devchain-cli 0.7.2 → 0.8.1

package/dist/drizzle/0027_legal_malice.sql

@@ -0,0 +1,17 @@
+CREATE TABLE `profile_provider_configs` (
+	`id` text PRIMARY KEY NOT NULL,
+	`profile_id` text NOT NULL,
+	`provider_id` text NOT NULL,
+	`options` text,
+	`env` text,
+	`created_at` text NOT NULL,
+	`updated_at` text NOT NULL,
+	FOREIGN KEY (`profile_id`) REFERENCES `agent_profiles`(`id`) ON UPDATE no action ON DELETE cascade,
+	FOREIGN KEY (`provider_id`) REFERENCES `providers`(`id`) ON UPDATE no action ON DELETE no action
+);
+--> statement-breakpoint
+CREATE INDEX `profile_provider_configs_profile_id_idx` ON `profile_provider_configs` (`profile_id`);
+--> statement-breakpoint
+CREATE INDEX `profile_provider_configs_provider_id_idx` ON `profile_provider_configs` (`provider_id`);
+--> statement-breakpoint
+ALTER TABLE `agents` ADD `provider_config_id` text REFERENCES profile_provider_configs(id);

package/dist/drizzle/0028_populate_provider_configs.sql

@@ -0,0 +1,19 @@
+-- Phase 2 Migration: Populate profile_provider_configs from existing profiles
+-- For each profile with a providerId, create a corresponding provider config record
+
+INSERT INTO profile_provider_configs (id, profile_id, provider_id, options, env, created_at, updated_at)
+SELECT
+    lower(hex(randomblob(4))) || '-' || lower(hex(randomblob(2))) || '-4' || substr(lower(hex(randomblob(2))),2) || '-' || substr('89ab',abs(random()) % 4 + 1, 1) || substr(lower(hex(randomblob(2))),2) || '-' || lower(hex(randomblob(6))) as id,
+    id as profile_id,
+    provider_id,
+    options,
+    NULL as env,
+    datetime('now') as created_at,
+    datetime('now') as updated_at
+FROM agent_profiles
+WHERE provider_id IS NOT NULL
+  AND NOT EXISTS (
+    SELECT 1 FROM profile_provider_configs ppc
+    WHERE ppc.profile_id = agent_profiles.id
+      AND ppc.provider_id = agent_profiles.provider_id
+  );

package/dist/drizzle/0029_merge_profiles_by_family_slug.sql

