slack-smart-bot 1.8.1 → 1.10.0

data/lib/slack/smart-bot/process_first.rb

@@ -1,27 +1,49 @@
 class SlackSmartBot
-  def process_first(user, text, dest, dchannel, typem, files, ts, thread_ts)
+  def process_first(user, text, dest, dchannel, typem, files, ts, thread_ts, routine, routine_name, routine_type)
     nick = user.name
     rules_file = ""
     text.gsub!(/^!!/,'^') # to treat it just as ^
+    shared = []
+    if @shares.key?(@channels_name[dest]) and (ts.to_s!='' or config.simulate) and (user.id!=config.nick_id or (user.id == config.nick_id and !text.match?(/\A\*?Shares from channel/)))
+      @shares[@channels_name[dest]].each do |row|
+        if row[:user_deleted]==''
+          if ((row[:type] == 'text' and text.include?(row[:condition][1..-2])) or (row[:type]=='regexp' and text.match?(/#{row[:condition][1..-2]}/im))) and !shared.include?(row[:to_channel])
+            if config.simulate
+              link = text
+            else
+              link = client.web_client.chat_getPermalink(channel: dest, message_ts: ts).permalink
+            end
+            respond "*<#{link}|Shared> by <@#{row[:user_created]}> from <##{dest}>* using share id #{row[:share_id]}", row[:to_channel]
+            shared << row[:to_channel]
+            sleep 0.2
+          end
+        end
+      end
+    end
+
     if typem == :on_call
       rules_file = config.rules_file
     elsif dest[0] == "C" or dest[0] == "G" # on a channel or private channel
       rules_file = config.rules_file
 
-      if @rules_imported.key?(user.id) and @rules_imported[user.id].key?(dchannel)
-        unless @bots_created.key?(@rules_imported[user.id][dchannel])
+      if @rules_imported.key?(user.name) and @rules_imported[user.name].key?(dchannel)
+        unless @bots_created.key?(@rules_imported[user.name][dchannel])
           get_bots_created()
         end
-        if @bots_created.key?(@rules_imported[user.id][dchannel])
-          rules_file = @bots_created[@rules_imported[user.id][dchannel]][:rules_file]
+        if @bots_created.key?(@rules_imported[user.name][dchannel])
+          rules_file = @bots_created[@rules_imported[user.name][dchannel]][:rules_file]
         end
       end
-    elsif dest[0] == "D" and @rules_imported.key?(user.id) and @rules_imported[user.id].key?(user.id) #direct message
-      unless @bots_created.key?(@rules_imported[user.id][user.id])
+    elsif dest[0] == "D" and @rules_imported.key?(user.name) and @rules_imported[user.name].key?(user.name) #direct message
+      unless @bots_created.key?(@rules_imported[user.name][user.name])
         get_bots_created()
       end
-      if @bots_created.key?(@rules_imported[user.id][user.id])
-        rules_file = @bots_created[@rules_imported[user.id][user.id]][:rules_file]
+      if @bots_created.key?(@rules_imported[user.name][user.name])
+        rules_file = @bots_created[@rules_imported[user.name][user.name]][:rules_file]
+      end
+    elsif dest[0] == 'D' and (!@rules_imported.key?(user.name) or ( @rules_imported.key?(user.name) and !@rules_imported[user.name].key?(user.name)))
+      if File.exist?("#{config.path}/rules/general_rules.rb")
+        rules_file = "/rules/general_rules.rb"
       end
     end
 
@@ -80,21 +102,21 @@ class SlackSmartBot
       text.match(/^()\^\s*(.+)\s*/im) or
       text.match(/^()!\s*(.+)\s*/im) or
       text.match(/^()<@#{config[:nick_id]}>\s+(.+)\s*/im)
-       command2 = $2
-       if text.match?(/^()\^\s*(.+)/im)
-         add_double_excl = true
-         addexcl = false
-         if command2.match?(/^![^!]/) or command2.match?(/^\^/)
+      command2 = $2
+      if text.match?(/^()\^\s*(.+)/im)
+        add_double_excl = true
+        addexcl = false
+        if command2.match?(/^![^!]/) or command2.match?(/^\^/)
           command2[0]=''
-         elsif command2.match?(/^!!/)
+        elsif command2.match?(/^!!/)
           command2[0]=''
           command2[1]=''
-         end
-       else
+        end
+      else
         add_double_excl = false
         addexcl = true
-       end
-       command = command2
+      end
+      command = command2
     else
       addexcl = false
       if text.include?('$') #for shortcuts inside commands
@@ -111,6 +133,10 @@ class SlackSmartBot
           command.gsub!("$#{sc}", @shortcuts[nick][sc])
         elsif @shortcuts.key?(:all) and @shortcuts[:all].keys.include?(sc)
           command.gsub!("$#{sc}", @shortcuts[:all][sc])
+        elsif @shortcuts_global.key?(nick) and @shortcuts_global[nick].keys.include?(sc)
+          command.gsub!("$#{sc}", @shortcuts_global[nick][sc])
+        elsif @shortcuts_global.key?(:all) and @shortcuts_global[:all].keys.include?(sc)
+          command.gsub!("$#{sc}", @shortcuts_global[:all][sc])
         end
       end
       command.scan(/\$([^\s]+)/i).flatten.each do |sc|
@@ -119,6 +145,10 @@ class SlackSmartBot
           command.gsub!("$#{sc}", @shortcuts[nick][sc])
         elsif @shortcuts.key?(:all) and @shortcuts[:all].keys.include?(sc)
           command.gsub!("$#{sc}", @shortcuts[:all][sc])
+        elsif @shortcuts_global.key?(nick) and @shortcuts_global[nick].keys.include?(sc)
+          command.gsub!("$#{sc}", @shortcuts_global[nick][sc])
+        elsif @shortcuts_global.key?(:all) and @shortcuts_global[:all].keys.include?(sc)
+          command.gsub!("$#{sc}", @shortcuts_global[:all][sc])
         end
       end
       text = command
@@ -126,13 +156,19 @@ class SlackSmartBot
       text = "^" + text if add_double_excl
     end
     if command.scan(/^(shortcut|sc)\s+([^:]+)\s*$/i).any? or
-       (@shortcuts.keys.include?(:all) and @shortcuts[:all].keys.include?(command)) or
-       (@shortcuts.keys.include?(nick) and @shortcuts[nick].keys.include?(command))
+      (@shortcuts.keys.include?(:all) and @shortcuts[:all].keys.include?(command)) or
+      (@shortcuts.keys.include?(nick) and @shortcuts[nick].keys.include?(command)) or
+      (@shortcuts_global.keys.include?(:all) and @shortcuts_global[:all].keys.include?(command)) or
+      (@shortcuts_global.keys.include?(nick) and @shortcuts_global[nick].keys.include?(command))
       command = $2.downcase unless $2.nil?
       if @shortcuts.keys.include?(nick) and @shortcuts[nick].keys.include?(command)
         text = @shortcuts[nick][command].dup
       elsif @shortcuts.keys.include?(:all) and @shortcuts[:all].keys.include?(command)
         text = @shortcuts[:all][command].dup
+      elsif @shortcuts_global.keys.include?(nick) and @shortcuts_global[nick].keys.include?(command)
+        text = @shortcuts_global[nick][command].dup
+      elsif @shortcuts_global.keys.include?(:all) and @shortcuts_global[:all].keys.include?(command)
+        text = @shortcuts_global[:all][command].dup
       else
         respond "Shortcut not found", dest unless dest[0] == "C" and dchannel != dest #on extended channel
         return :next #jal
@@ -146,6 +182,9 @@ class SlackSmartBot
     begin
       t = Thread.new do
         begin
+          processed = false
+          processed_rules = false
+
           Thread.current[:dest] = dest
           Thread.current[:user] = user
           Thread.current[:command] = command
@@ -154,30 +193,37 @@ class SlackSmartBot
           Thread.current[:files?] = !files.nil? && files.size>0
           Thread.current[:ts] = ts
           Thread.current[:thread_ts] = thread_ts
+          Thread.current[:routine] = routine
+          Thread.current[:routine_name] = routine_name
+          Thread.current[:routine_type] = routine_type
+          Thread.current[:dchannel] = dchannel
           if thread_ts.to_s == ''
             Thread.current[:on_thread] = false
             Thread.current[:thread_ts] = Thread.current[:ts] # to create the thread if necessary
           else
             Thread.current[:on_thread] = true
           end
-          if (dest[0] == "C") || (dest[0] == "G") and @rules_imported.key?(user.id) &&
-            @rules_imported[user.id].key?(dchannel) && @bots_created.key?(@rules_imported[user.id][dchannel])
-              Thread.current[:using_channel] = @rules_imported[user.id][dchannel]
-          elsif dest[0] == "D" && @rules_imported.key?(user.id) && @rules_imported[user.id].key?(user.id) and
-            @bots_created.key?(@rules_imported[user.id][user.id])
-              Thread.current[:using_channel] = @rules_imported[user.id][user.id]
+          if (dest[0] == "C") || (dest[0] == "G") and @rules_imported.key?(user.name) &&
+            @rules_imported[user.name].key?(dchannel) && @bots_created.key?(@rules_imported[user.name][dchannel])
+              Thread.current[:using_channel] = @rules_imported[user.name][dchannel]
+          elsif dest[0] == "D" && @rules_imported.key?(user.name) && @rules_imported[user.name].key?(user.name) and
+            @bots_created.key?(@rules_imported[user.name][user.name])
+              Thread.current[:using_channel] = @rules_imported[user.name][user.name]
           else
               Thread.current[:using_channel] = ''
           end
-
-          processed = process(user, command, dest, dchannel, rules_file, typem, files, Thread.current[:thread_ts])
+          if typem == :on_pub or typem == :on_pg
+            processed = false
+          else
+            processed = process(user, command, dest, dchannel, rules_file, typem, files, Thread.current[:thread_ts])
+          end
           @logger.info "command: #{nick}> #{command}" if processed
           on_demand = false
-          if command.match(/^@?(#{config[:nick]}):*\s+(.+)/im) or
-            command.match(/^()!!(.+)/im) or
-            command.match(/^()\^(.+)/im) or
-            command.match(/^()!(.+)/im) or
-            command.match(/^()<@#{config[:nick_id]}>\s+(.+)/im)
+          if command.match(/\A@?(#{config[:nick]}):*\s+(.+)/im) or
+            command.match(/\A()!!(.+)/im) or
+            command.match(/\A()\^(.+)/im) or
+            command.match(/\A()!(.+)/im) or
+            command.match(/\A()<@#{config[:nick_id]}>\s+(.+)/im)
             command2 = $2
             Thread.current[:command] = command2
             if command2.match?(/^()!!(.+)/im) or
@@ -187,96 +233,148 @@ class SlackSmartBot
             command = command2
             on_demand = true
           end
-          if @status == :on and
-             (@questions.key?(nick) or
-             (@repl_sessions.key?(nick) and dest==@repl_sessions[nick][:dest] and 
-               ((@repl_sessions[nick][:on_thread] and thread_ts == @repl_sessions[nick][:thread_ts]) or
-                (!@repl_sessions[nick][:on_thread] and !Thread.current[:on_thread] ))) or 
-             (@listening.key?(nick) and typem != :on_extended and 
-               ((@listening[nick].key?(dest) and !Thread.current[:on_thread]) or 
-                (@listening[nick].key?(thread_ts) and Thread.current[:on_thread] ) )) or
-              dest[0] == "D" or on_demand)
-            @logger.info "command: #{nick}> #{command}" unless processed
-            #todo: verify this
+          unless config.on_maintenance or @status != :on
+            if typem == :on_pub or typem == :on_pg or typem == :on_extended
+              if command.match(/\A\s*(#{@salutations.join("|")})\s+(rules|help)\s*(.+)?$/i) or command.match(/\A(#{@salutations.join("|")}),? what can I do/i)
+                $2.to_s.match?(/rules/i) ? specific = true : specific = false
+                help_command = $3
+                if typem == :on_extended and specific
+                  bot_rules(dest, help_command, typem, rules_file, user)
+                else
+                  bot_help(user, user.name, dest, dchannel, specific, help_command, rules_file)
+                end
+                processed = true
+              end    
+            end
+            processed = (processed || general_bot_commands(user, command, dest, files) )
+            processed = (processed || general_commands(user, command, dest, files) ) if defined?(general_commands)
+            @logger.info "command: #{nick}> #{command}" if processed
+          end
 
-            if dest[0] == "C" or dest[0] == "G" or (dest[0] == "D" and typem == :on_call)
-              if typem != :on_call and @rules_imported.key?(user.id) and @rules_imported[user.id].key?(dchannel)
-                if @bots_created.key?(@rules_imported[user.id][dchannel])
-                  if @bots_created[@rules_imported[user.id][dchannel]][:status] != :on
-                    respond "The bot on that channel is not :on", dest
-                    rules_file = ""
+          if !config.on_maintenance and !processed and typem != :on_pub and typem != :on_pg
+            if @status == :on and
+              (!answer.empty? or
+              (@repl_sessions.key?(nick) and dest==@repl_sessions[nick][:dest] and 
+                ((@repl_sessions[nick][:on_thread] and thread_ts == @repl_sessions[nick][:thread_ts]) or
+                  (!@repl_sessions[nick][:on_thread] and !Thread.current[:on_thread] ))) or 
+              (@listening.key?(nick) and typem != :on_extended and 
+                ((@listening[nick].key?(dest) and !Thread.current[:on_thread]) or 
+                  (@listening[nick].key?(thread_ts) and Thread.current[:on_thread] ) )) or
+                dest[0] == "D" or on_demand)
+              @logger.info "command: #{nick}> #{command}" unless processed
+              #todo: verify this
+
+              if dest[0] == "C" or dest[0] == "G" or (dest[0] == "D" and typem == :on_call)
+                if typem != :on_call and @rules_imported.key?(user.name) and @rules_imported[user.name].key?(dchannel)
+                  if @bots_created.key?(@rules_imported[user.name][dchannel])
+                    if @bots_created[@rules_imported[user.name][dchannel]][:status] != :on
+                      respond "The bot on that channel is not :on", dest
+                      rules_file = ""
+                    end
                   end
                 end
-              end
-              unless rules_file.empty?
-                begin
-                  eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file)
-                rescue Exception => stack
-                  @logger.fatal "ERROR ON RULES FILE: #{rules_file}"
-                  @logger.fatal stack
+                unless rules_file.empty?
+                  begin
+                    eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file)
+                  rescue Exception => stack
+                    @logger.fatal "ERROR ON RULES FILE: #{rules_file}"
+                    @logger.fatal stack
+                  end
+                  if defined?(rules)
+                    command[0] = "" if command[0] == "!"
+                    command.gsub!(/^@\w+:*\s*/, "")
+                    if method(:rules).parameters.size == 4
+                      processed_rules = rules(user, command, processed, dest)
+                    elsif method(:rules).parameters.size == 5
+                      processed_rules = rules(user, command, processed, dest, files)
+                    else
+                      processed_rules = rules(user, command, processed, dest, files, rules_file)
+                    end
+                  else
+                    @logger.warn "It seems like rules method is not defined"
+                  end
                 end
-                if defined?(rules)
-                  command[0] = "" if command[0] == "!"
-                  command.gsub!(/^@\w+:*\s*/, "")
-                  if method(:rules).parameters.size == 4
-                    rules(user, command, processed, dest)
-                  elsif method(:rules).parameters.size == 5
-                    rules(user, command, processed, dest, files)
+              elsif @rules_imported.key?(user.name) and @rules_imported[user.name].key?(user.name)
+                if @bots_created.key?(@rules_imported[user.name][user.name])
+                  if @bots_created[@rules_imported[user.name][user.name]][:status] == :on
+                    begin
+                      eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file) and !['.','..'].include?(config.path + rules_file)
+                    rescue Exception => stack
+                      @logger.fatal "ERROR ON imported RULES FILE: #{rules_file}"
+                      @logger.fatal stack
+                    end
                   else
-                    rules(user, command, processed, dest, files, rules_file)
+                    respond "The bot on <##{@rules_imported[user.name][user.name]}|#{@bots_created[@rules_imported[user.name][user.name]][:channel_name]}> is not :on", dest
+                    rules_file = ""
                   end
-                else
-                  @logger.warn "It seems like rules method is not defined"
                 end
-              end
-            elsif @rules_imported.key?(user.id) and @rules_imported[user.id].key?(user.id)
-              if @bots_created.key?(@rules_imported[user.id][user.id])
-                if @bots_created[@rules_imported[user.id][user.id]][:status] == :on
-                  begin
-                    eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file) and !['.','..'].include?(config.path + rules_file)
-                  rescue Exception => stack
-                    @logger.fatal "ERROR ON imported RULES FILE: #{rules_file}"
-                    @logger.fatal stack
+
+                unless rules_file.empty?
+                  if defined?(rules)
+                    command[0] = "" if command[0] == "!"
+                    command.gsub!(/^@\w+:*\s*/, "")
+                    if method(:rules).parameters.size == 4
+                      processed_rules = rules(user, command, processed, dest)
+                    elsif method(:rules).parameters.size == 5
+                      processed_rules = rules(user, command, processed, dest, files)
+                    else
+                      processed_rules = rules(user, command, processed, dest, files, rules_file)
+                    end
+                  else
+                    @logger.warn "It seems like rules method is not defined"
                   end
-                else
-                  respond "The bot on <##{@rules_imported[user.id][user.id]}|#{@bots_created[@rules_imported[user.id][user.id]][:channel_name]}> is not :on", dest
-                  rules_file = ""
                 end
-              end
+              elsif dest[0] == 'D' and 
+                (!@rules_imported.key?(user.name) or ( @rules_imported.key?(user.name) and !@rules_imported[user.name].key?(user.name))) and 
+                rules_file.include?('general_rules.rb')
+                begin
+                  eval(File.new(config.path+rules_file).read) if File.exist?(config.path+rules_file) and !['.','..'].include?(config.path + rules_file)
+                rescue Exception => stack
+                  @logger.fatal "ERROR ON imported RULES FILE: #{rules_file}"
+                  @logger.fatal stack
+                end
 
-              unless rules_file.empty?
-                if defined?(rules)
+                if defined?(general_rules)
                   command[0] = "" if command[0] == "!"
                   command.gsub!(/^@\w+:*\s*/, "")
-                  if method(:rules).parameters.size == 4
-                    rules(user, command, processed, dest)
-                  elsif method(:rules).parameters.size == 5
-                    rules(user, command, processed, dest, files)
+                  #todo: check to change processed > processed_rules
+                  if method(:general_rules).parameters.size == 4
+                    processed = general_rules(user, command, processed, dest)
+                  elsif method(:general_rules).parameters.size == 5
+                    processed = general_rules(user, command, processed, dest, files)
                   else
-                    rules(user, command, processed, dest, files, rules_file)
+                    processed = general_rules(user, command, processed, dest, files, rules_file)
                   end
                 else
-                  @logger.warn "It seems like rules method is not defined"
+                  @logger.warn "It seems like general_rules method is not defined"
+                end
+                unless processed
+                  dont_understand('')
+                end
+              else
+                @logger.info "it is a direct message with no rules file selected so no rules file executed."
+                if command.match?(/^\s*bot\s+rules\s*(.*)$/i)
+                  respond "No rules running. You can use the command `use rules from CHANNEL` to specify the rules you want to use on this private conversation.\n`bot help` to see available commands.", dest
+                end
+                unless processed
+                  dont_understand('')
                 end
               end
-            else
-              @logger.info "it is a direct message with no rules file selected so no rules file executed."
-              if command.match?(/^\s*bot\s+rules\s*$/i)
-                respond "No rules running. You can use the command `use rules from CHANNEL` to specify the rules you want to use on this private conversation.\n`bot help` to see available commands.", dest
-              end
-              unless processed
-                dont_understand('')
-              end
-            end
 
-            if processed and @listening.key?(nick)
-              if Thread.current[:on_thread] and @listening[nick].key?(Thread.current[:thread_ts])
-                @listening[nick][Thread.current[:thread_ts]] = Time.now
-              elsif !Thread.current[:on_thread] and @listening[nick].key?(dest)
-                @listening[nick][dest] = Time.now
+              processed = (processed_rules || processed)
+
+              if processed and @listening.key?(nick)
+                if Thread.current[:on_thread] and @listening[nick].key?(Thread.current[:thread_ts])
+                  @listening[nick][Thread.current[:thread_ts]] = Time.now
+                elsif !Thread.current[:on_thread] and @listening[nick].key?(dest)
+                  @listening[nick][dest] = Time.now
+                end
               end
             end
-
+          end
+          
+          if processed and config.general_message != '' and !routine
+            respond eval("\"" + config.general_message + "\"")
           end
         rescue Exception => stack
           @logger.fatal stack

data/lib/slack/smart-bot/treat_message.rb

@@ -1,222 +1,361 @@
 class SlackSmartBot
   def treat_message(data, remove_blocks = true)
-
+    @buffered = false if config[:testing]
     begin
-      unless data.text.to_s.match(/\A\s*\z/)
-        #to remove italic, bold... from data.text since there is no method on slack api
-        #only works when no @user or #channel mentioned
-        if remove_blocks and !data.blocks.nil? and data.blocks.size > 0
-          data.blocks.each do |b|
-            if b.type == 'rich_text'
-              if b.elements.size > 0
-                b.elements.each do |e|
-                  if e.type == 'rich_text_section'
-                    if e.elements.size > 0 and (e.elements.type.uniq - ['link', 'text']) == []
-                      data.text = ''
-                      e.elements.each do |el|
-                        if el.type == 'text'
-                          data.text += el.text
-                        else
-                          data.text += el.url
+      begin
+        unless data.text.to_s.match(/\A\s*\z/)
+          #to remove italic, bold... from data.text since there is no method on slack api
+          if remove_blocks and !data.blocks.nil? and data.blocks.size > 0
+            data_text = ''
+            data.blocks.each do |b|
+              if b.type == 'rich_text'
+                if b.elements.size > 0
+                  b.elements.each do |e|
+                    if e.type == 'rich_text_section' or e.type == 'rich_text_preformatted'
+                      if e.elements.size > 0 and (e.elements.type.uniq - ['link', 'text', 'user', 'channel']) == []
+                        data_text += '```' if e.type == 'rich_text_preformatted'
+                        e.elements.each do |el|
+                          if el.type == 'text'
+                            data_text += el.text
+                          elsif el.type == 'user'
+                            data_text += "<@#{el.user_id}>"
+                          elsif el.type == 'channel'
+                            tch = data.text.scan(/(<##{el.channel_id}\|[^\>]*>)/).join
+                            data_text += tch
+                          else
+                            data_text += el.url
+                          end
                         end
+                        data_text += '```' if e.type == 'rich_text_preformatted'
                       end
                     end
-                    break
                   end
                 end
               end
-              break
             end
+            data.text = data_text unless data_text == ''
           end
+          data.text = CGI.unescapeHTML(data.text)
+          data.text.gsub!("\u00A0", " ") #to change &nbsp; (asc char 160) into blank space
         end
-        data.text = CGI.unescapeHTML(data.text)
-        data.text.gsub!("\u00A0", " ") #to change &nbsp; (asc char 160) into blank space
+        data.text.gsub!('‘', "'")
+        data.text.gsub!('’', "'")
+        data.text.gsub!('“', '"') 
+        data.text.gsub!('”', '"')
+      rescue Exception => exc
+        @logger.warn "Impossible to unescape or clean format for data.text:#{data.text}"
+        @logger.warn exc.inspect
       end
-      data.text.gsub!('’', "'")
-      data.text.gsub!('“', '"')
-    rescue
-      @logger.warn "Impossible to unescape or clean format for data.text:#{data.text}"
-    end
-    if config[:testing] and config.on_master_bot
-      open("#{config.path}/buffer.log", "a") { |f|
-        f.puts "|#{data.channel}|#{data.user}|#{data.text}"
-      }
-    end
-    if data.channel[0] == "D" or data.channel[0] == "C" or data.channel[0] == "G" #Direct message or Channel or Private Channel
-      dest = data.channel
-    else # not treated
-      dest = nil
-    end
-    #todo: sometimes data.user is nil, check the problem.
-    @logger.warn "!dest is nil. user: #{data.user}, channel: #{data.channel}, message: #{data.text}" if dest.nil?
-    if !data.files.nil? and data.files.size == 1 and data.text.to_s == "" and data.files[0].filetype == "ruby"
-      data.text = "ruby"
-    end
-    if !dest.nil? and config.on_master_bot and !data.text.nil? and data.text.match(/^ping from (.+)\s*$/) and data.user == config[:nick_id]
-      @pings << $1
-    end
-    typem = :dont_treat
-    if !dest.nil? and !data.text.nil? and !data.text.to_s.match?(/^\s*$/)
-      if data.text.match(/^<@#{config[:nick_id]}>\s(on\s)?<#(\w+)\|([^>]+)>\s*:?\s*(.*)/im)
-        channel_rules = $2
-        channel_rules_name = $3
-        # to be treated only on the bot of the requested channel
-        if @channel_id == channel_rules
-          data.text = $4
-          typem = :on_call
-        end
-      elsif dest == @master_bot_id
-        if config.on_master_bot #only to be treated on master bot channel
-          typem = :on_master
+      
+      unless data.key?(:routine)
+        data.routine = false 
+        data.routine_name = ''
+        data.routine_type = ''
+      end
+      if config[:testing] and config.on_master_bot and !@buffered
+        @buffered = true
+        open("#{config.path}/buffer.log", "a") { |f|
+          f.puts "|#{data.channel}|#{data.user}|#{data.user_name}|#{data.text}"
+        }
+      end
+      if data.key?(:dest) and data.dest.to_s!='' # for run routines and publish on different channels
+        dest = data.dest
+      elsif data.channel[0] == "D" or data.channel[0] == "C" or data.channel[0] == "G" #Direct message or Channel or Private Channel
+        dest = data.channel
+      else # not treated
+        dest = nil
+      end
+      #todo: sometimes data.user is nil, check the problem.
+      @logger.warn "!dest is nil. user: #{data.user}, channel: #{data.channel}, message: #{data.text}" if dest.nil?
+      if !data.files.nil? and data.files.size == 1 and data.text.to_s == "" and data.files[0].filetype == "ruby"
+        data.text = "ruby"
+      end
+      if !dest.nil? and config.on_master_bot and !data.text.nil? and data.text.match(/^ping from (.+)\s*$/) and data.user == config[:nick_id]
+        @pings << $1
+      end
+      typem = :dont_treat
+      if data.nil? or data.user.nil? or data.user.to_s==''
+        user_info = nil
+        @users = get_users() if @users.empty?
+      else
+        #todo: when changed @questions user_id then move user_info inside the ifs to avoid calling it when not necessary
+        user_info = @users.select{|u| u.id == data.user or (u.key?(:enterprise_user) and u.enterprise_user.id == data.user)}[-1]
+        if user_info.nil? or user_info.empty?
+          @users = get_users() 
+          user_info = @users.select{|u| u.id == data.user or (u.key?(:enterprise_user) and u.enterprise_user.id == data.user)}[-1]
         end
-      elsif @bots_created.key?(dest)
-        if @channel_id == dest #only to be treated by the bot on the channel
-          typem = :on_bot
+      end
+      if !dest.nil? and !data.text.nil? and !data.text.to_s.match?(/\A\s*\z/)
+        if data.channel[0] == "D" and !data.text.to_s.match?(/^\s*<@#{config[:nick_id]}>\s+/) and 
+          (data.text.to_s.match?(/^\s*(on)?\s*<#\w+\|[^>]*>/i) or data.text.to_s.match?(/^\s*(on)?\s*#\w+/i))
+          data.text = "<@#{config[:nick_id]}> " + data.text.to_s
         end
-      elsif dest[0] == "D" #Direct message
-        get_rules_imported()
-        if @rules_imported.key?(data.user) && @rules_imported[data.user].key?(data.user) and
-          @bots_created.key?(@rules_imported[data.user][data.user])
-          if @channel_id == @rules_imported[data.user][data.user]
-            #only to be treated by the channel we are 'using'
+        #todo: we need to add mixed channels: @smart-bot on private1 #bot1cm <#CXDDFRDDF|bot2cu>: echo A
+        if data.text.match(/\A\^\^+/) # to open a thread it will be only when starting by single ^
+          typem = :dont_treat
+        elsif data.text.match(/^\s*<@#{config[:nick_id]}>\s+(on\s+)?((<#\w+\|[^>]*>\s*)+)\s*:?\s*(.*)/im) or 
+          data.text.match(/^\s*<@#{config[:nick_id]}>\s+(on\s+)?((#[a-zA-Z0-9\-\_]+\s*)+)\s*:?\s*(.*)/im) or
+          data.text.match(/^\s*<@#{config[:nick_id]}>\s+(on\s+)?(([a-zA-Z0-9\-\_]+\s*)+)\s*:\s*(.*)/im)
+          channels_rules = $2 #multiple channels @smart-bot on #channel1 #channel2 echo AAA
+          data_text = $4
+          channel_rules_name = ''
+          channel_rules = ''
+          channels_arr = channels_rules.scan(/<#(\w+)\|([^>]*)>/)
+          if channels_arr.size == 0
+            channels_arr = []
+            channels_rules.scan(/([^\s]+)/).each do |cn|
+              cna = cn.join.gsub('#','')
+              if @channels_name.key?(cna)
+                channels_arr << [cna, @channels_name[cna]]
+              else
+                channels_arr << [@channels_id[cna], cna]
+              end
+            end
+          else
+            channels_arr.each do |row|
+              row[0] = @channels_id[row[1]] if row[0] == ''
+              row[1] = @channels_name[row[0]] if row[1] == ''              
+            end
+          end
+          
+          # to be treated only on the bots of the requested channels
+          channels_arr.each do |tcid, tcname|
+            if @channel_id == tcid
+              data.text = data_text
+              typem = :on_call
+              channel_rules = tcid
+              channel_rules_name = tcname
+              break
+            elsif @bots_created.key?(@channel_id) and @bots_created[@channel_id][:extended].include?(tcname)
+              data.text = data_text
+              typem = :on_call
+              channel_rules = @channel_id
+              channel_rules_name = @channels_name[@channel_id]
+              break
+            end
+          end
+
+        elsif data.channel == @master_bot_id
+          if config.on_master_bot #only to be treated on master bot channel
+            typem = :on_master
+          end
+        elsif @bots_created.key?(data.channel)
+          if @channel_id == data.channel #only to be treated by the bot on the channel
+            typem = :on_bot
+          end
+        elsif data.channel[0] == "D" #Direct message
+          get_rules_imported()
+          if @rules_imported.key?(user_info.name) && @rules_imported[user_info.name].key?(user_info.name) and
+            @bots_created.key?(@rules_imported[user_info.name][user_info.name])
+            if @channel_id == @rules_imported[user_info.name][user_info.name]
+              #only to be treated by the channel we are 'using'
+              typem = :on_dm
+            end
+          elsif config.on_master_bot
+            #only to be treated by master bot
             typem = :on_dm
           end
-        elsif config.on_master_bot
-          #only to be treated by master bot
-          typem = :on_dm
-        end
-      elsif dest[0] == "C" or dest[0] == "G"
-        #only to be treated on the channel of the bot. excluding running ruby
-        if !config.on_master_bot and @bots_created.key?(@channel_id) and @bots_created[@channel_id][:extended].include?(@channels_name[dest]) and
-           !data.text.match?(/^!?\s*(ruby|code)\s+/) and !data.text.match?(/^!?!?\s*(ruby|code)\s+/) and !data.text.match?(/^\^?\s*(ruby|code)\s+/)
-          typem = :on_extended
-        elsif config.on_master_bot and (data.text.match?(/^!?\s*(ruby|code)\s+/) or data.text.match?(/^!?!?\s*(ruby|code)\s+/) or data.text.match?(/^\^?\s*(ruby|code)\s+/)  )
-          #or in case of running ruby, the master bot
+        elsif data.channel[0] == "C" or data.channel[0] == "G"
+          #only to be treated on the channel of the bot. excluding running ruby
+          if !config.on_master_bot and @bots_created.key?(@channel_id) and @bots_created[@channel_id][:extended].include?(@channels_name[data.channel]) and
+            !data.text.match?(/^!?\s*(ruby|code)\s+/) and !data.text.match?(/^!?!?\s*(ruby|code)\s+/) and !data.text.match?(/^\^?\s*(ruby|code)\s+/)
+            typem = :on_extended
+          elsif config.on_master_bot and (data.text.match?(/^!?\s*(ruby|code)\s+/) or data.text.match?(/^!?!?\s*(ruby|code)\s+/) or data.text.match?(/^\^?\s*(ruby|code)\s+/)  )
+            #or in case of running ruby, the master bot
+            @bots_created.each do |k, v|
+              if v.key?(:extended) and v[:extended].include?(@channels_name[data.channel])
+                typem = :on_extended
+                break
+              end
+            end
+          end
+          extended = false
           @bots_created.each do |k, v|
-            if v.key?(:extended) and v[:extended].include?(@channels_name[dest])
-              typem = :on_extended
+            if v.key?(:extended) and v[:extended].include?(@channels_name[data.channel])
+              extended = true
               break
             end
           end
-        end
-        if dest[0] == "G" and config.on_master_bot and typem != :on_extended #private group
-          typem = :on_pg
+          if data.channel[0] == "G" and config.on_master_bot and !extended #private group
+            typem = :on_pg
+          elsif data.channel[0] == 'C' and config.on_master_bot and !extended #public group
+            typem = :on_pub
+          end
         end
       end
-    end
-    unless typem == :dont_treat
-      if (Time.now - @last_activity_check) > 60 * 30 #every 30 minutes
-        @last_activity_check = Time.now
-        @listening.each do |k,v|
-          v.each do |kk, vv|
-            @listening[k].delete(kk) if (Time.now - vv) > 60 * 30
-          end
-          @listening.delete(k) if @listening[k].empty?
+      load "#{config.path}/rules/general_commands.rb" if File.exists?("#{config.path}/rules/general_commands.rb") and @datetime_general_commands != File.mtime("#{config.path}/rules/general_commands.rb")
+      unless typem == :dont_treat or user_info.nil?
+        if (Time.now - @last_activity_check) > 60 * 30 #every 30 minutes
+          @last_activity_check = Time.now
+          @listening.each do |k,v|
+            v.each do |kk, vv|
+              @listening[k].delete(kk) if (Time.now - vv) > 60 * 30
+            end
+            @listening.delete(k) if @listening[k].empty?
+          end
         end
-      end
-      begin
-        #todo: when changed @questions user_id then move user_info inside the ifs to avoid calling it when not necessary
-        user_info = client.web_client.users_info(user: data.user)
-        #user_info.user.id = data.user #todo: remove this line when slack issue with Wxxxx Uxxxx fixed
-        data.user = user_info.user.id  #todo: remove this line when slack issue with Wxxxx Uxxxx fixed
-        if @questions.key?(user_info.user.name)
-          if data.text.match?(/^\s*(Bye|Bæ|Good\sBye|Adiós|Ciao|Bless|Bless\sBless|Adeu)\s(#{@salutations.join("|")})\s*$/i)
-            @questions.delete(user_info.user.name)
-            command = data.text
+        begin
+          #user_info.id = data.user #todo: remove this line when slack issue with Wxxxx Uxxxx fixed
+          data.user = user_info.id  #todo: remove this line when slack issue with Wxxxx Uxxxx fixed
+          if data.thread_ts.nil?
+            qdest = dest
           else
-            command = @questions[user_info.user.name]
-            @questions[user_info.user.name] = data.text
+            qdest = data.thread_ts
+          end
+          if !answer(user_info.name, qdest).empty?
+            if data.text.match?(/\A\s*(Bye|Bæ|Good\sBye|Adiós|Ciao|Bless|Bless\sBless|Adeu)\s(#{@salutations.join("|")})\s*$/i)
+              answer_delete(user_info.name, qdest)
+              command = data.text
+            else
+              command = answer(user_info.name, qdest)
+              @answer[user_info.name][qdest] = data.text
+              @questions[user_info.name] = data.text # to be backwards compatible #todo remove it when 2.0
+            end
+          elsif @repl_sessions.key?(user_info.name) and data.channel==@repl_sessions[user_info.name][:dest] and 
+            ((@repl_sessions[user_info.name][:on_thread] and data.thread_ts == @repl_sessions[user_info.name][:thread_ts]) or
+            (!@repl_sessions[user_info.name][:on_thread] and data.thread_ts.to_s == '' ))
+            
+            if data.text.match(/^\s*```(.*)```\s*$/im)
+                @repl_sessions[user_info.name][:command] = $1
+            else   
+              @repl_sessions[user_info.name][:command] = data.text
+            end
+            command = 'repl'
+          else
+            command = data.text
           end
-        elsif @repl_sessions.key?(user_info.user.name) and dest==@repl_sessions[user_info.user.name][:dest] and 
-          ((@repl_sessions[user_info.user.name][:on_thread] and data.thread_ts == @repl_sessions[user_info.user.name][:thread_ts]) or
-           (!@repl_sessions[user_info.user.name][:on_thread] and data.thread_ts.to_s == '' ))
-          
-          if data.text.match(/^\s*```(.*)```\s*$/im)
-              @repl_sessions[user_info.user.name][:command] = $1
-          else   
-            @repl_sessions[user_info.user.name][:command] = data.text
-          end
-          command = 'repl'
-        else
-          command = data.text
-        end
 
-        #when added special characters on the message
-        if command.match(/^\s*```(.*)```\s*$/im)
-          command = $1
-        elsif command.size >= 2 and
-           ((command[0] == "`" and command[-1] == "`") or (command[0] == "*" and command[-1] == "*") or (command[0] == "_" and command[-1] == "_"))
-          command = command[1..-2]
-        end
+          #when added special characters on the message
+          if command.match(/\A\s*```(.*)```\s*\z/im)
+            command = $1
+          elsif command.size >= 2 and
+            ((command[0] == "`" and command[-1] == "`") or (command[0] == "*" and command[-1] == "*") or (command[0] == "_" and command[-1] == "_"))
+            command = command[1..-2]
+          end
 
-        #ruby file attached
-        if !data.files.nil? and data.files.size == 1 and
-           (command.match?(/^(ruby|code)\s*$/) or (command.match?(/^\s*$/) and data.files[0].filetype == "ruby") or
-            (typem == :on_call and data.files[0].filetype == "ruby"))
-          res = Faraday.new("https://files.slack.com", headers: { "Authorization" => "Bearer #{config[:token]}" }).get(data.files[0].url_private)
-          command += " ruby" if command != "ruby"
-          command = "#{command} #{res.body.to_s.force_encoding("UTF-8")}"
-        end
+          #ruby file attached
+          if !data.files.nil? and data.files.size == 1 and
+            (command.match?(/^(ruby|code)\s*$/) or (command.match?(/^\s*$/) and data.files[0].filetype == "ruby") or
+              (typem == :on_call and data.files[0].filetype == "ruby"))
+            res = Faraday.new("https://files.slack.com", headers: { "Authorization" => "Bearer #{config[:token]}" }).get(data.files[0].url_private)
+            command += " ruby" if command != "ruby"
+            command = "#{command} #{res.body.to_s.force_encoding("UTF-8")}"
+          end
 
-        if typem == :on_call
-          command = "!" + command unless command[0] == "!" or command.match?(/^\s*$/) or command[0] == "^"
+          if typem == :on_call
+            command = "!" + command unless command[0] == "!" or command.match?(/^\s*$/) or command[0] == "^"
 
-          #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
-          channel_found = channels.detect { |c| c.name == channel_rules_name }
-          members = client.web_client.conversations_members(channel: @channels_id[channel_rules_name]).members unless channel_found.nil?
-          if channel_found.nil?
-            @logger.fatal "Not possible to find the channel #{channel_rules_name}"
-          elsif channel_found.name == config.master_channel
-            respond "You cannot use the rules from Master Channel on any other channel.", dest
-          elsif @status != :on
-            respond "The bot in that channel is not :on", dest
-          elsif data.user == channel_found.creator or members.include?(data.user)
-            process_first(user_info.user, command, dest, channel_rules, typem, data.files, data.ts, data.thread_ts)
-          else
-            respond "You need to join the channel <##{channel_found.id}> to be able to use the rules.", dest
-          end
-        elsif config.on_master_bot and typem == :on_extended and
-              command.size > 0 and command[0] != "-"
-          # to run ruby only from the master bot for the case more than one extended
-          process_first(user_info.user, command, dest, @channel_id, typem, data.files, data.ts, data.thread_ts)
-        elsif !config.on_master_bot and @bots_created[@channel_id].key?(:extended) and
-              @bots_created[@channel_id][:extended].include?(@channels_name[data.channel]) and
-              command.size > 0 and command[0] != "-"
-          process_first(user_info.user, command, dest, @channel_id, typem, data.files, data.ts, data.thread_ts)
-        elsif (dest[0] == "D" or @channel_id == data.channel or data.user == config[:nick_id]) and
-              command.size > 0 and command[0] != "-"
-          process_first(user_info.user, command, dest, data.channel, typem, data.files, data.ts, data.thread_ts)
-          # if @botname on #channel_rules: do something
+            #todo: add pagination for case more than 1000 channels on the workspace
+            channels = get_channels()
+            channel_found = channels.detect { |c| c.name == channel_rules_name }
+            members = get_channel_members(@channels_id[channel_rules_name]) unless channel_found.nil?
+            if channel_found.nil?
+              @logger.fatal "Not possible to find the channel #{channel_rules_name}"
+            elsif channel_found.name == config.master_channel
+              respond "You cannot use the rules from Master Channel on any other channel.", data.channel
+            elsif @status != :on
+              respond "The bot in that channel is not :on", data.channel
+            elsif data.user == channel_found.creator or members.include?(data.user)
+              process_first(user_info, command, dest, channel_rules, typem, data.files, data.ts, data.thread_ts, data.routine, data.routine_name, data.routine_type)
+            else
+              respond "You need to join the channel <##{channel_found.id}> to be able to use the rules.", data.channel
+            end
+          elsif config.on_master_bot and typem == :on_extended and
+                command.size > 0 and command[0] != "-"
+            # to run ruby only from the master bot for the case more than one extended
+            process_first(user_info, command, dest, @channel_id, typem, data.files, data.ts, data.thread_ts, data.routine, data.routine_name, data.routine_type)
+          elsif !config.on_master_bot and @bots_created[@channel_id].key?(:extended) and
+                @bots_created[@channel_id][:extended].include?(@channels_name[data.channel]) and
+                command.size > 0 and command[0] != "-"
+            process_first(user_info, command, dest, @channel_id, typem, data.files, data.ts, data.thread_ts, data.routine, data.routine_name, data.routine_type)
+          elsif (dest[0] == "D" or @channel_id == data.channel or data.user == config[:nick_id]) and
+                command.size > 0 and command[0] != "-"
+            process_first(user_info, command, dest, data.channel, typem, data.files, data.ts, data.thread_ts, data.routine, data.routine_name, data.routine_type)
+            # if @botname on #channel_rules: do something
+          elsif typem == :on_pub or typem == :on_pg
+            process_first(user_info, command, dest, channel_rules, typem, data.files, data.ts, data.thread_ts, data.routine, data.routine_name, data.routine_type)
+          end
+        rescue Exception => stack
+          @logger.fatal stack
         end
-      rescue Exception => stack
-        @logger.fatal stack
-      end
-    else
-      if !config.on_master_bot and !dest.nil? and (dest == @master_bot_id or dest[0] == "D") and
-         data.text.match?(/^\s*bot\s+status\s*$/i) and @admin_users_id.include?(data.user)
-        respond "ping from #{config.channel}", dest
-      elsif !config.on_master_bot and !dest.nil? and data.user == config[:nick_id] and dest == @master_bot_id
-        # to treat on other bots the status messages populated on master bot
-        case data.text
-        when /^Bot has been (closed|killed) by/i
-          sleep 2
-          get_bots_created()
-        when /^Changed status on (.+) to :(.+)/i
-          sleep 2
-          get_bots_created()
-        when /extended the rules from (.+) to be used on (.+)\.$/i
-          sleep 2
-          get_bots_created()
-        when /removed the access to the rules of (.+) from (.+)\.$/i
-          sleep 2
-          get_bots_created()
+
+      else
+        @logger.warn "Pay attention there is no user on users with id #{data.user}" if user_info.nil?
+        if !config.on_master_bot and !dest.nil? and (data.channel == @master_bot_id or dest[0] == "D") and
+          data.text.match?(/^\s*(!|!!|\^)?\s*bot\s+status\s*$/i) and @admin_users_id.include?(data.user)
+          respond "ping from #{config.channel}", dest
+        elsif !config.on_master_bot and !dest.nil? and data.user == config[:nick_id] and dest == @master_bot_id
+          # to treat on other bots the status messages populated on master bot
+          case data.text
+          when /General message has been set\./i, /General message won't be displayed anymore./i
+            sleep 2
+            if File.exist?("#{config.path}/config_tmp.status")
+              file_cts = IO.readlines("#{config.path}/config_tmp.status").join
+              unless file_cts.to_s() == ""
+                file_cts = eval(file_cts)
+                if file_cts.is_a?(Hash) and file_cts.key?(:general_message)
+                  config.general_message = file_cts.general_message
+                end
+              end
+            end
+          when /From now on I'll be on maintenance status/i
+            sleep 2
+            if File.exist?("#{config.path}/config_tmp.status")
+              file_cts = IO.readlines("#{config.path}/config_tmp.status").join
+              unless file_cts.to_s() == ""
+                file_cts = eval(file_cts)
+                if file_cts.is_a?(Hash) and file_cts.key?(:on_maintenance)
+                  config.on_maintenance = file_cts.on_maintenance
+                  config.on_maintenance_message = file_cts.on_maintenance_message
+                end
+              end
+            end
+          when /From now on I won't be on maintenance/i
+            sleep 2
+            if File.exist?("#{config.path}/config_tmp.status")
+              file_cts = IO.readlines("#{config.path}/config_tmp.status").join
+              unless file_cts.to_s() == ""
+                file_cts = eval(file_cts)
+                if file_cts.is_a?(Hash) and file_cts.key?(:on_maintenance)
+                  config.on_maintenance = file_cts.on_maintenance
+                  config.on_maintenance_message = file_cts.on_maintenance_message
+                end
+              end
+            end
+            
+          when /^Bot has been (closed|killed) by/i
+            sleep 2
+            get_bots_created()
+          when /^Changed status on (.+) to :(.+)/i
+            sleep 2
+            get_bots_created()
+          when /extended the rules from (.+) to be used on (.+)\.$/i
+            sleep 2
+            get_bots_created()
+          when /removed the access to the rules of (.+) from (.+)\.$/i
+            sleep 2
+            get_bots_created()
+          when /global shortcut added/
+            sleep 2
+            if File.exist?("#{config.path}/shortcuts/shortcuts_global.rb")
+              file_sc = IO.readlines("#{config.path}/shortcuts/shortcuts_global.rb").join
+              unless file_sc.to_s() == ""
+                @shortcuts_global = eval(file_sc)
+              end
+            end
+          when /global shortcut deleted/
+            sleep 2
+            if File.exist?("#{config.path}/shortcuts/shortcuts_global.rb")
+              file_sc = IO.readlines("#{config.path}/shortcuts/shortcuts_global.rb").join
+              unless file_sc.to_s() == ""
+                @shortcuts_global = eval(file_sc)
+              end
+            end
+          end
         end
       end
+    rescue Exception => stack
+      @logger.fatal stack
     end
   end
 end