docs: final polish — roadmap, typo, full characters gallery (Cycle 22)#26
Merged
Conversation
…ters gallery (Cycle 22) Close out the docs cleanup arc (last three audit items). 1. roadmap.md — fix the drift AND its root cause. Cycle 19 was stuck "In progress (this PR)" though merged; Cycles 20/21 sat under "Planned" though shipped. Add 19 (#23), 20 (#24), 21 (#25), 22 to the Shipped table with correct refs. KILL the self-staling convention: no more "In progress (this PR)" row — the table is factual merged history (status never goes stale), and forward-looking work lives in a separate "Ideas / not yet scheduled" section that references no in-flight PR. Convention recorded in the cycle doc + a note in the roadmap. 2. legal-and-assets.md — fix the dangling "TMAFE (The Microsoft Agent Fan ... community)" sentence. No invented acronym expansion; reworded to the accurate "TMAFE is the de-facto community archive." 3. characters.md — expand the gallery to the real original four. Captured Merlin, Peedy, Robby with the existing Playwright tooling (live MASH, no voice needed), committed portrait + animation GIF for each (rendered PNG/GIF only — no .acs, ADR-0028). Each image viewed to confirm it rendered (wizard, green parrot, robot). Factual detail only (kind + animation count, counts verified via parseAcs: Genie 76 / Merlin 73 / Peedy 85 / Robby 68). "Any .acs works" framing intact. Docs + image assets only; CI green; verified by code-reviewer (roadmap matches merged history, convention gone, no invented lore, no committed .acs, images render, links resolve). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
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.
Closes out the docs cleanup arc — the last three items from the final audit.
Cycle spec:
docs/cycles/cycle-22-final-polish.md.1. Roadmap — fix the drift and its root cause
2. Typo
docs/legal-and-assets.md— fixed the danglingTMAFE (The Microsoft Agent Fan ... community)sentence. No invented acronym expansion — reworded to the accurate "TMAFE is the de-facto community archive."3. Characters gallery — show the real cast
Captured Merlin, Peedy, Robby with the existing capture tooling (live MASH dev server; portraits/animation need no voice container), and committed a portrait + animation GIF for each. characters.md now shows the original four (Genie, Merlin, Peedy, Robby), each with factual detail only (kind + animation count) and the "any
.acsworks" framing intact..acscommitted (ADR-0028 / ADR-0006).parseAcs): Genie 76 / Merlin 73 / Peedy 85 / Robby 68.Verification
pnpm -r typecheck && pnpm -r test && pnpm lint && pnpm format— green (docs + assets only)..acsstaged; all 8 character image refs resolve; every link resolves; no-app.pngleftovers.code-reviewercleared all six obligations: roadmap matchesgit log --merges, convention gone, no invented lore, IP hygiene, images non-blank, scope docs-only.This should bring the docs to fully DONE (the audit's remaining items were exactly these). Base
main, not merged.🤖 Generated with Claude Code