@atlashub/smartstack-cli 4.17.1 → 4.18.0

package/templates/skills/ralph-loop/steps/step-04-check.md

@@ -47,9 +47,12 @@ fi
 
 ### 1.7. Category Completeness Check (RUNS EVERY ITERATION)
 
-See `references/category-completeness.md` for detailed implementation.
+> **Implementation:** See `references/category-completeness.md` for the full `checkCategoryCompleteness()` function and artifact verification logic.
+> Do NOT duplicate the implementation here — load and execute the reference.
 
 ```javascript
+// Load references/category-completeness.md — execute checkCategoryCompleteness() as defined there
+// (includes guardrail injection AND artifact verification)
 const { missing, guardrailsNeeded } = checkCategoryCompleteness(prd);
 
 if (guardrailsNeeded.length > 0) {
@@ -57,26 +60,7 @@ if (guardrailsNeeded.length > 0) {
   writeJSON('.ralph/prd.json', prd);
   console.log(`PRD updated: +${guardrailsNeeded.length} guardrails, ${prd.tasks.filter(t => t.status === 'pending').length} pending tasks`);
 }
-
-// Artifact verification (check if completed tasks have files)
-const completedCats = new Set(prd.tasks.filter(t => t.status === 'completed').map(t => t.category));
-const artifactChecks = {
-  'frontend': () => glob.sync('**/src/pages/**/*.tsx').length > 0,
-  'test': () => glob.sync('tests/**/*Tests.cs').length > 0,
-  'api': () => glob.sync('src/*/Controllers/**/*Controller.cs').length > 0
-};
-for (const [cat, check] of Object.entries(artifactChecks)) {
-  if (completedCats.has(cat) && !check()) {
-    console.warn(`ARTIFACT MISSING: ${cat} tasks marked completed but no files found — resetting to pending`);
-    prd.tasks.filter(t => t.category === cat && t.status === 'completed')
-      .forEach(t => {
-        t.status = 'pending';
-        t.error = 'Artifacts missing — re-execute';
-        t.completed_at = null;
-      });
-    writeJSON('.ralph/prd.json', prd);
-  }
-}
+// Artifact verification is handled inside checkCategoryCompleteness() — see reference file.
 ```
 
 ## 2. Check Iteration Limit

package/templates/skills/ralph-loop/steps/step-05-report.md

@@ -131,13 +131,18 @@ Write to `.ralph/reports/{feature-slug}.md`:
 
 ```javascript
 // Clean up section-split temporary files
-if (prd._sectionSplit?.enabled) {
+// GUARD: only delete if the overall PRD status is 'completed'.
+// If status is 'partial' or 'failed', keep temporary PRD files so the user
+// can resume with `-r` — deleting them here would make recovery impossible.
+if (prd._sectionSplit?.enabled && prd.status === 'completed') {
   for (const phase of prd._sectionSplit.phases) {
     if (fileExists(phase.prdFile) && phase.prdFile !== '.ralph/prd.json') {
       deleteFile(phase.prdFile);
     }
   }
   delete prd._sectionSplit;
+} else if (prd._sectionSplit?.enabled && prd.status !== 'completed') {
+  console.log(`Section-split temp files preserved (status: ${prd.status}) — use /ralph-loop -r to resume`);
 }
 
 prd.updated_at = new Date().toISOString();