cc-devflow 4.5.7 → 4.5.9

package/lib/skill-runtime/__tests__/planner.tdd.test.js

@@ -120,7 +120,7 @@ describe('TDD Order Validation', () => {
       });
     });
 
-    test('should write currentTaskId and activePhase into manifest', async () => {
+    test('should write currentTaskId while deriving activePhase from the task graph', async () => {
       const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-planner-'));
       const changeId = 'REQ-321';
       const tasksPath = path.join(repoRoot, 'devflow', 'changes', `${changeId}-change`, 'planning', 'tasks.md');
@@ -146,7 +146,7 @@ describe('TDD Order Validation', () => {
         overwrite: true
       });
 
-      expect(manifest.activePhase).toBe(1);
+      expect(manifest.activePhase).toBeUndefined();
       expect(manifest.currentTaskId).toBe('T002');
       expect(manifest.tasks[1].reviews).toEqual({ spec: 'pending', code: 'pending' });
     });

package/lib/skill-runtime/__tests__/schemas.test.js

@@ -8,7 +8,6 @@ describe('Manifest schema hard constraints', () => {
       createdAt: '2026-04-10T01:00:00.000Z',
       updatedAt: '2026-04-10T01:05:00.000Z',
       currentTaskId: 'T001',
-      activePhase: 1,
       tasks: [
         {
           id: 'T001',
@@ -73,7 +72,39 @@ describe('Manifest schema hard constraints', () => {
     });
 
     expect(manifest.currentTaskId).toBe('T001');
-    expect(manifest.activePhase).toBe(1);
+    expect(manifest.activePhase).toBeUndefined();
+  });
+
+  test('accepts legacy activePhase input without preserving the retired field', () => {
+    const manifest = parseManifest({
+      changeId: 'REQ-560',
+      goal: 'Validate legacy manifest compatibility',
+      createdAt: '2026-04-10T01:00:00.000Z',
+      updatedAt: '2026-04-10T01:05:00.000Z',
+      currentTaskId: 'T001',
+      activePhase: 1,
+      tasks: [
+        {
+          id: 'T001',
+          title: '[TEST] Counter behavior',
+          type: 'OTHER',
+          phase: 1,
+          dependsOn: [],
+          run: ['echo ok'],
+          status: 'pending',
+          attempts: 0,
+          maxRetries: 1
+        }
+      ],
+      metadata: {
+        source: 'default',
+        generatedBy: 'test',
+        planVersion: 1
+      }
+    });
+
+    expect(manifest.currentTaskId).toBe('T001');
+    expect(manifest.activePhase).toBeUndefined();
   });
 
   test('rejects invalid currentTaskId for tasks.md manifest', () => {

package/lib/skill-runtime/planner.js

@@ -480,7 +480,7 @@ function deriveManifestExecutionState(tasks) {
 function applyManifestExecutionState(manifest, updatedAt = nowIso()) {
   const executionState = deriveManifestExecutionState(manifest.tasks || []);
   manifest.currentTaskId = executionState.currentTaskId;
-  manifest.activePhase = executionState.activePhase;
+  delete manifest.activePhase;
   manifest.updatedAt = updatedAt;
   return manifest;
 }
@@ -504,7 +504,6 @@ async function createTaskManifest({ repoRoot, changeId, goal, overwrite = false
     createdAt: previous?.createdAt || nowIso(),
     updatedAt: nowIso(),
     currentTaskId: null,
-    activePhase: null,
     tasks,
     metadata: {
       source: hasTasksFile ? 'tasks.md' : 'default',

package/lib/skill-runtime/query.js

@@ -81,7 +81,7 @@ async function getNextTask(repoRoot, changeId, options = {}) {
   const manifestPath = getTaskManifestPath(repoRoot, changeId, options);
   const manifest = await readQueryArtifact(manifestPath);
   const executionState = deriveManifestExecutionState(manifest.tasks || []);
-  const activePhase = manifest.activePhase ?? executionState.activePhase;
+  const activePhase = executionState.activePhase;
   const completedIds = new Set(
     (manifest.tasks || [])
       .filter((task) => isTaskCompletedStatus(task.status))

package/lib/skill-runtime/schemas.js

@@ -107,7 +107,7 @@ const ManifestSchema = z.object({
   createdAt: z.string().datetime(),
   updatedAt: z.string().datetime(),
   currentTaskId: z.string().regex(TASK_ID_PATTERN).nullable().default(null),
-  activePhase: z.number().int().min(1).nullable().default(null),
+  activePhase: z.number().int().min(1).nullable().optional(),
   tasks: z.array(TaskSchema),
   metadata: z.object({
     source: z.enum(['tasks.md', 'default']).default('default'),
@@ -161,7 +161,7 @@ const ManifestSchema = z.object({
     ? Math.min(...unfinished.map((task) => task.phase || 1))
     : null;
 
-  if (manifest.activePhase !== null && manifest.activePhase !== derivedActivePhase) {
+  if (manifest.activePhase !== undefined && manifest.activePhase !== null && manifest.activePhase !== derivedActivePhase) {
     ctx.addIssue({
       code: z.ZodIssueCode.custom,
       path: ['activePhase'],
@@ -542,7 +542,9 @@ function parseWithSchema(schema, input, label) {
 }
 
 function parseManifest(input) {
-  return parseWithSchema(ManifestSchema, input, 'Manifest');
+  const manifest = parseWithSchema(ManifestSchema, input, 'Manifest');
+  delete manifest.activePhase;
+  return manifest;
 }
 
 function parseCheckpoint(input) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "cc-devflow",
-    "version": "4.5.7",
+    "version": "4.5.9",
     "description": "Multi-platform CLI and skill pack for agent coding",
     "main": "bin/cc-devflow.js",
     "bin": {
@@ -10,11 +10,16 @@
         "bin/",
         "lib/",
         "config/",
+        "docs/assets/",
         ".claude/skills/cc-roadmap/",
+        ".claude/skills/cc-next/",
+        ".claude/skills/cc-dev/",
         ".claude/skills/cc-plan/",
         ".claude/skills/cc-investigate/",
         ".claude/skills/cc-do/",
         ".claude/skills/cc-review/",
+        ".claude/skills/cc-pr-review/",
+        ".claude/skills/cc-pr-land/",
         ".claude/skills/cc-check/",
         ".claude/skills/cc-act/",
         ".claude/skills/cc-spec-init/",