Skip to content

[codex] Limit foreground API timeout#28

Merged
jverre merged 1 commit into
mainfrom
codex/foreground-api-timeout
Jun 12, 2026
Merged

[codex] Limit foreground API timeout#28
jverre merged 1 commit into
mainfrom
codex/foreground-api-timeout

Conversation

@jverre

@jverre jverre commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Reduce foreground hook Opik API timeout from 30s to 2s so Claude Code does not wait tens of seconds when Opik is unreachable.
  • Add an explicit API constructor for longer background calls and keep the detached /context worker on the 30s timeout.
  • Cover the timeout split with a small unit test.

Validation

  • go test ./...
  • go vet ./...
  • git diff --check

@jverre jverre marked this pull request as ready for review June 12, 2026 12:27
@jverre jverre merged commit fe284cc into main Jun 12, 2026
1 check passed
@jverre jverre deleted the codex/foreground-api-timeout branch June 12, 2026 21:15
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