docs: close the seventh audit's verified findings - #835
Merged
Conversation
- export Mermaid diagram source into llms-full.txt as fenced blocks (51 diagrams across 47 pages were being stripped with the JSX) - rewrite sdk/simulate around the SDK's single dual-mode run_test: cloud args and local LiveKit args dispatch differently and do not combine; examples no longer mix them - align guardrail status codes with the gateway: 403 with a guardrail_error/content_blocked body on block, 200 plus the x-agentcc-guardrail-triggered header on warn and log; the 446/246 convention never shipped and is gone from all five Protect pages - document that an exact cache hit skips request-side guardrail screening while response-side checks still run - stop presenting Experiment as a product; signals flow into Evaluate Signed-off-by: khushalsonawat <khushalsonawat@gmail.com>
hadarishav
approved these changes
Aug 25, 2026
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.
What
Closes the five actionable findings from the seventh docs audit, every one verified against product source before a page changed.
llms-full.txtas fenced blocks (51 diagrams across 47 pages were stripped with the JSX)sdk/simulaterewritten around the SDK's single dual-moderun_test(): cloud args and local LiveKit args dispatch differently and never combine; the examples had been mixing them, which silently dropped parametersguardrail_error/content_blockedbody; warn and log return 200 plus thex-agentcc-guardrail-triggeredheader. The 446/246 convention never shippedWhy
An independent 742-page audit surfaced five claims that break working code or misrepresent the product. Verification confirmed three fully, narrowed one (cache bypass is input-side only), and disproved parts of another (446/246 exist only in dead code and an unwired client SDK).
What cases were covered
run_testverified against the releasedagent-simulate0.1.3 wheel and its dispatch logic, not just repo source; the reference page checked out fully and was left untouchedHow
Four source-verification agents (simulate SDK, gateway pipeline, status codes, product framing) plus an extractor change, each landing the smallest edit that resolves its finding.
Recording
Not applicable: text and export changes only.
🤖 Generated with Claude Code