pg_sql_triggers 1.2.0 → 1.4.0

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

@@ -0,0 +1,177 @@
+<%# Audit Log Index Page %>
+<div style="max-width: 1400px; margin: 0 auto; padding: 2rem;">
+  <div style="display: flex; justify-content: space-between; align-items: center; margin-bottom: 2rem;">
+    <h1 style="margin: 0;">Audit Log</h1>
+    <div style="display: flex; gap: 1rem;">
+      <%= link_to "Dashboard", dashboard_path, style: "padding: 0.5rem 1rem; background: #6c757d; color: white; text-decoration: none; border-radius: 4px; display: inline-block;" %>
+      <% csv_params = params.permit(:trigger_name, :operation, :status, :environment, :actor_id).to_h %>
+      <%= link_to "Export CSV", audit_logs_path(csv_params.merge(format: :csv)), style: "padding: 0.5rem 1rem; background: #28a745; color: white; text-decoration: none; border-radius: 4px; display: inline-block;" %>
+    </div>
+  </div>
+
+  <!-- Filters -->
+  <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; margin-bottom: 1rem;">Filters</h3>
+    <%= form_with url: audit_logs_path, method: :get, local: true, style: "display: grid; grid-template-columns: repeat(auto-fit, minmax(200px, 1fr)); gap: 1rem;" do |f| %>
+      <div>
+        <%= f.label :trigger_name, "Trigger Name", style: "display: block; margin-bottom: 0.5rem; font-weight: 600;" %>
+        <%= f.select :trigger_name, options_for_select([["All", ""]] + @available_trigger_names.map { |n| [n, n] }, params[:trigger_name]), {}, { style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" } %>
+      </div>
+
+      <div>
+        <%= f.label :operation, "Operation", style: "display: block; margin-bottom: 0.5rem; font-weight: 600;" %>
+        <%= f.select :operation, options_for_select([["All", ""]] + @available_operations.map { |o| [o.humanize, o] }, params[:operation]), {}, { style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" } %>
+      </div>
+
+      <div>
+        <%= f.label :status, "Status", style: "display: block; margin-bottom: 0.5rem; font-weight: 600;" %>
+        <%= f.select :status, options_for_select([["All", ""], ["Success", "success"], ["Failure", "failure"]], params[:status]), {}, { style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" } %>
+      </div>
+
+      <div>
+        <%= f.label :environment, "Environment", style: "display: block; margin-bottom: 0.5rem; font-weight: 600;" %>
+        <%= f.select :environment, options_for_select([["All", ""]] + @available_environments.map { |e| [e.humanize, e] }, params[:environment]), {}, { style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" } %>
+      </div>
+
+      <div>
+        <%= f.label :sort, "Sort Order", style: "display: block; margin-bottom: 0.5rem; font-weight: 600;" %>
+        <%= f.select :sort, options_for_select([["Newest First", "desc"], ["Oldest First", "asc"]], params[:sort] || "desc"), {}, { style: "width: 100%; padding: 0.5rem; border: 1px solid #ced4da; border-radius: 4px;" } %>
+      </div>
+
+      <div style="display: flex; align-items: flex-end; gap: 0.5rem;">
+        <%= f.submit "Apply Filters", style: "padding: 0.5rem 1rem; background: #007bff; color: white; border: none; border-radius: 4px; cursor: pointer; font-weight: 600;" %>
+        <%= link_to "Clear", audit_logs_path, style: "padding: 0.5rem 1rem; background: #6c757d; color: white; text-decoration: none; border-radius: 4px; display: inline-block;" %>
+      </div>
+    <% end %>
+  </div>
+
+  <!-- Results Summary -->
+  <div style="background: #f8f9fa; padding: 1rem; border-radius: 4px; margin-bottom: 1rem;">
+    <strong>Total Results:</strong> <%= @total_count %> entries
+    <% if params[:trigger_name].present? || params[:operation].present? || params[:status].present? || params[:environment].present? %>
+      <span style="color: #6c757d;">(filtered)</span>
+    <% end %>
+  </div>
+
+  <!-- Audit Log Table -->
+  <% if @audit_logs.any? %>
+    <div style="background: white; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); overflow-x: auto;">
+      <table style="width: 100%; border-collapse: collapse;">
+        <thead>
+          <tr style="background: #f8f9fa; border-bottom: 2px solid #dee2e6;">
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Time</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Trigger</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Operation</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Status</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Environment</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Actor</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Reason</th>
+            <th style="padding: 0.75rem; text-align: left; font-weight: 600;">Error</th>
+          </tr>
+        </thead>
+        <tbody>
+          <% @audit_logs.each do |log| %>
+            <tr style="border-bottom: 1px solid #dee2e6;">
+              <td style="padding: 0.75rem;">
+                <span title="<%= log.created_at.strftime("%Y-%m-%d %H:%M:%S %Z") %>" style="cursor: help;">
+                  <%= distance_of_time_in_words_to_now(log.created_at) %> ago
+                </span>
+                <br>
+                <small style="color: #6c757d;"><%= log.created_at.strftime("%Y-%m-%d %H:%M:%S") %></small>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.trigger_name.present? %>
+                  <%= link_to log.trigger_name, trigger_path(PgSqlTriggers::TriggerRegistry.find_by(trigger_name: log.trigger_name)), style: "color: #007bff; text-decoration: none;" rescue log.trigger_name %>
+                <% else %>
+                  <span style="color: #6c757d;">—</span>
+                <% end %>
+              </td>
+              <td style="padding: 0.75rem;">
+                <code style="background: #f8f9fa; padding: 0.25rem 0.5rem; border-radius: 3px; font-size: 0.875rem;"><%= log.operation %></code>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.status == "success" %>
+                  <span style="background: #d4edda; color: #155724; padding: 0.25rem 0.5rem; border-radius: 3px; font-size: 0.875rem; font-weight: 600;">Success</span>
+                <% else %>
+                  <span style="background: #f8d7da; color: #721c24; padding: 0.25rem 0.5rem; border-radius: 3px; font-size: 0.875rem; font-weight: 600;">Failure</span>
+                <% end %>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.environment.present? %>
+                  <span class="badge badge-info"><%= log.environment %></span>
+                <% else %>
+                  <span style="color: #6c757d;">—</span>
+                <% end %>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.actor.present? %>
+                  <% actor_type = log.actor.is_a?(Hash) ? (log.actor["type"] || log.actor[:type]) : nil %>
+                  <% actor_id = log.actor.is_a?(Hash) ? (log.actor["id"] || log.actor[:id]) : nil %>
+                  <% if actor_type && actor_id %>
+                    <code style="background: #f8f9fa; padding: 0.25rem 0.5rem; border-radius: 3px; font-size: 0.875rem;"><%= "#{actor_type}:#{actor_id}" %></code>
+                  <% else %>
+                    <span style="color: #6c757d;">—</span>
+                  <% end %>
+                <% else %>
+                  <span style="color: #6c757d;">—</span>
+                <% end %>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.reason.present? %>
+                  <span title="<%= log.reason %>" style="cursor: help;">
+                    <%= truncate(log.reason, length: 50) %>
+                  </span>
+                <% else %>
+                  <span style="color: #6c757d;">—</span>
+                <% end %>
+              </td>
+              <td style="padding: 0.75rem;">
+                <% if log.error_message.present? %>
+                  <span style="color: #dc3545; cursor: help;" title="<%= log.error_message %>">
+                    <%= truncate(log.error_message, length: 50) %>
+                  </span>
+                <% else %>
+                  <span style="color: #6c757d;">—</span>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+
+    <!-- Pagination -->
+    <% if @total_pages > 1 %>
+      <div style="display: flex; justify-content: center; align-items: center; gap: 1rem; margin-top: 2rem;">
+        <% if @page > 1 %>
+          <% prev_params = params.except(:page).permit(:trigger_name, :operation, :status, :environment, :sort, :per_page).to_h %>
+          <%= link_to "« Previous", audit_logs_path(prev_params.merge(page: @page - 1)), style: "padding: 0.5rem 1rem; background: #007bff; color: white; text-decoration: none; border-radius: 4px;" %>
+        <% else %>
+          <span style="padding: 0.5rem 1rem; background: #e9ecef; color: #6c757d; border-radius: 4px; cursor: not-allowed;">« Previous</span>
+        <% end %>
+
+        <span style="color: #6c757d;">
+          Page <%= @page %> of <%= @total_pages %>
+        </span>
+
+        <% if @page < @total_pages %>
+          <% next_params = params.except(:page).permit(:trigger_name, :operation, :status, :environment, :sort, :per_page).to_h %>
+          <%= link_to "Next »", audit_logs_path(next_params.merge(page: @page + 1)), style: "padding: 0.5rem 1rem; background: #007bff; color: white; text-decoration: none; border-radius: 4px;" %>
+        <% else %>
+          <span style="padding: 0.5rem 1rem; background: #e9ecef; color: #6c757d; border-radius: 4px; cursor: not-allowed;">Next »</span>
+        <% end %>
+      </div>
+    <% end %>
+  <% else %>
+    <div style="background: white; padding: 3rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); text-align: center;">
+      <h3 style="margin-top: 0; color: #6c757d;">No audit log entries found</h3>
+      <p style="color: #6c757d;">
+        <% if params[:trigger_name].present? || params[:operation].present? || params[:status].present? || params[:environment].present? %>
+          Try adjusting your filters or <%= link_to "clear filters", audit_logs_path, style: "color: #007bff;" %>.
+        <% else %>
+          Audit log entries will appear here as operations are performed.
+        <% end %>
+      </p>
+    </div>
+  <% end %>
+</div>

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

@@ -2,9 +2,6 @@
   <h2 style="margin: 0;">Trigger Dashboard</h2>
   <div style="display: flex; gap: 1rem;">
     <%= link_to "View Tables", tables_path, class: "btn btn-primary", style: "font-size: 1rem; padding: 0.75rem 1.5rem; text-decoration: none;" %>
-    <%= link_to "Generate New Trigger", new_generator_path,
-        class: "btn btn-success",
-        style: "font-size: 1rem; padding: 0.75rem 1.5rem; text-decoration: none;" %>
   </div>
 </div>
 
@@ -40,6 +37,7 @@
         <th>Version</th>
         <th>Status</th>
         <th>Source</th>
+        <th>Last Applied</th>
         <th>Actions</th>
       </tr>
     </thead>
@@ -60,11 +58,20 @@
           </td>
           <td><span class="badge badge-info"><%= trigger.source %></span></td>
           <td>
-            <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
-              <div style="display: flex; gap: 0.5rem; flex-wrap: wrap;">
+            <% if trigger.installed_at.present? %>
+              <span title="<%= trigger.installed_at.strftime("%Y-%m-%d %H:%M:%S %Z") %>" style="cursor: help;">
+                <%= distance_of_time_in_words_to_now(trigger.installed_at) %> ago
+              </span>
+            <% else %>
+              <span style="color: #6c757d;">Never</span>
+            <% end %>
+          </td>
+          <td>
+            <div style="display: flex; gap: 0.5rem; flex-wrap: wrap;">
+              <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
                 <% 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| %>
+                  <%= form_with url: disable_trigger_path(trigger), method: :post, local: false, 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;">
@@ -78,7 +85,7 @@
                              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| %>
+                  <%= form_with url: enable_trigger_path(trigger), method: :post, local: false, 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;">
@@ -91,10 +98,25 @@
                              title: 'Enable Trigger',
                              message: "Are you sure you want to enable trigger '#{trigger.trigger_name}'?" %>
                 <% end %>
-              </div>
-            <% else %>
-              —
-            <% end %>
+              <% end %>
+
+              <% if PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger) %>
+                <% begin %>
+                  <% drift_info = trigger.drift_result %>
+                  <% if drift_info && drift_info[:state] == 'drifted' %>
+                    <%= render 'pg_sql_triggers/triggers/re_execute_modal', trigger: trigger, drift_info: drift_info, redirect_to: dashboard_path, button_size: :small %>
+                  <% end %>
+                <% rescue StandardError %>
+                  <%# Skip if drift detection fails %>
+                <% end %>
+
+                <%= render 'pg_sql_triggers/triggers/drop_modal', trigger: trigger, redirect_to: dashboard_path, button_size: :small %>
+              <% end %>
+
+              <% unless PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) || PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger) %>
+                <span style="color: #6c757d;">—</span>
+              <% end %>
+            </div>
           </td>
         </tr>
       <% end %>
@@ -104,7 +126,7 @@
   <div style="background: #e7f3ff; border-left: 4px solid #007bff; padding: 1.5rem; border-radius: 4px;">
     <h3 style="margin-top: 0;">No triggers yet</h3>
     <p style="margin-bottom: 1rem;">Get started by generating your first trigger using the form-based wizard.</p>
-    <%= link_to "Generate New Trigger", new_generator_path, class: "btn btn-primary" %>
+    <%= link_to "View Tables", tables_path, class: "btn btn-primary" %>
   </div>
 <% end %>
 

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

@@ -1,6 +1,5 @@
 <div style="display: flex; justify-content: space-between; align-items: center; margin-bottom: 2rem;">
   <h2 style="margin: 0;">Database Tables & Triggers</h2>
-  <%= link_to "Generate New Trigger", new_generator_path, class: "btn btn-success" %>
 </div>
 
 <!-- Statistics -->
@@ -9,11 +8,37 @@
     <div style="font-size: 2rem; font-weight: 600; color: #28a745;"><%= @tables_with_trigger_count %></div>
     <div style="color: #6c757d;">Tables with Triggers</div>
   </div>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1);">
+    <div style="font-size: 2rem; font-weight: 600; color: #6c757d;"><%= @tables_without_trigger_count %></div>
+    <div style="color: #6c757d;">Tables without Triggers</div>
+  </div>
+  <div style="background: white; padding: 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1);">
+    <div style="font-size: 2rem; font-weight: 600; color: #007bff;"><%= @total_tables_count %></div>
+    <div style="color: #6c757d;">Total Tables</div>
+  </div>
+</div>
+
+<!-- Filter Controls -->
+<div style="background: white; padding: 1rem 1.5rem; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); margin-bottom: 2rem;">
+  <div style="display: flex; align-items: center; gap: 1.5rem; flex-wrap: wrap;">
+    <label style="color: #495057; font-weight: 600; font-size: 0.875rem;">Filter:</label>
+    <div style="display: flex; gap: 1rem; flex-wrap: wrap;">
+      <%= link_to "All Tables", tables_path(filter: 'all', page: 1, per_page: @per_page),
+          style: "padding: 0.5rem 1rem; background: #{@filter == 'all' ? '#007bff' : '#f8f9fa'}; color: #{@filter == 'all' ? 'white' : '#495057'}; text-decoration: none; border-radius: 4px; border: 1px solid #{@filter == 'all' ? '#007bff' : '#dee2e6'}; font-size: 0.875rem; font-weight: #{@filter == 'all' ? '600' : '400'};" %>
+      <%= link_to "With Triggers", tables_path(filter: 'with_triggers', page: 1, per_page: @per_page),
+          style: "padding: 0.5rem 1rem; background: #{@filter == 'with_triggers' ? '#28a745' : '#f8f9fa'}; color: #{@filter == 'with_triggers' ? 'white' : '#495057'}; text-decoration: none; border-radius: 4px; border: 1px solid #{@filter == 'with_triggers' ? '#28a745' : '#dee2e6'}; font-size: 0.875rem; font-weight: #{@filter == 'with_triggers' ? '600' : '400'};" %>
+      <%= link_to "Without Triggers", tables_path(filter: 'without_triggers', page: 1, per_page: @per_page),
+          style: "padding: 0.5rem 1rem; background: #{@filter == 'without_triggers' ? '#6c757d' : '#f8f9fa'}; color: #{@filter == 'without_triggers' ? 'white' : '#495057'}; text-decoration: none; border-radius: 4px; border: 1px solid #{@filter == 'without_triggers' ? '#6c757d' : '#dee2e6'}; font-size: 0.875rem; font-weight: #{@filter == 'without_triggers' ? '600' : '400'};" %>
+    </div>
+  </div>
 </div>
 
 <!-- Tables List -->
 <% if @tables_with_triggers.any? %>
   <div style="background: white; border-radius: 4px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); overflow: hidden;">
