activeadmin_mcp 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99312ac0bef360e80a8d058b79adf5f0af182d3bedc19fdfbf4caa69964f7e44
-  data.tar.gz: bba982b59472e271c18214bc4674e60a3ae96729b78172163d54eed756b3997c
+  metadata.gz: e336c562299eb5de7192243106f42dc18602589811fff5eb0282f12b605ea9b8
+  data.tar.gz: 006f6aac09bceb97605517a9b1b7346a4aef6fc2ddc4c59ab48b36b658825078
 SHA512:
-  metadata.gz: 861ea8e3d8eeff901f12f4fa99385db60fd0854a0eae22462cf314cce67820632033b8e55ebc13252bcaa3389f0b6aceda4cc712d5bceb4ba0c342ac10a5815d
-  data.tar.gz: 035b92456d5b3ca111c5e4715c8f495f52531ab053eb7974f1c1dda12de73d87f08ba34b40f226723d8a207bc98a89c929c4d6bb8ecdc9ee0873fdc6fbefb275
+  metadata.gz: 602940dbd9eb2a0337220672258f343aff84d0851eaec4b0199b38cb99f843adca48ee085ab201ed896ec29f260fe5e8ae6a8844b567990611e4c866d3abd05f
+  data.tar.gz: 54e2309b725add92b02af4ebbaefdafd5cfa8c349890211c22f0bb391f00863764e89cf50f7e5f6ace5f293962f473ad75f5197f7813702b939cfc2dc95d5b9d

data/lib/activeadmin_mcp/form_field_collector.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module ActiveadminMcp
+  # Records the field names declared by an ActiveAdmin `form do ... end` block.
+  #
+  # ActiveAdmin form blocks are arbitrary Formtastic DSL — `input`, `inputs`,
+  # `actions`, helper calls, conditionals — so the block is run against this
+  # stand-in form builder. Every `input :field` records `:field`; every other
+  # message (including unknown helpers) is swallowed and returns self, so the
+  # block executes without a real view context. Nested `has_many` associations
+  # are intentionally not descended into: the updater only writes flat
+  # attributes, and descending would record association fields as top-level.
+  class FormFieldCollector
+    attr_reader :fields
+
+    def initialize
+      @fields = []
+    end
+
+    def collect(&block)
+      instance_exec(self, &block)
+      @fields.uniq
+    end
+
+    def input(name, *_args, **_opts, &_block)
+      @fields << name.to_sym if name.respond_to?(:to_sym)
+      self
+    end
+
+    def inputs(*_args, **_opts, &block)
+      instance_exec(self, &block) if block
+      self
+    end
+
+    def has_many(*_args, **_opts)
+      self
+    end
+
+    def method_missing(_name, *_args, **_opts, &block)
+      instance_exec(self, &block) if block
+      self
+    end
+
+    def respond_to_missing?(_name, _include_private = false)
+      true
+    end
+  end
+end

data/lib/activeadmin_mcp/record_updater.rb

@@ -58,24 +58,47 @@ module ActiveadminMcp
       adapter_class.new(config, @current_user).authorized?(UPDATE, record)
     end
 
-    # Runs the incoming attributes through the resource controller's own
-    # permitted_params (compiled from `permit_params`), so we accept exactly what
-    # the admin form accepts. Fails closed if that cannot be resolved.
+    # Resolves the fields we may write, accepting exactly what the admin form
+    # accepts. Prefers the resource's own `permit_params` (via the controller's
+    # compiled permitted_params); when a resource declares its writable fields
+    # through a `form do ... end` block instead — as ActiveAdmin's default
+    # permitted_params then returns nil — derives them from the form inputs.
+    # Fails closed if neither can be resolved.
     def permitted_attributes(config, attributes)
+      from_permit_params(config, attributes) ||
+        from_form(config, attributes) ||
+        raise(PermitError, "Could not determine permitted attributes: #{@resource[:name]}")
+    end
+
+    def from_permit_params(config, attributes)
       param_key = config.param_key.to_sym
       controller = config.controller.new
-
-      unless controller.respond_to?(:permitted_params, true)
-        raise PermitError, "Resource does not declare permit_params: #{@resource[:name]}"
-      end
+      return nil unless controller.respond_to?(:permitted_params, true)
 
       controller.params = ActionController::Parameters.new(param_key => attributes)
-      permitted = controller.send(:permitted_params)[param_key]
-      permitted ? permitted.to_h.symbolize_keys : {}
-    rescue PermitError
-      raise
-    rescue StandardError => e
-      raise PermitError, "Could not determine permitted attributes: #{e.message}"
+      permitted = controller.send(:permitted_params)
+      scoped = permitted && permitted[param_key]
+      scoped ? scoped.to_h.symbolize_keys : nil
+    rescue StandardError
+      nil
+    end
+
+    def from_form(config, attributes)
+      fields = form_fields(config)
+      return nil if fields.empty?
+
+      ActionController::Parameters.new(attributes).permit(*fields).to_h.symbolize_keys
+    rescue StandardError
+      nil
+    end
+
+    def form_fields(config)
+      return [] unless config.respond_to?(:page_presenters)
+
+      block = config.page_presenters[:form]&.block
+      return [] unless block
+
+      FormFieldCollector.new.collect(&block)
     end
 
     def error(message, details: nil)

data/lib/activeadmin_mcp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveadminMcp
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/activeadmin_mcp.rb

@@ -3,6 +3,7 @@
 require_relative "activeadmin_mcp/version"
 require_relative "activeadmin_mcp/configuration"
 require_relative "activeadmin_mcp/resource_registry"
+require_relative "activeadmin_mcp/form_field_collector"
 require_relative "activeadmin_mcp/record_updater"
 require_relative "activeadmin_mcp/request_handler"
 require_relative "activeadmin_mcp/engine"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activeadmin_mcp
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - harunkumars
@@ -97,6 +97,7 @@ files:
 - lib/activeadmin_mcp.rb
 - lib/activeadmin_mcp/configuration.rb
 - lib/activeadmin_mcp/engine.rb
+- lib/activeadmin_mcp/form_field_collector.rb
 - lib/activeadmin_mcp/record_updater.rb
 - lib/activeadmin_mcp/request_handler.rb
 - lib/activeadmin_mcp/resource_registry.rb