Skip to content

feat(operator): thread ids on requests, run ids on results, and the audit run reads - #285

Merged
jfrench9 merged 1 commit into
mainfrom
feature/operator-run-history
Oct 23, 2026
Merged

jfrench9 merged 1 commit into
mainfrom
feature/operator-run-history

Conversation

@jfrench9

Copy link
Copy Markdown
Member

Summary

Companion to RoboFinSystems/robosystems#1936 and the TypeScript twin RoboFinSystems/robosystems-typescript-client#301. Additive, so a client minor; nothing the integration template imports moves.

  • `OperatorQueryRequest.thread_id` rides the body: it groups a run with the earlier runs of one conversation, and with no `history` given the server seeds the run from the thread's own record. Sent only when set.
  • `OperatorResult.operation_id` names the run on every path (a sync 200, the followed stream, the status poll), so a caller can read its record back.
  • Regenerated against the local API: `list_operator_threads`, `list_operator_runs` and `get_operator_run` under the graph audit, with their models.

The new request field waits on the robosystems release carrying #1936; against an older server it is ignored.

Verified

Wire-shape tests cover the body (thread field present only when set) and the run id on sync and followed results. Lint, typecheck and the full suite green.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AZDcFWJRoqZQpZ3ZguTVzu

…udit run reads

The operator facade sends an optional thread_id and every result carries
its operation_id, so a caller can read the run back. Regenerated against
the API's operator run endpoints: list_operator_threads, list_operator_runs
and get_operator_run under the graph audit.

Claude-Session: https://claude.ai/code/session_01AZDcFWJRoqZQpZ3ZguTVzu
@jfrench9
jfrench9 merged commit 8865297 into main Oct 23, 2026
6 checks passed
@jfrench9
jfrench9 deleted the feature/operator-run-history branch October 23, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant