pg_sql_triggers 1.1.0 → 1.2.0

data/app/controllers/pg_sql_triggers/triggers_controller.rb

@@ -0,0 +1,165 @@
+# frozen_string_literal: true
+
+module PgSqlTriggers
+  # Controller for managing individual triggers via web UI
+  # Provides actions to enable and disable triggers
+  class TriggersController < ApplicationController
+    before_action :set_trigger, only: %i[show enable disable drop re_execute]
+    before_action :check_viewer_permission, only: [:show]
+    before_action :check_operator_permission, only: %i[enable disable]
+    before_action :check_admin_permission, only: %i[drop re_execute]
+
+    def show
+      # Load trigger details and drift information
+      @drift_info = calculate_drift_info
+    end
+
+    def enable
+      # Check kill switch before enabling trigger
+      check_kill_switch(operation: :ui_trigger_enable, confirmation: params[:confirmation_text])
+
+      @trigger.enable!(confirmation: params[:confirmation_text])
+      flash[:success] = "Trigger '#{@trigger.trigger_name}' enabled successfully."
+      redirect_to redirect_path
+    rescue PgSqlTriggers::KillSwitchError => e
+      flash[:error] = e.message
+      redirect_to redirect_path
+    rescue StandardError => e
+      Rails.logger.error("Enable failed: #{e.message}\n#{e.backtrace.join("\n")}")
+      flash[:error] = "Failed to enable trigger: #{e.message}"
+      redirect_to redirect_path
+    end
+
+    def disable
+      # Check kill switch before disabling trigger
+      check_kill_switch(operation: :ui_trigger_disable, confirmation: params[:confirmation_text])
+
+      @trigger.disable!(confirmation: params[:confirmation_text])
+      flash[:success] = "Trigger '#{@trigger.trigger_name}' disabled successfully."
+      redirect_to redirect_path
+    rescue PgSqlTriggers::KillSwitchError => e
+      flash[:error] = e.message
+      redirect_to redirect_path
+    rescue StandardError => e
+      Rails.logger.error("Disable failed: #{e.message}\n#{e.backtrace.join("\n")}")
+      flash[:error] = "Failed to disable trigger: #{e.message}"
+      redirect_to redirect_path
+    end
+
+    def drop
+      # Validate required parameters
+      if params[:reason].blank?
+        flash[:error] = "Reason is required for dropping a trigger."
+        redirect_to redirect_path
+        return
+      end
+
+      # Check kill switch before dropping trigger
+      check_kill_switch(operation: :trigger_drop, confirmation: params[:confirmation_text])
+
+      # Drop the trigger
+      @trigger.drop!(
+        reason: params[:reason],
+        confirmation: params[:confirmation_text],
+        actor: current_actor
+      )
+
+      flash[:success] = "Trigger '#{@trigger.trigger_name}' dropped successfully."
+      redirect_to dashboard_path
+    rescue PgSqlTriggers::KillSwitchError => e
+      flash[:error] = e.message
+      redirect_to redirect_path
+    rescue ArgumentError => e
+      flash[:error] = "Invalid request: #{e.message}"
+      redirect_to redirect_path
+    rescue StandardError => e
+      Rails.logger.error("Drop failed: #{e.message}\n#{e.backtrace.join("\n")}")
+      flash[:error] = "Failed to drop trigger: #{e.message}"
+      redirect_to redirect_path
+    end
+
+    def re_execute
+      # Validate required parameters
+      if params[:reason].blank?
+        flash[:error] = "Reason is required for re-executing a trigger."
+        redirect_to redirect_path
+        return
+      end
+
+      # Check kill switch before re-executing trigger
+      check_kill_switch(operation: :trigger_re_execute, confirmation: params[:confirmation_text])
+
+      # Re-execute the trigger
+      @trigger.re_execute!(
+        reason: params[:reason],
+        confirmation: params[:confirmation_text],
+        actor: current_actor
+      )
+
+      flash[:success] = "Trigger '#{@trigger.trigger_name}' re-executed successfully."
+      redirect_to redirect_path
+    rescue PgSqlTriggers::KillSwitchError => e
+      flash[:error] = e.message
+      redirect_to redirect_path
+    rescue ArgumentError => e
+      flash[:error] = "Invalid request: #{e.message}"
+      redirect_to redirect_path
+    rescue StandardError => e
+      Rails.logger.error("Re-execute failed: #{e.message}\n#{e.backtrace.join("\n")}")
+      flash[:error] = "Failed to re-execute trigger: #{e.message}"
+      redirect_to redirect_path
+    end
+
+    private
+
+    def set_trigger
+      @trigger = PgSqlTriggers::TriggerRegistry.find(params[:id])
+    rescue ActiveRecord::RecordNotFound
+      flash[:error] = "Trigger not found."
+      redirect_to root_path
+    end
+
+    def check_viewer_permission
+      return if PgSqlTriggers::Permissions.can?(current_actor, :view_triggers)
+
+      redirect_to root_path, alert: "Insufficient permissions. Viewer role required."
+    end
+
+    def check_operator_permission
+      return if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger)
+
+      redirect_to root_path, alert: "Insufficient permissions. Operator role required."
+    end
+
+    def check_admin_permission
+      return if PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger)
+
+      redirect_to root_path, alert: "Insufficient permissions. Admin role required."
+    end
+
+    def redirect_path
+      # Redirect back to the referring page if possible, otherwise to dashboard
+      params[:redirect_to].presence || request.referer || root_path
+    end
+
+    def calculate_drift_info
+      # Get drift information for this trigger
+      drift_reporter = PgSqlTriggers::Drift::Reporter.new
+      drift_summary = drift_reporter.summary
+
+      # Find this trigger in the drift summary
+      drifted_triggers = drift_summary[:triggers] || []
+      drift_data = drifted_triggers.find { |t| t[:trigger_name] == @trigger.trigger_name }
+
+      {
+        has_drift: drift_data.present?,
+        drift_type: drift_data&.dig(:drift_type),
+        expected_sql: drift_data&.dig(:expected_sql),
+        actual_sql: drift_data&.dig(:actual_sql)
+      }
+    rescue StandardError => e
+      Rails.logger.error("Failed to calculate drift: #{e.message}")
+      { has_drift: false, drift_type: nil, expected_sql: nil, actual_sql: nil }
+    end
+  end
+end

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;">