fix(web): scope traits by provider instance

[aaditagrawal]

What Changed

• Synced the upstream reasoning-selection fix so composer trait/model options are read and persisted by provider instance id instead of driver kind.
• Passed the selected provider instance through trait controls and added coverage for custom instance option persistence.
• Added fork guardrails documenting protected fork features and fork-only handling for PRs, issues, GitHub Actions, workflows, checks, comments, labels, releases, and related repo operations.
• Updated README, CONTRIBUTING, AGENTS/CLAUDE, and the custom upstream-sync workflow notes to keep multi-provider support, usage monitoring, provider management, and orchestration reliability alive during upstream syncs.

Why

The fork supports custom provider instances and more providers than upstream. Reasoning and other provider trait selections must stay scoped to the selected instance so one configured provider does not overwrite another. The docs changes make that preservation rule explicit before future upstream merges.

UI Changes

No visual UI changes. This only preserves per-instance composer trait state and updates documentation.

Checklist

• This PR is small and focused
• I explained what changed and why
• No before/after screenshots are needed because there are no visual UI changes
• No video is needed because there are no animation/interaction changes

Verification

• bun fmt
• bun lint
• bun typecheck
• bun run test src/composerDraftStore.test.ts

Summary by CodeRabbit

• Documentation

  • Clarified fork policy and protected features: multi-provider support, usage/rate-limit monitoring, provider management, and orchestration reliability
  • Added Droid to the supported providers list
  • Expanded guidance on fork focus and upstream sync requirements

• New Features

  • Enhanced instance-specific provider configuration and management capabilities

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 233c10b6-1cac-4aec-b95f-2816561cbc0b

📥 Commits

Reviewing files that changed from the base of the PR and between 6cc364d and 394937a.

📒 Files selected for processing (10)

• AGENTS.md
• CLAUDE.md
• CONTRIBUTING.md
• README.md
• apps/web/src/components/chat/ChatComposer.tsx
• apps/web/src/components/chat/TraitsPicker.tsx
• apps/web/src/components/chat/composerProviderState.tsx
• apps/web/src/composerDraftStore.test.ts
• apps/web/src/composerDraftStore.ts
• docs/custom-alpha-workflow.md

---

📝 Walkthrough

Walkthrough

This PR enables provider model options to be selected by custom instance ID, routing this through the composer state, traits picker UI, and draft store. Supporting documentation updates establish the fork's commitment to preserving multi-provider support, usage monitoring, provider management UX, and orchestration reliability across upstream synchronization.

Changes

Instance-aware provider model selection and fork feature preservation

Layer / File(s)	Summary
Instance-aware provider traits type contracts apps/web/src/components/chat/TraitsPicker.tsx, apps/web/src/components/chat/composerProviderState.tsx	ProviderInstanceId type is imported and added as an optional prop to TraitsMenuContentProps and the internal TraitsRenderInput shape.
ChatComposer model options indexed by instance apps/web/src/components/chat/ChatComposer.tsx	composerProviderState lookup switches from selectedProvider to selectedInstanceId for model options retrieval, and trait UI rendering explicitly passes instanceId: selectedInstanceId to both menu and picker components.
TraitsPicker and composer state instance prop threading apps/web/src/components/chat/TraitsPicker.tsx, apps/web/src/components/chat/composerProviderState.tsx	TraitsMenuContent and TraitsPicker destructure and conditionally forward instanceId through the component tree; renderTraitsControl passes instanceId to trait components when provided.
Draft store instance-aware model option persistence apps/web/src/composerDraftStore.ts	setProviderModelOptions accepts optional instanceId to derive instance-specific keys and conditionally update sticky activation and activeProvider only when an instance is provided.
Test coverage for instance-aware model option persistence apps/web/src/composerDraftStore.test.ts	New test verifies that model option updates persist to secondary custom instances with correct sticky state and active provider switching.
Fork policies and protected features documentation AGENTS.md, CONTRIBUTING.md, README.md, CLAUDE.md, docs/custom-alpha-workflow.md	Fork governance documents define and protect four core capabilities (multi-provider runtime support, usage/limit monitoring, provider management UX, orchestration reliability), add Droid to the supported provider list, clarify fork-only GitHub operations, and introduce a maintainer verification checklist for upstream syncs.

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested labels

vouch:trusted, size:M

Poem

A rabbit hops through instance trees,
Where providers bend to custom keys,
The fork stands firm on features four,
Protected, documented, nevermore,
A Droid joins the burrow's lore! 🐰

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix(web): scope traits by provider instance' directly describes the main code change: scoping composer trait/model options by provider instance ID instead of driver kind.
Description check	✅ Passed	The description covers all required template sections: What Changed, Why, UI Changes, and Checklist. It thoroughly explains the code fix, documentation updates, and reasoning.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/sync-upstream-traits-and-fork-rules

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}