pg_sql_triggers 1.1.1 → 1.2.0

data/app/models/pg_sql_triggers/trigger_registry.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module PgSqlTriggers
+  # rubocop:disable Metrics/ClassLength
   class TriggerRegistry < PgSqlTriggers::ApplicationRecord
     self.table_name = "pg_sql_triggers_registry"
 
@@ -150,6 +151,51 @@ module PgSqlTriggers
       end
     end
 
+    def drop!(reason:, confirmation: nil, actor: nil)
+      # Check kill switch before dropping trigger
+      PgSqlTriggers::SQL::KillSwitch.check!(
+        operation: :trigger_drop,
+        environment: Rails.env,
+        confirmation: confirmation,
+        actor: actor || { type: "Console", id: "TriggerRegistry#drop!" }
+      )
+
+      # Validate reason is provided
+      raise ArgumentError, "Reason is required" if reason.nil? || reason.to_s.strip.empty?
+
+      log_drop_attempt(reason)
+
+      # Execute DROP TRIGGER in transaction
+      ActiveRecord::Base.transaction do
+        drop_trigger_from_database
+        destroy!
+        log_drop_success
+      end
+    end
+
+    def re_execute!(reason:, confirmation: nil, actor: nil)
+      # Check kill switch before re-executing trigger
+      PgSqlTriggers::SQL::KillSwitch.check!(
+        operation: :trigger_re_execute,
+        environment: Rails.env,
+        confirmation: confirmation,
+        actor: actor || { type: "Console", id: "TriggerRegistry#re_execute!" }
+      )
+
+      # Validate reason is provided
+      raise ArgumentError, "Reason is required" if reason.nil? || reason.to_s.strip.empty?
+      raise StandardError, "Cannot re-execute: missing function_body" if function_body.blank?
+
+      log_re_execute_attempt(reason)
+
+      # Execute the trigger creation/update in transaction
+      ActiveRecord::Base.transaction do
+        drop_existing_trigger_for_re_execute
+        recreate_trigger
+        update_registry_after_re_execute
+      end
+    end
+
     private
 
     def quote_identifier(identifier)
@@ -170,5 +216,82 @@ module PgSqlTriggers
     def verify!
       update!(last_verified_at: Time.current)
     end
+
+    # Drop trigger helpers
+    def log_drop_attempt(reason)
+      return unless defined?(Rails.logger)
+
+      Rails.logger.info "[TRIGGER_DROP] Dropping: #{trigger_name} on #{table_name}"
+      Rails.logger.info "[TRIGGER_DROP] Reason: #{reason}"
+    end
+
+    def log_drop_success
+      return unless defined?(Rails.logger)
+
+      Rails.logger.info "[TRIGGER_DROP] Successfully removed from registry"
+    end
+
+    def drop_trigger_from_database
+      trigger_exists = check_trigger_exists
+      return unless trigger_exists
+
+      execute_drop_sql
+    end
+
+    def check_trigger_exists
+      introspection = PgSqlTriggers::DatabaseIntrospection.new
+      introspection.trigger_exists?(trigger_name)
+    rescue StandardError => e
+      Rails.logger.warn("Could not check trigger existence: #{e.message}") if defined?(Rails.logger)
+      false
+    end
+
+    def execute_drop_sql
+      quoted_table = quote_identifier(table_name)
+      quoted_trigger = quote_identifier(trigger_name)
+      sql = "DROP TRIGGER IF EXISTS #{quoted_trigger} ON #{quoted_table};"
+      ActiveRecord::Base.connection.execute(sql)
+      Rails.logger.info "[TRIGGER_DROP] Dropped from database" if defined?(Rails.logger)
+    rescue ActiveRecord::StatementInvalid, StandardError => e
+      Rails.logger.error("[TRIGGER_DROP] Failed: #{e.message}") if defined?(Rails.logger)
+      raise
+    end
+
+    # Re-execute trigger helpers
+    def log_re_execute_attempt(reason)
+      return unless defined?(Rails.logger)
+
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Re-executing: #{trigger_name} on #{table_name}"
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Reason: #{reason}"
+      drift = drift_result
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Current state: #{drift[:state]}"
+    end
+
+    def drop_existing_trigger_for_re_execute
+      introspection = PgSqlTriggers::DatabaseIntrospection.new
+      return unless introspection.trigger_exists?(trigger_name)
+
+      quoted_table = quote_identifier(table_name)
+      quoted_trigger = quote_identifier(trigger_name)
+      drop_sql = "DROP TRIGGER IF EXISTS #{quoted_trigger} ON #{quoted_table};"
+      ActiveRecord::Base.connection.execute(drop_sql)
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Dropped existing" if defined?(Rails.logger)
+    rescue StandardError => e
+      Rails.logger.warn("[TRIGGER_RE_EXECUTE] Drop failed: #{e.message}") if defined?(Rails.logger)
+    end
+
+    def recreate_trigger
+      ActiveRecord::Base.connection.execute(function_body)
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Re-created trigger" if defined?(Rails.logger)
+    rescue ActiveRecord::StatementInvalid, StandardError => e
+      Rails.logger.error("[TRIGGER_RE_EXECUTE] Failed: #{e.message}") if defined?(Rails.logger)
+      raise
+    end
+
+    def update_registry_after_re_execute
+      update!(enabled: true, last_executed_at: Time.current)
+      Rails.logger.info "[TRIGGER_RE_EXECUTE] Updated registry" if defined?(Rails.logger)
+    end
   end
