Skip to content

docs: refresh grid screenshot, drop redundant terminal shot, clean root junk#139

Merged
almogdepaz merged 1 commit into
mainfrom
docs/refresh-screenshots
May 13, 2026
Merged

docs: refresh grid screenshot, drop redundant terminal shot, clean root junk#139
almogdepaz merged 1 commit into
mainfrom
docs/refresh-screenshots

Conversation

@almogdepaz
Copy link
Copy Markdown
Owner

Three small things:

  1. Replace docs/desktop-grid.png with a current screenshot of the grid view (the previous image was stale relative to recent UI changes).

  2. Drop docs/desktop-terminal.png from the README — it was the single-terminal-with-sidebar shot. The grid screenshot already implies the layout, and the mobile session list covers the cross-machine UX. README now leads with grid + mobile.

  3. Remove ~40 untracked PNGs and 4 review-*.md files from the repo root. All accumulated from past playwright/manual repro sessions and code review notes. .gitignore's *.png + !docs/*.png allowlist already prevents this kind of thing from being committed, so only the working tree was dirty. Only review-dev.md was actually tracked — the other deletions are silent working-tree cleanup.

Not touched (separate decision):

  • PLAN-*.md files in root (tracked — triaging next)
  • AGENTS.md, broker_stall.md, context.md, REVIEW_FIX_PLAN.md, sandbox-test.txt, tmux-client-97039.log (mixed tracked/untracked — separate scope)
  • docs/WhatsApp*.jpeg, docs/de/ (tracked but unreferenced — separate scope)

- Replace docs/desktop-grid.png with a current shot of the grid view
  (the previous one was stale relative to recent UI changes).
- Drop docs/desktop-terminal.png from the README — it just showed a
  single terminal with the sidebar, already implied by the grid shot
  + mobile session list. README now leads with grid + mobile.
- Remove untracked review-*.md notes from the repo root that had
  accumulated from past review sessions (review-dev.md was the only
  one tracked; the other three were already untracked).
- Remove ~40 untracked screenshot PNGs from the repo root (debug
  shots from playwright/manual repro sessions). .gitignore's '*.png'
  + 'docs/*.png' allowlist already protects against re-introducing
  them, this just cleans the working tree.
@almogdepaz almogdepaz merged commit 213dd8b into main May 13, 2026
1 check passed
almogdepaz added a commit that referenced this pull request May 13, 2026
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant