actionagent 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82a881aeb6aa97004b573137a8afdd5d61ac5d60f8d8cc1b28ca16e3712ec4c1
-  data.tar.gz: ccaba27a88b3297711515bfd8714e4449a3715c388af11874bfc60a63eaf3228
+  metadata.gz: 9d2d4c834fd9d4b7990cd9deb27061d5b56e25e1379ae66c1b7251c0c890b645
+  data.tar.gz: 84903be1227333260c6b7b90079c4a67f1d5fc60c07bafc6b28651b81b8804cf
 SHA512:
-  metadata.gz: fef536ffb278c2d0a9b93dbf52a689de927b3b0c9dfd572ed68c0e3fac5a06bf661bd130151094c45c9c722ef093d641c8bbbd3b7d7c1c84b789ea4b88071803
-  data.tar.gz: abfbab6a0e0251233da133a0ce00572997ecd5ee8b3db952e6a854cb1b9b8f94e4182ac9976712b4457cc38d70b02cb4797b5b0b60ec61c2fa32100b631728fe
+  metadata.gz: d11a547b4172f715efa0a16c3da0f548e06ff17ac2b7c7524b9a696d16a4f1248123ca9d2f859187862112b1b3fbfdafd73a556891eaf4bcd810492eb92be81f
+  data.tar.gz: bda14fd2e530157019b6b823581d08c42eb9896a4db726f1325b03b339e14f21b0e97cb003bc693e131440ed6a56d5ffca7f330ba147ddcf8d89a77994fedc85

data/lib/action_agent/engine.rb

@@ -26,6 +26,47 @@ module ActionAgent
       app.config.filter_parameters += [ :credential, :api_key, :access_token ]
     end
 
+    # The controllers under app/controllers/action_agent/api are ActionAgent::Api,
+    # 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"`
+    # — common enough that Rails documents it — makes Zeitwerk expect
+    # ActionAgent::API::TracesController in a file that defines
+    # ActionAgent::Api::TracesController, and every request to the mount raises
+    # Zeitwerk::NameError.
+    #
+    # Scoped to this engine's own path rather than set through `inflect`: the
+    # loader is shared, so a blanket rule would re-spell the host's own API
+    # constants. `camelize` receives the absolute path, which is the only hook
+    # that can tell this engine's api/ from the host's.
+    initializer "action_agent.inflections", before: :set_autoload_paths do
+      engine_root = root.to_s
+
+      Rails.autoloaders.main.inflector.singleton_class.prepend(Module.new do
+        define_method(:camelize) do |basename, abspath|
+          next "Api" if basename == "api" && abspath.to_s.start_with?(engine_root)
+
+          super(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.
+    ActionAgent.singleton_class.prepend(Module.new do
+      def const_missing(name)
+        return const_get(:Api) if name == :API
+
+        super
+      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.1"
 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.1
 platform: ruby
 authors:
 - Justin Bowen