]> git.netwichtig.de Git - user/henk/code/ruby/rbot.git/commitdiff
New modular framework is in place. Nothing works until core/auth.rb is done, though
authorGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Tue, 1 Aug 2006 17:14:01 +0000 (17:14 +0000)
committerGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Tue, 1 Aug 2006 17:14:01 +0000 (17:14 +0000)
lib/rbot/botuser.rb
lib/rbot/core/core.rb [new file with mode: 0644]
lib/rbot/ircbot.rb
lib/rbot/plugins.rb

index 28a5410814eb95ecaf18db02cf2a5f28c0d292e7..67d7d842bcabde86d32d98a591197e6e96abc94f 100644 (file)
@@ -171,13 +171,13 @@ module Irc
 \r
   # This method raises a TypeError if _user_ is not of class User\r
   #\r
-  def error_if_not_user(user)\r
+  def Irc.error_if_not_user(user)\r
     raise TypeError, "#{user.inspect} must be of type Irc::User and not #{user.class}" unless user.class <= User\r
   end\r
 \r
   # This method raises a TypeError if _chan_ is not of class Chan\r
   #\r
-  def error_if_not_channel(chan)\r
+  def Irc.error_if_not_channel(chan)\r
     raise TypeError, "#{chan.inspect} must be of type Irc::User and not #{chan.class}" unless chan.class <= Channel\r
   end\r
 \r
@@ -219,25 +219,26 @@ module Irc
       # the command as a symbol with the :command method and the whole\r
       # path as :path\r
       #\r
-      #   Command.new("core::auth::save").path => [:"", :core, :"core::auth", :"core::auth::save"]\r
+      #   Command.new("core::auth::save").path => [:"*", :"core", :"core::auth", :"core::auth::save"]\r
       #\r
       #   Command.new("core::auth::save").command => :"core::auth::save"\r
       #\r
       def initialize(cmd)\r
         cmdpath = sanitize_command_path(cmd).split('::')\r
