@howaboua/pi-codex-conversion 1.0.6 → 1.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@howaboua/pi-codex-conversion",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "Codex-oriented tool and prompt adapter for pi coding agent",
   "type": "module",
   "repository": {

package/src/patch/paths.ts

@@ -8,20 +8,17 @@ export function normalizePatchPath({ path }: { path: string }): string {
 	return withoutAt.replace(/^['"]|['"]$/g, "");
 }
 
-// Patch paths are intentionally confined to ctx.cwd so the adapter cannot write
-// outside the active workspace even if the incoming patch text is malicious.
+// Relative patch paths stay anchored to ctx.cwd. Absolute patch paths are
+// accepted as-is so the adapter can match Codex-style path usage.
 export function resolvePatchPath({ cwd, patchPath }: { cwd: string; patchPath: string }): string {
 	const normalized = normalizePatchPath({ path: patchPath });
 	if (!normalized) {
 		throw new DiffError("Patch path cannot be empty");
 	}
-	if (isAbsolute(normalized)) {
-		throw new DiffError("We do not support absolute paths.");
-	}
 
-	const absolutePath = resolve(cwd, normalized);
+	const absolutePath = isAbsolute(normalized) ? normalized : resolve(cwd, normalized);
 	const rel = relative(cwd, absolutePath);
-	if (rel.startsWith("..") || isAbsolute(rel)) {
+	if (!isAbsolute(normalized) && (rel.startsWith("..") || isAbsolute(rel))) {
 		throw new DiffError(`Path escapes working directory: ${normalized}`);
 	}
 	return absolutePath;

package/src/prompt/build-system-prompt.ts

@@ -5,6 +5,8 @@ export interface PromptSkill {
 }
 
 const CODEX_GUIDELINES = [
+	"Prefer a single `apply_patch` call that updates all related files together when one coherent patch will do.",
+	"When multiple tool calls are independent, emit them together so they can execute in parallel instead of serializing them.",
 	"Use `parallel` only when tool calls are independent and can safely run at the same time.",
 	"Use `write_stdin` when an exec session returns `session_id`, and continue until `exit_code` is present.",
 	"Do not request `tty` unless interactive terminal behavior is required.",

package/src/tools/exec-session-manager.ts

@@ -65,7 +65,7 @@ export interface ExecSessionManager {
 }
 
 const DEFAULT_EXEC_YIELD_TIME_MS = 10_000;
-const DEFAULT_WRITE_YIELD_TIME_MS = 250;
+const DEFAULT_WRITE_YIELD_TIME_MS = 10_000;
 const DEFAULT_MAX_OUTPUT_TOKENS = 10_000;
 const MIN_YIELD_TIME_MS = 250;
 const MAX_YIELD_TIME_MS = 30_000;
@@ -292,18 +292,24 @@ export function createExecSessionManager(): ExecSessionManager {
 		notify(session);
 	}
 
-	function waitForActivity(session: ExecSession, yieldTimeMs: number): Promise<number> {
-		if (session.buffer !== session.emittedBuffer || session.exitCode !== undefined && session.exitCode !== null) {
+	function waitForExitOrTimeout(session: ExecSession, yieldTimeMs: number): Promise<number> {
+		if (session.exitCode !== undefined && session.exitCode !== null) {
 			return Promise.resolve(0);
 		}
 
 		const startedAt = Date.now();
 		return new Promise((resolvePromise) => {
 			const onWake = () => {
+				if (session.exitCode === undefined || session.exitCode === null) {
+					return;
+				}
 				cleanup();
 				resolvePromise(Date.now() - startedAt);
 			};
-			const timeout = setTimeout(onWake, yieldTimeMs);
+			const timeout = setTimeout(() => {
+				cleanup();
+				resolvePromise(Date.now() - startedAt);
+			}, yieldTimeMs);
 			const cleanup = () => {
 				clearTimeout(timeout);
 				session.listeners.delete(onWake);
@@ -432,7 +438,7 @@ export function createExecSessionManager(): ExecSessionManager {
 			sessions.set(session.id, session);
 			rememberCommand(session.id, session.command);
 
-			const waitedMs = await waitForActivity(session, clampYieldTime(input.yield_time_ms, DEFAULT_EXEC_YIELD_TIME_MS));
+			const waitedMs = await waitForExitOrTimeout(session, clampYieldTime(input.yield_time_ms, DEFAULT_EXEC_YIELD_TIME_MS));
 			return makeResult(session, waitedMs, input.max_output_tokens);
 		},
 		write: async (input) => {
@@ -450,7 +456,7 @@ export function createExecSessionManager(): ExecSessionManager {
 			}
 			const waitedMs =
 				session.exitCode === undefined
-					? await waitForActivity(session, clampYieldTime(input.yield_time_ms, DEFAULT_WRITE_YIELD_TIME_MS))
+					? await waitForExitOrTimeout(session, clampYieldTime(input.yield_time_ms, DEFAULT_WRITE_YIELD_TIME_MS))
 					: 0;
 			return makeResult(session, waitedMs, input.max_output_tokens);
 		},