aia 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d72af72c970719f5d9508d14fc7903e46b7b751028d75cb045c8c23418151b44
-  data.tar.gz: 7a2b43f842f38d7e2f94f2118c85f087a4f97b944958b7a63c5d2798bab05cb9
+  metadata.gz: c8f6a82b1c6cc29b4245e51ed43a325a835a921a0da5c2818dbdfbdeeeb97c40
+  data.tar.gz: ff0cff98ed30d391970a11a44519cb9ea95082bfae2a08e5ebf03b71eeefac15
 SHA512:
-  metadata.gz: a27344680f0eb4ac9f1a9535982b53b102bb2face18b66f293918996a9b27e6d29aea2c46480671c2abecbb9a7e4f8c1400c84d5f157f7083c357a8766b399da
-  data.tar.gz: 3b740188dc4b8a74873fca77112fde35737574bd47c2a2c7f3e888cc831b669fb520b3e5e212b6124960ca0afe09ebe5fa004bf8e18f8507e32f5ea58afb5ae5
+  metadata.gz: ec35e9ae94d6c74ad6ecf87ad6a8511de29407d1bac99db1ff1266b7072ac078746bf9fcabb45f6aeed1858198b318ff2906e8858b74185de8107cfdcf48bfca
+  data.tar.gz: 7ee21fda6bdf851e2418d739450f2b8d6ffd8580e5aaaef7b84e5c16c83ded0eefb83e6c8a943ffa5bbe09960463eb8617c44adf9a66f6ad87b460edc50ed6b7

data/.version

@@ -1 +1 @@
-1.1.0
+1.1.1

data/CHANGELOG.md

@@ -1,4 +1,19 @@
 # Changelog
