From 349c2bbfed397445d929aa179d23386e3a77c8e4 Mon Sep 17 00:00:00 2001
From: Joe Rafaniello <jrafanie@gmail.com>
Date: Wed, 6 May 2026 11:39:04 -0400
Subject: [PATCH] WIP - Basic CRUD support for configuration script source
 payloads

This should start to enable service composition UI by allowing the json payload
to be persisted.

CP4AIOPS-11000
---
 ...onfiguration_script_payloads_controller.rb | 23 +++++++++++++++++++
 ...configuration_script_sources_controller.rb | 21 +++++++++++++++--
 .../configuration_script_payloads.rb          |  7 ++++++
 config/api.yml                                |  5 ++++
 4 files changed, 54 insertions(+), 2 deletions(-)

diff --git a/app/controllers/api/configuration_script_payloads_controller.rb b/app/controllers/api/configuration_script_payloads_controller.rb
index a11c11f40e..62e7c67f76 100644
--- a/app/controllers/api/configuration_script_payloads_controller.rb
+++ b/app/controllers/api/configuration_script_payloads_controller.rb
@@ -14,6 +14,29 @@ def api_resource_action_options
       %w[include_encrypted_attributes]
     end
 
+    def create_resource(type, _id, data)
+      # Get source_id and manager_id from configuration_script_source if provided
+      source_id = parse_id(data.delete('configuration_script_source'), :configuration_script_sources)
+      manager_id = parse_id(data.delete('manager_resource'), :providers) || data.delete('ems_id')
+
+      raise BadRequestError, _("Must specify name") unless data['name']
+      raise BadRequestError, _("Must specify payload") unless data['payload']
+      raise BadRequestError, _("Must specify manager_resource or configuration_script_source") unless manager_id || source_id
+
+      # Get manager_id from source if not already provided
+      if source_id && !manager_id
+        source = resource_search(source_id, :configuration_script_sources)
+        manager_id = source.manager_id
+      end
+
+      payload_attrs = data.slice('name', 'description', 'payload', 'payload_type').deep_symbolize_keys
+      payload_attrs[:manager_id] = manager_id
+      payload_attrs[:configuration_script_source_id] = source_id if source_id
+      payload_attrs[:payload_type] ||= 'json'
+
+      ConfigurationScriptPayload.create!(payload_attrs)
+    end
+
     def edit_resource(type, id, data)
       resource = resource_search(id, type)
 
diff --git a/app/controllers/api/configuration_script_sources_controller.rb b/app/controllers/api/configuration_script_sources_controller.rb
index 8bb487319d..a9bd8d860b 100644
--- a/app/controllers/api/configuration_script_sources_controller.rb
+++ b/app/controllers/api/configuration_script_sources_controller.rb
@@ -20,8 +20,25 @@ def create_resource(type, _id, data)
       # Since we are passing a custom hash to create_ems_resource (instead of data variable)
       # we need to manually remove it from data.
       data.delete('id')
-      create_ems_resource(type, {'ems_id' => manager_id, 'name' => data['name']}, :supports => true) do |_manager, klass|
-        {:task_id => klass.create_in_provider_queue(manager_id, data.deep_symbolize_keys)}
+
+      # Handle manual scm_type - create directly without provider queue
+      if data['scm_type'] == 'manual'
+        raise BadRequestError, _("Must specify name") unless data['name']
+        raise BadRequestError, _("Must specify manager_resource or ems_id") unless manager_id
+
+        # Create the script source directly
+        script_source = ConfigurationScriptSource.create!(
+          :name        => data['name'],
+          :description => data['description'],
+          :manager_id  => manager_id,
+          :scm_type    => 'manual'
+        )
+        script_source
+      else
+        # Use provider queue for git and other scm_types
+        create_ems_resource(type, {'ems_id' => manager_id, 'name' => data['name']}, :supports => true) do |_manager, klass|
+          {:task_id => klass.create_in_provider_queue(manager_id, data.deep_symbolize_keys)}
+        end
       end
     end
 
diff --git a/app/controllers/api/subcollections/configuration_script_payloads.rb b/app/controllers/api/subcollections/configuration_script_payloads.rb
index 4fc2a524e0..fae77ee7b0 100644
--- a/app/controllers/api/subcollections/configuration_script_payloads.rb
+++ b/app/controllers/api/subcollections/configuration_script_payloads.rb
@@ -4,6 +4,13 @@ module ConfigurationScriptPayloads
       def configuration_script_payloads_query_resource(object)
         object.configuration_script_payloads
       end
+
+      def create_resource_configuration_script_payloads(parent, _type, _id, data)
+        # When creating as a subcollection, automatically associate with parent script source
+        data['configuration_script_source'] = {"id" => parent.id.to_s}
+        data['manager_resource'] = {"id" => parent.manager_id.to_s} if parent.manager_id
+        create_resource(:configuration_script_payloads, nil, data)
+      end
     end
   end
 end
diff --git a/config/api.yml b/config/api.yml
index a5ba4384c6..6241e864fd 100644
--- a/config/api.yml
+++ b/config/api.yml
@@ -1000,6 +1000,8 @@
       - :name: read
         :identifier: embedded_configuration_script_payload_view
       :post:
+      - :name: create
+        :identifier: embedded_configuration_script_payload_add
       - :name: edit
         :identifier: embedded_configuration_script_payload_edit
     :resource_actions:
@@ -1013,6 +1015,9 @@
       :get:
       - :name: read
         :identifier: embedded_configuration_script_payload_view
+      :post:
+      - :name: create
+        :identifier: embedded_configuration_script_payload_add
     :authentications_subcollection_actions:
       :get:
       - :name: read