agent-harness 0.11.1 → 0.11.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf9a5083c42a6675255f4c0ea0516ab99473813020e3a875b02eeda98b937db8
-  data.tar.gz: e070d74c397e02260d2994783c42124f45ac41b15a7c070834dd43c9c59e712a
+  metadata.gz: 50b5bc213cf4a1b6f9441a06a82c38210c7e6907eb085e6cadf71e305e4b7897
+  data.tar.gz: 9785ea0b1b35f5aa52528741ce070287f2d433235ffeaf3d6df0ab7337f486b0
 SHA512:
-  metadata.gz: 7bce4c50dc634010ce8d7ac38c4adc661fdff51e2b055d9a585c0a7ef745f4a365a7aa9c68fedac63cfcbce7d7bea18f74a69b1622f2cde8fc5a6d38312ce2df
-  data.tar.gz: cd9adee894f938a0bb104cd4648766f2b6b6f89e1080874436dd774f8313213bdea4d63d3fed0a81b7bddeca319c8e54a39588726fb40f724bb8d858a85314fc
+  metadata.gz: 2343d812d85375faad3a55e0462eb228003ff5eddc1920fd1edb75aedf381e5b86e3062c191ee64d24c39f245c1f27ded7d0e74247526727901b6b5391c2518a
+  data.tar.gz: cfe73ff0ffce00f5e0be1726b3dc28d0cdb73aab6546988c19b79846c1d16014476302f2d258d13bb703124edbf86066a96cd43044047ebacc81ad0cf5c853e5

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.11.1"
+  ".": "0.11.2"
 }

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 ## [Unreleased]
 
