docs: retire the merged-stack instructions from the Workspaces guide - #658
dormouse-bot wants to merge 2 commits into
Conversation
Deploying mouseterm with
|
| Latest commit: |
1627e92
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://789b867b.mouseterm.pages.dev |
| Branch Preview URL: | https://fix-workspaces-guide-poststa.mouseterm.pages.dev |
dormouse-bot
left a comment
dormouse-bot
left a comment
There was a problem hiding this comment.
Feedback on work in progress, not a merge verdict — mark the PR ready when you want the full review.
The replacement text asserts the branches no longer exist, and all eleven are still on the remote: gh api repos/diffplug/dormouse/git/ref/heads/workspaces-window (and each of the other ten) resolves today. That matters here more than it would elsewhere, since accuracy about the current state is the whole reason for the edit — an agent that trusts the line will read origin/workspaces-fidelity in a fetch listing as a leftover it should clean up, or as evidence the guide is stale and stop trusting the rest of section 1. The merge is the durable fact; whether the branches get deleted later is not, so phrasing it as "merged and no longer worked on" stays true either way. Inline suggestion below.
The dates and the table check out: all eleven PRs merged 2026-09-15T02:00Z, and the merge commits on main do name the branches (Merge branch 'workspaces-harness' into workspaces-transfer-fixes), which is the justification for keeping the Branch column.
|
this file will soon be deleted completely, no point editing it |
2 participants
TESTING_AND_MODIFICATION_GUIDE.mdopens by telling an agent to work on the stack's tip branch, merge changes forward through the later branches, and land the PRs in order with merge commits. All eleven of those PRs (#614–#623, #630) merged on 2026-09-15, so an agent following section 1 or 1a today looks for branches and worktrees that no longer exist, and a Workspaces change made onmainreads as a mistake against instructions that assume the stack is still open.This replaces both sections with a note that the work is on
mainand that the owning specs in section 6 lead, keeping the PR table as an index of which PR introduced what — that survives the merge and is what makes agit blamehit on one of these branches legible. The dead Worktree column is dropped; the Branch column stays because the merge commits name those branches. Two later phrases that said "the stack" now say "Workspaces", since the reader no longer has a stack in front of them.Sections 2 through 8 and the Transfer findings are untouched: the setup steps, the
cargo testnode-binary trap, the test inventory, the manual checklist, the "rules that bite", and the known gaps all describemainas it stands and are the reason to keep this file.Nothing else in the repo links to this guide, so it is only findable by listing the repo root. If you want it reachable from the documented entry point, a one-line pointer in
AGENTS.mdwould do it — held back here because it is a separate concern and lands against that file's spec-lint word budget.No test: the change is documentation with no executable behavior.
pnpm lint:specsand the five sibling lints report the same state asmain(two pre-existingstandalone/sidecar/node_modulespath hits that only appear when dependencies are not installed).