hubot 13.1.1 → 13.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hubot",
-  "version": "13.1.1",
+  "version": "13.1.3",
   "author": "hubot",
   "keywords": [
     "github",
@@ -29,7 +29,7 @@
     "start": "bin/hubot",
     "gen": "bin/hubot --create myhubot",
     "pretest": "standard",
-    "test": "node --test",
+    "test": "node --test --test-timeout=20000",
     "test:smoke": "node src/**/*.js",
     "test:e2e": "bin/e2e-test.sh",
     "build:local": "npx @hubot-friends/sfab --folder ./docs --destination ./_site --verbose --serve /hubot/ --watch-path ./docs --scripts ./sfab-hooks",
@@ -45,6 +45,6 @@
   "dependencies": {
     "express": "^5.1.0",
     "express-basic-auth": "^1.2.1",
-    "pino": "^9.6.0"
+    "pino": "^9.12.0"
   }
 }

package/src/Robot.mjs

@@ -501,8 +501,9 @@ class Robot {
   //
   // Returns nothing.
   async loadAdapter (adapterPath = null) {
-    if (this.adapter) {
+    if (this.adapter && this.adapter.use) {
       this.adapter = await this.adapter.use(this)
+      this.adapterName = this.adapter.name ?? this.adapter.constructor.name
       return
     }
     this.logger.debug(`Loading adapter ${adapterPath ?? 'from npmjs:'} ${this.adapterName}`)
@@ -521,6 +522,8 @@ class Robot {
       this.logger.error(`Cannot load adapter ${adapterPath ?? '[no path set]'} ${this.adapterName} - ${error}`)
       throw error
     }
+
+    this.adapterName = this.adapter.name ?? this.adapter.constructor.name
   }
 
   async requireAdapterFrom (adapaterPath) {

package/src/adapters/Shell.mjs

@@ -39,11 +39,26 @@ class Shell extends Adapter {
     super(robot)
     this.name = 'Shell'
     const levels = ['trace', 'debug', 'info', 'warn', 'error', 'fatal']
+    const logLevel = process.env.HUBOT_LOG_LEVEL || 'info'
+    const levelPriorities = levels.reduce((acc, current, idx) => {
+      acc[current] = idx
+      return acc
+    }, {})
+
+    const configuredPriority = levelPriorities[logLevel]
+
+    const noop = async () => {}
+
     levels.forEach(level => {
-      robot.logger[level] = async (...args) => {
-        const color = levelColors[level] || ''
-        const msg = `${color}[${level}]${reset} ${args.map(a => typeof a === 'object' ? JSON.stringify(a) : a).join(' ')}`
-        await this.send({ user: { name: 'Logger', room: 'Shell' } }, msg)
+      const priority = levelPriorities[level]
+      if (priority >= configuredPriority) {
+        robot.logger[level] = async (...args) => {
+          const color = levelColors[level] || ''
+          const msg = `${color}[${level}]${reset} ${args.map(a => typeof a === 'object' ? JSON.stringify(a) : a).join(' ')}`
+          await this.send({ user: { name: 'Logger', room: 'Shell' } }, msg)
+        }
+      } else {
+        robot.logger[level] = noop
       }
     })
     this.robot.on('scripts have loaded', () => {