-        seq = cmdpath.inject([""]) { |list, cmd|\r
-          list << (list.last ? list.last + "::" : "") + cmd\r
+        seq = cmdpath.inject(["*"]) { |list, cmd|\r
+          list << (list.length > 1 ? list.last + "::" : "") + cmd\r
         }\r
         @path = seq.map { |k|\r
           k.to_sym\r
         }\r
         @command = path.last\r
+        debug "Created command #{@command.inspect} with path #{@path.join(', ')}"\r
       end\r
     end\r
 \r
     # This method raises a TypeError if _user_ is not of class User\r
     #\r
-    def error_if_not_command(cmd)\r
+    def Irc.error_if_not_command(cmd)\r
       raise TypeError, "#{cmd.inspect} must be of type Irc::Auth::Command and not #{cmd.class}" unless cmd.class <= Command\r
     end\r
 \r
@@ -256,7 +257,7 @@ module Irc
       #\r
       def set_permission(cmd, val)\r
         raise TypeError, "#{val.inspect} must be true or false" unless [true,false].include?(val)\r
-        error_if_not_command(cmd)\r
+        Irc::error_if_not_command(cmd)\r
         cmd.path.each { |k|\r
           set_permission(k.to_s, true) unless @perm.has_key?(k)\r
         }\r
@@ -266,8 +267,8 @@ module Irc
       # Tells if command _cmd_ is permitted. We do this by returning\r
       # the value of the deepest Command#path that matches.\r
       #\r
-      def allow?(cmd)\r
-        error_if_not_command(cmd)\r
+      def permit?(cmd)\r
+        Irc::error_if_not_command(cmd)\r
         allow = nil\r
         cmd.path.reverse.each { |k|\r
           if @perm.has_key?(k)\r
@@ -313,7 +314,7 @@ module Irc
       # Checks if BotUser is allowed to do something on channel _chan_,\r
       # or on all channels if _chan_ is nil\r
       #\r
-      def allow?(cmd, chan=nil)\r
+      def permit?(cmd, chan=nil)\r
         if chan\r
           k = chan.to_s.to_sym\r
         else\r
@@ -321,7 +322,7 @@ module Irc
         end\r
         allow = nil\r
         if @perm.has_key?(k)\r
-          allow = @perm[k].allow?(cmd)\r
+          allow = @perm[k].permit?(cmd)\r
         end\r
         return allow\r
       end\r
@@ -356,7 +357,7 @@ module Irc
 \r
       # This method checks if BotUser has a Netmask that matches _user_\r
       def knows?(user)\r
-        error_if_not_user(user)\r
+        Irc::error_if_not_user(user)\r
         known = false\r
         @netmasks.each { |n|\r
           if user.matches?(n)\r
@@ -424,7 +425,7 @@ module Irc
 \r
       # Anon knows everybody\r
       def knows?(user)\r
-        error_if_not_user(user)\r
+        Irc::error_if_not_user(user)\r
         return true\r
       end\r
 \r
@@ -437,7 +438,7 @@ module Irc
 \r
     # Returns the only instance of AnonBotUserClass\r
     #\r
-    def Auth::anonbotuser\r
+    def Auth.anonbotuser\r
       return AnonBotUserClass.instance\r
     end\r
 \r
@@ -449,14 +450,14 @@ module Irc
         super("owner")\r
       end\r
 \r
-      def allow?(cmd, chan=nil)\r
+      def permit?(cmd, chan=nil)\r
         return true\r
       end\r
     end\r
 \r
     # Returns the only instance of BotOwnerClass\r
     #\r
-    def Auth::botowner\r
+    def Auth.botowner\r
       return BotOwnerClass.instance\r
     end\r
 \r
@@ -520,8 +521,8 @@ module Irc
 \r
       # Maps <code>Irc::User</code> to BotUser\r
       def irc_to_botuser(ircuser)\r
-        error_if_not_user(ircuser)\r
-        return @botusers[ircuser] || anonbotuser\r
+        Irc::error_if_not_user(ircuser)\r
+        return @botusers[ircuser] || Auth::anonbotuser\r
       end\r
 \r
       # creates a new BotUser\r
@@ -541,7 +542,7 @@ module Irc
       # It is possible to autologin by Netmask, on request\r
       #\r
       def login(ircuser, botusername, pwd, bymask = false)\r
-        error_if_not_user(ircuser)\r
+        Irc::error_if_not_user(ircuser)\r
         n = BotUser.sanitize_username(name)\r
         k = n.to_sym\r
         raise "No such BotUser #{n}" unless include?(k)\r
@@ -569,11 +570,10 @@ module Irc
       # * anonbotuser on _chan_\r
       # * anonbotuser on all channels\r
       #\r
-      def allow?(user, cmdtxt, chan=nil)\r
-        error_if_not_user(user)\r
+      def permit?(user, cmdtxt, chan=nil)\r
+        botuser = irc_to_botuser(user)\r
         cmd = Command.new(cmdtxt)\r
-        allow = nil\r
-        botuser = @botusers[user]\r
+\r
         case chan\r
         when User\r
           chan = "?"\r
@@ -581,20 +581,27 @@ module Irc
           chan = chan.name\r
         end\r
 \r
-        allow = botuser.allow?(cmd, chan) if chan\r
+        allow = nil\r
+\r
+        allow = botuser.permit?(cmd, chan) if chan\r
         return allow unless allow.nil?\r
-        allow = botuser.allow?(cmd)\r
+        allow = botuser.permit?(cmd)\r
         return allow unless allow.nil?\r
 \r
-        unless botuser == anonbotuser\r
-          allow = anonbotuser.allow?(cmd, chan) if chan\r
+        unless botuser == Auth::anonbotuser\r
+          allow = Auth::anonbotuser.permit?(cmd, chan) if chan\r
           return allow unless allow.nil?\r
-          allow = anonbotuser.allow?(cmd)\r
+          allow = Auth::anonbotuser.permit?(cmd)\r
           return allow unless allow.nil?\r
         end\r
 \r
         raise "Could not check permission for user #{user.inspect} to run #{cmdtxt.inspect} on #{chan.inspect}"\r
       end\r
+\r
+      # Checks if command _cmd_ is allowed to User _user_ on _chan_\r
+      def allow?(cmdtxt, user, chan=nil)\r
+        permit?(user, cmdtxt, chan)\r
+      end\r
     end\r
 \r
     # Returns the only instance of AuthManagerClass\r
diff --git a/lib/rbot/core/core.rb b/lib/rbot/core/core.rb
new file mode 100644 (file)
index 0000000..c9210d5
--- /dev/null
@@ -0,0 +1,155 @@
+#-- vim:sw=2:et\r
+#++\r
+\r
+\r
+class Core < CoreBotModule\r
+\r
+  # TODO cleanup\r
+  # handle incoming IRC PRIVMSG +m+\r
+  def listen(m)\r
+    return unless m.class <= PrivMessage\r
+    if(m.private? && m.message =~ /^\001PING\s+(.+)\001/)\r
+      @bot.notice m.sourcenick, "\001PING #$1\001"\r
+      @bot.irclog "@ #{m.sourcenick} pinged me"\r
+      return\r
+    end\r
+\r
+    if(m.address?)\r
+      case m.message\r
+      when (/^join\s+(\S+)\s+(\S+)$/i)\r
+        @bot.join $1, $2 if(@bot.auth.allow?("join", m.source, m.replyto))\r
+      when (/^join\s+(\S+)$/i)\r
+        @bot.join $1 if(@bot.auth.allow?("join", m.source, m.replyto))\r
+      when (/^part$/i)\r
+        @bot.part m.target if(m.public? && @bot.auth.allow?("join", m.source, m.replyto))\r
+      when (/^part\s+(\S+)$/i)\r
+        @bot.part $1 if(@bot.auth.allow?("join", m.source, m.replyto))\r
+      when (/^quit(?:\s+(.*))?$/i)\r
+        @bot.quit $1 if(@bot.auth.allow?("quit", m.source, m.replyto))\r
+      when (/^restart(?:\s+(.*))?$/i)\r
+        @bot.restart $1 if(@bot.auth.allow?("quit", m.source, m.replyto))\r
+      when (/^hide$/i)\r
+        @bot.join 0 if(@bot.auth.allow?("join", m.source, m.replyto))\r
+      when (/^save$/i)\r
+        if(@bot.auth.allow?("config", m.source, m.replyto))\r
+          @bot.save\r
+          m.okay\r
+        end\r
+      when (/^nick\s+(\S+)$/i)\r
+        @bot.nickchg($1) if(@bot.auth.allow?("nick", m.source, m.replyto))\r
+      when (/^say\s+(\S+)\s+(.*)$/i)\r
+        @bot.say $1, $2 if(@bot.auth.allow?("say", m.source, m.replyto))\r
+      when (/^action\s+(\S+)\s+(.*)$/i)\r
+        @bot.action $1, $2 if(@bot.auth.allow?("say", m.source, m.replyto))\r
+        # when (/^topic\s+(\S+)\s+(.*)$/i)\r
+        #   topic $1, $2 if(@bot.auth.allow?("topic", m.source, m.replyto))\r
+      when (/^mode\s+(\S+)\s+(\S+)\s+(.*)$/i)\r
+        @bot.mode $1, $2, $3 if(@bot.auth.allow?("mode", m.source, m.replyto))\r
+      when (/^ping$/i)\r
+        @bot.say m.replyto, "pong"\r
+      when (/^rescan$/i)\r
+        if(@bot.auth.allow?("config", m.source, m.replyto))\r
+          m.reply "saving ..."\r
+          @bot.save\r
+          m.reply "rescanning ..."\r
+          @bot.rescan\r
+          m.reply "done. #{@plugins.status(true)}"\r
+        end\r
+      when (/^quiet$/i)\r
+        if(@bot.auth.allow?("talk", m.source, m.replyto))\r
+          m.okay\r
+          @bot.set_quiet\r
+        end\r
+      when (/^quiet in (\S+)$/i)\r
+        where = $1\r
+        if(@bot.auth.allow?("talk", m.source, m.replyto))\r
+          m.okay\r
+          where.gsub!(/^here$/, m.target) if m.public?\r
+          @bot.set_quiet(where)\r
+        end\r
+      when (/^talk$/i)\r
+        if(@bot.auth.allow?("talk", m.source, m.replyto))\r
+          @bot.reset_quiet\r
+          m.okay\r
+        end\r
+      when (/^talk in (\S+)$/i)\r
+        where = $1\r
+        if(@bot.auth.allow?("talk", m.source, m.replyto))\r
+          where.gsub!(/^here$/, m.target) if m.public?\r
+          @bot.reset_quiet(where)\r
+          m.okay\r
+        end\r
+      when (/^status\??$/i)\r
+        m.reply status if @bot.auth.allow?("status", m.source, m.replyto)\r
+      when (/^registry stats$/i)\r
+        if @bot.auth.allow?("config", m.source, m.replyto)\r
+          m.reply @registry.stat.inspect\r
+        end\r
+      when (/^(help\s+)?config(\s+|$)/)\r
+        @config.privmsg(m)\r
+      when (/^(version)|(introduce yourself)$/i)\r
+        @bot.say m.replyto, "I'm a v. #{$version} rubybot, (c) Tom Gilbert - http://linuxbrit.co.uk/rbot/"\r
+      when (/^help(?:\s+(.*))?$/i)\r
+        @bot.say m.replyto, help($1)\r
+        #TODO move these to a "chatback" plugin\r
+      when (/^(botsnack|ciggie)$/i)\r
+        @bot.say m.replyto, @lang.get("thanks_X") % m.sourcenick if(m.public?)\r
+        @bot.say m.replyto, @lang.get("thanks") if(m.private?)\r
+      when (/^(hello|howdy|hola|salut|bonjour|sup|niihau|hey|hi(\W|$)|yo(\W|$)).*/i)\r
+        @bot.say m.replyto, @lang.get("hello_X") % m.sourcenick if(m.public?)\r
+        @bot.say m.replyto, @lang.get("hello") if(m.private?)\r
+      end\r
+    else\r
+      # stuff to handle when not addressed\r
+      case m.message\r
+      when (/^\s*(hello|howdy|hola|salut|bonjour|sup|niihau|hey|hi|yo(\W|$))[\s,-.]+#{Regexp.escape(@bot.nick)}$/i)\r
+        @bot.say m.replyto, @lang.get("hello_X") % m.sourcenick\r
+      when (/^#{Regexp.escape(@bot.nick)}!*$/)\r
+        @bot.say m.replyto, @lang.get("hello_X") % m.sourcenick\r
+      else\r
+        # @keywords.privmsg(m)\r
+      end\r
+    end\r
+  end\r
+\r
+  # handle help requests for "core" topics\r
+  def help(topic="")\r
+    case topic\r
+    when "quit"\r
+      return "quit [<message>] => quit IRC with message <message>"\r
+    when "restart"\r
+      return "restart => completely stop and restart the bot (including reconnect)"\r
+    when "join"\r
+      return "join <channel> [<key>] => join channel <channel> with secret key <key> if specified. #{myself} also responds to invites if you have the required access level"\r
+    when "part"\r
+      return "part <channel> => part channel <channel>"\r
+    when "hide"\r
+      return "hide => part all channels"\r
+    when "save"\r
+      return "save => save current dynamic data and configuration"\r
+    when "rescan"\r
+      return "rescan => reload modules and static facts"\r
+    when "nick"\r
+      return "nick <nick> => attempt to change nick to <nick>"\r
+    when "say"\r
+      return "say <channel>|<nick> <message> => say <message> to <channel> or in private message to <nick>"\r
+    when "action"\r
+      return "action <channel>|<nick> <message> => does a /me <message> to <channel> or in private message to <nick>"\r
+    when "quiet"\r
+      return "quiet [in here|<channel>] => with no arguments, stop speaking in all channels, if \"in here\", stop speaking in this channel, or stop speaking in <channel>"\r
+    when "talk"\r
+      return "talk [in here|<channel>] => with no arguments, resume speaking in all channels, if \"in here\", resume speaking in this channel, or resume speaking in <channel>"\r
+    when "version"\r
+      return "version => describes software version"\r
+    when "botsnack"\r
+      return "botsnack => reward #{myself} for being good"\r
+    when "hello"\r
+      return "hello|hi|hey|yo [#{myself}] => greet the bot"\r
+    else\r
+      return "Core help topics: quit, restart, config, join, part, hide, save, rescan, nick, say, action, topic, quiet, talk, version, botsnack, hello"\r
+    end\r
+  end\r
+end\r
+\r
+core = Core.new\r
+\r
index 42f39b163fad89c2ab002de850d1c29da24bbd1d..d567189b7c74c04e1a6b8a68983f5b6d4813d6bb 100644 (file)
@@ -350,8 +350,10 @@ class IrcBot
     Dir.mkdir("#{botclass}/plugins") unless File.exist?("#{botclass}/plugins")
     @plugins = Plugins::pluginmanager
     @plugins.bot_associate(self)
-    @plugins.load_core(Config::coredir)
-    @plugins.load_plugins(["#{botclass}/plugins"])
+    @plugins.add_botmodule_dir(Config::coredir)
+    @plugins.add_botmodule_dir("#{botclass}/plugins")
+    @plugins.add_botmodule_dir(Config::datadir + "/plugins")
+    @plugins.scan
 
     @socket = IrcSocket.new(@config['server.name'], @config['server.port'], @config['server.bindhost'], @config['server.sendq_delay'], @config['server.sendq_burst'])
     @client = IrcClient.new
@@ -364,7 +366,6 @@ class IrcBot
     # in all channels, a list of channels otherwise
     @quiet = nil
 
-
     @client[:welcome] = proc {|data|
       irclog "joined server #{@client.server} as #{myself}", "server"
 
@@ -735,8 +736,6 @@ class IrcBot
     case where
     when Channel
       irclog "* #{myself} #{message}", where
-    when User
-      irclog "* #{myself}[#{where}] #{message}", $1
     else
       irclog "* #{myself}[#{where}] #{message}", where
     end
@@ -1003,7 +1002,6 @@ class IrcBot
     end
   end
 
-  # respond to being kicked from a channel
   def irclogkick(m)
     if(m.address?)
       debug "kicked from channel #{m.channel}"
index 546a9b30882cfc3a8613fc326e3b79945dae8b25..bb4c744a6c238c9a821db33a44b3c4db8642ec4a 100644 (file)
@@ -98,13 +98,21 @@ module Plugins
 
   class BotModule
     attr_reader :bot   # the associated bot
+    attr_reader :botmodule_class # the botmodule class (:coremodule or :plugin)
+
     # initialise your bot module. Always call super if you override this method,
     # as important variables are set up for you
-    def initialize
-      @bot = Plugins.pluginmanager.bot
+    def initialize(kl)
+      @manager = Plugins::pluginmanager
+      @bot = @manager.bot
+
+      @botmodule_class = kl.to_sym
       @botmodule_triggers = Array.new
+
       @handler = MessageMapper.new(self)
       @registry = BotRegistryAccessor.new(@bot, self.class.to_s.gsub(/^.*::/, ""))
+
+      @manager.add_botmodule(kl, self)
     end
 
     def flush_registry
@@ -148,7 +156,12 @@ module Plugins
     # return an identifier for this plugin, defaults to a list of the message
     # prefixes handled (used for error messages etc)
     def name
-      self.class.downcase.sub(/(plugin)?$/,"")
+      self.class.to_s.downcase.sub(/^#<module:.*?>::/,"").sub(/(plugin)?$/,"")
+    end
+
+    # just calls name
+    def to_s
+      name
     end
 
     # return a help string for your module. for complex modules, you may wish
@@ -163,10 +176,10 @@ module Plugins
     # register the plugin as a handler for messages prefixed +name+
     # this can be called multiple times for a plugin to handle multiple
     # message prefixes
-    def register(name, kl, opts={})
-      raise ArgumentError, "Third argument must be a hash!" unless opts.kind_of?(Hash)
-      return if Plugins.pluginmanager.botmodules[kl].has_key?(name)
-      Plugins.pluginmanager.botmodules[kl][name] = self
+    def register(name, opts={})
+      raise ArgumentError, "Second argument must be a hash!" unless opts.kind_of?(Hash)
+      return if @manager.knows?(name, @botmodule_class)
+      @manager.register(name, @botmodule_class, self)
       @botmodule_triggers << name unless opts.fetch(:hidden, false)
     end
 
@@ -179,20 +192,18 @@ module Plugins
   end
 
   class CoreBotModule < BotModule
-    def register(name, opts={})
-      raise ArgumentError, "Second argument must be a hash!" unless opts.kind_of?(Hash)
-      super(name, :core, opts)
+    def initialize
+      super(:coremodule)
     end
   end
 
   class Plugin < BotModule
-    def register(name, opts={})
-      raise ArgumentError, "Second argument must be a hash!" unless opts.kind_of?(Hash)
-      super(name, :plugin, opts)
+    def initialize
+      super(:plugin)
     end
   end
 
-  # class to manage multiple plugins and delegate messages to them for
+  # Singleton to manage multiple plugins and delegate messages to them for
   # handling
   class PluginManagerClass
     include Singleton
@@ -201,29 +212,67 @@ module Plugins
 
     def initialize
       bot_associate(nil)
+
+      @dirs = []
     end
 
-    # Associate with bot _bot_
-    def bot_associate(bot)
+    # Reset lists of botmodules
+    def reset_botmodule_lists
       @botmodules = {
-        :core => Hash.new,
-        :plugin => Hash.new
+        :coremodule => [],
+        :plugin => []
       }
 
-      # associated IrcBot class
+      @commandmappers = {
+        :coremodule => {},
+        :plugin => {}
+      }
+
+    end
+
+    # Associate with bot _bot_
+    def bot_associate(bot)
+      reset_botmodule_lists
       @bot = bot
     end
 
-    # Returns a hash of the registered message prefixes and associated
-    # plugins
+    # Returns +true+ if _name_ is a known botmodule of class kl
+    def knows?(name, kl)
+      return @commandmappers[kl.to_sym].has_key?(name.to_sym)
+    end
+
+    # Returns +true+ if _name_ is a known botmodule of class kl
+    def register(name, kl, botmodule)
+      raise TypeError, "Third argument #{botmodule.inspect} is not of class BotModule" unless botmodule.class <= BotModule
+      @commandmappers[kl.to_sym][name.to_sym] = botmodule
+    end
+
+    def add_botmodule(kl, botmodule)
+      raise TypeError, "Second argument #{botmodule.inspect} is not of class BotModule" unless botmodule.class <= BotModule
+      raise "#{kl.to_s} #{botmodule.name} already registered!" if @botmodules[kl.to_sym].include?(botmodule)
+      @botmodules[kl.to_sym] << botmodule
+    end
+
+    # Returns an array of the loaded plugins
+    def core_modules
+      @botmodules[:coremodule]
+    end
+
+    # Returns an array of the loaded plugins
     def plugins
       @botmodules[:plugin]
     end
 
+    # Returns a hash of the registered message prefixes and associated
+    # plugins
+    def plugin_commands
+      @commandmappers[:plugin]
+    end
+
     # Returns a hash of the registered message prefixes and associated
     # core modules
-    def core_modules
-      @botmodules[:core]
+    def core_commands
+      @commandmappers[:coremodule]
     end
 
     # Makes a string of error _err_ by adding text _str_
@@ -246,6 +295,7 @@ module Plugins
       plugin_module = Module.new
 
       desc = desc.to_s + " " if desc
+
       begin
         plugin_string = IO.readlines(fname).join("")
         debug "loading #{desc}#{fname}"
@@ -272,31 +322,12 @@ module Plugins
     end
     private :load_botmodule_file
 
-    # Load core botmodules
-    def load_core(dir)
-      # TODO FIXME should this be hardcoded?
-      if(FileTest.directory?(dir))
-        d = Dir.new(dir)
-        d.sort.each { |file|
-          next unless(file =~ /[^.]\.rb$/)
-
-          did_it = load_botmodule_file("#{dir}/#{file}", "core module")
-          case did_it
-          when Symbol
-            # debug "loaded core botmodule #{dir}/#{file}"
-          when Exception
-            raise "failed to load core botmodule #{dir}/#{file}!"
-          end
-        }
-      end
-    end
-
-    # dirlist:: array of directories to scan (in order) for plugins
+    # add one or more directories to the list of directories to
+    # load botmodules from
     #
-    # create a new plugin handler, scanning for plugins in +dirlist+
-    def load_plugins(dirlist)
-      @dirs = dirlist
-      scan
+    def add_botmodule_dir(*dirlist)
+      @dirs += dirlist
+      debug "Botmodule loading path: #{@dirs.join(', ')}"
     end
 
     # load plugins from pre-assigned list of directories
@@ -310,11 +341,8 @@ module Plugins
         processed[pn.intern] = :blacklisted
       }
 
-      dirs = Array.new
-      # TODO FIXME should this be hardcoded?
-      dirs << Config::datadir + "/plugins"
-      dirs += @dirs
-      dirs.reverse.each {|dir|
+      dirs = @dirs
+      dirs.each {|dir|
         if(FileTest.directory?(dir))
           d = Dir.new(dir)
           d.sort.each {|file|
@@ -349,6 +377,7 @@ module Plugins
           }
         end
       }
+      debug "finished loading plugins: #{status(true)}"
     end
 
     # call the save method for each active plugin
@@ -360,6 +389,7 @@ module Plugins
     # call the cleanup method for each active plugin
     def cleanup
       delegate 'cleanup'
+      reset_botmodule_lists
     end
 
     # drop all plugins and rescan plugins on disk
@@ -367,21 +397,31 @@ module Plugins
     def rescan
       save
       cleanup
-      plugins.clear
       scan
     end
 
     def status(short=false)
+      list = ""
+      if self.core_length > 0
+        list << "#{self.core_length} core module#{'s' if core_length > 1}"
+        if short
+          list << " loaded"
+        else
+          list << ": " + core_modules.collect{ |p| p.name}.sort.join(", ")
+        end
+      else
+        list << "no core botmodules loaded"
+      end
       # Active plugins first
       if(self.length > 0)
-        list = "#{self.length} plugin#{'s' if length > 1}"
+        list << "; #{self.length} plugin#{'s' if length > 1}"
         if short
           list << " loaded"
         else
-          list << ": " + @@plugins.values.uniq.collect{|p| p.name}.sort.join(", ")
+          list << ": " + plugins.collect{ |p| p.name}.sort.join(", ")
         end
       else
-        list = "no plugins active"
+        list << "no plugins active"
       end
       # Ignored plugins next
       unless @ignored.empty?
@@ -402,7 +442,11 @@ module Plugins
     end
 
     def length
-      plugins.values.uniq.length
+      plugins.length
+    end
+
+    def core_length
+      core_modules.length
     end
 
     # return help for +topic+ (call associated plugin's help method)
@@ -431,57 +475,87 @@ module Plugins
       when /^(\S+)\s*(.*)$/
         key = $1
         params = $2
-        if(@@plugins.has_key?(key))
-          begin
-            return @@plugins[key].help(key, params)
-          rescue Exception => err
-            #rescue TimeoutError, StandardError, NameError, SyntaxError => err
-            error report_error("plugin #{@@plugins[key].name} help() failed:", err)
+        [core_commands, plugin_commands].each { |pl|
+          if(pl.has_key?(key))
+            begin
+              return pl[key].help(key, params)
+            rescue Exception => err
+              #rescue TimeoutError, StandardError, NameError, SyntaxError => err
+              error report_error("#{p.botmodule_class} #{plugins[key].name} help() failed:", err)
+            end
+          else
+            return false
           end
-        else
-          return false
-        end
+        }
       end
     end
 
     # see if each plugin handles +method+, and if so, call it, passing
     # +message+ as a parameter
     def delegate(method, *args)
+      debug "Delegating #{method.inspect}"
       [core_modules, plugins].each { |pl|
-        pl.values.uniq.each {|p|
+        pl.each {|p|
           if(p.respond_to? method)
             begin
+              debug "#{p.botmodule_class} #{p.name} responds"
               p.send method, *args
             rescue Exception => err
-              #rescue TimeoutError, StandardError, NameError, SyntaxError => err
-              error report_error("plugin #{p.name} #{method}() failed:", err)
+              error report_error("#{p.botmodule_class} #{p.name} #{method}() failed:", err)
+              raise if err.class <= BDB::Fatal
             end
           end
         }
       }
+      debug "Finished delegating #{method.inspect}"
     end
 
     # see if we have a plugin that wants to handle this message, if so, pass
     # it to the plugin and return true, otherwise false
     def privmsg(m)
-      [core_modules, plugins].each { |pl|
-        return unless(m.plugin)
-        if (pl.has_key?(m.plugin) &&
-          pl[m.plugin].respond_to?("privmsg") &&
-          @bot.auth.allow?(m.plugin, m.source, m.replyto))
-          begin
-            pl[m.plugin].privmsg(m)
-          rescue BDB::Fatal => err
-            error error_report("plugin #{pl[m.plugin].name} privmsg() failed:", err)
-            raise
-          rescue Exception => err
-            #rescue TimeoutError, StandardError, NameError, SyntaxError => err
-            error "plugin #{pl[m.plugin].name} privmsg() failed: #{err.class}: #{err}\n#{error err.backtrace.join("\n")}"
+      debug "Delegating privmsg with key #{m.plugin}"
+      return unless m.plugin
+      begin
+        [core_commands, plugin_commands].each { |pl|
+          # We do it this way to skip creating spurious keys
+          # FIXME use fetch?
+          k = m.plugin.to_sym
+          if pl.has_key?(k)
+            p = pl[k]
+          else
+            p = nil
           end
-          return true
-        end
+          if p
+            # TODO This should probably be checked elsewhere
+            debug "Checking auth ..."
+            if @bot.auth.allow?(m.plugin, m.source, m.replyto)
+              debug "Checking response ..."
+              if p.respond_to?("privmsg")
+                begin
+                  debug "#{p.botmodule_class} #{p.name} responds"
+                  p.privmsg(m)
+                rescue Exception => err
+                  error report_error("#{p.botmodule_class} #{p.name} privmsg() failed:", err)
+                  raise if err.class <= BDB::Fatal
+                end
+                debug "Successfully delegated privmsg with key #{m.plugin}"
+                return true
+              else
+                debug "#{p.botmodule_class} #{p.name} is registered, but it doesn't respond to privmsgs"
+              end
+            else
+              debug "#{p.botmodule_class} #{p.name} is registered, but #{m.source} isn't allowed to use #{m.plugin} on #{m.replyto}"
+            end
+          else
+            debug "No #{pl.values.first.botmodule_class} registered #{m.plugin}" unless pl.empty?
+          end
+          debug "Finished delegating privmsg with key #{m.plugin}" + ( pl.empty? ? "" : " to #{pl.values.first.botmodule_class}s" )
+        }
         return false
-      }
+      rescue Exception => e
+        error report_error("couldn't delegate #{m}", e)
+      end
+      debug "Finished delegating privmsg with key #{m.plugin}"
     end
   end