Skip to content

feat: map harness session id to Codex threads#21

Merged
mikehostetler merged 2 commits into
mainfrom
feat/harness-session-id-bridge
May 8, 2026
Merged

feat: map harness session id to Codex threads#21
mikehostetler merged 2 commits into
mainfrom
feat/harness-session-id-bridge

Conversation

@mikehostetler

Copy link
Copy Markdown
Contributor

Summary

  • default Codex thread_id from RunRequest.session_id
  • preserve existing Codex metadata/runtime override precedence
  • update jido_harness lock to include RunRequest.session_id
  • add regression coverage for harness session_id and metadata override behavior

Refs #19.

Verification

  • mix test test/jido/codex/adapter_test.exs test/jido/codex/options_test.exs
  • mix test
  • mix format --check-formatted
  • mix compile --warnings-as-errors
  • git diff --check

Note: the stdin race tracked by #19 still depends on upstream external_runtime_transport; this PR lands the harness-layer session bridge while keeping the upstream blocker visible.

@mikehostetler mikehostetler merged commit cddcfc2 into main May 8, 2026
17 checks passed
@mikehostetler mikehostetler deleted the feat/harness-session-id-bridge branch May 8, 2026 12:45
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