+    <div style="margin-bottom: 1rem; color: #6c757d; font-size: 0.875rem; padding: 1rem 1rem 0 1rem;">
+      Showing <%= (@page - 1) * @per_page + 1 %>-<%= [@page * @per_page, @total_tables].min %> of <%= @total_tables %> tables
+    </div>
     <table style="margin: 0;">
       <thead>
         <tr>
@@ -87,17 +112,62 @@
             </td>
             <td>
               <%= link_to "View Details", table_path(table[:table_name]), class: "btn btn-primary", style: "padding: 0.25rem 0.5rem; font-size: 0.875rem;" %>
-              <%= link_to "Create Trigger", new_generator_path(pg_sql_triggers_generator_form: { table_name: table[:table_name] }), class: "btn btn-success", style: "padding: 0.25rem 0.5rem; font-size: 0.875rem; margin-top: 0.25rem; display: block;" %>
             </td>
           </tr>
         <% end %>
       </tbody>
     </table>
+
+    <!-- Pagination Controls -->
+    <% if @total_pages > 1 %>
+      <div style="display: flex; justify-content: space-between; align-items: center; margin-top: 1rem; padding: 1rem; border-top: 1px solid #dee2e6;">
+        <div>
+          <% if @page > 1 %>
+            <%= link_to "← Previous", tables_path(filter: @filter, page: @page - 1, per_page: @per_page),
+                style: "padding: 0.5rem 1rem; background: #007bff; color: white; text-decoration: none; border-radius: 4px; margin-right: 0.5rem;" %>
+          <% end %>
+          <% if @page < @total_pages %>
+            <%= link_to "Next →", tables_path(filter: @filter, page: @page + 1, per_page: @per_page),
+                style: "padding: 0.5rem 1rem; background: #007bff; color: white; text-decoration: none; border-radius: 4px;" %>
+          <% end %>
+        </div>
+        <div style="color: #6c757d; font-size: 0.875rem;">
+          Page <%= @page %> of <%= @total_pages %>
+        </div>
+        <div style="display: flex; align-items: center; gap: 0.5rem;">
+          <label style="color: #6c757d; font-size: 0.875rem;">Per page:</label>
+          <select onchange="window.location.href='<%= tables_path %>?filter=<%= @filter %>&page=1&per_page=' + this.value"
+                  style="padding: 0.25rem 0.5rem; border: 1px solid #ced4da; border-radius: 4px;">
+            <option value="10" <%= 'selected' if @per_page == 10 %>>10</option>
+            <option value="20" <%= 'selected' if @per_page == 20 %>>20</option>
+            <option value="50" <%= 'selected' if @per_page == 50 %>>50</option>
+            <option value="100" <%= 'selected' if @per_page == 100 %>>100</option>
+          </select>
+        </div>
+      </div>
+    <% end %>
   </div>
 <% else %>
   <div style="background: #e7f3ff; border-left: 4px solid #007bff; padding: 1.5rem; border-radius: 4px;">
