Skip to content

Remove old review screenshots, archive/, unused icon drafts, and Codex scripts - #1848

Merged
r3dbars merged 3 commits into
mainfrom
claude/project-thread-bj8ydh
Sep 25, 2026
Merged

r3dbars merged 3 commits into
mainfrom
claude/project-thread-bj8ydh

Conversation

@r3dbars

@r3dbars r3dbars commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Requested by Justin · project thread

Why

Justin said yes to clearing four piles of old files that nothing uses anymore (follow-up to #1845). Everything here is still in git history.

Product Impact

  • Affects: docs only (repo files and dev tooling; no app code)
  • Lane: agent workflow
  • Why this matters: about 7,700 fewer lines and ~8 MB of files that agents and code search would otherwise trip over.

What changed

Pile Removed Kept, and why
.agent-review/ Screenshots, fixtures and probes from PRs that already merged (issue-1734, issue-1743, reliability, termination, language, menu bar and Home mockups) The folder's README.md and AGENTS.md, since the PR template and WORKFLOW.md still send new UI evidence there. visuals/meeting-microphone-settings.png, because docs/support/airpods-capture-repair-2026-09-09.md links it.
archive/ The old beta backend worker (Cloudflare worker, SQL, wrangler config) Nothing. Its mentions in AGENT_START.md, CLAUDE.md, docs/agent-onboarding.md, docs/docs.md and docs/repo-layout.md are removed.
docs/assets/app-icon-options/ Icon rounds 1-7, the other round 8 colors, render/generator scripts and preview PNGs round8/H-mono-light.svg and H-mono-dark.svg (the chosen icon, named in AppIcon/ and MenuBarGlyph.swift), and website/ (logos and site mockups). The README is rewritten to say what's there.
scripts/ops/ The local Codex issue runner (agent-todo-runner.rb, its security test, agent-todo-launchagent.sh), build-codex-memory-index.py, nightly-transcripted-archive-miner.sh, and docs/agent-issue-orchestration.md Their test-matrix rules, scripts/README.md entries and the linux-checks.sh ruby suite entry are removed too. The now-empty ruby list uses ${arr[@]+...} so it's safe under set -u on macOS bash 3.2.

Deliberately left alone:

  • config/security/nightly-security-manifest.json still allowlists archive/**. The history secret scan looks back 12 commits, and this commit's deletions are in that window.
  • scripts/dev/agent-context.py uses archive/README.md as a made-up path in its fallback self-test. It doesn't need the file to exist.
  • WORKFLOW.md's front matter and .github/ISSUE_TEMPLATE/agent_task.md still mention the agent todo label. That's out of scope here.

How I checked it

  • scripts/dev/agent-preflight.sh
  • bash scripts/dev/linux-checks.sh: 48 passed, 0 failed (one fewer than main because the deleted ruby suite is gone)
  • python3 scripts/dev/check-source-pins.py --changed-only: every resolved pin holds
  • Grepped Tests/ and Tools/*/Tests for the edited docs and paths. NightlySecurityContractTests reads scripts/README.md and the test matrix, and every string it checks is still there.
  • git grep finds no references left to the deleted scripts or docs

Checks I could not run, and why:

  • The Swift build and tests (no Swift toolchain here). No Swift or build file changed; CI runs them.

Mac or hardware test still needed? No.

Risk Review

🤖 Generated with Claude Code

https://claude.ai/code/session_01Gc3ecXeWwy6DfwkU7PT1wi


Generated by Claude Code

Justin said yes to clearing four piles of unused files. All stay in git
history.

- .agent-review: visuals and probes from merged PRs. The folder and its
  rules stay, since the PR template and WORKFLOW.md still point new UI
  evidence there. meeting-microphone-settings.png stays (a support doc
  links it).
- archive/: the old beta backend worker. Doc mentions removed.
- docs/assets/app-icon-options: rounds 1-7, the other round 8 colors, and
  the generator scripts. The chosen H-mono light/dark SVGs and website/
  stay, since the app icon, menu bar glyph and site mockups reference them.
- scripts/ops: the local Codex issue runner (agent-todo-*), its
  LaunchAgent helper, and the Codex memory index + nightly miner, plus
  docs/agent-issue-orchestration.md. Test-matrix rules, scripts/README
  entries and the linux-checks ruby suite go with them.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gc3ecXeWwy6DfwkU7PT1wi
@r3dbars r3dbars self-assigned this Sep 25, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gc3ecXeWwy6DfwkU7PT1wi
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gc3ecXeWwy6DfwkU7PT1wi
@r3dbars
r3dbars marked this pull request as ready for review September 25, 2026 01:54
@r3dbars
r3dbars merged commit 8964afb into main Sep 25, 2026
8 checks passed
@r3dbars
r3dbars deleted the claude/project-thread-bj8ydh branch September 25, 2026 02:09
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.

2 participants