amsd-pipeline-workflow 0.1.9 → 0.1.11

package/driver/writer-stamps.js

@@ -17,6 +17,9 @@ function applyGateStamps(story, res) {
   else if (uit && uit.passed === true) { del('suiteState'); del('suiteStateStep'); }
   const gate = res && res.gate;
   if (gate && gate.ran !== false && gate.passed === false) { set('reviewStatus', 'escalated'); set('reproGate', 'failed'); }
+  // A REAL APPROVAL CLEARS AN EARLIER ATTEMPT'S ESCALATION (lib/review-cycle.sh ~:180–195, the pipeline's own: reviewStatus
+  // "escalated" → null). A Writer result of 'done' comes only after the review approved. B65: the stale stamp blocked the gate.
+  else if (res && res.status === 'done' && story.reviewStatus === 'escalated') set('reviewStatus', null);
   return changed;
 }
 

package/generator/writer-workflow.js

@@ -384,10 +384,14 @@ async function reproStep(prep) {
   if (!source) return { committed: false, reason: 'no testable source file in the change' }
   const cand = prep.candidates[source]
   const fb = { typecheck: '', assertions: '', commit_rejected: '' }
+  // the answer names the source the workflow chose, and only that (B61: unexplained, the model named the codeline, then the test)
+  const schema = { ...SCHEMAS.repro, properties: { ...SCHEMAS.repro.properties,
+    source: { type: 'string', enum: [source], description: 'the source file your test covers: exactly this path' },
+    test: { type: 'string', description: \`the path of the test file you wrote, relative to the codeline root (planned: \${cand.target})\` } } }
   for (let i = 0; i < PLAN.repro.maxAttempts; i++) {
     const model = PLAN.repro.ladder[Math.min(i, PLAN.repro.ladder.length - 1)]
     const a = await agentCall(relativize(cand.prompt) + fb.typecheck + fb.assertions + fb.commit_rejected,
-      { agentType: PLAN.agents.reproWriter, model, schema: SCHEMAS.repro, label: \`repro test #\${i + 1}\`, phase: 'Writer' })
+      { agentType: PLAN.agents.reproWriter, model, schema, label: \`repro test #\${i + 1}\`, phase: 'Writer' })
     const v = (a && a.validation) || { valid: false, committed: false, class: 'no_file', feedback: '', reason: a ? 'the answer carries no validation' : 'the agent ended without an answer' }
     if (v.valid) return { committed: !!v.committed, attempts: i + 1, test: a.test, reason: v.reason }
     const slot = v.class === 'unchanged' ? 'commit_rejected' : v.class

package/hooks/repro-checks.js

@@ -20,7 +20,10 @@ function args(argv) { const a = {}; for (let i = 0; i < argv.length; i += 2) a[a
 function judge(a, input) {
   const pj = JSON.parse(fs.readFileSync(a.pipeline, 'utf8'));
   const prep = JSON.parse(fs.readFileSync(path.join(a.state, 'repro-prep.json'), 'utf8'));
-  const source = String(input.source || ''); const test = String(input.test || '');
+  const source = String(input.source || '');
+  // a test named by its absolute path inside the codeline is that codeline-relative path (B61); anything else stays as named
+  const named = String(input.test || ''); const rel = path.isAbsolute(named) ? path.relative(pj.codeline, named) : named;
+  const test = rel && !rel.startsWith('..') && !path.isAbsolute(rel) ? rel : named;
   const cand = (prep.candidates || {})[source];
   if (!cand) return { valid: false, committed: false, class: 'no_file', feedback: '', reason: `'${source}' is not a source this change offered for a test` };
   const allowed = new Set([cand.target]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amsd-pipeline-workflow",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "The AMSD pipeline as Claude workflows. Installed by the amsd-pipeline installer: install.sh --workflows",
   "license": "UNLICENSED",
   "bin": {