Skip to content

test(cli): include Opus 5 in provider expectations#2425

Merged
boudra merged 1 commit into
getpaseo:mainfrom
dwyanewang:fix/cli-claude-opus-5-expectations
Jul 25, 2026
Merged

test(cli): include Opus 5 in provider expectations#2425
boudra merged 1 commit into
getpaseo:mainfrom
dwyanewang:fix/cli-claude-opus-5-expectations

Conversation

@dwyanewang

Copy link
Copy Markdown
Contributor

Linked issue

Closes #2424.

Type of change

  • Bug fix
  • New feature (with prior issue + design alignment)
  • Refactor / code improvement
  • Docs

What does this PR do

The Claude model manifest now contains 12 entries after Opus 5 was added, but the CLI provider integration test still expected 11. That made cli-tests (shard 2/3) fail deterministically before it could verify the remaining provider commands.

This updates the CLI expectations to:

  • include claude-opus-5 as the latest release
  • describe claude-opus-4-8 as the previous release

The change is limited to the stale provider-test expectations.

How did you verify it

Before the change, I built the server stack and ran packages/cli/tests/15-provider.test.ts. The Claude assertion failed with:

AssertionError: claude output should match the current catalog size
12 !== 11

After the change, I ran the same provider test with the CI-pinned Codex CLI (@openai/codex@0.105.0), OpenCode, and an isolated CODEX_HOME. All 11 provider command tests passed, including the Claude catalog assertion.

I also ran:

npm run build:server
npm run format
npm run typecheck
npm run lint

Lint completed with 0 warnings and 0 errors.

Checklist

  • One focused change. Unrelated cleanups split out.
  • npm run typecheck passes
  • npm run lint passes
  • npm run format ran (Biome)
  • UI changes include screenshots or video for every affected platform (not applicable; test-only change)
  • Tests added or updated where it made sense

@boudra
boudra merged commit 51ab86b into getpaseo:main Jul 25, 2026
22 checks passed
Paramstr added a commit to Paramstr/paseo that referenced this pull request Jul 25, 2026
Brings in the one commit on upstream/main since the v0.2.1 merge
(test(cli): include Opus 5 in provider expectations, getpaseo#2425). Test-only,
matches the Opus 5 manifest already in the fork. No conflicts.

Verified: typecheck 0, lint 0. Not built per request; CLI test itself
runs in CI.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

bug: CLI provider test is stale after Opus 5 catalog update

2 participants