fix(docs): correct hallucinations and wrong values in tutorials#334
Merged
Conversation
Fixes found by systematic cross-check of every tutorial against the actual source code and config schema. Changes per file: quickstart.md - cmcp-gateway → cmcp-runtime (package name) verifying-a-trace-claim.md - cmcp-gateway → cmcp-runtime; fix cmcp_verify attribution comment cedar-policy-walkthrough.md - cmcp-gateway → cmcp-runtime - cedar-policy-analysis (nonexistent pip package) → cedar-policy-cli (real Rust crate: cargo install cedar-policy-cli) - Add cMCP:: namespace to all Action and Resource references in Cedar policy snippets (bare Action::"call_tool" is rejected by the schema) response-inspection.md - cmcp-gateway → cmcp-runtime - Remove fabricated inspection_config_path config key (not in _KNOWN_TOP_KEYS; would cause ConfigError at startup) - Fix all pattern names: underscore-separated → hyphen-separated to match patterns_v1.json; remove conv_reset (does not exist) tls-pinning.md - cmcp-gateway → cmcp-runtime - Fix sentinel behavior: runtime warns and falls back to CA verification (does not fail closed as tutorial claimed) - Fix catalog hash computation: use json.dumps canonical form not raw file bytes (matches catalog/loader.py _catalog_hash()) tee-attestation.md - cmcp-gateway → cmcp-runtime - Remove CMCP_AMD_SEV_SNP_REPORT_PATH export (env var does not exist; device path is hardcoded in sev_snp.py) - Fix measurement prefix sha256: → sha384: (SEV-SNP uses SHA-384) in both the sample claim output and expected_measurement config multi-tenant-config.md - cmcp-gateway → cmcp-runtime Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Both tenant A and tenant B policy blocks used Action::"call_tool" instead of the required cMCP::Action::"call_tool" namespace. Co-Authored-By: Claude Sonnet 4.6 <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
1 participant
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.
Summary
Systematic audit of all 7 tutorial files against actual source code found 9 categories of issues. All fixed.
Fixes
pip install cmcp-gateway→pip install cmcp-runtime(wrong PyPI package name)cedar-policy-walkthrough.mdcedar-policy-analysis(nonexistent pip package) →cedar-policy-cli(Rust crate via cargo); addedcMCP::namespace to all CedarActionandResourcereferencesresponse-inspection.mdinspection_config_pathconfig key (not in_KNOWN_TOP_KEYS, causesConfigError); fixed all pattern names from underscore to hyphen-separated to matchpatterns_v1.json; removedconv_reset(does not exist)tls-pinning.mdcatalog/loader.py _catalog_hash(), not raw file bytestee-attestation.mdCMCP_AMD_SEV_SNP_REPORT_PATHexport (env var does not exist; path hardcoded insev_snp.py); fixed measurement prefixsha256:→sha384:in sample output andexpected_measurementconfigTest plan
pip install cmcp-runtimeresolves correctlycedar validatewith thecMCP::namespacepatterns_v1.json🤖 Generated with Claude Code