aia 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f3c92628b523cd32fa77fa52f1546f1ec43ee9b65c645b33adb508086ef0c13
-  data.tar.gz: cc9e9aab97552cff87e4b05b001d8330a99a5fea9000c6d2b76f07b1e1a7054e
+  metadata.gz: 9fa126ad31719e30c2d357d3573ba4b7832fe99ab082799c0e7b37c3b0d49314
+  data.tar.gz: 20706361c420669cb035fb03c996ef28895399ba850985715e44f6b9411ca47a
 SHA512:
-  metadata.gz: abf1b9d9f077b6d8aadd6c259e1cf3125007718b7912be857003f4189d57be4c20451093e87c6ec47f0a3be06c4ddf7cf767ffff5dc55be4177241e2d6fa8d7d
-  data.tar.gz: 14e85d72e471d1bf8572d8fb2b417a6e1e0c641bc34e376b8748dd8b2f55665221fa6ff78966692f0ecce65ee990067c65a898db0d96f718ad713ec8f02048ec
+  metadata.gz: 6154106f03fa35cfbf562282eec6512dc210df4b40ae6c900eede45df5a5253f232fc6bbe7c9a0cf930a152558ffde96d45aa1894d8c338af5256f65a9ccbe82
+  data.tar.gz: 6ce2ca929686cd6e262f5f009581a087b710129f5d89a0d020d612dffd7bc5b555f794a41cfc38ca3f8a3b1941edfeff7cb08331f2c7ea036dfce2e4372c3ae0

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [2.0.1] 2026-11-02
+
+### Fixed
+
+- **`--chat` startup output is readable again.** ruby_llm 2.1 starts stdio MCP servers with their stderr connected to aia's, so server banners, npm notices, stack traces and validation dumps were printed over the "Connecting MCP servers" spinner and garbled it. That stderr now goes to `~/.config/aia/logs/mcp_servers.log`. Set `logger.mcp.server_stderr` to another path, `STDERR` (the 2.0.0 behavior) or `null` to discard it.
+
+### Changed
+
+- `logger.mcp.file` defaults to `~/.config/aia/logs/mcp.log` (was `STDOUT`), so MCP connection warnings no longer print over the spinner. Failed servers are still listed in the startup banner. A config file that sets `file: STDOUT` keeps the old behavior.
+- Log directories are created when they don't exist.
+
 ## [2.0.0] 2026-11-02
 
 AIA 2.0 rebuilds the CLI on [robot_lab](https://github.com/MadBomber/robot_lab): every session is a crew of robots, MCP servers come from ruby_llm 2.1's built-in client, and any provider RubyLLM registers can be used. This entry summarizes everything since 1.1.1, the last 1.x release, including the `2.0.0.pre.alpha` and `2.0.0.pre.beta2` pre-releases.

data/lib/aia/config/defaults.yml

@@ -162,10 +162,15 @@ defaults:
       file: STDOUT
       level: warn
       flush: true
+    # MCP connection logging goes to a file so warnings never interleave
+    # with the "Connecting MCP servers" spinner; failures still appear in
+    # the startup banner's FAILED list.
     mcp:
-      file: STDOUT
+      file: ~/.config/aia/logs/mcp.log
       level: warn
       flush: true
+      # stderr of stdio MCP server processes: a file path, STDERR, or null
+      server_stderr: ~/.config/aia/logs/mcp_servers.log
 
   # ---------------------------------------------------------------------------
   # Pipeline/Workflow Configuration

data/lib/aia/logger.rb

@@ -18,8 +18,9 @@
 #       file: STDOUT
 #       level: warn
 #     mcp:
-#       file: STDOUT
+#       file: ~/.config/aia/logs/mcp.log
 #       level: warn
+#       server_stderr: ~/.config/aia/logs/mcp_servers.log  # stdio server stderr
 #
 # Lumberjack provides structured logging, context isolation,
 # automatic log file rolling, and multi-process safe file writes.
@@ -30,6 +31,7 @@
 #   AIA::LoggerManager.clear_test_logs!  # between tests
 #   entries = AIA::LoggerManager.aia_logger.device.entries  # inspect logs
 
+require 'fileutils'
 require 'lumberjack'
 
 module AIA
@@ -287,6 +289,7 @@ module AIA
           Lumberjack::Device::Writer.new($stderr, autoflush: flush)
         else
           path = File.expand_path(file)
+          FileUtils.mkdir_p(File.dirname(path))
           # Daily date rolling via Logger::LogDevice, which also makes the
           # file safe for multiple loggers to write to
           Lumberjack::Device::LogFile.new(

data/lib/aia/patches/mcp_stdio_stderr.rb

@@ -0,0 +1,102 @@
+# lib/aia/patches/mcp_stdio_stderr.rb
+#
+# Redirect the stderr of stdio MCP server processes away from the terminal.
+#
+# ruby_llm 2.1's RubyLLM::MCP::Stdio#start launches each server with
+# Open3.popen2, so the child inherits aia's stderr ("Its stderr is the
+# parent's"). Every server banner, npm notice, stack trace and Python
+# validation dump then lands on the terminal, interleaved with the
+# TTY::Spinner::Multi "Connecting MCP servers" display, which corrupts it.
+# (ruby_llm-mcp 1.x used popen3 and drained stderr, so aia 1.x was quiet.)
+#
+# The destination comes from AIA.config.logger.mcp.server_stderr:
+#   a file path  - appended to (default: ~/.config/aia/logs/mcp_servers.log)
+#   STDERR       - the old passthrough behavior
+#   null         - discard (also: none, /dev/null)
+#
+# Stdio#start has no hook for spawn options, so this re-implements its
+# three-line body. It is guarded against drift: if a future ruby_llm drops
+# Stdio#start, the patch is skipped with a warning instead of breaking MCP.
+
+require 'fileutils'
+require 'open3'
+
+module AIA
+  module MCPStdioStderr
+    DEFAULT_PATH = '~/.config/aia/logs/mcp_servers.log'.freeze
+    DISCARD      = %w[NULL NONE /DEV/NULL].freeze
+
+    class << self
+      # The spawn option value for a server's stderr.
+      #
+      # @param setting [String, nil] logger.mcp.server_stderr
+      # @return [Array(String, String), nil] [path, mode] for Process.spawn's
+      #   err: option, or nil to inherit the parent's stderr
+      def spawn_target(setting = configured_setting)
+        value = setting.to_s.strip
+        value = DEFAULT_PATH if value.empty?
+
+        case value.upcase
+        when 'STDERR' then nil
+        when *DISCARD then [File::NULL, 'w']
+        else log_file_target(value)
+        end
+      end
+
+      # Spawn options for Open3 with the stderr redirect added.
+      #
+      # @param options [Hash] options ruby_llm would pass (e.g. chdir:)
+      # @param target [Array, nil] from .spawn_target
+      # @return [Hash]
+      def spawn_options(options, target = spawn_target)
+        target ? options.merge(err: target) : options
+      end
+
+      # Install the patch onto RubyLLM::MCP::Stdio (idempotent).
+      #
+      # @return [Class, nil] the patched class, or nil when it can't be patched
+      def install!(stdio = stdio_class)
+        return unless stdio&.private_method_defined?(:start)
+
+        stdio.prepend(StdioPatch) unless stdio.ancestors.include?(StdioPatch)
+        stdio
+      end
+
+      private
+
+      def configured_setting
+        AIA.config&.logger&.mcp&.server_stderr
+      rescue StandardError
+        nil # config not loaded yet (or malformed): fall back to the default file
+      end
+
+      def log_file_target(value)
+        path = File.expand_path(value)
+        FileUtils.mkdir_p(File.dirname(path))
+        [path, 'a']
+      rescue SystemCallError
+        [File::NULL, 'w'] # unwritable location: still keep the terminal clean
+      end
+
+      def stdio_class
+        RubyLLM::MCP::Stdio if defined?(RubyLLM::MCP::Stdio)
+      end
+    end
+
+    # Prepended onto RubyLLM::MCP::Stdio.
+    module StdioPatch
+      private
+
+      def start
+        options = @directory ? { chdir: @directory } : {}
+        options = MCPStdioStderr.spawn_options(options)
+        @stdin, @stdout, @process = Open3.popen2(@env, *@command, **options)
+        @buffer = +''
+      end
+    end
+  end
+end
+
+unless AIA::MCPStdioStderr.install!
+  warn 'AIA: RubyLLM::MCP::Stdio#start not found; MCP server stderr will reach the terminal'
+end

data/lib/aia/version.rb

@@ -1,13 +1,5 @@
 # lib/aia/version.rb
-#
-# This file defines the version of the AIA application.
-# The VERSION constant defines the current version of the AIA application,
-# which is read from the .version file in the project root.
 
-# The AIA module serves as the namespace for the AIA application, which
-# provides an interface for interacting with AI models and managing prompts.
 module AIA
-  # The VERSION constant defines the current version of the AIA application,
-  # which is read from the .version file in the project root.
-  VERSION = File.read(File.join(File.dirname(__FILE__), '..', '..', '.version')).strip
+  VERSION = '2.0.1'.freeze
 end

data/lib/aia.rb

@@ -53,6 +53,7 @@ require_relative 'aia/robot_sync'
 require_relative 'aia/robot_builder'
 require_relative 'aia/directive_processor'
 require_relative 'aia/variable_input_collector'
+require_relative 'aia/patches/mcp_stdio_stderr'
 require_relative 'aia/ui_presenter'
 require_relative 'aia/input_collector'
 require_relative 'aia/handler_context'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aia
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Dewayne VanHoozer
@@ -418,7 +418,6 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".version"
 - CHANGELOG.md
 - LICENSE
 - README.md
@@ -470,6 +469,7 @@ files:
 - lib/aia/model_registry.rb
 - lib/aia/network_builder.rb
 - lib/aia/network_memory_manager.rb
+- lib/aia/patches/mcp_stdio_stderr.rb
 - lib/aia/patches/ruby_llm_streaming_error.rb
 - lib/aia/patches/ruby_llm_tool_error.rb
 - lib/aia/patches/ruby_llm_tool_param_compat.rb

data/.version

@@ -1 +0,0 @@
-2.0.0