@dzhechkov/skills-feature-adr 1.3.64 → 1.3.65

package/.dz-manifest.json

@@ -13,7 +13,7 @@
       },
       {
         "path": "README.md",
-        "sha256": "b3095a1ad16fe17c103fb167e2844dabfe61f2a67dde3f797521c7916f41ddb3"
+        "sha256": "4a887b8fe2a52c75be7189ddeef4393dc2a8a872d747bdbde3caba94488960dd"
       },
       {
         "path": "bin/cli.js",
@@ -25,7 +25,7 @@
       },
       {
         "path": "package.json",
-        "sha256": "98aaa022a135e9cf92c534e191c96bdcd0254bf1ea8d7aba42745a82071ad4c2"
+        "sha256": "1f95669ddfb76990bfeb0e1be603c27651f47a9233020ed10c37e8705543b688"
       },
       {
         "path": "scripts/sync-templates.js",
@@ -317,7 +317,7 @@
       },
       {
         "path": "templates/.claude/workflows/feature-adr.js",
-        "sha256": "e02d770867542d0d1505baca64c3008ff1bbc0048b291bd30be7ccc41bdc06a1"
+        "sha256": "8afa77bb421ea4052eec0feab5f49edc1f2b1cea69238971f8ef1b3c8328a483"
       },
       {
         "path": "templates/lib/memory-protocol.md",
@@ -345,5 +345,5 @@
       }
     ]
   },
-  "signature": "ac1qFz1SL9en2rAGmTGAYgsXlnlNG7tX3LCKhQFdt0nngZoLWW3LIT9pXBKrSAwKLAsqLXc1EVcmx2fmupeyAg=="
+  "signature": "3OT+p3Tm5tBnn9T2RKEPlCnhCiaemKAOraeoPA4zYR28p1MpACnkeEmVyXPpwnqhG/a6uD/AjC/DrRuihMdHCw=="
 }

package/README.md

@@ -274,8 +274,13 @@ Workflow({ scriptPath: '.claude/workflows/feature-adr.js',
 
 > Note: under the override, coder **and** QE both run on Codex — the cross-model self-QE guard is
 > consciously suspended (a Claude QE reviewer is exactly the agent that dies under limit pressure). The
-> ` (usage-switched)` suffix + `usageEvents` keep it auditable; re-run an independent QE after limits
-> reset if the switch fired during Step 8.
+> ` (usage-switched)` suffix + `usageEvents` keep it auditable — and since v1.3.65 the suspension is a
+> **machine debt**, not a note: the run writes `features/<slug>/.fa-state/reqe-due.json` (idempotent,
+> run-stamped, crash-resume-safe) and returns `reqeDue: true`; after limits reset, `dz usage` shows a
+> `re-QE due:` line, `dz reqe --slug <s>` prints the cross-family review brief, and
+> `dz reqe --slug <s> --done --report <f>` settles FAIL-CLOSED — only against an existing graded
+> report, never the run's own 08_qe_report.md (real-path and inode compared). RU: «перепроверь другой
+> семьёй потом» стало долгом на диске с полным жизненным циклом.
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dzhechkov/skills-feature-adr",
-  "version": "1.3.64",
+  "version": "1.3.65",
   "description": "Adaptive Feature Development skill pack for Claude Code — 11-step pipeline with Complexity Router (S/M/L/XL), ADR-driven architecture, 15 agentic-qe skills, multi-agent fleet QE. Supports --full-qe, --full-qe-extended, --with-learning, and --knowledge-extractor modes.",
   "bin": {
     "skills-feature-adr": "./bin/cli.js"

package/sbom.json

@@ -35,7 +35,7 @@
       "hashes": [
         {
           "alg": "SHA-256",
-          "content": "b3095a1ad16fe17c103fb167e2844dabfe61f2a67dde3f797521c7916f41ddb3"
+          "content": "4a887b8fe2a52c75be7189ddeef4393dc2a8a872d747bdbde3caba94488960dd"
         }
       ]
     },
@@ -65,7 +65,7 @@
       "hashes": [
         {
           "alg": "SHA-256",
-          "content": "98aaa022a135e9cf92c534e191c96bdcd0254bf1ea8d7aba42745a82071ad4c2"
+          "content": "1f95669ddfb76990bfeb0e1be603c27651f47a9233020ed10c37e8705543b688"
         }
       ]
     },
@@ -795,7 +795,7 @@
       "hashes": [
         {
           "alg": "SHA-256",
-          "content": "e02d770867542d0d1505baca64c3008ff1bbc0048b291bd30be7ccc41bdc06a1"
+          "content": "8afa77bb421ea4052eec0feab5f49edc1f2b1cea69238971f8ef1b3c8328a483"
         }
       ]
     },

package/templates/.claude/workflows/feature-adr.js

@@ -1002,7 +1002,11 @@ const qeModel = resolveStageModel('qe')
 // Single tested source of truth (feature-adr-routing.ts:qeShouldUseCodex) — closes the self-QE hole where
 // the legacy qeReviewer='codex' knob used to re-route QE back to codex even when the CODER was codex.
 if (MODELS.qe === undefined && QE_REVIEWER === 'codex' && coderIsCodex()) log('QE: coder is codex — enforcing cross-model Claude QE (ignoring qeReviewer=codex to avoid self-review)')