+## [0.11.2](https://github.com/viamin/agent-harness/compare/agent-harness/v0.11.1...agent-harness/v0.11.2) (2026-04-27)
+
+
+### Bug Fixes
+
+* 162: Support provider-agnostic MCP configuration ([#165](https://github.com/viamin/agent-harness/issues/165)) ([27f4814](https://github.com/viamin/agent-harness/commit/27f48146e99d8fdba0346235a3e5f19138266652))
+
 ## [0.11.1](https://github.com/viamin/agent-harness/compare/agent-harness/v0.11.0...agent-harness/v0.11.1) (2026-04-26)
 
 

data/lib/agent_harness/configuration.rb

@@ -78,6 +78,22 @@ module AgentHarness
       @custom_provider_classes[name.to_sym] = klass
     end
 
+    # Set MCP servers from an array of server definitions.
+    #
+    # @param servers [Array<Hash, McpServer>] server definitions
+    def mcp_servers=(servers)
+      normalized = McpConfigTranslator.normalize_servers(servers)
+      @mcp_servers = normalized.each_with_object({}) { |s, h| h[s.name.to_sym] = s }
+    end
+
+    # Load MCP server definitions from a configuration file.
+    #
+    # @param path [String] file path
+    def load_mcp_servers_file(path)
+      loaded = McpConfigLoader.load_file(path)
+      @mcp_servers = loaded.each_with_object({}) { |s, h| h[s.name.to_sym] = s }
+    end
+
     # Configure a provider-agnostic sub-agent definition.
     #
     # @param name [Symbol, String] sub-agent name

data/lib/agent_harness/mcp_config_loader.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require "json"
+require "yaml"
+
+module AgentHarness
+  class McpConfigLoader
+    ENV_VAR_PATTERN = /\$\{([A-Z0-9_]+)\}/
+
+    class << self
+      def load_file(path)
+        parsed = parse_file(path)
+        servers = parsed.is_a?(Hash) ? (parsed["servers"] || parsed[:servers] || parsed) : parsed
+
+        unless servers.is_a?(Array)
+          raise McpConfigurationError,
+            "MCP config file must contain a top-level servers array"
+        end
+
+        servers.map do |server|
+          McpServer.from_hash(interpolate_env(server))
+        end
+      end
+
+      private
+
+      def parse_file(path)
+        ext = File.extname(path).downcase
+        content = File.read(path)
+
+        case ext
+        when ".json"
+          JSON.parse(content)
+        when ".yml", ".yaml"
+          YAML.safe_load(content, aliases: false) || {}
+        else
+          raise McpConfigurationError,
+            "Unsupported MCP config file format '#{ext}'. Use .json, .yml, or .yaml"
+        end
+      rescue Errno::ENOENT => e
+        raise McpConfigurationError, "MCP config file not found: #{e.message}"
+      rescue JSON::ParserError, Psych::SyntaxError => e
+        raise McpConfigurationError, "Failed to parse MCP config file: #{e.message}"
+      end
+
+      def interpolate_env(value)
+        case value
+        when Array
+          value.map { |entry| interpolate_env(entry) }
+        when Hash
+          value.each_with_object({}) do |(key, entry), memo|
+            memo[key] = interpolate_env(entry)
+          end
+        when String
+          value.gsub(ENV_VAR_PATTERN) { ENV.fetch(Regexp.last_match(1), "") }
+        else
+          value
+        end
+      end
+    end
+  end
+end

data/lib/agent_harness/mcp_config_translator.rb

@@ -0,0 +1,85 @@
+# frozen_string_literal: true
+
+module AgentHarness
+  module McpConfigTranslator
+    module_function
+
+    def for_provider(provider, mcp_servers)
+      servers = normalize_servers(mcp_servers)
+
+      case provider.to_sym
+      when :anthropic, :claude, :claude_code
+        translate_for_claude(servers)
+      when :codex
+        translate_for_codex(servers)
+      when :openai
+        translate_for_openai(servers)
+      else
+        servers.map(&:to_h)
+      end
+    end
+
+    def normalize_servers(mcp_servers)
+      Array(mcp_servers).map do |server|
+        case server
+        when McpServer
+          server
+        when Hash
+          McpServer.from_hash(server)
+        else
+          raise McpConfigurationError, "MCP server must be a Hash or McpServer, got #{server.class}"
+        end
+      end
+    end
+
+    def translate_for_claude(mcp_servers)
+      {
+        mcpServers: mcp_servers.each_with_object({}) do |server, memo|
+          entry = server.stdio? ? {command: server.command} : {url: server.url}
+          entry[:args] = server.args if server.stdio? && !server.args.empty?
+          entry[:env] = server.env unless server.env.empty?
+          entry[:headers] = server.headers if server.http? && !server.headers.empty?
+          memo[server.name] = entry
+        end
+      }
+    end
+
+    def translate_for_codex(mcp_servers)
+      {
+        mcp_servers: mcp_servers.each_with_object({}) do |server, memo|
+          entry = server.stdio? ? {command: server.command, transport: server.transport} : {url: server.url, transport: server.transport}
+          entry[:args] = server.args if server.stdio? && !server.args.empty?
+          entry[:env] = server.env unless server.env.empty?
+          entry[:headers] = server.headers if server.http? && !server.headers.empty?
+          memo[server.name] = entry
+        end
+      }
+    end
+
+    def translate_for_openai(mcp_servers)
+      mcp_servers.map do |server|
+        unless server.http?
+          raise McpTransportUnsupportedError.new(
+            "Provider 'openai' only supports remote MCP servers over HTTP/SSE (server: '#{server.name}')",
+            provider: :openai
+          )
+        end
+
+        unsupported_headers = server.headers.keys - ["Authorization"]
+        unless unsupported_headers.empty?
+          raise McpConfigurationError,
+            "OpenAI MCP translation only supports the Authorization header (server: '#{server.name}')"
+        end
+
+        tool = {
+          type: "mcp",
+          server_label: server.name,
+          server_url: server.url,
+          require_approval: "never"
+        }
+        tool[:authorization] = server.headers["Authorization"] if server.headers.key?("Authorization")
+        tool
+      end
+    end
+  end
+end

data/lib/agent_harness/mcp_server.rb

@@ -10,7 +10,8 @@ module AgentHarness
   #   McpServer.new(
   #     name: "filesystem",
   #     transport: "stdio",
-  #     command: ["npx", "-y", "@modelcontextprotocol/server-filesystem", "/workspace"],
+  #     command: "npx",
+  #     args: ["-y", "@modelcontextprotocol/server-filesystem", "/workspace"],
   #     env: { "DEBUG" => "0" }
   #   )
   #
@@ -23,21 +24,24 @@ module AgentHarness
   class McpServer
     VALID_TRANSPORTS = %w[stdio http sse].freeze
 
-    attr_reader :name, :transport, :command, :args, :env, :url
+    attr_reader :name, :transport, :command, :args, :env, :url, :headers
 
     # @param name [String] unique name for this MCP server
     # @param transport [String] one of "stdio", "http", "sse"
-    # @param command [Array<String>, nil] command to launch (stdio only)
+    # @param command [String, Array<String>, nil] executable to launch (stdio only).
+    #   Arrays are accepted for backwards compatibility and normalized into
+    #   +command+ plus +args+.
     # @param args [Array<String>, nil] additional args for the command
     # @param env [Hash<String,String>, nil] environment variables for the server process
     # @param url [String, nil] URL for HTTP/SSE transport
-    def initialize(name:, transport:, command: nil, args: nil, env: nil, url: nil)
+    # @param headers [Hash<String,String>, nil] HTTP headers for remote MCP servers
+    def initialize(name:, transport:, command: nil, args: nil, env: nil, url: nil, headers: nil)
       @name = name
       @transport = transport.to_s
-      @command = command
-      @args = args || []
+      @command, @args = normalize_command_and_args(command, args)
       @env = env || {}
       @url = url
+      @headers = headers || {}
 
       validate!
     end
@@ -63,7 +67,8 @@ module AgentHarness
         command: hash[:command],
         args: hash[:args],
         env: hash[:env],
-        url: hash[:url]
+        url: hash[:url],
+        headers: hash[:headers]
       )
     end
 
@@ -75,6 +80,12 @@ module AgentHarness
       %w[http sse].include?(@transport)
     end
 
+    def command_argv
+      return [] unless stdio?
+
+      [@command, *@args]
+    end
+
     # Check if the MCP server is reachable based on its transport type.
     #
     # For stdio servers, checks that a command is present.
@@ -101,6 +112,7 @@ module AgentHarness
         h[:args] = @args unless @args.empty?
       else
         h[:url] = @url
+        h[:headers] = @headers unless @headers.empty?
       end
       h[:env] = @env unless @env.empty?
       h
@@ -118,6 +130,7 @@ module AgentHarness
 
       validate_args!
       validate_env!
+      validate_headers!
       validate_stdio! if stdio?
       validate_http! if http?
       validate_no_stdio_only_fields_on_http! if http?
@@ -137,15 +150,17 @@ module AgentHarness
         "MCP server '#{@name}' env must be a Hash with String keys and values"
     end
 
-    def validate_stdio!
-      if @command.nil? || !@command.is_a?(Array) || @command.empty?
-        raise McpConfigurationError,
-          "MCP server '#{@name}' with stdio transport requires a non-empty command array"
-      end
+    def validate_headers!
+      return if @headers.is_a?(Hash) && @headers.keys.all? { |k| k.is_a?(String) } && @headers.values.all? { |v| v.is_a?(String) }
 
-      unless @command.all? { |c| c.is_a?(String) }
+      raise McpConfigurationError,
+        "MCP server '#{@name}' headers must be a Hash with String keys and values"
+    end
+
+    def validate_stdio!
+      if @command.nil? || !@command.is_a?(String) || @command.strip.empty?
         raise McpConfigurationError,
-          "MCP server '#{@name}' command must contain only strings"
+          "MCP server '#{@name}' with stdio transport requires a non-empty command string"
       end
 
       return if @url.nil?
@@ -173,6 +188,17 @@ module AgentHarness
         "MCP server '#{@name}' with #{@transport} transport should not have args (args are only valid for stdio)"
     end
 
+    def normalize_command_and_args(command, args)
+      normalized_args = args || []
+      return [command, normalized_args] unless command.is_a?(Array)
+
+      unless normalized_args.is_a?(Array)
+        return [command.first, normalized_args]
+      end
+
+      [command.first, command[1..] + normalized_args]
+    end
+
     def http_ping_ok?(timeout: 5)
       require "net/http"
       uri = URI.parse(@url)

data/lib/agent_harness/providers/anthropic.rb

@@ -15,6 +15,7 @@ module AgentHarness
     #   response = provider.send_message(prompt: "Hello!")
     class Anthropic < Base
       include RateLimitResetParsing
+      include McpConfigFileSupport
 
       # Model name pattern for Anthropic Claude models
       MODEL_PATTERN = /^claude-[\d.-]+-(?:opus|sonnet|haiku)(?:-\d{8})?$/i
@@ -756,85 +757,8 @@ module AgentHarness
         servers
       end
 
-      def write_mcp_config_file(mcp_servers, working_dir: nil)
-        require "tempfile"
-        require "tmpdir"
-        require "securerandom"
-
-        config = build_claude_mcp_config(mcp_servers)
-        config_json = JSON.generate(config)
-
-        if @executor.is_a?(DockerCommandExecutor)
-          # When running inside a Docker container, write the config file
-          # inside the container so the CLI process can read it.
-          # Track the path so cleanup_mcp_tempfiles! can remove it after execution.
-          container_path = "/tmp/agent_harness_mcp_#{SecureRandom.hex(8)}.json"
-          result = @executor.execute(
-            ["sh", "-c", "cat > #{container_path}"],
-            stdin_data: config_json,
-            timeout: 5
-          )
-          unless result.success?
-            raise McpConfigurationError,
-              "Failed to write MCP config inside container: #{result.stderr}"
-          end
-
-          @mcp_docker_config_paths ||= []
-          @mcp_docker_config_paths << container_path
-
-          container_path
-        else
-          dir = working_dir || Dir.tmpdir
-          file = Tempfile.new(["agent_harness_mcp_", ".json"], dir)
-          file.write(config_json)
-          file.close
-
-          # Hold a reference so the Tempfile is not garbage-collected (and
-          # therefore deleted) before the CLI process reads it.
-          # Cleaned up by cleanup_mcp_tempfiles! after execution.
-          @mcp_config_tempfiles ||= []
-          @mcp_config_tempfiles << file
-
-          file.path
-        end
-      end
-
-      def build_claude_mcp_config(mcp_servers)
-        servers = {}
-        mcp_servers.each do |server|
-          h = if server.stdio?
-            entry = {command: server.command.first}
-            remaining_args = server.command[1..] + server.args
-            entry[:args] = remaining_args unless remaining_args.empty?
-            entry
-          else
-            {url: server.url}
-          end
-          h[:env] = server.env unless server.env.empty?
-          servers[server.name] = h
-        end
-        {mcpServers: servers}
-      end
-
-      def cleanup_mcp_tempfiles!
-        if @mcp_config_tempfiles
-          @mcp_config_tempfiles.each do |file|
-            file.close unless file.closed?
-            file.unlink
-          rescue
-            nil
-          end
-          @mcp_config_tempfiles = nil
-        end
-
-        if @mcp_docker_config_paths
-          @mcp_docker_config_paths.each do |path|
-            @executor.execute(["rm", "-f", path], timeout: 5)
-          rescue
-            nil
-          end
-          @mcp_docker_config_paths = nil
-        end
+      def mcp_provider_key
+        :anthropic
       end
 
       def build_tool_control_flags(tools_option, skip_permission_mode: false)

data/lib/agent_harness/providers/base.rb

@@ -419,7 +419,13 @@ module AgentHarness
       end
 
       def normalize_mcp_servers(options)
-        servers = options[:mcp_servers]
+        if options.key?(:mcp_servers)
+          servers = options[:mcp_servers]
+        else
+          # Configuration stores mcp_servers as a Hash keyed by name; extract values.
+          config_servers = AgentHarness.configuration.mcp_servers
+          servers = config_servers.is_a?(Hash) ? config_servers.values : config_servers
+        end
         return options if servers.nil?
 
         unless servers.is_a?(Array)

data/lib/agent_harness/providers/codex.rb

@@ -9,6 +9,7 @@ module AgentHarness
     # Provides integration with the OpenAI Codex CLI tool.
     class Codex < Base
       include RateLimitResetParsing
+      include McpConfigFileSupport
 
       SUPPORTED_CLI_VERSION = "0.116.0"
       SUPPORTED_CLI_REQUIREMENT = Gem::Requirement.new(">= #{SUPPORTED_CLI_VERSION}", "< 0.117.0").freeze
@@ -186,7 +187,7 @@ module AgentHarness
           vision: false,
           tool_use: true,
           json_mode: false,
-          mcp: false,
+          mcp: true,
           dangerous_mode: true
         }
       end
