Skip to content

Commit efd3172

Browse files
committed
fix: resolve fallback fast cutoff per model
1 parent 8afc1f7 commit efd3172

2 files changed

Lines changed: 10 additions & 7 deletions

File tree

extensions/codex/src/app-server/run-attempt.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5623,6 +5623,7 @@ describe("runCodexAppServerAttempt", () => {
56235623
name: "fast off",
56245624
fastMode: false,
56255625
configuredServiceTier: "priority",
5626+
expectedServiceTier: null,
56265627
},
56275628
{
56285629
name: "fast auto active",
@@ -5633,7 +5634,7 @@ describe("runCodexAppServerAttempt", () => {
56335634
name: string;
56345635
fastMode: EmbeddedRunAttemptParams["fastMode"];
56355636
configuredServiceTier?: "priority";
5636-
expectedServiceTier?: "priority";
5637+
expectedServiceTier?: "priority" | null;
56375638
}>)(
56385639
"maps $name to app-server resume and turn service tier",
56395640
async ({ fastMode, configuredServiceTier, expectedServiceTier }) => {

src/auto-reply/reply/agent-runner-utils.ts

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -217,19 +217,21 @@ export function resolveRunFastModeForFallbackCandidate(params: {
217217
model: string;
218218
sessionEntry?: Pick<SessionEntry, "fastMode">;
219219
}) {
220-
if (params.run.fastModeOverride) {
221-
return {
222-
fastMode: params.run.fastMode,
223-
fastModeAutoOnSeconds: params.run.fastModeAutoOnSeconds,
224-
};
225-
}
226220
const state = resolveFastModeState({
227221
cfg: params.config,
228222
provider: params.provider,
229223
model: params.model,
230224
agentId: params.run.agentId,
231225
sessionEntry: params.sessionEntry,
232226
});
227+
if (params.run.fastModeOverride) {
228+
return {
229+
fastMode: params.run.fastMode,
230+
fastModeAutoOnSeconds: params.run.fastModeAutoOnSecondsOverride
231+
? params.run.fastModeAutoOnSeconds
232+
: state.fastAutoOnSeconds,
233+
};
234+
}
233235
return {
234236
fastMode: state.mode,
235237
fastModeAutoOnSeconds: params.run.fastModeAutoOnSecondsOverride

0 commit comments

Comments
 (0)