pg_sql_triggers 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54f76c538693c37b1572cb914462269eb8c3600473c1a7c26efda0a7e02eefe5
-  data.tar.gz: 8574ff3bb8f835a11f8cec850c5adbff7f17be65a1461b8f8e87242ab3e50b62
+  metadata.gz: 6f8d9043692145293beaf91342280ca01702979a6e50190fd5930d8578170291
+  data.tar.gz: 18e9eaaa03de9932f9916bcc2110e9896a271c555a3b94f13348baf1e37c584c
 SHA512:
-  metadata.gz: 5e7ccb4983c43cad0aee550a741e341a3feda8a2fd5541be1786a92ffc26894a80eb8081e57493e6eb05697a362cafaa3fee325b288a716f4f1e8c3de989966e
-  data.tar.gz: f24548a2b4fa7f0bb274d11d2085b87e716c44e08923782eee93ffb0d86500892ffca3b6d7c69ba53e1f886743646ac0c8695097589ecb7db4280152b4c1dccf
+  metadata.gz: 8ed71a10f16385f318a4a596a444c07d2cb00a7d07119043c921cf43b1d2199db6d36cc30b5ea523bcbb527a817b97fbe7c507ac81a4cec3931f887c2a4606ca
+  data.tar.gz: af0162bab6005904e1649619e5a68e1c1f36733e4f8d34f0306c9a64af07356e9f19165dda1a60f19606b36b517ee1ad34224a50b479b02b0b8efd340175f1fc

data/CHANGELOG.md