+  # rubocop:enable Metrics/ClassLength
 end

data/app/views/pg_sql_triggers/dashboard/index.html.erb

@@ -46,7 +46,9 @@
     <tbody>
       <% @triggers.limit(10).each do |trigger| %>
         <tr>
-          <td><strong><%= trigger.trigger_name %></strong></td>
+          <td>
+            <%= link_to trigger.trigger_name, trigger_path(trigger), style: "color: #007bff; text-decoration: none; font-weight: 600;" %>
+          </td>
           <td><%= trigger.table_name %></td>
           <td><%= trigger.version %></td>
           <td>
@@ -57,7 +59,43 @@
             <% end %>
           </td>
           <td><span class="badge badge-info"><%= trigger.source %></span></td>
-          <td>—</td>
+          <td>
+            <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
+              <div style="display: flex; gap: 0.5rem; flex-wrap: wrap;">
+                <% if trigger.enabled %>
+                  <% form_id = "trigger-disable-#{trigger.id}-form" %>
+                  <%= form_with url: disable_trigger_path(trigger), method: :post, local: true, id: form_id, style: "margin: 0;" do |f| %>
+                    <%= f.hidden_field :redirect_to, value: dashboard_path %>
+                    <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+                        style="padding: 0.25rem 0.75rem; background: #dc3545; color: white; border: none; border-radius: 3px; cursor: pointer; font-size: 0.75rem;">
+                      Disable
+                    </button>
+                  <% end %>
+                  <%= render 'pg_sql_triggers/shared/confirmation_modal',
+                             operation: :ui_trigger_disable,
+                             form_id: form_id,
+                             title: 'Disable Trigger',
+                             message: "Are you sure you want to disable trigger '#{trigger.trigger_name}'?" %>
+                <% else %>
+                  <% form_id = "trigger-enable-#{trigger.id}-form" %>
+                  <%= form_with url: enable_trigger_path(trigger), method: :post, local: true, id: form_id, style: "margin: 0;" do |f| %>
+                    <%= f.hidden_field :redirect_to, value: dashboard_path %>
+                    <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+                        style="padding: 0.25rem 0.75rem; background: #28a745; color: white; border: none; border-radius: 3px; cursor: pointer; font-size: 0.75rem;">
+                      Enable
+                    </button>
+                  <% end %>
+                  <%= render 'pg_sql_triggers/shared/confirmation_modal',
+                             operation: :ui_trigger_enable,
+                             form_id: form_id,
+                             title: 'Enable Trigger',
+                             message: "Are you sure you want to enable trigger '#{trigger.trigger_name}'?" %>
+                <% end %>
+              </div>
+            <% else %>
+              —
+            <% end %>
+          </td>
         </tr>
       <% end %>
     </tbody>

data/app/views/pg_sql_triggers/sql_capsules/new.html.erb

