Error: outfile must end with .html

[codedust]

First of all: Thanks for your great tool!

The README says:

It is also possible to write mappings into a separate file as follow:

npx @jsenv/importmap-node-module demo.importmap --entrypoint index.html

However, this produces the following error:

Error: outfile must end with .html

The following command:

npx @jsenv/importmap-node-module importmap.html --entrypoint index.html

also produces an error:

Import resolution failed for "./importmap.html"

How do I write mappings into a separate file?

[dmail]

Hello thank you for your interest and opening an issue 👍

Error: outfile must end with .html

This is enforced by the cli at

importmap-node-module/src/cli.mjs

Lines 39 to 42 in 7e7a11a

	if (!outfile.endsWith(".html")) {
	console.error("Error: outfile must end with .html");
	process.exit(1);
	}

But it would be supported by the API as shown in

importmap-node-module/src/step_write_into_files/write_into_files.js

Lines 23 to 28 in 7e7a11a

	if (fileUrl.endsWith(".html")) {
	writeIntoHtmlFile(fileUrl, importmapAsJson, { logger });
	} else {
	writeFileSync(fileUrl, importmapAsJson);
	logger.info(`-> ${urlToFileSystemPath(fileUrl)}`);
	}

So I'll likely update the cli to accept .importmap and .json extensions.

Import resolution failed for "./importmap.html"

The error message is not very clear but this is because the HTML file where you want to write the importmap must exists. Importmap are injected into an existing HTML file. You could update your command to

npx @jsenv/importmap-node-module index.html --entrypoint index.html

Which could be shortened to

npx @jsenv/importmap-node-module index.html

[dmail]

I've pushed 24efe89

Once the GitHub workflow is over, if tests are still passing the version "7.1.0" will be published to NPM

[codedust]

Thanks for the fast reply! Unfortunately, 24efe89 introduced a bug that requires the file to end with .html AND .importmap AND JSON. I fixed that in #77