chore: untrack stale plan + clean root markdown junk#140
Merged
Conversation
Old ralph implementation plan for the broker migration. The migration is long done — the broker is mandatory in mainline and tmux ownership has been removed. The file was already covered by .gitignore (line 35) but tracked from before the rule was added. Removed from index only; working copy preserved for local reference.
Re-apply the cleanup originally pushed to PR #139 as commit f580769 but never merged — github merged the PR at the first commit's HEAD (86b1a48) before this follow-up commit was pushed. Tracked files removed: - PHASE-4-VISIBLE-REFLOW.md — followup plan to SCROLLBACK-REFLOW-PLAN.md (which is already gone); no inbound refs anywhere in the tree. - REVIEW_FIX_PLAN.md — fix plan for PR #123, merged 2026-04-30. Dead doc. - context.md — old edc-build output. AGENTS.md (when present) points to the new .context/ layout instead; this file is the unmigrated remnant. - sandbox-test.txt — "hello from sandbox test" (24 bytes), no inbound refs. Tests prefixed 'ralph-sandbox-test-' were a substring false match, not an actual user. - review-tasks/ (11 files) — tool output from the PR #106 differential review (long merged). One-time dump, never updated, no current value. Adding to .gitignore so a future review run doesn't track it again. Kept (referenced from code): - broker_stall.md — referenced by comment in src/server/backend.ts:182 Verification: bun test → 1483 pass / 0 fail.
almogdepaz
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two related cleanups bundled together.
1. Untrack
docs/ralph-broker-plan.mdOld ralph implementation plan for the broker migration. The migration is long done — the broker is mandatory in mainline and tmux session ownership has been removed.
The file was already covered by
.gitignore(line 35) but tracked from before that rule was added.git rm --cachedremoves the index entry; working copy preserved locally for reference.Zero inbound code or doc references.
2. Re-apply the cleanup lost from PR #139
PR #139 originally had two commits. github merged it at HEAD = first commit (
86b1a48), and the second commit (f580769 chore: remove stale docs and tool output from repo root) was pushed afterward — never made it to main.Re-applying it here:
Tracked files removed:
PHASE-4-VISIBLE-REFLOW.md— followup plan toSCROLLBACK-REFLOW-PLAN.md(already gone); no inbound refs.REVIEW_FIX_PLAN.md— fix plan for PR feat(broker): Rust PTY broker replaces tmux + in-process session ownership #123, merged 2026-04-30. Dead.context.md— old edc-build output.AGENTS.md(when present) points to the new.context/layout instead; this file is the unmigrated remnant.sandbox-test.txt—hello from sandbox test(24 bytes), no real users.review-tasks/(11 files) — tool output from the PR no-tmux backend, push notifications, terminal stability fixes #106 differential review. One-time dump, no current value. Adding to.gitignore.Kept:
broker_stall.md— referenced fromsrc/server/backend.ts:182Verification
bun test→ 1483 pass / 0 fail