fix(web): apply sticky model when reusing existing draft thread by murataslan1 · Pull Request #1291 · pingdotgg/t3code

murataslan1 · 2026-03-22T00:05:21Z

Summary

Fixes the model selection resetting to GPT 5.4 when creating a new thread.

Root Cause

handleNewThread() in useHandleNewThread.ts has 3 code paths:

Reuse stored draft thread (line 57) — sticky model was NOT applied
Reuse active draft in same project (line 79) — sticky model was NOT applied
Create brand new thread (line 95) — sticky model was applied

When a user clicks "New Thread" and a draft already exists for that project, path 1 was taken — navigating to the old draft without updating its model to match the user's last selection.

Fix

Apply stickyModel and stickyModelOptions to the draft thread in the stored draft path (path 1), matching the behavior already present in path 3.

Test plan

Select Claude Opus 4.6 and create a thread
Click "New Thread" on the same project
Verify the model picker shows Claude Opus 4.6 (not GPT 5.4)

Fixes #1276

Note

Apply sticky model when reusing an existing draft thread

When handleNewThread reuses an existing draft thread (stored or current route), it now sets the thread's provider and model from the composer's sticky model preference and applies any sticky model options. Previously, sticky model preferences were ignored in these reuse paths, so the thread would not reflect the user's saved model choice.

^{Macroscope summarized cf46b91.}

coderabbitai · 2026-03-22T00:05:27Z

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: 104c97a9-f8c1-44d0-9c94-32d807eac0ef

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

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

Tip

You can enable review details to help with troubleshooting, context usage and more.

Enable the reviews.review_details setting to include review details such as the model used, the time taken for each step and more in the review comments.

Copilot

Pull request overview

Fixes the web “New Thread” flow so that when an existing project draft thread is reused, it is updated to the user’s sticky model selection instead of reverting to the default model (e.g., GPT 5.4).

Changes:

Apply stickyModel and stickyModelOptions when reusing a stored draft thread in useHandleNewThread.
Set settingSources: ["user", "project"] for Claude Agent SDK query options in the server Claude adapter.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
apps/web/src/hooks/useHandleNewThread.ts	Updates the stored-draft reuse path to also apply sticky model + options before navigating/reusing.
apps/server/src/provider/Layers/ClaudeAdapter.ts	Adds Claude Agent SDK `settingSources` to query options (behavioral change to session startup).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-22T00:08:33Z

apps/server/src/provider/Layers/ClaudeAdapter.ts

          env: process.env,
+          settingSources: ["user", "project"],
          ...(input.cwd ? { additionalDirectories: [input.cwd] } : {}),


This PR is labeled/-described as a web-only fix for sticky model selection, but this change alters server-side Claude Agent session startup behavior by explicitly setting settingSources. Please either (a) document in the PR description why this server behavior change is required for the web sticky-model fix, including expected impact, or (b) move it to a separate PR to avoid shipping unrelated runtime behavior changes together.

Copilot · 2026-03-22T00:08:33Z

apps/web/src/hooks/useHandleNewThread.ts

+          if (stickyModel) {
+            setProvider(storedDraftThread.threadId, inferProviderForModel(stickyModel));
+            setModel(storedDraftThread.threadId, stickyModel);
+          }
+          if (Object.keys(stickyModelOptions).length > 0) {
+            setModelOptions(storedDraftThread.threadId, stickyModelOptions);
+          }


This adds sticky model/modelOptions application when reusing a stored draft thread, but the other reuse path (when latestActiveDraftThread exists for the same project and you return early around lines 86–100) still doesn’t apply stickyModel/stickyModelOptions. If the project->draft mapping is missing but the user is currently on a draft thread, clicking “New Thread” can still leave the draft on an outdated/default model. Consider applying the same sticky-model update in that early-return branch (or factor the sticky application into a shared helper).

UtkarshUsername · 2026-03-22T05:19:21Z

Also see #1277

murataslan1 · 2026-03-22T07:47:40Z

Addressed the feedback:

The settingSources change isn't in this PR, that's a separate one (fix(claude): add settingSources to load Skills, Commands, and CLAUDE.md #1290). Copilot might've gotten confused.
Added sticky model to the second reuse path too (the latestActiveDraftThread early return) — good catch from Copilot.
Also noticed fix(web): apply sticky model selection when reusing draft threads #1277 does the same thing with a cleaner helper function. Happy to close this if you'd rather go with that one.

UtkarshUsername · 2026-03-22T08:01:20Z

Are you a bot?

murataslan1 · 2026-03-22T08:05:38Z

Nope, just using Claude Code as my coding assistant.

UtkarshUsername · 2026-03-22T10:17:37Z

The settingSources is still in this PR

When creating a new thread, if a stored draft thread already existed for the project, the sticky model was not applied to it. This caused the model picker to show the default model (GPT 5.4) instead of the user's last selected model. Apply stickyModel and stickyModelOptions to the draft thread in all code paths, not just when creating a brand new thread. Fixes pingdotgg#1276

Address review feedback: the second reuse path (when latestActiveDraftThread exists for the same project) also needs stickyModel/stickyModelOptions applied, otherwise clicking "New Thread" while on an existing draft can leave it on an outdated model.

murataslan1 · 2026-03-22T10:55:52Z

You're right, my bad, the branch had the settingSources commit mixed in. Rebased on main so the diff only contains the sticky model fix now.

Copilot AI review requested due to automatic review settings March 22, 2026 00:05

github-actions bot added size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Mar 22, 2026

Copilot started reviewing on behalf of murataslan1 March 22, 2026 00:05 View session

Copilot AI reviewed Mar 22, 2026

View reviewed changes

github-actions bot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Mar 22, 2026

murataslan1 added 2 commits March 22, 2026 13:55

murataslan1 force-pushed the fix/model-reset-new-thread branch from 21f3e9b to cf46b91 Compare March 22, 2026 10:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(web): apply sticky model when reusing existing draft thread#1291

fix(web): apply sticky model when reusing existing draft thread#1291
murataslan1 wants to merge 2 commits intopingdotgg:mainfrom
murataslan1:fix/model-reset-new-thread

murataslan1 commented Mar 22, 2026 •

edited by macroscopeapp bot

Loading

Uh oh!

coderabbitai bot commented Mar 22, 2026 •

edited

Loading

Review skipped

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 22, 2026

Uh oh!

Copilot AI Mar 22, 2026

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026 •

edited

Loading

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

murataslan1 commented Mar 22, 2026 • edited by macroscopeapp bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Root Cause

Fix

Test plan

Apply sticky model when reusing an existing draft thread

Uh oh!

coderabbitai bot commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026

Uh oh!

UtkarshUsername commented Mar 22, 2026

Uh oh!

murataslan1 commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

murataslan1 commented Mar 22, 2026 •

edited by macroscopeapp bot

Loading

coderabbitai bot commented Mar 22, 2026 •

edited

Loading

murataslan1 commented Mar 22, 2026 •

edited

Loading

murataslan1 commented Mar 22, 2026 •

edited

Loading