@@ -0,0 +1,81 @@
+<div style="max-width: 900px; margin: 0 auto;">
+  <div style="display: flex; justify-content: space-between; align-items: center; margin-bottom: 2rem;">
+    <h2 style="margin: 0;">Create SQL Capsule</h2>
+    <%= link_to "← Back to Dashboard", dashboard_path, style: "color: #007bff; text-decoration: none;" %>
+  </div>
+
+  <div style="background: #fff3cd; border: 1px solid #ffc107; padding: 1rem; border-radius: 4px; margin-bottom: 2rem;">
+    <strong>⚠️ Warning:</strong> SQL Capsules allow direct SQL execution. Use with extreme caution.
+    This feature is intended for emergency operations only.
+  </div>
+
+  <%= form_with url: sql_capsules_path, method: :post, local: true do |f| %>
+    <div style="background: white; padding: 2rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1);">
+
+      <div style="margin-bottom: 1.5rem;">
+        <label for="name" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+          Capsule Name <span style="color: #dc3545;">*</span>
+        </label>
+        <%= text_field_tag :name, @capsule_name,
+            placeholder: "e.g., fix_user_permissions",
+            required: true,
+            pattern: "[a-zA-Z0-9_-]+",
+            autocomplete: "off",
+            style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" %>
+        <small style="color: #6c757d; display: block; margin-top: 0.25rem;">
+          Letters, numbers, underscores, and hyphens only
+        </small>
+      </div>
+
+      <div style="margin-bottom: 1.5rem;">
+        <label for="environment" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+          Environment <span style="color: #dc3545;">*</span>
+        </label>
+        <%= text_field_tag :environment, @environment,
+            placeholder: "e.g., production",
+            required: true,
+            autocomplete: "off",
+            style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" %>
+        <small style="color: #6c757d; display: block; margin-top: 0.25rem;">
+          The environment this capsule is intended for
+        </small>
+      </div>
+
+      <div style="margin-bottom: 1.5rem;">
+        <label for="purpose" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+          Purpose <span style="color: #dc3545;">*</span>
+        </label>
+        <%= text_area_tag :purpose, @purpose,
+            placeholder: "Describe what this SQL does and why it's needed...",
+            required: true,
+            rows: 3,
+            style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px; font-family: inherit;" %>
+        <small style="color: #6c757d; display: block; margin-top: 0.25rem;">
+          Detailed description of the operation and its purpose
+        </small>
+      </div>
+
+      <div style="margin-bottom: 1.5rem;">
+        <label for="sql" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+          SQL Statement <span style="color: #dc3545;">*</span>
+        </label>
+        <%= text_area_tag :sql, @sql,
+            placeholder: "-- Enter your SQL statement here\nSELECT * FROM users WHERE ...",
+            required: true,
+            rows: 12,
+            style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px; font-family: 'Monaco', 'Menlo', 'Courier New', monospace; font-size: 0.9rem;" %>
+        <small style="color: #6c757d; display: block; margin-top: 0.25rem;">
+          The SQL to execute. Review carefully before saving.
+        </small>
+      </div>
+
+      <div style="display: flex; gap: 1rem; justify-content: flex-end;">
+        <%= link_to "Cancel", dashboard_path,
+            style: "padding: 0.75rem 1.5rem; background: #6c757d; color: white; text-decoration: none; border-radius: 4px;" %>
+        <%= button_tag "Create Capsule",
+            type: "submit",
+            style: "padding: 0.75rem 1.5rem; background: #007bff; color: white; border: none; border-radius: 4px; cursor: pointer; font-size: 1rem;" %>
+      </div>
+    </div>
+  <% end %>
+</div>

data/app/views/pg_sql_triggers/sql_capsules/show.html.erb

