fix(connector-slack): no construct-time Slack auth (green main)

packages/connector-slack/src/slack-adapter.test.ts

@@ -55,6 +55,23 @@ function makeAdapter(): {
   return { adapter, setStatus, postMessage };
 }
 
+describe('SlackAdapter construction', () => {
+  it('does not throw or reject synchronously when constructed with fake tokens (no network call)', async () => {
+    // Guard: constructing must be network-free. With deferInitialization:true the
+    // Bolt App skips the construct-time auth.test() call, so no unhandled rejection fires.
+    let adapter: SlackAdapter | undefined;
+    expect(() => {
+      adapter = new SlackAdapter({
+        slackBotToken: 'xoxb-test',
+        slackAppToken: 'xapp-test',
+      } as never);
+    }).not.toThrow();
+    // Flush microtasks to surface any immediate promise rejection.
+    await Promise.resolve();
+    expect(adapter).toBeDefined();
+  });
+});
+
 describe('SlackAdapter.postError', () => {
   it('posts an errorBlock with the headline as the push fallback text', async () => {
     const { adapter, postMessage } = makeAdapter();

packages/connector-slack/src/slack-adapter.ts

@@ -58,6 +58,10 @@ export class SlackAdapter implements SurfaceAdapter {
       token: config.slackBotToken,
       appToken: config.slackAppToken,
       socketMode: true,
+      // deferInitialization prevents the Bolt App constructor from making a
+      // construct-time auth.test() network call. start() calls app.init()
+      // before app.start() to run the deferred initialization at the right time.
+      deferInitialization: true,
     });
     this.wire();
   }
@@ -71,6 +75,9 @@ export class SlackAdapter implements SurfaceAdapter {
   }
 
   async start(): Promise<void> {
+    // init() runs the deferred auth.test() that was skipped in the constructor
+    // (because deferInitialization:true); start() throws if init() was not called.
+    await this.app.init();
     await this.app.start();
   }