@xpr/nestjs-slack 0.4.0 → 0.6.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xpr/nestjs-slack",
-  "version": "0.4.0",
+  "version": "0.6.0",
   "description": "NestJS server implementation of the Slack Assistant",
   "main": "index.js",
   "scripts": {

package/slack.d.ts

@@ -1,5 +1,5 @@
-import { type CustomTransportStrategy, type MessageHandler, Server } from "@nestjs/microservices";
-import { App, type AppOptions } from "@slack/bolt";
+import { type CustomTransportStrategy, type MessageHandler, Server } from '@nestjs/microservices';
+import { App, type AppOptions } from '@slack/bolt';
 export type SlackOptions = {
     /**
      * Slack application options with required types

package/slack.js

@@ -14,8 +14,6 @@ class Slack extends microservices_1.Server {
     async listen(callback) {
         for (const handler of this.getHandlers().values()) {
             this.register(handler);
-            const { type, event } = handler.extras;
-            this.logger.log(`Handler [${type}] registered with (${event})`);
         }
         await this.app().start();
         callback();
@@ -24,7 +22,7 @@ class Slack extends microservices_1.Server {
         this.#app && (await this.#app.stop());
     }
     on() {
-        throw new Error("Use SlackEvent decorator to register events");
+        throw new Error('Use SlackEvent decorator to register events');
     }
     unwrap() {
         return this.#app;
@@ -36,6 +34,7 @@ class Slack extends microservices_1.Server {
      */
     register(handler) {
         const { type, event } = handler.extras;
+        this.logger.log(`Registering handler of type [${type}] with (${event})`);
         switch (type) {
             case decorators_1.EventTypes.Shortcut:
                 return this.app().shortcut(event, handler);
@@ -61,7 +60,7 @@ class Slack extends microservices_1.Server {
                 this.options.slack.logger = (0, utils_1.adjustLogger)(this.logger);
             }
             this.#app = new bolt_1.App(this.options.slack);
-            this.logger.log("Bolt app created");
+            this.logger.log('Bolt app created');
         }
         return this.#app;
     }

package/utils.js

@@ -37,21 +37,24 @@ function messageDecorator(type, pattern) {
     })(target, key, descriptor);
 }
 function adjustLogger(logger) {
-    // minimal logger interface required by the library
+    // this patch required to support NestJS logger that not use the standard "info"
+    // and use "log" instead
+    if (logger["log"] && !logger["info"]) {
+        logger["info"] = logger["log"].bind(logger);
+    }
+    // minimal logger interface required by @slack/bolt library
     for (const method of ["debug", "info", "warn", "error"]) {
         if (typeof logger[method] !== "function") {
             throw new Error(`Logger must implement "${method}" method`);
         }
     }
-    const noop = () => {
-    };
     return Object.assign(logger, {
         setLevel(_) {
             // don't allow library to set the level
         },
         setName(name) {
-            // setContext is alias for setName
-            (logger.setName ?? logger.setContext ?? noop)(name);
+            // search for all possible setName/setContext methods, use "noop" if none
+            (logger.setName ?? logger.setContext ?? ((i) => i))(name);
         },
         getLevel() {
             // always return the lowest supported level (by library, not real logger)