Repository navigation
Conversation
|
Claude finished @alukach's task in 8s —— View job ✅ No blocking issues — safe to merge. Docs-only change to Simplify (ponytail)
💰 Estimated review cost: $0.10 · 0m07s · 4 turns |
alukach
force-pushed
the
docs/pr-description-demos
branch
from
October 17, 2026 06:36
71ecb8c to
7120c02
Compare
AGENTS.md (which CLAUDE.md includes) now says what a PR description holds and in what order, how to record and embed demos with demo/, and how the stub server works and is extended. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
asciinema.org's SVG previews show as broken images on GitHub, whose image proxy can't fetch them, so demos are GIFs on the demo-assets branch linked to the player, as #59's record.py now prints. Membership lists are pages on source.coop since its #673, like every other listing. The stub's mechanics are left to demo/README.md; this keeps the rules for extending it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
alukach
force-pushed
the
docs/pr-description-demos
branch
from
October 17, 2026 06:55
7120c02 to
28b5fc6
Compare
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.
Adds a "Pull request descriptions" section to
AGENTS.md.CLAUDE.mdis@AGENTS.md, so Claude reads it there, and so do other agents and contributors. It writes down the style used in #57:demo-assetsbranch, linked to its player, where reviewers can pause and rewind. That's the markdownrecord.py --upload --gif-base …prints. asciinema.org's own SVG and PNG previews show as broken images on GitHub, whose image proxy can't fetch them. Uploads are public to anyone with the link, so record only stub data, and runasciinema authso recordings stay on an account. Treat recording as a review pass: feat: manage members and invitations (member, invitation) #57's demos caught two awkward messages.demo/stub.py): how it runs is indemo/README.md, so this section keeps only the rules for extending it.Stacked on #59 (
chore/demo-recorder), whosedemo/tooling this section describes. Merge #59 first, then retarget this PR tomain.Note for #57: #57 calls
/accounts/{id}/members,/products/{account}/{product}/membersand/memberships/{id}/accept|reject|revoke, which source.coop#679 replaced with one/membershipsresource. Its own demos used a one-off stub of those old endpoints on itsdemos/pr-57branch.demo/stub.pyalready serves/memberships: listing them as pages, inviting, and changing a role or state. So once #57 moves there, porting its demos todemo/should need only fixtures and whatever else its commands call.Testing
Docs only.
cargo teststill passes, including the README and help checks.🤖 Generated with Claude Code