ruboty-response 0.0.5 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 42dcfa387c9eac4ee09cceaf46031a3b7116a58c
-  data.tar.gz: e47320e880fa614e368746658559a838ede56bfa
+  metadata.gz: ef9eead7d7e1d71867c4d51a86ce2029ff6beed1
+  data.tar.gz: c6d6cfc76726664c87f96711b1e12bd7f1d6c6a7
 SHA512:
-  metadata.gz: a0e44ea5f5c4a39fa5fc54a4f1062b647b4d52627eca86fae15688abd2a52e30fcef6ebbdfa23ee2c5aa03baa723a9ac7f5897a208c62b13bdd356e907b4a49e
-  data.tar.gz: 154673114d852d54280a42a1268815273178d09f9ea59da8cbf39af3beda6829aeef86996451cd4dca0ca1cad8081bcffb61c2f489042ed0a46159861ac187e5
+  metadata.gz: 1bf4b9b94f8f4a89d2760d59899ef8d6f705c575568b8c3197c5aa99f8a289cbb22495a12c441ac12904ee5e13281e8917d4901055f333d0efb59b69d7b0e68b
+  data.tar.gz: 41f2334593dfa03f937fb73eeab187aabd2ce78bf5f1cf704ef8914b8566939cfddc21dfe64ddd4c6203b772aaac6ee5db5f1b33bc00cfdfe4a67b18f17f30dc

data/lib/ruboty/handlers/response.rb

@@ -11,13 +11,18 @@ module Ruboty
 
       def catchall(message)
         responses.each do |id, hash|
-          next unless /#{hash[:regex]}/ =~ message[:keyword] rescue false
+          next unless message[:keyword] =~ /#{hash[:regex]}/ rescue false
 
           # If the response is a code
-          if hash[:response] =~ /\A`.+`\z/
+          if match_data = hash[:response].match(/\A`(?<code>.+)`\z/)
             thread = Thread.start do
-              result = `#{hash[:response].match(/\A`(.+)`\z/)[1]}`.chomp!
-              message.reply(result)
+              stdout, stderr, status = Open3.capture3(match_data[:code])
+              message.reply(stdout.chomp) if stdout
+              message.reply(stderr.chomp) if stderr
+
+              unless status.success?
+                message.reply("Command Failed:\n#{match_data[:code]}", code: true)
+              end
             end
             Thread.start do
               sleep 10
@@ -27,6 +32,8 @@ module Ruboty
             message.reply(hash[:response])
           end
         end
+      rescue => e
+        Ruboty.logger.error("Error: #{e.class}: #{e.message}}")
       end
 
       def add(message)

data/lib/ruboty/response/version.rb

@@ -1,5 +1,5 @@
 module Ruboty
   module Response
-    VERSION = '0.0.5'
+    VERSION = '1.0.0'
   end
 end

data/ruboty-response.gemspec

@@ -20,6 +20,4 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'pry'
-  spec.add_development_dependency 'awesome_print'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruboty-response
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 1.0.0
 platform: ruby
 authors:
 - kaihar4
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-04 00:00:00.000000000 Z
+date: 2015-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruboty
@@ -52,34 +52,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: pry
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: awesome_print
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: Ruboty handler to register a response.
 email:
 - kaihar4@gmail.com