Repository navigation
Conversation
Codex 0.156.1 rejects gpt-6.1-sol for ChatGPT subscriptions, so local subscription runs with that model fail at the threat model step.
|
@dpanta94 is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
🤖 auto-maintain reviewAutomated, advisory triage for
Review panel: 🟡 medium highest severity deepsec maintainer code review: 🟢 low
General code review: 🟢 low
Scanner and matcher quality: 🟡 medium
Sandbox and credential boundaries: 🟢 low
Durability and distributed state: 🟢 low
Posted by auto-maintain. This automated code review is advisory; a human maintainer makes the call. |
What changed
Bumps
@openai/codexand@openai/codex-sdkfrom^0.156.1to^0.164.0inpackages/deepsecandpackages/processor, with the matchingpnpm-lock.yamlupdate.Why
With a ChatGPT subscription (
--model-auth local --agent codex), Codex 0.156.1 can't usegpt-6.1-sol.deepsec initstops at the threat model step withCodex Exec exited with code 1:and no further detail. Running the workspace's own Codex binary with the same login shows the API response:A caret range on a
0.xversion only matches patch releases, so^0.156.1resolves to0.156.xand new workspaces can't pick up a newer Codex until this range changes.Verification
pnpm testpasses (2417 passed, 1 skipped)pnpm lintpassespnpm knippassespnpm validateexits 0, which also coverspnpm -r build,pnpm bundle, andpnpm typecheck:deepsec.I also ran
pnpm deepsec init <workspace> <app> --model-auth local --agent codex --model gpt-6.1-sol --no-tuiagainst a one-file project, before and after the change:main(Codex 0.156.1), it fails at Threat model with the error above.NO_SCAN_CANDIDATES, which is expected for a project with nothing to scan.Notes for reviewer
This PR only changes the Codex packages and leaves
@anthropic-ai/claude-agent-sdkas it is.Because of the caret behavior described above, each model that Codex gates by version needs a deepsec release to reach users. A wider range such as
>=0.156.1 <1would avoid that, but0.xminor releases can change the SDK API, so this PR keeps the existing caret.Written with AI assistance (Claude Code) and reviewed by me.