Skip to content

Commit f89c66e

Browse files
committed
fix(auth): keep configured relogin order scoped
1 parent c68d751 commit f89c66e

2 files changed

Lines changed: 13 additions & 4 deletions

File tree

src/agents/auth-profiles/profiles.test.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -486,6 +486,7 @@ describe("promoteAuthProfileInOrder", () => {
486486
const newProfileId = "openai:new-login";
487487
const primaryProfileId = "openai:primary-login";
488488
const backupProfileId = "openai:backup-login";
489+
const unrelatedProfileId = "openai:unrelated-login";
489490
saveAuthProfileStore(
490491
{
491492
version: AUTH_STORE_VERSION,
@@ -511,6 +512,13 @@ describe("promoteAuthProfileInOrder", () => {
511512
refresh: "new-refresh",
512513
expires: Date.now() + 60 * 60 * 1000,
513514
},
515+
[unrelatedProfileId]: {
516+
type: "oauth",
517+
provider: "openai",
518+
access: "unrelated-access",
519+
refresh: "unrelated-refresh",
520+
expires: Date.now() + 30 * 60 * 1000,
521+
},
514522
},
515523
},
516524
agentDir,

src/agents/auth-profiles/profiles.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -114,10 +114,11 @@ export async function promoteAuthProfileInOrder(params: {
114114
if (!params.createIfMissing) {
115115
return false;
116116
}
117-
const providerProfiles = dedupeProfileIds([
118-
...(params.createFromOrder ?? []),
119-
...listProfilesForProvider(store, providerKey),
120-
]);
117+
const providerProfiles = dedupeProfileIds(
118+
params.createFromOrder !== undefined
119+
? params.createFromOrder
120+
: listProfilesForProvider(store, providerKey),
121+
);
121122
const next = dedupeProfileIds([
122123
params.profileId,
123124
...providerProfiles.filter((profileId) => profileId !== params.profileId),

0 commit comments

Comments
 (0)