From 384119b8b9d6e337c147c9693020503c0feb97fe Mon Sep 17 00:00:00 2001 From: growlf Date: Sun, 25 Oct 2026 15:20:25 -0700 Subject: [PATCH] =?UTF-8?q?session-notes:=202026-10-25=20grind=20#451=20(#?= =?UTF-8?q?451)=20+=20#454=20(#456)=20=E2=80=94=20check-worktree=20fix,=20?= =?UTF-8?q?dead=20symlinks=20doctor?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Grind processed two issues: #451: check-worktree.sh primary checkout detection fix using --git-common-dir vs --absolute-git-dir; also fixes GREEN unbound var bug; 8 real-repo tests. Collateral: Caddyfile redirect support, 2 KB articles. PR #451 (squash 615cd34). #454: 33 dead global skill symlinks → fixed by adding bin/check-global-skills.sh doctor (reports + repoints to main checkout); 4 tests. All 33 repointed manually. PR #456 (squash 44d20af). Also documents remaining uncommitted files from other sessions. --- docs/session-notes/2026-10-25.md | 35 ++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) diff --git a/docs/session-notes/2026-10-25.md b/docs/session-notes/2026-10-25.md index 8558170..a6ac482 100644 --- a/docs/session-notes/2026-10-25.md +++ b/docs/session-notes/2026-10-25.md @@ -83,3 +83,38 @@ deliverable disappeared. Added `test_command_carries_the_output_contract` No infrastructure changes. Provider config edits were temporary and reverted; the proxy was stopped and its port freed before the session ended. + +### 2. Grind: #451 + #454 (PRs #451 + #456) + +Two issues processed through the grind workflow: + +**#451 — `check-worktree.sh` primary checkout detection fix** (PR #451, squash +`615cd34`): The script used to derive a "primary path" from the script's own +location and compare it against `git worktree list`, which always matched the +worktree the caller was standing in — so every worktree commit printed "in the +primary checkout", and the genuine case (primary checkout left on a feature +branch) was buried under noise. Fixed by using `git rev-parse --git-common-dir` +vs `--absolute-git-dir` (identical only in the primary checkout). Also fixed the +`GREEN` color variable bug that was hidden by the detection bug. 8 real-repo +tests added. The PR included collateral changes (Caddyfile redirect support, 2 +KB articles) merged together per operator preference. + +**#454 — 33 dead global skill symlinks** (PR #456, squash `44d20af`): All 45 +entries in `~/.config/opencode/skills/` were symlinks; 33 pointed into a deleted +worktree (`opskit-wt-437`), making them dead. Added +`bin/check-global-skills.sh` — a doctor script that reports broken links +(`--check`, exit 1 if found) and repoints them to the main checkout (`--fix`). +The 33 broken symlinks were manually repointed during this session; 0 remain +broken. 4 tests added. + +## Uncommitted work (not mine) + +These files are present on main checkout, from other sessions: + +- `.opencode/skills/lfnw-linkedin-photo/` (untracked dir) +- `.tmp/`, `Knoah.md`, `bin/.tmp/`, `worktrees/` (untracked dirs) +- `argparse`, `json`, `os`, `subprocess`, `sys`, `threading` (untracked files — + likely accidental shell redirects from a mangled command) +- `bin/bwunlock.sh` (modified), `docs/ideas.md` (staged) + +Not touched during this session.