@@ -0,0 +1,116 @@
+-- Phase 2 Migration: Auto-merge profiles by familySlug
+-- For each (projectId, familySlug) group with multiple profiles:
+-- 1. Keep oldest as canonical
+-- 2. Capture agent→providerId mapping (BEFORE profile deletion)
+-- 3. Move provider configs to canonical
+-- 4. Set agents.provider_config_id based on original providerId
+-- 5. Update agents to point to canonical profile
+-- 6. Delete merged profiles
+
+-- Step 1: Create temp table identifying canonical profiles (oldest per group)
+CREATE TEMP TABLE canonical_profiles AS
+WITH ranked AS (
+    SELECT
+        id,
+        project_id,
+        family_slug,
+        created_at,
+        ROW_NUMBER() OVER (PARTITION BY project_id, family_slug ORDER BY created_at ASC) as rn
+    FROM agent_profiles
+    WHERE family_slug IS NOT NULL
+)
+SELECT id as canonical_id, project_id, family_slug
+FROM ranked
+WHERE rn = 1
+AND EXISTS (
+    SELECT 1 FROM ranked r2
+    WHERE r2.project_id = ranked.project_id
+    AND r2.family_slug = ranked.family_slug
+    AND r2.rn > 1
+);
+
+--> statement-breakpoint
+
+-- Step 2: Create temp table with profiles to merge (non-canonical in groups)
+CREATE TEMP TABLE profiles_to_merge AS
+WITH ranked AS (
+    SELECT
+        id,
+        project_id,
+        family_slug,
+        ROW_NUMBER() OVER (PARTITION BY project_id, family_slug ORDER BY created_at ASC) as rn
+    FROM agent_profiles
+    WHERE family_slug IS NOT NULL
+)
+SELECT ranked.id as merge_id, cp.canonical_id
+FROM ranked
+JOIN canonical_profiles cp
+    ON ranked.project_id = cp.project_id
+    AND ranked.family_slug = cp.family_slug
+WHERE ranked.rn > 1;
+
+--> statement-breakpoint
+
+-- Step 3: Capture agent→providerId mapping BEFORE profiles are merged/deleted
+-- This preserves which provider each agent was originally using
+CREATE TEMP TABLE agent_original_provider AS
+SELECT a.id as agent_id, ap.provider_id
+FROM agents a
+JOIN agent_profiles ap ON a.profile_id = ap.id
+WHERE a.profile_id IN (SELECT merge_id FROM profiles_to_merge);
+
+--> statement-breakpoint
+
+-- Step 4: Update provider configs - move from merged profiles to canonical
+UPDATE profile_provider_configs
+SET profile_id = (
+    SELECT canonical_id FROM profiles_to_merge ptm
+    WHERE ptm.merge_id = profile_provider_configs.profile_id
+)
+WHERE profile_id IN (SELECT merge_id FROM profiles_to_merge);
+
+--> statement-breakpoint
+
+-- Step 5: Set agents.provider_config_id based on original providerId
+-- Match each agent to the config in their new canonical profile that has the same providerId
+UPDATE agents
+SET provider_config_id = (
+    SELECT ppc.id
+    FROM agent_original_provider aop
+    JOIN profiles_to_merge ptm ON agents.profile_id = ptm.merge_id
+    JOIN profile_provider_configs ppc ON ppc.profile_id = ptm.canonical_id
+    WHERE aop.agent_id = agents.id
+    AND ppc.provider_id = aop.provider_id
+    LIMIT 1
+)
+WHERE profile_id IN (SELECT merge_id FROM profiles_to_merge)
+AND provider_config_id IS NULL;
+
+--> statement-breakpoint
+
+-- Step 6: Update agents - point to canonical profile
+UPDATE agents
+SET profile_id = (
+    SELECT canonical_id FROM profiles_to_merge ptm
+    WHERE ptm.merge_id = agents.profile_id
+)
+WHERE profile_id IN (SELECT merge_id FROM profiles_to_merge);
+
+--> statement-breakpoint
+
+-- Step 7: Delete merged profiles
+DELETE FROM agent_profiles
+WHERE id IN (SELECT merge_id FROM profiles_to_merge);
+
+--> statement-breakpoint
+
+-- Step 8: Cleanup temp tables
+DROP TABLE agent_original_provider;
+
+--> statement-breakpoint
+
+DROP TABLE profiles_to_merge;
+
+--> statement-breakpoint
+
+DROP TABLE canonical_profiles;

package/dist/drizzle/0030_set_agents_provider_config_id.sql

@@ -0,0 +1,14 @@
+-- Phase 2 Migration: Set agents.providerConfigId
+-- For agents not handled by 0029 (non-merged profiles), set providerConfigId
+-- Match config by profile's provider_id (not by creation order)
+
+UPDATE agents
+SET provider_config_id = (
+    SELECT ppc.id
+    FROM profile_provider_configs ppc
+    JOIN agent_profiles ap ON ap.id = agents.profile_id
+    WHERE ppc.profile_id = agents.profile_id
+    AND ppc.provider_id = ap.provider_id
+    LIMIT 1
+)
+WHERE provider_config_id IS NULL;

package/dist/drizzle/0031_remove_legacy_profile_columns.sql

