Repository navigation
[AAASM-6439] 📝 (python-sdk-docs): Resync quick-start snippets with the examples' enforcement_mode - #388
Conversation
…ples' enforcement_mode The examples now pass enforcement_mode="observe" in each offline quick start because the default posture fails closed when registration cannot reach a gateway. The vendored snippets and the generated tabs still showed the old call, so copying them verbatim into an offline script failed closed. The local import headers and the completed with-bodies are kept. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
The snippets pass enforcement_mode="observe" for offline demos, which makes the SDK's local checks fail open. Copying one into a service without a gateway would silently run ungoverned, so say so next to the snippets and name what to change. Found in independent review of the snippet resync. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
|
OWNER-MERGE-REVIEW head=a996c3df130e37e1cd48eb3c04aab6973f28b1b0 verdict=PASS blocking=0 reviewer=claude-independent-agent |
…tart observe note The paragraph asserted fail-open and default-enforce behaviour with no ClaimBinding. Keep only the configuration instruction and allow-list the two remaining sentences with written justifications. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
…inst a local policy The snippets still pass a gateway URL, so 'without a gateway' was loose. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
|
OWNER-MERGE-REVIEW head=69be38db6a42a51418aa40493b6b3dd314f85b04 verdict=PASS blocking=0 reviewer=claude-independent-agent |
Brings in the FUNDING.yml-only change (HORO-2531) so the branch is current. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
|
OWNER-MERGE-REVIEW head=0759ac390b14cdd94a479d54715a24e9eb83a47f verdict=PASS blocking=0 reviewer=claude-independent-agent |
Brings in the SDK org identity change (AAASM-6444) so the branch is current. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4
|
|
OWNER-MERGE-REVIEW head=ff5bcad3cf326c909b30a81b738421520db9bffa verdict=PASS blocking=0 reviewer=claude-independent-agent |
|
OWNER-ADMIN-MERGE (same-identity REVIEW_REQUIRED deadlock exception, engineering-workflow exception 2) for head ff5bcad. Verified identity: Chisanan232 (author; org AI-agent-assembly role=admin, checked just now). Only remaining blocker: GitHub required approving review, which the author cannot supply for their own PR. At this head (main with #390 merged in): independent adversarial review clean (no Critical/High; delta reviews clean; quick-start claim-binding test passes, unit suite 1384 passed), OWNER-MERGE-REVIEW PASS comment posted, 31 checks green, mergeable, 0 unresolved threads, 0 REQUEST_CHANGES. Merge by merge commit pinned to this head, on the owner explicit instruction; host Allow once / Deny prompt applies. |
What changes for a reader
Every per-framework quick start now passes
enforcement_mode="observe", as the runnable examples already do. Without it the defaultenforceposture fails closed when registration cannot reach a gateway, so the offline quick start the docs show did not run verbatim. Found by the J08 candidate harness (16 of 16 snippets differed from the examples' quick-start region).Scope
quickstart_snippets/*.py: only theenforcement_modeargument and its comment are added (+6 lines each, 96 insertions, 0 deletions). The local import headers and completed with-bodies stay, because four upstream regions end mid-withand are not valid Python on their own.docs/quick-start.md: regenerated byscripts/generate_quickstart_tabs.py;--checkreports it up to date.Evidence
Not covered
The four upstream examples' regions are syntactically incomplete (google-adk, microsoft-agent-framework, pydantic-ai, smolagents); that is an examples-repo defect and is not changed here.
🤖 Generated with Claude Code
https://claude.ai/code/session_01TLpw696KjUDUg4QjH83En4