feat(executions): add puppet and package execution types

Add support for 'puppet' and 'package' execution types throughout the application.

Backend changes:
- Update database schema with migration to add puppet and package to type constraint
- Update ExecutionResult type definition to include new types
- Modify execution repository and route handlers to support new types

Frontend changes:
- Add color coding for new types (puppet: orange, package: green)
- Add display label mapping (puppet → 'puppet run', package → 'package')
- Update ExecutionsPage component with type badges and labels
- Update type definitions across components (PackageInstallInterface, PuppetRunInterface, TaskRunInterface, RealtimeOutputViewer, NodeDetailPage)

This enables proper tracking and display of Puppet runs and package installations in the execution history.

Author: alvagante

backend/src/bolt/types.ts

@@ -152,7 +152,7 @@ export interface Facts {
  */
 export interface ExecutionResult {
   id: string;
-  type: "command" | "task" | "facts";
+  type: "command" | "task" | "facts" | "puppet" | "package";
   targetNodes: string[];
   action: string;
   parameters?: Record<string, unknown>;

backend/src/database/ExecutionRepository.ts

@@ -27,7 +27,7 @@ interface DbRow {
 /**
  * Execution types
  */
-export type ExecutionType = "command" | "task" | "facts";
+export type ExecutionType = "command" | "task" | "facts" | "puppet" | "package";
 
 /**
  * Execution status

backend/src/database/migrations.sql

@@ -6,3 +6,38 @@ ALTER TABLE executions ADD COLUMN command TEXT;
 
 -- Add expert_mode column if it doesn't exist
 ALTER TABLE executions ADD COLUMN expert_mode INTEGER DEFAULT 0;
+
+-- Migration: Update execution type constraint to include 'puppet' and 'package' types
+-- This migration recreates the executions table with the updated type constraint
+
+-- Step 1: Create new table with updated constraint
+CREATE TABLE IF NOT EXISTS executions_new (
+  id TEXT PRIMARY KEY,
+  type TEXT NOT NULL CHECK(type IN ('command', 'task', 'facts', 'puppet', 'package')),
+  target_nodes TEXT NOT NULL,
+  action TEXT NOT NULL,
+  parameters TEXT,
+  status TEXT NOT NULL CHECK(status IN ('running', 'success', 'failed', 'partial')),
+  started_at TEXT NOT NULL,
+  completed_at TEXT,
+  results TEXT NOT NULL,
+  error TEXT,
+  command TEXT,
+  expert_mode INTEGER DEFAULT 0
+);
+
+-- Step 2: Copy data from old table to new table
+INSERT INTO executions_new SELECT * FROM executions;
+
+-- Step 3: Drop old table
+DROP TABLE executions;
+
+-- Step 4: Rename new table to original name
+ALTER TABLE executions_new RENAME TO executions;
+
+-- Step 5: Recreate indexes
+CREATE INDEX IF NOT EXISTS idx_executions_started ON executions(started_at DESC);
+CREATE INDEX IF NOT EXISTS idx_executions_status ON executions(status);
+CREATE INDEX IF NOT EXISTS idx_executions_type ON executions(type);
+CREATE INDEX IF NOT EXISTS idx_executions_status_started ON executions(status, started_at DESC);
+CREATE INDEX IF NOT EXISTS idx_executions_type_started ON executions(type, started_at DESC);

backend/src/database/schema.sql

@@ -1,7 +1,7 @@
 -- Executions table for storing command and task execution history
 CREATE TABLE IF NOT EXISTS executions (
   id TEXT PRIMARY KEY,
-  type TEXT NOT NULL CHECK(type IN ('command', 'task', 'facts')),
+  type TEXT NOT NULL CHECK(type IN ('command', 'task', 'facts', 'puppet', 'package')),
   target_nodes TEXT NOT NULL,  -- JSON array of target node IDs
   action TEXT NOT NULL,
   parameters TEXT,  -- JSON object of parameters

backend/src/routes/executions.ts

@@ -13,7 +13,7 @@ const ExecutionIdParamSchema = z.object({
 });
 
 const ExecutionFiltersQuerySchema = z.object({
-  type: z.enum(['command', 'task', 'facts']).optional(),
+  type: z.enum(['command', 'task', 'facts', 'puppet', 'package']).optional(),
   status: z.enum(['running', 'success', 'failed', 'partial']).optional(),
   targetNode: z.string().optional(),
   startDate: z.string().datetime().optional(),

backend/src/routes/packages.ts

@@ -96,7 +96,7 @@ export function createPackagesRouter(
 
       // Create initial execution record
       const executionId = await executionRepository.create({
-        type: "task",
+        type: "package",
         targetNodes: [nodeId],
         action: taskName,
         parameters: { packageName, ensure, version, settings },

backend/src/routes/puppet.ts

@@ -70,7 +70,7 @@ export function createPuppetRouter(
 
         // Create initial execution record
         const executionId = await executionRepository.create({
-          type: "task",
+          type: "puppet",
           targetNodes: [nodeId],
           action: "psick::puppet_agent",
           parameters: config,

frontend/src/components/PackageInstallInterface.svelte

@@ -27,7 +27,7 @@
 
   interface ExecutionResult {
     id: string;
-    type: 'command' | 'task' | 'facts';
+    type: 'command' | 'task' | 'facts' | 'puppet' | 'package';
     targetNodes: string[];
     action: string;
     parameters?: Record<string, unknown>;

frontend/src/components/PuppetRunInterface.svelte

@@ -25,7 +25,7 @@
 
   interface ExecutionResult {
     id: string;
-    type: 'command' | 'task' | 'facts';
+    type: 'command' | 'task' | 'facts' | 'puppet' | 'package';
     targetNodes: string[];
     action: string;
     parameters?: Record<string, unknown>;
@@ -296,7 +296,7 @@
           >
             <span
               class="pointer-events-none inline-block h-5 w-5 transform rounded-full bg-white shadow ring-0 transition duration-200 ease-in-out {noop ? 'translate-x-5' : 'translate-x-0'}"
-            />
+            ></span>
           </button>
         </div>
 
@@ -321,7 +321,7 @@
           >
             <span
               class="pointer-events-none inline-block h-5 w-5 transform rounded-full bg-white shadow ring-0 transition duration-200 ease-in-out {noNoop ? 'translate-x-5' : 'translate-x-0'}"
-            />
+            ></span>
           </button>
         </div>
 
@@ -346,7 +346,7 @@
           >
             <span
               class="pointer-events-none inline-block h-5 w-5 transform rounded-full bg-white shadow ring-0 transition duration-200 ease-in-out {debug ? 'translate-x-5' : 'translate-x-0'}"
-            />
+            ></span>
           </button>
         </div>
 
@@ -355,6 +355,7 @@
           type="button"
           class="text-sm text-purple-600 hover:text-purple-700 dark:text-purple-400 dark:hover:text-purple-300"
           onclick={() => showAdvanced = !showAdvanced}
+          aria-label={showAdvanced ? 'Hide advanced options' : 'Show advanced options'}
         >
           {showAdvanced ? 'Hide' : 'Show'} advanced options
         </button>

frontend/src/components/RealtimeOutputViewer.svelte

@@ -15,11 +15,11 @@
 
   // Auto-scroll state
   let autoScroll = $state(true);
-  let stdoutContainer: HTMLElement | null = null;
-  let stderrContainer: HTMLElement | null = null;
+  let stdoutContainer = $state<HTMLElement | null>(null);
+  let stderrContainer = $state<HTMLElement | null>(null);
 
   // Elapsed time tracking
-  let startTime: number | null = null;
+  let startTime = $state<number | null>(null);
   let elapsedTime = $state(0);
   let elapsedInterval: ReturnType<typeof setInterval> | null = null;
 
@@ -557,11 +557,11 @@
   }
 
   /* Dark mode scrollbar */
-  .dark .realtime-output-viewer ::-webkit-scrollbar-thumb {
+  :global(.dark) .realtime-output-viewer ::-webkit-scrollbar-thumb {
     background: rgba(75, 85, 99, 0.5);
   }
 
-  .dark .realtime-output-viewer ::-webkit-scrollbar-thumb:hover {
+  :global(.dark) .realtime-output-viewer ::-webkit-scrollbar-thumb:hover {
     background: rgba(75, 85, 99, 0.7);
   }
 </style>