actionagent 1.2.0 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82a881aeb6aa97004b573137a8afdd5d61ac5d60f8d8cc1b28ca16e3712ec4c1
-  data.tar.gz: ccaba27a88b3297711515bfd8714e4449a3715c388af11874bfc60a63eaf3228
+  metadata.gz: f6e31cd17a84e6621881630137c3478268d2e31c8bba138516a397c23964c4f9
+  data.tar.gz: 28e83e7b84704c5ff9de2b475998273ac3dd328929515472f31e3743981927de
 SHA512:
-  metadata.gz: fef536ffb278c2d0a9b93dbf52a689de927b3b0c9dfd572ed68c0e3fac5a06bf661bd130151094c45c9c722ef093d641c8bbbd3b7d7c1c84b789ea4b88071803
-  data.tar.gz: abfbab6a0e0251233da133a0ce00572997ecd5ee8b3db952e6a854cb1b9b8f94e4182ac9976712b4457cc38d70b02cb4797b5b0b60ec61c2fa32100b631728fe
+  metadata.gz: 3a9e49dce9507b1243dd494b842c967bb5cb04921a7825ae2e16e37ed3cf25e845ae5b86de26b50cacdb01e7120859624ea7667d05c0d04048489bc7af5b534e
+  data.tar.gz: bebd359520c4ea07d69288ef8381afd5e5f663ea6df3264e17db81c2bc1a6d78836be216a8677666ede6223a1069f39e3e8e3a709025adcf8c9817a2be9750c9

data/app/models/action_agent/telemetry_trace.rb

@@ -135,7 +135,7 @@ module ActionAgent
       # Add account if in multi-tenant mode
       attrs[:account] = account if ActionAgent.multi_tenant? && account
 
-      create!(attrs)
+      create!(attrs).tap { |record| AgentRegistrar.call(record) }
     end
 
     # Sums a span's token counts (used to decide which spans carry the

data/app/services/action_agent/agent_execution_service.rb

@@ -354,8 +354,14 @@ module ActionAgent
         define_singleton_method(:name) { klass_name }
 
         # Persist the conversation (agent_contexts / agent_messages /
-        # agent_generations) via solid_agent. contextable: false — the context
-        # is loaded explicitly in the action below.
+        # agent_generations) via solid_agent. Auto-context is switched off —
+        # the context is loaded explicitly in the action below.
+        #
+        # The keyword that switches it off was renamed (contextable: ->
+        # contextual:) between solid_agent 0.1 and 0.2, and the gemspec floor
+        # admits both, so it is resolved from the installed method rather than
+        # hard-coded: passing the wrong one is an ArgumentError that only
+        # surfaces when a run executes.
         #
         # The model classes are named explicitly because solid_agent infers
         # bare "AgentContext"/"AgentMessage"/"AgentGeneration" and resolves
@@ -363,10 +369,12 @@ module ActionAgent
         # inferred names only resolve in a host app that happens to have
         # top-level models of its own.
         include SolidAgent::HasContext
-        has_context contextable: false,
+        has_context(
+          ActionAgent.solid_agent_auto_context_keyword => false,
           class_name: "ActionAgent::AgentContext",
           message_class: "ActionAgent::AgentMessage",
           generation_class: "ActionAgent::AgentGeneration"
+        )
 
         if effective_provider == :mock
           # Test environment only (see #provider_available?).

data/app/services/action_agent/agent_toolbox.rb

@@ -399,7 +399,7 @@ module ActionAgent
         if defined?(SolidAgent::ToolCache)
           SolidAgent::ToolCache.fetch(tool: name.to_s, args: kwargs, ttl: CACHE_TTL, &block)
         else
-          key = "solid_agent:tool_cache:#{name}:#{Digest::SHA256.hexdigest(kwargs.sort.to_h.to_json)}"
+          key = fallback_cache_key(name, kwargs)
           cached = Rails.cache.read(key)
           return cached.merge(cached: true) unless cached.nil?
 
@@ -411,6 +411,29 @@ module ActionAgent
         end
       end
 
+      # Byte-for-byte the key SolidAgent::ToolCache would compute, so an app
+      # that upgrades solid_agent mid-TTL keeps reading what it already
+      # cached instead of silently starting over. Nested hashes and
+      # symbol/string keys have to normalize the same way, which a plain
+      # `kwargs.sort.to_h.to_json` does not do.
+      #
+      # test/integration/solid_agent/tool_cache_test.rb asserts the two
+      # schemes still agree.
+      def fallback_cache_key(name, kwargs)
+        "solid_agent:tool_cache:#{name}:#{Digest::SHA256.hexdigest(normalize_cache_args(kwargs).to_json)}"
+      end
+
+      def normalize_cache_args(args)
+        case args
+        when Hash
+          args.map { |key, value| [ key.to_s, normalize_cache_args(value) ] }.sort_by(&:first)
+        when Array
+          args.map { |value| normalize_cache_args(value) }
+        else
+          args
+        end
+      end
+
       # SSRF guard for fetch_url: reject hosts that resolve to loopback,
       # private, or link-local addresses.
       def public_host?(host)

data/lib/action_agent/engine.rb

@@ -13,6 +13,13 @@ module ActionAgent
 
     engine_name "action_agent"
 
+    # Basenames this engine spells differently from Zeitwerk's default
+    # camelization, consulted by the inflections initializer below. Empty
+    # today: every file here is named for the constant default camelization
+    # produces. An engine file that wants a genuine acronym in its constant
+    # adds its basename here rather than relying on the host to register one.
+    INFLECTION_OVERRIDES = {}.freeze
+
     config.action_agent = ActiveSupport::OrderedOptions.new
 
     # The dashboard's JS and CSS ship prebuilt in the gem. Adding the
@@ -26,6 +33,75 @@ module ActionAgent
       app.config.filter_parameters += [ :credential, :api_key, :access_token ]
     end
 
+    # This engine's constants are spelled the way Zeitwerk's own inflector
+    # spells them — Api, McpCatalog, ApiKey — but an engine's files are
+    # autoloaded by the host's `rails.main` loader, under the *host's*
+    # inflections. A host that declares `inflect.acronym "API"` or "MCP" (both
+    # common, and documented by Rails) makes Zeitwerk expect
+    # ActionAgent::API::TracesController or ActionAgent::MCPCatalog from files
+    # that define ActionAgent::Api::TracesController and
+    # ActionAgent::McpCatalog. The constant never resolves and the request
+    # raises Zeitwerk::NameError.
+    #
+    # Every path under this engine therefore camelizes with Zeitwerk's default
+    # rules, ignoring whatever acronyms the host has registered. Applied by
+    # path rather than through `inflect`: the loader is shared with the host,
+    # so a blanket rule would re-spell the host's own constants. `camelize`
+    # receives the absolute path, which is the only hook that can tell this
+    # engine's files from the host's.
+    #
+    # Basenames whose spelling this engine cannot express through default
+    # camelization (a genuine acronym it wants uppercased) go in OVERRIDES.
+    initializer "action_agent.inflections", before: :set_autoload_paths do
+      engine_root = File.join(root.to_s, "")
+      default = Zeitwerk::Inflector.new
+
+      Rails.autoloaders.main.inflector.singleton_class.prepend(Module.new do
+        define_method(:camelize) do |basename, abspath|
+          next super(basename, abspath) unless abspath.to_s.start_with?(engine_root)
+
+          ActionAgent::Engine::INFLECTION_OVERRIDES.fetch(basename) { default.camelize(basename, abspath) }
+        end
+      end)
+    end
+
+    # Rails resolves a route's controller by camelizing the stored path
+    # ("action_agent/api/traces") with the host's *global* inflections, and no
+    # engine-level setting scopes that — so an acronym host looks up
+    # ActionAgent::API::TracesController and raises NameError even though the
+    # autoloader named the module correctly above. No single spelling satisfies
+    # both kinds of host: a plain host camelizes to Api, an acronym host to API.
+    # So the namespace answers to both. `const_missing` rather than an eager
+    # alias because the controllers are autoloaded on demand, and naming them at
+    # boot would load the whole dashboard.
+    # The router does not consult the autoloader's inflector, so an acronym
+    # host asks for ActionAgent::API::MCPServersController while the constants
+    # are Api::McpServersController. Rather than enumerate the pairs, an
+    # all-caps run in a missing constant is retried in the spelling default
+    # camelization produces: API -> Api, MCPServersController -> McpServers-
+    # Controller. Only the engine's own namespaces are touched, and only for a
+    # constant that is already missing.
+    inflection_shim = Module.new do
+      def const_missing(name)
+        relaxed = name.to_s.gsub(/([A-Z])([A-Z]+)(?=[A-Z][a-z]|\d|\z)/) { "#{$1}#{$2.downcase}" }
+
+        return super if relaxed == name.to_s || !const_defined?(relaxed, false)
+
+        const_get(relaxed, false)
+      end
+    end
+
+    ActionAgent.singleton_class.prepend(inflection_shim)
+
+    # ActionAgent::Api is autoloaded, so it cannot be reopened at this point.
+    # Zeitwerk hands it over as soon as it is defined, which is before the
+    # router can ask it for a controller.
+    initializer "action_agent.api_inflection_shim" do
+      Rails.autoloaders.main.on_load("ActionAgent::Api") do |mod, _abspath|
+        mod.singleton_class.prepend(inflection_shim)
+      end
+    end
+
     initializer "action_agent.assets", before: :append_assets_path do |app|
       builds = root.join("app", "assets", "builds").to_s
       next unless File.directory?(builds)

