fix(web): describe what Ultracode does in the Reasoning picker - #6092
fix(web): describe what Ultracode does in the Reasoning picker#6092delltrak wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
ApprovabilityVerdict: Approved c60cd35 This PR adds descriptive text to the 'Ultracode' option in the Reasoning picker UI. The changes are purely additive - adding an optional description field to existing data structures and rendering it in the UI. No runtime logic changes beyond displaying text. You can customize Macroscope's approvability policy. Learn more. |
cd45c04 to
81626e9
Compare
Ultracode sits next to Low/Medium/High/Extra High/Max with a bare label, so nothing in the UI says it also turns on standing multi-agent workflow orchestration. Claude Code's own picker spells that out. ProviderOptionChoice already declares an optional description, it just was never plumbed through. Pass it along in buildSelectOptionDescriptor, render it in TraitsPicker, and set it on ultracode for the three models that offer it. Options without a description render exactly as before.
81626e9 to
c60cd35
Compare
Dismissing prior approval to re-evaluate c60cd35
Closes #6091
Reasoning → Ultracodesits next to Low/Medium/High/Extra High/Max with a bare label. Nothing in the UI says it also turns on standing multi-agent workflow orchestration. Claude Code's own picker spells it out (xhigh + dynamic workflow orchestration).ProviderOptionChoiceinpackages/contracts/src/model.tsalready declares an optionaldescription. It just was never plumbed through, so this wires it up and sets one on Ultracode:providerSnapshot.ts:buildSelectOptionDescriptorstops droppingdescriptionwhen it maps choicesTraitsPicker.tsx: rendersoption.descriptionunder the label when there is oneClaudeProvider.ts: sets it onultracodefor the three models that offer it (fable-5, opus-5, opus-4-8)Data-driven rather than a hardcoded ultracode branch in the picker, since #5696 is moving provider-specific logic out of that file. Any provider can annotate any select option now.
Options without a description render exactly as before. No contract change, nothing to migrate.
The description is capped at
max-w-56and wraps, so the menu settles around 250px instead of growing with the text. Menus whose options carry no description keep their current width.Renaming the label would have been a smaller diff but a worse result.
TraitsPickertruncates choice labels, and mobile's compact settings pill reuses the sameoption.label, soOpus 5 · Ultracode (multi-agent workflows) · Fullwould land on the one surface that already hides this option on purpose.#5696 touches
ClaudeProvider.tsandTraitsPicker.tsxtoo. Whichever lands second needs a rebase.Testing
vp fmtandvp lint: cleantypecheckont3,@t3tools/web,@t3tools/contracts: passesvp run --filter @t3tools/web test: 2161 passed across 234 filesvp run --filter t3 test: 2371 passed, 7 skipped