Persist shell adapter history between invocations of hubot

package.json

@@ -3,41 +3,37 @@
   "version": "2.9.3",
 
   "author": "hubot",
-
   "keywords": [
     "github",
     "hubot",
     "campfire",
     "bot"
   ],
-
   "description": "A simple helpful robot for your Company",
-
-  "licenses": [{
-    "type": "MIT",
-    "url": "https://raw.github.com/github/hubot/master/LICENSE"
-  }],
-
-  "repository" : {
+  "licenses": [
+    {
+      "type": "MIT",
+      "url": "https://raw.github.com/github/hubot/master/LICENSE"
+    }
+  ],
+  "repository": {
     "type": "git",
     "url": "https://github.com/github/hubot.git"
   },
-
   "dependencies": {
-    "coffee-script":      "1.6.3",
-    "optparse":           "1.0.4",
+    "coffee-script": "1.6.3",
+    "optparse": "1.0.4",
     "scoped-http-client": "0.9.8",
-    "log":                "1.4.0",
-    "express":            "3.18.1"
+    "log": "1.4.0",
+    "express": "3.3.4",
+    "express": "3.18.1",
+    "readline-history": "~1.2.0"
   },
-
   "engines": {
     "node": ">= 0.8.x",
-    "npm":  ">= 1.1.x"
+    "npm": ">= 1.1.x"
   },
-
   "main": "./index",
-
   "bin": {
     "hubot": "./bin/hubot"
   }

src/adapters/shell.coffee

@@ -1,9 +1,15 @@
-Readline = require 'readline'
+Readline = require 'readline-history'
 
 Robot         = require '../robot'
 Adapter       = require '../adapter'
 {TextMessage} = require '../message'
 
+
+historySize = if process.env.HUBOT_SHELL_HISTSIZE?
+                parseInt(process.env.HUBOT_SHELL_HISTSIZE)
+              else
+                1024
+
 class Shell extends Adapter
   send: (envelope, strings...) ->
     unless process.platform is 'win32'
@@ -20,29 +26,41 @@ class Shell extends Adapter
     @send envelope, strings...
 
   run: ->
-    self = @
     stdin = process.openStdin()
     stdout = process.stdout
 
-    @repl = Readline.createInterface stdin, stdout, null
+    @repl = null
+    Readline.createInterface
+      path: ".hubot_history",
+      input: stdin,
+      output: stdout,
+      maxLength: historySize, # number of entries
+      next: (rl) =>
+        @repl = rl
+        @repl.on 'close', =>
+          stdin.destroy()
+          @robot.shutdown()
+          process.exit 0
 
-    @repl.on 'close', =>
-      stdin.destroy()
-      @robot.shutdown()
-      process.exit 0
+        @repl.on 'line', (buffer) =>
 
-    @repl.on 'line', (buffer) =>
-      @repl.close() if buffer.toLowerCase() is 'exit'
-      @repl.prompt()
-      user_id = parseInt(process.env.HUBOT_SHELL_USER_ID or '1')
-      user_name = process.env.HUBOT_SHELL_USER_NAME or 'Shell'
-      user = @robot.brain.userForId user_id, name: user_name, room: 'Shell'
-      @receive new TextMessage user, buffer, 'messageId'
+          switch buffer.toLowerCase()
+            when "exit"
+              @repl.close()
+            when "history"
+              stdout.write "#{line}\n" for line in @repl.history
+            else
+              user_id = parseInt(process.env.HUBOT_SHELL_USER_ID or '1')
+              user_name = process.env.HUBOT_SHELL_USER_NAME or 'Shell'
+              user = @robot.brain.userForId user_id, name: user_name, room: 'Shell'
+              @receive new TextMessage user, buffer, 'messageId'
+          @repl.prompt(true)
 
-    self.emit 'connected'
+        @emit 'connected'
+
+        @repl.setPrompt "#{@robot.name}> "
+        @repl.prompt(true)
 
-    @repl.setPrompt "#{@robot.name}> "
-    @repl.prompt()
 
 exports.use = (robot) ->
   new Shell robot