hubot 11.1.6 → 11.1.8

package/README.md

@@ -47,4 +47,6 @@ See the [LICENSE](LICENSE.md) file for license rights and limitations (MIT).
 
 [Hubot by Hubotics](https://www.theoldrobots.com/hubot.html)
 
-[Automating Inefficiencies](https://zachholman.com/2011/01/automating-inefficiencies/)
+[Automating Inefficiencies](https://zachholman.com/2011/01/automating-inefficiencies/)
+
+[Getting Started with Hubot](https://www.youtube.com/watch?v=A7fh6RIzGrw)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hubot",
-  "version": "11.1.6",
+  "version": "11.1.8",
   "author": "hubot",
   "keywords": [
     "github",

package/src/Adapter.mjs

@@ -79,7 +79,7 @@ class Adapter extends EventEmitter {
   // Returns an Array of User objects.
   // @deprecated Use @robot.brain
   users () {
-    this.robot.logger.warning('@users() is going to be deprecated in 11.0.0 use @robot.brain.users()')
+    this.robot.logger.warn('@users() is going to be deprecated in 11.0.0 use @robot.brain.users()')
     return this.robot.brain.users()
   }
 
@@ -88,7 +88,7 @@ class Adapter extends EventEmitter {
   // Returns a User instance of the specified user.
   // @deprecated Use @robot.brain
   userForId (id, options) {
-    this.robot.logger.warning('@userForId() is going to be deprecated in 11.0.0 use @robot.brain.userForId()')
+    this.robot.logger.warn('@userForId() is going to be deprecated in 11.0.0 use @robot.brain.userForId()')
     return this.robot.brain.userForId(id, options)
   }
 
@@ -97,7 +97,7 @@ class Adapter extends EventEmitter {
   // Returns a User instance for the user with the specified name.
   // @deprecated Use @robot.brain
   userForName (name) {
-    this.robot.logger.warning('@userForName() is going to be deprecated in 11.0.0 use @robot.brain.userForName()')
+    this.robot.logger.warn('@userForName() is going to be deprecated in 11.0.0 use @robot.brain.userForName()')
     return this.robot.brain.userForName(name)
   }
 
@@ -108,7 +108,7 @@ class Adapter extends EventEmitter {
   // Returns an Array of User instances matching the fuzzy name.
   // @deprecated Use @robot.brain
   usersForRawFuzzyName (fuzzyName) {
-    this.robot.logger.warning('@userForRawFuzzyName() is going to be deprecated in 11.0.0 use @robot.brain.userForRawFuzzyName()')
+    this.robot.logger.warn('@userForRawFuzzyName() is going to be deprecated in 11.0.0 use @robot.brain.userForRawFuzzyName()')
     return this.robot.brain.usersForRawFuzzyName(fuzzyName)
   }
 
@@ -119,7 +119,7 @@ class Adapter extends EventEmitter {
   // Returns an Array of User instances matching the fuzzy name.
   // @deprecated Use @robot.brain
   usersForFuzzyName (fuzzyName) {
-    this.robot.logger.warning('@userForFuzzyName() is going to be deprecated in 11.0.0 use @robot.brain.userForFuzzyName()')
+    this.robot.logger.warn('@userForFuzzyName() is going to be deprecated in 11.0.0 use @robot.brain.userForFuzzyName()')
     return this.robot.brain.usersForFuzzyName(fuzzyName)
   }
 
@@ -131,7 +131,7 @@ class Adapter extends EventEmitter {
   // Returns a ScopedClient instance.
   // @deprecated Use node.js fetch.
   http (url) {
-    this.robot.logger.warning('@http() is going to be deprecated in 11.0.0 use @robot.http()')
+    this.robot.logger.warn('@http() is going to be deprecated in 11.0.0 use @robot.http()')
     return this.robot.http(url)
   }
 }

package/src/Robot.mjs

@@ -133,8 +133,8 @@ class Robot {
     const name = this.name.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, '\\$&')
 
     if (regexStartsWithAnchor) {
-      this.logger.warning('Anchors don’t work well with respond, perhaps you want to use \'hear\'')
-      this.logger.warning(`The regex in question was ${regex.toString()}`)
+      this.logger.warn('Anchors don’t work well with respond, perhaps you want to use \'hear\'')
+      this.logger.warn(`The regex in question was ${regex.toString()}`)
     }
 
     if (!this.alias) {
@@ -336,7 +336,7 @@ class Robot {
     if (typeof script?.default === 'function') {
       result = await script.default(this)
     } else {
-      this.logger.warning(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to export default, got ${typeof script}`)
+      this.logger.warn(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to export default, got ${typeof script}`)
     }
     return result
   }
@@ -348,7 +348,7 @@ class Robot {
     if (typeof script === 'function') {
       result = await script(this)
     } else {
-      this.logger.warning(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to module.exports, got ${typeof script}`)
+      this.logger.warn(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to module.exports, got ${typeof script}`)
     }
     return result
   }

package/src/adapters/Shell.mjs

@@ -75,20 +75,15 @@ class Shell extends Adapter {
           this.#rl.prompt()
           break
       }
+      if (input.length > 0) {
+        fs.appendFileSync(historyPath, `${input}\n`)
+      }
       const history = fs.readFileSync(historyPath, 'utf-8').split('\n').reverse()
       this.#rl.history = history
-      this.#rl.on('line', line => {
-        const input = line.trim()
-        if (input.length === 0) return
-        fs.appendFile(historyPath, `${input}\n`, err => {
-          if (err) console.error(err)
-        })
-      })
       let userId = process.env.HUBOT_SHELL_USER_ID || '1'
       if (userId.match(/A\d+z/)) {
         userId = parseInt(userId)
       }
-
       const userName = process.env.HUBOT_SHELL_USER_NAME || 'Shell'
       const user = this.robot.brain.userForId(userId, { name: userName, room: 'Shell' })
       await this.receive(new TextMessage(user, input, 'messageId'))