@@ -0,0 +1,85 @@
+<div style="max-width: 900px; margin: 0 auto;">
+  <div style="display: flex; justify-content: space-between; align-items: center; margin-bottom: 2rem;">
+    <h2 style="margin: 0;">SQL Capsule: <%= @capsule.name %></h2>
+    <%= link_to "← Back to Dashboard", dashboard_path, style: "color: #007bff; text-decoration: none;" %>
+  </div>
+
+  <div style="background: white; padding: 2rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+    <h3 style="margin-top: 0; margin-bottom: 1rem; color: #495057;">Capsule Details</h3>
+
+    <div style="display: grid; grid-template-columns: 200px 1fr; gap: 1rem; margin-bottom: 1rem;">
+      <div style="font-weight: 600; color: #6c757d;">Name:</div>
+      <div><%= @capsule.name %></div>
+
+      <div style="font-weight: 600; color: #6c757d;">Environment:</div>
+      <div><span class="badge badge-info"><%= @capsule.environment %></span></div>
+
+      <div style="font-weight: 600; color: #6c757d;">Created:</div>
+      <div><%= @capsule.created_at.strftime("%Y-%m-%d %H:%M:%S %Z") %></div>
+
+      <div style="font-weight: 600; color: #6c757d;">Checksum:</div>
+      <div style="font-family: 'Monaco', 'Menlo', 'Courier New', monospace; font-size: 0.85rem; color: #6c757d;">
+        <%= @checksum %>
+      </div>
+    </div>
+
+    <div style="margin-top: 1.5rem;">
+      <div style="font-weight: 600; color: #6c757d; margin-bottom: 0.5rem;">Purpose:</div>
+      <div style="background: #f8f9fa; padding: 1rem; border-radius: 4px; border-left: 3px solid #007bff;">
+        <%= @capsule.purpose %>
+      </div>
+    </div>
+
+    <div style="margin-top: 1.5rem;">
+      <div style="font-weight: 600; color: #6c757d; margin-bottom: 0.5rem;">SQL Statement:</div>
+      <div style="background: #f8f9fa; padding: 1rem; border-radius: 4px; border-left: 3px solid #28a745; overflow-x: auto;">
+        <pre style="margin: 0; font-family: 'Monaco', 'Menlo', 'Courier New', monospace; font-size: 0.9rem; white-space: pre-wrap; word-wrap: break-word;"><%= @capsule.sql %></pre>
+      </div>
+    </div>
+  </div>
+
+  <% if @can_execute %>
+    <div style="background: #fff3cd; border: 1px solid #ffc107; padding: 1.5rem; border-radius: 4px; margin-bottom: 2rem;">
+      <h4 style="margin-top: 0; color: #856404;">⚠️ Execution Warning</h4>
+      <p style="margin-bottom: 1rem; color: #856404;">
+        This will execute the SQL statement directly against the database.
+        This operation cannot be undone. Make sure you understand the implications.
+      </p>
+
+      <% if kill_switch_active? %>
+        <p style="margin-bottom: 1rem; color: #856404;">
+          <strong>Kill Switch is ACTIVE for <%= current_environment %> environment.</strong><br>
+          You must provide confirmation text to proceed.
+        </p>
+      <% end %>
+
+      <% form_id = "execute-capsule-form" %>
+      <%= form_with url: execute_sql_capsule_path(@capsule.name), method: :post, local: true, id: form_id do |f| %>
+        <div style="display: flex; gap: 1rem; align-items: flex-end;">
+          <div style="flex: 1;">
+            <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+                style="width: 100%; padding: 0.75rem 1.5rem; background: #dc3545; color: white; border: none; border-radius: 4px; cursor: pointer; font-size: 1rem; font-weight: 600;">
+              Execute SQL Capsule
+            </button>
+          </div>
+        </div>
+      <% end %>
+
+      <%= render 'pg_sql_triggers/shared/confirmation_modal',
+                 operation: :execute_sql_capsule,
+                 form_id: form_id,
+                 title: 'Execute SQL Capsule',
+                 message: "Are you sure you want to execute SQL capsule '#{@capsule.name}'? This will run the SQL statement directly against the database." %>
+    </div>
+  <% else %>
+    <div style="background: #f8d7da; border: 1px solid #f5c6cb; padding: 1rem; border-radius: 4px; color: #721c24;">
+      <strong>Insufficient Permissions:</strong> You need Admin role to execute SQL capsules.
+    </div>
+  <% end %>
+
+  <div style="margin-top: 2rem;">
+    <%= link_to "Create Another Capsule", new_sql_capsule_path,
+        class: "btn btn-primary",
+        style: "padding: 0.75rem 1.5rem; background: #007bff; color: white; text-decoration: none; border-radius: 4px; display: inline-block;" %>
+  </div>
+</div>

data/app/views/pg_sql_triggers/tables/show.html.erb

@@ -46,8 +46,8 @@
         <div style="border: 1px solid #dee2e6; border-radius: 4px; padding: 1rem; background: #f8f9fa;">
           <div style="display: flex; justify-content: space-between; align-items: start; margin-bottom: 1rem;">
             <div>
