Skip to content

Commit f7b7c3c

Browse files
Fix current CI baseline failures
1 parent 99fe86c commit f7b7c3c

3 files changed

Lines changed: 22 additions & 19 deletions

File tree

extensions/openrouter/provider-routing.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ function mergeOpenRouterProviderRouting(params: {
5656
const modelRouting = readRecord(params.modelParams?.provider);
5757
const extraRouting = readRecord(params.extraParams.provider);
5858
const merged = {
59-
...(providerRouting ?? {}),
60-
...(modelRouting ?? {}),
61-
...(extraRouting ?? {}),
59+
...providerRouting,
60+
...modelRouting,
61+
...extraRouting,
6262
};
6363
return Object.keys(merged).length > 0 ? merged : undefined;
6464
}
@@ -78,8 +78,8 @@ export function resolveOpenRouterExtraParamsForTransport(
7878
}
7979
return {
8080
patch: {
81-
...(providerConfigParams ?? {}),
82-
...(modelParams ?? {}),
81+
...providerConfigParams,
82+
...modelParams,
8383
...ctx.extraParams,
8484
...(providerRouting ? { provider: providerRouting } : {}),
8585
},

src/commands/status.summary.redaction.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ function createRecentSessionRow() {
1414
remainingTokens: 4,
1515
percentUsed: 5,
1616
model: "gpt-5",
17+
configuredModel: "gpt-5",
18+
selectedModel: "gpt-5",
19+
modelSelectionReason: null,
1720
contextTokens: 200_000,
1821
flags: ["id:sess-1"],
1922
};

src/infra/secret-file.test.ts

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -112,20 +112,6 @@ describe("readSecretFileSync", () => {
112112
});
113113

114114
it.each([
115-
{
116-
name: "returns undefined from the non-throwing helper for rejected files",
117-
pathValue: async () =>
118-
createSecretPath(async (dir) => {
119-
const target = path.join(dir, "target.txt");
120-
const link = path.join(dir, "secret-link.txt");
121-
await fsPromises.writeFile(target, "top-secret\n", "utf8");
122-
await fsPromises.symlink(target, link);
123-
return link;
124-
}),
125-
label: "Telegram bot token",
126-
options: { rejectSymlink: true },
127-
expected: undefined,
128-
},
129115
{
130116
name: "returns undefined from the non-throwing helper for blank file paths",
131117
pathValue: async () => " ",
@@ -144,6 +130,20 @@ describe("readSecretFileSync", () => {
144130
const file = await pathValue();
145131
expect(tryReadSecretFileSync(file, label, options)).toBe(expected);
146132
});
133+
134+
it("throws from the non-throwing helper for rejected symlinks", async () => {
135+
const link = await createSecretPath(async (dir) => {
136+
const target = path.join(dir, "target.txt");
137+
const symlink = path.join(dir, "secret-link.txt");
138+
await fsPromises.writeFile(target, "top-secret\n", "utf8");
139+
await fsPromises.symlink(target, symlink);
140+
return symlink;
141+
});
142+
143+
expect(() =>
144+
tryReadSecretFileSync(link, "Telegram bot token", { rejectSymlink: true }),
145+
).toThrow("must not be a symlink");
146+
});
147147
});
148148

149149
describe("writePrivateSecretFileAtomic", () => {

0 commit comments

Comments
 (0)