slack-smart-bot 1.8.2 → 1.9.1

data/lib/slack/smart-bot/commands/general/bot_status.rb

@@ -14,7 +14,7 @@ class SlackSmartBot
       gems_remote = `gem list slack-smart-bot --remote`
       version_remote = gems_remote.to_s().scan(/slack-smart-bot \((\d+\.\d+\.\d+)/).join
       version_message = ""
-      if version_remote != VERSION
+      if Gem::Version.new(version_remote) > Gem::Version.new(VERSION)
         version_message = " There is a new available version: #{version_remote}."
       end
       require "socket"

data/lib/slack/smart-bot/commands/general/bye_bot.rb

@@ -4,8 +4,8 @@ class SlackSmartBot
   # help: `Bye Bot`
   # help: `Bye Smart`
   # help: `Bye NAME_OF_THE_BOT`
-  # help:    Also apart of Bye you can use _Bæ, Good Bye, Adiós, Ciao, Bless, Bless Bless, Adeu_
   # help:    Bot stops listening to you
+  # help:    Also apart of Bye you can use _Bæ, Good Bye, Adiós, Ciao, Bless, Bless Bless, Adeu_
   # help:
   def bye_bot(dest, from, display_name)
     if @status == :on

data/lib/slack/smart-bot/commands/general/hi_bot.rb

@@ -5,9 +5,9 @@ class SlackSmartBot
   # help: `Hi Smart`
   # help: `Hello Bot` `Hola Bot` `Hallo Bot` `What's up Bot` `Hey Bot` `Hæ Bot`
   # help: `Hello THE_NAME_OF_THE_BOT`
-  # help:    Also apart of Hello you can use _Hallo, Hi, Hola, What's up, Hey, Hæ_
   # help:    Bot starts listening to you
   # help:    After that if you want to avoid a single message to be treated by the smart bot, start the message by -
+  # help:    Also apart of Hello you can use _Hallo, Hi, Hola, What's up, Hey, Hæ_
   # help:
   def hi_bot(user, dest, dchannel, from, display_name)
     if @status == :on

data/lib/slack/smart-bot/commands/general/use_rules.rb

@@ -15,14 +15,10 @@ class SlackSmartBot
       respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
     else
       #todo: add pagination for case more than 1000 channels on the workspace
-      channels = client.web_client.conversations_list(
-        types: "private_channel,public_channel",
-        limit: "1000",
-        exclude_archived: "true",
-      ).channels
+      channels = get_channels()
 
       channel_found = channels.detect { |c| c.name == channel }
-      members = client.web_client.conversations_members(channel: @channels_id[channel]).members unless channel_found.nil?
+      members = get_channel_members(@channels_id[channel]) unless channel_found.nil?
 
       if channel_found.nil?
         respond "The channel you are trying to use doesn't exist", dest

data/lib/slack/smart-bot/commands/general/whats_new.rb

@@ -0,0 +1,19 @@
+class SlackSmartBot
+
+    # help: ----------------------------------------------
+    # help: `What's new`
+    # help:    It will display the last user changes on Slack Smart Bot
+    # help:
+    def whats_new(user, dest, dchannel, from, display_name)
+      if @status == :on
+        save_stats(__method__)
+        whats_new_file = (__FILE__).gsub(/lib\/slack\/smart-bot\/commands\/general\/whats_new\.rb$/, "whats_new.txt")
+        whats_new = File.read(whats_new_file)
+        whats_new.split(/^\-\-\-\-\-\-+$/).each do |msg|
+            respond msg
+            sleep 0.3
+        end
+      end
+    end
+  end
+  

data/lib/slack/smart-bot/commands/on_bot/add_shortcut.rb

@@ -7,8 +7,10 @@ class SlackSmartBot
   # help: `add sc for all NAME: COMMAND`
   # help: `shortcut NAME: COMMAND`
   # help: `shortcut for all NAME: COMMAND`
+  # help: `add global sc for all NAME: COMMAND`
   # help:    It will add a shortcut that will execute the command we supply.
   # help:    In case we supply 'for all' then the shorcut will be available for everybody
+  # help:    If 'global' or 'generic' supplied and in Master channel then the shortcut will be available in all Bot channels.
   # help:    If you want to use a shortcut as a inline shortcut inside a command you can do it by adding a $ fex: _!run tests $cust1_
   # help:    Example:
   # help:        _add shortcut for all Spanish account: code require 'iso/iban'; 10.times {puts ISO::IBAN.random('ES')}_
@@ -17,7 +19,7 @@ class SlackSmartBot
   # help:        _shortcut Spanish Account_
   # help:        _Spanish Account_
   # help:
-  def add_shortcut(dest, user, typem, for_all, shortcut_name, command, command_to_run)
+  def add_shortcut(dest, user, typem, for_all, shortcut_name, command, command_to_run, global)
     save_stats(__method__)
     unless typem == :on_extended
       from = user.name
@@ -25,43 +27,73 @@ class SlackSmartBot
         (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
         respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
       else
-        @shortcuts[from] = Hash.new() unless @shortcuts.keys.include?(from)
 
-        found_other = false
-        if for_all.to_s != ""
-          @shortcuts.each { |sck, scv|
-            if sck != :all and sck != from and scv.key?(shortcut_name)
-              found_other = true
+        if global 
+          if !config.on_master_bot or typem != :on_master
+            respond "It is only possible to add global shortcuts from Master channel"
+          else
+            @shortcuts_global[from] = Hash.new() unless @shortcuts_global.keys.include?(from)
+            found_other = false
+            if for_all.to_s != ""
+              @shortcuts_global.each { |sck, scv|
+                if sck != :all and sck != from and scv.key?(shortcut_name)
+                  found_other = true
+                end
+              }
             end
-          }
-        end
-        if !config.admins.include?(from) and @shortcuts[:all].include?(shortcut_name) and !@shortcuts[from].include?(shortcut_name)
-          respond "Only the creator of the shortcut can modify it", dest
-        elsif found_other
-          respond "You cannot create a shortcut for all with the same name than other user is using", dest
-        elsif !@shortcuts[from].include?(shortcut_name)
-          #new shortcut
-          @shortcuts[from][shortcut_name] = command_to_run
-          @shortcuts[:all][shortcut_name] = command_to_run if for_all.to_s != ""
-          update_shortcuts_file()
-          respond "shortcut added", dest
+            if @shortcuts_global[:all].include?(shortcut_name) or @shortcuts_global[from].include?(shortcut_name)
+              respond "Global shortcut name already in use. Please use another shortcut name."
+            elsif found_other
+              respond "You cannot create a global shortcut for all with the same name than other user is using."
+            elsif !@shortcuts_global[from].include?(shortcut_name)
+              #new shortcut
+              @shortcuts_global[from][shortcut_name] = command_to_run
+              @shortcuts_global[:all][shortcut_name] = command_to_run if for_all.to_s != ""
+              update_shortcuts_file()
+              respond "global shortcut added"
+            else
+              respond "Not possible to add the global shortcut" #todo: check if this is ever gonna be the case
+            end            
+          end
         else
-          #are you sure? to avoid overwriting existing
-          unless @questions.keys.include?(from)
-            ask("The shortcut already exists, are you sure you want to overwrite it?", command, from, dest)
+          @shortcuts[from] = Hash.new() unless @shortcuts.keys.include?(from)
+
+          found_other = false
+          if for_all.to_s != ""
+            @shortcuts.each { |sck, scv|
+              if sck != :all and sck != from and scv.key?(shortcut_name)
+                found_other = true
+              end
+            }
+          end
+          if !config.admins.include?(from) and @shortcuts[:all].include?(shortcut_name) and !@shortcuts[from].include?(shortcut_name)
+            respond "Only the creator of the shortcut can modify it", dest
+          elsif found_other
+            respond "You cannot create a shortcut for all with the same name than other user is using", dest
+          elsif !@shortcuts[from].include?(shortcut_name)
+            #new shortcut
+            @shortcuts[from][shortcut_name] = command_to_run
+            @shortcuts[:all][shortcut_name] = command_to_run if for_all.to_s != ""
+            update_shortcuts_file()
+            respond "shortcut added", dest
           else
-            case @questions[from]
-            when /^(yes|yep)/i
-              @shortcuts[from][shortcut_name] = command_to_run
-              @shortcuts[:all][shortcut_name] = command_to_run if for_all.to_s != ""
-              update_shortcuts_file()
-              respond "shortcut added", dest
-              @questions.delete(from)
-            when /^no/i
-              respond "ok, I won't add it", dest
-              @questions.delete(from)
+            #are you sure? to avoid overwriting existing
+            if answer.empty?
+              ask("The shortcut already exists, are you sure you want to overwrite it?", command, from, dest)
             else
-              ask "I don't understand, yes or no?", command, from, dest
+              case answer
+              when /^(yes|yep)/i
+                @shortcuts[from][shortcut_name] = command_to_run
+                @shortcuts[:all][shortcut_name] = command_to_run if for_all.to_s != ""
+                update_shortcuts_file()
+                respond "shortcut added", dest
+                answer_delete(from)
+              when /^no/i
+                respond "ok, I won't add it", dest
+                answer_delete(from)
+              else
+                ask "I don't understand, yes or no?", command, from, dest
+              end
             end
           end
         end

data/lib/slack/smart-bot/commands/on_bot/admin/extend_rules.rb

@@ -14,15 +14,11 @@ class SlackSmartBot
         respond "Only admins can extend the rules. Admins on this channel: #{config.admins}", dest
       else
         #todo: add pagination for case more than 1000 channels on the workspace
-        channels = client.web_client.conversations_list(
-          types: "private_channel,public_channel",
-          limit: "1000",
-          exclude_archived: "true",
-        ).channels
+        channels = get_channels()
 
         channel_found = channels.detect { |c| c.name == channel }
         get_channels_name_and_id()
-        members = client.web_client.conversations_members(channel: @channels_id[channel]).members unless channel_found.nil?
+        members = get_channel_members(@channels_id[channel]) unless channel_found.nil?
         get_bots_created()
         channels_in_use = []
         @bots_created.each do |k, v|
@@ -39,7 +35,7 @@ class SlackSmartBot
         elsif !members.include?(user.id)
           respond "You need to join that channel first", dest
         elsif !members.include?(config[:nick_id])
-          respond "You need to add first to the channel the smart bot user: #{config[:nick]}", dest
+          respond "You need to add first to the channel the smart bot user: <@#{config[:nick_id]}>", dest
         else
           channels_in_use.each do |channel_in_use|
             respond "The rules from channel <##{@channels_id[channel_in_use]}> are already in use on that channel", dest

data/lib/slack/smart-bot/commands/on_bot/admin/pause_bot.rb

@@ -13,6 +13,7 @@ class SlackSmartBot
       respond "zZzzzzZzzzzZZZZZZzzzzzzzz", dest
       @status = :paused
       @bots_created[@channel_id][:status] = :paused
+      update_bots_file()
       unless config.on_master_bot
         send_msg_channel config.master_channel, "Changed status on #{config.channel} to :paused"
       end

data/lib/slack/smart-bot/commands/on_bot/admin/see_routines.rb

@@ -38,8 +38,14 @@ class SlackSmartBot
           respond "Routines on channel *#{rout_ch.get_values(:channel_name).values.flatten.uniq[0]}*", dest
           rout_ch.each do |k, v|
             msg = []
-            ch != v[:dest] ? directm = " (*DM to #{v[:creator]}*)" : directm = ""
-            msg << "*`#{k}`*#{directm}"
+            if v[:dest][0] == 'D'
+              extram = " (*DM to #{v[:creator]}*)"
+            elsif v[:dest] != ch
+              extram = " (*publish on <##{v[:dest]}>*)"
+            else
+              extram = ''
+            end
+            msg << "*`#{k}`*#{extram}"
             msg << "\tCreator: #{v[:creator]}"
             msg << "\tStatus: #{v[:status]}"
             msg << "\tEvery: #{v[:every]}" unless v[:every] == ""

data/lib/slack/smart-bot/commands/on_bot/admin/start_bot.rb

@@ -12,6 +12,7 @@ class SlackSmartBot
       respond "This bot is running and listening from now on. You can pause again: pause this bot", dest
       @status = :on
       @bots_created[@channel_id][:status] = :on
+      update_bots_file()
       unless config.on_master_bot
         send_msg_channel config.master_channel, "Changed status on #{config.channel} to :on"
       end

data/lib/slack/smart-bot/commands/on_bot/delete_repl.rb

@@ -3,7 +3,6 @@ class SlackSmartBot
   # help: `delete repl SESSION_NAME`
   # help: `delete irb SESSION_NAME`
   # help: `remove repl SESSION_NAME`
-  # help: 
   # help:     Will delete the specified REPL
   # help:     Only the creator of the REPL or an admin can delete REPLs
   # help:
@@ -20,6 +19,7 @@ class SlackSmartBot
           @repls.delete(session_name)
           update_repls()
           File.rename("#{config.path}/repl/#{@channel_id}/#{session_name}.input", "#{config.path}/repl/#{@channel_id}/#{session_name}_#{Time.now.strftime("%Y%m%d%H%M%S%N")}.deleted")
+          File.delete("#{config.path}/repl/#{@channel_id}/#{session_name}.output") if File.exist?("#{config.path}/repl/#{@channel_id}/#{session_name}.output")
           File.delete("#{config.path}/repl/#{@channel_id}/#{session_name}.run") if File.exist?("#{config.path}/repl/#{@channel_id}/#{session_name}.run")
           respond "REPL #{session_name} deleted"
         else

data/lib/slack/smart-bot/commands/on_bot/delete_shortcut.rb

@@ -2,10 +2,12 @@ class SlackSmartBot
   # help: ----------------------------------------------
   # help: `delete shortcut NAME`
   # help: `delete sc NAME`
+  # help: `delete global sc NAME`
   # help:    It will delete the shortcut with the supplied name
+  # help:    'global' or 'generic' can only be used on Master channel.
   # help:
 
-  def delete_shortcut(dest, user, shortcut, typem, command)
+  def delete_shortcut(dest, user, shortcut, typem, command, global)
     save_stats(__method__)
     unless typem == :on_extended
       from = user.name
@@ -15,32 +17,61 @@ class SlackSmartBot
       else
         deleted = false
 
-        if !config.admins.include?(from) and @shortcuts[:all].include?(shortcut) and !@shortcuts[from].include?(shortcut)
-          respond "Only the creator of the shortcut or an admin user can delete it", dest
-        elsif (@shortcuts.keys.include?(from) and @shortcuts[from].keys.include?(shortcut)) or
-              (config.admins.include?(from) and @shortcuts[:all].include?(shortcut))
-          #are you sure? to avoid deleting by mistake
-          unless @questions.keys.include?(from)
-            ask("are you sure you want to delete it?", command, from, dest)
+        if global
+          if !config.on_master_bot or typem != :on_master
+            respond "It is only possible to delete global shortcuts from Master channel"
           else
-            case @questions[from]
-            when /^(yes|yep)/i
-              @questions.delete(from)
-              respond "shortcut deleted!", dest
-              respond("#{shortcut}: #{@shortcuts[from][shortcut]}", dest) if @shortcuts.key?(from) and @shortcuts[from].key?(shortcut)
-              respond("#{shortcut}: #{@shortcuts[:all][shortcut]}", dest) if @shortcuts.key?(:all) and @shortcuts[:all].key?(shortcut)
-              @shortcuts[from].delete(shortcut) if @shortcuts.key?(from) and @shortcuts[from].key?(shortcut)
-              @shortcuts[:all].delete(shortcut) if @shortcuts.key?(:all) and @shortcuts[:all].key?(shortcut)
+            if !config.admins.include?(from) and @shortcuts_global[:all].include?(shortcut) and 
+              (!@shortcuts_global.key?(from) or !@shortcuts_global[from].include?(shortcut))
+              respond "Only the creator of the shortcut or an admin user can delete it"
+            elsif (@shortcuts_global.key?(from) and @shortcuts_global[from].keys.include?(shortcut)) or
+              (config.admins.include?(from) and @shortcuts_global[:all].include?(shortcut))
+              
+              respond "global shortcut deleted!", dest
+              if @shortcuts_global.key?(from) and @shortcuts_global[from].key?(shortcut)
+                respond("#{shortcut}: #{@shortcuts_global[from][shortcut]}", dest) 
+              elsif @shortcuts_global.key?(:all) and @shortcuts_global[:all].key?(shortcut)
+                respond("#{shortcut}: #{@shortcuts_global[:all][shortcut]}", dest)
+              end
+              @shortcuts_global[from].delete(shortcut) if @shortcuts_global.key?(from) and @shortcuts_global[from].key?(shortcut)
+              @shortcuts_global[:all].delete(shortcut) if @shortcuts_global.key?(:all) and @shortcuts_global[:all].key?(shortcut)
               update_shortcuts_file()
-            when /^no/i
-              @questions.delete(from)
-              respond "ok, I won't delete it", dest
             else
-              ask("I don't understand, are you sure you want to delete it? (yes or no)", command, from, dest)
+              respond 'shortcut not found'
             end
           end
         else
-          respond "shortcut not found", dest
+          if !config.admins.include?(from) and @shortcuts[:all].include?(shortcut) and 
+            (!@shortcuts.key?(from) or !@shortcuts[from].include?(shortcut))
+            respond "Only the creator of the shortcut or an admin user can delete it", dest
+          elsif (@shortcuts.keys.include?(from) and @shortcuts[from].keys.include?(shortcut)) or
+                (config.admins.include?(from) and @shortcuts[:all].include?(shortcut))
+            #are you sure? to avoid deleting by mistake
+            if answer.empty?
+              ask("are you sure you want to delete it?", command, from, dest)
+            else
+              case answer
+              when /^(yes|yep)/i
+                answer_delete(from)
+                respond "shortcut deleted!", dest
+                if @shortcuts.key?(from) and @shortcuts[from].key?(shortcut)
+                  respond("#{shortcut}: #{@shortcuts[from][shortcut]}", dest) 
+                elsif @shortcuts.key?(:all) and @shortcuts[:all].key?(shortcut)
+                  respond("#{shortcut}: #{@shortcuts[:all][shortcut]}", dest)
+                end
+                @shortcuts[from].delete(shortcut) if @shortcuts.key?(from) and @shortcuts[from].key?(shortcut)
+                @shortcuts[:all].delete(shortcut) if @shortcuts.key?(:all) and @shortcuts[:all].key?(shortcut)
+                update_shortcuts_file()
+              when /^no/i
+                answer_delete(from)
+                respond "ok, I won't delete it", dest
+              else
+                ask("I don't understand, are you sure you want to delete it? (yes or no)", command, from, dest)
+              end
+            end
+          else
+            respond "shortcut not found", dest
+          end
         end
       end
     end

data/lib/slack/smart-bot/commands/on_bot/get_repl.rb

@@ -3,7 +3,6 @@ class SlackSmartBot
   # help: `get repl SESSION_NAME`
   # help: `get irb SESSION_NAME`
   # help: `get live SESSION_NAME`
-  # help: 
   # help:     Will get the Ruby commands sent on that SESSION_NAME.
   # help:
   def get_repl(dest, user, session_name)
@@ -16,22 +15,23 @@ class SlackSmartBot
       Dir.mkdir("#{config.path}/repl") unless Dir.exist?("#{config.path}/repl")
       Dir.mkdir("#{config.path}/repl/#{@channel_id}") unless Dir.exist?("#{config.path}/repl/#{@channel_id}")
       if File.exist?("#{config.path}/repl/#{@channel_id}/#{session_name}.run")
-        if @repls.key?(session_name) and @repls[session_name][:type] == :private and 
+        if @repls.key?(session_name) and (@repls[session_name][:type] == :private or @repls[session_name][:type] == :private_clean) and 
           @repls[session_name][:creator_name]!=user.name and 
           !config.admins.include?(user.name)
           respond "The REPL with session name: #{session_name} is private", dest
         else
+          content = "require 'nice_http'\n"
           if @repls.key?(session_name)
             @repls[session_name][:accessed] = Time.now.to_s
             @repls[session_name][:gets] += 1
             update_repls()
           end
-
-          content = "require 'nice_http'\n"
-          if File.exist?("#{project_folder}/.smart-bot-repl")
+          if !@repls.key?(session_name) or 
+            (File.exist?("#{project_folder}/.smart-bot-repl") and @repls[session_name][:type] != :private_clean and @repls[session_name][:type] != :public_clean)
             content += File.read("#{project_folder}/.smart-bot-repl")
             content += "\n"
           end
+
           content += File.read("#{config.path}/repl/#{@channel_id}/#{session_name}.run").gsub(/^(quit|exit|bye)$/i,'') #todo: remove this gsub it will never contain it
           File.write("#{config.path}/repl/#{@channel_id}/#{session_name}.rb", content, mode: "w+")
           send_file(dest, "REPL #{session_name} on #{config.channel}", "#{config.path}/repl/#{@channel_id}/#{session_name}.rb", " REPL #{session_name} on #{config.channel}", 'text/plain', "ruby")

data/lib/slack/smart-bot/commands/on_bot/repl.rb

@@ -5,15 +5,16 @@ class SlackSmartBot
   # help: `irb`
   # help: `repl SESSION_NAME`
   # help: `private repl SESSION_NAME`
+  # help: `clean repl SESSION_NAME`
   # help: `repl ENV_VAR=VALUE`
   # help: `repl SESSION_NAME ENV_VAR=VALUE ENV_VAR='VALUE'`
   # help: `repl SESSION_NAME: "DESCRIPTION"`
   # help: `repl SESSION_NAME: "DESCRIPTION" ENV_VAR=VALUE ENV_VAR='VALUE'`
-  # help: 
   # help:     Will run all we write as a ruby command and will keep the session values. 
   # help:     SESSION_NAME only admits from a to Z, numbers, - and _
   # help:     If no SESSION_NAME supplied it will be treated as a temporary REPL
   # help:     If 'private' specified the repl will be accessible only by you and it will be displayed only to you when `see repls`
+  # help:     If 'clean' specified the repl won't pre execute the code written on the .smart-bot-repl file
   # help:     To avoid a message to be treated, start the message with '-'.
   # help:     Send _quit_, _bye_ or _exit_ to finish the session.
   # help:     Send puts, print, p or pp if you want to print out something when using `run repl` later.
@@ -78,7 +79,13 @@ class SlackSmartBot
           }
           update_repls()        
         end
-    
+        react :running
+        if Thread.current[:ts].to_s == ''
+          @ts_react = Thread.current[:thread_ts]
+        else
+          @ts_react = Thread.current[:ts]
+        end        
+
         message = "Session name: *#{session_name}*
         From now on I will execute all you write as a Ruby command and I will keep the session open until you send `quit` or `bye` or `exit`. 
         I will respond with the result so it is not necessary you send `print`, `puts`, `p` or `pp` unless you want it as the output when calling `run repl`. 
@@ -96,6 +103,19 @@ class SlackSmartBot
         File.write("#{config.path}/repl/#{@channel_id}/#{@repl_sessions[from][:name]}.output", "", mode: "a+")
         File.write("#{config.path}/repl/#{@channel_id}/#{@repl_sessions[from][:name]}.run", "", mode: "a+")
         
+        if type != :private_clean and type != :public_clean
+          pre_execute = '
+            if File.exist?(\"./.smart-bot-repl\")
+              begin
+                eval(File.read(\"./.smart-bot-repl\"), bindme' + serialt + ')
+              rescue Exception => resp_repl
+              end
+            end
+          '
+        else
+          pre_execute = ''
+        end
+
         process_to_run = '
             ruby -e "' + env_vars.join("\n") + '
             require \"amazing_print\"
@@ -105,17 +125,12 @@ class SlackSmartBot
               (obj.methods - Object.methods)
             end
             
-            file_input_repl = File.open(\"' + Dir.pwd + '/repl/' + @channel_id + '/' + session_name + '.input\", \"r\")
-            if File.exist?(\"./.smart-bot-repl\")
-              begin
-                eval(File.read(\"./.smart-bot-repl\"), bindme' + serialt + ')
-              rescue Exception => resp_repl
-              end
-            end
+            file_input_repl = File.open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.input\", \"r\")
+            ' + pre_execute + '
             while true do 
               sleep 0.2 
               code_to_run_repl = file_input_repl.read
-              if code_to_run_repl.to_s!=''
+              if code_to_run_repl.to_s!=\"\"
                 add_to_run_repl = true
                 if code_to_run_repl.to_s.match?(/^quit$/i) or 
                   code_to_run_repl.to_s.match?(/^exit$/i) or 
@@ -134,16 +149,16 @@ class SlackSmartBot
                   end
                   if resp_repl.to_s != \"\"
                     if code_to_run_repl.match?(/^\s*p\s+/i)
-                      open(\"' + Dir.pwd + '/repl/' + @channel_id + '/' + session_name + '.output\", \"a+\") {|f|
+                      open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.output\", \"a+\") {|f|
                         f.puts \"\`\`\`\n#{resp_repl.inspect}\n\`\`\`\"
                       }
                     else
-                      open(\"' + Dir.pwd + '/repl/' + @channel_id + '/' + session_name + '.output\", \"a+\") {|f|
+                      open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.output\", \"a+\") {|f|
                         f.puts \"\`\`\`\n#{resp_repl.ai}\n\`\`\`\"
                       }
                     end
                     unless error or !add_to_run_repl
-                      open(\"' + Dir.pwd + '/repl/' + @channel_id + '/' + session_name + '.run\", \"a+\") {|f|
+                      open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.run\", \"a+\") {|f|
                         f.puts code_to_run_repl
                       }
                     end
@@ -152,7 +167,6 @@ class SlackSmartBot
               end
             end"
         '
-
         unless rules_file.empty? # to get the project_folder
           begin
             eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file)
@@ -160,12 +174,12 @@ class SlackSmartBot
         end
         started = Time.now
         process_to_run = ("cd #{project_folder} &&" + process_to_run) if defined?(project_folder)
-        
+
         stdin, stdout, stderr, wait_thr = Open3.popen3(process_to_run)
         timeout = 30 * 60 # 30 minutes
         
         file_output_repl = File.open("#{config.path}/repl/#{@channel_id}/#{session_name}.output", "r")
-
+        @repl_sessions[from][:pid] = wait_thr.pid
         while (wait_thr.status == 'run' or wait_thr.status == 'sleep') and @repl_sessions.key?(from)
           begin
             if (Time.now-@repl_sessions[from][:finished]) > timeout
@@ -173,6 +187,14 @@ class SlackSmartBot
                   f.puts 'quit'
                 }
                 respond "REPL session finished: #{@repl_sessions[from][:name]}", dest
+                unreact :running, @ts_react
+                pids = `pgrep -P #{@repl_sessions[from][:pid]}`.split("\n").map(&:to_i) #todo: it needs to be adapted for Windows
+                pids.each do |pid|
+                  begin
+                    Process.kill("KILL", pid)
+                  rescue
+                  end
+                end
                 @repl_sessions.delete(from)
                 break
             end
@@ -199,10 +221,12 @@ class SlackSmartBot
         code.gsub!("\\r", "\r")
         # Disabled for the moment since it is deleting lines with '}'
         #code.gsub!(/^\W*$/, "") #to remove special chars from slack when copy/pasting.
-        if code.match?(/System/i) or code.match?(/Kernel/i) or code.include?("File") or
+        if code.match?(/System/i) or code.match?(/Kernel/i) or code.include?("File.") or
           code.include?("`") or code.include?("exec") or code.include?("spawn") or code.include?("IO.") or
           code.match?(/open3/i) or code.match?(/bundle/i) or code.match?(/gemfile/i) or code.include?("%x") or
-          code.include?("ENV") or code.match?(/=\s*IO/)
+          code.include?("ENV") or code.match?(/=\s*IO/) or code.include?("Dir.") or 
+          code.match?(/=\s*File/) or code.match?(/=\s*Dir/) or code.match?(/<\s*File/) or code.match?(/<\s*Dir/) or
+          code.match?(/\w+:\s*File/) or code.match?(/\w+:\s*Dir/)
           respond "Sorry I cannot run this due security reasons", dest
         else
           @repl_sessions[from][:input]<<code
@@ -212,6 +236,14 @@ class SlackSmartBot
               f.puts code
             }
             respond "REPL session finished: #{@repl_sessions[from][:name]}", dest
+            unreact :running, @ts_react
+            pids = `pgrep -P #{@repl_sessions[from][:pid]}`.split("\n").map(&:to_i) #todo: it needs to be adapted for Windows
+            pids.each do |pid|
+              begin
+                Process.kill("KILL", pid)
+              rescue
+              end
+            end
             @repl_sessions.delete(from)
           when /^\s*-/i
             #ommit