Import assertions

[dmail]

Fixes #211

TODO list:

• Do not write css file when it's only referenced by js import assertion
• Ensure soucemap url for css inlined in js targets the css sourcemap file
• Test inline js importing css with import assertion
• Test inline js referencing css with import.meta.url
• Write babel plugin replacing import assertion with the query string param
• Update jsenv browser system to wrap server response when ?import_type is in the module url
• Test js importing css with launchChromium and launchFirefox
• Test with stylesheet polyfill https://github.com/calebdwilliams/construct-style-sheets
• Add check for css import assertion at the bottom of getFeatureReport
• Add check for json import assertion at the bottom of getFeatureReport
• Write a test using dynamic import (https://github.com/tc39/proposal-import-assertions#dynamic-import)
  • If that's not supported for some reason (rollup or something don't like that) then we should throw an error explaining
• Write a test where css is imported both by js and html
• Update @jsenv/template-pwa We'll see later

Changes

• Rename launchNode into nodeRuntime (same for browsers)
• launch becomes runtime, launchParams -> runtimeParams
• Support css and json import assertions
• Compile server now detect runtimeSupport from testPlan
• Restructure many test files
• Improve response validation code and error messages
• Update some deps
• Ensure inline scripts gets a sourcemap comment as well
• Ensure sourcemap file are properly versioned too

[github-actions]

Performance impact

Impact on 15 metrics when merging import_type_css into master. Before drawing conclusion, keep in mind performance variability.

package metrics

Metric	Before merge	After merge	Impact
npm tarball size	525.79 KB	544.38 KB	+18.59 KB / +3.53%	↗️
npm tarball unpacked size	2.09 MB	2.16 MB	+79.28 KB / +3.71%	↗️
npm tarball file count	379	387	+8 / +2.11%	↗️

dev server metrics

Metric	Before merge	After merge	Impact
start duration	0.594 seconds	0.6 seconds	+0.006 seconds / +1.01%	↗️
dev server memory heap used	8.74 MB	8.83 MB	+91.27 KB / +1.02%	↗️
number of fs read operation	0	0		👻
number of fs write operation	24	24		👻

build metrics

Metric	Before merge	After merge	Impact
build duration	1.53 seconds	1.55 seconds	+0.013 seconds / +0.88%	↗️
build memory heap used	42.96 MB	43.56 MB	+620.26 KB / +1.41%	↗️
number of fs read operation	0	0		👻
number of fs write operation	152	152		👻

test metrics

Metric	Before merge	After merge	Impact
test plan duration	16.32 seconds	17.46 seconds	+1.13 seconds / +6.97%	↗️
test plan memory heap used	44.88 MB	47.59 MB	+2.71 MB / +6.04%	↗️
number of fs read operation	0	0		👻
number of fs write operation	8352	8432	+80 / +0.96%	↗️

_{Generated by @jsenv/performance-impact during performance_impact#1342708727 on 69b98b6}

[codecov]

Codecov Report

Merging #212 (138d748) into master (d1c5738) will increase coverage by 0.36%.
The diff coverage is 86.94%.

❗ Current head 138d748 differs from pull request most recent head 69b98b6. Consider uploading reports for the commit 69b98b6 to get more accurate results