From 63e41d1735ca6092438099e9390164ef1ec9fceb Mon Sep 17 00:00:00 2001
From: Cafexss <coffeemjj@gmail.com>
Date: Fri, 13 Mar 2026 14:43:49 +0800
Subject: [PATCH] refactor: remove redundant ?? undefined in Slack probe

Remove redundant nullish coalescing with undefined (?? undefined) in
Slack probe result handling. The properties from Slack auth.test() are
already typed as string | undefined, so ?? undefined is a no-op.
---
 src/slack/probe.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/slack/probe.ts b/src/slack/probe.ts
index 22857ca2bc6..165c5af636b 100644
--- a/src/slack/probe.ts
+++ b/src/slack/probe.ts
@@ -26,8 +26,8 @@ export async function probeSlack(token: string, timeoutMs = 2500): Promise<Slack
       ok: true,
       status: 200,
       elapsedMs: Date.now() - start,
-      bot: { id: result.user_id ?? undefined, name: result.user ?? undefined },
-      team: { id: result.team_id ?? undefined, name: result.team ?? undefined },
+      bot: { id: result.user_id, name: result.user },
+      team: { id: result.team_id, name: result.team },
     };
   } catch (err) {
     const message = err instanceof Error ? err.message : String(err);