hubot 10.0.5 → 11.0.1

package/README.md

@@ -6,9 +6,8 @@
 
 # Hubot
 
-**Note: v10.0.4 contains the removal of CoffeeScript**
-
-Semver is looking for **BREAKING CHANGE** singular, not **BREAKING CHANGES**. As a result, the removal of CoffeeScript was marked as the `v10.0.4` release.
+**Note: v10.0.4 accidentaly contains the removal of CoffeeScript; v10.0.5 puts it back in**
+**Note: v11 removes CoffeeScript and converts this codebase to ESM**
 
 Hubot is a framework to build chat bots, modeled after GitHub's Campfire bot of the same name, hubot.
 He's pretty cool. He's [extendable with scripts](https://hubotio.github.io/hubot/docs#scripts) and can work

package/bin/{hubot.js → Hubot.mjs}

@@ -1,15 +1,13 @@
 'use strict'
 
-const fs = require('fs')
-const pathResolve = require('path').resolve
-
-const OptParse = require('../src/OptParse.js')
-
-const Hubot = require('..')
-const create = require('../src/GenHubot.js')
+import fs from 'node:fs'
+import { resolve as pathResolve } from 'node:path'
+import OptParse from '../src/OptParse.mjs'
+import Hubot from '../index.mjs'
+import create from '../src/GenHubot.mjs'
 
 const switches = [
-  ['-a', '--adapter HUBOT_ADAPTER', 'The Adapter to use, e.g. "shell" (to load the default hubot shell adapter)'],
+  ['-a', '--adapter HUBOT_ADAPTER', 'The Adapter to use, e.g. "Shell" (to load the default hubot Shell adapter)'],
   ['-f', '--file HUBOT_FILE', 'Path to adapter file, e.g. "./adapters/CustomAdapter.mjs"'],
   ['-c', '--create HUBOT_CREATE', 'Create a deployable hubot'],
   ['-d', '--disable-httpd HUBOT_HTTPD', 'Disable the HTTP server'],
@@ -97,13 +95,13 @@ if (options.file) {
 }
 
 const robot = Hubot.loadBot(options.adapter, options.enableHttpd, options.name, options.alias)
-module.exports = robot
+export default robot
 
 async function loadScripts () {
   await robot.load(pathResolve('.', 'scripts'))
   await robot.load(pathResolve('.', 'src', 'scripts'))
 
-  loadExternalScripts()
+  await loadExternalScripts()
 
   const tasks = options.scripts.map((scriptPath) => {
     if (scriptPath[0] === '/') {
@@ -115,25 +113,19 @@ async function loadScripts () {
   await Promise.all(tasks)
 }
 
-function loadExternalScripts () {
+async function loadExternalScripts () {
   const externalScripts = pathResolve('.', 'external-scripts.json')
-
-  if (!fs.existsSync(externalScripts)) {
-    return
-  }
-
-  fs.readFile(externalScripts, function (error, data) {
-    if (error) {
-      throw error
-    }
-
+  try {
+    const data = await fs.promises.readFile(externalScripts)
     try {
       robot.loadExternalScripts(JSON.parse(data))
     } catch (error) {
       console.error(`Error parsing JSON data from external-scripts.json: ${error}`)
       process.exit(1)
     }
-  })
+  } catch (e) {
+    robot.logger.info('No external-scripts.json found. Skipping.')
+  }
 }
 
 (async () => {

package/bin/hubot

@@ -1,9 +1,3 @@
-#!/usr/bin/env coffee
+#!/usr/bin/env node
 
-# While all other files have been converted to JavaScript via https://github.com/github/hubot/pull/1347,
-# we left the `bin/hubot` file to remain in CoffeeScript in order prevent
-# breaking existing 3rd party adapters of which some are still written in
-# CoffeeScript themselves. We will deprecate and eventually remove this file
-# in a future version of hubot
-
-require './hubot.js'
+import('./Hubot.mjs').then(async ({ default: robot }) => {})

package/index.mjs

@@ -0,0 +1,52 @@
+'use strict'
+
+import User from './src/User.mjs'
+import Brain from './src/Brain.mjs'
+import Robot from './src/Robot.mjs'
+import Adapter from './src/Adapter.mjs'
+import Response from './src/Response.mjs'
+import Middleware from './src/Middleware.mjs'
+import { Listener, TextListener } from './src/Listener.mjs'
+import { TextMessage, EnterMessage, LeaveMessage, TopicMessage, CatchAllMessage, Message } from './src/Message.mjs'
+import { DataStore, DataStoreUnavailable } from './src/DataStore.mjs'
+
+const loadBot = (adapter, enableHttpd, name, alias) => new Robot(adapter, enableHttpd, name, alias)
+export {
+  Adapter,
+  User,
+  Brain,
+  Robot,
+  Response,
+  Listener,
+  TextListener,
+  Message,
+  TextMessage,
+  EnterMessage,
+  LeaveMessage,
+  TopicMessage,
+  CatchAllMessage,
+  DataStore,
+  DataStoreUnavailable,
+  Middleware,
+  loadBot
+}
+
+export default {
+  Adapter,
+  User,
+  Brain,
+  Robot,
+  Response,
+  Listener,
+  TextListener,
+  Message,
+  TextMessage,
+  EnterMessage,
+  LeaveMessage,
+  TopicMessage,
+  CatchAllMessage,
+  DataStore,
+  DataStoreUnavailable,
+  Middleware,
+  loadBot
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hubot",
-  "version": "10.0.5",
+  "version": "11.0.1",
   "author": "hubot",
   "keywords": [
     "github",
@@ -15,7 +15,6 @@
     "url": "https://github.com/hubotio/hubot.git"
   },
   "dependencies": {
-    "coffeescript": "^2.7.0",
     "connect-multiparty": "^2.2.0",
     "express": "^4.18.2",
     "express-basic-auth": "^1.2.1",
@@ -28,7 +27,7 @@
     "node": ">= 18",
     "npm": ">= 9"
   },
-  "main": "./index",
+  "main": "./index.mjs",
   "bin": {
     "hubot": "./bin/hubot"
   },

package/src/{adapter.js → Adapter.mjs}

@@ -1,6 +1,6 @@
 'use strict'
 
-const EventEmitter = require('events').EventEmitter
+import EventEmitter from 'node:events'
 
 class Adapter extends EventEmitter {
   // An adapter is a specific interface to a chat source for robots.
@@ -136,4 +136,4 @@ class Adapter extends EventEmitter {
   }
 }
 
-module.exports = Adapter
+export default Adapter

package/src/{brain.js → Brain.mjs}

@@ -1,8 +1,7 @@
 'use strict'
 
-const EventEmitter = require('events').EventEmitter
-
-const User = require('./user')
+import EventEmitter from 'node:events'
+import User from './User.mjs'
 
 // If necessary, reconstructs a User object. Returns either:
 //
@@ -239,4 +238,4 @@ class Brain extends EventEmitter {
   }
 }
 
-module.exports = Brain
+export default Brain

package/src/{datastore.js → DataStore.mjs}

@@ -1,6 +1,6 @@
 'use strict'
 
-class DataStore {
+export class DataStore {
   // Represents a persistent, database-backed storage for the robot. Extend this.
   //
   // Returns a new Datastore with no storage.
@@ -83,9 +83,9 @@ class DataStore {
   }
 }
 
-class DataStoreUnavailable extends Error {}
+export class DataStoreUnavailable extends Error {}
 
-module.exports = {
+export default {
   DataStore,
   DataStoreUnavailable
 }

package/src/{GenHubot.js → GenHubot.mjs}

@@ -1,6 +1,6 @@
-const { spawnSync } = require('child_process')
-const File = require('fs')
-const path = require('path')
+import { spawnSync } from 'node:child_process'
+import File from 'node:fs'
+import path from 'node:path'
 
 function runCommands (hubotDirectory, options) {
   options.hubotInstallationPath = options?.hubotInstallationPath ?? 'hubot'
@@ -13,8 +13,19 @@ function runCommands (hubotDirectory, options) {
   const envFilePath = path.resolve(process.cwd(), '.env')
   process.chdir(hubotDirectory)
 
-  spawnSync('npm', ['init', '-y'])
-  spawnSync('npm', ['i', options.hubotInstallationPath].concat(options.adapter, 'hubot-help', 'hubot-rules', 'hubot-diagnostics'))
+  let output = spawnSync('npm', ['init', '-y'])
+  console.log('npm init', output.stderr.toString())
+  if (options.hubotInstallationPath !== 'hubot') {
+    output = spawnSync('npm', ['pack', `${options.hubotInstallationPath}`])
+    console.log('npm pack', output.stderr.toString(), output.stdout.toString())
+    const customHubotPackage = JSON.parse(File.readFileSync(`${options.hubotInstallationPath}/package.json`, 'utf8'))
+    output = spawnSync('npm', ['i', `${customHubotPackage.name}-${customHubotPackage.version}.tgz`])
+    console.log(`npm i ${customHubotPackage.name}-${customHubotPackage.version}.tgz`, output.stderr.toString(), output.stdout.toString())
+  } else {
+    output = spawnSync('npm', ['i', 'hubot@latest'])
+  }
+  output = spawnSync('npm', ['i', 'hubot-help@latest', 'hubot-rules@latest', 'hubot-diagnostics@latest'].concat([options.adapter]).filter(Boolean))
+  console.log('npm i', output.stderr.toString(), output.stdout.toString())
   spawnSync('mkdir', ['scripts'])
   spawnSync('touch', ['external-scripts.json'])
 
@@ -50,6 +61,7 @@ export default (robot) => {
   packageJson.scripts = {
     start: 'hubot'
   }
+  packageJson.description = 'A simple helpful robot for your Company'
   if (options.adapter) {
     packageJson.scripts.start += ` --adapter ${options.adapter}`
   }
@@ -79,7 +91,7 @@ export default (robot) => {
     console.log('.env file not found, continuing to the next operation.')
   }
 }
-module.exports = (hubotDirectory, options) => {
+export default (hubotDirectory, options) => {
   try {
     runCommands(hubotDirectory, options)
   } catch (error) {

package/src/{httpclient.js → HttpClient.mjs}

@@ -33,10 +33,10 @@ Implement a phased approach to deprecate `robot.http` all together in favor of `
 2. Add a deprecation warning to `robot.http`
 3. Remove `robot.http` in a future release
 */
-const path = require('path')
-const http = require('http')
-const https = require('https')
-const qs = require('querystring')
+import path from 'node:path'
+import http from 'node:http'
+import https from 'node:https'
+import qs from 'node:querystring'
 
 const nonPassThroughOptions = [
   'headers', 'hostname', 'encoding', 'auth', 'port',
@@ -308,5 +308,8 @@ const reduce = function (a, b) {
   }
   return a
 }
-
-exports.create = (url, options) => new ScopedClient(url, options)
+export default {
+  create (url, options) {
+    return new ScopedClient(url, options)
+  }
+}

package/src/{listener.js → Listener.mjs}

@@ -1,9 +1,8 @@
 'use strict'
 
-const inspect = require('util').inspect
-
-const TextMessage = require('./message').TextMessage
-const Middleware = require('./middleware')
+import { inspect } from 'node:util'
+import { TextMessage } from './Message.mjs'
+import Middleware from './Middleware.mjs'
 
 class Listener {
   // Listeners receive every message from the chat source and decide if they
@@ -108,7 +107,7 @@ class TextListener extends Listener {
   }
 }
 
-module.exports = {
+export {
   Listener,
   TextListener
 }

package/src/{message.js → Message.mjs}

@@ -1,6 +1,6 @@
 'use strict'
 
-class Message {
+export class Message {
   // Represents an incoming message from the chat.
   //
   // user - A User instance that sent the message.
@@ -18,7 +18,7 @@ class Message {
   }
 }
 
-class TextMessage extends Message {
+export class TextMessage extends Message {
   // Represents an incoming message from the chat.
   //
   // user - A User instance that sent the message.
@@ -52,23 +52,23 @@ class TextMessage extends Message {
 // user - A User instance for the user who entered.
 // text - Always null.
 // id   - A String of the message ID.
-class EnterMessage extends Message {}
+export class EnterMessage extends Message {}
 
 // Represents an incoming user exit notification.
 //
 // user - A User instance for the user who left.
 // text - Always null.
 // id   - A String of the message ID.
-class LeaveMessage extends Message {}
+export class LeaveMessage extends Message {}
 
 // Represents an incoming topic change notification.
 //
 // user - A User instance for the user who changed the topic.
 // text - A String of the new topic
 // id   - A String of the message ID.
-class TopicMessage extends TextMessage {}
+export class TopicMessage extends TextMessage {}
 
-class CatchAllMessage extends Message {
+export class CatchAllMessage extends Message {
   // Represents a message that no matchers matched.
   //
   // message - The original message.
@@ -78,7 +78,7 @@ class CatchAllMessage extends Message {
   }
 }
 
-module.exports = {
+export default {
   Message,
   TextMessage,
   EnterMessage,

package/src/{middleware.js → Middleware.mjs}

@@ -42,4 +42,4 @@ class Middleware {
   }
 }
 
-module.exports = Middleware
+export default Middleware

package/src/{OptParse.js → OptParse.mjs}

@@ -1,4 +1,4 @@
-const EventEmitter = require('node:events')
+import EventEmitter from 'node:events'
 class OptParse extends EventEmitter {
   constructor (switches) {
     super()
@@ -41,4 +41,4 @@ ${this.switches.map(([key, description]) => `  ${key}, ${description}`).join('\n
   }
 }
 
-module.exports = OptParse
+export default OptParse

package/src/{response.js → Response.mjs}

@@ -123,4 +123,4 @@ class Response {
   }
 }
 
-module.exports = Response
+export default Response

package/src/{robot.js → Robot.mjs}

@@ -1,21 +1,23 @@
 'use strict'
-const EventEmitter = require('events').EventEmitter
-const fs = require('fs')
-const path = require('path')
-const pathToFileURL = require('url').pathToFileURL
 
-const pino = require('pino')
-const HttpClient = require('./httpclient')
-
-const Brain = require('./brain')
-const Response = require('./response')
-const Listener = require('./listener')
-const Message = require('./message')
-const Middleware = require('./middleware')
-
-const HUBOT_DEFAULT_ADAPTERS = ['campfire', 'shell']
+import EventEmitter from 'node:events'
+import fs from 'node:fs'
+import path from 'node:path'
+import { pathToFileURL, fileURLToPath } from 'node:url'
+import pino from 'pino'
+import HttpClient from './HttpClient.mjs'
+import Brain from './Brain.mjs'
+import Response from './Response.mjs'
+import { Listener, TextListener } from './Listener.mjs'
+import Message from './Message.mjs'
+import Middleware from './Middleware.mjs'
+
+const HUBOT_DEFAULT_ADAPTERS = ['Campfire', 'Shell']
 const HUBOT_DOCUMENTATION_SECTIONS = ['description', 'dependencies', 'configuration', 'commands', 'notes', 'author', 'authors', 'examples', 'tags', 'urls']
 
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+
 class Robot {
   // Robots receive messages from a chat source (Campfire, irc, etc), and
   // dispatch them to matching listeners.
@@ -37,6 +39,14 @@ class Robot {
     this.brain = new Brain(this)
     this.alias = alias
     this.adapter = null
+    this.adaptername = 'Shell'
+    if (adapter && typeof (adapter) === 'object') {
+      this.adapter = adapter
+      this.adapterName = adapter.name ?? adapter.constructor.name
+    } else {
+      this.adapterName = adapter ?? this.adaptername
+    }
+
     this.shouldEnableHttpd = httpd ?? true
     this.datastore = null
     this.Response = Response
@@ -56,7 +66,6 @@ class Robot {
     this.globalHttpOptions = {}
 
     this.parseVersion()
-    this.adapterName = adapter ?? 'shell'
     this.errorHandlers = []
 
     this.on('error', (err, res) => {
@@ -78,7 +87,7 @@ class Robot {
   //
   // Returns nothing.
   listen (matcher, options, callback) {
-    this.listeners.push(new Listener.Listener(this, matcher, options, callback))
+    this.listeners.push(new Listener(this, matcher, options, callback))
   }
 
   // Public: Adds a Listener that attempts to match incoming messages based on
@@ -91,7 +100,7 @@ class Robot {
   //
   // Returns nothing.
   hear (regex, options, callback) {
-    this.listeners.push(new Listener.TextListener(this, regex, options, callback))
+    this.listeners.push(new TextListener(this, regex, options, callback))
   }
 
   // Public: Adds a Listener that attempts to match incoming messages directed
@@ -320,24 +329,22 @@ class Robot {
   }
 
   async loadmjs (filePath) {
-    const script = await import(pathToFileURL(filePath))
+    const forImport = this.prepareForImport(filePath)
+    const script = await import(forImport)
     if (typeof script?.default === 'function') {
       script.default(this)
     } else {
-      this.logger.warning(`Expected ${filePath} to assign a function to export default, got ${typeof script}`)
+      this.logger.warning(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to export default, got ${typeof script}`)
     }
   }
 
-  async loadcoffee (filePath) {
-    return await this.loadjs(filePath)
-  }
-
   async loadjs (filePath) {
-    const script = require(filePath)
+    const forImport = this.prepareForImport(filePath)
+    const script = (await import(forImport)).default
     if (typeof script === 'function') {
       script(this)
     } else {
-      this.logger.warning(`Expected ${filePath} to assign a function to module.exports, got ${typeof script}`)
+      this.logger.warning(`Expected ${filePath} (after preparing for import ${forImport}) to assign a function to module.exports, got ${typeof script}`)
     }
   }
 
@@ -352,7 +359,7 @@ class Robot {
     const full = path.join(filepath, path.basename(filename))
 
     // see https://github.com/hubotio/hubot/issues/1355
-    if (['js', 'mjs', 'coffee'].indexOf(ext) === -1) {
+    if (['js', 'mjs'].indexOf(ext) === -1) {
       this.logger.debug(`Skipping unsupported file type ${full}`)
       return
     }
@@ -386,15 +393,19 @@ class Robot {
   // packages - An Array of packages containing hubot scripts to load.
   //
   // Returns nothing.
-  loadExternalScripts (packages) {
+  async loadExternalScripts (packages) {
     this.logger.debug('Loading external-scripts from npm packages')
 
     try {
       if (Array.isArray(packages)) {
-        return packages.forEach(pkg => require(pkg)(this))
+        for await (const pkg of packages) {
+          (await import(pkg)).default(this)
+        }
+        return
+      }
+      for await (const key of Object.keys(packages)) {
+        (await import(key)).default(this, packages[key])
       }
-
-      Object.keys(packages).forEach(key => require(key)(this, packages[key]))
     } catch (error) {
       this.logger.error(`Error loading scripts from npm package - ${error.stack}`)
       throw error
@@ -413,9 +424,9 @@ class Robot {
     const limit = process.env.EXPRESS_LIMIT || '100kb'
     const paramLimit = parseInt(process.env.EXPRESS_PARAMETER_LIMIT) || 1000
 
-    const express = require('express')
-    const basicAuth = require('express-basic-auth')
-    const multipart = require('connect-multiparty')
+    const express = (await import('express')).default
+    const basicAuth = (await import('express-basic-auth')).default
+    const multipart = (await import('connect-multiparty')).default
 
     const app = express()
 
@@ -475,19 +486,23 @@ class Robot {
   //
   // Returns nothing.
   async loadAdapter (adapterPath = null) {
+    if (this.adapter) {
+      this.adapter = await this.adapter.use(this)
+      return
+    }
     this.logger.debug(`Loading adapter ${adapterPath ?? 'from npmjs:'} ${this.adapterName}`)
-    const ext = path.extname(adapterPath ?? '') ?? '.js'
+    const ext = path.extname(adapterPath ?? '') ?? '.mjs'
     try {
       if (Array.from(HUBOT_DEFAULT_ADAPTERS).indexOf(this.adapterName) > -1) {
-        this.adapter = this.requireAdapterFrom(path.resolve(path.join(__dirname, 'adapters', this.adapterName)))
-      } else if (['.js', '.cjs', '.coffee'].includes(ext)) {
-        this.adapter = this.requireAdapterFrom(path.resolve(adapterPath))
+        this.adapter = await this.requireAdapterFrom(path.resolve(path.join(__dirname, 'adapters', `${this.adapterName}.mjs`)))
+      } else if (['.js', '.cjs'].includes(ext)) {
+        this.adapter = await this.requireAdapterFrom(path.resolve(adapterPath))
       } else if (['.mjs'].includes(ext)) {
-        this.adapter = await this.importAdapterFrom(pathToFileURL(path.resolve(adapterPath)).href)
+        this.adapter = await this.importAdapterFrom(path.resolve(adapterPath))
       } else {
         const adapterPathInCurrentWorkingDirectory = this.adapterName
         try {
-          this.adapter = this.requireAdapterFrom(adapterPathInCurrentWorkingDirectory)
+          this.adapter = await this.requireAdapterFrom(adapterPathInCurrentWorkingDirectory)
         } catch (err) {
           if (err.name === 'SyntaxError') {
             this.adapter = await this.importAdapterFrom(adapterPathInCurrentWorkingDirectory)
@@ -502,12 +517,13 @@ class Robot {
     }
   }
 
-  requireAdapterFrom (adapaterPath) {
-    return require(adapaterPath).use(this)
+  async requireAdapterFrom (adapaterPath) {
+    return await this.importAdapterFrom(adapaterPath)
   }
 
   async importAdapterFrom (adapterPath) {
-    return await (await import(adapterPath)).default.use(this)
+    const forImport = this.prepareForImport(adapterPath)
+    return await (await import(forImport)).default.use(this)
   }
 
   // Public: Help Commands for Running Scripts.
@@ -519,12 +535,12 @@ class Robot {
 
   // Private: load help info from a loaded script.
   //
-  // path - A String path to the file on disk.
+  // filePath - A String path to the file on disk.
   //
   // Returns nothing.
-  parseHelp (path) {
+  parseHelp (filePath) {
     const scriptDocumentation = {}
-    const body = fs.readFileSync(require.resolve(path), 'utf-8')
+    const body = fs.readFileSync(path.resolve(filePath), 'utf-8')
 
     const useStrictHeaderRegex = /^["']use strict['"];?\s+/
     const lines = body.replace(useStrictHeaderRegex, '').split(/(?:\n|\r\n|\r)/)
@@ -657,11 +673,15 @@ class Robot {
     this.events.removeAllListeners()
   }
 
+  prepareForImport (filePath) {
+    return pathToFileURL(filePath)
+  }
+
   // Public: The version of Hubot from npm
   //
   // Returns a String of the version number.
   parseVersion () {
-    const pkg = require(path.join(__dirname, '..', 'package.json'))
+    const pkg = fs.readFileSync(path.join(__dirname, '..', 'package.json'))
     this.version = pkg.version
 
     return this.version
@@ -720,8 +740,6 @@ class Robot {
   }
 }
 
-module.exports = Robot
-
 function isCatchAllMessage (message) {
   return message instanceof Message.CatchAllMessage
 }
@@ -748,9 +766,7 @@ function removeCommentPrefix (line) {
   return line.replace(/^[#/]+\s*/, '')
 }
 
-function extend (obj/* , ...sources */) {
-  const sources = [].slice.call(arguments, 1)
-
+function extend (obj, ...sources) {
   sources.forEach((source) => {
     if (typeof source !== 'object') {
       return
@@ -763,3 +779,5 @@ function extend (obj/* , ...sources */) {
 
   return obj
 }
+
+export default Robot

package/src/{user.js → User.mjs}

@@ -1,6 +1,6 @@
 'use strict'
 
-const DataStoreUnavailable = require('./datastore').DataStoreUnavailable
+import { DataStoreUnavailable } from './DataStore.mjs'
 
 class User {
   // Represents a participating user in the chat.
@@ -62,4 +62,4 @@ class User {
   }
 }
 
-module.exports = User
+export default User

package/src/adapters/{campfire.js → Campfire.mjs}

@@ -1,16 +1,9 @@
 'use strict'
 
-const HTTPS = require('https')
-const EventEmitter = require('events').EventEmitter
-
-const Adapter = require('../adapter')
-
-const Message = require('../message')
-
-const TextMessage = Message.TextMessage
-const EnterMessage = Message.EnterMessage
-const LeaveMessage = Message.LeaveMessage
-const TopicMessage = Message.TopicMessage
+import HTTPS from 'node:https'
+import EventEmitter from 'node:events'
+import Adapter from '../Adapter.mjs'
+import { TextMessage, EnterMessage, LeaveMessage, TopicMessage } from '../Message.mjs'
 
 class Campfire extends Adapter {
   send (envelope/* , ...strings */) {
@@ -157,8 +150,6 @@ class Campfire extends Adapter {
   }
 }
 
-exports.use = robot => new Campfire(robot)
-
 class CampfireStreaming extends EventEmitter {
   constructor (options, robot) {
     super()
@@ -392,3 +383,9 @@ class CampfireStreaming extends EventEmitter {
     return request.on('error', err => logger.error(`Campfire request error: ${err}`))
   }
 }
+
+export default {
+  use (robot) {
+    return new Campfire(robot)
+  }
+}

package/src/adapters/{shell.js → Shell.mjs}

@@ -1,9 +1,9 @@
 'use strict'
 
-const fs = require('fs')
-const readline = require('node:readline')
-const Adapter = require('../adapter')
-const { TextMessage } = require('../message')
+import fs from 'node:fs'
+import readline from 'node:readline'
+import Adapter from '../Adapter.mjs'
+import { TextMessage } from '../Message.mjs'
 
 const historySize = process.env.HUBOT_SHELL_HISTSIZE != null ? parseInt(process.env.HUBOT_SHELL_HISTSIZE) : 1024
 const historyPath = '.hubot_history'
@@ -16,7 +16,7 @@ const completer = line => {
 }
 const showHelp = () => {
   console.log('usage:')
-  console.log('\\q, exit - close shell and exit')
+  console.log('\\q, exit - close Shell and exit')
   console.log('\\?, help - show this help')
   console.log('\\c, clear - clear screen')
 }
@@ -112,4 +112,8 @@ class Shell extends Adapter {
 
 // Prevent output buffer "swallowing" every other character on OSX / Node version > 16.19.0.
 process.stdout._handle.setBlocking(false)
-exports.use = robot => new Shell(robot)
+export default {
+  use (robot) {
+    return new Shell(robot)
+  }
+}

package/src/datastores/{memory.js → Memory.mjs}

@@ -1,6 +1,6 @@
 'use strict'
 
-const DataStore = require('../datastore.js').DataStore
+import { DataStore } from '../DataStore.mjs'
 
 class InMemoryDataStore extends DataStore {
   constructor (robot) {
@@ -20,4 +20,4 @@ class InMemoryDataStore extends DataStore {
   }
 }
 
-module.exports = InMemoryDataStore
+export default InMemoryDataStore

package/es2015.js

@@ -1,31 +0,0 @@
-'use strict'
-
-const User = require('./src/user')
-const Brain = require('./src/brain')
-const Robot = require('./src/robot')
-const Adapter = require('./src/adapter')
-const Response = require('./src/response')
-const Listener = require('./src/listener')
-const Message = require('./src/message')
-const DataStore = require('./src/datastore')
-
-module.exports = {
-  User,
-  Brain,
-  Robot,
-  Adapter,
-  Response,
-  Listener: Listener.Listener,
-  TextListener: Listener.TextListener,
-  Message: Message.Message,
-  TextMessage: Message.TextMessage,
-  EnterMessage: Message.EnterMessage,
-  LeaveMessage: Message.LeaveMessage,
-  TopicMessage: Message.TopicMessage,
-  CatchAllMessage: Message.CatchAllMessage,
-  DataStore: DataStore.DataStore,
-  DataStoreUnavailable: DataStore.DataStoreUnavailable,
-  loadBot (adapter, enableHttpd, name, alias) {
-    return new module.exports.Robot(adapter, enableHttpd, name, alias)
-  }
-}

package/index.js

@@ -1,40 +0,0 @@
-'use strict'
-require('coffeescript/register')
-
-const inherits = require('util').inherits
-
-const hubotExport = require('./es2015')
-
-// make all es2015 class declarations compatible with CoffeeScript’s extend
-// see https://github.com/hubotio/evolution/pull/4#issuecomment-306437501
-module.exports = Object.keys(hubotExport).reduce((map, current) => {
-  if (current !== 'loadBot') {
-    map[current] = makeClassCoffeeScriptCompatible(hubotExport[current])
-  } else {
-    map[current] = hubotExport[current]
-  }
-  return map
-}, {})
-
-function makeClassCoffeeScriptCompatible (klass) {
-  function CoffeeScriptCompatibleClass () {
-    const Hack = Function.prototype.bind.apply(klass, [null].concat([].slice.call(arguments)))
-    const instance = new Hack()
-
-    // pass methods from child to returned instance
-    for (const key in this) {
-      instance[key] = this[key]
-    }
-
-    // support for constructor methods which call super()
-    // in which this.* properties are set
-    for (const key in instance) {
-      this[key] = instance[key]
-    }
-
-    return instance
-  }
-  inherits(CoffeeScriptCompatibleClass, klass)
-
-  return CoffeeScriptCompatibleClass
-}