Skip to content

docs: point README samples at the demo GIFs - #44

Merged
Kiran01bm merged 1 commit into
mainfrom
kiran01bm/readme-demo-slim
Aug 18, 2026
Merged

docs: point README samples at the demo GIFs#44
Kiran01bm merged 1 commit into
mainfrom
kiran01bm/readme-demo-slim

Conversation

@Kiran01bm

@Kiran01bm Kiran01bm commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Trims the README's duplicated console samples in favor of the animated demos and inlines all four GIFs in the demos page.

Why

After the demos landed (#42), the improve, refuse, and lint sections carried both a GIF and a near-identical verbatim console block — two renderings of the same output to keep in sync by hand. The demos page described the recordings in a table without showing them.

What

  • README: improve/refuse/lint keep their exit-code/gating prose and point at the GIF plus docs/cli-output-examples.md; the diff sample stays as the one copyable text rendering, with cli-output-examples.md's back-pointer updated to match. The lint section's exit-code claim now matches the code (error-severity findings exit non-zero; warnings alone pass).
  • docs/demos/README.md: the tape table becomes per-tape sections with each GIF embedded.
  • Tapes: all four GIFs re-rendered.
Before                                  After
┌───────────────────────────────┐      ┌───────────────────────────────┐
│ README                        │      │ README                        │
│  hero GIF (improve)           │      │  hero GIF (improve)           │
│  diff:    prose + console     │      │  diff:    prose + console     │
│  improve: prose + console     │      │  improve: prose ──┐           │
│  refuse:  prose + console     │      │  refuse:  prose ──┼─ links ─┐ │
│  lint:    prose + console     │      │  lint:    prose ──┘         │ │
└───────────────────────────────┘      └─────────────────────────────┼─┘
                                                                     │
┌───────────────────────────────┐      ┌─────────────────────────────▼─┐
│ docs/demos/README.md          │      │ docs/demos/README.md          │
│  table describing the tapes   │      │  per-tape sections, each GIF  │
│  (GIFs not shown)             │      │  embedded inline              │
└───────────────────────────────┘      └───────────────────────────────┘

The improve, refuse, and lint verbatim console samples duplicated what
the animated demos record: each section keeps its exit-code/gating prose
and points at the GIF and the JSON contract doc, the diff sample stays
as the one copyable text rendering, and docs/demos/README.md inlines all
four GIFs in per-tape sections. Also reverts the neutral-prompt review
nit as a deliberate authorial choice; GIFs re-rendered to match.
@Kiran01bm Kiran01bm changed the title docs: point README samples at the demo GIFs; keep the author's prompt docs: point README samples at the demo GIFs Aug 18, 2026
@Kiran01bm
Kiran01bm marked this pull request as ready for review August 18, 2026 12:01
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Kiran01bm
Kiran01bm merged commit 5044628 into main Aug 18, 2026
8 checks passed
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