insika 0.9.0 → 1.0.1

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: insika
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Guilherme Lages Santos
@@ -24,33 +24,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: ruby_llm
+  name: json_schemer
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: '2.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: '2.5'
 - !ruby/object:Gem::Dependency
-  name: ruby_llm-mcp
+  name: ruby_llm
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.1'
 - !ruby/object:Gem::Dependency
   name: falcon
   requirement: !ruby/object:Gem::Requirement
@@ -173,6 +173,7 @@ files:
 - docs/BENCHMARK.md
 - docs/CHANNELS.md
 - docs/CONTEXT.md
+- docs/CROSS-HARNESS-BENCH.md
 - docs/DEMO.md
 - docs/DEPLOY.md
 - docs/EMBEDDING.md
@@ -184,17 +185,21 @@ files:
 - docs/MEDIA.md
 - docs/OBSERVABILITY.md
 - docs/OUTCOMES.md
+- docs/PERFORMANCE.md
 - docs/PLUGINS.md
 - docs/POLICY.md
 - docs/REFINEMENT.md
 - docs/RELEASING.md
 - docs/ROUTER.md
+- docs/RUBYLLM_2_MIGRATION.md
 - docs/RUNNING-LOCAL.md
 - docs/SANDBOX.md
 - docs/SCHEDULING.md
 - docs/SECURITY.md
+- docs/SHARED-CONVERSATIONS.md
 - docs/SKILLS.md
 - docs/SOAK.md
+- docs/STUDIO.md
 - docs/TEMPLATES.md
 - docs/TOOLS.md
 - docs/WHY.md
@@ -221,6 +226,7 @@ files:
 - docs/ship.md
 - docs/start-here.md
 - lib/insika.rb
+- lib/insika/agent_defaults.rb
 - lib/insika/agent_file_store.rb
 - lib/insika/agent_profile.rb
 - lib/insika/alert_dispatcher.rb
@@ -258,6 +264,7 @@ files:
 - lib/insika/commands/delete_artifact.rb
 - lib/insika/commands/delete_concept.rb
 - lib/insika/commands/delete_data_tool.rb
+- lib/insika/commands/delete_kit.rb
 - lib/insika/commands/delete_llm_provider.rb
 - lib/insika/commands/delete_mcp.rb
 - lib/insika/commands/delete_skill.rb
@@ -268,7 +275,6 @@ files:
 - lib/insika/commands/freeze_funnel_baseline.rb
 - lib/insika/commands/gate_harvest.rb
 - lib/insika/commands/gate_refinement.rb
-- lib/insika/commands/import_mcp_tools.rb
 - lib/insika/commands/import_tools.rb
 - lib/insika/commands/issue_tenant_token.rb
 - lib/insika/commands/judge_shadow_pairs.rb
@@ -288,6 +294,7 @@ files:
 - lib/insika/commands/restore_data_tool.rb
 - lib/insika/commands/restore_system_file.rb
 - lib/insika/commands/resume_task.rb
+- lib/insika/commands/review_guardrail_flag.rb
 - lib/insika/commands/revoke_contact.rb
 - lib/insika/commands/revoke_token.rb
 - lib/insika/commands/rollback_harvest.rb
@@ -310,6 +317,7 @@ files:
 - lib/insika/commands/write_concept.rb
 - lib/insika/commands/write_data_tool.rb
 - lib/insika/commands/write_golden.rb
+- lib/insika/commands/write_kit.rb
 - lib/insika/commands/write_skill.rb
 - lib/insika/commands/write_system_file.rb
 - lib/insika/compaction.rb
@@ -324,12 +332,14 @@ files:
 - lib/insika/context/providers/fence_notice.rb
 - lib/insika/context/providers/knowledge.rb
 - lib/insika/context/providers/memory.rb
+- lib/insika/context/providers/pending_confirmation.rb
 - lib/insika/context/providers/prompt.rb
 - lib/insika/context/providers/request.rb
 - lib/insika/context/providers/session.rb
 - lib/insika/context/providers/skill.rb
 - lib/insika/context/providers/skill_trigger.rb
 - lib/insika/context/providers/tool_search.rb
+- lib/insika/context/reranker.rb
 - lib/insika/context_trace_store.rb
 - lib/insika/cron.rb
 - lib/insika/delegation_store.rb
@@ -353,6 +363,7 @@ files:
 - lib/insika/evals/baseline.rb
 - lib/insika/evals/golden.rb
 - lib/insika/evals/judge.rb
