@dotdotgod/cli 0.1.43 → 0.1.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotdotgod/cli",
-  "version": "0.1.43",
+  "version": "0.1.44",
   "description": "CLI for dotdotgod validation, project memory snapshots, and graph indexing.",
   "type": "module",
   "license": "Elastic-2.0",

package/src/commands/plan-stage-contract.mjs

@@ -41,8 +41,9 @@ export const PLAN_BLOCKER_LABELS = new Map([
   ['invalid-task-slug', 'Plan task directory must be kebab-case'],
   ['empty-plan', 'Plan README is empty'],
   ['missing-stage', 'Missing required stage directory'],
-  ['missing-internal-stage', 'Missing internal stage file'],
   ['missing-stage-readme', 'Missing stage README.md'],
+  ['missing-internal-stage', 'Missing internal stage checkpoint'],
+  ['open-internal-stage', 'Internal stage checkpoint is not completed'],
   ['invalid-markdown-name', 'Invalid Markdown file name'],
   ['missing-section', 'Missing required section'],
   ['empty-section', 'Required section has no content'],
@@ -66,8 +67,9 @@ export function buildPlanValidationNextStagePrompt(planPath, stage, stagePath) {
   return [
     `Continue Plan Mode stage advancement for ${planPath}.`,
     `Current stage: ${stage}.`,
-    `Make ${planPath} satisfy the current stage's durable README requirements first. Use ${stagePath} only as an optional checkpoint/workflow note for this stage; do not make numbered checkpoint creation the goal and do not create later checkpoint files yet.`,
-    stage === '05-workstream-handoff' ? 'For this stage, record `Split decision: yes` or `Split decision: no` in `## Workstream Handoff`. If splitting, make `## Workstream Map`, `## Workstreams`, and `## Integration Sequence` describe multiple actionable workstreams with required context, allowed/forbidden edits, tasks, validation, dependencies, and handoff output. If not splitting, record the no-split rationale and keep `## Todo Contract` actionable so Pi can extract execution todos.' : undefined,
+    `Update the durable plan for human readers, then make ${stagePath} the machine-readable checkpoint that satisfies the current stage validation contract. Do not create later checkpoint files yet.`,
+    `In ${stagePath}, set Status: completed and include the current stage required sections plus \`## Stage ${stage.slice(0, 2)} Construction Checklist\` rows in the canonical \`- [x] Category: evidence\` form.`,
+    stage === '05-workstream-handoff' ? 'For this stage checkpoint, record `Split decision: yes` or `Split decision: no` in `## Workstream Handoff`. If splitting, make `## Workstream Map`, `## Workstreams`, and `## Integration Sequence` describe multiple actionable workstreams with required context, allowed/forbidden edits, tasks, validation, dependencies, and handoff output. If not splitting, record the no-split rationale and keep `## Todo Contract` actionable so Pi can extract execution todos.' : undefined,
     `After completing this stage, stop and rerun \`dotdotgod plan validate ${planPath} --stage ${stage} --json\`.`,
   ].filter(Boolean).join('\n\n');
 }

package/src/commands/plan.mjs

@@ -201,17 +201,37 @@ function readInternalStageState(taskDir, stageName) {
 
 function validateInternalStageState(root, taskDir, stageName, blockers) {
   const state = readInternalStageState(taskDir, stageName);
-  if (!state) return;
+  const expectedPath = internalStageFile(taskDir, stageName);
+  const rel = projectRelative(root, state?.file ?? expectedPath);
+  if (!state) {
+    addBlocker(blockers, 'missing-internal-stage', `Missing internal stage checkpoint: ${rel}`, { path: rel, stage: stageName });
+    return { state: undefined, valid: false };
+  }
+
+  let valid = true;
   const stage = contentMatch(state.content, /^Stage:\s*(\S+)\s*$/m);
   const status = contentMatch(state.content, /^Status:\s*(created|blocked|completed)\s*$/m);
   const updated = contentMatch(state.content, /^Updated:\s*(\S+)\s*$/m);
-  const rel = projectRelative(root, state.file);
-  const looksLikeGeneratorState = /^Stage:\s*\S+\s*$/m.test(state.content) || /^Status:\s*\S+\s*$/m.test(state.content) || /^Updated:\s*\S+\s*$/m.test(state.content);
-  if (!looksLikeGeneratorState) return;
-  if (stage && stage !== stageName) addBlocker(blockers, 'missing-internal-stage', `Internal stage file has wrong Stage field: ${rel}`, { path: rel, stage: stageName });
-  if (!stage) addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing Stage: ${rel}`, { path: rel, stage: stageName });
-  if (!status) addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing valid Status: ${rel}`, { path: rel, stage: stageName });
-  if (!updated) addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing Updated: ${rel}`, { path: rel, stage: stageName });
+  if (stage && stage !== stageName) {
+    valid = false;
+    addBlocker(blockers, 'missing-internal-stage', `Internal stage file has wrong Stage field: ${rel}`, { path: rel, stage: stageName });
+  }
+  if (!stage) {
+    valid = false;
+    addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing Stage: ${rel}`, { path: rel, stage: stageName });
+  }
+  if (!status) {
+    valid = false;
+    addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing valid Status: ${rel}`, { path: rel, stage: stageName });
+  } else if (status !== 'completed') {
+    valid = false;
+    addBlocker(blockers, 'open-internal-stage', `Internal stage checkpoint is not completed: ${rel}`, { path: rel, stage: stageName });
+  }
+  if (!updated) {
+    valid = false;
+    addBlocker(blockers, 'missing-internal-stage', `Internal stage file is missing Updated: ${rel}`, { path: rel, stage: stageName });
+  }
+  return { state, valid };
 }
 
 function contentMatch(content, pattern) {
@@ -492,28 +512,28 @@ export function validatePlanArtifact(planPath, options = {}) {
   const durableFiles = discoverDurablePlanMarkdownFiles(taskDir);
   if (durableFiles.length > 1) summary.splitFiles.detected = true;
   for (const stageName of stagesToValidate) {
-    validateInternalStageState(root, taskDir, stageName, blockers);
-    const files = durableFiles.length > 0 ? durableFiles : [absolutePlanPath];
-    const stageReadme = absolutePlanPath;
-    summary.stages.present += 1;
-    let stageValid = true;
+    const internal = validateInternalStageState(root, taskDir, stageName, blockers);
+    summary.stages.present += internal.state ? 1 : 0;
+    let stageValid = internal.valid;
+    const files = internal.state ? [internal.state.file] : [];
+    const fallbackPath = projectRelative(root, internalStageFile(taskDir, stageName));
     for (const header of PLAN_REQUIRED_HEADERS[stageName]) {
-      const found = findSection(files, header);
+      const found = files.length > 0 ? findSection(files, header) : undefined;
       if (!found) {
         stageValid = false;
-        addBlocker(blockers, 'missing-section', `Missing required section: ## ${header}`, { path: projectRelative(root, stageReadme), stage: stageName, section: header });
+        addBlocker(blockers, 'missing-section', `Missing required section in internal checkpoint: ## ${header}`, { path: fallbackPath, stage: stageName, section: header });
         continue;
       }
       summary.requiredSections.present += 1;
       const relFound = projectRelative(root, found.file);
       if (!contentHasValue(found.content)) {
         stageValid = false;
-        addBlocker(blockers, 'empty-section', `Required section has no content: ## ${header}`, { path: relFound, stage: stageName, section: header });
+        addBlocker(blockers, 'empty-section', `Required checkpoint section has no content: ## ${header}`, { path: relFound, stage: stageName, section: header });
         continue;
       }
       if (isPlaceholderContent(found.content)) {
         stageValid = false;
-        addBlocker(blockers, 'placeholder-section', `Required section only contains placeholder content: ## ${header}`, { path: relFound, stage: stageName, section: header });
+        addBlocker(blockers, 'placeholder-section', `Required checkpoint section only contains placeholder content: ## ${header}`, { path: relFound, stage: stageName, section: header });
         continue;
       }
       summary.requiredSections.valid += 1;
@@ -541,8 +561,8 @@ export function validatePlanArtifact(planPath, options = {}) {
         addBlocker(blockers, 'unresolved-assumption', 'Assumptions contain unresolved items.', { path: relFound, stage: stageName, section: header });
       }
     }
-    if (!validateWorkstreamHandoffContract(root, files, stageName, blockers)) stageValid = false;
-    if (!validateConstructionChecklist(root, files, stageName, blockers)) stageValid = false;
+    if (files.length > 0 && !validateWorkstreamHandoffContract(root, files, stageName, blockers)) stageValid = false;
+    if (files.length > 0 && !validateConstructionChecklist(root, files, stageName, blockers)) stageValid = false;
     if (stageValid) summary.stages.valid += 1;
   }
 

package/src/validate/run.mjs

@@ -35,6 +35,7 @@ export function runValidate(argv) {
     if (path.includes('/node_modules') || path === 'node_modules') return true;
     if (path.includes('/.git') || path === '.git') return true;
     if (path === CACHE_DIR || path.startsWith(`${CACHE_DIR}/`)) return true;
+    if (/^docs\/(?:plan|archive\/plan)\/[a-z0-9]+(?:-[a-z0-9]+)*\/\.dotdotgod-plan(?:\/|$)/.test(path)) return true;
     if (!options.includeLocalMemory && (path === 'docs/plan' || path.startsWith('docs/plan/') || path === 'docs/archive' || path.startsWith('docs/archive/'))) return true;
     return false;
   };