-    <h3 style="margin-top: 0;">No tables with triggers found</h3>
-    <p style="margin-bottom: 1rem;">No tables with triggers were found in the database. Create your first trigger to get started.</p>
-    <%= link_to "Generate New Trigger", new_generator_path, class: "btn btn-primary" %>
+    <h3 style="margin-top: 0;">
+      <% if @filter == 'with_triggers' %>
+        No tables with triggers found
+      <% elsif @filter == 'without_triggers' %>
+        No tables without triggers found
+      <% else %>
+        No tables found
+      <% end %>
+    </h3>
+    <p style="margin-bottom: 1rem;">
+      <% if @filter == 'with_triggers' %>
+        No tables with triggers were found in the database. Create your first trigger to get started.
+      <% elsif @filter == 'without_triggers' %>
+        All tables in the database have triggers.
+      <% else %>
+        No tables were found in the database.
+      <% end %>
+    </p>
+    <% if @filter == 'with_triggers' || @filter == 'all' %>
+      <% end %>
   </div>
 <% end %>

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

@@ -1,7 +1,6 @@
 <div style="margin-bottom: 2rem;">
   <h2>Table: <%= @table_info[:table_name] %></h2>
   <%= link_to "← Back to Tables", tables_path, class: "btn", style: "background: #6c757d; color: white; text-decoration: none; margin-right: 1rem;" %>