@@ -199,6 +200,27 @@ module AgentHarness
 
       def cli_env_overrides = {"PAID_CODEX_SUBSCRIPTION_AUTH" => "1"}
 
+      def send_message(prompt:, **options)
+        super
+      ensure
+        cleanup_mcp_tempfiles!
+      end
+
+      def supports_mcp?
+        true
+      end
+
+      def supported_mcp_transports
+        %w[stdio http sse]
+      end
+
+      def build_mcp_flags(mcp_servers, working_dir: nil)
+        return [] if mcp_servers.empty?
+
+        config_path = write_mcp_config_file(mcp_servers, working_dir: working_dir)
+        ["--mcp-config", config_path]
+      end
+
       def test_command_overrides
         ["--skip-git-repo-check", "--output-last-message"]
       end
@@ -449,6 +471,11 @@ module AgentHarness
           cmd += sandbox_bypass_flags
         end
 
+        # Add MCP server flags (validated/normalized by Base#send_message)
+        if options[:mcp_servers]&.any?
+          cmd += build_mcp_flags(options[:mcp_servers])
+        end
+
         if options[:session]
           cmd += session_flags(options[:session])
         end
@@ -1259,6 +1286,10 @@ module AgentHarness
         config_dir = ENV["CODEX_CONFIG_DIR"] || File.expand_path("~/.codex")
         File.join(config_dir, "config.json")
       end
+
+      def mcp_provider_key
+        :codex
+      end
     end
   end
 end

data/lib/agent_harness/providers/mcp_config_file_support.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+module AgentHarness
+  module Providers
+    # Shared concern for writing and cleaning up MCP configuration tempfiles.
+    #
+    # Include this module in any provider that needs to write MCP config files
+    # for CLI processes. The including class must define an +mcp_provider_key+
+    # method that returns the symbol passed to +McpConfigTranslator.for_provider+.
+    module McpConfigFileSupport
+      def write_mcp_config_file(mcp_servers, working_dir: nil)
+        require "tempfile"
+        require "tmpdir"
+        require "securerandom"
+
+        config = McpConfigTranslator.for_provider(mcp_provider_key, mcp_servers)
+        config_json = JSON.generate(config)
+
+        if @executor.is_a?(DockerCommandExecutor)
+          # When running inside a Docker container, write the config file
+          # inside the container so the CLI process can read it.
+          # Track the path so cleanup_mcp_tempfiles! can remove it after execution.
+          container_path = "/tmp/agent_harness_mcp_#{SecureRandom.hex(8)}.json"
+          result = @executor.execute(
+            ["sh", "-c", "cat > #{container_path}"],
+            stdin_data: config_json,
+            timeout: 5
+          )
+          unless result.success?
+            raise McpConfigurationError,
+              "Failed to write MCP config inside container: #{result.stderr}"
+          end
+
+          @mcp_docker_config_paths ||= []
+          @mcp_docker_config_paths << container_path
+
+          container_path
+        else
+          dir = working_dir || Dir.tmpdir
+          file = Tempfile.new(["agent_harness_mcp_", ".json"], dir)
+          file.write(config_json)
+          file.close
+
+          # Hold a reference so the Tempfile is not garbage-collected (and
+          # therefore deleted) before the CLI process reads it.
+          # Cleaned up by cleanup_mcp_tempfiles! after execution.
+          @mcp_config_tempfiles ||= []
+          @mcp_config_tempfiles << file
+
+          file.path
+        end
+      end
+
+      def cleanup_mcp_tempfiles!
+        if @mcp_config_tempfiles
+          @mcp_config_tempfiles.each do |file|
+            file.close unless file.closed?
+            file.unlink
+          rescue
+            nil
+          end
+          @mcp_config_tempfiles = nil
+        end
+
+        if @mcp_docker_config_paths
+          @mcp_docker_config_paths.each do |path|
+            @executor.execute(["rm", "-f", path], timeout: 5)
+          rescue
+            nil
+          end
+          @mcp_docker_config_paths = nil
+        end
+      end
+    end
+  end
+end

