cc-devflow 4.5.5 → 4.5.7

package/lib/skill-runtime/intent.js

@@ -68,11 +68,11 @@ function summarizeGateSection(gates = []) {
   });
 }
 
-async function readLatestCheckpoints(repoRoot, changeId, tasks = []) {
+async function readLatestCheckpoints(repoRoot, changeId, tasks = [], options = {}) {
   const checkpoints = [];
 
   for (const task of tasks) {
-    const checkpoint = await readJson(getCheckpointPath(repoRoot, changeId, task.id), null);
+    const checkpoint = await readJson(getCheckpointPath(repoRoot, changeId, task.id, options), null);
     if (!checkpoint) {
       continue;
     }
@@ -90,28 +90,28 @@ async function readLatestCheckpoints(repoRoot, changeId, tasks = []) {
   return checkpoints;
 }
 
-async function cleanupLegacyArtifacts(repoRoot, changeId, manifest) {
+async function cleanupLegacyArtifacts(repoRoot, changeId, manifest, options = {}) {
   const taskIds = (manifest?.tasks || []).map((task) => task.id);
-  const change = getChangePaths(repoRoot, changeId);
+  const change = getChangePaths(repoRoot, changeId, options);
 
   await Promise.all(
-    getLegacyIntentProjectionPaths(repoRoot, changeId, taskIds).map((target) => removePath(target))
+    getLegacyIntentProjectionPaths(repoRoot, changeId, taskIds, options).map((target) => removePath(target))
   );
   await removePath(change.workersDir);
 }
 
-async function clearHandoffArtifacts(repoRoot, changeId, keep = null) {
+async function clearHandoffArtifacts(repoRoot, changeId, keep = null, options = {}) {
   await Promise.all(
-    getIntentHandoffArtifactPaths(repoRoot, changeId)
+    getIntentHandoffArtifactPaths(repoRoot, changeId, options)
       .filter((target) => target !== keep)
       .map((target) => removePath(target))
   );
 }
 
-async function writeResumeIndex(repoRoot, changeId, state, manifest, report) {
-  await ensureIntentScaffold(repoRoot, changeId);
+async function writeResumeIndex(repoRoot, changeId, state, manifest, report, options = {}) {
+  await ensureIntentScaffold(repoRoot, changeId, options);
 
-  const checkpoints = await readLatestCheckpoints(repoRoot, changeId, manifest?.tasks || []);
+  const checkpoints = await readLatestCheckpoints(repoRoot, changeId, manifest?.tasks || [], options);
   const lastGoodCheckpoint = checkpoints.find((checkpoint) => checkpoint.status === 'passed') || checkpoints[0] || null;
   const blockers = [
     ...(manifest?.tasks || [])
@@ -119,8 +119,8 @@ async function writeResumeIndex(repoRoot, changeId, state, manifest, report) {
       .map((task) => `${task.id}: ${task.lastError || 'Task failed'}`),
     ...(report?.blockingFindings || [])
   ];
-  const hasPrBrief = await exists(getIntentPrBriefPath(repoRoot, changeId));
-  const hasReleaseNote = await exists(getReleaseNotePath(repoRoot, changeId));
+  const hasPrBrief = await exists(getIntentPrBriefPath(repoRoot, changeId, options));
+  const hasReleaseNote = await exists(getReleaseNotePath(repoRoot, changeId, options));
   const stage = deriveLifecycleStage({ state, manifest, report, hasPrBrief });
   const approval = getApprovalState(state, manifest);
   const goal = manifest?.goal || state?.goal || changeId;
@@ -131,7 +131,7 @@ async function writeResumeIndex(repoRoot, changeId, state, manifest, report) {
     hasPrBrief,
     hasReleaseNote
   });
-  const resumePath = getIntentResumeIndexPath(repoRoot, changeId);
+  const resumePath = getIntentResumeIndexPath(repoRoot, changeId, options);
 
   const content = [
     `# Resume Index: ${changeId}`,
@@ -167,7 +167,7 @@ async function writeResumeIndex(repoRoot, changeId, state, manifest, report) {
     ])
   ].join('\n');
 
-  await clearHandoffArtifacts(repoRoot, changeId, resumePath);
+  await clearHandoffArtifacts(repoRoot, changeId, resumePath, options);
   await writeText(resumePath, `${content}\n`);
 
   return {

package/lib/skill-runtime/operations/autopilot-shared.js

@@ -28,11 +28,11 @@ function hasUnresolvedTasks(manifest) {
   );
 }
 
-async function loadState(repoRoot, changeId) {
+async function loadState(repoRoot, changeId, options = {}) {
   const [state, manifest, report] = await Promise.all([
-    readJson(getRuntimeStatePath(repoRoot, changeId), null),
-    readJson(getTaskManifestPath(repoRoot, changeId), null),
-    readJson(getReportCardPath(repoRoot, changeId), null)
+    readJson(getRuntimeStatePath(repoRoot, changeId, options), null),
+    readJson(getTaskManifestPath(repoRoot, changeId, options), null),
+    readJson(getReportCardPath(repoRoot, changeId, options), null)
   ]);
 
   return { state, manifest, report };

package/lib/skill-runtime/paths.js

@@ -1,7 +1,7 @@
 /**
  * [INPUT]: 依赖 fs/path，接收 repoRoot/changeId/taskId/workerId 等定位参数。
  * [OUTPUT]: 对外提供 devflow canonical layout 的唯一路径解析能力。
- * [POS]: skill runtime 的路径真相源，强制新 change 目录使用 REQ/FIX 前缀。
+ * [POS]: skill runtime 的路径真相源，完整 change key 才是目录身份。
  * [PROTOCOL]: 变更时更新此头部，然后检查 CLAUDE.md
  */
 
@@ -85,17 +85,27 @@ function listChangeKeys(repoRoot) {
     .sort();
 }
 
-function findExistingChangeKey(repoRoot, changeId) {
+function findExistingChangeKeys(repoRoot, changeId) {
   const canonicalPrefix = `${getChangeKeyPrefix(changeId)}-`;
   const legacyKey = slugifySegment(changeId, 'change');
   const legacyPrefix = `${legacyKey}-`;
 
-  return listChangeKeys(repoRoot).find((name) => (
+  return listChangeKeys(repoRoot).filter((name) => (
     name === changeId
     || name === legacyKey
     || name.startsWith(canonicalPrefix)
     || name.startsWith(legacyPrefix)
-  )) || null;
+  ));
+}
+
+function findExactChangeKey(repoRoot, changeId, changeKey) {
+  const slug = getChangeSlug(changeId, changeKey);
+  const aliases = new Set([
+    changeKey,
+    `${slugifySegment(changeId, 'change')}-${slug}`
+  ]);
+
+  return listChangeKeys(repoRoot).find((name) => aliases.has(name)) || null;
 }
 
 function assertChangeKey(changeId, changeKey) {
@@ -132,14 +142,56 @@ function resolveChangeKey(repoRoot, changeId, options = {}) {
     return assertChangeKey(changeId, options.changeKey);
   }
 
-  const existing = findExistingChangeKey(repoRoot, changeId);
-  if (existing) {
-    return existing;
+  const requestedSlug = options.slug || stripChangeIdPrefix(changeId, options.goal);
+  if (requestedSlug) {
+    const requestedKey = buildChangeKey(changeId, { slug: requestedSlug });
+    return findExactChangeKey(repoRoot, changeId, requestedKey) || requestedKey;
+  }
+
+  const existing = findExistingChangeKeys(repoRoot, changeId);
+  if (existing.length === 1) {
+    return existing[0];
+  }
+  if (existing.length > 1) {
+    throw new Error(
+      `Ambiguous changeId "${changeId}": found ${existing.join(', ')}. Pass an explicit changeKey.`
+    );
   }
 
   return buildChangeKey(changeId, options);
 }
 
+function nextChangeKey(repoRoot, prefix, description) {
+  const upper = String(prefix || '').toUpperCase();
+  if (upper !== 'REQ' && upper !== 'FIX') {
+    throw new Error(`Invalid prefix "${prefix}". Use REQ or FIX.`);
+  }
+
+  const numberPattern = new RegExp(`^${upper}-(\\d+)`);
+  let maxNum = 0;
+  let padWidth = 3;
+
+  for (const name of listChangeKeys(repoRoot)) {
+    const match = name.match(numberPattern);
+    if (match) {
+      const numStr = match[1];
+      const num = parseInt(numStr, 10);
+      if (num > maxNum) maxNum = num;
+      if (numStr.length > padWidth) padWidth = numStr.length;
+    }
+  }
+
+  const paddedNum = String(maxNum + 1).padStart(padWidth, '0');
+  const changeId = `${upper}-${paddedNum}`;
+  const slug = slugifySegment(
+    stripChangeIdPrefix(changeId, description) || description,
+    'change'
+  );
+  const changeKey = `${changeId}-${slug}`;
+
+  return { changeId, changeKey };
+}
+
 function getChangePaths(repoRoot, changeId, options = {}) {
   const changeKey = resolveChangeKey(repoRoot, changeId, options);
   const changeDir = path.join(getChangesRoot(repoRoot), changeKey);
@@ -205,6 +257,7 @@ module.exports = {
   getDevflowRoot,
   getChangesRoot,
   getWorkspacesRoot,
+  nextChangeKey,
   resolveChangeKey,
   buildChangeKey,
   getChangePaths,

package/lib/skill-runtime/query-registry.js

@@ -48,10 +48,10 @@ function createQueryRegistry(entries) {
       };
     }
 
-    const artifactRefs = resolveArtifactRefs(entry, context, 'artifactRefs');
-    const requiredArtifactRefs = resolveArtifactRefs(entry, context, 'requiredArtifactRefs');
-
+    let artifactRefs = [];
     try {
+      artifactRefs = resolveArtifactRefs(entry, context, 'artifactRefs');
+      const requiredArtifactRefs = resolveArtifactRefs(entry, context, 'requiredArtifactRefs');
       const missingRefs = requiredArtifactRefs.filter((ref) => !fs.existsSync(ref));
       if (missingRefs.length > 0) {
         throw namedError(

package/lib/skill-runtime/query.js

@@ -1,5 +1,5 @@
 /**
- * [INPUT]: 依赖 store/artifacts/lifecycle 读取 requirement 工件，接收 repoRoot 和 changeId。
+ * [INPUT]: 依赖 store/artifacts/lifecycle 读取 requirement 工件，接收 repoRoot、changeId 和可选 changeKey。
  * [OUTPUT]: 对外提供 typed query registry 与兼容查询函数，附 named error 和 trace shape。
  * [POS]: skill runtime 的薄查询兼容层，只读 artifact 与共享 lifecycle 语义，不再自带流程推导副本。
  * [PROTOCOL]: 变更时更新此头部，然后检查 CLAUDE.md
@@ -65,8 +65,8 @@ async function readQueryArtifact(filePath, { required = true } = {}) {
  * @param {string} changeId - 需求 ID
  * @returns {Promise<Object>} 进度统计对象
  */
-async function getProgress(repoRoot, changeId) {
-  const manifestPath = getTaskManifestPath(repoRoot, changeId);
+async function getProgress(repoRoot, changeId, options = {}) {
+  const manifestPath = getTaskManifestPath(repoRoot, changeId, options);
   const manifest = await readQueryArtifact(manifestPath);
   return deriveTaskProgress(manifest.tasks || []);
 }
@@ -77,8 +77,8 @@ async function getProgress(repoRoot, changeId) {
  * @param {string} changeId - 需求 ID
  * @returns {Promise<Object|null>} 下一个任务对象或 null
  */
-async function getNextTask(repoRoot, changeId) {
-  const manifestPath = getTaskManifestPath(repoRoot, changeId);
+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;
@@ -117,18 +117,18 @@ async function getNextTask(repoRoot, changeId) {
  * @param {string} changeId - 需求 ID
  * @returns {Promise<Object>} 完整状态对象
  */
-async function getFullState(repoRoot, changeId) {
-  const statePath = getRuntimeStatePath(repoRoot, changeId);
-  const reportPath = getReportCardPath(repoRoot, changeId);
-  const prBriefPath = getIntentPrBriefPath(repoRoot, changeId);
+async function getFullState(repoRoot, changeId, options = {}) {
+  const statePath = getRuntimeStatePath(repoRoot, changeId, options);
+  const reportPath = getReportCardPath(repoRoot, changeId, options);
+  const prBriefPath = getIntentPrBriefPath(repoRoot, changeId, options);
 
   const [state, manifest, hasPrBrief] = await Promise.all([
     readQueryArtifact(statePath),
-    readQueryArtifact(getTaskManifestPath(repoRoot, changeId)),
+    readQueryArtifact(getTaskManifestPath(repoRoot, changeId, options)),
     exists(prBriefPath)
   ]);
-  const progress = await getProgress(repoRoot, changeId);
-  const nextTask = await getNextTask(repoRoot, changeId);
+  const progress = await getProgress(repoRoot, changeId, options);
+  const nextTask = await getNextTask(repoRoot, changeId, options);
   const report = await readQueryArtifact(reportPath, { required: false });
 
   return {
@@ -160,8 +160,8 @@ async function getFullState(repoRoot, changeId) {
   };
 }
 
-async function getShipReadiness(repoRoot, changeId) {
-  const reportPath = getReportCardPath(repoRoot, changeId);
+async function getShipReadiness(repoRoot, changeId, options = {}) {
+  const reportPath = getReportCardPath(repoRoot, changeId, options);
   const report = await readJson(reportPath, null);
 
   if (!report) {
@@ -178,12 +178,12 @@ async function getShipReadiness(repoRoot, changeId) {
   return deriveShipReadiness(report, { reportPath });
 }
 
-function queryArtifactRefs(repoRoot, changeId, names) {
+function queryArtifactRefs(repoRoot, changeId, names, options = {}) {
   const refs = {
-    manifest: getTaskManifestPath(repoRoot, changeId),
-    state: getRuntimeStatePath(repoRoot, changeId),
-    report: getReportCardPath(repoRoot, changeId),
-    prBrief: getIntentPrBriefPath(repoRoot, changeId)
+    manifest: getTaskManifestPath(repoRoot, changeId, options),
+    state: getRuntimeStatePath(repoRoot, changeId, options),
+    report: getReportCardPath(repoRoot, changeId, options),
+    prBrief: getIntentPrBriefPath(repoRoot, changeId, options)
   };
 
   return names.map((name) => refs[name]).filter(Boolean);
@@ -192,26 +192,26 @@ function queryArtifactRefs(repoRoot, changeId, names) {
 const registry = createQueryRegistry([
   {
     id: 'progress',
-    artifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['manifest']),
-    requiredArtifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['manifest']),
-    handler: ({ repoRoot, changeId }) => getProgress(repoRoot, changeId)
+    artifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['manifest'], { changeKey }),
+    requiredArtifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['manifest'], { changeKey }),
+    handler: ({ repoRoot, changeId, changeKey }) => getProgress(repoRoot, changeId, { changeKey })
   },
   {
     id: 'next-task',
-    artifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['manifest']),
-    requiredArtifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['manifest']),
-    handler: ({ repoRoot, changeId }) => getNextTask(repoRoot, changeId)
+    artifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['manifest'], { changeKey }),
+    requiredArtifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['manifest'], { changeKey }),
+    handler: ({ repoRoot, changeId, changeKey }) => getNextTask(repoRoot, changeId, { changeKey })
   },
   {
     id: 'full-state',
-    artifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['state', 'manifest', 'report', 'prBrief']),
-    requiredArtifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['state', 'manifest']),
-    handler: ({ repoRoot, changeId }) => getFullState(repoRoot, changeId)
+    artifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['state', 'manifest', 'report', 'prBrief'], { changeKey }),
+    requiredArtifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['state', 'manifest'], { changeKey }),
+    handler: ({ repoRoot, changeId, changeKey }) => getFullState(repoRoot, changeId, { changeKey })
   },
   {
     id: 'ship-readiness',
-    artifactRefs: ({ repoRoot, changeId }) => queryArtifactRefs(repoRoot, changeId, ['report']),
-    handler: ({ repoRoot, changeId }) => getShipReadiness(repoRoot, changeId)
+    artifactRefs: ({ repoRoot, changeId, changeKey }) => queryArtifactRefs(repoRoot, changeId, ['report'], { changeKey }),
+    handler: ({ repoRoot, changeId, changeKey }) => getShipReadiness(repoRoot, changeId, { changeKey })
   }
 ]);
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "cc-devflow",
-    "version": "4.5.5",
+    "version": "4.5.7",
     "description": "Multi-platform CLI and skill pack for agent coding",
     "main": "bin/cc-devflow.js",
     "bin": {
@@ -14,6 +14,7 @@
         ".claude/skills/cc-plan/",
         ".claude/skills/cc-investigate/",
         ".claude/skills/cc-do/",
+        ".claude/skills/cc-review/",
         ".claude/skills/cc-check/",
         ".claude/skills/cc-act/",
         ".claude/skills/cc-spec-init/",