-  <%= link_to "Create Trigger for this Table", new_generator_path(pg_sql_triggers_generator_form: { table_name: @table_info[:table_name] }), class: "btn btn-success" %>
 </div>
 
 <!-- Table Information -->
@@ -64,8 +63,8 @@
             </div>
           </div>
 
-          <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
-            <div style="margin-bottom: 1rem; display: flex; gap: 0.5rem;">
+          <div style="margin-bottom: 1rem; display: flex; gap: 0.5rem; flex-wrap: wrap;">
+            <% if PgSqlTriggers::Permissions.can?(current_actor, :enable_trigger) %>
               <% 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| %>
@@ -95,8 +94,21 @@
                            title: 'Enable Trigger',
                            message: "Are you sure you want to enable trigger '#{trigger.trigger_name}' on table '#{@table_info[:table_name]}'?" %>
               <% end %>
-            </div>
-          <% end %>
+            <% end %>
+
+            <% if PgSqlTriggers::Permissions.can?(current_actor, :drop_trigger) %>
+              <% begin %>
+                <% drift_info = trigger.drift_result %>
+                <% if drift_info && drift_info[:state] == 'drifted' %>
+                  <%= render 'pg_sql_triggers/triggers/re_execute_modal', trigger: trigger, drift_info: drift_info, redirect_to: table_path(@table_info[:table_name]) %>
+                <% end %>
+              <% rescue StandardError %>
+                <%# Skip if drift detection fails %>
+              <% end %>
+
+              <%= render 'pg_sql_triggers/triggers/drop_modal', trigger: trigger, redirect_to: table_path(@table_info[:table_name]) %>
+            <% end %>
+          </div>
 
           <% if trigger.definition.present? %>
             <% definition = JSON.parse(trigger.definition) rescue {} %>
@@ -125,7 +137,6 @@
   <% else %>
     <div style="background: #e7f3ff; border-left: 4px solid #007bff; padding: 1rem; border-radius: 4px;">
       <p style="margin: 0;">No registered triggers for this table.</p>
-      <%= link_to "Create a trigger", new_generator_path(pg_sql_triggers_generator_form: { table_name: @table_info[:table_name] }), class: "btn btn-primary", style: "margin-top: 0.5rem;" %>
     </div>
   <% end %>
 </div>

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

@@ -1,12 +1,21 @@
-<% form_id = "trigger-drop-#{trigger.id}-form" %>
+<% form_id = local_assigns[:form_id] || "trigger-drop-#{trigger.id}-form" %>
+<% redirect_path = local_assigns[:redirect_to] || params[:redirect_to] || trigger_path(trigger) %>
+<% button_size = local_assigns[:button_size] || :medium %>
 
 <%= 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>
+  <%= f.hidden_field :redirect_to, value: redirect_path %>
+
+  <% if button_size == :small %>
+    <button type="button" onclick="showDropModal('<%= form_id %>')"
+        style="padding: 0.25rem 0.75rem; background: #6c757d; color: white; border: none; border-radius: 3px; cursor: pointer; font-size: 0.75rem;">
+      Drop
+    </button>
+  <% else %>
+    <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 %>
 <% end %>
 
 <!-- Drop Modal -->

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

@@ -1,12 +1,21 @@
-<% form_id = "trigger-re-execute-#{trigger.id}-form" %>
+<% form_id = local_assigns[:form_id] || "trigger-re-execute-#{trigger.id}-form" %>
+<% redirect_path = local_assigns[:redirect_to] || params[:redirect_to] || trigger_path(trigger) %>
+<% button_size = local_assigns[:button_size] || :medium %>
 
 <%= 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>
+  <%= f.hidden_field :redirect_to, value: redirect_path %>
+
+  <% if button_size == :small %>
+    <button type="button" onclick="showReExecuteModal('<%= form_id %>')"
+        style="padding: 0.25rem 0.75rem; background: #ffc107; color: #000; border: none; border-radius: 3px; cursor: pointer; font-size: 0.75rem;">
+      Re-Execute
+    </button>
+  <% else %>
+    <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 %>
 <% end %>
 
 <!-- Re-Execute Modal -->

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

