Docs/conventions hygiene batch: CLAUDE.md rules + proposal doc corrections

[WilfordGrimley]

Summary

Consolidated docs/conventions hygiene batch — 7 items, each a decided convention or correction from today's work needing its permanent home.

1. CLAUDE.md conventions:
   • Report-relay: replies must now carry the full GitHub blob URL to the pushed report, not just branch+path+summary.
   • New merge-duty rule: never delete a branch in the same action as merging its PR; precondition gh pr list --base <branch> must return empty before deleting. Cites how PR Proposal G, PR3: opaque-blob saved-decks API #88 was lost (stacked-PR base-deletion trap).
   • New rule: search for an existing recovery before rebuilding a lost/auto-closed PR. Cites Proposal G, PR3: opaque-blob saved-decks API #88 being rebuilt in parallel by two sessions, with Proposal G, PR3: opaque-blob saved-decks API (recreated after #88's base-deletion auto-close) #95 duplicating Proposal G, PR3: opaque-blob saved-decks API #94's already-shipped recovery.
2. Proprietary-name genericization: "Proxxied" (proposal-b, proposal-c, proposal-g) and "Steam Deck" (proposal-b) design-reference mentions genericized. A broad follow-up grep across all of docs/ (including an independent research-agent sweep) found no further hits. "Moxfield" kept as-is (named factual import source); historical docs/reports/ files untouched.
3. proposal-c-context-menu-restyle.md: Part (b) (solid-color utilitarian restyle direction) marked SUPERSEDED by Proposal H, which absorbs the restyle direction. Kept as a historical record, not an active plan.
4. proposal-b-bleed-normalization.md: decision 4's persistence note corrected. It originally (written before PR-2 shipped) said persistence was "project state (projectSlice), not session-only," which reads as SavedDeck-style project/cloud persistence. What actually shipped is identifier-keyed localStorage (deliberately device-local, mirroring favoritesSlice), consistent with projectSlice holding a live Redux copy but localStorage being what actually survives reload — and consistent with proposal-g's own §5, which already correctly describes this as deliberately kept out of SavedDeck state.
5. proposal-h-unified-display-page.md §1: corrected alex-taxiera/proxy-print's license label from MIT to AGPL-3.0 (verified directly against its actual GitHub license metadata) — confirmed correct by the owner. acoreyj/proxies-at-home's own MIT label is correct and unchanged.
6. docs/lessons.md — two new entries, both checked-for-and-absent beforehand (PR /whatsthat: fix oversized starburst + swap accent to an AA-verified navy against the orange bg #91's own diff only touched cardPanel.tsx/whatsthat.tsx, no docs):
   • "A value carried 'verbatim' out of its old context can silently stop meaning what it meant" — generalizes PR /whatsthat: fix oversized starburst + swap accent to an AA-verified navy against the orange bg #91's starburst 140%-of-column width (tuned for a 33% column, carried unchanged into a 58% column) together with the existing PR Fix Level 1's missing Scryfall reference image (regression, PR #49) #78 "extracts X verbatim" entry, as two instances of the same class.
   • "Bootswatch Superhero hardcodes some component colors as literal properties, not CSS custom-property references" — PR /whatsthat: fix oversized starburst + swap accent to an AA-verified navy against the orange bg #91's .btn-primary background-color finding: verify computed styles on a live element, not just that a --bs-* custom property resolved correctly.
7. Standard close: docs_lint.py and pinned prettier (v2.7.1) run clean across every file this batch touched. Wiki: no separate wiki edit needed — this batch is docs/CLAUDE.md-only (no user- or admin-facing product change), and the automated wiki republish on merge covers today's process-evolution documentation itself.

Test plan

• python3 .github/scripts/docs_lint.py — clean.
• npx prettier@2.7.1 --check (pinned to the repo's .pre-commit-config.yaml version) on every touched file — clean.
• Manual + independent research-agent grep sweep of docs/ for other proprietary product/company names used as design comparisons — none found beyond the two already fixed.
• Cross-checked proposal-b's corrected persistence language against real shipped code (frontend/src/store/slices/projectSlice.ts, frontend/src/common/cookies.ts, frontend/src/store/listenerMiddleware.ts, frontend/src/common/constants.ts) and against proposal-g's own §5 description of the same mechanism.
• Verified PR /whatsthat: fix oversized starburst + swap accent to an AA-verified navy against the orange bg #91 and PR Fix Level 1's missing Scryfall reference image (regression, PR #49) #78 (cited in the new lessons.md entries) via pull_request_read — both real, merged, content matches what's cited; confirmed PR /whatsthat: fix oversized starburst + swap accent to an AA-verified navy against the orange bg #91 never touched docs/lessons.md itself, so the new entries aren't duplicates.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AHfxN9bbWAWHs8rfzVtBWt