Skip to content

fix(server): load Jude OpenCode subscription models from the right config - #6

Closed
digitalbase wants to merge 1 commit into
mainfrom
codex/fix-opencode-model-discovery
Closed

fix(server): load Jude OpenCode subscription models from the right config#6
digitalbase wants to merge 1 commit into
mainfrom
codex/fix-opencode-model-discovery

Conversation

@digitalbase

Copy link
Copy Markdown

OpenCode in Jude could miss models supplied by Anthropic and Codex subscriptions because its inventory commands inherited the backend process directory instead of using the configured server working directory. That allowed OpenCode to resolve a different project config and plugin set.

The inventory path now carries the server cwd through to both opencode models --verbose and opencode agent list, with regression coverage that verifies model discovery stays anchored to that directory.

Tests: vp test run apps/server/src/provider/Layers/OpenCodeProvider.test.ts apps/server/src/provider/opencodeRuntime.cliParsers.test.ts; vp run --filter t3 typecheck

Model: GPT-5
Harness: Codex desktop

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Aug 14, 2026
@github-actions

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.3 KiB 11.3 KiB −25 B (−0.2%) 15.1 KiB
Codex Thread snapshot wire 5.5 KiB 5.5 KiB +2 B (+0.0%) 7.3 KiB
Codex Live turn WebSocket wire 5.9 KiB 5.9 KiB −27 B (−0.4%) 7.8 KiB
Codex Live turn WebSocket decoded 49.7 KiB 49.7 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 11.3 KiB 11.3 KiB +6 B (+0.1%) 15.1 KiB
Claude Thread snapshot wire 5.5 KiB 5.5 KiB +7 B (+0.1%) 7.3 KiB
Claude Live turn WebSocket wire 5.9 KiB 5.9 KiB −1 B (−0.0%) 7.8 KiB
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: 80f0be1 · PR result: 7db0d7c · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.6 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@digitalbase

Copy link
Copy Markdown
Author

Closing as superseded by #5, which already landed the same change on main.

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

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant