agent-harness 0.17.0 → 0.17.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3de614caae7f2e65904c6118c209a2e149769e518e4a200c3efb77e8424db93
-  data.tar.gz: 8bd46e4f308b5de943554bd1dc3a4b31ab9180e2d1a2b61fd126840de6ff9029
+  metadata.gz: f15e1c2315d17c1250d547c58c3f04b2d9daf491a3b56fcb23e58b824a68e6a1
+  data.tar.gz: 42bd06fe77703e25a8c61ec8667de90d1b3e84e483d1651b36c0e0013cdc610a
 SHA512:
-  metadata.gz: 4981d9c872f63e2739f7bda8f182387aac3218f7522e66cdc137aa1d34dfb51516b4e82e3193b8ee4b983c5a19cb68185d7bab5b630ace98a6b9bfad1b2ffb8d
-  data.tar.gz: d84d943eccbc10258eafbd60f53b6aad796cc2f2b9b663addb6c178cc29934ef3ee2d0c92925d627edb4c875c39a912915f0f7f04ed273be451a4a0a314bc047
+  metadata.gz: ba35bf2c9b5e6de554664cfbf1dacbb70d0d9ceb017c6fc1be8bb8c63b73ab295b1d01e28de3650ac74d08d4908650dd11dfc267099482e2ea933a24ae33d5e9
+  data.tar.gz: c8fcf04ce5c2ab05c2007fc6642457adbe2bafb48cc11e0290b97834d269c4fbe8be411d9227aceef8315975d97a9ff38d8beae270d34365f07f2e851c883ddb

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.17.0"
+  ".": "0.17.2"
 }

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 ## [Unreleased]
 
