hubotでscriptをcronでまわすには・・？

Question

hubot(slack)でcronをまわして定期的にslackの任意のチャンネルにshellの出力結果を吐き出したいのですが、coffee scriptの書き方が今ひとつわかりません。

module.exports = (robot) ->
  cronJob = new cronJob('*/10 * * * *', () ->
    envelope = room: "general"
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    msg.send "Command: #{command}"
    @exec command, (error, stdout, stderr) ->
      msg.send error if error?
      msg.send stdout if stdout?
      msg.send stderr if stderr?
  )
cronJob.start()

こんな感じでかいたみたのですが、実行されていないようでどこか間違っているような気がします。
アドバイスお願い致します。。

追記： coffee scriptを書き換えてみました。

cron = require('cron').CronJob

module.exports = (robot) ->
  new cron '*/1 * * * *', () =>
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    @exec command, (error, stdout, stderr) ->
      robot.send {room: "#general"}, "stdout"
  , null, true, "Asia/Tokyo"

これでshellの実行結果がslackに投稿されれば良いのですが、stdoutの部分が展開されず、文字列として表示されてしまいます。。。
(error,stdout,stderr)の部分がどういうものなのか理解しきれていないのですが、以下のようなコードの場合は動作するので、参考にしてみたもののも上手くいきません＿|￣|○

module.exports = (robot) ->
  robot.respond /artisan_up/, (msg) ->
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    msg.send "Command: #{command}"
    @exec command, (error, stdout, stderr) ->
      msg.send error if error?
      msg.send stdout if stdout?
      msg.send stderr if stderr?

追記２:shell(hoge.sh)については以下のとおりです。

logwatch --print

Answer 1

cron = require('cron').CronJob

module.exports = (robot) ->
  new cron '*/1 * * * *', () =>
    @exec = require('child_process').exec
    command = "/webyone_hubot/scripts/shell/logwatch.sh"
    @exec command, (error, stdout, stderr) ->
      robot.send {room: "#general"}, stdout
  , null, true, "Asia/Tokyo"

これで動きましたｗ
このコードが動くのではないかと最初踏んでいたのですが、なぜか動かず、その後試行錯誤して改めてこのコードを試してみると動きましたｗ
どこが原因だったか判明していませんが、これでクローズします。
コメント頂けた方、ありがとうございました！！

Answer 2

cronJob 自体についてはわかりませんが、CoffeeScript としておかしなところを指摘してみます。

cronJob = new cronJob('*/10 * * * *', () ->

変数の cronJob とクラス名の cronJob が被ってしまっています。そもそも cronJob クラスがどこから来たのか提示されたコードからだとよくわからないのですが、事前に読まれていたとしても上書きされてしまうのはまずい気がします。

cronJob.start()

この cronJob がクラスなのか 2 行目で作った変数なのかがわからないのですが、もし変数のつもりであるなら、2 行目で宣言された cronJob 変数は (robot) -> 関数内のローカル変数ですので、この位置からは参照できません。

そしておまけですが、@exec の @ は恐らく必要ないです。あっても動作に支障はないかとは思います。

Comments

[twin_bird]

回答ありがとうございます！もう少しシンプルな形に変更してみたのですが、変数？部分の展開が上手くいきません。。（質問欄参照）

[thinca]

`"stdout"` のようにクォートで囲えば当然 `stdout` と言う文字列そのものが送られます。変数 `stdout` を指定してみてください。

[twin_bird]

バックティック（`）で指定してみましたが、展開されないようです。。hubotはpm2でデーモンを永続化しているので、そちらのログを確認してみましたがそれらしきエラー等見当たりませんでした。。

[thinca]

すいません。バックティックは Markdown の記法のつもりで書きました(そしてコメントでは Markdown は使えませんでした…)。
いずれにしても基本的な CoffeeScript の文法部分になるので、わからない場合は基礎の入門からやり直したほうが良いかと思います。