singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(gsd): stamp defer and milestone captures as executed after triage

Browse source
This commit is contained in:
Tibsfox 2026-04-05 10:38:29 -07:00
parent 092d1c0a9e
commit b46b113360
1 changed files with 19 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
src/resources/extensions/gsd/triage-resolution.ts
View file

@ -479,15 +479,18 @@ export function executeTriageResolutions(
}
}
// Also process deferred captures that target milestone IDs — create
// milestone directories so deriveState() discovers them.
const deferred = loadAllCaptures(basePath).filter(
c => c.status === "resolved" && !c.executed && c.classification === "defer",
// Also process deferred and milestone-class captures (#3542).
// A defer/milestone capture's "action" is the triage decision itself —
// once classified and resolved, the capture is done. The target milestone
// picks up the work naturally from its planning context.
const deferrable = loadAllCaptures(basePath).filter(
c => c.status === "resolved" && !c.executed &&
(c.classification === "defer" || c.classification === "milestone"),
);
if (deferred.length > 0) {
// Group deferred captures by target milestone
if (deferrable.length > 0) {
// Group captures that reference a specific milestone — create dirs as needed.
const byMilestone = new Map<string, CaptureEntry[]>();
for (const cap of deferred) {
for (const cap of deferrable) {
const target = cap.resolution?.match(/\b(M\d{3}(?:-[a-z0-9]{6})?)\b/)?.[1];
if (target) {
const list = byMilestone.get(target) ?? [];
@ -502,12 +505,18 @@ export function executeTriageResolutions(
if (created) {
result.deferredMilestones++;
result.actions.push(`Created milestone ${milestoneId} for ${captures.length} deferred capture(s)`);
for (const cap of captures) {
markCaptureExecuted(basePath, cap.id);
}
}
}
}
// Stamp ALL defer/milestone captures as executed (#3542 gaps 1-3).
// Previously only captures that triggered dir creation were stamped.
// Captures without a milestone ID in resolution text, or targeting an
// existing directory, were silently dropped — never stamped.
for (const cap of deferrable) {
if (!cap.executed) {
markCaptureExecuted(basePath, cap.id);
}
}
}
if (actionable.length === 0) return result;