automagik-forge 0.1.13 → 0.1.14

package/backend/migrations/20250715154859_add_task_templates.sql

@@ -1,25 +0,0 @@
--- Add task templates tables
-CREATE TABLE task_templates (
-    id            BLOB PRIMARY KEY,
-    project_id    BLOB,  -- NULL for global templates
-    title         TEXT NOT NULL,
-    description   TEXT,
-    template_name TEXT NOT NULL,  -- Display name for the template
-    created_at    TEXT NOT NULL DEFAULT (datetime('now', 'subsec')),
-    updated_at    TEXT NOT NULL DEFAULT (datetime('now', 'subsec')),
-    FOREIGN KEY (project_id) REFERENCES projects(id) ON DELETE CASCADE
-);
-
--- Add index for faster queries
-CREATE INDEX idx_task_templates_project_id ON task_templates(project_id);
-
--- Add unique constraints to prevent duplicate template names within same scope
--- For project-specific templates: unique within each project
-CREATE UNIQUE INDEX idx_task_templates_unique_name_project 
-ON task_templates(project_id, template_name) 
-WHERE project_id IS NOT NULL;
-
--- For global templates: unique across all global templates
-CREATE UNIQUE INDEX idx_task_templates_unique_name_global 
-ON task_templates(template_name) 
-WHERE project_id IS NULL;

package/backend/migrations/20250716143725_add_default_templates.sql

