@dtlabs/ai-toolkit 0.24.5 → 0.24.7

package/CHANGELOG.md

@@ -1,4 +1,22 @@
 ## 0.1.4 (2026-07-21)
+## [0.24.7](https://github.com/dtlab-tech/ai-toolkit/compare/v0.24.6...v0.24.7) (2026-10-30)
+
+
+### Bug Fixes
+
+* **pm-phase1:** route validator findings to a document by their prefix when the flags are omitted ([7bd9a38](https://github.com/dtlab-tech/ai-toolkit/commit/7bd9a38de571c07d233a5212dfba9213a1b7d562))
+* **pm-phase1:** route validator findings to a document by their prefix when the flags are omitted ([f957d72](https://github.com/dtlab-tech/ai-toolkit/commit/f957d726294d2b563e67ba1c3f4e59f9c8d57be4))
+* **pm-phase1:** route validator findings to a document by their prefix when the flags are omitted ([e33faef](https://github.com/dtlab-tech/ai-toolkit/commit/e33faefc4488f256e42ec459cefa547f7b5efbf2))
+
+## [0.24.6](https://github.com/dtlab-tech/ai-toolkit/compare/v0.24.5...v0.24.6) (2026-10-29)
+
+
+### Bug Fixes
+
+* **pm-phase1:** feed validation findings back to the generators and d… ([6fcb46d](https://github.com/dtlab-tech/ai-toolkit/commit/6fcb46d4e0d6d1d9ad8e2bf97313ce23691ce665))
+* **pm-phase1:** feed validation findings back to the generators and drop duplicate Related blocks ([86d01df](https://github.com/dtlab-tech/ai-toolkit/commit/86d01dffbaccd9f5b99e3a356cd13e5d902d699b))
+* **pm-phase1:** feed validation findings back to the generators and drop duplicate Related blocks ([9411ea5](https://github.com/dtlab-tech/ai-toolkit/commit/9411ea529d8eccdc54abcd4f8f296542b73e14b9))
+
 ## [0.24.5](https://github.com/dtlab-tech/ai-toolkit/compare/v0.24.4...v0.24.5) (2026-10-27)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dtlabs/ai-toolkit",
-  "version": "0.24.5",
+  "version": "0.24.7",
   "description": "AI Toolkit — reusable agents and procedures for developers to build AI-powered applications.",
   "files": [
     "docs/",

package/src/claude/workflows/pm-phase1.js

@@ -55,6 +55,28 @@ return c.run(meta.name, featureDir, prefix, async () => {
     return `${prompt}\n\nTemplate (${label}):\n${readTemplate(templateFile)}`
   }
 
+  // Gaps the validator reported on the previous cycle, set only by the revision branch below
+  // before it regenerates. Without this a "revision" was a blind full regeneration from
+  // feature.md alone: the generators never saw what was wrong, so three cycles were three
+  // independent attempts at the same input and convergence depended on luck.
+  let revisionFindings = []
+  function revisionSection(findings) {
+    if (!findings.length) return ''
+    return '\n\nThe previous version of these documents was validated and these gaps were reported. ' +
+      'Resolve every one that concerns the document you are generating, and do not raise a new open ' +
+      'question for anything feature.md already decides:\n' + findings.map(f => `- ${f}`).join('\n')
+  }
+
+  // The use-case agent is told not to write the "Related Business Rules"/"Related Acceptance
+  // Criteria" lines (the host computes them from req/index.md and appends them), but it
+  // sometimes writes its own anyway — with guessed IDs and "—" for the ACs it cannot see. Appending
+  // the host's block after it left two conflicting blocks in the file. Drop anything the model
+  // wrote from its first such line onward, so exactly one host-computed block remains.
+  function stripModelCrossReferences(detail) {
+    const idx = detail.search(/^\*\*Related (?:Business Rules|Acceptance Criteria):\*\*/m)
+    return idx === -1 ? detail : detail.slice(0, idx)
+  }
+
   // Deliberately NOT file('req') — same reasoning as FTR-020's wbsDir: this is a directory
   // nested inside the feature's own already-prefixed folder, so a second prefix would be
   // redundant.
@@ -162,7 +184,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
     const base = `${featurePathArg}\n\nGenerate the full detail for this use case:\n` +
       `UC ID: ${row.id}\nTitle: ${row.title}\nActor: ${row.actor}\nPriority: ${row.priority}\n\n` +
       `Full use-case index (for cross-UC awareness, do not restate):\n${compactIndex}`
-    return `${base}\n\nTemplate (req/use-cases/{UC-ID}.md):\n${ucTemplate}`
+    return `${base}\n\nTemplate (req/use-cases/{UC-ID}.md):\n${ucTemplate}${revisionSection(revisionFindings)}`
   }
 
   // forceAll: bypasses resume-by-existence entirely, re-dispatching the skeleton and every
@@ -193,7 +215,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
     // 1. Skeleton (no declared worker() output — host writes it itself, mirroring FTR-020's
     // skeleton step)
     if (regenSkeleton) {
-      const skeleton = await c.worker(reqId, withTemplate(featurePath, 'req/index.md', 'gaia-requirements-index-template.md'))
+      const skeleton = await c.worker(reqId, withTemplate(featurePath, 'req/index.md', 'gaia-requirements-index-template.md') + revisionSection(revisionFindings))
       c.writeOutput(reqIndexPath, skeleton)
     }
 
@@ -231,7 +253,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
         valid = isValidUseCaseFile(detail, row.id)
       }
       if (!valid) throw new Error(`Requirements use case "${row.id}" failed structural validation after ${USE_CASE_RETRY_ATTEMPTS} attempts — missing the required "### ${row.id}: " heading`)
-      c.writeOutput(ucPath, detail.trimEnd() + ucCrossReferences(c.read(reqIndexPath), row.id))
+      c.writeOutput(ucPath, stripModelCrossReferences(detail).trimEnd() + ucCrossReferences(c.read(reqIndexPath), row.id))
     }
     // No assembly/write step here — generation only ever produces req/index.md + its per-UC
     // files. Assembly happens in-memory, on demand, at the top level below (never persisted by
@@ -484,7 +506,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
     const base = `${featurePathArg}\n\nGenerate the full technical design for this component:\n` +
       `Path: ${row.filePath}\nChange: ${row.changeType}\nDomain: ${row.domain}\nAddresses: ${row.addresses}\n\n` +
       `Full component index (for cross-component awareness, do not restate):\n${compactIndex}`
-    return `${base}\n\nTemplate (spec/components/{fileSlug}.md):\n${componentTemplate}`
+    return `${base}\n\nTemplate (spec/components/{fileSlug}.md):\n${componentTemplate}${revisionSection(revisionFindings)}`
   }
 
   // A row's own content fingerprint — the same four fields FTR-030's row-diffing has always
@@ -543,7 +565,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
 
     // 1. Skeleton (no declared worker() output — host writes it itself)
     if (regenSkeleton) {
-      const skeleton = await c.worker(specId, withTemplate(featurePath, 'spec/index.md', 'gaia-tech-spec-index-template.md'))
+      const skeleton = await c.worker(specId, withTemplate(featurePath, 'spec/index.md', 'gaia-tech-spec-index-template.md') + revisionSection(revisionFindings))
       c.writeOutput(specIndexPath, applyComponentSlugs(skeleton, manifest))
     }
 
@@ -722,8 +744,14 @@ return c.run(meta.name, featureDir, prefix, async () => {
       // now-stale self-report. The only authoritative signal for the final verdict is whether any
       // finding remains, from any source, after every check this cycle has run.
       const mergedValid = mergedFindings.length === 0
-      const mergedNeedsTechSpecFix = result.needsTechSpecFix || newFindings.some(f => f.startsWith('Tech-Spec:'))
-      const mergedNeedsRequirementsFix = result.needsRequirementsFix || newFindings.some(f => f.startsWith('Requirements:'))
+      // Routing reads the prefix of EVERY finding, the validator's own included: the validator is
+      // required to prefix each finding with the document it concerns (gaia-validate-feature-docs,
+      // output contract), and its own example verdict omits needsRequirementsFix/needsTechSpecFix
+      // altogether. Reading the prefix only on host-computed findings made a verdict with prefixed
+      // findings and no flags fail with "flagged neither document" (seen on FTR-034). A finding
+      // with no prefix and no flag still fails closed below.
+      const mergedNeedsTechSpecFix = result.needsTechSpecFix || mergedFindings.some(f => f.startsWith('Tech-Spec:'))
+      const mergedNeedsRequirementsFix = result.needsRequirementsFix || mergedFindings.some(f => f.startsWith('Requirements:'))
 
       // c.worker() already wrote result.report verbatim via its own outputs[0] contract; only
       // overwrite it when this cycle actually added findings the LLM's own report text doesn't
@@ -740,6 +768,7 @@ return c.run(meta.name, featureDir, prefix, async () => {
       }
       if (cycle === 3) throw new Error('Validation gaps remain after 3 cycles')
       if (!mergedNeedsRequirementsFix && !mergedNeedsTechSpecFix) throw new Error(`Validation agent reported gaps on cycle ${cycle} but flagged neither document for revision`)
+      revisionFindings = mergedFindings
       if (mergedNeedsRequirementsFix) { await generateRequirements(true); assembledRequirements = assembleRequirements() }
       if (mergedNeedsTechSpecFix) { await generateTechSpec(true); assembledTechSpec = assembleTechSpec() }
     }