Skip to content

Commit 2880019

Browse files
committed
fix(prompt): default size probe survives the gpt row
openai/gpt-4.1 now resolves to the gpt family, so the default column measured the narrate-before-tools nudge. Probe an unrecognized provider instead — default family regardless of shipped rows — and assert the default column carries neither the claude block nor the gpt nudge.
1 parent f0fa3f4 commit 2880019

2 files changed

Lines changed: 9 additions & 7 deletions

File tree

‎src/agent/prompt-sizes.test.ts‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -345,10 +345,11 @@ describe("grok tool-budget residual (CL-8297)", () => {
345345
test("default-family and orchestrator prompts carry no tool budget", () => {
346346
const defaultPrompt = assembleDirectorPrompt("builder", "default");
347347
expect(defaultPrompt).not.toContain("Tool budget:");
348-
// The default probe (openai/gpt-4.1) resolves to the default family, so
349-
// the default column carries no family residual at all — not the claude
350-
// task_guidance block either.
348+
// The default probe resolves to the default family, so the default
349+
// column carries no family residual — neither the claude task_guidance
350+
// block nor the gpt narrate-before-tools nudge.
351351
expect(defaultPrompt).not.toContain("<task_guidance>");
352+
expect(defaultPrompt).not.toContain("Narrate before tools (GPT worker):");
352353
expect(assembleDirectorPrompt("skywalker", "grok")).not.toContain(
353354
"Tool budget:",
354355
);

‎src/agent/prompt-sizes.ts‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,11 +57,12 @@ export const CANONICAL_PROMPT_ENV: EnvironmentInfo = {
5757
};
5858

5959
const GROK_PROVIDER = { providerName: "xai/default", model: "grok-4.6" };
60-
// Default-family probe: openai/gpt-4.1 resolves to the default family (the
61-
// gpt row lands later in CL-8310), so the default column carries no residual.
60+
// Default-family probe: an unrecognized provider stays on the default
61+
// family no matter how many family rows land (claude/gpt already ship),
62+
// so the default column carries no residual.
6263
const DEFAULT_PROVIDER = {
63-
providerName: "openai",
64-
model: "gpt-4.1",
64+
providerName: "unknown-provider",
65+
model: "unknown-model",
6566
};
6667

6768
/** Families in the size table: default assembly vs Grok (+finish-bias note). */

0 commit comments

Comments
 (0)