[codex] Break web composer circular dependency

[PlunderStruck]

What changed

• extracted the pure composer provider-state normalization into apps/web/src/composerProviderState.ts
• updated non-UI callers to import the pure helper directly
• trimmed composerProviderRegistry.tsx back to traits rendering only
• added a markdown checklist plan at docs/web-circular-deps-plan.md

Why

• ctx-analysis.md and the live dependency graph both identified a real web import cycle through composerDraftStore.ts, modelSelection.ts, composerProviderRegistry.tsx, and TraitsPicker.tsx
• the cycle made modelSelection.ts pull chat UI and store setup into non-UI code paths

Impact

• removes the web circular dependency while keeping composer behavior unchanged
• leaves the remaining contracts cycle untouched because it is still the separate type-coupling issue we discussed

Validation

• bun fmt
• bun lint
• bun typecheck
• bun run test
• refreshed SocratiCode graph: only the contracts cycle remains
• refreshed SCIP index: modelSelection.ts now depends on composerProviderState.ts instead of the chat registry

Note

Break circular dependency by extracting getComposerProviderState into a standalone module

• Moves getComposerProviderState, ComposerProviderState, and ComposerProviderStateInput out of composerProviderRegistry.tsx into a new composerProviderState.ts module.
• composerProviderRegistry now only contains UI rendering logic (renderTraitsMenuContent, renderTraitsPicker), dropping all state-computation imports.
• ChatView.tsx and modelSelection.ts are updated to import from the new module.

^{Macroscope summarized 5415598.}

---

Note

Low Risk
Low risk refactor that primarily moves provider-state normalization into a standalone module; main risk is subtle behavioral drift in model option normalization or ultrathink styling if the extraction missed edge cases.

Overview
Breaks a web circular dependency by extracting the pure composer provider-state normalization (getComposerProviderState and related types) into a new apps/web/src/composerProviderState.ts module.

Updates non-UI callers (modelSelection.ts, ChatView.tsx) to import the new helper directly, and trims composerProviderRegistry.tsx down to traits UI rendering only. Provider-state tests are repointed to the new module, and a new docs/web-circular-deps-plan.md documents the dependency cycle and verification steps.

^{Written by Cursor Bugbot for commit 5415598. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 5c527875-f2b4-4ee7-9efe-78a49bfe624a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}