data/lib/agent_harness/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AgentHarness
-  VERSION = "0.11.1"
+  VERSION = "0.11.2"
 end

data/lib/agent_harness.rb

@@ -278,6 +278,8 @@ end
 # Core components
 require_relative "agent_harness/errors"
 require_relative "agent_harness/mcp_server"
+require_relative "agent_harness/mcp_config_loader"
+require_relative "agent_harness/mcp_config_translator"
 require_relative "agent_harness/sub_agent_config"
 require_relative "agent_harness/sub_agent_file_loader"
 require_relative "agent_harness/sub_agent_translator"
@@ -301,6 +303,7 @@ require_relative "agent_harness/providers/adapter"
 require_relative "agent_harness/providers/base"
 require_relative "agent_harness/providers/token_usage_parsing"
 require_relative "agent_harness/providers/rate_limit_reset_parsing"
+require_relative "agent_harness/providers/mcp_config_file_support"
 require_relative "agent_harness/providers/anthropic"
 require_relative "agent_harness/providers/aider"
 require_relative "agent_harness/providers/codex"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: agent-harness
 version: !ruby/object:Gem::Version
-  version: 0.11.1
+  version: 0.11.2
 platform: ruby
 authors:
 - Bart Agapinan
@@ -107,6 +107,8 @@ files:
 - lib/agent_harness/error_taxonomy.rb
 - lib/agent_harness/errors.rb
 - lib/agent_harness/execution_preparation.rb
+- lib/agent_harness/mcp_config_loader.rb
+- lib/agent_harness/mcp_config_translator.rb
 - lib/agent_harness/mcp_server.rb
 - lib/agent_harness/openai_compatible_transport.rb
 - lib/agent_harness/orchestration/circuit_breaker.rb
@@ -126,6 +128,7 @@ files:
 - lib/agent_harness/providers/gemini.rb
 - lib/agent_harness/providers/github_copilot.rb
 - lib/agent_harness/providers/kilocode.rb
+- lib/agent_harness/providers/mcp_config_file_support.rb
 - lib/agent_harness/providers/mistral_vibe.rb
 - lib/agent_harness/providers/opencode.rb
 - lib/agent_harness/providers/rate_limit_reset_parsing.rb