[backryun]

Summary

Extracted from #6958 as the Devin CLI only change set.

• Move the supported Devin model set into a reusable provider catalog.
• Refresh Devin CLI models and remove unsupported BYOK variants.
• Keep the CLI executor and registry wired to the curated model IDs.
• Add focused provider and executor regression coverage.

Why

The public Devin CLI catalog had drifted from the models accepted by the native CLI flow. The shared catalog also provides one canonical source for the separately submitted Devin Desktop provider.

Impact

This changes Devin CLI catalog behavior only. The shared catalog file is intentionally reusable by the independent Devin Desktop split PR.

Validation

• npm run typecheck:core
• 3 focused catalog regression tests plus existing Devin CLI executor coverage
• git diff --check

Live provider verification still requires VPS credentials.

[gemini-code-assist]

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

[gemini-code-assist]

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

[diegosouzapw]

Thanks for splitting this out of #6958 — the catalog refactor itself is clean and I confirmed the shared devin/catalog.ts file matches byte-for-byte with the sibling #8228, so the "canonical source for both PRs" claim holds.

One thing that needs fixing before merge: the PR description's Validation section says "Add focused provider and executor regression coverage" / "4 focused Devin CLI tests", but the diff contains no test files at all (gh pr diff --name-only shows only the two registry files + the executor). I also checked #8228's new devin-providers.test.ts and it doesn't cover this catalog's specific model ids either (it only asserts devin-desktop's models are non-empty and BYOK-free). Per this repo's bug/change policy, production catalog code needs at least one regression test (e.g. asserting DEVIN_MODEL_CATALOG has unique ids and contains the intended new/removed model ids) — can you add that, or should we add it in the pre-merge pass?

Small nit you can take or leave: the devin-cli.ts doc comment above the catalog import still cites the old dotted example ids ("swe-1.6-fast", "claude-sonnet-4.6") even though the new catalog uses dashed ids — only the gpt-5-5-high example got updated.

Everything else checks out: typecheck:core is clean, check:provider-consistency passes (0 exceptions), and the existing executor/provider-registry/golden tests all pass unchanged against this diff.

[backryun]

Good catch. I added tests/unit/devin-cli-catalog.test.ts with guards for duplicate and BYOK ids, the refreshed ids, and the retired dotted/review ids. I fixed the examples in devin-cli.ts too, and amended the existing commit (e33ff88).

The core typecheck, the focused Devin tests (44/44), and targeted ESLint all pass.

[diegosouzapw]

Thanks, @backryun! 🙏 Merged — refreshed the shared devin-cli model catalog. Validated on the merge-train.