Skip to content

Commit b835528

Browse files
committed
docs(station): close out Plan 16 on main + flag follow-ups
- Status.md: Plan 16 In Progress → Recently Done (PR #23, 2026-04-20) - Plan 16 header: Active → Shipped (squash 28d181e) - memory.md: drop Plan 16 from in-flight, refresh Last completed, flag ui-ux-testing branch is now 14 behind main and needs rebase - Backlog: mark go-install bug fixed; add 2 new entries: doc-drift in Bonsai/CLAUDE.md ProjectStructure tree (lines 19, 109) and stale agent worktree pile under .claude/worktrees/
1 parent 28d181e commit b835528

4 files changed

Lines changed: 9 additions & 6 deletions

File tree

station/Playbook/Backlog.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,9 @@ Items that should be worked together are tagged with a group letter. See the gro
5757
<!-- "Spinner error swallowing" — scope-deferred to Plan 15 harness work, 2026-04-17 -->
5858
<!-- ".golangci.yml + test/lint/fmt Makefile targets" — shipped 2026-04-17, Plan 17 / PR #24 (demo GIF item remains, see Group C below) -->
5959
- **[debt] Testing infrastructure for triggers and sensors** `[Group B]` — No testing infrastructure exists for hook-based triggers, prompt hooks, context-guard regex patterns, path-scoped rules, or skill auto-invocation. Need: (1) unit tests for context-guard regex patterns (positive/negative cases), (2) integration test harness for sensor scripts (mock stdin, verify stdout/exit codes), (3) end-to-end test framework for trigger activation (simulate user prompts, verify correct ability loads), (4) prompt hook evaluation testing (verify Haiku correctly classifies intents). The trigger system is expanding significantly — without test infra, regressions will be invisible. *(added 2026-04-16, source: user)*
60-
- **[bug] `go install .` installs binary as `Bonsai` (capital B), not `bonsai`** `[Group F]` — Module path is `github.com/LastStep/Bonsai`, so Go names the installed binary after the last path segment → `Bonsai`. Linux is case-sensitive, so `bonsai` fails to resolve on PATH. README instructs `go install .` — anyone following it gets a broken install until they rename. The `Makefile` dodges this with `-o bonsai`, but `go install` has no such override. **Fix options:** (1) rename repo/module to lowercase `bonsai` (breaking — changes import path), (2) move `main.go` to `cmd/bonsai/main.go` so `go install ./cmd/bonsai` produces correct name (clean, no rename), (3) add a README note about the rename step (worst — puts burden on users). Option 2 is the right fix. *(added 2026-04-17, source: session — found during UI/UX testing setup)*
60+
<!-- "go install . installs binary as Bonsai (capital B)" — fixed 2026-04-20 via Plan 16 / PR #23 (option 2: main.go → cmd/bonsai/main.go + root embed.go) -->
61+
- **[debt] `Bonsai/CLAUDE.md` ProjectStructure tree references stale `main.go` location** `[doc-drift]` — After Plan 16 merge, lines 19 (`├── main.go ← entry point, embeds catalog/ via embed.FS`) and 109 (`Catalog is embedded via embed.FS in main.go`) still describe pre-move layout. Linter/user already fixed line 118 install command, but the structural tree + key-concept bullet were missed. Should now read: `├── cmd/bonsai/main.go` + `├── embed.go ← root embed package (CatalogFS, GuideContent)` and "embedded via embed.FS in `embed.go`". Trivial fix; surfaced post-merge. *(added 2026-04-20, source: session — found during Plan 16 close-out)*
62+
- **[debt] Stale agent worktrees accumulating under `.claude/worktrees/`** `[housekeeping]``git worktree list` shows ~15 leftover worktrees from prior plan dispatches (some on UNC `//wsl.localhost/...` paths from cross-OS sessions, some on Linux paths). Branches mostly already merged. The UNC ones can't be removed from this side; the Linux-side ones are safely prunable. Risk: confused state when re-dispatching to a worktree-named branch, occasional `branch -D` failures (hit during PR #23 merge). Suggested: add a station routine or `bonsai` chore to prune merged worktrees periodically, plus a one-time manual sweep. *(added 2026-04-20, source: session — found during Plan 16 close-out)*
6163
<!-- "Review panel borders break in non-fullscreen terminals" — fixed 2026-04-17 via width-aware TitledPanel (ansi.Truncate + term.GetSize) -->
6264
<!-- "Explicit feedback for required-only sections" — fixed 2026-04-17 via collapsed chip line in PickItems -->
6365
<!-- Implicit fix: Getwd error now surfaces via FatalPanel instead of becoming a confusing "open .bonsai.yaml: no such file or directory" — see mustCwd() in cmd/root.go -->

station/Playbook/Plans/Active/16-go-install-binary-name.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Plan 16 — Fix `go install` binary name
22

33
**Tier:** 1 (Patch)
4-
**Status:** Active
4+
**Status:** Shipped — PR #23 merged 2026-04-20 (squash `28d181e`)
55
**Agent:** general-purpose
66
**Source:** `Playbook/Backlog.md` P1 bug (added 2026-04-17)
77

station/Playbook/Status.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@ description: Live task tracker. Update this file at the start and end of every w
1818
<!-- UI/UX overhaul Phase 2 — merged PR #20, moved to Recently Done 2026-04-17 -->
1919
<!-- ActionUnchanged follow-ups — merged PR #22, moved to Recently Done 2026-04-17 -->
2020
| BubbleTea foundation + theme system (migrate init/add/remove/update onto shared harness) | 15 | general-purpose | On `ui-ux-testing` branch. 3 iterations planned. Iter 1 = harness + theme + init migration |
21-
| Fix `go install` binary name (module path → `/cmd/bonsai`) | 16 | general-purpose | Draft [PR #23](https://github.com/LastStep/Bonsai/pull/23) still open, awaiting user merge. |
21+
<!-- Plan 16 (`go install` binary name) — merged PR #23 2026-04-20, moved to Recently Done -->
22+
2223
<!-- UI/UX Overhaul Phase 3 (Plan 14) — merged via PR #24, moved to Recently Done 2026-04-17 -->
2324
<!-- Release prep (Plan 17) — merged PR #24, moved to Recently Done 2026-04-17 -->
2425
<!-- Code index refresh — completed 2026-04-16, PR #12 -->
@@ -38,6 +39,7 @@ description: Live task tracker. Update this file at the start and end of every w
3839

3940
| Task | Plan | Agent | Date |
4041
|------|------|-------|------|
42+
| Fix `go install` binary name — main.go → cmd/bonsai/ + root embed package (PR #23) | 16 | general-purpose | 2026-04-20 |
4143
| Release prep — Go toolchain 1.24.13 + triggerSection frontmatter fix + OSS polish (PR #24) | 17 | general-purpose | 2026-04-17 |
4244
| UI/UX overhaul — Phase 3 visual identity + init polish (merged via PR #24 bundle) | 14 | general-purpose | 2026-04-17 |
4345
| ActionUnchanged follow-ups — chmod bug + test gaps (PR #22) | 13 | tech-lead | 2026-04-17 |

station/agent/Core/memory.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,9 @@ description: Tech Lead Agent working memory — flags, work state, notes.
1515

1616
**Current task:** None in-session. Awaiting next pickup.
1717
**In flight (other tracks):**
18-
- Plan 15 (BubbleTea foundation) — on `ui-ux-testing` branch, parallel session driving iter 2/3
19-
- Plan 16 (go install binary name) — draft PR #23 still open, awaiting user merge
18+
- Plan 15 (BubbleTea foundation) — on `ui-ux-testing` branch, parallel session driving iter 2/3. **Branch is now 14 commits behind main after PR #23 merge — needs rebase before iter 2 starts (`cmd/init.go` and `internal/tui/` likely conflict).**
2019
**Blocked on:** Nothing in this session.
21-
**Last completed:** Plan 17 merged to main via PR #24 (`bc565bf`, 2026-04-17). Bundled: go toolchain 1.24.3→1.24.13, triggerSection frontmatter fix (`injectTriggerSection` helper + 4-case test), `.golangci.yml` + Makefile test/lint/fmt/tidy + parallel CI lint job, and Plan 14 TUI code (palette tokens, banner, width-aware panels, mustCwd error surfacing).
20+
**Last completed:** Plan 16 merged to main via PR #23 (`28d181e`, 2026-04-20). Moved `main.go``cmd/bonsai/main.go` + introduced root `embed.go` package (keeps `//go:embed catalog/` and `docs/custom-files.md` resolving from repo root). `go install github.com/LastStep/Bonsai/cmd/bonsai@latest` now produces lowercase `bonsai` binary on PATH. Verified locally before merge.
2221

2322
## Notes
2423

0 commit comments

Comments
 (0)