From c713d48d08af5044cb0fab4505365fd98e31b9d6 Mon Sep 17 00:00:00 2001
From: kaitranntt <kaitran.ntt@gmail.com>
Date: Tue, 27 Jan 2026 22:25:31 -0500
Subject: [PATCH] refactor(oauth): derive auth-code providers from
 OAUTH_FLOW_TYPES (DRY)

- Update checkAuthCodePorts() to filter from CLIPROXY_PROFILES

- Update header comment to list all OAuth ports including Claude

- Addresses PR #384 review round 3
---
 src/management/oauth-port-diagnostics.ts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/management/oauth-port-diagnostics.ts b/src/management/oauth-port-diagnostics.ts
index 8c261457..d50c16cf 100644
--- a/src/management/oauth-port-diagnostics.ts
+++ b/src/management/oauth-port-diagnostics.ts
@@ -8,7 +8,11 @@
  * - Gemini: 8085
  * - Codex: 1455
  * - Agy: 51121
+ * - iFlow: 11451
+ * - Kiro: 9876
+ * - Claude: 54545
  * - Qwen: Device Code Flow (no port needed)
+ * - GHCP: Device Code Flow (no port needed)
  */
 
 import {
@@ -156,7 +160,8 @@ export async function checkAllOAuthPorts(): Promise<OAuthPortDiagnostic[]> {
  * Check OAuth ports for providers that use Authorization Code flow only
  */
 export async function checkAuthCodePorts(): Promise<OAuthPortDiagnostic[]> {
-  const providers: CLIProxyProvider[] = ['gemini', 'codex', 'agy', 'iflow', 'kiro', 'claude'];
+  // Filter providers that use authorization_code flow (DRY: derive from OAUTH_FLOW_TYPES)
+  const providers = CLIPROXY_PROFILES.filter((p) => OAUTH_FLOW_TYPES[p] === 'authorization_code');
   const results: OAuthPortDiagnostic[] = [];
 
   for (const provider of providers) {