data/lib/action_agent/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActionAgent
-  VERSION = "1.2.0"
+  VERSION = "1.2.2"
 end

data/lib/action_agent.rb

@@ -20,6 +20,25 @@ module ActionAgent
       global = defined?(::ActiveRecord::Base) ? ::ActiveRecord::Base.table_name_prefix : ""
       "#{global}#{@table_name_prefix ||= "active_agent_"}"
     end
+
+    # Which keyword the installed solid_agent uses to switch has_context's
+    # auto-context off: `contextable:` up to 0.1, `contextual:` from 0.2. The
+    # gemspec floor admits both, and passing the wrong one raises an
+    # ArgumentError deep inside a run rather than at boot — so
+    # AgentExecutionService asks rather than assumes.
+    #
+    # Covered by test/integration/solid_agent, which runs this engine against
+    # solid_agent's main branch as well as the released gem.
+    def solid_agent_auto_context_keyword
+      @solid_agent_auto_context_keyword ||= begin
+        keywords = ::SolidAgent::HasContext::ClassMethods
+          .instance_method(:has_context).parameters
+          .select { |type, _| [ :key, :keyreq ].include?(type) }
+          .map(&:last)
+
+        keywords.include?(:contextual) ? :contextual : :contextable
+      end
+    end
   end
 end
 

data/lib/generators/action_agent/templates/create_active_agent_dashboard_tables.rb.erb

@@ -26,13 +26,13 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :agent_class_name
       t.string :action_name
       t.integer :status, default: 0, null: false
-      t.column :appearance, json_type, default: {}
-      t.column :instruction_sets, json_type, default: []
-      t.column :tools, json_type, default: []
-      t.column :mcp_servers, json_type, default: []
-      t.column :model_config, json_type, default: {}
-      t.column :response_format, json_type, default: {}
-      t.column :action_prompts, json_type, default: [], null: false
+      t.column :appearance, json_type, **json_default({})
+      t.column :instruction_sets, json_type, **json_default([])
+      t.column :tools, json_type, **json_default([])
+      t.column :mcp_servers, json_type, **json_default([])
+      t.column :model_config, json_type, **json_default({})
+      t.column :response_format, json_type, **json_default({})
+      t.column :action_prompts, json_type, **json_default([], null: false)
 
       # Agents discovered from reported telemetry rather than authored here.
       t.string :service_name
@@ -52,7 +52,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
     create_table "#{prefix}agent_versions" do |t|
       t.bigint :agent_id, null: false
       t.integer :version_number, default: 1, null: false
