@rsvelte/vite-plugin-svelte-native 0.3.8 → 0.3.10

package/index.cjs

@@ -254,4 +254,4 @@ module.exports.decodeParseEnvelope = decodeParseEnvelope;
 // with `submodules/svelte/packages/svelte/package.json` by hand; run
 // `node scripts/dev/check-vps-version.mjs` (also wired into CI) to
 // catch drift.
-module.exports.VERSION = '5.56.9';
+module.exports.VERSION = '5.56.10';

package/index.d.ts

@@ -193,6 +193,13 @@ export function compileModule(
  * {@link decodeEnvelope} to obtain
  * the legacy {@link CompileResult} shape; or pass the buffer through
  * worker `postMessage` (transferable) to avoid a copy.
+ *
+ * Unlike {@link compile}, this does NOT go through the wrapper that resolves
+ * upstream's function-valued options, so a `runes` / `css` / `customElement` /
+ * `warningFilter` callback is **rejected** here rather than honoured — evaluate
+ * it yourself and pass the plain value. `runes` and `warningFilter` used to be
+ * dropped in silence instead, which produced a successful compile with the
+ * wrong result; `css` and `customElement` already threw.
  */
 export function compileEnvelope(source: string, options?: CompileOptions): Buffer;
 export function compileModuleEnvelope(
@@ -328,6 +335,13 @@ export function compileModuleBuffers(
 /**
  * The legacy JSON-on-the-boundary path. Kept exported for parity tests
  * and as an escape hatch — production callers should use {@link compile}.
+ *
+ * Unlike {@link compile}, this does NOT go through the wrapper that resolves
+ * upstream's function-valued options, so a `runes` / `css` / `customElement` /
+ * `warningFilter` callback is **rejected** here rather than honoured — evaluate
+ * it yourself and pass the plain value. `runes` and `warningFilter` used to be
+ * dropped in silence instead, which produced a successful compile with the
+ * wrong result; `css` and `customElement` already threw.
  */
 export function compileLegacy(source: string, options?: CompileOptions): CompileResult;
 export function compileModuleLegacy(
@@ -359,12 +373,29 @@ export interface ParseOptions {
 	 * JSON-parse step.
 	 */
 	skipCssAst?: boolean;
+	/**
+	 * `modern` from `svelte/compiler`'s `parse()`. **Defaults to `false`**,
+	 * which returns the LEGACY AST (`{ html, instance, module, css }`); pass
+	 * `true` for the modern `Root` shape. The modern AST stays opt-in until
+	 * Svelte 6, and this binding defaults the same way official does.
+	 */
+	modern?: boolean;
+	/**
+	 * `loose` from `svelte/compiler`'s `parse()`. Recover from a parse error
+	 * and return an AST anyway — what an editor integration needs to parse a
+	 * document mid-keystroke. Not blanket recovery: a source official still
+	 * rejects under `loose` (e.g. a stray `</div>`) throws here too.
+	 */
+	loose?: boolean;
 }
 
 /**
  * Parse a Svelte component and return the AST as a JSON string. The
  * caller is responsible for `JSON.parse` on the returned value.
  *
+ * Without `{ modern: true }` this returns the LEGACY AST, matching
+ * `svelte/compiler`'s default.
+ *
  * For the fastest path skip JSON entirely: use {@link parseEnvelope}
  * with `decodeParseEnvelope` from
  * `@rsvelte/vite-plugin-svelte-native/parse-envelope.js`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rsvelte/vite-plugin-svelte-native",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "description": "NAPI bindings to the rsvelte compiler — used by the @rsvelte vite-plugin-svelte shim",
   "license": "MIT",
   "repository": {
@@ -30,11 +30,11 @@
     "index.d.ts"
   ],
   "optionalDependencies": {
-    "@rsvelte/vite-plugin-svelte-native-darwin-x64": "^0.3.8",
-    "@rsvelte/vite-plugin-svelte-native-darwin-arm64": "^0.3.8",
-    "@rsvelte/vite-plugin-svelte-native-linux-x64-gnu": "^0.3.8",
-    "@rsvelte/vite-plugin-svelte-native-linux-arm64-gnu": "^0.3.8",
-    "@rsvelte/vite-plugin-svelte-native-win32-x64-msvc": "^0.3.8"
+    "@rsvelte/vite-plugin-svelte-native-darwin-arm64": "^0.3.10",
+    "@rsvelte/vite-plugin-svelte-native-darwin-x64": "^0.3.10",
+    "@rsvelte/vite-plugin-svelte-native-linux-arm64-gnu": "^0.3.10",
+    "@rsvelte/vite-plugin-svelte-native-linux-x64-gnu": "^0.3.10",
+    "@rsvelte/vite-plugin-svelte-native-win32-x64-msvc": "^0.3.10"
   },
   "publishConfig": {
     "access": "public"

package/parse-envelope.js

@@ -19,7 +19,7 @@ const MAGIC = 0x3156_5052; // "RPV1" little-endian
 // reorder, `typeParameters` on function-like nodes, Identifier `optional`);
 // v4 adds the object-method `typeParameters`-after-`body` flag byte.
 // Keep in lockstep with `napi_raw_parse.rs`'s `VERSION`.
-const VERSION = 4;
+const VERSION = 8;
 const HEADER_LEN = 24;
 
 // Tags — must mirror napi_raw_parse.rs.
@@ -80,7 +80,7 @@ const TAG_AWAIT_BLOCK = 0x72;
 const TAG_KEY_BLOCK = 0x73;
 const TAG_SNIPPET_BLOCK = 0x74;
 
-// JsNode (estree) tags — 0x80..0xCC. Mirrors `napi_raw_parse.rs`.
+// JsNode (estree) tags — 0x80..0xD1. Mirrors `napi_raw_parse.rs`.
 const JS_IDENTIFIER = 0x80;
 const JS_PRIVATE_IDENTIFIER = 0x81;
 const JS_LITERAL = 0x82;
@@ -140,6 +140,8 @@ const JS_CONTINUE_STATEMENT = 0xb7;
 const JS_EMPTY_STATEMENT = 0xb8;
 const JS_DEBUGGER_STATEMENT = 0xb9;
 const JS_IMPORT_DECLARATION = 0xba;
+const JS_IMPORT_ATTRIBUTE = 0xd2;
+const JS_EXPORT_ALL_DECLARATION = 0xd3;
 const JS_IMPORT_SPECIFIER = 0xbb;
 const JS_IMPORT_DEFAULT_SPECIFIER = 0xbc;
 const JS_IMPORT_NAMESPACE_SPECIFIER = 0xbd;
@@ -152,7 +154,6 @@ const JS_PROPERTY_DEFINITION = 0xc3;
 const JS_STATIC_BLOCK = 0xc4;
 const JS_DECORATOR = 0xc5;
 const JS_TS_TYPE_ANNOTATION = 0xc6;
-const JS_TS_ENUM_DECLARATION = 0xc7;
 const JS_TS_MODULE_DECLARATION = 0xc8;
 const JS_COMMENT = 0xc9;
 const JS_NULL = 0xca;
@@ -561,6 +562,8 @@ function readNodeBody(ctx, tag, start, end) {
 			return readJsBareExpr(ctx, 'DebuggerStatement', start, end);
 		case JS_IMPORT_DECLARATION:
 			return readJsImportDeclaration(ctx, start, end);
+		case JS_IMPORT_ATTRIBUTE:
+			return readJsImportAttribute(ctx, start, end);
 		case JS_IMPORT_SPECIFIER:
 			return readJsImportSpecifier(ctx, start, end);
 		case JS_IMPORT_DEFAULT_SPECIFIER:
@@ -571,6 +574,8 @@ function readNodeBody(ctx, tag, start, end) {
 			return readJsExportNamedDeclaration(ctx, start, end);
 		case JS_EXPORT_DEFAULT_DECLARATION:
 			return readJsExportDefaultDeclaration(ctx, start, end);
+		case JS_EXPORT_ALL_DECLARATION:
+			return readJsExportAllDeclaration(ctx, start, end);
 		case JS_EXPORT_SPECIFIER:
 			return readJsExportSpecifier(ctx, start, end);
 		case JS_CLASS_BODY:
@@ -585,8 +590,6 @@ function readNodeBody(ctx, tag, start, end) {
 			return readJsBareExpr(ctx, 'Decorator', start, end);
 		case JS_TS_TYPE_ANNOTATION:
 			return readJsTSTypeAnnotation(ctx, start, end);
-		case JS_TS_ENUM_DECLARATION:
-			return readJsBareExpr(ctx, 'TSEnumDeclaration', start, end);
 		case JS_TS_PARAMETER_PROPERTY:
 			return readJsBareExpr(ctx, 'TSParameterProperty', start, end);
 		case JS_TS_MODULE_DECLARATION:
@@ -978,11 +981,18 @@ function readJsBareExpr(ctx, typeName, start, end) {
 function readJsImportExpression(ctx, start, end) {
 	const loc = readTypedLoc(ctx);
 	const source = readNode(ctx);
+	const options = readChildArray(ctx);
+	const ts = readBool(ctx);
 	const node = { type: 'ImportExpression', start, end };
 	if (loc !== null) node.loc = loc;
 	node.source = source;
-	// The upstream serializer also adds `options: null`. Match that.
-	node.options = null;
+	// acorn-typescript spells the second argument as an `arguments` list and
+	// omits the key when there is none; acorn always writes `options`.
+	if (ts) {
+		if (options.length > 0) node.arguments = options;
+	} else {
+		node.options = options.length > 0 ? options[0] : null;
+	}
 	return node;
 }
 
@@ -1416,7 +1426,19 @@ function readJsImportDeclaration(ctx, start, end) {
 	node.specifiers = specifiers;
 	node.source = source;
 	if (importKind !== null) node.importKind = importKind;
-	node.attributes = attributes;
+	// See `readJsExportNamedDeclaration`.
+	if (importKind === null || attributes.length > 0) node.attributes = attributes;
+	return node;
+}
+
+function readJsImportAttribute(ctx, start, end) {
+	const loc = readTypedLoc(ctx);
+	const key = readNode(ctx);
+	const value = readNode(ctx);
+	const node = { type: 'ImportAttribute', start, end };
+	if (loc !== null) node.loc = loc;
+	node.key = key;
+	node.value = value;
 	return node;
 }
 
@@ -1464,15 +1486,35 @@ function readJsExportNamedDeclaration(ctx, start, end) {
 	node.specifiers = specifiers;
 	node.source = source;
 	if (exportKind !== null) node.exportKind = exportKind;
-	node.attributes = attributes;
+	// acorn always writes `attributes`; acorn-typescript writes it only where the
+	// source had a `with` clause, and `exportKind` is set for TypeScript alone.
+	if (exportKind === null || attributes.length > 0) node.attributes = attributes;
+	return node;
+}
+
+function readJsExportAllDeclaration(ctx, start, end) {
+	const loc = readTypedLoc(ctx);
+	const exported = readOptNode(ctx);
+	const source = readNode(ctx);
+	const exportKind = readOptStr(ctx);
+	const attributes = readChildArray(ctx);
+	const node = { type: 'ExportAllDeclaration', start, end };
+	if (loc !== null) node.loc = loc;
+	if (exportKind !== null) node.exportKind = exportKind;
+	node.exported = exported;
+	node.source = source;
+	// See `readJsExportNamedDeclaration`.
+	if (exportKind === null || attributes.length > 0) node.attributes = attributes;
 	return node;
 }
 
 function readJsExportDefaultDeclaration(ctx, start, end) {
 	const loc = readTypedLoc(ctx);
 	const declaration = readNode(ctx);
+	const exportKind = readOptStr(ctx);
 	const node = { type: 'ExportDefaultDeclaration', start, end };
 	if (loc !== null) node.loc = loc;
+	if (exportKind !== null) node.exportKind = exportKind;
 	node.declaration = declaration;
 	return node;
 }
@@ -1652,7 +1694,14 @@ function readJsComment(ctx, start, end) {
 	const kindByte = readU8(ctx);
 	const kind = kindByte === 1 ? 'Block' : 'Line';
 	const value = readStr(ctx);
+	// Only an in-tag comment's `loc` carries `character`; a script comment's
+	// comes from the JS parser, which does not produce one.
+	const hasCharacter = readU8(ctx) === 1;
 	const loc = readSourceLocation(ctx);
+	if (!hasCharacter) {
+		delete loc.start.character;
+		delete loc.end.character;
+	}
 	return { type: kind, start, end, value, loc };
 }