@code-yeongyu/senpi-codemode 2026.9.12 → 2026.9.13-2

package/CHANGELOG.md

@@ -12,6 +12,58 @@
 
 ### Removed
 
+## [2026.9.13-2] - 2026-09-13
+
+### Breaking Changes
+
+### Added
+
+- Added `PI_SESSION_CWD` and `PI_GOAL_STORE_FILE` session environment keys for kernels and shell children, including clearing of inherited values when absent (fixes #1663).
+
+- Interactive foreground eval cells detach on queued steering without cancelling their computation or in-flight tools. An occupied detached slot keeps the call waiting ([#1637](https://github.com/code-yeongyu/senpi/issues/1637)).
+
+### Changed
+
+### Fixed
+
+### Removed
+
+## [2026.9.13] - 2026-09-13
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+### Removed
+
+## [2026.9.12-3] - 2026-09-12
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+### Removed
+
+## [2026.9.12-2] - 2026-09-12
+
+### Breaking Changes
+
+### Added
+
+### Changed
+
+### Fixed
+
+### Removed
+
 ## [2026.9.12] - 2026-09-12
 
 ### Breaking Changes

package/README.md

@@ -57,8 +57,9 @@ schema; it is not an installation failure.
 ### Session environment
 
 Every kernel starts with the active session's `PI_*` environment — `PI_SESSION_ID`,
-`PI_SESSION_FILE` (when the session is persistent), `PI_PROVIDER`, `PI_MODEL`, and
-`PI_REASONING_LEVEL` (when set) — resolved at session start, mirroring the bash tool's
+`PI_SESSION_FILE` (when the session is persistent), `PI_SESSION_CWD` (the session's
+working directory), `PI_GOAL_STORE_FILE` (the authoritative goal-store path, when the
+host provides it), `PI_PROVIDER`, `PI_MODEL`, and `PI_REASONING_LEVEL` (when set) — resolved at session start, mirroring the bash tool's
 session environment contract. The values are visible to `env()`/`process.env`/`os.environ`
 inside cells and are inherited by every child process a cell spawns
 (`Bun.$`, `Bun.spawn`, `child_process`, `subprocess`, ...). Inherited `PI_*` values from
@@ -67,6 +68,11 @@ what a child spawned from the bash tool sees. The values snapshot at kernel star
 mid-session model switch updates the bash tool's next command but not already-running
 kernels; a new session starts fresh kernels with fresh values.
 
+`PI_GOAL_STORE_FILE` is supplied by the host's optional `ExtensionContext.goalStoreFile`
+getter and may name a file that does not exist yet. It honors session-directory overrides
+and in-memory sessions; it cannot be derived reliably from `PI_SESSION_FILE`. If the host
+omits the getter, the variable is unset rather than inherited from the launching process.
+
 ## Settings
 
 Configuration is loaded in this order:
@@ -177,6 +183,12 @@ cell keeps only its own language kernel busy. A new same-language call returns
 a busy error with its cell id and output tail; calls in other languages continue
 normally. Do not re-run the cell.
 
+Queued steering also detaches an eligible interactive foreground call, including
+one paused in a host tool bridge, without cancelling its computation. If the
+language's detached slot is occupied, steering leaves the call waiting. Follow-up
+messages, explicit `on_timeout: "error"`, and print/JSON calls do not trigger this
+transition; caller abort and existing deadlines retain their cancellation behavior.
+
 Every cell, detached or not, is bounded by two kill deadlines. The run budget
 (`runBudgetSeconds`, or the call's `timeout`) charges only the cell's own
 execution time and is paused while a host tool call is in flight, so a cell
@@ -185,8 +197,8 @@ The hard limit (`hardLimitSeconds`, raised by a larger `timeout`) is wall-clock
 and bounds parked cells too. A cell killed by either deadline reports which one
 in its result or completion notification, together with whether kernel state
 survived; the tool schema states the configured numbers. The `timeout` value
-never changes when an interactive call detaches: that is `cellTimeoutSeconds`
-capped by `foregroundWindowSeconds`.
+never changes the idle detach deadline: that is `cellTimeoutSeconds` capped by
+`foregroundWindowSeconds`; queued steering can detach the call earlier.
 
 While any cell is detached, the interactive footer shows a highlighted
 `↗ <language> · <summary>` status on the extension status line (the cell id

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/senpi-codemode",
-	"version": "2026.9.12",
+	"version": "2026.9.13-2",
 	"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.9.12",
-		"typebox": "1.3.18"
+		"@earendil-works/pi-ai": "npm:@code-yeongyu/senpi-ai@2026.9.13-2",
+		"typebox": "1.3.27"
 	},
 	"peerDependencies": {
-		"@code-yeongyu/senpi": "2026.9.12"
+		"@code-yeongyu/senpi": "2026.9.13-2"
 	},
 	"devDependencies": {
-		"@code-yeongyu/senpi": "2026.9.12"
+		"@code-yeongyu/senpi": "2026.9.13-2"
 	},
 	"keywords": [
 		"senpi",

package/src/kernels/js/worker-core.js

@@ -9,6 +9,8 @@ const INTERRUPT_ACK_OP = "interrupt-ack";
 const SESSION_ENVIRONMENT_KEYS = [
 	"PI_SESSION_ID",
 	"PI_SESSION_FILE",
+	"PI_SESSION_CWD",
+	"PI_GOAL_STORE_FILE",
 	"PI_PROVIDER",
 	"PI_MODEL",
 	"PI_REASONING_LEVEL",

package/src/kernels/session-env.ts

@@ -12,6 +12,8 @@
 export const SESSION_ENVIRONMENT_KEYS = [
 	"PI_SESSION_ID",
 	"PI_SESSION_FILE",
+	"PI_SESSION_CWD",
+	"PI_GOAL_STORE_FILE",
 	"PI_PROVIDER",
 	"PI_MODEL",
 	"PI_REASONING_LEVEL",
@@ -22,6 +24,8 @@ export type SessionEnvironment = Readonly<Record<string, string>>;
 
 /** Structural slice of `ExtensionContext` the session environment is resolved from. */
 export interface SessionEnvironmentSource {
+	readonly cwd: string;
+	readonly goalStoreFile?: string;
 	readonly sessionManager: {
 		getSessionId(): string;
 		getSessionFile(): string | undefined;
@@ -33,6 +37,8 @@ export interface SessionEnvironmentSource {
 export function sessionEnvironmentFrom(source: SessionEnvironmentSource): SessionEnvironment {
 	const env: Record<string, string> = {};
 	env.PI_SESSION_ID = source.sessionManager.getSessionId();
+	env.PI_SESSION_CWD = source.cwd;
+	if (source.goalStoreFile) env.PI_GOAL_STORE_FILE = source.goalStoreFile;
 	const sessionFile = source.sessionManager.getSessionFile();
 	if (sessionFile) env.PI_SESSION_FILE = sessionFile;
 	const model = source.model;

package/src/tool/detached-cell-manager.ts

@@ -144,7 +144,12 @@ export class EvalDetachedCellManager {
 	}
 
 	detach(cell: ManagedCell): boolean {
-		if (!cell.canDetach || !allowsDetachedCellTransition(cell.state, "detached")) return false;
+		if (
+			!cell.canDetach ||
+			!allowsDetachedCellTransition(cell.state, "detached") ||
+			this.#detachedByLanguage.has(cell.input.language)
+		)
+			return false;
 		cell.state = "detached";
 		cell.wasDetached = true;
 		this.#detachedByLanguage.set(cell.input.language, cell);

package/src/tool/run-eval-cell.ts

@@ -49,6 +49,12 @@ export async function runEvalCell(
 	let detached = false;
 	let execution: CellExecution;
 	const cell = cellManager.create(invocation.cellId, invocation.input, (error) => execution.cancel(error));
+	const detach = (): boolean => {
+		if (!cellManager.detach(cell)) return false;
+		detached = true;
+		execution.detach();
+		return true;
+	};
 	execution = new CellExecution({
 		callerSignal: invocation.signal,
 		cellId: invocation.cellId,
@@ -58,12 +64,7 @@ export async function runEvalCell(
 						timeoutMs: detachAfterMs,
 						maxPauseGraceMs: foregroundWindowMs,
 						onTimeout: (error: Error) => {
-							if (cellManager.detach(cell)) {
-								detached = true;
-								execution.detach();
-								return;
-							}
-							execution.cancel(error);
+							if (!detach()) execution.cancel(error);
 						},
 					},
 				}
@@ -74,6 +75,16 @@ export async function runEvalCell(
 			bridgeAbortController.abort(error);
 		},
 	});
+	const steeringSignal =
+		detaches && invocation.ctx.mode !== "print" && invocation.ctx.mode !== "json"
+			? invocation.ctx.steeringSignal
+			: undefined;
+	const onSteering = (): void => {
+		if (!steeringSignal?.aborted || detached || !state.active || invocation.signal.aborted) return;
+		// Losing the slot keeps this call foreground; only the existing deadlines/caller may cancel it.
+		detach();
+	};
+	steeringSignal?.addEventListener("abort", onSteering, { once: true });
 	const running = executeCell(
 		options,
 		invocation,
@@ -83,6 +94,7 @@ export async function runEvalCell(
 		execution,
 		bridgeContext,
 		bridgeAbortController,
+		onSteering,
 	);
 	let settleEventEmitted = false;
 	const emitSettled = (outcome: EvalExecutionSettleOutcome): void => {
@@ -110,12 +122,16 @@ export async function runEvalCell(
 			throw error;
 		},
 	);
-	const outcome = await Promise.race([
-		finalized.then((result) => ({ kind: "result" as const, result })),
-		execution.detached.then(() => ({ kind: "detached" as const })),
-	]);
-	if (outcome.kind === "detached") return resultAfterDetach(cellManager.peek(invocation.cellId), invocation.input);
-	return outcome.result;
+	try {
+		const outcome = await Promise.race([
+			finalized.then((result) => ({ kind: "result" as const, result })),
+			execution.detached.then(() => ({ kind: "detached" as const })),
+		]);
+		if (outcome.kind === "detached") return resultAfterDetach(cellManager.peek(invocation.cellId), invocation.input);
+		return outcome.result;
+	} finally {
+		steeringSignal?.removeEventListener("abort", onSteering);
+	}
 }
 
 async function executeCell(
@@ -127,6 +143,7 @@ async function executeCell(
 	execution: CellExecution,
 	bridgeContext: ExtensionContext,
 	bridgeAbortController: AbortController,
+	onReady: () => void,
 ): Promise<AgentToolResult<EvalToolDetails>> {
 	let handler: CellHandler | undefined;
 	try {
@@ -168,6 +185,8 @@ async function executeCell(
 			() => activeHandler.liveResult(),
 			(error) => execution.cancel(error),
 		);
+		// Includes a steer already queued at execute start or received while acquiring the kernel.
+		onReady();
 		if ("setContext" in options.kernelManager && typeof options.kernelManager.setContext === "function") {
 			options.kernelManager.setContext(bridgeContext);
 		}