feat: migrate from axios to fetch

packages/oauth/package.json

@@ -41,7 +41,7 @@
   },
   "dependencies": {
     "@slack/logger": "^4.0.1",
-    "@slack/web-api": "^7.15.0",
+    "@slack/web-api": "^8.0.0-rc.1",
     "@types/jsonwebtoken": "^9",
     "@types/node": ">=20",
     "jsonwebtoken": "^9"

packages/socket-mode/examples/proxy.js

@@ -1,17 +1,18 @@
 const { SocketModeClient, LogLevel } = require('@slack/socket-mode');
-const HttpsProxyAgent = require('https-proxy-agent');
-const clientOptions = { agent: new HttpsProxyAgent('http://localhost:9001') };
+const { ProxyAgent } = require('undici');
+
+const dispatcher = new ProxyAgent('http://localhost:9001');
 
 const socketModeClient = new SocketModeClient({
   appToken: process.env.SLACK_APP_TOKEN,
   logLevel: LogLevel.DEBUG,
-  clientOptions,
+  dispatcher,
 });
 
 // const { WebClient } = require('@slack/web-api');
 // const webClient = new WebClient(process.env.SLACK_BOT_TOKEN, {
 //   logLevel: LogLevel.DEBUG,
-//   clientOptions,
+//   fetch: (url, options) => fetch(url, { ...options, dispatcher })
 // });
 
 socketModeClient.on('slack_event', async ({ ack, body }) => {

packages/socket-mode/package.json

@@ -45,23 +45,31 @@
     "prepack": "npm run build",
     "test": "npm run test:unit && npm run test:integration",
     "test:coverage": "npm run build && node --experimental-test-coverage --test-reporter=spec --test-reporter-destination=stdout --test-reporter=lcov --test-reporter-destination=lcov.info --test-reporter=junit --test-reporter-destination=test-results.xml --import tsx --test src/*.test.ts",
-    "test:integration": "npm run build && node --import tsx --test test/integration.test.js",
+    "test:integration": "npm run build && bash -c 'node --import tsx --test test/integrations/*.test.js'",
+    "test:types": "tsd",
     "test:unit": "npm run build && bash -c 'node --test-reporter=spec --test-reporter-destination=stdout --test-reporter=junit --test-reporter-destination=test-results.xml --import tsx --test src/*.test.ts'",
     "watch": "npx nodemon --watch 'src' --ext 'ts' --exec npm test"
   },
   "dependencies": {
     "@slack/logger": "^4.0.1",
-    "@slack/web-api": "^7.15.0",
+    "@slack/web-api": "^8.0.0-rc.1",
     "@types/node": ">=20",
-    "@types/ws": "^8",
-    "eventemitter3": "^5",
-    "ws": "^8"
+    "eventemitter3": "^5"
+  },
+  "peerDependencies": {
+    "undici": "^7.0.0"
   },
   "devDependencies": {
     "@types/proxyquire": "^1.3.31",
     "@types/sinon": "^21",
     "nodemon": "^3.1.0",
     "proxyquire": "^2.1.3",
-    "sinon": "^21"
+    "sinon": "^21",
+    "tsd": "^0.33.0",
+    "undici": "^7.25.0",
+    "ws": "^8"
+  },
+  "tsd": {
+    "directory": "test/types"
   }
 }

packages/socket-mode/src/SlackWebSocket.test.ts

@@ -4,26 +4,34 @@ import { ConsoleLogger } from '@slack/logger';
 import EventEmitter from 'eventemitter3';
 import proxyquire from 'proxyquire';
 import sinon from 'sinon';
+import { CloseEvent, ErrorEvent, MessageEvent } from 'undici';
 
 proxyquire.noPreserveCache();
 
 import logModule from './logger';
 
-// A slightly spruced up event emitter aiming at mocking out the `ws` library's `WebSocket` class
-class WSMock extends EventEmitter {
-  // biome-ignore lint/suspicious/noExplicitAny: event listeners can accept any args
-  addEventListener(evt: string, fn: (...args: any[]) => void) {
-    this.addListener.call(this, evt, fn);
-  }
+// Minimal mock of undici's WebSocket (EventTarget-based)
+class WSMock extends EventTarget {
+  static CONNECTING = 0;
+  static OPEN = 1;
+  static CLOSING = 2;
+  static CLOSED = 3;
+  readyState = 1;
+  close() {}
+  send(_data: string) {}
 }
 
 describe('SlackWebSocket', () => {
   const sandbox = sinon.createSandbox();
   let SlackWebSocket: typeof import('./SlackWebSocket').SlackWebSocket;
   beforeEach(() => {
     SlackWebSocket = proxyquire.load('./SlackWebSocket', {
-      ws: {
+      undici: {
         WebSocket: WSMock,
+        CloseEvent,
+        ErrorEvent,
+        MessageEvent,
+        ping: () => {},
       },
     }).SlackWebSocket;
   });
@@ -58,17 +66,19 @@ describe('SlackWebSocket', () => {
   });
   describe('WebSocket event handling', () => {
     it('should call disconnect() if websocket emits an error', async () => {
-      // an exposed event emitter pretending it's a websocket
       const ws = new WSMock();
-      // mock out the `ws` library and have it return our event emitter mock
       SlackWebSocket = proxyquire.load('./SlackWebSocket', {
-        ws: {
+        undici: {
           WebSocket: class Fake {
             constructor() {
               // biome-ignore lint/correctness/noConstructorReturn: for test mocking purposes
               return ws;
             }
           },
+          CloseEvent,
+          ErrorEvent,
+          MessageEvent,
+          ping: () => {},
         },
       }).SlackWebSocket;
       const sws = new SlackWebSocket({
@@ -79,7 +89,7 @@ describe('SlackWebSocket', () => {
       });
       const discStub = sinon.stub(sws, 'disconnect');
       sws.connect();
-      ws.emit('error', { error: new Error('boom') });
+      ws.dispatchEvent(new ErrorEvent('error', { error: new Error('boom'), message: 'boom' }));
       sinon.assert.calledOnce(discStub);
     });
   });