docs: record verified merged worktree retirement - #150
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The new roadmap text references repo-relative audit/* files that don’t exist in the repository and includes a time-sensitive “no Actions secrets configured” claim that should be clarified/timestamped for long-term accuracy.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR updates docs/roadmap.md to record follow-through actions after the 2026-09-07 historical audit, specifically the retirement of merged worktrees/branches and the current status/evidence around the CI-image proposal failure tracked in #149.
Changes:
- Add a “Worktree retirement follow-through” section documenting the removal of four merged worktrees and associated local/remote branch deletions.
- Record evidence that v17’s checkout-history repair work is superseded by merged work (#143), with #144 closed.
- Summarize the current state of issue #149, including the missing credential path for workflow-file updates and required retry behavior.
File summaries
| File | Description |
|---|---|
| docs/roadmap.md | Adds a retirement follow-through subsection and notes on #143/#144 and #149 status/evidence. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| listing verified their absence. The recovery set records these operations in | ||
| `audit/merged-retirement-20260909.json` and | ||
| `audit/merged-refs-retirement-20260909.json`. No force deletion was used. |
There was a problem hiding this comment.
Clarified in 31a5ee9: these are files in the audit directory of the maintainer-held recovery set outside this repository, not repository paths.
| Three worktrees remain: main, parked #86, and v17. Review of closed #144 confirms | ||
| that merged #143 supersedes v17's shallow-checkout repair by retaining full Chat | ||
| history. The current workflow suite passed 59 tests with 19 platform skips; |
| proposal failure and retry defect. Read-only inspection found no repository | ||
| Actions secrets configured: the proposed separate credential is absent. The | ||
| image itself already built and passed its verification job. Repairing the | ||
| proposal requires both an authorized workflow-file write path and retry logic | ||
| that does not confuse an existing branch with a completed update and PR. |
There was a problem hiding this comment.
Clarified in 31a5ee9: the observation is dated 2026-09-09 UTC and scoped to the repository and repository-accessible organization secrets returned to the authenticated maintainer. Both read-only REST lists were empty at inspection time.
Record the verified reduction from seven to three worktrees, safe local branch deletion and leased remote deletion, v17 supersession evidence, and the missing CI-image credential tracked in #149.
Validation: fresh clean-state, exact-tip ancestry, process ownership, filesystem absence, and remote-ref checks; 59 workflow tests passed with 19 platform skips; 50 specification guards passed; git diff --check passed. The recovery set contains machine-readable removal receipts. PR #86 and the divergent v17 history remain preserved.