@@ -7,6 +7,150 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.3.0] - 2026-01-05
+
+### Added
+- **Enhanced Console API**: Added missing drift query methods to Registry API for consistency
+  - `PgSqlTriggers::Registry.drifted` - Returns all drifted triggers
+  - `PgSqlTriggers::Registry.in_sync` - Returns all in-sync triggers
+  - `PgSqlTriggers::Registry.unknown_triggers` - Returns all unknown (external) triggers
+  - `PgSqlTriggers::Registry.dropped` - Returns all dropped triggers
+  - All console APIs now follow consistent naming conventions (query methods vs action methods)
+
+- **Controller Concerns**: Extracted common controller functionality into reusable concerns
+  - `KillSwitchProtection` concern - Handles kill switch checking and confirmation helpers
+  - `PermissionChecking` concern - Handles permission checks and actor management
+  - `ErrorHandling` concern - Handles error formatting and flash message helpers
+  - All controllers now inherit from `ApplicationController` which includes these concerns
+  - Improved code organization and maintainability
+
+- **YARD Documentation**: Comprehensive YARD documentation added to all public APIs
+  - `PgSqlTriggers::Registry` module - All public methods fully documented
+  - `PgSqlTriggers::TriggerRegistry` model - All public methods fully documented
+  - `PgSqlTriggers::Generator::Service` - All public class methods fully documented
+  - `PgSqlTriggers::SQL::Executor` - Already had documentation (verified)
+  - All documentation includes parameter types, return values, and examples
+
+### Added
+- **Complete UI Action Buttons**: All trigger operations now accessible via web UI
+  - Enable/Disable buttons in dashboard and table detail views
+  - Drop trigger button with confirmation modal (Admin permission required)
+  - Re-execute trigger button with drift diff display (Admin permission required)
+  - All buttons respect permission checks and show/hide based on user role
+  - Kill switch integration with confirmation modals for all actions
+  - Buttons styled with environment-aware colors (warning colors for production)
+
+- **Enhanced Dashboard**:
+  - "Last Applied" column showing `installed_at` timestamps in human-readable format
+  - Tooltips with exact timestamps on hover
+  - Default sorting by `installed_at` (most recent first)
+  - Drop and Re-execute buttons in dashboard table (Admin only)
+  - Permission-aware button visibility throughout
+
+- **Trigger Detail Page Enhancements**:
+  - Breadcrumb navigation (Dashboard → Tables → Table → Trigger)
+  - Enhanced `installed_at` display with relative time formatting
+  - `last_verified_at` timestamp display
+  - All action buttons (enable/disable/drop/re-execute) accessible from detail page
+
+- **Comprehensive Audit Logging System**:
+  - New `pg_sql_triggers_audit_log` table for tracking all operations
+  - `AuditLog` model with logging methods (`log_success`, `log_failure`)
+  - Audit logging integrated into all trigger operations:
+    - `enable!` - logs success/failure with before/after state
+    - `disable!` - logs success/failure with before/after state  
+    - `drop!` - logs success/failure with reason and state changes
+    - `re_execute!` - logs success/failure with drift diff information
+  - All operations track actor (who performed the action)
+  - Complete state capture (before/after) for all operations
+  - Error messages logged for failed operations
+  - Environment and confirmation text tracking
+
+- **Enhanced Actor Tracking**:
+  - All trigger operations now accept `actor` parameter
+  - Console APIs updated to pass actor information
+  - UI controllers pass `current_actor` to all operations
+  - Actor information stored in audit logs for complete audit trail
+
+- **Permissions Enforcement System**:
+  - Permission checks enforced across all controllers (Viewer, Operator, Admin)
+  - `PermissionsHelper` module for view-level permission checks
+  - Permission helper methods in `ApplicationController` for consistent authorization
+  - All UI buttons and actions respect permission levels
+  - Console APIs (`Registry.enable/disable/drop/re_execute`, `SQL::Executor.execute`) check permissions
+  - Permission errors raise `PermissionError` with clear messages
+  - Configurable permission checker via `permission_checker` configuration option
+
+- **Enhanced Error Handling System**:
+  - Comprehensive error hierarchy with base `Error` class and specialized error types
+  - Error classes: `PermissionError`, `KillSwitchError`, `DriftError`, `ValidationError`, `ExecutionError`, `UnsafeMigrationError`, `NotFoundError`
+  - Error codes for programmatic handling (e.g., `PERMISSION_DENIED`, `KILL_SWITCH_ACTIVE`, `DRIFT_DETECTED`)
+  - Standardized error messages with recovery suggestions
+  - Enhanced error display in UI with user-friendly formatting
+  - Context information included in all errors for better debugging
+  - Error handling helpers in `ApplicationController` for consistent error formatting
+
+- **Comprehensive Documentation**:
+  - New `ui-guide.md` - Quick start guide for web interface
+  - New `permissions.md` - Complete guide to configuring and using permissions
+  - New `audit-trail.md` - Guide to viewing and exporting audit logs
+  - New `troubleshooting.md` - Common issues and solutions with error code reference
+  - Updated documentation index with links to all new guides
+
+- **Audit Log UI**:
+  - Web interface for viewing audit log entries (`/audit_logs`)
+  - Filterable by trigger name, operation, status, and environment
+  - Sortable by date (ascending/descending)
+  - Pagination support (default 50 entries per page, max 200)
+  - CSV export functionality with applied filters
+  - Comprehensive view showing operation details, actor information, status, and error messages
+  - Links to trigger detail pages from audit log entries
+  - Navigation menu integration
+
+- **Enhanced Database Tables & Triggers Page**:
+  - Pagination support for tables list (default 20 per page, configurable up to 100)
+  - Filter functionality to view:
+    - All tables
+    - Tables with triggers only
+    - Tables without triggers only
+  - Enhanced statistics dashboard showing:
+    - Count of tables with triggers
+    - Count of tables without triggers
+    - Total tables count
+  - Filter controls with visual indicators for active filter
+  - Pagination controls preserve filter selection when navigating pages
+  - Context-aware empty state messages based on selected filter
+
+### Changed
+- **Code Organization**: Refactored `ApplicationController` to use concerns instead of inline methods
+  - Reduced code duplication across controllers
+  - Improved separation of concerns
+  - Better testability and maintainability
+
+- **Service Object Patterns**: Standardized service object patterns across all service classes
+  - All service objects follow consistent class method patterns
+  - Consistent stateless service object conventions
+
+- **Goal.md**: Updated to reflect actual implementation status
+  - Added technical notes documenting improvements
+  - Updated console API section with all implemented methods
+  - Documented code organization improvements
+
+- Dashboard default sorting changed to `installed_at` (most recent first) instead of `created_at`
+- Trigger detail page breadcrumbs improved navigation flow
+- All trigger action buttons use consistent styling and permission checks
+
+### Fixed
+- Actor tracking now properly passed through all operation methods
+- Improved error handling with audit log integration
+
+### Security
+- All operations now tracked in audit log for compliance and debugging
+- Actor information captured for all operations (UI, Console, CLI)
+- Complete state change tracking for audit trail
+- Permission enforcement ensures only authorized users can perform operations
+- Permission checks enforced at controller, API, and view levels
+
 ## [1.2.0] - 2026-01-02
 
 ### Added