@@ -1,174 +0,0 @@
--- Add default global templates
-
--- 1. Bug Analysis template
-INSERT INTO task_templates (
-    id,
-    project_id,
-    title,
-    description,
-    template_name,
-    created_at,
-    updated_at
-) VALUES (
-    randomblob(16),
-    NULL, -- Global template
-    'Analyze codebase for potential bugs and issues',
-    'Perform a comprehensive analysis of the project codebase to identify potential bugs, code smells, and areas of improvement.
-
-## Analysis Checklist:
-
-### 1. Static Code Analysis
-- [ ] Run linting tools to identify syntax and style issues
-- [ ] Check for unused variables, imports, and dead code
-- [ ] Identify potential type errors or mismatches
-- [ ] Look for deprecated API usage
-
-### 2. Common Bug Patterns
-- [ ] Check for null/undefined reference errors
-- [ ] Identify potential race conditions
-- [ ] Look for improper error handling
-- [ ] Check for resource leaks (memory, file handles, connections)
-- [ ] Identify potential security vulnerabilities (XSS, SQL injection, etc.)
-
-### 3. Code Quality Issues
-- [ ] Identify overly complex functions (high cyclomatic complexity)
-- [ ] Look for code duplication
-- [ ] Check for missing or inadequate input validation
-- [ ] Identify hardcoded values that should be configurable
-
-### 4. Testing Gaps
-- [ ] Identify untested code paths
-- [ ] Check for missing edge case tests
-- [ ] Look for inadequate error scenario testing
-
-### 5. Performance Concerns
-- [ ] Identify potential performance bottlenecks
-- [ ] Check for inefficient algorithms or data structures
-- [ ] Look for unnecessary database queries or API calls
-
-## Deliverables:
-1. Prioritized list of identified issues
-2. Recommendations for fixes
-3. Estimated effort for addressing each issue',
-    'Bug Analysis',
-    datetime('now', 'subsec'),
-    datetime('now', 'subsec')
-);
-
--- 2. Unit Test template
-INSERT INTO task_templates (
-    id,
-    project_id,
-    title,
-    description,
-    template_name,
-    created_at,
-    updated_at
-) VALUES (
-    randomblob(16),
-    NULL, -- Global template
-    'Add unit tests for [component/function]',
-    'Write unit tests to improve code coverage and ensure reliability.
-
-## Unit Testing Checklist
-
-### 1. Identify What to Test
-- [ ] Run coverage report to find untested functions
-- [ ] List the specific functions/methods to test
-- [ ] Note current coverage percentage
-
-### 2. Write Tests
-- [ ] Test the happy path (expected behavior)
-- [ ] Test edge cases (empty inputs, boundaries)
-- [ ] Test error cases (invalid inputs, exceptions)
-- [ ] Mock external dependencies
-- [ ] Use descriptive test names
-
-### 3. Test Quality
-- [ ] Each test focuses on one behavior
-- [ ] Tests can run independently
-- [ ] No hardcoded values that might change
-- [ ] Clear assertions that verify the behavior
-
-## Examples to Cover:
-- Normal inputs → Expected outputs
-- Empty/null inputs → Proper handling
-- Invalid inputs → Error cases
-- Boundary values → Edge case behavior
-
-## Goal
-Achieve at least 80% coverage for the target component
-
-## Deliverables
-1. New test file(s) with comprehensive unit tests
-2. Updated coverage report
-3. All tests passing',
-    'Add Unit Tests',
-    datetime('now', 'subsec'),
-    datetime('now', 'subsec')
-);
-
--- 3. Code Refactoring template
-INSERT INTO task_templates (
-    id,
-    project_id,
-    title,
-    description,
-    template_name,
-    created_at,
-    updated_at
-) VALUES (
-    randomblob(16),
-    NULL, -- Global template
-    'Refactor [component/module] for better maintainability',
-    'Improve code structure and maintainability without changing functionality.
-
-## Refactoring Checklist
-
-### 1. Identify Refactoring Targets
-- [ ] Run code analysis tools (linters, complexity analyzers)
-- [ ] Identify code smells (long methods, duplicate code, large classes)
-- [ ] Check for outdated patterns or deprecated approaches
-- [ ] Review areas with frequent bugs or changes
-
-### 2. Plan the Refactoring
-- [ ] Define clear goals (what to improve and why)
-- [ ] Ensure tests exist for current functionality
-- [ ] Create a backup branch
-- [ ] Break down into small, safe steps
-
-### 3. Common Refactoring Actions
-- [ ] Extract methods from long functions
-- [ ] Remove duplicate code (DRY principle)
-- [ ] Rename variables/functions for clarity
-- [ ] Simplify complex conditionals
-- [ ] Extract constants from magic numbers/strings
-- [ ] Group related functionality into modules
-- [ ] Remove dead code
-
-### 4. Maintain Functionality
-- [ ] Run tests after each change
-- [ ] Keep changes small and incremental
-- [ ] Commit frequently with clear messages
-- [ ] Verify no behavior has changed
-
-### 5. Code Quality Improvements
-- [ ] Apply consistent formatting
-- [ ] Update to modern syntax/features
-- [ ] Improve error handling
-- [ ] Add type annotations (if applicable)
-
-## Success Criteria
-- All tests still pass
-- Code is more readable and maintainable
-- No new bugs introduced
-- Performance not degraded
-
-## Deliverables
-1. Refactored code with improved structure
-2. All tests passing
-3. Brief summary of changes made',
-    'Code Refactoring',
-    datetime('now', 'subsec'),
-    datetime('now', 'subsec')
-);

package/backend/migrations/20250716161432_update_executor_names_to_kebab_case.sql

@@ -1,20 +0,0 @@
--- Migration to update executor type names from snake_case/camelCase to kebab-case
--- This handles the change from charmopencode -> charm-opencode and setup_script -> setup-script
-
--- Update task_attempts.executor column
-UPDATE task_attempts 
-SET executor = 'charm-opencode' 
-WHERE executor = 'charmopencode';
-
-UPDATE task_attempts 
-SET executor = 'setup-script' 
-WHERE executor = 'setup_script';
-
--- Update execution_processes.executor_type column
-UPDATE execution_processes 
-SET executor_type = 'charm-opencode' 
-WHERE executor_type = 'charmopencode';
-
-UPDATE execution_processes 
-SET executor_type = 'setup-script' 
-WHERE executor_type = 'setup_script';

