Repository navigation
docs(examples): show startup content capture with configure - #172
Merged
Merged
Conversation
Add a standalone example that calls configure once before the first AI invocation. Handlers are built without a capture_content argument.
Contributor
Author
|
bugbot run |
The capture example imported the registry at module load, so provider clients were constructed before .env was read.
Contributor
Author
|
bugbot run |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 87c3d67. Configure here.
jeffdupont
approved these changes
Oct 15, 2026
jeffdupont
left a comment
Contributor
There was a problem hiding this comment.
configure(options=SDKOptions(capture_content=...)) matches the lazy resolution in types.py (effective_capture_content), so calling it after handler registration is correct. Nit, non-blocking: this is the only example with its own __main__ instead of an entry in main.py.
Drop the example's own entrypoint and register it with the other examples.
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 standalone example that turns conversation content on for the process with
configure(options=SDKOptions(capture_content=True))beforeinit_clientand the first AI call. The registered handlers are built with nocapture_contentargument.It is not wired into
main.py.Note
Overview
Adds a
capture-contentexample that turns on process-wide conversation content capture withconfigure(options=SDKOptions(capture_content=True))before the firstconfig(...).invoke, using handlers from the global registry that do not setcapture_contentthemselves (per-handler factory flags would override this default).main.pyregisters the example undercapture-contentand documents the CLI usage alongside the other examples.Reviewed by Cursor Bugbot for commit f5fa6fa. Bugbot is set up for automated code reviews on this repo. Configure here.