-      t.column :configuration_snapshot, json_type, default: {}, null: false
+      t.column :configuration_snapshot, json_type, **json_default({}, null: false)
       t.string :change_summary
       t.string :created_by
       t.timestamps
@@ -65,10 +65,10 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :action_name
       t.integer :status, default: 0, null: false
       t.text :input_prompt
-      t.column :input_params, json_type, default: {}
+      t.column :input_params, json_type, **json_default({})
       t.text :output
-      t.column :output_metadata, json_type, default: {}
-      t.column :logs, json_type, default: []
+      t.column :output_metadata, json_type, **json_default({})
+      t.column :logs, json_type, **json_default([])
       t.text :error_message
       t.text :error_backtrace
       t.integer :input_tokens
@@ -93,11 +93,11 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :model, default: "gpt-4o-mini"
       t.text :instructions
       t.string :preset_type
-      t.column :appearance, json_type, default: {}
-      t.column :instruction_sets, json_type, default: []
-      t.column :tools, json_type, default: []
-      t.column :mcp_servers, json_type, default: {}
-      t.column :model_config, json_type, default: {}
+      t.column :appearance, json_type, **json_default({})
+      t.column :instruction_sets, json_type, **json_default([])
+      t.column :tools, json_type, **json_default([])
+      t.column :mcp_servers, json_type, **json_default({})
+      t.column :model_config, json_type, **json_default({})
       t.boolean :featured, default: false
       t.boolean :free_tier, default: false
       t.boolean :public, default: true
@@ -114,7 +114,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :contextable_type
       t.bigint :contextable_id
       t.text :instructions
-      t.column :options, json_type, default: {}
+      t.column :options, json_type, **json_default({})
       t.string :trace_id
       t.integer :total_input_tokens, default: 0
       t.integer :total_output_tokens, default: 0
@@ -130,11 +130,11 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :content_checksum
       t.string :tool_call_id
       t.string :tool_name
-      t.column :tool_arguments, json_type, default: {}
+      t.column :tool_arguments, json_type, **json_default({})
       t.column :tool_result, json_type
-      t.column :attachments, json_type, default: []
-      t.column :metadata, json_type, default: {}
-      t.column :provenance, json_type, default: {}
+      t.column :attachments, json_type, **json_default([])
+      t.column :metadata, json_type, **json_default({})
+      t.column :provenance, json_type, **json_default({})
       t.timestamps
       t.index :agent_context_id
     end
@@ -150,9 +150,9 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.integer :cached_tokens, default: 0
       t.integer :reasoning_tokens, default: 0
       t.float :duration_seconds
-      t.column :tool_calls, json_type, default: []
+      t.column :tool_calls, json_type, **json_default([])
       t.column :raw_response, json_type
-      t.column :provenance, json_type, default: {}
+      t.column :provenance, json_type, **json_default({})
       t.string :trace_id
       t.timestamps
       t.index :agent_context_id
@@ -183,8 +183,8 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :judge_kind, default: "rules", null: false
       t.string :judge_model
       t.integer :sample_size, default: 20, null: false
-      t.column :criteria, json_type, default: [], null: false
-      t.column :config, json_type, default: {}, null: false
+      t.column :criteria, json_type, **json_default([], null: false)
+      t.column :config, json_type, **json_default({}, null: false)
       t.timestamps
       t.index [ :agent_id, :name ], unique: true
     end
@@ -192,7 +192,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
     create_table "#{prefix}evaluation_runs" do |t|
       t.bigint :evaluation_id, null: false
       t.integer :status, default: 0, null: false
-      t.column :scores, json_type, default: {}
+      t.column :scores, json_type, **json_default({})
       t.integer :samples_evaluated, default: 0
       t.integer :samples_passed, default: 0
       t.text :error_message
@@ -208,11 +208,11 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.string :sandbox_type, default: "playwright_mcp"
       t.string :cloud_run_job_id
       t.string :cloud_run_url
-      t.column :runs, json_type, default: []
+      t.column :runs, json_type, **json_default([])
       # MCP servers this session was started with, so the MCP Services view
       # can show a launched server as running rather than offering to start
       # a second copy.