+- lib/insika/evals/moderator_compare.rb
 - lib/insika/evals/pairwise.rb
 - lib/insika/evals/persona.rb
 - lib/insika/evals/report.rb
@@ -384,24 +395,27 @@ files:
 - lib/insika/hooks.rb
 - lib/insika/http_client.rb
 - lib/insika/inbound_log.rb
+- lib/insika/kits.rb
 - lib/insika/knowledge.rb
 - lib/insika/knowledge_store.rb
 - lib/insika/llm_configurator.rb
 - lib/insika/llm_provider_store.rb
+- lib/insika/llm_trace_store.rb
 - lib/insika/loop_detector.rb
 - lib/insika/mcp_client.rb
-- lib/insika/mcp_http_client.rb
 - lib/insika/mcp_json.rb
 - lib/insika/mcp_live_tool.rb
 - lib/insika/mcp_store.rb
-- lib/insika/mcp_tool_ingestor.rb
 - lib/insika/mcp_tool_registry.rb
 - lib/insika/media.rb
 - lib/insika/memory_audit_store.rb
 - lib/insika/memory_store.rb
+- lib/insika/memory_trim.rb
 - lib/insika/message_origin.rb
 - lib/insika/middleware.rb
+- lib/insika/model_metrics_store.rb
 - lib/insika/model_policy.rb
+- lib/insika/model_registry.rb
 - lib/insika/model_resolver.rb
 - lib/insika/model_selection.rb
 - lib/insika/model_visible.rb
@@ -451,9 +465,11 @@ files:
 - lib/insika/safety/grounding_enforcer.rb
 - lib/insika/safety/grounding_validator.rb
 - lib/insika/safety/input_guardrail.rb
+- lib/insika/safety/judge_moderator.rb
 - lib/insika/safety/moderator.rb
 - lib/insika/safety/output_filter.rb
 - lib/insika/safety/output_validator.rb
+- lib/insika/safety/prompt_echo.rb
 - lib/insika/safety/safe_responses.rb
 - lib/insika/sandbox.rb
 - lib/insika/sandbox/boundary.rb
@@ -484,6 +500,7 @@ files:
 - lib/insika/session_store.rb
 - lib/insika/settings_store.rb
 - lib/insika/shadow_pair_store.rb
+- lib/insika/shared_conversations.rb
 - lib/insika/shutdown.rb
 - lib/insika/skill_catalog.rb
 - lib/insika/skill_store.rb
@@ -493,24 +510,55 @@ files:
 - lib/insika/spoken_transcript.rb
 - lib/insika/steer_injector.rb
 - lib/insika/store.rb
+- lib/insika/stores/json_value.rb
 - lib/insika/stores/memory.rb
+- lib/insika/stores/postgres.rb
 - lib/insika/stores/sqlite.rb
+- lib/insika/stores/turn_counter.rb
 - lib/insika/studio/app.rb
 - lib/insika/studio/assets/dist/application.css
 - lib/insika/studio/assets/dist/application.js
 - lib/insika/studio/assets/dist/favicon.svg
+- lib/insika/studio/assets/dist/fonts/ibm-plex-mono-latin-400-normal.woff2
+- lib/insika/studio/assets/dist/fonts/ibm-plex-mono-latin-500-normal.woff2
+- lib/insika/studio/assets/dist/fonts/ibm-plex-mono-latin-600-normal.woff2
+- lib/insika/studio/assets/dist/fonts/ibm-plex-sans-latin-wght-normal.woff2
 - lib/insika/studio/forms.rb
 - lib/insika/studio/nav_icons.rb
+- lib/insika/studio/views/_agent_config_retrieval.erb
+- lib/insika/studio/views/_agent_picker.erb
 - lib/insika/studio/views/_agent_tab_cache.erb
 - lib/insika/studio/views/_agent_tab_config.erb
 - lib/insika/studio/views/_agent_tab_history.erb
 - lib/insika/studio/views/_agent_tab_loops.erb
 - lib/insika/studio/views/_agent_tab_memory.erb
 - lib/insika/studio/views/_agent_tab_outcomes.erb
+- lib/insika/studio/views/_agent_tab_performance.erb
 - lib/insika/studio/views/_agent_tab_prompts.erb
 - lib/insika/studio/views/_agent_tab_skills.erb
+- lib/insika/studio/views/_agent_tab_tools.erb
 - lib/insika/studio/views/_agents_master.erb
+- lib/insika/studio/views/_cfg_alerts.erb
+- lib/insika/studio/views/_cfg_budget.erb
+- lib/insika/studio/views/_cfg_distill.erb
+- lib/insika/studio/views/_cfg_followup.erb
+- lib/insika/studio/views/_cfg_grounding.erb
+- lib/insika/studio/views/_cfg_guardrails.erb
+- lib/insika/studio/views/_cfg_harvest.erb
+- lib/insika/studio/views/_cfg_reliability.erb
+- lib/insika/studio/views/_chats_master.erb
+- lib/insika/studio/views/_customers_master.erb
+- lib/insika/studio/views/_empty.erb
+- lib/insika/studio/views/_facts_master.erb
+- lib/insika/studio/views/_harvest_candidate_detail.erb
+- lib/insika/studio/views/_identity.erb
+- lib/insika/studio/views/_kpi.erb
 - lib/insika/studio/views/_message.erb
+- lib/insika/studio/views/_model_chart.erb
+- lib/insika/studio/views/_refinement_master.erb
+- lib/insika/studio/views/_tool_edit.erb
+- lib/insika/studio/views/_tool_users.erb
+- lib/insika/studio/views/_why.erb
 - lib/insika/studio/views/agent_detail.erb
 - lib/insika/studio/views/agents.erb
 - lib/insika/studio/views/approvals.erb
@@ -520,26 +568,31 @@ files:
 - lib/insika/studio/views/customer.erb
 - lib/insika/studio/views/customers.erb
 - lib/insika/studio/views/evals.erb
+- lib/insika/studio/views/fact_detail.erb
 - lib/insika/studio/views/facts.erb
 - lib/insika/studio/views/followups.erb
 - lib/insika/studio/views/funnel.erb
 - lib/insika/studio/views/harvest.erb
 - lib/insika/studio/views/home.erb
+- lib/insika/studio/views/inbox.erb
+- lib/insika/studio/views/kits.erb
 - lib/insika/studio/views/knowledge.erb
 - lib/insika/studio/views/layout.erb
+- lib/insika/studio/views/live.erb
 - lib/insika/studio/views/login.erb
 - lib/insika/studio/views/mcp.erb
+- lib/insika/studio/views/models.erb
 - lib/insika/studio/views/not_found.erb
 - lib/insika/studio/views/parity.erb
 - lib/insika/studio/views/playground.erb
 - lib/insika/studio/views/refinement.erb
+- lib/insika/studio/views/refinement_detail.erb
 - lib/insika/studio/views/session.erb
 - lib/insika/studio/views/settings.erb
 - lib/insika/studio/views/skills.erb
 - lib/insika/studio/views/system_files.erb
 - lib/insika/studio/views/task.erb
 - lib/insika/studio/views/tasks.erb
-- lib/insika/studio/views/tool_edit.erb
 - lib/insika/studio/views/tools.erb
 - lib/insika/subagent_graph.rb
 - lib/insika/system_file_store.rb
@@ -548,6 +601,7 @@ files:
 - lib/insika/telemetry.rb
 - lib/insika/telemetry/pricing.rb
 - lib/insika/telemetry/recorder.rb
+- lib/insika/telemetry/ruby_llm_instrumenter.rb
 - lib/insika/templates.rb
 - lib/insika/templates/browser-agent/README.md
 - lib/insika/templates/browser-agent/agent.rb
@@ -562,6 +616,7 @@ files:
 - lib/insika/templates/travel-planner/README.md
 - lib/insika/templates/travel-planner/agent.rb
 - lib/insika/testing/store_contract.rb
+- lib/insika/text_tool_call.rb
 - lib/insika/tick.rb
 - lib/insika/timezone.rb
 - lib/insika/token_estimator.rb
@@ -580,6 +635,7 @@ files:
 - lib/insika/tools/a2a_remote.rb
 - lib/insika/tools/agent_enum.rb
 - lib/insika/tools/concurrency.rb
+- lib/insika/tools/confirm_pending.rb
 - lib/insika/tools/data_defined_tool.rb
 - lib/insika/tools/generate_image.rb
 - lib/insika/tools/load_knowledge.rb

data/lib/insika/commands/import_mcp_tools.rb

@@ -1,48 +0,0 @@
-# frozen_string_literal: true
-
-require "time"
-
-module Insika
-  module Commands
-    # Control command: LIVE MCP ingestion. Receives the NAME of
-    # an MCP instance, delegates to McpToolIngestor (discovers via an injectable client
-    # -> builds the manifest -> reuses :import_tools: batch upsert + hot reload)
-    # and returns the per-tool report in the shape of :import_tools, plus
-    # `instance:`. Idempotent (re-ingesting reconciles). A per-tool failure is isolated
-    # in `errors[]` (R4); an absent/disabled/url-less instance raises.
-    #   -> { instance, version, created: [names], updated: [names], errors: [{tool,error}] }
-    class ImportMcpTools
-      def initialize(ingestor:, event_stream:)
-        @ingestor = ingestor
-        @event_stream = event_stream
-      end
-
-      def call(command)
-        p = symbolize(command.payload)
-        name = Insika::Coercion.presence(p[:name]) ||
-               (raise Insika::ValidationError, "import_mcp_tools: 'name' (MCP instance) is required")
-
-        report = @ingestor.ingest(name)
-        emit(report)
-        report
-      end
-
-      private
-
-      # Emits only COUNTS + the instance name (never headers/env/secrets): 0 leakage.
-      def emit(report)
-        @event_stream.emit(Insika::Event.new(
-                             type: :mcp_tools_imported,
-                             data: { instance: report[:instance],
-                                     created: report[:created].size, updated: report[:updated].size,
-                                     errors: report[:errors].size },
-                             meta: { at: Time.now.utc.iso8601 }
-                           ))
-      end
-
-      def symbolize(payload)
-        (payload || {}).each_with_object({}) { |(k, v), acc| acc[k.to_sym] = v }
-      end
-    end
-  end
-end

data/lib/insika/mcp_http_client.rb

@@ -1,67 +0,0 @@
-# frozen_string_literal: true
-
-require "json"
-
-module Insika
-  # MINIMAL MCP client over HTTP JSON-RPC. Discovers the tools
-  # of an MCP instance with HTTP transport by making a JSON-RPC 2.0 `tools/list`
-  # POST to the instance endpoint, behind the EgressGuard (SSRF — the url comes
-  # from editable config). It is the DEFAULT client injected into the
-  # McpToolIngestor; tests pass a Fake (duck-typed) in its place.
-  #
-  # Contract (MCP client duck-type): `#list_tools -> [{name, description,
-  # inputSchema}]` — the same MCP envelope that the ToolManifest adapter normalizes.
-  #
-  # SCOPE (bounded): only the minimal handshake of ONE stateless `tools/list`
-  # POST. Does NOT implement the full MCP session lifecycle (initialize/protocol
-  # negotiation/session-id/notifications) nor the stdio transport — that is the
-  # "real MCP transport", later work (out-of-scope, see spec). It serves
-  # simple HTTP MCP servers (direct JSON-RPC) and proves the ingestion seam.
-  class McpHttpClient
-    JSONRPC_VERSION = "2.0"
-
-    def initialize(url:, http: Insika::HttpClient.new, egress: Insika::EgressGuard,
-                   egress_options: {}, headers: {}, timeout: nil)
-      @url = url.to_s
-      @http = http
-      @egress = egress
-      @egress_options = egress_options
-      @headers = { "Content-Type" => "application/json", "Accept" => "application/json" }.merge(headers || {})
-      @timeout = timeout
-    end
-
-    # -> [ { "name", "description", "inputSchema" } ]. Raises Insika::Error on
-    # blocked egress, HTTP != 2xx, invalid JSON or a JSON-RPC error.
-    def list_tools
-      reason = @egress.violation(@url, **@egress_options)
-      raise Insika::Error, "MCP target blocked: #{reason}" if reason
-
-      result = @http.request(method: "POST", url: @url, headers: @headers,
-                             body: request_body("tools/list", {}), timeout: @timeout)
-      Array(rpc_result(result).fetch("tools", []))
-    end
-
-    private
-
-    def request_body(method, params)
-      JSON.generate(jsonrpc: JSONRPC_VERSION, id: 1, method: method, params: params)
-    end
-
-    # Validates the JSON-RPC envelope and returns `result` (Hash). HTTP/parse/error -> Error.
-    def rpc_result(result)
-      status = result[:status].to_i
-      raise Insika::Error, "MCP HTTP #{status}: #{result[:body].to_s[0, 200]}" if status >= 400
-
-      parsed = begin
-        JSON.parse(result[:body].to_s)
-      rescue JSON::ParserError => e
-        raise Insika::Error, "MCP response is not JSON: #{e.message}"
-      end
-      if (err = parsed["error"])
-        raise Insika::Error, "MCP JSON-RPC error: #{err["message"] || err.inspect}"
-      end
-
-      parsed["result"] || {}
-    end
-  end
-end

