@code-yeongyu/senpi 2026.8.23 → 2026.8.25

package/node_modules/@code-yeongyu/senpi-codemode/src/extension/session-manager-proxy.ts

@@ -21,12 +21,30 @@ export class CodemodeSessionNotStartedError extends Error {
 	}
 }
 
+const defaultTeardownFailureReporter = (error: unknown): void => {
+	globalThis.process.stderr.write(`[senpi-codemode] ${describeTeardownFailure(error)}\n`);
+};
+
+function describeTeardownFailure(error: unknown): string {
+	const message = error instanceof Error ? error.message : String(error);
+	if (error instanceof AggregateError && error.errors.length > 0) {
+		const causes = error.errors.map((cause) => (cause instanceof Error ? cause.message : String(cause))).join("; ");
+		return `session teardown failed: ${message} (${causes})`;
+	}
+	return `session teardown failed: ${message}`;
+}
+
 export class SessionManagerProxy implements CodemodeSessionManager, EvalExecutionTracker {
 	#current: CodemodeSessionManager | undefined;
 	#generation = 0;
 	#started = false;
 	#acceptingExecutions = false;
 	readonly #executions = new Set<TrackedExecution>();
+	readonly #onTeardownFailure: (error: unknown) => void;
+
+	constructor(onTeardownFailure: (error: unknown) => void = defaultTeardownFailureReporter) {
+		this.#onTeardownFailure = onTeardownFailure;
+	}
 
 	beginReplacement(): number {
 		this.#generation++;
@@ -37,19 +55,19 @@ export class SessionManagerProxy implements CodemodeSessionManager, EvalExecutio
 
 	async replace(generation: number, next: CodemodeSessionManager): Promise<boolean> {
 		if (generation !== this.#generation) {
-			await next.dispose();
+			await this.#disposeQuietly(next);
 			return false;
 		}
 		await this.#settleExecutions();
 		if (generation !== this.#generation) {
-			await next.dispose();
+			await this.#disposeQuietly(next);
 			return false;
 		}
 		const current = this.#current;
 		this.#current = undefined;
-		await current?.dispose();
+		await this.#disposeQuietly(current);
 		if (generation !== this.#generation) {
-			await next.dispose();
+			await this.#disposeQuietly(next);
 			return false;
 		}
 		this.#current = next;
@@ -100,7 +118,23 @@ export class SessionManagerProxy implements CodemodeSessionManager, EvalExecutio
 		await this.#settleExecutions();
 		const current = this.#current;
 		this.#current = undefined;
-		await current?.dispose();
+		await this.#disposeQuietly(current);
+	}
+
+	/**
+	 * Session teardown is best-effort: a kernel or bridge that fails to confirm
+	 * close (e.g. a SIGKILLed interpreter missing its reap window throws
+	 * KernelRetirementError into the manager's dispose AggregateError) must not
+	 * reject the session lifecycle handler that triggered the teardown — the
+	 * extension host surfaces such rejections as user-facing extension errors.
+	 */
+	async #disposeQuietly(manager: CodemodeSessionManager | undefined): Promise<void> {
+		if (manager === undefined) return;
+		try {
+			await manager.dispose();
+		} catch (error) {
+			this.#onTeardownFailure(error);
+		}
 	}
 
 	#abortExecutions(): void {

package/node_modules/@code-yeongyu/senpi-codemode/src/kernels/AGENTS.md

@@ -0,0 +1,40 @@
+# src/kernels
+
+Persistent kernels for four runtimes plus shared subprocess lifecycle. Earned
+by score 13 — distinct multi-runtime domain (31 files, TS hosts plus embedded
+runner/prelude assets).
+
+## WHERE TO LOOK
+
+| Task | Path |
+| --- | --- |
+| JavaScript host API | `js/context-manager.ts` (`JavaScriptKernel`), `js/worker-host.ts` |
+| JS worker runtime, entries | `js/worker-runtime.js`, `js/worker-entry.js`, `js/inline-worker-entry.js`, `js/inline-worker.ts`, `js/worker-core.js` (+ `worker-core.d.ts`) |
+| JS import rewriting, queueing | `js/rewrite-imports.ts`, `js/run-queue.ts`, `js/prelude.ts`, `js/local-module-loader.ts` |
+| Python kernel | `py/kernel.ts`, `py/transport.ts`, `py/process.ts`, `py/prelude.py` |
+| Ruby kernel | `rb/kernel.ts` + `rb/prelude.rb`, `rb/runner.rb` |
+| Julia kernel | `jl/kernel.ts` + `jl/prelude.jl`, `jl/runner.jl` |
+| Shared subprocess layer | `shared/subprocess-kernel.ts`, `subprocess-{contract,process,queue,run}.ts`, `runtime-asset.ts` |
+
+## CONVENTIONS
+
+- Each language dir pairs a typed TS host/controller with an embedded runner or
+  prelude asset; `shared/runtime-asset.ts` ships them.
+- Transport messages are discriminated by string `type` (`ready`, `result`,
+  `tool-call`, `closed`, `init-failed`, ...) exchanged as framed bridge
+  messages, one JSON line per frame.
+- JS persistent cell bindings are rewritten onto `globalThis`; imports are
+  AST-parsed (Babel) and rewritten to bridge-compatible dynamic imports.
+- JS runs on worker threads with an inline-worker fallback; py/rb/jl run as
+  framed subprocesses through `shared/`.
+- Subprocess retirement/restart, worker recovery, timeout, and interrupt
+  semantics live here, never in the tool layer.
+
+## ANTI-PATTERNS
+
+- Never treat arbitrary objects as bridge messages without discriminant
+  validation.
+- Never rewrite imports by filename/regex — `rewrite-imports.ts` applies
+  source-position edits from the parsed program.
+- An optional interpreter being absent (py/rb/jl not installed) is a capability
+  gap, not an installation failure or error path.

package/node_modules/@code-yeongyu/senpi-codemode/src/tool/AGENTS.md

@@ -0,0 +1,40 @@
+# src/tool
+
+Eval tool core: input schema, cell execution lifecycle, detached-cell state
+machine, status events, and all call/result rendering. Earned by score 12 —
+highest reference density in the package (`createEvalTool` and `EvalToolDetails`
+anchor most suites).
+
+## WHERE TO LOOK
+
+| Task | Path |
+| --- | --- |
+| Tool registration, options | `eval-tool.ts`, `eval-tool-options.ts`, `eval-request.ts` |
+| Wire contract, TypeBox schemas | `types.ts` (`createEvalInputSchema`, `fullEvalInputSchema`) |
+| Cell execution, settlement | `cell-handler.ts`, `cell-execution.ts`, `cell-runtime.ts` |
+| Detached cells | `detached-cell-manager.ts` + `detached-cell-{state,snapshot,notification}.ts`, `detached-notification-queue.ts`, `detached-eval-result.ts` |
+| Call/result rendering | `render.ts`, `runtime-label.ts`, `json-tree.ts`, `image.ts`, `tool-widgets.ts` |
+| Status events, execution events | `status-events.ts`, `eval-execution-event.ts` |
+| Interrupt, capture | `interrupt-note.ts`, `call-capture.ts` |
+
+## CONVENTIONS
+
+- Wire/schema fields are snake_case (`cell_id`, `on_timeout`); internal TS
+  fields are camelCase. Schemas and shared eval types live only in `types.ts`.
+- Rendering is bounded by explicit line/code-point budgets with an injectable
+  render clock; nothing here depends on wall-clock luck.
+- Detached execution is a first-class state machine — snapshot, notification
+  queue, spill-file notice, result conversion — never folded into ordinary
+  cell execution.
+- Unicode tree glyphs and status icons are intentional UI conventions.
+
+## ANTI-PATTERNS
+
+- Never add unbounded output: previews, JSON tree depth/lines/scalar length,
+  widget lines, and collapsed errors all cap.
+- Never bypass the kernel bridge message contract with ad-hoc return values;
+  kernels import `KernelInterruptHandle` from `types.ts`, so discriminants and
+  lifecycle state are cross-runtime contracts — change them only with all four
+  runtimes and the detached path in mind.
+- `render.ts` (1,030 LOC) is the package's largest file and the highest-risk
+  hotspot for regressions; changes there need render contracts first.

package/node_modules/@code-yeongyu/senpi-codemode/src/tool/detached-cell-manager.ts

@@ -93,7 +93,7 @@ export class EvalDetachedCellManager {
 		this.#artifactsDir = options.artifactsDir;
 		this.#onStatusChange = options.onStatusChange;
 		this.#onWakeSourceState = options.onWakeSourceState;
-		this.#notificationQueue = new DetachedNotificationQueue(options.notifier, options.artifactsDir);
+		this.#notificationQueue = new DetachedNotificationQueue(options.notifier);
 		this.#now = options.now ?? Date.now;
 		this.#hardLimitSeconds = options.hardLimitSeconds ?? DEFAULT_HARD_LIMIT_SECONDS;
 	}

package/node_modules/@code-yeongyu/senpi-codemode/src/tool/detached-cell-notification.ts

@@ -12,7 +12,6 @@ export function detachedNotificationSpillPath(artifactsDir: string | undefined,
 export async function buildDetachedCellNotification(
 	snapshot: EvalDetachedCellSnapshot,
 	spillPath: string | undefined,
-	artifactsDir: string | undefined,
 ): Promise<EvalDetachedCellNotification> {
 	const body = notificationBody(snapshot);
 	const overflow = Buffer.byteLength(body, "utf8") > NOTIFICATION_TAIL_BYTES;
@@ -21,7 +20,9 @@ export async function buildDetachedCellNotification(
 		try {
 			await mkdir(dirname(spillPath), { recursive: true });
 			await writeFile(spillPath, body, "utf8");
-			spillNotice = `\nBuffered output overflowed; full output: ${localUri(spillPath, artifactsDir)}`;
+			// The agent read tool resolves plain paths only, so the notice must carry
+			// the absolute spill path, never the kernel-helper local:// scheme.
+			spillNotice = `\nBuffered output overflowed; full output: ${spillPath}`;
 		} catch (error) {
 			const message = error instanceof Error ? error.message : String(error);
 			spillNotice = `\nBuffered output overflow could not be spilled: ${message}`;
@@ -84,12 +85,6 @@ function safeCellId(cellId: string): string {
 	return cellId.replace(/[^a-zA-Z0-9_-]/gu, "_");
 }
 
-function localUri(path: string, artifactsDir: string | undefined): string {
-	if (artifactsDir === undefined) return `local://${path}`;
-	const root = join(artifactsDir, "local");
-	return path.startsWith(`${root}/`) ? `local://${path.slice(root.length + 1)}` : `local://${path}`;
-}
-
 function truncateTailUtf8(text: string, maxBytes: number): string {
 	if (Buffer.byteLength(text, "utf8") <= maxBytes) return text;
 	const bytes = Buffer.from(text, "utf8");

package/node_modules/@code-yeongyu/senpi-codemode/src/tool/detached-notification-queue.ts

@@ -7,14 +7,12 @@ export interface PendingDetachedNotification {
 }
 
 export class DetachedNotificationQueue {
-	readonly #artifactsDir: string | undefined;
 	readonly #notifier: EvalDetachedCellNotifier | undefined;
 	#pending: PendingDetachedNotification[] = [];
 	#flush: Promise<void> | undefined;
 
-	constructor(notifier: EvalDetachedCellNotifier | undefined, artifactsDir: string | undefined) {
+	constructor(notifier: EvalDetachedCellNotifier | undefined) {
 		this.#notifier = notifier;
-		this.#artifactsDir = artifactsDir;
 	}
 
 	enqueue(notification: PendingDetachedNotification): void {
@@ -32,9 +30,7 @@ export class DetachedNotificationQueue {
 		const flush = Promise.resolve().then(async () => {
 			const pending = this.#pending.splice(0);
 			const notifications = await Promise.all(
-				pending.map(
-					async (item) => await buildDetachedCellNotification(item.snapshot(), item.spillPath, this.#artifactsDir),
-				),
+				pending.map(async (item) => await buildDetachedCellNotification(item.snapshot(), item.spillPath)),
 			);
 			this.#notifier?.notify(notifications);
 		});

package/node_modules/@earendil-works/pi-agent-core/dist/agent-loop.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-loop.d.ts","sourceRoot":"","sources":["../src/agent-loop.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAIN,KAAK,OAAO,EAEZ,WAAW,EAIX,MAAM,uBAAuB,CAAC;AAU/B,OAAO,KAAK,EACX,YAAY,EACZ,UAAU,EACV,eAAe,EAEf,YAAY,EACZ,SAAS,EACT,aAAa,EAEb,QAAQ,EACR,MAAM,YAAY,CAAC;AAEpB,MAAM,MAAM,cAAc,GAAG,CAAC,KAAK,EAAE,UAAU,KAAK,OAAO,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC;AAEzE;;;GAGG;AACH,wBAAgB,SAAS,CACxB,OAAO,EAAE,YAAY,EAAE,EACvB,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,WAAW,CAAC,UAAU,EAAE,YAAY,EAAE,CAAC,CAiBzC;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAChC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,WAAW,CAAC,UAAU,EAAE,YAAY,EAAE,CAAC,CAwBzC;AAED,wBAAsB,YAAY,CACjC,OAAO,EAAE,YAAY,EAAE,EACvB,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,OAAO,CAAC,YAAY,EAAE,CAAC,CAgBzB;AAED,wBAAsB,oBAAoB,CACzC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,OAAO,CAAC,YAAY,EAAE,CAAC,CAiBzB;AA6QD,uGAAuG;AACvG,wBAAsB,oBAAoB,CACzC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,IAAI,CAAC,eAAe,EAAE,cAAc,GAAG,kBAAkB,CAAC,EAClE,MAAM,CAAC,EAAE,WAAW,GAClB,OAAO,CAAC,OAAO,CAAC,CAgBlB;AAqlBD,MAAM,WAAW,qBAAqB;IACrC,QAAQ,EAAE,aAAa,CAAC;IACxB,IAAI,EAAE,SAAS,CAAC;IAChB,IAAI,EAAE,OAAO,CAAC;CACd;AAED,wBAAgB,6BAA6B,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,aAAa,GAAG,aAAa,CAYrG;AAED,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,aAAa,GAAG,qBAAqB,CAOpG"}
+{"version":3,"file":"agent-loop.d.ts","sourceRoot":"","sources":["../src/agent-loop.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAIN,KAAK,OAAO,EAEZ,WAAW,EAIX,MAAM,uBAAuB,CAAC;AAU/B,OAAO,KAAK,EACX,YAAY,EACZ,UAAU,EACV,eAAe,EAEf,YAAY,EACZ,SAAS,EACT,aAAa,EAEb,QAAQ,EACR,MAAM,YAAY,CAAC;AAEpB,MAAM,MAAM,cAAc,GAAG,CAAC,KAAK,EAAE,UAAU,KAAK,OAAO,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC;AAEzE;;;GAGG;AACH,wBAAgB,SAAS,CACxB,OAAO,EAAE,YAAY,EAAE,EACvB,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,WAAW,CAAC,UAAU,EAAE,YAAY,EAAE,CAAC,CAiBzC;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,CAChC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,WAAW,CAAC,UAAU,EAAE,YAAY,EAAE,CAAC,CAwBzC;AAED,wBAAsB,YAAY,CACjC,OAAO,EAAE,YAAY,EAAE,EACvB,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,OAAO,CAAC,YAAY,EAAE,CAAC,CAgBzB;AAED,wBAAsB,oBAAoB,CACzC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,QAAQ,EAAE,QAAQ,GAChB,OAAO,CAAC,YAAY,EAAE,CAAC,CAiBzB;AA6QD,uGAAuG;AACvG,wBAAsB,oBAAoB,CACzC,OAAO,EAAE,YAAY,EACrB,MAAM,EAAE,IAAI,CAAC,eAAe,EAAE,cAAc,GAAG,kBAAkB,CAAC,EAClE,MAAM,CAAC,EAAE,WAAW,GAClB,OAAO,CAAC,OAAO,CAAC,CAgBlB;AA2lBD,MAAM,WAAW,qBAAqB;IACrC,QAAQ,EAAE,aAAa,CAAC;IACxB,IAAI,EAAE,SAAS,CAAC;IAChB,IAAI,EAAE,OAAO,CAAC;CACd;AAED,wBAAgB,6BAA6B,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,aAAa,GAAG,aAAa,CAYrG;AAED,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,aAAa,GAAG,qBAAqB,CAOpG"}

package/node_modules/@earendil-works/pi-agent-core/dist/agent-loop.js

@@ -514,6 +514,9 @@ async function streamAssistantResponse(context, config, signal, emit, streamFunc
     }
 }
 const ABORTED = Symbol("aborted");
+function abortError(reason) {
+    return reason instanceof Error ? reason : new Error("Request was aborted");
+}
 function normalizeTimeoutMs(timeoutMs) {
     return typeof timeoutMs === "number" && Number.isFinite(timeoutMs) && timeoutMs > 0 ? timeoutMs : undefined;
 }
@@ -539,7 +542,7 @@ function createAssistantEventReader(iterator, timeoutMs, signal, onIdleTimeout,
         next: async () => {
             if (signal?.aborted) {
                 void iterator.return?.();
-                return Promise.reject(new Error("Request was aborted"));
+                return Promise.reject(abortError(signal.reason));
             }
             // The start bound applies only until the provider proves the request is
             // alive with its first event; afterwards the idle bound governs as before.
@@ -548,7 +551,7 @@ function createAssistantEventReader(iterator, timeoutMs, signal, onIdleTimeout,
             const makeTimeoutError = useStartBound
                 ? (ms) => new StreamStartTimeoutError(ms)
                 : (ms) => new StreamIdleTimeoutError(ms);
-            const result = await readNextAssistantEvent(iterator, readTimeoutMs, makeTimeoutError, abortPromise, onIdleTimeout, stream);
+            const result = await readNextAssistantEvent(iterator, readTimeoutMs, makeTimeoutError, abortPromise, onIdleTimeout, stream, signal);
             if (!result.done)
                 sawFirstEvent = true;
             return result;
@@ -556,7 +559,7 @@ function createAssistantEventReader(iterator, timeoutMs, signal, onIdleTimeout,
         dispose: () => removeAbortListener?.(),
     };
 }
-async function readNextAssistantEvent(iterator, idleTimeoutMs, makeTimeoutError, abortPromise, onIdleTimeout, stream) {
+async function readNextAssistantEvent(iterator, idleTimeoutMs, makeTimeoutError, abortPromise, onIdleTimeout, stream, signal) {
     if (idleTimeoutMs === undefined && abortPromise === undefined) {
         return iterator.next();
     }
@@ -595,7 +598,7 @@ async function readNextAssistantEvent(iterator, idleTimeoutMs, makeTimeoutError,
         void next.then((result) => {
             if (result === ABORTED) {
                 void iterator.return?.();
-                settle(() => reject(new Error("Request was aborted")));
+                settle(() => reject(abortError(signal?.reason)));
                 return;
             }
             settle(() => resolve(result));