data/COVERAGE.md

@@ -1,8 +1,8 @@
 # Code Coverage Report
 
-**Total Coverage: 95.33%**
+**Total Coverage: 95.99%**
 
-Covered: 2040 / 2140 lines
+Covered: 2319 / 2416 lines
 
 ---
 
@@ -10,6 +10,9 @@ Covered: 2040 / 2140 lines
 
 | File | Coverage | Covered Lines | Missed Lines | Total Lines |
 |------|----------|---------------|--------------|-------------|
+| `lib/pg_sql_triggers/drift.rb` | 100.0% ✅ | 13 | 0 | 13 |
+| `lib/pg_sql_triggers/drift/db_queries.rb` | 100.0% ✅ | 24 | 0 | 24 |
+| `lib/pg_sql_triggers/dsl.rb` | 100.0% ✅ | 9 | 0 | 9 |
 | `lib/pg_sql_triggers/dsl/trigger_definition.rb` | 100.0% ✅ | 37 | 0 | 37 |
 | `lib/pg_sql_triggers/generator.rb` | 100.0% ✅ | 4 | 0 | 4 |
 | `lib/pg_sql_triggers/generator/form.rb` | 100.0% ✅ | 36 | 0 | 36 |
@@ -20,41 +23,45 @@ Covered: 2040 / 2140 lines
 | `lib/pg_sql_triggers/migrator/pre_apply_diff_reporter.rb` | 100.0% ✅ | 75 | 0 | 75 |
 | `lib/pg_sql_triggers/migrator/safety_validator.rb` | 100.0% ✅ | 110 | 0 | 110 |
 | `lib/pg_sql_triggers/permissions.rb` | 100.0% ✅ | 11 | 0 | 11 |
-| `lib/pg_sql_triggers/permissions/checker.rb` | 100.0% ✅ | 15 | 0 | 15 |
-| `lib/pg_sql_triggers/registry.rb` | 100.0% ✅ | 41 | 0 | 41 |
+| `lib/pg_sql_triggers/permissions/checker.rb` | 100.0% ✅ | 17 | 0 | 17 |
 | `lib/pg_sql_triggers/registry/validator.rb` | 100.0% ✅ | 5 | 0 | 5 |
 | `lib/pg_sql_triggers/sql/capsule.rb` | 100.0% ✅ | 28 | 0 | 28 |
 | `lib/pg_sql_triggers/sql/executor.rb` | 100.0% ✅ | 63 | 0 | 63 |
 | `lib/pg_sql_triggers/testing.rb` | 100.0% ✅ | 6 | 0 | 6 |
 | `lib/pg_sql_triggers/testing/syntax_validator.rb` | 100.0% ✅ | 58 | 0 | 58 |
 | `lib/pg_sql_triggers/testing/dry_run.rb` | 100.0% ✅ | 24 | 0 | 24 |
-| `config/initializers/pg_sql_triggers.rb` | 100.0% ✅ | 10 | 0 | 10 |
+| `app/models/pg_sql_triggers/audit_log.rb` | 100.0% ✅ | 28 | 0 | 28 |
+| `app/models/pg_sql_triggers/trigger_registry.rb` | 100.0% ✅ | 176 | 0 | 176 |
+| `app/controllers/concerns/pg_sql_triggers/error_handling.rb` | 100.0% ✅ | 19 | 0 | 19 |
+| `app/controllers/concerns/pg_sql_triggers/kill_switch_protection.rb` | 100.0% ✅ | 17 | 0 | 17 |
 | `app/models/pg_sql_triggers/application_record.rb` | 100.0% ✅ | 3 | 0 | 3 |
