fix(orchestrate): treat whitespace Slack tokens as unset

slackTokenConfigured() already trimmed ORCHESTRATE_SLACK_BOT_TOKEN, but
createSlackWebClient() used the raw env value, so a spaces-only token
skipped channel guards while still initializing an unusable WebClient.
Share one trimmed reader so both helpers agree.

Author: cursoragent

orchestrate/skills/orchestrate/scripts/adapters/slack/client.ts

@@ -5,12 +5,17 @@ export const SLACK_TOKEN_ENV = "ORCHESTRATE_SLACK_BOT_TOKEN";
 
 const LEGACY_SLACK_TOKEN_ENV = "SLACK_BOT_TOKEN";
 
+function readSlackToken(): string | undefined {
+  const token = process.env[SLACK_TOKEN_ENV]?.trim();
+  return token || undefined;
+}
+
 export function slackTokenConfigured(): boolean {
-  return Boolean(process.env[SLACK_TOKEN_ENV]?.trim());
+  return Boolean(readSlackToken());
 }
 
 export function createSlackWebClient(): WebClient | undefined {
-  const token = process.env[SLACK_TOKEN_ENV];
+  const token = readSlackToken();
   if (!token) {
     // Losing Slack silently after the rename would look like an outage, so
     // name the old variable when it's the only one set.