@code-yeongyu/senpi-codemode 2026.8.29 → 2026.8.30-3

package/CHANGELOG.md

@@ -12,6 +12,68 @@
 
 ### Removed
 
+## [2026.8.30-3] - 2026-08-30
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+### Removed
+
+## [2026.8.30-2] - 2026-08-30
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+### Removed
+
+## [2026.8.30] - 2026-08-30
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+- The eval prompt's dependency-graph section is now `<workflow>` and states its contract directly:
+  define the workflow spec in code, one node per logically distinct step, rather than hand-authoring
+  the graph as a single opaque call.
+
+### Fixed
+
+- The JavaScript kernel persistence transform no longer truncates declarations whose multi-line
+  initializers contain interior `//` comments (previously emitted unparseable code such as
+  `globalThis["jobs"] = {;`, failing cells with `Unexpected token ';'. Expected a property name.`),
+  and no longer re-evaluates comment-bearing initializers when persisting bindings — such
+  declarations are kept verbatim and their bindings persisted by reference.
+- Last-expression capture no longer inserts `return` before continuation lines (`else`/`catch`/`finally`
+  clauses and leading-`.`/operator method-chain lines), and now scans template literals (including
+  nested templates in interpolations), regexes, and comments with the same literal-aware scanner as
+  the persistence transform — fixing `return else …`, `return .replace(…)`, and mid-argument
+  `return )` corruption of valid cells.
+- Last-expression capture now follows real ASI statement semantics: a parenthesized/bracketed/template
+  line after a closed block starts a new statement (echo restored), expressions split after a trailing
+  operator or `await` stay one statement, tagged templates split across lines invoke the tag, regexes
+  directly after a control-structure condition no longer desync the scanner, and labeled final
+  statements are left uncaptured instead of emitting invalid `return label: …`.
+- Destructuring patterns carrying interior line comments now persist their bindings, and declarations
+  with a dangling trailing comma are left untransformed so the original syntax error surfaces instead
+  of being silently "repaired".
+- Rewritten destructuring assignments are emitted with a leading defensive semicolon so they can no
+  longer ASI-merge into a preceding unterminated expression statement as a bogus call
+  (`foo()\n({…} = …)` previously became `foo()({…} = …)`).
+
+### Removed
+
 ## [2026.8.29] - 2026-08-29
 
 ### Breaking Changes

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/senpi-codemode",
-	"version": "2026.8.29",
+	"version": "2026.8.30-3",
 	"description": "Source-only senpi extension package for codemode evaluation tools",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -30,14 +30,14 @@
 	},
 	"dependencies": {
 		"@babel/parser": "8.0.4",
-		"@earendil-works/pi-ai": "npm:@code-yeongyu/senpi-ai@2026.8.29",
+		"@earendil-works/pi-ai": "npm:@code-yeongyu/senpi-ai@2026.8.30-3",
 		"typebox": "1.3.18"
 	},
 	"peerDependencies": {
-		"@code-yeongyu/senpi": "2026.8.29"
+		"@code-yeongyu/senpi": "2026.8.30-3"
 	},
 	"devDependencies": {
-		"@code-yeongyu/senpi": "2026.8.29"
+		"@code-yeongyu/senpi": "2026.8.30-3"
 	},
 	"keywords": [
 		"senpi",

package/src/kernels/js/worker-indirect-eval.js

@@ -258,20 +258,22 @@ function rewriteDeclaration(code, declarationStart, start, end, keyword) {
 	const preserveDeclaration = source.includes("//") || source.includes("/*");
 	for (const [segmentStart, segmentEnd] of splitDeclarators(code, start, end)) {
 		const segment = code.slice(segmentStart, segmentEnd);
-		if (!segment.trim()) continue;
+		if (!segment.trim()) return undefined;
 		const initializerStart = findTopLevelEquals(segment);
 		if (initializerStart < 0 && keyword === "const") return undefined;
 		const pattern = trimPattern(initializerStart < 0 ? segment : segment.slice(0, initializerStart));
 		const bindings = [];
 		collectPatternNames(pattern, bindings);
 		if (bindings.length === 0) return undefined;
+		if (preserveDeclaration) {
+			for (const name of bindings) assignments.push(`globalThis[${JSON.stringify(name)}] = ${name};`);
+			continue;
+		}
 		const target = rewriteBindingPattern(pattern);
 		if (target === undefined) return undefined;
 		const initializer = initializerStart < 0 ? "undefined" : segment.slice(initializerStart + 1).trim();
-		const [assignmentInitializer, comment] = splitTrailingLineComment(initializer);
-		const assignmentComment = preserveDeclaration ? "" : comment;
 		assignments.push(
-			`${target.startsWith("{") || target.startsWith("[") ? `(${target} = ${assignmentInitializer})` : `${target} = ${assignmentInitializer}`};${assignmentComment}`,
+			`${target.startsWith("{") || target.startsWith("[") ? `;(${target} = ${initializer})` : `${target} = ${initializer}`};`,
 		);
 	}
 	if (assignments.length === 0) return undefined;
@@ -408,6 +410,7 @@ function splitDeclarators(code, start, end) {
 		if (!/\s/u.test(char)) canStartRegex = isExpressionOperator(char) || char === "(" || char === "[" || char === "{";
 	}
 	if (segmentStart < end && code.slice(segmentStart, end).trim() !== ";") ranges.push([segmentStart, end - (code[end - 1] === ";" ? 1 : 0)]);
+	else if (ranges.length > 0 && code.slice(segmentStart, end).trim() !== ";") ranges.push([end, end]);
 	return ranges;
 }
 
@@ -501,51 +504,9 @@ function applyTextEdits(code, edits) {
 	return output;
 }
 
-function splitTrailingLineComment(source) {
-	let canStartRegex = true;
-	for (let index = 0; index < source.length; index += 1) {
-		const char = source[index];
-		const next = source[index + 1];
-		if (char === "/" && next === "/") return [source.slice(0, index).trimEnd(), source.slice(index)];
-		if (char === "/" && next === "*") {
-			index = skipBlockComment(source, index) - 1;
-			continue;
-		}
-		if (char === "'" || char === '"') {
-			index = skipQuotedLiteral(source, index) - 1;
-			canStartRegex = false;
-			continue;
-		}
-		if (char === "`") {
-			index = skipTemplateLiteral(source, index) - 1;
-			canStartRegex = false;
-			continue;
-		}
-		if (char === "/" && canStartRegex) {
-			index = skipRegexLiteral(source, index) - 1;
-			canStartRegex = false;
-			continue;
-		}
-		if (isIdentifierStart(char)) {
-			const end = readIdentifier(source, index);
-			canStartRegex = REGEX_PREFIX_KEYWORDS.has(source.slice(index, end));
-			index = end - 1;
-			continue;
-		}
-		if (isDecimalDigit(char)) {
-			index = skipNumberLiteral(source, index) - 1;
-			canStartRegex = false;
-			continue;
-		}
-		if (!/\s/u.test(char)) canStartRegex = isExpressionOperator(char) || char === "(" || char === "[" || char === "{";
-	}
-	return [source, ""];
-}
-
 function trimPattern(source) {
 	let start = 0;
-	let end = source.length;
-	while (start < end) {
+	while (start < source.length) {
 		if (/\s/u.test(source[start])) {
 			start += 1;
 			continue;
@@ -560,12 +521,30 @@ function trimPattern(source) {
 		}
 		break;
 	}
-	while (true) {
-		while (end > start && /\s/u.test(source[end - 1])) end -= 1;
-		if (end < start + 2 || source.slice(end - 2, end) !== "*/") break;
-		const commentStart = source.lastIndexOf("/*", end - 2);
-		if (commentStart < start) break;
-		end = commentStart;
+	let end = start;
+	for (let index = start; index < source.length; index += 1) {
+		const char = source[index];
+		const next = source[index + 1];
+		if (/\s/u.test(char)) continue;
+		if (char === "/" && next === "/") {
+			index = skipLineComment(source, index) - 1;
+			continue;
+		}
+		if (char === "/" && next === "*") {
+			index = skipBlockComment(source, index) - 1;
+			continue;
+		}
+		if (char === "'" || char === '"') {
+			index = skipQuotedLiteral(source, index) - 1;
+			end = index + 1;
+			continue;
+		}
+		if (char === "`") {
+			index = skipTemplateLiteral(source, index) - 1;
+			end = index + 1;
+			continue;
+		}
+		end = index + 1;
 	}
 	return source.slice(start, end);
 }
@@ -733,66 +712,136 @@ function captureLastExpression(code) {
 	return `${head}return ${tail.replace(/;+$/u, "")};`;
 }
 
+const LABEL_PREFIX_RE = /^[\p{ID_Start}$_][\p{ID_Continue}\u200c\u200d$]*\s*:/u;
+
 function isStatementOnly(source) {
-	return /^(?:const|let|var|if|for|while|switch|try|catch|finally|class|function|import|export|throw|return|do|break|continue|debugger)\b/u.test(
-		source,
-	);
+	if (
+		/^(?:const|let|var|if|for|while|switch|try|catch|finally|class|function|import|export|throw|return|do|break|continue|debugger)\b/u.test(
+			source,
+		)
+	)
+		return true;
+	return LABEL_PREFIX_RE.test(source);
 }
 
+const CONTROL_PAREN_KEYWORDS = new Set(["catch", "for", "if", "switch", "while", "with"]);
+
 function findLastTopLevelStatementStart(code) {
 	let start = 0;
 	let round = 0;
 	let square = 0;
 	let curly = 0;
-	let quote = "";
-	let escaped = false;
-	let lineComment = false;
-	let blockComment = false;
+	let canStartRegex = true;
+	let lastSignificant = "";
+	let pendingControlParen = false;
+	const controlParens = [];
 	for (let index = 0; index < code.length; index += 1) {
 		const char = code[index];
 		const next = code[index + 1];
-		if (lineComment) {
-			if (char === "\n") lineComment = false;
+		if (char === "/" && next === "/") {
+			index = skipLineComment(code, index) - 1;
 			continue;
 		}
-		if (blockComment) {
-			if (char === "*" && next === "/") {
-				blockComment = false;
-				index += 1;
-			}
+		if (char === "/" && next === "*") {
+			index = skipBlockComment(code, index) - 1;
 			continue;
 		}
-		if (quote) {
-			if (escaped) {
-				escaped = false;
-			} else if (char === "\\") {
-				escaped = true;
-			} else if (char === quote) {
-				quote = "";
-			}
+		if (char === "'" || char === '"' || char === "`") {
+			index = (char === "`" ? skipTemplateLiteral(code, index) : skipQuotedLiteral(code, index)) - 1;
+			canStartRegex = false;
+			lastSignificant = char;
+			pendingControlParen = false;
 			continue;
 		}
-		if (char === "/" && next === "/") {
-			lineComment = true;
-			index += 1;
+		if (char === "/" && canStartRegex) {
+			index = skipRegexLiteral(code, index) - 1;
+			canStartRegex = false;
+			lastSignificant = char;
+			pendingControlParen = false;
 			continue;
 		}
-		if (char === "/" && next === "*") {
-			blockComment = true;
-			index += 1;
+		if (isIdentifierStart(char)) {
+			const end = readIdentifier(code, index);
+			const token = code.slice(index, end);
+			canStartRegex = REGEX_PREFIX_KEYWORDS.has(token);
+			pendingControlParen = CONTROL_PAREN_KEYWORDS.has(token);
+			lastSignificant = code[end - 1];
+			index = end - 1;
 			continue;
 		}
-		if (char === "'" || char === '"' || char === "`") {
-			quote = char;
+		if (isDecimalDigit(char)) {
+			index = skipNumberLiteral(code, index) - 1;
+			canStartRegex = false;
+			lastSignificant = char;
+			pendingControlParen = false;
 			continue;
 		}
-		if (char === "(") round += 1;
-		else if (char === ")") round -= 1;
-		else if (char === "[") square += 1;
-		else if (char === "]") square -= 1;
-		else if (char === "{") curly += 1;
-		else if (char === "}") curly -= 1;
-		else if ((char === ";" || char === "\n") && round === 0 && square === 0 && curly === 0) start = index + 1;
+		if (char === "(") {
+			round += 1;
+			controlParens.push(pendingControlParen);
+			pendingControlParen = false;
+			canStartRegex = true;
+			lastSignificant = char;
+			continue;
+		}
+		if (char === ")") {
+			round = Math.max(0, round - 1);
+			canStartRegex = controlParens.pop() === true;
+			lastSignificant = char;
+			continue;
+		}
+		if (char === "[" || char === "{") {
+			if (char === "[") square += 1;
+			else curly += 1;
+			canStartRegex = true;
+			lastSignificant = char;
+			pendingControlParen = false;
+			continue;
+		}
+		if (char === "]" || char === "}") {
+			if (char === "]") square = Math.max(0, square - 1);
+			else curly = Math.max(0, curly - 1);
+			canStartRegex = false;
+			lastSignificant = char;
+			pendingControlParen = false;
+			continue;
+		}
+		if (char === ";" && round === 0 && square === 0 && curly === 0) {
+			const nextIndex = nextSignificantIndex(code, index + 1);
+			if (nextIndex < code.length && !isContinuationKeyword(code, nextIndex)) start = nextIndex;
+			canStartRegex = true;
+			lastSignificant = char;
+			pendingControlParen = false;
+			continue;
+		}
+		if (isLineTerminator(char) && round === 0 && square === 0 && curly === 0) {
+			if (!canStartRegex) {
+				const nextIndex = nextSignificantIndex(code, index + 1);
+				if (nextIndex < code.length && !isStatementContinuation(code, nextIndex, lastSignificant)) start = nextIndex;
+			}
+			canStartRegex = true;
+			continue;
+		}
+		if (!/\s/u.test(char)) {
+			canStartRegex = isExpressionOperator(char) || char === ",";
+			lastSignificant = char;
+			pendingControlParen = false;
+		}
 	}
 	return start;
 }
+
+const STATEMENT_CONTINUATION_KEYWORDS = new Set(["catch", "else", "finally"]);
+
+function isContinuationKeyword(code, index) {
+	if (!isIdentifierStart(code[index])) return false;
+	return STATEMENT_CONTINUATION_KEYWORDS.has(code.slice(index, readIdentifier(code, index)));
+}
+
+function isStatementContinuation(code, index, previousChar) {
+	const char = code[index];
+	if (char === "(" || char === "[" || char === "`") return previousChar !== "}" && previousChar !== "";
+	if (char === "!" || char === "~") return false;
+	if (char !== undefined && isDeclarationContinuation(char)) return true;
+	return isContinuationKeyword(code, index);
+}

package/src/prompt/eval-prompt.ts

@@ -163,7 +163,7 @@ tool_schema(name?) → dict
 completion(prompt, model?="default", system?=None, schema?=None) → str | dict
     Oneshot, stateless (no history/tools). \`model\`: \`"smol"\` fast | \`"default"\` session | \`"slow"\` most capable. \`schema\` (JSON-Schema) → structured output, parsed object.
 {{#if spawns}}agent(prompt, agent?="{{spawnDefaultAgent}}", model?=None, label?=None, schema?=None, handle?=False) → str | dict
-    Run a subagent → final output. \`agent\` picks another discovered agent; omit it to use \`{{spawnDefaultAgent}}\`. \`schema\` as in completion(). Background via \`local://\` files named in the prompt. \`handle\` → DAG node dict { text, output, handle: \`agent://<id>\`, id, agent } (parsed under \`data\` when \`schema\` set).
+    Run a subagent → final output. \`agent\` picks another discovered agent; omit it to use \`{{spawnDefaultAgent}}\`. \`schema\` as in completion(). Background via \`local://\` files named in the prompt. \`handle\` → workflow node dict { text, output, handle: \`agent://<id>\`, id, agent } (parsed under \`data\` when \`schema\` set).
 {{#if js}}    JS: options are ONE trailing object — agent(prompt, { agent, schema, handle }).
 {{/if}}{{/if}}parallel(thunks) → list
     Thunks through a bounded pool (wide as a \`task\` batch — don't pre-shrink), input order kept; returns when all finish, a throwing thunk propagates.
@@ -176,14 +176,14 @@ phase(title) → None
 \`\`\`
 </prelude>
 {{#if spawns}}
-<dag>
-Pipe handles through stage helpers to build a dependency graph — acyclic waves:
+<workflow>
+Define the workflow spec IN CODE: partition the work into logically distinct steps, one node per step, then wire them as acyclic waves — never hand-author the graph as a single opaque call.
 - **Name nodes.** Capture each \`agent(…, {{#if py}}handle=True{{/if}}{{#if js}}{ handle: true }{{/if}}{{#if jl}}handle=true{{/if}})\` result; carries \`handle\` (\`agent://<id>\`) + \`output\`.
 - **Wire edges by reference.** Put an upstream node's \`handle\`/\`output\` in the dependent stage's prompt — large transcript never re-inlined. Bulk: \`write("local://<name>.md", …)\`, pass the URI.
 - **\`pipeline(items, *stages)\` = staged waves**, barrier between stages (every item clears stage N before any enters N+1). **\`parallel(thunks)\` = one wave** of independent nodes.
 - **Isolate failure.** A raising node re-raises the lowest-index error, aborts its wave; wrap risky nodes in try/except so a failure degrades only its dependent subtree, independent branches finish.
 - **Acyclic only.** A node never waits on its own descendant.
-</dag>
+</workflow>
 {{/if}}
 
 <critical>