Add option to remove doublequotes in module names in TOC

[nicknisi]

add externalModulesOnly option (boolean) which, if true, will not wrap module names in doublequotes. I was researching why these names are wrapped in quotes and stumbled across #68 discussing this issue, which suggested a possible CLI flag to disable this functionality if desired.

Here's a screenshot of the change in a project where I am not using namespaces (internal modules).

Also, I'm open to alternatives for the name of the option. Some others I've thought of include

• assumeNoNamespace
• removeQuotes
• externalModules

Any help on the naming would be greatly appreciated.

[aciccarello]

Yes! I think it would be great to get rid of those quotes. Honestly I'd like them to be off by default.

[nicknisi]

I would too 😄, honestly. It looks like they are there to distinguish namespaces (formerly internal modules) with external modules (using the ESM export keyword. I personally don't use namespace all that often, so I'm not sure how big of an issue that would be to turn them off by default.

[blakeembrey]

Is there any way we can check if a "module" is external (e.g. in that case, node.js fs) or internal?

[jaredcnance]

has this PR been abandoned? what still needs to be done to get this cleaned up?

[nicknisi]

Oh wow, sorry I never followed up. I'll re-orient myself and follow up properly.