+## [0.17.2](https://github.com/viamin/agent-harness/compare/agent-harness/v0.17.1...agent-harness/v0.17.2) (2026-05-05)
+
+
+### Bug Fixes
+
+* 160: Add support for the pi agent CLI ([#203](https://github.com/viamin/agent-harness/issues/203)) ([0aeb607](https://github.com/viamin/agent-harness/commit/0aeb607ea98b52ba8202726dc946b8c1db09a3cd))
+
+## [0.17.1](https://github.com/viamin/agent-harness/compare/agent-harness/v0.17.0...agent-harness/v0.17.1) (2026-05-05)
+
+
+### Bug Fixes
+
+* 173: Smoke test contract timeout (30s) overrides caller timeout, breaking slow models ([#205](https://github.com/viamin/agent-harness/issues/205)) ([3a1e301](https://github.com/viamin/agent-harness/commit/3a1e301e36ef8957fd440f2782b3b8e4687b473c))
+
 ## [0.17.0](https://github.com/viamin/agent-harness/compare/agent-harness/v0.16.1...agent-harness/v0.17.0) (2026-05-03)
 
 

data/README.md

@@ -5,7 +5,7 @@ A unified Ruby interface for CLI-based AI coding agents like Claude Code, Cursor
 ## Features
 
 - **Unified Interface**: Single API for multiple AI coding agents
-- **9 Built-in Providers**: Claude Code, Cursor, Gemini CLI, GitHub Copilot, Codex, Aider, OpenCode, Kilocode, Mistral Vibe
+- **10 Built-in Providers**: Claude Code, Cursor, Gemini CLI, GitHub Copilot, Codex, Pi, Aider, OpenCode, Kilocode, Mistral Vibe
 - **Full Orchestration**: Provider switching, circuit breakers, rate limiting, and health monitoring
 - **Flexible Configuration**: YAML, Ruby DSL, or environment variables
 - **Token Tracking**: Monitor usage across providers for cost and limit management
@@ -104,6 +104,7 @@ end
 | `:gemini` | `gemini` | Google Gemini CLI |
 | `:github_copilot` | `copilot` | GitHub Copilot CLI |
 | `:codex` | `codex` | OpenAI Codex CLI |
+| `:pi` | `pi` | Pi coding agent CLI |
 | `:aider` | `aider` | Aider coding assistant |
 | `:opencode` | `opencode` | OpenCode CLI |
 | `:kilocode` | `kilo` | Kilocode CLI |
@@ -187,7 +188,7 @@ puts contract[:supported_versions][:requirement]
 
 # List all registered providers
 AgentHarness::Providers::Registry.instance.all
-# => [:claude, :cursor, :gemini, :github_copilot, :codex, :opencode, :kilocode, :aider, :mistral_vibe]
+# => [:claude, :cursor, :gemini, :github_copilot, :pi, :codex, :opencode, :kilocode, :aider, :mistral_vibe]
 ```
 
 For Claude, the install contract is the first-class source of truth for:

data/lib/agent_harness/configuration.rb

@@ -377,7 +377,7 @@ module AgentHarness
   # Provider-specific configuration
   class ProviderConfig
     attr_accessor :enabled, :type, :priority, :models, :default_flags, :timeout, :model,
-      :externally_sandboxed
+      :provider, :externally_sandboxed
 
     attr_reader :name
 
@@ -390,6 +390,7 @@ module AgentHarness
       @default_flags = []
       @timeout = nil
       @model = nil
+      @provider = nil
       @externally_sandboxed = false
     end
 

data/lib/agent_harness/provider_health_check.rb

@@ -324,7 +324,18 @@ module AgentHarness
         # contract[:timeout]. When the provider overrides #smoke_test without
         # publishing a contract, forward the validated health-check timeout so
         # the override can honour it instead of running without any limit.
-        smoke_timeout = smoke_contract ? nil : timeout
+        # However, when the caller explicitly provides a timeout that exceeds
+        # the contract timeout, honour the caller's intent so slow models
+        # are not prematurely killed by the contract default.
+        contract_timeout = smoke_contract&.dig(:timeout)
+        valid_contract_timeout = contract_timeout.is_a?(Numeric) && contract_timeout.positive?
+        smoke_timeout = if valid_contract_timeout && timeout && timeout > contract_timeout
+          timeout
+        elsif smoke_contract
+          nil
+        else
+          timeout
+        end
         smoke = provider_instance.smoke_test(timeout: smoke_timeout, provider_runtime: provider_runtime)
         unless smoke[:ok]
           return build_result(

data/lib/agent_harness/provider_runtime.rb

@@ -46,9 +46,9 @@ module AgentHarness
       validate_optional_string!(:base_url, base_url)
       validate_optional_string!(:api_provider, api_provider)
 
-      @model = model
-      @base_url = base_url
-      @api_provider = api_provider
+      @model = normalize_optional_string(model)
+      @base_url = normalize_optional_string(base_url)
+      @api_provider = normalize_optional_string(api_provider)
 
       env_hash = env.nil? ? {} : env
       unless env_hash.is_a?(Hash)
@@ -113,9 +113,9 @@ module AgentHarness
             "chat_tools must be an Array of Hashes; invalid element at index #{index}: #{tool.inspect} (#{tool.class})"
         end
       end
-      @chat_base_url = chat_base_url
-      @chat_model = chat_model
-      @chat_api_key = chat_api_key
+      @chat_base_url = normalize_optional_string(chat_base_url)
+      @chat_model = normalize_optional_string(chat_model)
+      @chat_api_key = normalize_optional_string(chat_api_key)
       @chat_max_tokens = chat_max_tokens
       @chat_tools = normalized_chat_tools&.freeze
 
@@ -191,5 +191,12 @@ module AgentHarness
 
       raise ArgumentError, "#{name} must be a String or nil (got #{value.class})"
     end
+
+    def normalize_optional_string(value)
+      return value unless value.respond_to?(:strip)
+
+      normalized = value.strip
+      normalized.empty? ? nil : normalized
+    end
   end
 end

data/lib/agent_harness/providers/pi.rb

@@ -0,0 +1,214 @@
+# frozen_string_literal: true
+
+require "rubygems/requirement"
+
+module AgentHarness
+  module Providers
+    # Pi coding agent CLI provider
+    #
+    # Provides integration with the Pi terminal coding agent.
+    class Pi < Base
+      CLI_PACKAGE = "@mariozechner/pi-coding-agent"
+      SUPPORTED_CLI_VERSION = "0.73.0"
+      SUPPORTED_CLI_REQUIREMENT = Gem::Requirement.new("= #{SUPPORTED_CLI_VERSION}").freeze
+
+      class << self
+        def provider_name
+          :pi
+        end
+
+        def binary_name
+          "pi"
+        end
+
+        def available?
+          executor = AgentHarness.configuration.command_executor
+          !!executor.which(binary_name)
+        end
+
+        def provider_metadata_overrides
+          {
+            auth: {
+              service: :pi,
+              api_family: :multi_provider
+            }
+          }
+        end
+
+        def firewall_requirements
+          {
+            domains: [
+              "pi.dev"
+            ],
+            ip_ranges: []
+          }
+        end
+
+        def instruction_file_paths
+          [
+            {
+              path: "AGENTS.md",
+              description: "Pi agent instructions",
+              symlink: false
+            },
+            {
+              path: "SYSTEM.md",
+              description: "Pi system prompt override",
+              symlink: false
+            }
+          ]
+        end
+
+        def discover_models
+          return [] unless available?
+          []
+        end
+
+        def installation_contract(version: SUPPORTED_CLI_VERSION)
+          version = version.strip if version.respond_to?(:strip)
+
+          unless version.is_a?(String) && !version.empty?
+            raise ArgumentError,
+              "Unsupported Pi CLI version #{version.inspect}; " \
+              "supported versions must satisfy #{SUPPORTED_CLI_REQUIREMENT}"
+          end
+
+          parsed_version = begin
+            Gem::Version.new(version)
+          rescue ArgumentError
+            raise ArgumentError,
+              "Unsupported Pi CLI version #{version.inspect}; " \
+              "supported versions must satisfy #{SUPPORTED_CLI_REQUIREMENT}"
+          end
+
+          unless SUPPORTED_CLI_REQUIREMENT.satisfied_by?(parsed_version)
+            raise ArgumentError,
+              "Unsupported Pi CLI version #{version.inspect}; " \
+              "supported versions must satisfy #{SUPPORTED_CLI_REQUIREMENT}"
+          end
+
+          package = "#{CLI_PACKAGE}@#{version}".freeze
+          install_command_prefix = ["npm", "install", "-g", "--ignore-scripts"].freeze
+          install_command = (install_command_prefix + [package]).freeze
+          supported_versions = [version].freeze
+          version_requirement = SUPPORTED_CLI_REQUIREMENT.requirements
+            .map { |op, ver| "#{op} #{ver}".freeze }
+            .freeze
+
+          contract = {
+            source: :npm,
+            package: package,
+            package_name: CLI_PACKAGE,
+            version: version,
+            version_requirement: version_requirement,
+            binary_name: binary_name,
+            install_command_prefix: install_command_prefix,
+            install_command: install_command,
+            supported_versions: supported_versions
+          }
+
+          contract.each_value do |value|
+            value.freeze if value.is_a?(String)
+          end
+          contract.freeze
+        end
+
+        def smoke_test_contract
+          Base::DEFAULT_SMOKE_TEST_CONTRACT
+        end
+      end
+
+      def name
+        "pi"
+      end
+
+      def display_name
+        "Pi Coding Agent"
+      end
+
+      def configuration_schema
+        {
+          fields: [
+            {
+              name: :model,
+              type: :string,
+              label: "Model",
+              required: false,
+              hint: "Pi model pattern or ID passed to --model"
+            },
+            {
+              name: :provider,
+              type: :string,
+              label: "Provider",
+              required: false,
+              hint: "Pi provider name passed to --provider"
+            }
+          ],
+          auth_modes: %i[api_key oauth],
+          openai_compatible: false
+        }
+      end
+
+      def capabilities
+        {
+          streaming: false,
+          file_upload: true,
+          vision: true,
+          tool_use: true,
+          # Pi's non-interactive CLI currently exposes only text print mode.
+          # Keep JSON mode disabled until the CLI ships a structured output flag.
+          json_mode: false,
+          mcp: false,
+          dangerous_mode: false
+        }
+      end
+
+      def error_patterns
+        COMMON_ERROR_PATTERNS
+      end
+
+      def supports_tool_control?
+        true
+      end
+
+      def auth_type
+        :oauth
+      end
+
+      def execution_semantics
+        {
+          prompt_delivery: :flag,
+          output_format: :text,
+          sandbox_aware: false,
+          uses_subcommand: false,
+          non_interactive_flag: "-p",
+          legitimate_exit_codes: [0],
+          stderr_is_diagnostic: true,
+          parses_rate_limit_reset: false
+        }
+      end
+
+      protected
+
+      def build_command(prompt, options)
+        runtime = options[:provider_runtime]
+        provider = runtime&.api_provider || @config.provider
+        model = runtime&.model || @config.model
+
+        cmd = [self.class.binary_name, "--no-session"]
+        cmd += @config.default_flags if @config.default_flags&.any?
+        cmd += runtime.flags if runtime
+        cmd += ["--provider", provider] if provider
+        cmd += ["--model", model] if model
+        cmd << "--no-tools" if options[:tools] == :none
+        cmd += ["-p", prompt]
+
+        cmd
+      end
+
+      def default_timeout
+        300
+      end
+    end
+  end
+end

data/lib/agent_harness/providers/registry.rb

@@ -27,6 +27,7 @@ module AgentHarness
           class_name: :GithubCopilot,
           aliases: [:copilot]
         },
+        {name: :pi, require_path: "agent_harness/providers/pi", class_name: :Pi, aliases: []},
         {name: :codex, require_path: "agent_harness/providers/codex", class_name: :Codex, aliases: []},
         {name: :opencode, require_path: "agent_harness/providers/opencode", class_name: :Opencode, aliases: []},
         {name: :kilocode, require_path: "agent_harness/providers/kilocode", class_name: :Kilocode, aliases: []},

data/lib/agent_harness/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: agent-harness
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.2
 platform: ruby
 authors:
 - Bart Agapinan
@@ -133,6 +133,7 @@ files:
 - 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/pi.rb
 - lib/agent_harness/providers/rate_limit_reset_parsing.rb
 - lib/agent_harness/providers/registry.rb
 - lib/agent_harness/providers/token_usage_parsing.rb