fix: update misleading comment in orderProfilesByMode to reflect both sort criteria

adhitShet · claude · gumadeiras · commit 04271651d4fc · 2026-02-19T19:42:13.000-05:00
Comment now accurately describes that sorting is by type preference (oauth &gt; token &gt; api_key) then by lastUsed oldest-first for round-robin within type.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/src/agents/auth-profiles/order.ts b/src/agents/auth-profiles/order.ts
@@ -144,7 +144,7 @@ function orderProfilesByMode(order: string[], store: AuthProfileStore): string[]
     }
   }
 
-  // Sort available profiles by lastUsed (oldest first = round-robin)
+  // Sort available profiles by type preference, then by lastUsed (oldest first = round-robin within type)
   const scored = available.map((profileId) => {
     const type = store.profiles[profileId]?.type;
     const typeScore = type === "oauth" ? 0 : type === "token" ? 1 : type === "api_key" ? 2 : 3;

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ function orderProfilesByMode(order: string[], store: AuthProfileStore): string[]`
`144`	`144`	`}`
`145`	`145`	`}`
`146`	`146`
`147`		`- // Sort available profiles by lastUsed (oldest first = round-robin)`
	`147`	`+ // Sort available profiles by type preference, then by lastUsed (oldest first = round-robin within type)`
`148`	`148`	`const scored = available.map((profileId) => {`
`149`	`149`	`const type = store.profiles[profileId]?.type;`
`150`	`150`	`const typeScore = type === "oauth" ? 0 : type === "token" ? 1 : type === "api_key" ? 2 : 3;`