@shepai/cli 1.55.0 → 1.55.2

package/dist/packages/core/src/infrastructure/persistence/sqlite/mappers/feature.mapper.js

@@ -80,9 +80,9 @@ export function fromDatabase(row) {
         relatedArtifacts: JSON.parse(row.related_artifacts),
         createdAt: new Date(row.created_at),
         updatedAt: new Date(row.updated_at),
-        ...(row.plan !== null && { plan: JSON.parse(row.plan) }),
-        ...(row.agent_run_id !== null && { agentRunId: row.agent_run_id }),
-        ...(row.spec_path !== null && { specPath: row.spec_path }),
+        ...(row.plan != null && { plan: JSON.parse(row.plan) }),
+        ...(row.agent_run_id != null && { agentRunId: row.agent_run_id }),
+        ...(row.spec_path != null && { specPath: row.spec_path }),
         // Assemble workflow flags from flat columns
         push: row.push === 1,
         openPr: row.open_pr === 1,
@@ -91,22 +91,22 @@ export function fromDatabase(row) {
             allowPlan: row.allow_plan === 1,
             allowMerge: row.allow_merge === 1,
         },
-        ...(row.worktree_path !== null && { worktreePath: row.worktree_path }),
+        ...(row.worktree_path != null && { worktreePath: row.worktree_path }),
         // Repository reference
-        ...(row.repository_id !== null && { repositoryId: row.repository_id }),
+        ...(row.repository_id != null && { repositoryId: row.repository_id }),
         // Assemble pr from flat columns (only when pr_url exists)
-        ...(row.pr_url !== null && {
+        ...(row.pr_url != null && {
             pr: {
                 url: row.pr_url,
                 number: row.pr_number,
                 status: row.pr_status,
-                ...(row.commit_hash !== null && { commitHash: row.commit_hash }),
-                ...(row.ci_status !== null && { ciStatus: row.ci_status }),
-                ...(row.ci_fix_attempts !== null && { ciFixAttempts: row.ci_fix_attempts }),
-                ...(row.ci_fix_history !== null && { ciFixHistory: JSON.parse(row.ci_fix_history) }),
+                ...(row.commit_hash != null && { commitHash: row.commit_hash }),
+                ...(row.ci_status != null && { ciStatus: row.ci_status }),
+                ...(row.ci_fix_attempts != null && { ciFixAttempts: row.ci_fix_attempts }),
+                ...(row.ci_fix_history != null && { ciFixHistory: JSON.parse(row.ci_fix_history) }),
             },
         }),
         // Feature dependency
-        ...(row.parent_id !== null && { parentId: row.parent_id }),
+        ...(row.parent_id != null && { parentId: row.parent_id }),
     };
 }

package/dist/packages/core/src/infrastructure/persistence/sqlite/migrations.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"migrations.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/persistence/sqlite/migrations.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAmU3C;;;GAGG;AACH,eAAO,MAAM,qBAAqB,QAA4C,CAAC;AAE/E;;;;;;;GAOG;AACH,wBAAsB,mBAAmB,CAAC,EAAE,EAAE,QAAQ,CAAC,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA4B9E"}
+{"version":3,"file":"migrations.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/persistence/sqlite/migrations.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AA4V3C;;;GAGG;AACH,eAAO,MAAM,qBAAqB,QAA4C,CAAC;AAE/E;;;;;;;GAOG;AACH,wBAAsB,mBAAmB,CAAC,EAAE,EAAE,QAAQ,CAAC,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA4B9E"}

package/dist/packages/core/src/infrastructure/persistence/sqlite/migrations.js

@@ -309,6 +309,14 @@ ALTER TABLE features ADD COLUMN ci_fix_history TEXT;
         sql: '',
         handler: (db) => {
             const columns = db.pragma('table_info(features)');
+            // Backfill ci_fix columns for users whose old migration 19 added parent_id
+            // instead of ci_fix_attempts/ci_fix_history (pre-rebase ordering).
+            if (!columns.some((c) => c.name === 'ci_fix_attempts')) {
+                db.exec('ALTER TABLE features ADD COLUMN ci_fix_attempts INTEGER');
+            }
+            if (!columns.some((c) => c.name === 'ci_fix_history')) {
+                db.exec('ALTER TABLE features ADD COLUMN ci_fix_history TEXT');
+            }
             if (!columns.some((c) => c.name === 'parent_id')) {
                 db.exec('ALTER TABLE features ADD COLUMN parent_id TEXT');
             }
@@ -316,6 +324,23 @@ ALTER TABLE features ADD COLUMN ci_fix_history TEXT;
             db.exec('CREATE INDEX IF NOT EXISTS idx_features_parent_id ON features(parent_id)');
         },
     },
+    {
+        version: 21,
+        // Migration 021: Backfill columns that may be missing due to migration 19 rebase reordering.
+        // If the old migration 19 (parent_id) ran before the rebase, ci_fix_attempts/ci_fix_history
+        // were never added. Migration 20's handler now covers fresh installs; this covers
+        // databases that already ran migration 20 with the old handler.
+        sql: '',
+        handler: (db) => {
+            const columns = db.pragma('table_info(features)');
+            if (!columns.some((c) => c.name === 'ci_fix_attempts')) {
+                db.exec('ALTER TABLE features ADD COLUMN ci_fix_attempts INTEGER');
+            }
+            if (!columns.some((c) => c.name === 'ci_fix_history')) {
+                db.exec('ALTER TABLE features ADD COLUMN ci_fix_history TEXT');
+            }
+        },
+    },
 ];
 /**
  * The latest schema version (highest migration version number).