fix(devin): shorten the Devin provider label to one word - #4326
Conversation
The provider rail sits beside one-word names like Cursor, Kimi and Grok, and "Cognition (Devin/Windsurf)" is the registry's disambiguation for an add-provider list rather than a name to read at a glance. The rail now says "Devin"; the registry label is unchanged.
|
✅ Deterministic PR hygiene checks passed. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe Devin provider display name changes from ChangesDevin label update
Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
The dashboard provider rail showed
devinas "Cognition (Devin/Windsurf)". That string is the registry's disambiguation for an add-provider list, and it reads badly in a narrow rail next to one-word neighbours like Cursor, Kimi and Grok. The rail label is now "Devin".Only the GUI display name changes. The registry label, the provider id, and
devin-cli's "Devin CLI" are untouched.Verification
bun test tests/providers/devin-cli-adapter.test.ts— 16 pass, 0 fail; the display-name assertion moves with it.Repository-wide
bun run testandbun run typecheck: NOT RUN locally, per the operator constraint for this session. CI covers them on this head.Checklist
devSummary by CodeRabbit