-| `app/controllers/pg_sql_triggers/application_controller.rb` | 100.0% ✅ | 28 | 0 | 28 |
-| `lib/pg_sql_triggers.rb` | 100.0% ✅ | 45 | 0 | 45 |
-| `lib/pg_sql_triggers/dsl.rb` | 100.0% ✅ | 9 | 0 | 9 |
-| `lib/pg_sql_triggers/drift/db_queries.rb` | 100.0% ✅ | 24 | 0 | 24 |
-| `lib/pg_sql_triggers/drift.rb` | 100.0% ✅ | 13 | 0 | 13 |
+| `app/controllers/pg_sql_triggers/application_controller.rb` | 100.0% ✅ | 13 | 0 | 13 |
+| `app/helpers/pg_sql_triggers/permissions_helper.rb` | 100.0% ✅ | 16 | 0 | 16 |
+| `app/controllers/pg_sql_triggers/dashboard_controller.rb` | 100.0% ✅ | 26 | 0 | 26 |
+| `config/initializers/pg_sql_triggers.rb` | 100.0% ✅ | 10 | 0 | 10 |
+| `lib/pg_sql_triggers/errors.rb` | 100.0% ✅ | 83 | 0 | 83 |
+| `app/controllers/pg_sql_triggers/triggers_controller.rb` | 100.0% ✅ | 75 | 0 | 75 |
+| `lib/pg_sql_triggers.rb` | 100.0% ✅ | 40 | 0 | 40 |
 | `lib/pg_sql_triggers/migrator/pre_apply_comparator.rb` | 99.19% ✅ | 122 | 1 | 123 |
 | `lib/pg_sql_triggers/drift/detector.rb` | 98.48% ✅ | 65 | 1 | 66 |
-| `app/controllers/pg_sql_triggers/sql_capsules_controller.rb` | 97.18% ✅ | 69 | 2 | 71 |
-| `app/controllers/pg_sql_triggers/dashboard_controller.rb` | 96.67% ✅ | 29 | 1 | 30 |
-| `app/controllers/pg_sql_triggers/triggers_controller.rb` | 96.43% ✅ | 81 | 3 | 84 |
+| `app/controllers/pg_sql_triggers/audit_logs_controller.rb` | 97.73% ✅ | 43 | 1 | 44 |
+| `app/controllers/pg_sql_triggers/sql_capsules_controller.rb` | 97.14% ✅ | 68 | 2 | 70 |
 | `lib/generators/pg_sql_triggers/trigger_migration_generator.rb` | 96.3% ✅ | 26 | 1 | 27 |
-| `lib/pg_sql_triggers/sql/kill_switch.rb` | 96.0% ✅ | 96 | 4 | 100 |
+| `lib/pg_sql_triggers/sql/kill_switch.rb` | 96.04% ✅ | 97 | 4 | 101 |
 | `lib/pg_sql_triggers/migrator.rb` | 95.42% ✅ | 125 | 6 | 131 |
 | `lib/pg_sql_triggers/registry/manager.rb` | 95.08% ✅ | 58 | 3 | 61 |
-| `app/controllers/pg_sql_triggers/tables_controller.rb` | 94.44% ✅ | 17 | 1 | 18 |
+| `app/controllers/pg_sql_triggers/tables_controller.rb` | 94.74% ✅ | 18 | 1 | 19 |
 | `lib/pg_sql_triggers/database_introspection.rb` | 94.29% ✅ | 66 | 4 | 70 |
 | `lib/pg_sql_triggers/drift/reporter.rb` | 94.12% ✅ | 96 | 6 | 102 |
 | `lib/pg_sql_triggers/engine.rb` | 92.86% ✅ | 13 | 1 | 14 |
-| `app/models/pg_sql_triggers/trigger_registry.rb` | 92.25% ✅ | 119 | 10 | 129 |
 | `lib/pg_sql_triggers/testing/safe_executor.rb` | 91.89% ✅ | 34 | 3 | 37 |
+| `lib/pg_sql_triggers/registry.rb` | 91.84% ✅ | 45 | 4 | 49 |
 | `app/controllers/pg_sql_triggers/generator_controller.rb` | 91.49% ✅ | 86 | 8 | 94 |
 | `lib/pg_sql_triggers/sql.rb` | 90.91% ✅ | 10 | 1 | 11 |
-| `lib/pg_sql_triggers/testing/function_tester.rb` | 89.55% ⚠️ | 60 | 7 | 67 |
-| `app/controllers/pg_sql_triggers/migrations_controller.rb` | 81.4% ⚠️ | 70 | 16 | 86 |
-| `config/routes.rb` | 12.5% ❌ | 3 | 21 | 24 |
+| `lib/pg_sql_triggers/testing/function_tester.rb` | 89.71% ⚠️ | 61 | 7 | 68 |
+| `app/controllers/concerns/pg_sql_triggers/permission_checking.rb` | 85.37% ⚠️ | 35 | 6 | 41 |
+| `app/controllers/pg_sql_triggers/migrations_controller.rb` | 82.76% ⚠️ | 72 | 15 | 87 |
+| `config/routes.rb` | 12.0% ❌ | 3 | 22 | 25 |
 
 ---