Repository navigation
feat(conversations): repeat operation_name and add the content filter on items - #191
Draft
brightsparc wants to merge 2 commits into
Draft
brightsparc wants to merge 2 commits into
brightsparc wants to merge 2 commits into
Conversation
… on items `items.list()`, `export_trajectory()`, `export_arrow()` and the export params take `operation_name` as one name or a list (any of which matches) and a `content` list of classes (`user_text`, `assistant_text`, `delegation`, `tool:<name>`, `tool_call:<name>`, `tool_result:<name>`), both sent as repeated query keys. `IntrospectionAttributes.content` types the ingest-stamped `introspection.content` summary. Sync twins regenerated. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Hj7wRZSdedpuVRaKkxsHuF
The export route takes the repeated `operation_name` but no `content`, so `export_trajectory()`, `export_arrow()`, `build_export_params()` and `ConversationExportParams` lose the option; `items.list()` keeps it. Sync twins regenerated. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Hj7wRZSdedpuVRaKkxsHuF
This branch was successfully deployed
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.
Why
GET /v1/conversations/{id}/itemsgains two server-side filters in introspection-org/introspection-cloud#3524 so a chat client reads only the rows it renders: a repeatableoperation_name(any of) andcontent, a closed set of content classes resolved from a plaintext summary stamped at ingest. The export route gains the repeatableoperation_nameonly. This SDK tookoperation_nameas one string and had nocontent. Nothing is removed: this SDK never exposed/turns.Change
items.list()takesoperation_nameasstr | list[str](a list is sent as the repeated key, any of which matches) and acontent: list[str]of classes:user_text,assistant_text,delegation,tool:<name>,tool_call:<name>,tool_result:<name>. Its docstring says how the two combine.export_trajectory(),export_arrow(),build_export_params()andConversationExportParamstakeoperation_nameasstr | list[str]; nocontent, since the export route has none.IntrospectionAttributes.content: IntrospectionContent | Nonetypes theintrospection.content.*summary (version, counts, tool-name arrays; never message content); exported fromschemas.genai_span.scripts/gen_sync.py.The only type change on an existing name is the widening of
operation_namefromstrtostr | list[str];griffe checkagainstv0.29.1reports no breakage.Verification
ruff format,ruff check,ty check,deptryclean;jscpd16.03 % against the 16.3 % threshold.pytest tests/778 passed, coverage 92.4 % against the 80 % floor. New cases: repeatedoperation_nameandcontentkeys on the items list, a singleoperation_nameunchanged, the typed content summary on an item, and the repeatedoperation_namewith nocontenton the trajectory export.griffe check introspection_sdk --search . --against v0.29.1 -e scripts/griffe_reexports.py: no breakage.Review round
Codex: the export carried a
contentthe server ignores. Removed in the last commit.Related
🤖 Generated with Claude Code
https://claude.ai/code/session_01Hj7wRZSdedpuVRaKkxsHuF