@@ -0,0 +1,68 @@
+-- Phase 4: Remove legacy providerId and options columns from agent_profiles
+-- Provider configuration now lives in profile_provider_configs table
+-- This migration:
+--   1. Rebuilds agent_profiles without providerId and options columns
+--   2. Replaces family_provider_unique index with family_unique index
+
+-- Note: foreign_keys are disabled in db.provider.ts before running migrations
+-- (PRAGMA foreign_keys=OFF has no effect inside a transaction)
+
+-- Step 1: Create new table without providerId and options columns
+CREATE TABLE `__new_agent_profiles` (
+	`id` text PRIMARY KEY NOT NULL,
+	`project_id` text,
+	`name` text NOT NULL,
+	`family_slug` text,
+	`system_prompt` text,
+	`instructions` text,
+	`temperature` integer,
+	`max_tokens` integer,
+	`created_at` text NOT NULL,
+	`updated_at` text NOT NULL,
+	FOREIGN KEY (`project_id`) REFERENCES `projects`(`id`) ON UPDATE no action ON DELETE cascade
+);
+--> statement-breakpoint
+
+-- Step 2: Copy data (excluding providerId and options)
+INSERT INTO `__new_agent_profiles` (
+	`id`,
+	`project_id`,
+	`name`,
+	`family_slug`,
+	`system_prompt`,
+	`instructions`,
+	`temperature`,
+	`max_tokens`,
+	`created_at`,
+	`updated_at`
+)
+SELECT
+	`id`,
+	`project_id`,
+	`name`,
+	`family_slug`,
+	`system_prompt`,
+	`instructions`,
+	`temperature`,
+	`max_tokens`,
+	`created_at`,
+	`updated_at`
+FROM `agent_profiles`;
+--> statement-breakpoint
+
+-- Step 3: Drop old table
+DROP TABLE `agent_profiles`;
+--> statement-breakpoint
+
+-- Step 4: Rename new table
+ALTER TABLE `__new_agent_profiles` RENAME TO `agent_profiles`;
+--> statement-breakpoint
+
+-- Step 5: Recreate indexes
+-- Keep: unique constraint on (project_id, name)
+CREATE UNIQUE INDEX `agent_profiles_project_name_unique` ON `agent_profiles` (`project_id`, `name`);
+--> statement-breakpoint
+
+-- New: unique constraint on (project_id, family_slug) - profiles are now provider-independent
+-- Note: Replaces old agent_profiles_family_provider_unique which included provider_id
+CREATE UNIQUE INDEX `agent_profiles_family_unique` ON `agent_profiles` (`project_id`, `family_slug`) WHERE `family_slug` IS NOT NULL;

package/dist/drizzle/0032_agents_provider_config_not_null.sql

@@ -0,0 +1,52 @@
+-- Phase 4: Make agents.providerConfigId NOT NULL
+-- Prerequisites:
+--   - Migration 0030 must have run to set all providerConfigId values
+--   - No agents should have NULL providerConfigId
+-- This migration will FAIL if any agent has NULL providerConfigId
+-- (the INSERT into the NOT NULL column will fail naturally)
+
+-- Rebuild agents table with NOT NULL constraint
+-- Note: foreign_keys are disabled in db.provider.ts before running migrations
+-- (PRAGMA foreign_keys=OFF has no effect inside a transaction)
+
+CREATE TABLE `__new_agents` (
+	`id` text PRIMARY KEY NOT NULL,
+	`project_id` text NOT NULL,
+	`profile_id` text NOT NULL,
+	`provider_config_id` text NOT NULL,
+	`name` text NOT NULL,
+	`description` text,
+	`created_at` text NOT NULL,
+	`updated_at` text NOT NULL,
+	FOREIGN KEY (`project_id`) REFERENCES `projects`(`id`) ON UPDATE no action ON DELETE cascade,
+	FOREIGN KEY (`profile_id`) REFERENCES `agent_profiles`(`id`) ON UPDATE no action ON DELETE no action,
+	FOREIGN KEY (`provider_config_id`) REFERENCES `profile_provider_configs`(`id`) ON UPDATE no action ON DELETE restrict
+);
+--> statement-breakpoint
+
+INSERT INTO `__new_agents` (
+	`id`,
+	`project_id`,
+	`profile_id`,
+	`provider_config_id`,
+	`name`,
+	`description`,
+	`created_at`,
+	`updated_at`
+)
+SELECT
+	`id`,
+	`project_id`,
+	`profile_id`,
+	`provider_config_id`,
+	`name`,
+	`description`,
+	`created_at`,
+	`updated_at`
+FROM `agents`;
+--> statement-breakpoint
+
+DROP TABLE `agents`;
+--> statement-breakpoint
+
+ALTER TABLE `__new_agents` RENAME TO `agents`;

package/dist/drizzle/0033_profile_provider_configs_name.sql