package/backend/migrations/20250716170000_add_parent_task_to_tasks.sql

@@ -1,7 +0,0 @@
-PRAGMA foreign_keys = ON;
-
--- Add parent_task_attempt column to tasks table
-ALTER TABLE tasks ADD COLUMN parent_task_attempt BLOB REFERENCES task_attempts(id);
-
--- Create index for parent_task_attempt lookups
-CREATE INDEX idx_tasks_parent_task_attempt ON tasks(parent_task_attempt);

package/backend/migrations/20250717000000_drop_task_attempt_activities.sql

@@ -1,9 +0,0 @@
--- Migration to drop task_attempt_activities table
--- This removes the task attempt activity tracking functionality
-
--- Drop indexes first
-DROP INDEX IF EXISTS idx_task_attempt_activities_execution_process_id;
-DROP INDEX IF EXISTS idx_task_attempt_activities_created_at;
-
--- Drop the table
-DROP TABLE IF EXISTS task_attempt_activities;

package/backend/migrations/20250719000000_add_cleanup_script_to_projects.sql

@@ -1,2 +0,0 @@
--- Add cleanup_script column to projects table
-ALTER TABLE projects ADD COLUMN cleanup_script TEXT;

package/backend/migrations/20250720000000_add_cleanupscript_to_process_type_constraint.sql

@@ -1,25 +0,0 @@
--- 1. Add the replacement column with the wider CHECK
-ALTER TABLE execution_processes
-  ADD COLUMN process_type_new TEXT NOT NULL DEFAULT 'setupscript'
-    CHECK (process_type_new IN ('setupscript',
-                                'cleanupscript',   -- new value 🎉
-                                'codingagent',
-                                'devserver'));
-
--- 2. Copy existing values across
-UPDATE execution_processes
-  SET process_type_new = process_type;
-
--- 3. Drop any indexes that mention the old column
-DROP INDEX IF EXISTS idx_execution_processes_type;
-
--- 4. Remove the old column (requires 3.35+)
-ALTER TABLE execution_processes DROP COLUMN process_type;
-
--- 5. Rename the new column back to the canonical name
-ALTER TABLE execution_processes
-  RENAME COLUMN process_type_new TO process_type;
-
--- 6. Re-create the index
-CREATE INDEX idx_execution_processes_type
-        ON execution_processes(process_type);

package/backend/migrations/20250723000000_add_wish_to_tasks.sql

@@ -1,7 +0,0 @@
-PRAGMA foreign_keys = ON;
-
--- Add wish_id column to tasks table as required field
-ALTER TABLE tasks ADD COLUMN wish_id TEXT NOT NULL DEFAULT '';
-
--- Create index for wish_id lookups
-CREATE INDEX idx_tasks_wish_id ON tasks(wish_id);

package/backend/migrations/20250724000000_remove_unique_wish_constraint.sql

@@ -1,5 +0,0 @@
-PRAGMA foreign_keys = ON;
-
--- Remove the unique constraint index for wish_id that was incorrectly added
--- wish_id is meant for grouping tasks, not uniqueness
-DROP INDEX IF EXISTS unique_wish_per_project;

package/backend/scripts/toast-notification.ps1

@@ -1,23 +0,0 @@
-param(
-    [Parameter(Mandatory=$true)]
-    [string]$Title,
-    
-    [Parameter(Mandatory=$true)]
-    [string]$Message,
-    
-    [Parameter(Mandatory=$false)]
-    [string]$AppName = "Automagik Forge"
-)
-
-[Windows.UI.Notifications.ToastNotificationManager, Windows.UI.Notifications, ContentType = WindowsRuntime] | Out-Null
-$Template = [Windows.UI.Notifications.ToastNotificationManager]::GetTemplateContent([Windows.UI.Notifications.ToastTemplateType]::ToastText02)
-$RawXml = [xml] $Template.GetXml()
-($RawXml.toast.visual.binding.text|where {$_.id -eq "1"}).AppendChild($RawXml.CreateTextNode($Title)) | Out-Null
-($RawXml.toast.visual.binding.text|where {$_.id -eq "2"}).AppendChild($RawXml.CreateTextNode($Message)) | Out-Null
-$SerializedXml = New-Object Windows.Data.Xml.Dom.XmlDocument
-$SerializedXml.LoadXml($RawXml.OuterXml)
-$Toast = [Windows.UI.Notifications.ToastNotification]::new($SerializedXml)
-$Toast.Tag = $AppName
-$Toast.Group = $AppName
-$Notifier = [Windows.UI.Notifications.ToastNotificationManager]::CreateToastNotifier($AppName)
-$Notifier.Show($Toast)