+## [1.1.1] - 2026-05-01
+
+### Bug Fixes
+- **Conversation context lost on Ollama model pipeline steps (Issue #152)**: `maybe_change_model` was unconditionally replacing the adapter whenever the configured model name (e.g. `ollama/qwen3`) did not literally appear in the resolved model ID (`qwen3`). Fixed by stripping the provider prefix before comparison and by never replacing the adapter when conversation messages exist in any chat instance — preserving all history across pipeline steps and role changes
+- **Role file front matter overwriting active model config**: `ChatLoop#process_role_context` was allowing `fetch_role` (which calls `apply_metadata_config`) to overwrite `AIA.config.models` with any `model:` key in the role file's YAML front matter. The active model is now saved before `fetch_role` and restored afterward, so a role's front matter cannot trigger a false mismatch in `maybe_change_model`
+- **Custom `--tools` directives not available in ERB prompts**: `AIA::Directive` subclasses defined in files loaded via `--tools` were required too late — after `PromptHandler.new` had already called `register_pm_directives`. Added `require_tool_files` as the first step in `ConfigValidator#tailor` so custom directive classes are defined and registered into `PM.directives` before `PromptHandler.new` runs; `<%= timestamp %>` and other custom ERB helpers now expand correctly
+- **Demo example path resolution**: `examples/prompts_dir/project_summary` referenced `../../*.gemspec` and `../../lib/aia/*.rb`, resolving two levels above the repo root from the `examples/` working directory; corrected to `../`
+
+### Improvements
+- **Demo scripts use develop-branch binary**: `examples/common.sh` now prepends `../bin` to `PATH` when the local `bin/aia` is executable, ensuring all demo scripts exercise working-tree changes rather than the installed gem
+
+### Testing
+- Added `test/aia/chat_loop_test.rb` — 6 tests covering `process_role_context`: early-return when no role configured, system message injection into chats, skipping chats that already have a system message, and model restoration after `fetch_role` side effects (Issue #152 regression tests)
+- Expanded `test/aia/chat_processor_service_test.rb` with 7 additional tests: provider-prefix stripping before model comparison, full-ID alias resolution (e.g. `claude-sonnet-4` vs `claude-sonnet-4-20250514`), adapter preservation when any chat has history, adapter replacement allowed when no history exists, and the exact Issue #152 regression scenario (OpenAI model active, role front matter specifies a Claude model)
+
 ## [1.1.0] - 2026-04-23
 
 ### New Features

data/examples/common.sh

@@ -18,3 +18,10 @@ while IFS= read -r var; do
 done < <(env | grep '^AIA_' | cut -d= -f1)
 
 CONFIG="aia_config.yml"
+
+# When running from the develop branch, prefer the local bin/aia over any
+# installed gem so that uncommitted working-tree changes are exercised.
+LOCAL_AIA="${SCRIPT_DIR}/../bin/aia"
+if [[ -x "${LOCAL_AIA}" ]]; then
+  export PATH="${SCRIPT_DIR}/../bin:${PATH}"
+fi

data/examples/prompts_dir/project_summary

@@ -4,9 +4,9 @@ temperature: 0.3
 ---
 Summarize this project based on the following information:
 
-Gemspec name: <%= `grep 'spec.name' ../../*.gemspec 2>/dev/null`.strip %>
+Gemspec name: <%= `grep 'spec.name' ../*.gemspec 2>/dev/null`.strip %>
 Ruby version: <%= RUBY_VERSION %>
 Current directory: $(pwd)
-Files in lib/: $(ls ../../lib/aia/*.rb 2>/dev/null | wc -l | tr -d ' ')
+Files in lib/: $(ls ../lib/aia/*.rb 2>/dev/null | wc -l | tr -d ' ')
 
 Provide a three-sentence overview.

data/lib/aia/chat_loop.rb

@@ -48,7 +48,14 @@ module AIA
       return if role.nil? || role.empty?
 
       prompt_handler = AIA::PromptHandler.new
+
+      # fetch_role applies the role file's metadata (including any model: key) to
+      # AIA.config. Preserve the user's chosen model so the chat session keeps
+      # using the model from the CLI, not one embedded in the role file.
+      saved_models = AIA.config.models
       role_parsed = prompt_handler.fetch_role(role)
+      AIA.config.models = saved_models
+
       return if role_parsed.nil?
 
       role_content = role_parsed.to_s

data/lib/aia/chat_processor_service.rb

@@ -99,9 +99,23 @@ module AIA
       model_name = first_model.respond_to?(:name) ? first_model.name : first_model.to_s
 
       # client_model is the full resolved ID (e.g. "claude-sonnet-4-20250514"),
-      # model_name is the configured alias (e.g. "claude-sonnet-4").
-      # The alias is always a prefix/substring of the resolved ID, so check that way.
-      unless client_model.downcase.include?(model_name.downcase)
+      # model_name is the configured alias (e.g. "claude-sonnet-4") or a
+      # provider-prefixed name (e.g. "ollama/qwen3").
+      #
+      # The adapter strips provider prefixes when creating the RubyLLM chat
+      # (e.g. "ollama/qwen3" → model: "qwen3"), so client_model has no prefix.
+      # Strip the prefix from model_name before comparing so that
+      # "qwen3".include?("qwen3")  →  true  (was: "qwen3".include?("ollama/qwen3") → false).
+      comparable_name = model_name.sub(%r{\A[^/]+/}, '')
+      unless client_model.downcase.include?(comparable_name.downcase)
+        # Never replace the adapter when conversation history exists — doing so
+        # destroys all prior context. Role/prompt files can change AIA.config.models
+        # via front matter metadata, but that must not evict an active chat session.
+        if AIA.client.respond_to?(:chats) &&
+           AIA.client.chats.values.any? { |chat| chat.messages.any? }
+          warn "Warning: Model config changed to '#{model_name}' but keeping '#{client_model}' to preserve conversation history."
+          return
+        end
         AIA.client = AIA.client.class.new
       end
     end

data/lib/aia/config/validator.rb

@@ -24,6 +24,10 @@ module AIA
         stdin_content = process_stdin_content
         config.stdin_content = stdin_content if stdin_content && !stdin_content.strip.empty?
 
+        # Require tool files early so AIA::Directive subclasses defined in them
+        # are registered into PM.directives before PromptHandler.new is called.
+        require_tool_files(config)
+
         # Process arguments and validate
         process_prompt_id_from_args(config, remaining_args)
         validate_and_set_context_files(config, remaining_args)
@@ -361,6 +365,19 @@ module AIA
         !Array(config.mcp_use).empty? || !Array(config.mcp_skip).empty?
       end
 
+      def require_tool_files(config)
+        Array(config.tools&.paths).each do |path|
+          expanded = File.expand_path(path)
+          if File.exist?(expanded)
+            require expanded
+          else
+            warn "Warning: Tool file not found: #{path}"
+          end
+        rescue LoadError, StandardError => e
+          warn "Warning: Failed to load tool '#{path}': #{e.message}"
+        end
+      end
+
       def load_local_tools(config)
         # Load required libraries (with gem activation and lazy-load triggering)
         Array(config.require_libs).each do |lib|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aia
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Dewayne VanHoozer