Skip to content

fix: force Codex inference over HTTP - #231

Merged
robbycochran merged 1 commit into
mainfrom
fix/codex-http-inference
Sep 19, 2026
Merged

robbycochran merged 1 commit into
mainfrom
fix/codex-http-inference

Conversation

@robbycochran

Copy link
Copy Markdown
Collaborator

The OpenShell inference route supports HTTP but rejects the Codex Responses WebSocket transport. Configure a dedicated Codex model provider with supports_websockets = false, while keeping the existing sandbox policy and provider bootstrap.

@robbycochran
robbycochran merged commit fb457d7 into main Sep 19, 2026
10 checks passed
@robbycochran
robbycochran deleted the fix/codex-http-inference branch September 19, 2026 02:34
@stackrox-openshell-workflow stackrox-openshell-workflow Bot added the stackrox-ai-review Opt in to StackRox AI review label Sep 19, 2026
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: stackrox/harness-openshell/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e1ac0e21-4c6a-457c-9a32-10695abc2758

📥 Commits

Reviewing files that changed from the base of the PR and between fe72bf1 and 78c3ce4.

📒 Files selected for processing (2)
  • tasks/github-pr-reviewer/workflow/codex-harness.yaml
  • test/pr_review_test.go
 ________________________________________________________________________________________________________________________
< Don't assume it - prove it. Prove your assumptions in the actual environment - with real data and boundary conditions. >
 ------------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stackrox-ai-review Opt in to StackRox AI review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant