aia 1.0.0.pre.beta → 1.1.0

data/lib/aia/directive.rb

@@ -84,5 +84,34 @@ module AIA
         ""
       end
     end
+
+    private
+
+    # Split args into positive and negative search terms.
+    # Tokens prefixed with -, ~, or ! are negative (exclusion) terms.
+    # All other tokens (bare or +-prefixed) are positive (inclusion) terms.
+    # All terms are downcased.
+    #
+    # @param args [Array<String>] raw argument tokens
+    # @return [Array<Array<String>>] [positive_terms, negative_terms]
+    def parse_search_terms(args)
+      positive = []
+      negative = []
+
+      Array(args).each do |arg|
+        arg.split.each do |token|
+          downcased = token.downcase
+          if downcased =~ /\A[-~!]/
+            negative << downcased[1..]
+          elsif downcased.start_with?('+')
+            positive << downcased[1..]
+          else
+            positive << downcased
+          end
+        end
+      end
+
+      [positive, negative]
+    end
   end
 end

data/lib/aia/directives/configuration_directives.rb

@@ -32,7 +32,8 @@ module AIA
         if AIA.config.respond_to?(setter)
           AIA.config.send(setter, new_value)
         else
-          puts "Warning: Unknown config option '#{config_item}'"
+          warn "Warning: Unknown config option '#{config_item}'"
+          AIA::LoggerManager.aia_logger.warn("Unknown config option '#{config_item}'")
         end
         ""
       end

data/lib/aia/directives/execution_directives.rb

@@ -10,7 +10,7 @@ module AIA
 
       begin
         String(eval(ruby_code))
-      rescue Exception => e
+      rescue StandardError => e
         <<~ERROR
           This ruby code failed: #{ruby_code}
           #{e.message}

data/lib/aia/directives/model_directives.rb

@@ -4,6 +4,8 @@ module AIA
   class ModelDirectives < Directive
     desc "List all available AI models"
     def available_models(args = nil, context_manager = nil)
+      positive_terms, negative_terms = parse_search_terms(Array(args))
+
       current_models = AIA.config.models
 
       model_names = current_models.map do |m|
@@ -13,9 +15,9 @@ module AIA
       using_local_provider = model_names.any? { |m| m.start_with?('ollama/', 'lms/') }
 
       if using_local_provider
-        show_local_models(model_names, args)
+        show_local_models(model_names, positive_terms, negative_terms)
       else
-        show_rubyllm_models(args)
+        show_rubyllm_models(positive_terms, negative_terms)
       end
 
       ""
@@ -80,7 +82,7 @@ module AIA
 
     # --- helpers (no desc → not registered) ---
 
-    def show_local_models(current_models, args)
+    def show_local_models(current_models, positive_terms, negative_terms)
       require 'net/http'
       require 'json'
 
@@ -91,15 +93,15 @@ module AIA
         if model_spec.start_with?('ollama/')
           api_base = ENV.fetch('OLLAMA_API_BASE', 'http://localhost:11434')
           api_base = api_base.gsub(%r{/v1/?$}, '')
-          show_ollama_models(api_base, args)
+          show_ollama_models(api_base, positive_terms, negative_terms)
         elsif model_spec.start_with?('lms/')
           api_base = ENV.fetch('LMS_API_BASE', 'http://localhost:1234')
-          show_lms_models(api_base, args)
+          show_lms_models(api_base, positive_terms, negative_terms)
         end
       end
     end
 
-    def show_ollama_models(api_base, args)
+    def show_ollama_models(api_base, positive_terms, negative_terms)
       begin
         uri = URI("#{api_base}/api/tags")
         response = Net::HTTP.get_response(uri)
@@ -127,8 +129,12 @@ module AIA
           modified = model['modified_at'] ? Time.parse(model['modified_at']).strftime('%Y-%m-%d') : 'unknown'
 
           entry = "- ollama/#{name} (size: #{size}, modified: #{modified})"
+          entry_down = entry.downcase
+
+          show_it = positive_terms.empty? || positive_terms.any? { |q| entry_down.include?(q) }
+          show_it &&= negative_terms.none? { |q| entry_down.include?(q) }
 
-          if args.nil? || args.empty? || args.any? { |q| entry.downcase.include?(q.downcase) }
+          if show_it
             puts entry
             counter += 1
           end
@@ -142,7 +148,7 @@ module AIA
       end
     end
 
-    def show_lms_models(api_base, args)
+    def show_lms_models(api_base, positive_terms, negative_terms)
       begin
         uri = URI("#{api_base.gsub(%r{/v1/?$}, '')}/v1/models")
         response = Net::HTTP.get_response(uri)
@@ -167,8 +173,12 @@ module AIA
         models.each do |model|
           name = model['id']
           entry = "- lms/#{name}"
+          entry_down = entry.downcase
+
+          show_it = positive_terms.empty? || positive_terms.any? { |q| entry_down.include?(q) }
+          show_it &&= negative_terms.none? { |q| entry_down.include?(q) }
 
-          if args.nil? || args.empty? || args.any? { |q| entry.downcase.include?(q.downcase) }
+          if show_it
             puts entry
             counter += 1
           end
@@ -192,22 +202,17 @@ module AIA
       "%.1f %s" % [bytes.to_f / (1024 ** exp), units[exp]]
     end
 
-    def show_rubyllm_models(args)
-      query = args
-
-      if query && 1 == query.size
-        query = query.first.split(',')
-      end
+    def show_rubyllm_models(positive_terms, negative_terms)
+      modality_terms = positive_terms.select { |q| q.include?('_to_') }
+      text_terms     = positive_terms.reject { |q| q.include?('_to_') }
 
       header = "\nAvailable LLMs"
-      header += " for #{query.join(' and ')}" if query
+      header += " for #{positive_terms.join(' and ')}" unless positive_terms.empty?
+      header += " excluding: #{negative_terms.join(', ')}" unless negative_terms.empty?
 
       puts header + ':'
       puts
 
-      q1 = query ? query.select { |q| q.include?('_to_') } : []
-      q2 = query ? query.reject { |q| q.include?('_to_') } : []
-
       counter = 0
 
       RubyLLM.models.all.each do |llm|
@@ -218,16 +223,12 @@ module AIA
         mode = "#{inputs} to #{outputs}"
         in_1m = llm.pricing.text_tokens.standard.to_h[:input_per_million]
         entry = "- #{llm.id} (#{llm.provider}) in: $#{in_1m} cw: #{cw} mode: #{mode} caps: #{caps}"
-
-        if query.nil? || query.empty?
-          counter += 1
-          puts entry
-          next
-        end
+        entry_down = entry.downcase
 
         show_it = true
-        q1.each { |q| show_it &&= llm.modalities.send("#{q}?") }
-        q2.each { |q| show_it &&= entry.include?(q) }
+        modality_terms.each { |q| show_it &&= llm.modalities.send("#{q}?") }
+        text_terms.each     { |q| show_it &&= entry_down.include?(q) }
+        negative_terms.each { |q| show_it &&= !entry_down.include?(q) }
 
         if show_it
           counter += 1

data/lib/aia/directives/web_and_file_directives.rb

@@ -2,11 +2,14 @@
 
 require 'faraday'
 require 'clipboard'
+require 'yaml'
+require 'io/console'
+require 'word_wrapper'
 
 module AIA
   class WebAndFileDirectives < Directive
+    include AIA::SkillUtils
     PUREMD_API_KEY = ENV.fetch('PUREMD_API_KEY', nil)
-    SKILLS_DIR     = File.expand_path('~/.claude/skills')
 
     desc "Fetch and include content from a webpage"
     def webpage(args, _context_manager = nil)
@@ -30,46 +33,94 @@ module AIA
     alias_method :website, :webpage
     alias_method :web,     :webpage
 
-    desc "List available Claude Code skills"
-    def skills(_args = [], _context_manager = nil)
-      unless Dir.exist?(SKILLS_DIR)
-        puts "No skills directory found at #{SKILLS_DIR}"
+    desc "List available AIA skills"
+    def skills(args = [], _context_manager = nil)
+      dir = aia_skills_dir
+
+      unless Dir.exist?(dir)
+        puts "No skills directory found at #{dir}"
         return nil
       end
 
-      entries = Dir.children(SKILLS_DIR)
-                   .select { |e| Dir.exist?(File.join(SKILLS_DIR, e)) }
-                   .sort
+      positive_terms, negative_terms = parse_search_terms(Array(args))
 
-      if entries.empty?
-        puts "No skills found in #{SKILLS_DIR}"
-      else
-        puts "\nAvailable Skills"
-        puts "================"
-        entries.each { |name| puts "  #{name}" }
-        puts "\nTotal: #{entries.size} skills"
+      skill_dirs = Dir.children(dir)
+                      .select { |e| Dir.exist?(File.join(dir, e)) }
+                      .sort
+
+      if skill_dirs.empty?
+        puts "No skills found in #{dir}"
+        return nil
+      end
+
+      skill_data = skill_dirs.filter_map do |name|
+        fm = parse_front_matter(File.join(dir, name, 'SKILL.md'))
+        text = "#{name} #{fm.values.join(' ').downcase}"
+        pos_ok = positive_terms.empty? || positive_terms.all? { |t| text.include?(t) }
+        neg_ok = negative_terms.none? { |t| text.include?(t) }
+        [name, fm] if pos_ok && neg_ok
+      end
+
+      if skill_data.empty?
+        puts "No skills matching your query"
+        return nil
       end
 
+      width = terminal_width
+      puts "\nAvailable Skills (#{dir}):"
+      puts "=" * [width, 60].min
+
+      skill_data.each do |name, fm|
+        display_name = fm['name'] || name
+        desc_text = fm['description'].to_s.strip
+        puts "\n  #{display_name}"
+        unless desc_text.empty?
+          puts WordWrapper::MinimumRaggedness.new(width - 4, desc_text).wrap
+            .split("\n").map { |l| "    #{l}" }.join("\n")
+        end
+      end
+
+      puts "\nTotal: #{skill_data.size} skill#{'s' if skill_data.size != 1}"
       nil
     end
 
-    desc "Include a Claude Code skill from ~/.claude/skills/"
+    desc "Include an AIA skill from the skills directory"
     def skill(args = [], _context_manager = nil)
+      args = Array(args)
       skill_name = args.first&.strip
+
       if skill_name.nil? || skill_name.empty?
-        STDERR.puts "Error: /skill requires a skill name"
+        warn "Error: /skill requires a skill name. Use /skills to list available skills."
+        AIA::LoggerManager.aia_logger.error("/skill requires a skill name")
         return nil
       end
 
-      skill_dir = resolve_skill_dir(skill_name)
+      dir = aia_skills_dir
+
+      unless Dir.exist?(dir)
+        warn "Error: Skills directory not found at #{dir}"
+        AIA::LoggerManager.aia_logger.error("Skills directory not found at #{dir}")
+        return nil
+      end
+
+      skill_dir = find_skill_dir(skill_name, dir)
       unless skill_dir
-        STDERR.puts "Error: No skill matching '#{skill_name}' found in #{SKILLS_DIR}"
+        if path_based_id?(skill_name)
+          warn "Error: No skill directory found at '#{File.expand_path(skill_name)}'. Use /skills to list available skills."
+          AIA::LoggerManager.aia_logger.error("No skill directory found at '#{File.expand_path(skill_name)}'")
+        else
+          warn "Error: No skill matching '#{skill_name}' found in #{dir}. Use /skills to list available skills."
+          AIA::LoggerManager.aia_logger.error("No skill matching '#{skill_name}' found in #{dir}")
+        end
         return nil
       end
 
+      return File.read(skill_dir) if File.file?(skill_dir)
+
       skill_path = File.join(skill_dir, 'SKILL.md')
       unless File.exist?(skill_path)
-        STDERR.puts "Error: Skill directory '#{File.basename(skill_dir)}' has no SKILL.md"
+        warn "Error: Skill directory '#{File.basename(skill_dir)}' has no SKILL.md. Use /skills to list available skills."
+        AIA::LoggerManager.aia_logger.error("Skill directory '#{File.basename(skill_dir)}' has no SKILL.md")
         return nil
       end
 
@@ -87,28 +138,15 @@ module AIA
 
     private
 
-    def resolve_skill_dir(skill_name)
-      return nil unless Dir.exist?(SKILLS_DIR)
-
-      exact = File.join(SKILLS_DIR, skill_name)
-      return safe_skill_path(exact)  if Dir.exist?(exact)
-
-      Dir.children(SKILLS_DIR)
-         .sort
-         .each do |entry|
-        next unless entry.start_with?(skill_name)
-        candidate = File.join(SKILLS_DIR, entry)
-        return safe_skill_path(candidate) if Dir.exist?(candidate)
-      end
-
-      nil
+    def aia_skills_dir
+      AIA.config.skills.dir
     end
 
-    def safe_skill_path(path)
-      resolved = File.realpath(path)
-      resolved.start_with?(File.realpath(SKILLS_DIR)) ? resolved : nil
-    rescue Errno::ENOENT
-      nil
+    def terminal_width
+      IO.console&.winsize&.last || 80
+    rescue StandardError
+      80
     end
+
   end
 end

data/lib/aia/errors.rb

@@ -1,5 +1,24 @@
 # lib/aia/errors.rb
 
 module AIA
-  class ConfigurationError < StandardError; end
+  # Base error class for all AIA errors
+  class Error < StandardError; end
+
+  # Raised for configuration validation failures
+  class ConfigurationError < Error; end
+
+  # Raised for prompt loading, parsing, or processing failures
+  class PromptError < Error; end
+
+  # Raised for RubyLLM::Tool loading or execution failures
+  class ToolError < Error; end
+
+  # Raised for MCP server connection or communication failures
+  class MCPError < Error; end
+
+  # Raised for LLM adapter failures (model setup, chat execution)
+  class AdapterError < Error; end
+
+  # Raised for directive parsing or execution failures
+  class DirectiveError < Error; end
 end

data/lib/aia/fzf.rb

@@ -2,6 +2,7 @@
 # fzf is a general-purpose command-line fuzzy finder
 
 require 'shellwords'
+require 'open3'
 require 'tempfile'
 
 class AIA::Fzf
@@ -37,19 +38,19 @@ class AIA::Fzf
 
   def build_command
     fzf_options = DEFAULT_PARAMETERS.dup
-    fzf_options << "--header='#{subject} which contain: #{query}\nPress ESC to cancel.'"
-    fzf_options << "--preview='cat #{directory}/{1}#{extension}'"
+    escaped_dir = Shellwords.escape(directory)
+    escaped_ext = Shellwords.escape(extension)
+    fzf_options << "--header=#{Shellwords.escape("#{subject} which contain: #{query}\nPress ESC to cancel.")}"
+    fzf_options << "--preview=#{Shellwords.escape("cat #{escaped_dir}/{1}#{escaped_ext}")}"
     fzf_options << "--prompt=#{Shellwords.escape(prompt)}"
 
-    fzf_command = "fzf #{fzf_options.join(' ')}"
-
-    @command = "cat #{tempfile_path} | #{fzf_command}"
+    @fzf_args = fzf_options
   end
 
 
   def run
-    # puts "Executing: #{@command}"
-    selected = `#{@command}`.strip
+    input = list.join("\n")
+    selected, status = Open3.capture2('fzf', *@fzf_args, stdin_data: input)
 
     selected.strip.empty? ? nil : selected.strip
   ensure

data/lib/aia/input_collector.rb

@@ -0,0 +1,24 @@
+# lib/aia/input_collector.rb
+# frozen_string_literal: true
+
+module AIA
+  class InputCollector
+    # Collect variable values from user input via HistoryManager
+    def collect(parameters)
+      return {} if parameters.nil? || parameters.empty?
+
+      values = {}
+      input_manager = AIA::HistoryManager.new
+
+      parameters.each do |name, default|
+        value = input_manager.request_variable_value(
+          variable_name: name,
+          default_value: default,
+        )
+        values[name] = value
+      end
+
+      values
+    end
+  end
+end

data/lib/aia/prompt_handler.rb

@@ -6,6 +6,14 @@ require 'erb'
 
 module AIA
   class PromptHandler
+    include AIA::SkillUtils
+
+    # Struct for path-based role content (bypasses PM parsing)
+    RoleContent = Struct.new(:content) do
+      def to_s; content; end
+      def metadata; nil; end
+    end
+
     # Root-level YAML keys that are shorthands for deeper config paths
     SHORTHAND_KEYS = %w[model temperature top_p next pipeline shell erb].freeze
 
@@ -46,7 +54,8 @@ module AIA
       parsed = if File.exist?(prompt_file_path)
                  PM.parse(prompt_id)
                else
-                 puts "Warning: Invalid prompt ID or file not found: #{prompt_id}"
+                 warn "Warning: Invalid prompt ID or file not found: #{prompt_id}"
+                 logger.warn("Invalid prompt ID or file not found: #{prompt_id}")
                  handle_missing_prompt(prompt_id)
                end
 
@@ -57,6 +66,7 @@ module AIA
 
     def fetch_role(role_id)
       return handle_missing_role("roles/") if role_id.nil?
+      return fetch_role_from_path(role_id) if path_based_id?(role_id)
 
       unless role_id.start_with?(AIA.config.prompts.roles_prefix)
         role_id = "#{AIA.config.prompts.roles_prefix}/#{role_id}"
@@ -65,9 +75,12 @@ module AIA
       role_file_path = File.join(@prompts_dir, "#{role_id}#{AIA.config.prompts.extname}")
 
       parsed = if File.exist?(role_file_path)
-                 PM.parse(role_id)
+                 # PM.parse(role_id) cannot resolve subdirectory IDs like "roles/jersey_mike"
+                 # and returns the ID string as content.  Parse from raw file content instead.
+                 PM.parse_string(File.read(role_file_path))
                else
-                 puts "Warning: Invalid role ID or file not found: #{role_id}"
+                 warn "Warning: Invalid role ID or file not found: #{role_id}"
+                 logger.warn("Invalid role ID or file not found: #{role_id}")
                  handle_missing_role(role_id)
                end
 
@@ -90,7 +103,8 @@ module AIA
       role_parsed = fetch_role(role_id)
       role_parsed.to_s
     rescue => e
-      puts "Warning: Could not load role '#{role_id}' for model: #{e.message}"
+      warn "Warning: Could not load role '#{role_id}' for model: #{e.message}"
+      logger.warn("Could not load role '#{role_id}' for model: #{e.message}")
       nil
     end
 
@@ -280,19 +294,33 @@ module AIA
     def handle_missing_prompt(prompt_id)
       prompt_id = prompt_id.to_s.strip
       if prompt_id.empty?
-        STDERR.puts "Error: Prompt ID cannot be empty"
+        warn "Error: Prompt ID cannot be empty"
         exit 1
       end
 
       if AIA.config.flags.fuzzy
         fuzzy_search_prompt(prompt_id)
       else
-        STDERR.puts "Error: Could not find prompt with ID: #{prompt_id}"
+        warn "Error: Could not find prompt with ID: #{prompt_id}"
         exit 1
       end
     end
 
 
+    def fetch_role_from_path(role_id)
+      expanded = File.expand_path(role_id)
+      expanded += '.md' if File.extname(expanded).empty?
+
+      unless File.exist?(expanded)
+        $stderr.puts "Warning: Role file not found at path: #{expanded}"
+        logger.warn("Role file not found at path: #{expanded}")
+        return handle_missing_role(role_id)
+      end
+
+      RoleContent.new(File.read(expanded))
+    end
+
+
     def fuzzy_search_prompt(prompt_id)
       new_prompt_id = search_prompt_id_with_fzf(prompt_id)
 
@@ -307,14 +335,14 @@ module AIA
     def handle_missing_role(role_id)
       role_id = role_id.to_s.strip
       if role_id.empty? || role_id == "roles/"
-        STDERR.puts "Error: Role ID cannot be empty"
+        warn "Error: Role ID cannot be empty"
         exit 1
       end
 
       if AIA.config.flags.fuzzy
         fuzzy_search_role(role_id)
       else
-        STDERR.puts "Error: Could not find role with ID: #{role_id}"
+        warn "Error: Could not find role with ID: #{role_id}"
         exit 1
       end
     end