@code-yeongyu/senpi-codemode 2026.10.12 → 2026.10.13

package/CHANGELOG.md

@@ -12,6 +12,20 @@
 
 ### Removed
 
+## [2026.10.13] - 2026-10-13
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+- Large JavaScript `eval` data cells no longer stall the shared RPC host on unnecessary import parsing. Cells without possible import syntax skip Babel, and the whole-AST walk runs only when a dynamic import is possible. The dynamic-import check stays linear on comment-heavy data; imports with comments or non-ASCII bindings still rewrite correctly ([#3206](https://github.com/code-yeongyu/senpi/pull/3206)).
+
+### Removed
+
 ## [2026.10.12] - 2026-10-12
 
 ### Breaking Changes

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/senpi-codemode",
-	"version": "2026.10.12",
+	"version": "2026.10.13",
 	"description": "Source-only senpi extension package for codemode evaluation tools",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -33,15 +33,15 @@
 	},
 	"dependencies": {
 		"@babel/parser": "8.0.4",
-		"@earendil-works/pi-ai": "npm:@code-yeongyu/senpi-ai@2026.10.12",
+		"@earendil-works/pi-ai": "npm:@code-yeongyu/senpi-ai@2026.10.13",
 		"quickjs-wasi": "3.6.2",
 		"typebox": "1.3.34"
 	},
 	"peerDependencies": {
-		"@code-yeongyu/senpi": "2026.10.12"
+		"@code-yeongyu/senpi": "2026.10.13"
 	},
 	"devDependencies": {
-		"@code-yeongyu/senpi": "2026.10.12"
+		"@code-yeongyu/senpi": "2026.10.13"
 	},
 	"keywords": [
 		"senpi",

package/src/kernels/js/rewrite-imports.ts

@@ -151,8 +151,16 @@ function applyEdits(code: string, edits: readonly TextEdit[]): string {
 	return output;
 }
 
+// Cells are prepared on the shared host loop (#3205), so Babel runs only for text that can be import syntax: the
+// keyword as a whole word, not `import-1` or `import.meta`. Anything after it (comments, non-ASCII names) still
+// parses; a false positive only costs a parse, a false negative would skip a rewrite. The AST walk runs when
+// `import` is followed by `(` or by a comment that could hide one. The gate never scans a comment body: a
+// comment-matching pattern backtracks quadratically on large data full of `import /*` text.
+const IMPORT_SYNTAX = /\bimport\b(?![-.$])/u;
+const DYNAMIC_IMPORT_SYNTAX = /\bimport\s*[(/]/u;
+
 export function rewriteImports(code: string): string {
-	if (!code.includes("import")) return code;
+	if (!IMPORT_SYNTAX.test(code)) return code;
 	const ast = parseProgram(code);
 	if (!ast) return code;
 	const edits: TextEdit[] = [];
@@ -161,9 +169,11 @@ export function rewriteImports(code: string): string {
 		if (node.type !== "ImportDeclaration" || typeof node.start !== "number" || typeof node.end !== "number") continue;
 		edits.push({ start: node.start, end: node.end, text: rewriteImportDeclaration(node) });
 	}
-	walkNodes(ast.program, (node) => {
-		const edit = dynamicImportEdit(node);
-		if (edit) edits.push(edit);
-	});
+	if (DYNAMIC_IMPORT_SYNTAX.test(code)) {
+		walkNodes(ast.program, (node) => {
+			const edit = dynamicImportEdit(node);
+			if (edit) edits.push(edit);
+		});
+	}
 	return applyEdits(code, edits);
 }