From 9d68d5f81b9db84a72ecc8b5e92eb2496a1005e4 Mon Sep 17 00:00:00 2001 From: JB Date: Fri, 4 Sep 2026 16:14:59 -0400 Subject: [PATCH] docs: approve Bash cwd dependency upgrade --- docs/workflow/bash-worktree-cwd-record.md | 24 ++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/docs/workflow/bash-worktree-cwd-record.md b/docs/workflow/bash-worktree-cwd-record.md index 26427c2..90772e0 100644 --- a/docs/workflow/bash-worktree-cwd-record.md +++ b/docs/workflow/bash-worktree-cwd-record.md @@ -2,9 +2,9 @@ ## Status -- Status: waiting -- Current phase: 2. Discover and decide requirements -- Next safe action: obtain approval for the Pi dependency upgrade scope and implementation plan before changing dependencies. +- Status: active +- Current phase: 5. Implement owned slices +- Next safe action: write the focused regression test against Pi 0.84.4, confirm it fails, then upgrade the aligned Pi packages to 0.85.0. ## Request and scope @@ -31,23 +31,25 @@ ## Approved implementation plan and gates - Plan/specification source: user request, `docs/todos.md`, and upstream Pi 0.84.4/0.85.0 source under `/srv/data/opensrc/repos/github.com/earendil-works/pi/`. -- Exact plan: pending approval. Proposed: (1) add a focused repository regression harness for dynamic Bash `ctx.cwd` with hidden-worktree root, nested package, and missing-directory cases; (2) update the aligned `@earendil-works/pi-ai`, `@earendil-works/pi-coding-agent`, and `@earendil-works/pi-tui` catalog versions to 0.85.0 and regenerate `bun.lock`; (3) run focused tests, typecheck, and the full suite; (4) update README compatibility text and resolve the TODO/workaround references; (5) verify in a fresh Pi 0.85.0 process without changing the currently running global Pi installation. -- Plan approval: pending. +- Exact plan: (1) add a focused repository regression harness for dynamic Bash `ctx.cwd` with hidden-worktree root, nested package, and missing-directory cases; (2) update the aligned `@earendil-works/pi-ai`, `@earendil-works/pi-coding-agent`, and `@earendil-works/pi-tui` catalog versions to 0.85.0 and regenerate `bun.lock`; (3) run focused tests, typecheck, and the full suite; (4) update README compatibility text and resolve the TODO/workaround references; (5) verify in a fresh Pi 0.85.0 process without changing the currently running global Pi installation. +- Plan approval: approved by the user through `ask_user` on 04SEP2026 (`approve_085`). - UI classification and direction approval: non-UI; no prototype gate applies. -- Risk decisions and scope: dependency upgrade requires explicit approval. Global Pi installation changes are excluded unless separately authorized. +- Risk decisions and scope: the user approved the aligned Pi 0.85.0 repository dependency upgrade. The user selected repository-only scope, so the active global Pi 0.84.4 installation remains unchanged. ## Ownership | Assignment | Owner identity | Deadline | Exclusive paths | Before/after Git evidence | Outcome | | --- | --- | --- | --- | --- | --- | -| Workflow setup and research | parent Pi session `01a06e08-8d07-76a7-a116-d7e3baf55b87` | 15 minutes | workflow record only | Before: clean `main` at `7e7cfdb`; after: pending initial record commit | In progress | +| Workflow setup and research | parent Pi session `01a06e08-8d07-76a7-a116-d7e3baf55b87` | 15 minutes | workflow record only | Before: clean `main` at `7e7cfdb`; after: clean feature worktree at `5f853dc` | Complete | +| Dependency, test, and docs implementation | parent Pi session `01a06e08-8d07-76a7-a116-d7e3baf55b87` | 15 minutes | `package.json`, `bun.lock`, focused test, `README.md`, `docs/todos.md`, workflow record | Before: clean feature worktree at `5f853dc`; after: pending | In progress | ## Verification matrix | Surface | Command | Exit code | Outcome | | --- | --- | --- | --- | | Source/cache preflight | `git status --short --branch`; `opensrc path ...` | 0 | Clean source; Pi 0.84.4 and 0.85.0 source available locally. | -| Focused cwd regression | pending | pending | pending | +| Focused cwd reproduction | inline Node harness against Pi 0.84.4 | 0 (asserted buggy outcome) | Both hidden-worktree requests observed the primary checkout; a missing requested cwd still executed a sentinel command. | +| Focused cwd regression | pending repository test | pending | pending | | Typecheck | `bun run typecheck` | pending | pending | | Full tests | `bun run test` | pending | pending | | Fresh-process integration | pending | pending | pending | @@ -75,6 +77,6 @@ ## Checkpoints and blockers -- Last committed valid checkpoint: none yet -- Blockers: dependency-upgrade and exact-plan approval. -- Exact next safe action: commit this initial record, present it, and ask the user to approve or reject the proposed dependency scope and plan. +- Last committed valid checkpoint: initial record commit `5f853dc` +- Blockers: none. +- Exact next safe action: commit the approved gate, then implement the focused regression test before updating dependencies.