fix(identity): avoid mutating fallback map during iteration

ndycode · ndycode · commit 2f3e2f4b9569 · 2026-02-28T02:20:15.000+08:00
diff --git a/index.ts b/index.ts
@@ -909,11 +909,15 @@ export const OpenAIOAuthPlugin: Plugin = async ({ client }: PluginInput) => {
 						}
 					}
 
+					const fallbackAccountIdsToDelete: string[] = [];
 					for (const [accountId, fallbackIndex] of entry.fallbackByAccountId) {
 						if (collapseFallbackIndex(fallbackIndex)) {
-							entry.fallbackByAccountId.delete(accountId);
+							fallbackAccountIdsToDelete.push(accountId);
 						}
 					}
+					for (const accountId of fallbackAccountIdsToDelete) {
+						entry.fallbackByAccountId.delete(accountId);
+					}
 				};
 
 				for (let i = 0; i < accounts.length; i += 1) {

Original file line number	Diff line number	Diff line change
`@@ -909,11 +909,15 @@ export const OpenAIOAuthPlugin: Plugin = async ({ client }: PluginInput) => {`
`909`	`909`	`}`
`910`	`910`	`}`
`911`	`911`
	`912`	`+ const fallbackAccountIdsToDelete: string[] = [];`
`912`	`913`	`for (const [accountId, fallbackIndex] of entry.fallbackByAccountId) {`
`913`	`914`	`if (collapseFallbackIndex(fallbackIndex)) {`
`914`		`- entry.fallbackByAccountId.delete(accountId);`
	`915`	`+ fallbackAccountIdsToDelete.push(accountId);`
`915`	`916`	`}`
`916`	`917`	`}`
	`918`	`+ for (const accountId of fallbackAccountIdsToDelete) {`
	`919`	`+ entry.fallbackByAccountId.delete(accountId);`
	`920`	`+ }`
`917`	`921`	`};`
`918`	`922`
`919`	`923`	`for (let i = 0; i < accounts.length; i += 1) {`