Repository navigation
fix(dw-demo-foldback): record "read X, did not reach Y" and an Instructions used line per step (5.8.5) - #176
Merged
Conversation
…ctions used line per step (5.8.5)
Justinvolved
force-pushed
the
fix/foldback-read-not-reached
branch
from
September 29, 2026 10:44
868fca6 to
b94b921
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skill side of justdynamics/Truvio.Commerce.Foundry#1416 (a colleague build journal showed traps that were documented in the skills and not reached at the step, and a fold-back could not tell "rule missing" from "rule not reached").
dw-demo-foldback/references/fold-back-workflow.mdStep 1b "Dedup check": when the lesson is already recorded, record read X, did not reach Y, classify the miss as routing, and fix the route (symptom-index row, one-line pointer in the file that was read,SKILL.mdrouting) instead of adding content. A miss without a record of the files read is reported as unclassified.dw-demo-base/references/orchestrator.md"Persist progress": theflow-state.jsonexample gains a per-stepinstructions_usedmap, described as the step's "Instructions used:" line (the same line a human-readable run journal carries on every step).Not in this PR: the issue's capture-script parameters (
-EvidenceAgainst,-SuspectedComponent) and their Pester check are Foundry harness work.Checks
python scripts/validate-skills.py: 0 errors, 14 warnings (unchanged).node scripts/build-manifest.mjs --check: up to date (both skills aredynamo: false).Part of justdynamics/Truvio.Commerce.Foundry#1416