-      t.column :mcp_servers, json_type, default: []
+      t.column :mcp_servers, json_type, **json_default([])
       t.integer :runs_count, default: 0
       t.integer :max_runs, default: 10
       t.integer :timeout_seconds, default: 300
@@ -234,7 +234,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.integer :status, default: 0, null: false
       t.text :result
       t.text :error
-      t.column :screenshots, json_type, default: []
+      t.column :screenshots, json_type, **json_default([])
       t.integer :tokens_used, default: 0
       t.integer :duration_ms
       t.datetime :started_at
@@ -250,7 +250,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.integer :status, default: 0, null: false
       t.integer :action_count, default: 0
       t.integer :duration_ms
-      t.column :metadata, json_type, default: {}
+      t.column :metadata, json_type, **json_default({})
       t.bigint :user_id
       t.bigint :account_id
       t.timestamps
@@ -269,7 +269,7 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
       t.integer :timestamp_ms, null: false
       t.string :screenshot_key
       t.string :dom_snapshot_key
-      t.column :metadata, json_type, default: {}
+      t.column :metadata, json_type, **json_default({})
       t.timestamps
       t.index [ :session_recording_id, :sequence ], unique: true,
         name: "index_active_agent_recording_actions_on_recording_and_sequence"
@@ -314,6 +314,21 @@ class CreateActiveAgentDashboardTables < ActiveRecord::Migration<%= migration_ve
 
   # jsonb where the adapter has it, json where it doesn't.
   def json_type
-    @json_type ||= connection.adapter_name.to_s.downcase.include?("postgres") ? :jsonb : :json
+    @json_type ||= postgres? ? :jsonb : :json
+  end
+
+  # MySQL rejects a default on a JSON column outright ("BLOB, TEXT, GEOMETRY or
+  # JSON column can't have a default value"), which aborts create_table, so the
+  # column is created without one there. `null: false` is dropped with it:
+  # nothing assigns these before validation, so a NOT NULL column with no
+  # default would reject the very inserts the default was there to satisfy.
+  def json_default(value, null: nil)
+    return {} unless postgres?
+
+    null.nil? ? { default: value } : { default: value, null: null }
+  end
+
+  def postgres?
+    connection.adapter_name.to_s.downcase.include?("postgres")
   end
 end

data/lib/generators/action_agent/templates/create_active_agent_telemetry_traces.rb.erb

@@ -15,9 +15,9 @@ class CreateActiveAgentTelemetryTraces < ActiveRecord::Migration<%= migration_ve
       t.string :service_name
       t.string :environment
       t.datetime :timestamp, index: true
-      t.column :spans, json_type, default: []
-      t.column :resource_attributes, json_type, default: {}
-      t.column :sdk_info, json_type, default: {}
+      t.column :spans, json_type, **json_default([])
+      t.column :resource_attributes, json_type, **json_default({})
+      t.column :sdk_info, json_type, **json_default({})
       t.decimal :total_duration_ms, precision: 12, scale: 3
       t.integer :total_input_tokens, default: 0
       t.integer :total_output_tokens, default: 0
@@ -53,6 +53,19 @@ class CreateActiveAgentTelemetryTraces < ActiveRecord::Migration<%= migration_ve
   # traverses spans in SQL on PostgreSQL, and jsonb_array_elements rejects a
   # json argument, so the column type and the query have to agree.
   def json_type
-    @json_type ||= connection.adapter_name.to_s.downcase.include?("postgres") ? :jsonb : :json
+    @json_type ||= postgres? ? :jsonb : :json
+  end
+
+  # MySQL rejects a default on a JSON column outright ("BLOB, TEXT, GEOMETRY or
+  # JSON column can't have a default value"), which aborts create_table, so the
+  # column is created without one there. ActionAgent::TelemetryTrace reads every
+  # JSON column through `Array(...)`/`|| {}`, and ingest always assigns all
+  # three, so a NULL reads the same as the empty default would.
+  def json_default(value)
+    postgres? ? { default: value } : {}
+  end
+
+  def postgres?
+    connection.adapter_name.to_s.downcase.include?("postgres")
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: actionagent
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.2
 platform: ruby
 authors:
 - Justin Bowen