-const qeIsCodex = qeShouldUseCodex()
+// reqe QE #3: the usage override can resolve the QE stage to codex EVEN when qeShouldUseCodex()
+// (which reads the pre-override config) says claude — dispatching that through the "claude" branch
+// would run the stub wrapper AND mislabel qeReviewerUsed='claude' (a codex-on-codex review recorded
+// as cross-family). The resolved agentType is the truth; either signal routes the codex branch.
+const qeIsCodex = qeShouldUseCodex() || (qeModel && qeModel.agentType === 'codex:codex-rescue')
 const qeClaudeModel = qeIsCodex ? {} : qeModel
 const qeClaudeOpts = mergeOpts({ label: stageLabel('qe:brutal', qeClaudeModel), phase: 'QE', agentType: 'qe-code-reviewer', schema: QE }, qeClaudeModel)
 // qe checkpoint: COMPOSITE (verdict + reviewer identity) keyed on the CODE stage's result — a re-coded
@@ -1056,6 +1060,38 @@ if (qeStage && qeStage.modelUsed) modelsUsed.qe = qeStage.modelUsed + (resumedSt
 const claimGate = step8ClaimGate(qe && qe.claimCheck ? qe.claimCheck : null)
 log(claimGate.note)
 
+// ── re-QE debt emission (backlog 6b40e667) — mirror of harness-core/src/reqe.ts ──
+// The cross-model guard was consciously SUSPENDED when the usage override made coder and QE the
+// same family (FR-2.9). Record that as a machine DEBT (features/<slug>/.fa-state/reqe-due.json) so
+// `dz reqe` / `dz usage` surface it after limits reset — a doc instruction on the weakest detection
+// layer becomes a fact on disk. Emitted ONLY for the actual same-family-under-override case: a
+// switch that kept cross-family QE, or the codex-unavailable Claude belt (no override), creates no
+// debt. A RESUMED qe never re-emits (the original run emitted; a settlement must not be clobbered).
+let reqeDue = false
+{
+  const reqeFamOf = function (s) { return /codex|gpt|openai/i.test(String(s || '')) ? 'openai' : 'claude' }
+  const qeLabel = String(modelsUsed.qe || '')
+  if (/\(usage-switched\)/.test(qeLabel) && reqeFamOf(coderUsed) === reqeFamOf(qeReviewerUsed)) {
+    reqeDue = true
+    const reqeDebt = { schema: 'reqe-due-1', slug: SLUG, coderFamily: reqeFamOf(coderUsed), qeFamily: reqeFamOf(qeReviewerUsed), qeGrade: (qe && qe.grade) ? String(qe.grade) : null, reason: 'usage-switched self-review: Step-8 QE ran on the coder’s own family under the limit override (FR-2.9)', emittedAt: null, runStamp: qeHash }
+    // IDEMPOTENT + VERIFIED (reqe QE #2 + r2 #2/#3): a RESUMED qe re-runs this block (the original
+    // run may have died between the qe checkpoint and this write), but: an existing due file is
+    // never clobbered; a settlement blocks re-emission ONLY when it carries THIS run's runStamp (an
+    // OLD settlement must not immunize the slug against a FRESH run's debt); the write itself is
+    // noclobber `set -C` (O_EXCL — a concurrent emitter or a planted/dangling symlink cannot be
+    // written through). The agent echoes the written file back; an unverifiable write is logged
+    // LOUDLY, and reqeDue stays true in the result either way.
+    const dueQ = shq(FDIR + '/.fa-state/reqe-due.json')
+    const emitCmd = 'if [ -e ' + dueQ + ' ]; then echo REQE-EXISTS; elif grep -qs ' + shq(qeHash) + ' ' + shq(FDIR + '/.fa-state') + '/reqe-settled*.json 2>/dev/null; then echo REQE-SETTLED-THIS-RUN; else mkdir -p ' + shq(FDIR + '/.fa-state') + ' && printf %s ' + shq(JSON.stringify(reqeDebt)) + ' | sed "s/\\"emittedAt\\":null/\\"emittedAt\\":\\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\\"/" | (set -C; cat > ' + dueQ + ') && cat ' + dueQ + '; fi'
+    const emitOut = await agent('Run EXACTLY this one shell command via your Bash tool and return its stdout VERBATIM, nothing else: ' + emitCmd, { label: 'reqe:emit', phase: 'QE', effort: 'low' })
+    const emitText = String(emitOut || '')
+    if (/REQE-SETTLED-THIS-RUN/.test(emitText)) log('re-QE debt: THIS run’s debt was already settled — not re-opened')
+    else if (/REQE-EXISTS/.test(emitText)) log('re-QE debt: already recorded for ' + SLUG + ' — not overwritten')
+    else if (emitText.indexOf('reqe-due-1') !== -1) log('re-QE DEBT recorded: Step-8 ran same-family under the usage override — after limits reset run `dz reqe --slug ' + SLUG + '` for the independent cross-family pass')
+    else log('re-QE debt write UNVERIFIED (agent returned no readback) — the debt may be missing on disk; reqeDue=true is still reported, record it manually via features/' + SLUG + '/.fa-state/reqe-due.json')
+  }
+}
+
 // ── AUTO-COST outcome recording (feature learned-cost-routing) ──
 // The two-phase label lands here: every auto-cost stage that produced an artifact records a PROVISIONAL (i);
 // the CODE stage's authoritative gate (ii) is the Step-8 QE grade (A/B = pass) — a produced-but-gate-FAILED
@@ -1275,6 +1311,7 @@ return {
   selfLearning: 'recall@Step0 + teach@Step8 (mandatory)',
   resumedStages: resumedStages,
   checkpointing: CHECKPOINTS_ON ? RESUME_MODE : 'off',
+  reqeDue: reqeDue,
   brain: BRAIN,
   polymorphism: POLY.hasManifest ? POLY.report : null,
   claimGate: claimGate,