-              <h4 style="margin: 0; color: #007bff;">
-                <%= trigger.trigger_name %>
+              <h4 style="margin: 0;">
+                <%= link_to trigger.trigger_name, trigger_path(trigger), style: "color: #007bff; text-decoration: none;" %>
                 <% if trigger.enabled %>
                   <span class="badge badge-success">Enabled</span>
                 <% else %>
@@ -64,6 +64,40 @@
             </div>
           </div>
 
+          <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
+            <div style="margin-bottom: 1rem; display: flex; gap: 0.5rem;">
+              <% if trigger.enabled %>
+                <% form_id = "table-trigger-disable-#{trigger.id}-form" %>
+                <%= form_with url: disable_trigger_path(trigger), method: :post, local: true, id: form_id, style: "margin: 0;" do |f| %>
+                  <%= f.hidden_field :redirect_to, value: table_path(@table_info[:table_name]) %>
+                  <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+                      style="padding: 0.5rem 1rem; background: #dc3545; color: white; border: none; border-radius: 4px; cursor: pointer; font-size: 0.875rem; font-weight: 600;">
+                    Disable Trigger
+                  </button>
+                <% end %>
+                <%= render 'pg_sql_triggers/shared/confirmation_modal',
+                           operation: :ui_trigger_disable,
+                           form_id: form_id,
+                           title: 'Disable Trigger',
+                           message: "Are you sure you want to disable trigger '#{trigger.trigger_name}' on table '#{@table_info[:table_name]}'?" %>
+              <% else %>
+                <% form_id = "table-trigger-enable-#{trigger.id}-form" %>
+                <%= form_with url: enable_trigger_path(trigger), method: :post, local: true, id: form_id, style: "margin: 0;" do |f| %>
+                  <%= f.hidden_field :redirect_to, value: table_path(@table_info[:table_name]) %>
+                  <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+                      style="padding: 0.5rem 1rem; background: #28a745; color: white; border: none; border-radius: 4px; cursor: pointer; font-size: 0.875rem; font-weight: 600;">
+                    Enable Trigger
+                  </button>
+                <% end %>
+                <%= render 'pg_sql_triggers/shared/confirmation_modal',
+                           operation: :ui_trigger_enable,
+                           form_id: form_id,
+                           title: 'Enable Trigger',
+                           message: "Are you sure you want to enable trigger '#{trigger.trigger_name}' on table '#{@table_info[:table_name]}'?" %>
+              <% end %>
+            </div>
+          <% end %>
+
           <% if trigger.definition.present? %>
             <% definition = JSON.parse(trigger.definition) rescue {} %>
             <div style="margin-bottom: 1rem;">

data/app/views/pg_sql_triggers/triggers/_drop_modal.html.erb