@@ -0,0 +1,38 @@
+-- Migration: Add name column to profile_provider_configs
+-- This enables multiple configs per provider per profile (GLM use case)
+
+-- Step 1: Add name column with default empty string
+ALTER TABLE profile_provider_configs ADD COLUMN name TEXT NOT NULL DEFAULT '';
+--> statement-breakpoint
+
+-- Step 2: Backfill existing rows with deterministic names (provider_name + order suffix if needed)
+-- Uses a CTE to assign row numbers per profile, then updates with provider-based names
+WITH ranked_configs AS (
+  SELECT
+    ppc.id,
+    p.name AS provider_name,
+    ROW_NUMBER() OVER (
+      PARTITION BY ppc.profile_id, ppc.provider_id
+      ORDER BY ppc.created_at ASC, ppc.id ASC
+    ) AS rn,
+    COUNT(*) OVER (
+      PARTITION BY ppc.profile_id, ppc.provider_id
+    ) AS total_for_provider
+  FROM profile_provider_configs ppc
+  JOIN providers p ON p.id = ppc.provider_id
+)
+UPDATE profile_provider_configs
+SET name = (
+  SELECT
+    CASE
+      WHEN rc.total_for_provider = 1 THEN rc.provider_name
+      ELSE rc.provider_name || '-' || rc.rn
+    END
+  FROM ranked_configs rc
+  WHERE rc.id = profile_provider_configs.id
+)
+WHERE name = '';
+--> statement-breakpoint
+
+-- Step 3: Create unique index on (profile_id, name)
+CREATE UNIQUE INDEX profile_provider_configs_profile_name_idx ON profile_provider_configs(profile_id, name);

package/dist/drizzle/0034_sessions_agent_running_unique.sql

@@ -0,0 +1,30 @@
+-- Migration 0034: Clean up duplicate sessions and add unique index
+-- This migration ensures only one running session per agent exists before adding the constraint
+
+-- Step 1: Clean up any existing duplicate running sessions
+-- For each agent with multiple running sessions, keep exactly one and mark the rest as 'stopped'
+-- Uses window function for deterministic ordering even when started_at timestamps are identical
+UPDATE sessions
+SET status = 'stopped',
+    ended_at = datetime('now'),
+    updated_at = datetime('now')
+WHERE id IN (
+    -- Find all duplicate sessions (rn > 1 means not the first for that agent)
+    SELECT id FROM (
+        SELECT id,
+               ROW_NUMBER() OVER (
+                   PARTITION BY agent_id
+                   ORDER BY started_at DESC, created_at DESC, id DESC
+               ) as rn
+        FROM sessions
+        WHERE status = 'running'
+          AND agent_id IS NOT NULL
+    ) ranked
+    WHERE rn > 1
+);
+--> statement-breakpoint
+
+-- Step 2: Add partial unique index to prevent future duplicates
+-- This is a safety net for the application-level lock (withAgentLock in SessionsService)
+-- to catch any edge cases where concurrent requests might create duplicate sessions
+CREATE UNIQUE INDEX `idx_sessions_agent_running` ON `sessions` (`agent_id`) WHERE status = 'running' AND agent_id IS NOT NULL;

package/dist/drizzle/0035_provider_config_position.sql

@@ -0,0 +1,26 @@
+-- Migration: Add position column to profile_provider_configs
+-- This enables drag-and-drop ordering of provider configs within a profile
+
+-- Step 1: Add position column with default 0
+ALTER TABLE profile_provider_configs ADD COLUMN position INTEGER NOT NULL DEFAULT 0;
+--> statement-breakpoint
+
+-- Step 2: Backfill positions based on createdAt order per profile
+-- Uses CTE with ROW_NUMBER() to assign sequential positions (0, 1, 2, ...)
+WITH ranked AS (
+  SELECT
+    id,
+    ROW_NUMBER() OVER (PARTITION BY profile_id ORDER BY created_at ASC, id ASC) - 1 AS pos
+  FROM profile_provider_configs
+)
+UPDATE profile_provider_configs
+SET position = (
+  SELECT pos
+  FROM ranked
+  WHERE ranked.id = profile_provider_configs.id
+);
+--> statement-breakpoint
+
+-- Step 3: Create unique index on (profile_id, position)
+CREATE UNIQUE INDEX profile_provider_configs_profile_position_idx
+ON profile_provider_configs(profile_id, position);

package/dist/drizzle/PHASE2_MIGRATION_GUIDE.md

@@ -0,0 +1,109 @@
+# Phase 2 Migration Guide: Profile Provider Configs
+
+## Overview
+
+Phase 2 migrations populate the `profile_provider_configs` table and merge duplicate profiles.
+
+**Migrations:**
+- `0028_populate_provider_configs.sql` - Create configs from existing profiles
+- `0029_merge_profiles_by_family_slug.sql` - Merge profiles with same familySlug
+- `0030_set_agents_provider_config_id.sql` - Set agent config references
+
+## Pre-Migration Checklist
+
+### 1. Stop All Sessions
+
+Before running migrations, ensure no active sessions exist:
+
+```bash
+sqlite3 ~/.devchain/devchain.db < apps/local-app/drizzle/migration-guard.sql
+```
+
+If any output is shown, stop the sessions first:
+```bash
+# List running sessions
+sqlite3 ~/.devchain/devchain.db "SELECT id, agent_id FROM sessions WHERE status='running';"
+
+# Stop sessions via UI or terminate tmux sessions
+```
+
+### 2. Create Backup
+
+**CRITICAL:** Always backup before migrating:
+
+```bash
+cp ~/.devchain/devchain.db ~/.devchain/devchain.db.backup-$(date +%Y%m%d-%H%M%S)
+```
+
+### 3. Verify Backup
+
+```bash
+sqlite3 ~/.devchain/devchain.db.backup-* "SELECT COUNT(*) FROM agent_profiles;"
+```
+
+## Running Migrations
+
+```bash
+pnpm --filter local-app db:migrate
+```
+
+Or manually:
+```bash
+sqlite3 ~/.devchain/devchain.db < apps/local-app/drizzle/0028_populate_provider_configs.sql
+sqlite3 ~/.devchain/devchain.db < apps/local-app/drizzle/0029_merge_profiles_by_family_slug.sql
+sqlite3 ~/.devchain/devchain.db < apps/local-app/drizzle/0030_set_agents_provider_config_id.sql
+```
+
+## Post-Migration Verification
+
+```bash
+# Verify configs created
+sqlite3 ~/.devchain/devchain.db "SELECT COUNT(*) FROM profile_provider_configs;"
+
+# Verify no duplicate familySlug groups
+sqlite3 ~/.devchain/devchain.db "SELECT project_id, family_slug, COUNT(*) FROM agent_profiles WHERE family_slug IS NOT NULL GROUP BY project_id, family_slug HAVING COUNT(*) > 1;"
+
+# Verify all agents have config
+sqlite3 ~/.devchain/devchain.db "SELECT COUNT(*) FROM agents WHERE provider_config_id IS NULL;"
+```
+
+## Rollback Procedure
+
+### Recommended: Restore from Backup
+
+```bash
+# Stop the application
+# Restore backup
+cp ~/.devchain/devchain.db.backup-YYYYMMDD-HHMMSS ~/.devchain/devchain.db
+
+# Restart the application
+```
+
+### Alternative: Partial Rollback Script
+
+**WARNING:** The partial rollback script cannot restore deleted profiles from the merge step.
+
+```bash
+sqlite3 ~/.devchain/devchain.db < apps/local-app/drizzle/phase2-rollback.sql
+```
+
+This will:
+1. Clear `agents.provider_config_id` (set to NULL)
+2. Delete all `profile_provider_configs` records
+3. Remove migration tracking entries
+
+**NOT restored:** Profiles deleted during merge (0029) cannot be restored without backup.
+
+## Troubleshooting
+
+### Migration fails with "database is locked"
+- Stop all sessions and the application
+- Ensure no other process is accessing the database
+
+### Agents have NULL providerConfigId after migration
+- Verify profile has a config: `SELECT * FROM profile_provider_configs WHERE profile_id = '<profile_id>';`
+- Re-run migration 0030
+
+### Profile configs missing
+- Check if profile has providerId: `SELECT provider_id FROM agent_profiles WHERE id = '<profile_id>';`
+- Profiles with NULL providerId don't get configs created