data/lib/insika/mcp_tool_ingestor.rb

@@ -1,165 +0,0 @@
-# frozen_string_literal: true
-
-require "json"
-
-module Insika
-  # LIVE MCP ingestion (/ spec): discovers the tools of an
-  # MCP instance at RUNTIME (no hand-written manifest) and ingests them as
-  # data-tools. Given an McpStore instance + an INJECTABLE MCP client
-  # (duck-typed: `#list_tools -> [{name, description, inputSchema}]`), it builds a
-  # ToolManifest and REUSES the ingestion path (the:import_tools Command:
-  # batch upsert into the ToolStore + hot reload + per-tool report + partial-
-  # failure isolation R4). The ToolManifest MCP adapter (`inputSchema`) is reused
-  # — no schema parsing here.
-  #
-  # GENERIC: nothing here mentions a consumer/gateway. The MCP instance is DATA in the store.
-  #
-  # BINDING STRATEGY (this stage's choice, bounded):
-  #   Each discovered tool becomes an HTTP data-tool that makes a JSON-RPC 2.0
-  #   `tools/call` POST to the instance endpoint (url). The tool name is resolved
-  #   at INGESTION (literal in the body); the model's arguments come in as `{{param}}`
-  #   per TOP-level property of the inputSchema (with quoting by type — strings
-  #   quoted, others raw, via the DataDefinedTool :body encode). This way the tool
-  #   runs through the SAME HTTP path as the other data-tools (egress guard, secret
-  #   headers, hot reload) — no new execution code.
-  #
-  #   Each tool gets `group: "mcp:<instance>"` so the per-group gating
-  #   (tools_allow_groups) works for free.
-  #
-  # DEFERRED / OUT-OF-SCOPE (documented — spec):
-  #   - Real MCP transport: only instances with a `url` (http transport) are ingestible;
-  #     stdio has no HTTP endpoint -> raises a clear error (later work).
-  #   - MCP session lifecycle (initialize/negotiation/session-id/notifications) and the
-  #     UNWRAP of the `tools/call` response (`{content:[{type,text}]}`) — the binding
-  #     makes a stateless POST and returns the raw body (extract body_raw). A server
-  #     that requires the handshake before answering `tools/list`/`tools/call` is not
-  #     reachable yet — that IS the real transport, not this minimal client.
-  #   - Tools whose name/top-level property is outside the ToolDefinition NAME_RE
-  #     (uppercase/hyphens) are ISOLATED into `errors[]` by the import (R4).
-  #
-  # CREDENTIAL INJECTION: the instance `env` (Hash) is sent verbatim as HTTP
-  # headers, on BOTH the discovery request (`tools/list`, this file) and every
-  # ingested tool's `tools/call` binding (`build_manifest`'s `defaults.headers`)
-  # — literal header-name -> value, the same convention as a data-tool's own
-  # `secret_headers` (the operator types the full value, "Bearer xxx" included;
-  # no magic prefixing). An instance with no `env` sends the bare
-  # `Content-Type` header only, byte-for-byte what shipped before this.
-  class McpToolIngestor
-    def initialize(mcp_store:, import_tools:, client_factory: nil)
-      @mcp_store = mcp_store
-      @import_tools = import_tools
-      # Per-instance client factory (default: minimal JSON-RPC HTTP client).
-      # Injectable for tests (Fake) and to swap for a real transport later.
-      @client_factory = client_factory || method(:default_client)
-    end
-
-    # Discovers + ingests the tools of instance `name`. `client` is injectable
-    # (Fake in tests); absent -> the factory builds one from the record. -> the
-    # import_tools report + `instance:` ({ instance:, version:, created:, updated:, errors: }).
-    def ingest(name, client: nil)
-      manifest = manifest_for(name, client: client)
-      report = @import_tools.call(Insika::Command.build(:import_tools, manifest, transport: :internal))
-      report.merge(instance: name.to_s)
-    end
-
-    # Discovers the tools and builds the manifest Hash (without ingesting) — isolable for testing.
-    def manifest_for(name, client: nil)
-      record = @mcp_store.get_raw(name.to_s)
-      raise Insika::NotFoundError, "MCP instance '#{name}' not found" if record.nil?
-      raise Insika::ValidationError, "MCP instance '#{name}' is disabled" unless record["enabled"]
-
-      url = presence(record["url"])
-      if url.nil?
-        raise Insika::ValidationError,
-              "MCP instance '#{name}' has no url: live ingestion requires HTTP transport " \
-              "(stdio is later work)"
-      end
-
-      tools = Array((client || @client_factory.call(record)).list_tools)
-      build_manifest(name.to_s, url, tools, env_headers(record))
-    end
-
-    private
-
-    # `headers` (an http/sse instance's credentials — `env` reads
-    # as `headers` for these transports via McpStore's migration-on-read, see
-    # McpStore#migrate_legacy_headers) -> literal HTTP headers (header-NAME ->
-    # value; the value carries its own scheme, e.g. "Bearer xxx" — no magic
-    # prefixing). NOT `secret_headers`: ToolManifest's own rule for a secret
-    # header requires a `{{secret.*}}` reference, which is the opposite of what
-    # a literal credential is — so an MCP instance's headers are NOT masked in
-    # a tool's edit page in /studio/tools, unlike a hand-authored data-tool's
-    # Authorization header. Acceptable for now (Studio is already
-    # operator-gated); a real fix routes this through the deployment's own
-    # {{secret.*}}/{{env.*}} resolution instead — later work.
-    def env_headers(record)
-      (record["headers"] || {}).each_with_object({}) { |(k, v), acc| acc[k.to_s] = v.to_s }
-    end
-
-    def build_manifest(name, url, tools, headers)
-      {
-        "version" => 1,
-        "defaults" => {
-          "method" => "POST",
-          "headers" => { "Content-Type" => "application/json" }.merge(headers),
-          "response" => { "extract" => "body_raw" },
-          "group" => "mcp:#{name}"
-        },
-        "tools" => tools.map { |raw| tool_entry(name, url, stringify(raw)) }
-      }
-    end
-
-    # Raw MCP entry -> manifest entry (MCP `inputSchema` envelope + JSON-RPC
-    # binding). ToolManifest normalizes the `inputSchema` (MCP adapter) and
-    # inherits the defaults; `group` falls through from the defaults.
-    def tool_entry(name, url, raw)
-      tool_name = raw["name"]
-      input_schema = raw["inputSchema"] || {}
-      {
-        "name" => tool_name,
-        "description" => presence(raw["description"]) || "Tool '#{tool_name}' from MCP server '#{name}'.",
-        "inputSchema" => input_schema,
-        "url" => url,
-        "side_effect" => true, # a tools/call is a side effect (checkpoint/skip-on-resume)
-        "body" => jsonrpc_call_body(tool_name, input_schema)
-      }
-    end
-
-    # JSON-RPC 2.0 `tools/call` body. `name` literal (resolved at ingestion);
-    # `arguments` per TOP-level property of the inputSchema, with `{{param}}` that
-    # the DataDefinedTool interpolates at TURN time.
-    def jsonrpc_call_body(tool_name, input_schema)
-      %({"jsonrpc":"2.0","id":1,"method":"tools/call",) +
-        %("params":{"name":#{JSON.generate(tool_name)},"arguments":#{arguments_fragment(input_schema)}}})
-    end
-
-    # -> "{...}" JSON with one placeholder per top-level property. Strings are
-    # quoted (the DataDefinedTool :body encode returns the escaped content WITHOUT
-    # quotes); other types raw (the encode returns value.to_json).
-    def arguments_fragment(input_schema)
-      props = (input_schema["properties"] || input_schema[:properties] || {})
-      return "{}" if props.nil? || props.empty?
-
-      pairs = props.map do |key, spec|
-        type = stringify(spec)["type"].to_s
-        placeholder = type == "string" ? %("{{#{key}}}") : "{{#{key}}}"
-        %(#{JSON.generate(key.to_s)}:#{placeholder})
-      end
-      "{#{pairs.join(',')}}"
-    end
-
-    def default_client(record)
-      Insika::McpHttpClient.new(url: record["url"], headers: env_headers(record))
-    end
-
-    def presence(str) = Insika::Coercion.presence(str)
-
-    def stringify(obj)
-      case obj
-      when Hash then obj.each_with_object({}) { |(k, v), acc| acc[k.to_s] = stringify(v) }
-      when Array then obj.map { |v| stringify(v) }
-      else obj
-      end
-    end
-  end
-end