[hyfdev]

Markdown island pages now hydrate correctly when the island import uses a trailing comma (or a $/_-prefixed component name) — instead of silently rendering static and leaking an orphan import into the client bundle.

The bug

The markdown compiler's island-import regexes in packages/md/src/extract-script.ts didn't tolerate a trailing comma in the import attributes and only matched \w+ component names:

<script>
import Counter from "./Counter.vue" with { island: "load", }
</script>

When the regex failed to match, extractScript returned no islandImports, so plugin.ts took the non-island branch (emit*Component) — the component rendered as static markup with no client hydration — and the un-stripped import line leaked into the client bundle as an orphan import. This affects all four adapters, since packages/md is the shared markdown parser for .md pages.

Expected-correct behavior

A trailing comma is valid syntax (and routinely auto-inserted by formatters like Prettier), and $/_-prefixed names are valid identifiers (the islands docs themselves use a _Counter example). Both should be parsed as islands.

Root cause

The identical island-import fix already landed for all four adapter plugins and void/pages/islands-plugin.ts in commit 3307e6fc ("fix: Island import attributes with trailing commas"), but the parallel markdown parser was missed. This change mirrors that fix on the one file it didn't reach.

The fix

In both ISLAND_IMPORT_RE (extraction) and ISLAND_IMPORT_LINE_RE (line-strip): allow an optional trailing comma (,?\s* before }) and widen the component name to [$_A-Za-z][$_\w]*, byte-identical to the merged adapter regex.

Tests

Regression tests added in extract-script.test.ts: a trailing-comma island import and a $-prefixed name — both fail on origin/main and pass after; existing cases unchanged (and with { island: "x", } only tolerates a single trailing comma, not a second attribute). Local gate: extract-script.test.ts 13/13; full --project unit zero-collateral vs a clean origin/main baseline; lint + fmt clean. (packages/md typecheck hits a pre-existing void/pages build-artifact resolution error unrelated to this regex change; CI's build:all resolves it.)

Verified extract-script.ts is the only remaining island-import regex without the trailing-comma fix.

Prepared with AI assistance.

[netlify]

❌ Deploy Preview for staging-void-cloud failed.

Name	Link
🔨 Latest commit	86aef7a
🔍 Latest deploy log	https://app.netlify.com/projects/staging-void-cloud/deploys/6a33ef0c02575400085cbea8