package/backend/src/app_state.rs

@@ -1,218 +0,0 @@
-use std::{collections::HashMap, sync::Arc, time::Duration};
-
-#[cfg(unix)]
-use nix::{sys::signal::Signal, unistd::Pid};
-use tokio::sync::{Mutex, RwLock as TokioRwLock};
-use uuid::Uuid;
-
-use crate::services::{generate_user_id, AnalyticsConfig, AnalyticsService};
-
-#[derive(Debug)]
-pub enum ExecutionType {
-    SetupScript,
-    CleanupScript,
-    CodingAgent,
-    DevServer,
-}
-
-#[derive(Debug)]
-pub struct RunningExecution {
-    pub task_attempt_id: Uuid,
-    pub _execution_type: ExecutionType,
-    pub child: command_group::AsyncGroupChild,
-}
-
-#[derive(Debug, Clone)]
-pub struct AppState {
-    running_executions: Arc<Mutex<HashMap<Uuid, RunningExecution>>>,
-    pub db_pool: sqlx::SqlitePool,
-    config: Arc<tokio::sync::RwLock<crate::models::config::Config>>,
-    pub analytics: Arc<TokioRwLock<AnalyticsService>>,
-    user_id: String,
-}
-
-impl AppState {
-    pub async fn new(
-        db_pool: sqlx::SqlitePool,
-        config: Arc<tokio::sync::RwLock<crate::models::config::Config>>,
-    ) -> Self {
-        // Initialize analytics with user preferences
-        let user_enabled = {
-            let config_guard = config.read().await;
-            config_guard.analytics_enabled.unwrap_or(true)
-        };
-
-        let analytics_config = AnalyticsConfig::new(user_enabled);
-        let analytics = Arc::new(TokioRwLock::new(AnalyticsService::new(analytics_config)));
-
-        Self {
-            running_executions: Arc::new(Mutex::new(HashMap::new())),
-            db_pool,
-            config,
-            analytics,
-            user_id: generate_user_id(),
-        }
-    }
-
-    pub async fn update_analytics_config(&self, user_enabled: bool) {
-        // Check if analytics was disabled before this update
-        let was_analytics_disabled = {
-            let analytics = self.analytics.read().await;
-            !analytics.is_enabled()
-        };
-
-        let new_config = AnalyticsConfig::new(user_enabled);
-        let new_service = AnalyticsService::new(new_config);
-        let mut analytics = self.analytics.write().await;
-        *analytics = new_service;
-
-        // If analytics was disabled and is now enabled, fire a session_start event
-        if was_analytics_disabled && analytics.is_enabled() {
-            analytics.track_event(&self.user_id, "session_start", None);
-        }
-    }
-
-    // Running executions getters
-    pub async fn has_running_execution(&self, attempt_id: Uuid) -> bool {
-        let executions = self.running_executions.lock().await;
-        executions
-            .values()
-            .any(|exec| exec.task_attempt_id == attempt_id)
-    }
-
-    pub async fn get_running_executions_for_monitor(&self) -> Vec<(Uuid, Uuid, bool, Option<i64>)> {
-        let mut executions = self.running_executions.lock().await;
-        let mut completed_executions = Vec::new();
-
-        for (execution_id, running_exec) in executions.iter_mut() {
-            match running_exec.child.try_wait() {
-                Ok(Some(status)) => {
-                    let success = status.success();
-                    let exit_code = status.code().map(|c| c as i64);
-                    completed_executions.push((
-                        *execution_id,
-                        running_exec.task_attempt_id,
-                        success,
-                        exit_code,
-                    ));
-                }
-                Ok(None) => {
-                    // Still running
-                }
-                Err(e) => {
-                    tracing::error!("Error checking process status: {}", e);
-                    completed_executions.push((
-                        *execution_id,
-                        running_exec.task_attempt_id,
-                        false,
-                        None,
-                    ));
-                }
-            }
-        }
-
-        // Remove completed executions from the map
-        for (execution_id, _, _, _) in &completed_executions {
-            executions.remove(execution_id);
-        }
-
-        completed_executions
-    }
-
-    // Running executions setters
-    pub async fn add_running_execution(&self, execution_id: Uuid, execution: RunningExecution) {
-        let mut executions = self.running_executions.lock().await;
-        executions.insert(execution_id, execution);
-    }
-
-    pub async fn stop_running_execution_by_id(
-        &self,
-        execution_id: Uuid,
-    ) -> Result<bool, Box<dyn std::error::Error + Send + Sync>> {
-        let mut executions = self.running_executions.lock().await;
-        let Some(exec) = executions.get_mut(&execution_id) else {
-            return Ok(false);
-        };
-
-        // hit the whole process group, not just the leader
-        #[cfg(unix)]
-        {
-            use nix::{sys::signal::killpg, unistd::getpgid};
-
-            let pgid = getpgid(Some(Pid::from_raw(exec.child.id().unwrap() as i32)))?;
-            for sig in [Signal::SIGINT, Signal::SIGTERM, Signal::SIGKILL] {
-                killpg(pgid, sig)?;
-                tokio::time::sleep(Duration::from_secs(2)).await;
-                if exec.child.try_wait()?.is_some() {
-                    break; // gone!
-                }
-            }
-        }
-
-        // final fallback – command_group already targets the group
-        exec.child.kill().await.ok();
-        exec.child.wait().await.ok(); // reap
-
-        // only NOW remove it
-        executions.remove(&execution_id);
-        Ok(true)
-    }
-
-    // Config getters
-    pub async fn get_sound_alerts_enabled(&self) -> bool {
-        let config = self.config.read().await;
-        config.sound_alerts
-    }
-
-    pub async fn get_push_notifications_enabled(&self) -> bool {
-        let config = self.config.read().await;
-        config.push_notifications
-    }
-
-    pub async fn get_sound_file(&self) -> crate::models::config::SoundFile {
-        let config = self.config.read().await;
-        config.sound_file.clone()
-    }
-
-    pub fn get_config(&self) -> &Arc<tokio::sync::RwLock<crate::models::config::Config>> {
-        &self.config
-    }
-
-    pub async fn track_analytics_event(
-        &self,
-        event_name: &str,
-        properties: Option<serde_json::Value>,
-    ) {
-        let analytics = self.analytics.read().await;
-        if analytics.is_enabled() {
-            analytics.track_event(&self.user_id, event_name, properties);
-        } else {
-            tracing::debug!("Analytics disabled, skipping event: {}", event_name);
-        }
-    }
-
-    pub async fn update_sentry_scope(&self) {
-        let config = self.get_config().read().await;
-        let username = config.github.username.clone();
-        let email = config.github.primary_email.clone();
-        drop(config);
-
-        let sentry_user = if username.is_some() || email.is_some() {
-            sentry::User {
-                id: Some(self.user_id.clone()),
-                username,
-                email,
-                ..Default::default()
-            }
-        } else {
-            sentry::User {
-                id: Some(self.user_id.clone()),
-                ..Default::default()
-            }
-        };
-
-        sentry::configure_scope(|scope| {
-            scope.set_user(Some(sentry_user));
-        });
-    }
-}