gasolin / webbybot Goto Github PK

View Code? Open in Web Editor NEW

68.0 7.0 11.0 179 KB

chatterbot written in es6 (es2015), fork from hubot

JavaScript 99.14% Shell 0.86%

chat chatbot bot hubot

webbybot's People

Contributors

Stargazers

Watchers

Forkers

raylin metalvegetarianoprogresivo scarletsky gitter-badger lawrips deleav chrisgalvan hashashin dkkv clickjankin

webbybot's Issues

Porting motivations

I saw a note somewhere about "porting Hubot from coffeescript to es6". What are the motivations for doing so vs just building on top of Hubot as-is?

move build-in shell adapter out of webbybot

port user.js

fix webpack/node require conflict issue

webpack treat all require as webpack's require, but in some cases we need node require to require plugins dynamically

port hubot-mock-adapter to es6 and remove lodash dependency

https://github.com/blalor/hubot-mock-adapter

replace HTTPClient to fetch-wrapped lib

We could add HTTPClient compatible API based on stable
https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API/Using_Fetch
http://www.w3ctech.com/topic/854
https://davidwalsh.name/fetch

bundled web dashboard

provide web interface to easy config the chatterbot

respond result is wrong

$ npm install hubot-calculator
$ npm run build
$ ./bin/webby.js
webby> webby calc 1+1

Expected result:

webby > 2

Actual result

webby > 0

The return result in hubot-calculator is correct (2), so there is something wrong in respond or middleware module.

use fetch instead of scoped-http-client

deprecated https://github.com/technoweenie/node-scoped-http-client because its no longer maintained

https://www.npmjs.com/package/node-fetch (preferred)

https://github.com/request/request

webbybot generator

to generate custom webbybot and javascript plugin

make HTTPClient replaceable

such as https://github.com/github/hubot/pull/1134/files

related to #54

port robot.js

fix robot tests

there's some skipped tests in robot_test related to middleware and listener

broadcast local address through mdns

broadcast webby.local so user can type that in searchbar and connect to webby web dashboard.

refactor: move loadFile and varients to expander.js file

support set command alias

port brain.js

remove dependency of babel-plugin-add-module-exports

we use babel-plugin-add-module-exports to smooth the porting effort to es6 modules, now all modules are es6 ready, we could start remove dependency of babel-plugin-add-module-exports

add coverage report

support multiple adaptors

add unit test framework

mock hubot require to webbybot to use adapters directly

currently webbybot needs to modify the require('hubot') statement from every hubot-adapters, its not that convenient.

We can use mockery or mock-require to proxy hubot require to webbybot, so every hubot requirement will be redirected to webbybot

use ES6 for..of loop to replace forEach and for..in loop。

buildin schedule support

https://www.npmjs.com/package/hubot-cronjob
https://github.com/matsukaz/hubot-schedule

wrote with es6

expose webby as library

refer to http://webpack.github.io/docs/configuration.html#output-librarytarget

reconfig webpack to expose webby as commonjs library

Cannot read property 'robot' of undefined with hubot-timezone

input webby time in london

and got debug log with error:

[Wed Apr 13 2016 13:58:51 GMT+0800 (CST)] DEBUG Executing listener callback for Message '${message}'
[Wed Apr 13 2016 13:58:51 GMT+0800 (CST)] ERROR TypeError: Cannot read property 'robot' of undefined
  at http (/home/gasolin/webby-confbot/node_modules/webbybot/dist/src/response.js:224:18)
  at getTimezoneInfo (/home/gasolin/webby-confbot/node_modules/hubot-timezone/src/timezone.coffee:37:3)
  at sendLocalTime (/home/gasolin/webby-confbot/node_modules/hubot-timezone/src/timezone.coffee:77:5)
  at convertTime (/home/gasolin/webby-confbot/node_modules/hubot-timezone/src/timezone.coffee:92:5)
  at TextListener.callback (/home/gasolin/webby-confbot/node_modules/hubot-timezone/src/timezone.coffee:116:5)
  at executeListener (/home/gasolin/webby-confbot/node_modules/webbybot/dist/src/listener.js:116:19)
  at allDone (/home/gasolin/webby-confbot/node_modules/webbybot/dist/src/middleware.js:85:16)
  at /home/gasolin/webby-confbot/node_modules/webbybot/node_modules/async/lib/async.js:380:13
  at /home/gasolin/webby-confbot/node_modules/webbybot/node_modules/async/lib/async.js:52:16
  at iterate (/home/gasolin/webby-confbot/node_modules/webbybot/node_modules/async/lib/async.js:260:24)
  at Object.async.forEachOfSeries.async.eachOfSeries (/home/gasolin/webby-confbot/node_modules/webbybot/node_modules/async/lib/async.js:281:9)
  at Object.async.inject.async.foldl.async.reduce (/home/gasolin/webby-confbot/node_modules/webbybot/node_modules/async/lib/async.js:374:15)
  at /home/gasolin/webby-confbot/node_modules/webbybot/dist/src/middleware.js:91:22
  at nextTickCallbackWith0Args (node.js:420:9)
  at process._tickCallback (node.js:349:13)