hubot 13.0.1 → 13.1.0

package/CONTRIBUTING.md

@@ -2,7 +2,7 @@
 
 Contributions to this project are [released](https://help.github.com/articles/github-terms-of-service/#6-contributions-under-repository-license) to the public under the [project's open source license](LICENSE.md).
 
-Everyone is welcome to contribute to Hubot. Contributing doesn’t just mean submitting pull requests—there are many different ways for you to get involved, including answering questions in [chat](https://hubot-slackin.herokuapp.com/), reporting or triaging [issues](https://github.com/github/hubot/issues), and participating in the [Hubot Evolution](https://github.com/hubotio/evolution) process.
+Everyone is welcome to contribute to Hubot. Contributing doesn’t just mean submitting pull requests—there are many different ways for you to get involved, including answering questions, reporting or triaging [issues](https://github.com/github/hubot/issues), and participating in using Hubot.
 
 No matter how you want to get involved, we ask that you first learn what’s expected of anyone who participates in the project by reading the [Contributor Covenant Code of Conduct](http://contributor-covenant.org). By participating, you are expected to uphold this code.
 

package/package.json

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

package/src/adapters/Shell.mjs

@@ -1,6 +1,6 @@
 'use strict'
 
-import fs from 'node:fs'
+import { stat, writeFile, unlink, appendFile, readFile } from 'node:fs/promises'
 import readline from 'node:readline'
 import Adapter from '../Adapter.mjs'
 import { TextMessage } from '../Message.mjs'
@@ -22,15 +22,37 @@ const showHelp = () => {
 }
 
 const bold = str => `\x1b[1m${str}\x1b[22m`
+const green = str => `\x1b[32m${str}\x1b[0m`
+const levelColors = {
+  error: '\x1b[31m',
+  warn: '\x1b[33m',
+  debug: '\x1b[35m',
+  info: '\x1b[34m',
+  trace: '\x1b[36m',
+  fatal: '\x1b[91m'
+}
+const reset = '\x1b[0m'
 
 class Shell extends Adapter {
   #rl = null
   constructor (robot) {
     super(robot)
     this.name = 'Shell'
+    const levels = ['trace', 'debug', 'info', 'warn', 'error', 'fatal']
+    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)
+      }
+    })
+    this.robot.on('scripts have loaded', () => {
+      this.#rl.prompt()
+    })
   }
 
   async send (envelope, ...strings) {
+    this.#rl.prompt()
     Array.from(strings).forEach(str => console.log(bold(str)))
   }
 
@@ -40,22 +62,25 @@ class Shell extends Adapter {
 
   async reply (envelope, ...strings) {
     strings = strings.map((s) => `${envelope.user.name}: ${s}`)
-    this.send(envelope, ...strings)
+    await this.send(envelope, ...strings)
   }
 
   async run () {
-    if (!fs.existsSync(historyPath)) {
-      fs.writeFileSync(historyPath, '')
-    }
-    const stats = fs.statSync(historyPath)
-    if (stats.size > historySize) {
-      fs.unlinkSync(historyPath)
+    try {
+      const stats = await stat(historyPath)
+      if (stats.size > historySize) {
+        await unlink(historyPath)
+        await writeFile(historyPath, '')
+      }
+    } catch (error) {
+      console.log(error)
+      await writeFile(historyPath, '')
     }
 
     this.#rl = readline.createInterface({
       input: this.robot.stdin ?? process.stdin,
       output: this.robot.stdout ?? process.stdout,
-      prompt: `${this.robot.name ?? this.robot.alias}> `,
+      prompt: green(`${this.robot.name ?? this.robot.alias}> `),
       completer
     })
     this.#rl.on('line', async (line) => {
@@ -69,6 +94,7 @@ class Shell extends Adapter {
         case '\\?':
         case 'help':
           showHelp()
+          this.#rl.prompt()
           break
         case '\\c':
         case 'clear':
@@ -95,14 +121,13 @@ class Shell extends Adapter {
 
     this.#rl.on('history', async (history) => {
       if (history.length === 0) return
-      await fs.promises.appendFile(historyPath, `${history[0]}\n`)
+      await appendFile(historyPath, `${history[0]}\n`)
     })
 
-    const existingHistory = (await fs.promises.readFile(historyPath, 'utf8')).split('\n')
+    const existingHistory = (await readFile(historyPath, 'utf8')).split('\n')
     existingHistory.reverse().forEach(line => this.#rl.history.push(line))
 
     try {
-      this.#rl.prompt()
       this.emit('connected', this)
     } catch (error) {
       console.log(error)