@tduyng/pi-prewalk 0.0.1 → 0.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tduyng/pi-prewalk",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Plan on the frontier model, execute on the cheap one. The session model swaps the moment the first edit lands.",
   "keywords": [
     "pi",

package/src/bash-writes.ts

@@ -1,5 +1,3 @@
-import { tmpdir } from 'node:os'
-import { resolve } from 'node:path'
 import { isScratchPath } from './logic.js'
 
 const HEREDOC = /(?<!<)<<(?!<)-?\s*(['"]?)(\w+)\1/
@@ -39,7 +37,6 @@ function redirectsToWorkspace(command: string, cwd: string): boolean {
 
 export function bashWritesFiles(command: string, cwd: string): boolean {
     if (redirectsToWorkspace(command, cwd)) return true
-    if (command.includes(resolve(tmpdir()))) return false
     const text = withoutInertText(command)
     return WRITER_VERB.test(text) || (INTERPRETER.test(text) && SCRIPT_WRITE.test(command))
 }

package/src/config.ts

@@ -95,11 +95,14 @@ export function readEffectivePrewalkConfig(
     return prewalkConfigFrom({ ...readSettingsFile(agentPath), ...readSettingsFile(projectPath) })
 }
 
-export function writePrewalkConfig(config: PrewalkConfig, path: string = prewalkWritePath()): void {
+export function writePrewalkConfig(config: PrewalkConfig, path: string = prewalkWritePath()): boolean {
     try {
         const target = existsSync(path) ? realpathSync(path) : path
         const temporary = `${target}.${process.pid}.tmp`
         writeFileSync(temporary, `${JSON.stringify(config, null, 2)}\n`)
         renameSync(temporary, target)
-    } catch {}
+        return true
+    } catch {
+        return false
+    }
 }

package/src/index.ts

@@ -165,6 +165,14 @@ type PlanToolDefinition = ToolDefinition<typeof PLAN_PARAMETERS> & { defaultActi
 export default function prewalkExtension(pi: ExtensionAPI): void {
     let state: PrewalkState | null = null
     let config: PrewalkConfig = readEffectivePrewalkConfig()
+
+    const saveConfig = (ctx: ExtensionContext): void => {
+        if (writePrewalkConfig(config)) return
+        ctx.ui.notify(
+            'prewalk: the config change applies to this session only; the file could not be written.',
+            'warning'
+        )
+    }
     let stallNudged = false
     let completionNudged = false
     let reviewSpent = false
@@ -423,7 +431,7 @@ export default function prewalkExtension(pi: ExtensionAPI): void {
                 planner: args.planner === null ? config.planner : [plannerRef],
                 executor: args.executor === null ? config.executor : [executorRef]
             }
-            writePrewalkConfig(config)
+            saveConfig(ctx)
         }
         persist()
         updateStatus(ctx)
@@ -681,10 +689,11 @@ export default function prewalkExtension(pi: ExtensionAPI): void {
     pi.on('input', async (event) => {
         const text = event.text.trimStart()
         if (text.startsWith('/') || text.startsWith(SKILL_BLOCK_PREFIX) || text.startsWith(NUDGE_PREFIX)) return
+        if (event.source === 'extension') return
         stallNudged = false
         // A steer joins the task in flight and a follow-up waits for that turn to settle, so the
-        // steer never opens a task here and the follow-up routes as one when it arrives. Every
-        // other message opens a scored task, whichever extension sent it.
+        // steer never opens a task here and the follow-up routes as one when it arrives. Extension
+        // messages are continuations or injected context, not a new task, so they leave the task alone.
         if (event.streamingBehavior === 'steer') {
             steer = event.text
             return
@@ -874,7 +883,7 @@ export default function prewalkExtension(pi: ExtensionAPI): void {
 
             if (parsed.kind === 'mode') {
                 config = { ...config, mode: parsed.mode }
-                writePrewalkConfig(config)
+                saveConfig(ctx)
                 ctx.ui.notify(`prewalk: ${MODE_LABEL[parsed.mode]}`, 'info')
                 return
             }
@@ -894,7 +903,7 @@ export default function prewalkExtension(pi: ExtensionAPI): void {
                     parsed.chain.push(choice.ref)
                 }
                 config = { ...config, [parsed.role]: parsed.chain }
-                writePrewalkConfig(config)
+                saveConfig(ctx)
                 // The badge reads the armed pair, so a chain that moves under a live session moves it too.
                 if (state !== null) {
                     const head = chainModel(ctx, parsed.chain)