docs: show what the demos and the quickstart actually print - #25
Merged
Conversation
Both pages describe outcomes in prose and never show the output. For pages whose whole pitch is "don't take the spec on trust, run it", the output is the argument. I ran the published quickstart end to end against cmcp-runtime 0.4.0 and demo 6 against weight-custody-manifest 0.25.0. Both do exactly what the pages say, so nothing here corrects a claim about behaviour. What changes is that a reader can now see it before they install anything. Quickstart: - Step 4 shows the response body the runtime really returns, replacing a one-line prose summary of it. - That body carries a call_id, and the same id lands in the audit chain, which is the thread from the block in step 4 to the signed record in step 5. The page never mentioned it, and it is the point. - Step 5 shows the CRYPTO-001 advisory the CLI prints before the checks. It is not a failure, but it is the first thing on screen and the page did not prepare anyone for it. - Says which version the page was last verified against, and that a step not behaving as written is a bug worth reporting. Demos: - The governance section said "these five govern the tool boundary" above six cards. Demo 10 is in that section and governs model calls, not the tool boundary. - The card times sum to thirteen minutes, not twelve. Fixed in the hero and in the meta, OG and Twitter descriptions. - Dropped export CMCP_BEARER_TOKEN from the quick start: demo.py sets it, as the repo README says. Added --no-pause, without which the runner waits for a keypress before every demo. - Added demo 6's verbatim output. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013EQx4N5BzTQbY8kvXUsdkY
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.
Both pages describe outcomes in prose and never show the output. For pages whose whole pitch is "don't take the spec on trust. Run it.", the output is the argument.
I ran them
cmcp-runtime 0.4.0: config validates, the runtime starts, the forbidden call returns403 POLICY_DENY, the session closes into a signed claim, andcmcp verifyreportsFAIL (partially_verified)and exits 1.weight-custody-manifest 0.25.0: all four bullets on the card are accurate.Nothing here corrects a claim about behaviour. Both pages were already right. What changes is that a reader can see it before installing anything.
Quickstart
call_id, and the same id lands in the audit chain. That is the thread from the block in step 4 to the signed record in step 5, and the page never mentioned it. It is arguably the point of the whole exercise: a refusal nobody can check afterwards is just a log line.CRYPTO-001advisory the CLI prints before the checks table. It is not a failure, but it is the first thing on screen and the page did not prepare anyone for it.Demos
export CMCP_BEARER_TOKEN=demo-tokendemo.pysets it, as the repo README says--no-pauseNoted, not fixed here
demos/demo.pysaysrun only demo 2 (1 through 9)in its--helpdocstring. There are ten.cmcp validate-configcrashes on a Windows console ('charmap' codec can't encode '✓') unlessPYTHONUTF8=1is set. The page scopes itself to macOS and Linux, so this is a cmcp issue rather than a page issue.🤖 Generated with Claude Code
https://claude.ai/code/session_013EQx4N5BzTQbY8kvXUsdkY