@@ -1,7 +1,17 @@
-<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;" %>
+<!-- Breadcrumb Navigation -->
+<div style="margin-bottom: 1rem; color: #6c757d; font-size: 0.875rem;">
+  <%= link_to "Dashboard", dashboard_path, style: "color: #007bff; text-decoration: none;" %> /
+  <%= link_to "Tables", tables_path, style: "color: #007bff; text-decoration: none;" %> /
+  <%= link_to @trigger.table_name, table_path(@trigger.table_name), style: "color: #007bff; text-decoration: none;" %> /
+  <span style="color: #495057; font-weight: 600;"><%= @trigger.trigger_name %></span>
+</div>
+
+<div style="display: flex; justify-content: space-between; align-items: center; margin-bottom: 2rem;">
+  <h2 style="margin: 0;">Trigger Details: <%= @trigger.trigger_name %></h2>
+  <div style="display: flex; gap: 0.5rem;">
+    <%= link_to "← Back to Dashboard", dashboard_path, class: "btn", style: "background: #6c757d; color: white; text-decoration: none; padding: 0.5rem 1rem;" %>
+    <%= link_to "View Table", table_path(@trigger.table_name), class: "btn", style: "background: #007bff; color: white; text-decoration: none; padding: 0.5rem 1rem;" %>
+  </div>
 </div>
 
 <!-- Drift Warning -->
@@ -55,8 +65,18 @@
 
   <% 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 style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Last Applied</div>
+      <div title="<%= @trigger.installed_at.strftime("%Y-%m-%d %H:%M:%S %Z") %>">
+        <%= distance_of_time_in_words_to_now(@trigger.installed_at) %> ago
+        <small style="color: #6c757d;">(<%= @trigger.installed_at.strftime("%Y-%m-%d %H:%M:%S") %>)</small>
+      </div>
+    </div>
+  <% end %>
+
+  <% if @trigger.last_verified_at.present? %>
+    <div style="margin-bottom: 1rem;">
+      <div style="color: #6c757d; font-size: 0.875rem; margin-bottom: 0.25rem;">Last Verified</div>
+      <div><%= @trigger.last_verified_at.strftime("%Y-%m-%d %H:%M:%S %Z") %></div>
     </div>
   <% end %>
 

data/config/routes.rb

@@ -7,20 +7,6 @@ begin
 
     resources :tables, only: %i[index show]
 
-    resources :generator, only: %i[new create] do
-      collection do
-        post :preview
-        post :validate_table
-        get :tables
-      end
-    end
-
-    resources :sql_capsules, only: %i[new create show] do
-      member do
-        post :execute
-      end
-    end
-
     resources :migrations, only: [] do
       collection do
         post :up
@@ -37,6 +23,8 @@ begin
         post :re_execute
       end
     end
+
+    resources :audit_logs, only: [:index]
   end
 rescue ArgumentError => e
   # Ignore duplicate route errors (routes may already be drawn in tests)

data/db/migrate/20260103000001_create_pg_sql_triggers_audit_log.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+class CreatePgSqlTriggersAuditLog < ActiveRecord::Migration[6.1]
+  def change
+    create_table :pg_sql_triggers_audit_log do |t|
+      t.string :trigger_name
+      t.string :operation, null: false
+      t.jsonb :actor # Store actor information (type, id)
+      t.string :environment
+      t.string :status, null: false # success, failure
+      t.text :reason
+      t.string :confirmation_text
+      t.jsonb :before_state # Store state before operation
+      t.jsonb :after_state # Store state after operation
+      t.text :diff # Store diff if applicable
+      t.text :error_message # Store error message if operation failed
+
+      t.timestamps
+    end
+
+    add_index :pg_sql_triggers_audit_log, :trigger_name
+    add_index :pg_sql_triggers_audit_log, :operation
+    add_index :pg_sql_triggers_audit_log, :status
+    add_index :pg_sql_triggers_audit_log, :environment
+    add_index :pg_sql_triggers_audit_log, :created_at
+    add_index :pg_sql_triggers_audit_log, %i[trigger_name created_at]
+  end
+end

data/db/migrate/20260228000001_add_for_each_to_pg_sql_triggers_registry.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+class AddForEachToPgSqlTriggersRegistry < ActiveRecord::Migration[6.1]
+  def change
+    add_column :pg_sql_triggers_registry, :for_each, :string, default: "row", null: false
+    add_index :pg_sql_triggers_registry, :for_each
+  end
+end