pg_sql_triggers 1.1.0 → 1.2.0

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>

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

@@ -0,0 +1,136 @@
+<% form_id = "trigger-re-execute-#{trigger.id}-form" %>
+
+<%= form_with url: re_execute_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="showReExecuteModal('<%= form_id %>')"
+      style="padding: 0.75rem 1.5rem; background: #ffc107; color: #000; border: none; border-radius: 4px; cursor: pointer; font-size: 0.875rem; font-weight: 600;">
+    Re-Execute Trigger
+  </button>
+<% end %>
+
+<!-- Re-Execute 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: 3% auto; padding: 0; border-radius: 8px; width: 90%; max-width: 800px; box-shadow: 0 4px 6px rgba(0,0,0,0.1);">
+    <!-- Header -->
+    <div style="background: #ffc107; color: #000; padding: 1.5rem; border-radius: 8px 8px 0 0;">
+      <div style="display: flex; justify-content: space-between; align-items: center;">
+        <h3 style="margin: 0;">🔄 Re-Execute Trigger</h3>
+        <button type="button" onclick="closeReExecuteModal('<%= form_id %>')"
+            style="background: none; border: none; color: #000; 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;">Notice:</strong>
+        <p style="margin: 0.5rem 0 0 0; color: #856404;">
+          This will drop and re-create trigger '<strong><%= trigger.trigger_name %></strong>' based on the expected definition from the registry.
+          This can fix drift issues.
+        </p>
+      </div>
+
+      <!-- Show Drift Diff -->
+      <% if drift_info[:expected_sql].present? %>
+        <div style="margin-bottom: 1.5rem;">
+          <h4 style="margin-bottom: 1rem;">SQL Comparison</h4>
+
+          <div style="margin-bottom: 1rem;">
+            <div style="color: #28a745; font-weight: 600; margin-bottom: 0.5rem;">✓ Expected SQL (will be applied):</div>
+            <pre style="background: #d4edda; padding: 1rem; border-radius: 4px; overflow-x: auto; border: 1px solid #c3e6cb; font-size: 0.85rem; max-height: 200px;"><code><%= drift_info[:expected_sql] %></code></pre>
+          </div>
+
+          <div>
+            <div style="color: #dc3545; font-weight: 600; margin-bottom: 0.5rem;">✗ Current SQL (will be replaced):</div>
+            <pre style="background: #f8d7da; padding: 1rem; border-radius: 4px; overflow-x: auto; border: 1px solid #f5c6cb; font-size: 0.85rem; max-height: 200px;"><code><%= drift_info[:actual_sql] || "Not found in database" %></code></pre>
+          </div>
+        </div>
+      <% end %>
+
+      <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="2"
+              placeholder="Why are you re-executing 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_re_execute) %>"
+                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_re_execute) %></code>
+            </small>
+          </div>
+        <% end %>
+
+        <div style="display: flex; gap: 1rem; justify-content: flex-end;">
+          <button type="button" onclick="closeReExecuteModal('<%= 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="submitReExecute('<%= form_id %>')"
+              style="padding: 0.75rem 1.5rem; background: #ffc107; color: #000; border: none; border-radius: 4px; cursor: pointer; font-weight: 600;">
+            Re-Execute Trigger
+          </button>
+        </div>
+      </form>
+    </div>
+  </div>
+</div>
+
+<script>
+function showReExecuteModal(formId) {
+  document.getElementById(formId + '-modal').style.display = 'block';
+}
+
+function closeReExecuteModal(formId) {
+  document.getElementById(formId + '-modal').style.display = 'none';
+}
+
+function submitReExecute(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 re-executing 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();
+}
+</script>

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