@@ -0,0 +1,129 @@
+<% form_id = "trigger-drop-#{trigger.id}-form" %>
+
+<%= form_with url: drop_trigger_path(trigger), method: :post, local: true, id: form_id, style: "margin: 0; display: inline-block;" do |f| %>
+  <%= f.hidden_field :redirect_to, value: params[:redirect_to] || trigger_path(trigger) %>
+
+  <button type="button" onclick="showDropModal('<%= form_id %>')"
+      style="padding: 0.75rem 1.5rem; background: #6c757d; color: white; border: none; border-radius: 4px; cursor: pointer; font-size: 0.875rem; font-weight: 600;">
+    Drop Trigger
+  </button>
+<% end %>
+
+<!-- Drop Modal -->
+<div id="<%= form_id %>-modal" style="display: none; position: fixed; z-index: 1000; left: 0; top: 0; width: 100%; height: 100%; overflow: auto; background-color: rgba(0,0,0,0.4);">
+  <div style="background-color: #fefefe; margin: 5% auto; padding: 0; border-radius: 8px; width: 90%; max-width: 600px; box-shadow: 0 4px 6px rgba(0,0,0,0.1);">
+    <!-- Header -->
+    <div style="background: #dc3545; color: white; padding: 1.5rem; border-radius: 8px 8px 0 0;">
+      <div style="display: flex; justify-content: space-between; align-items: center;">
+        <h3 style="margin: 0;">⚠️ Drop Trigger</h3>
+        <button type="button" onclick="closeDropModal('<%= form_id %>')"
+            style="background: none; border: none; color: white; font-size: 1.5rem; cursor: pointer; padding: 0; width: 30px; height: 30px;">
+          &times;
+        </button>
+      </div>
+    </div>
+
+    <!-- Body -->
+    <div style="padding: 1.5rem;">
+      <div style="background: #fff3cd; border-left: 4px solid #ffc107; padding: 1rem; margin-bottom: 1.5rem;">
+        <strong style="color: #856404;">Warning:</strong>
+        <p style="margin: 0.5rem 0 0 0; color: #856404;">
+          This will permanently drop trigger '<strong><%= trigger.trigger_name %></strong>' from the database and remove it from the registry.
+          This action cannot be undone.
+        </p>
+      </div>
+
+      <form id="<%= form_id %>-reason-form">
+        <div style="margin-bottom: 1rem;">
+          <label for="<%= form_id %>-reason" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+            Reason <span style="color: #dc3545;">*</span>
+          </label>
+          <textarea id="<%= form_id %>-reason" name="reason" required rows="3"
+              placeholder="Why are you dropping this trigger? This will be logged."
+              style="width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px; font-family: inherit;"></textarea>
+        </div>
+
+        <% if kill_switch_active? %>
+          <div style="margin-bottom: 1rem;">
+            <label for="<%= form_id %>-confirmation" style="display: block; font-weight: 600; margin-bottom: 0.5rem;">
+              Confirmation Text <span style="color: #dc3545;">*</span>
+            </label>
+            <input type="text" id="<%= form_id %>-confirmation" name="confirmation_text" required
+                placeholder="<%= expected_confirmation_text(:trigger_drop) %>"
+                autocomplete="off"
+                style="width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px; font-family: 'Monaco', 'Menlo', monospace;">
+            <small style="color: #6c757d; display: block; margin-top: 0.25rem;">
+              Type: <code><%= expected_confirmation_text(:trigger_drop) %></code>
+            </small>
+          </div>
+        <% end %>
+
+        <div style="display: flex; gap: 1rem; justify-content: flex-end;">
+          <button type="button" onclick="closeDropModal('<%= form_id %>')"
+              style="padding: 0.75rem 1.5rem; background: #6c757d; color: white; border: none; border-radius: 4px; cursor: pointer;">
+            Cancel
+          </button>
+          <button type="button" onclick="submitDrop('<%= form_id %>')"
+              style="padding: 0.75rem 1.5rem; background: #dc3545; color: white; border: none; border-radius: 4px; cursor: pointer; font-weight: 600;">
+            Drop Trigger
+          </button>
+        </div>
+      </form>
+    </div>
+  </div>
+</div>
+
+<script>
+function showDropModal(formId) {
+  document.getElementById(formId + '-modal').style.display = 'block';
+}
+
+function closeDropModal(formId) {
+  document.getElementById(formId + '-modal').style.display = 'none';
+}
+
+function submitDrop(formId) {
+  const form = document.getElementById(formId);
+  const reasonForm = document.getElementById(formId + '-reason-form');
+  const reason = document.getElementById(formId + '-reason').value;
+
+  if (!reason.trim()) {
+    alert('Please provide a reason for dropping this trigger.');
+    return;
+  }
+
+  <% if kill_switch_active? %>
+    const confirmation = document.getElementById(formId + '-confirmation').value;
+    if (!confirmation.trim()) {
+      alert('Please provide the confirmation text.');
+      return;
+    }
+
+    // Add confirmation to main form
+    const confirmationInput = document.createElement('input');
+    confirmationInput.type = 'hidden';
+    confirmationInput.name = 'confirmation_text';
+    confirmationInput.value = confirmation;
+    form.appendChild(confirmationInput);
+  <% end %>
+
+  // Add reason to main form
+  const reasonInput = document.createElement('input');
+  reasonInput.type = 'hidden';
+  reasonInput.name = 'reason';
+  reasonInput.value = reason;
+  form.appendChild(reasonInput);
+
+  form.submit();
+}
+
+// Close modal when clicking outside
+window.onclick = function(event) {
+  const modals = document.querySelectorAll('[id$="-modal"]');
+  modals.forEach(modal => {
+    if (event.target == modal) {
+      modal.style.display = 'none';
+    }
+  });
+}
+</script>