Uh oh!
There was an error while loading. Please reload this page.
fix: experimental plan mode build incorrectly uses plan's model - #9299
fix: experimental plan mode build incorrectly uses plan's model#9299stevoland wants to merge 1 commit into
Conversation
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
| @@ -42,7 +48,7 @@ export const PlanExitTool = Tool.define("plan_exit", { | |||
| const answer = answers[0]?.[0] | |||
| if (answer === "No") throw new Question.RejectedError() | |||
| const model = await getLastModel(ctx.sessionID) | |||
| const model = await getAgentModel("build") | |||
There was a problem hiding this comment.
eh it should do:
- check if build agent has default model IF NOT then getLastModel(ctx.sessionID)
There was a problem hiding this comment.
@rekram1-node my bad. done and added the reverse for for build -> plan
4e8d252 to
cec9d87Comparef1ae801 to
08fa7f7CompareThis is needed, made a PR as well didn't see this one (closed mine) |
clement-heliot
commented
Mar 4, 2026
What is missing for this to get merged? The plan/build workflow with different configured models is broken right now |
stevoland
commented
Mar 5, 2026
@Antonio-Bennett I blew my one chance :) You should raise yours and I'll close this, you might get lucky |
Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR. |
What does this PR do?
Resolves: #9296
Planned with configured GTP-5.2, handover to build errored with GPT-5.2 rather than the configured opus-4.5:
How did you verify your code works?
repeated previous prompt and build agent model was correctly invoked