@@ -0,0 +1,186 @@
+<div style="margin-bottom: 2rem;">
+  <h2>Trigger Details: <%= @trigger.trigger_name %></h2>
+  <%= link_to "← Back to Dashboard", dashboard_path, class: "btn", style: "background: #6c757d; color: white; text-decoration: none; margin-right: 1rem;" %>
+  <%= link_to "View Table", table_path(@trigger.table_name), class: "btn", style: "background: #007bff; color: white; text-decoration: none;" %>
+</div>
+
+<!-- Drift Warning -->
+<% if @drift_info[:has_drift] %>
+  <div style="background: #fff3cd; border-left: 4px solid #ffc107; padding: 1.5rem; border-radius: 4px; margin-bottom: 2rem;">
+    <h3 style="margin-top: 0; color: #856404;">⚠ Drift Detected</h3>
+    <p style="color: #856404; margin-bottom: 0;">
+      This trigger has drifted from its expected state. Type: <strong><%= @drift_info[:drift_type] %></strong>
+    </p>
+  </div>
+<% end %>
+
+<!-- Trigger Summary -->
+<div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+  <h3 style="margin-top: 0;">Summary</h3>
+
+  <div style="display: grid; grid-template-columns: repeat(auto-fit, minmax(200px, 1fr)); gap: 1.5rem; margin-bottom: 1.5rem;">
+    <div>
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Status</div>
+      <div>
+        <% if @trigger.enabled %>
+          <span class="badge badge-success">Enabled</span>
+        <% else %>
+          <span class="badge badge-danger">Disabled</span>
+        <% end %>
+      </div>
+    </div>
+
+    <div>
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Table</div>
+      <div><strong><%= @trigger.table_name %></strong></div>
+    </div>
+
+    <div>
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Version</div>
+      <div><strong><%= @trigger.version %></strong></div>
+    </div>
+
+    <div>
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Source</div>
+      <div><span class="badge badge-info"><%= @trigger.source %></span></div>
+    </div>
+  </div>
+
+  <% if @trigger.environment.present? %>
+    <div style="margin-bottom: 1rem;">
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Environment</div>
+      <div><strong><%= @trigger.environment %></strong></div>
+    </div>
+  <% end %>
+
+  <% if @trigger.installed_at.present? %>
+    <div style="margin-bottom: 1rem;">
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Installed At</div>
+      <div><%= @trigger.installed_at.strftime("%Y-%m-%d %H:%M:%S %Z") %></div>
+    </div>
+  <% end %>
+
+  <div>
+    <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Created At</div>
+    <div><%= @trigger.created_at.strftime("%Y-%m-%d %H:%M:%S %Z") %></div>
+  </div>
+</div>
+
+<!-- Trigger Definition -->
+<% if @trigger.definition.present? %>
+  <% definition = JSON.parse(@trigger.definition) rescue {} %>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+    <h3 style="margin-top: 0;">Trigger Configuration</h3>
+
+    <div style="margin-bottom: 1rem;">
+      <strong>Function:</strong>
+      <code style="background: #e9ecef; padding: 0.25rem 0.5rem; border-radius: 2px;"><%= definition["function_name"] || "N/A" %></code>
+    </div>
+
+    <% if definition["timing"].present? %>
+      <div style="margin-bottom: 1rem;">
+        <strong>Timing:</strong>
+        <span class="badge badge-info"><%= definition["timing"].upcase %></span>
+      </div>
+    <% end %>
+
+    <% if definition["events"].present? %>
+      <div style="margin-bottom: 1rem;">
+        <strong>Events:</strong>
+        <% definition["events"].each do |event| %>
+          <span class="badge badge-info" style="margin-left: 0.25rem;"><%= event.upcase %></span>
+        <% end %>
+      </div>
+    <% end %>
+
+    <% if definition["condition"].present? %>
+      <div style="margin-bottom: 1rem;">
+        <strong>Condition (WHEN):</strong>
+        <pre style="background: #f8f9fa; padding: 0.5rem; border-radius: 4px; margin-top: 0.25rem; overflow-x: auto;"><code><%= definition["condition"] %></code></pre>
+      </div>
+    <% end %>
+
+    <% if definition["environments"].present? %>
+      <div>
+        <strong>Environments:</strong>
+        <% definition["environments"].each do |env| %>
+          <span class="badge badge-secondary" style="margin-left: 0.25rem;"><%= env %></span>
+        <% end %>
+      </div>
+    <% end %>
+  </div>
+<% end %>
+
+<!-- SQL Diff (if drift detected) -->
+<% if @drift_info[:has_drift] && @drift_info[:expected_sql].present? %>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+    <h3 style="margin-top: 0;">SQL Drift Comparison</h3>
+
+    <div style="margin-bottom: 1.5rem;">
+      <h4 style="color: #28a745; margin-bottom: 0.5rem;">Expected SQL (from DSL)</h4>
+      <pre style="background: #d4edda; padding: 1rem; border-radius: 4px; overflow-x: auto; border: 1px solid #c3e6cb;"><code><%= @drift_info[:expected_sql] %></code></pre>
+    </div>
+
+    <div>
+      <h4 style="color: #dc3545; margin-bottom: 0.5rem;">Actual SQL (from database)</h4>
+      <pre style="background: #f8d7da; padding: 1rem; border-radius: 4px; overflow-x: auto; border: 1px solid #f5c6cb;"><code><%= @drift_info[:actual_sql] || "Not found in database" %></code></pre>
+    </div>
+  </div>
+<% end %>
+
+<!-- Function Body -->
+<% if @trigger.function_body.present? %>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+    <h3 style="margin-top: 0;">Function Body</h3>
+    <pre style="background: #f8f9fa; padding: 1rem; border-radius: 4px; overflow-x: auto;"><code><%= @trigger.function_body %></code></pre>
+  </div>
+<% end %>
+
+<!-- Action Buttons -->
+<% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1);">
+    <h3 style="margin-top: 0;">Actions</h3>
+
+    <div style="display: flex; gap: 1rem; flex-wrap: wrap;">
+      <% if @trigger.enabled %>
+        <% form_id = "trigger-detail-disable-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: trigger_path(@trigger) %>
+          <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+              style="padding: 0.75rem 1.5rem; 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}'?" %>
+      <% else %>
+        <% form_id = "trigger-detail-enable-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: trigger_path(@trigger) %>
+          <button type="button" onclick="showKillSwitchModal('<%= form_id %>')"
+              style="padding: 0.75rem 1.5rem; 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}'?" %>
+      <% end %>
+
+      <!-- Re-execute button (for drifted triggers) -->
+      <% if @drift_info[:has_drift] && PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger) %>
+        <%= render 'pg_sql_triggers/triggers/re_execute_modal', trigger: @trigger, drift_info: @drift_info %>
+      <% end %>
+
+      <!-- Drop button (Admin only) -->
+      <% if PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger) %>
+        <%= render 'pg_sql_triggers/triggers/drop_modal', trigger: @trigger %>
+      <% end %>
+    </div>
+  </div>
+<% end %>

data/config/routes.rb

@@ -28,6 +28,15 @@ begin
         post :redo
       end
     end
+
+    resources :triggers, only: [:show] do
+      member do
+        post :enable
+        post :disable
+        post :drop
+        post :re_execute
+      end
+    end
   end
 rescue ArgumentError => e
   # Ignore duplicate route errors (routes may already be drawn in tests)

data/docs/README.md

@@ -61,6 +61,6 @@ When updating documentation:
 
 ## External Resources
 
-- [GitHub Repository](https://github.com/samaswin87/pg_sql_triggers)
-- [Example Application](https://github.com/samaswin87/pg_triggers_example)
+- [GitHub Repository](https://github.com/samaswin/pg_sql_triggers)
+- [Example Application](https://github.com/samaswin/pg_triggers_example)
 - [RubyGems](https://rubygems.org/gems/pg_sql_triggers)