fix(nightingale-slack): remove dependency request

packages/nightingale-example/package.json

@@ -69,14 +69,16 @@
       "console",
       "debug",
       "errors",
-      "jsonConsole"
+      "jsonConsole",
+      "slack"
     ]
   },
   "dependencies": {
     "nightingale": "14.0.2",
     "nightingale-console": "14.0.2",
     "nightingale-console-output": "14.0.2",
     "nightingale-json-formatter": "14.0.2",
+    "nightingale-slack": "14.0.2",
     "nightingale-string": "14.0.2",
     "nightingale-types": "14.0.2"
   },

packages/nightingale-example/src/slack.ts

@@ -0,0 +1,23 @@
+import { Logger, configure, Level } from 'nightingale';
+import { SlackHandler } from 'nightingale-slack';
+
+if (!process.env.WEBHOOK_URL) {
+  throw new Error('Missing WEBHOOK_URL environment variable');
+}
+
+configure([
+  {
+    handlers: [
+      new SlackHandler(
+        {
+          webhookUrl: process.env.WEBHOOK_URL,
+        },
+        Level.ALL,
+      ),
+    ],
+  },
+]);
+
+const logger = new Logger('app');
+
+logger.success('Test', { value: 1 });

packages/nightingale-slack/dist/definitions/index.d.ts

@@ -2,6 +2,9 @@ import type { Handle, Handler, Level } from 'nightingale-types';
 import type { SlackConfig } from './SlackConfig';
 export type { SlackConfig } from './SlackConfig';
 export { default as createBody } from './createBody';
+declare global {
+    const fetch: typeof import('node-fetch').default;
+}
 export declare class SlackHandler implements Handler {
     minLevel: Level;
     handle: Handle;

packages/nightingale-slack/dist/index-node18.mjs

@@ -1,4 +1,3 @@
-import { post } from 'request';
 import { Level } from 'nightingale-levels';
 import markdownFormatter from 'nightingale-markdown-formatter';
 import rawFormatter from 'nightingale-raw-formatter';
@@ -32,14 +31,15 @@ function createBody(record, slackConfig) {
   };
 }
 
+// temp fix for global fetch: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/60924
+
 const createHandler = slackConfig => record => {
   const body = createBody(record, slackConfig);
-  post({
-    url: slackConfig.webhookUrl,
-    body,
-    json: true
-  }).on('error', err2 => {
-    console.error(err2.stack);
+  fetch(slackConfig.webhookUrl, {
+    method: 'POST',
+    body: JSON.stringify(body)
+  }).catch(error => {
+    console.error(error.stack);
   });
 };
 class SlackHandler {

packages/nightingale-slack/package.json

@@ -56,12 +56,12 @@
     "nightingale-levels": "14.0.2",
     "nightingale-markdown-formatter": "14.0.2",
     "nightingale-raw-formatter": "14.0.2",
-    "nightingale-types": "14.0.2",
-    "request": "^2.88.0"
+    "nightingale-types": "14.0.2"
   },
   "devDependencies": {
     "@babel/core": "7.22.9",
-    "@types/request": "2.48.8",
+    "@types/node": "18.17.1",
+    "@types/node-fetch": "2.6.4",
     "pob-babel": "36.2.0",
     "typescript": "5.1.6"
   }

packages/nightingale-slack/src/index.ts

@@ -5,25 +5,30 @@ import type {
   Handler,
   Level,
 } from 'nightingale-types';
-import { post } from 'request';
 import type { SlackConfig } from './SlackConfig';
 import createBody from './createBody';
 
 export type { SlackConfig } from './SlackConfig';
 
 export { default as createBody } from './createBody';
 
+// temp fix for global fetch: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/60924
+declare global {
+  // eslint-disable-next-line @typescript-eslint/consistent-type-imports
+  const fetch: typeof import('node-fetch').default;
+}
+
 const createHandler =
   (slackConfig: SlackConfig) =>
   <T extends Metadata>(record: LogRecord<T>) => {
     const body = createBody(record, slackConfig);
 
-    post({ url: slackConfig.webhookUrl, body, json: true }).on(
-      'error',
-      (err2: Error) => {
-        console.error(err2.stack);
-      },
-    );
+    fetch(slackConfig.webhookUrl, {
+      method: 'POST',
+      body: JSON.stringify(body),
+    }).catch((error: Error) => {
+      console.error(error.stack);
+    });
   };
 
 export class SlackHandler implements Handler {