@opengsd/gsd-pi 1.3.0 → 1.4.0

package/dist/resources/extensions/gsd/auto-prompts.js

@@ -10,7 +10,7 @@
 import { loadFile, parseContinue, parseSummary, loadActiveOverrides, formatOverridesSection } from "./files.js";
 import { hasVerdict, extractVerdict } from "./verdict-parser.js";
 import { loadPrompt, inlineTemplate } from "./prompt-loader.js";
-import { resolveMilestoneFile, resolveSliceFile, resolveSlicePath, resolveTasksDir, resolveTaskFiles, resolveTaskFile, relMilestoneFile, relSliceFile, relSlicePath, relMilestonePath, resolveGsdRootFile, relGsdRootFile, resolveRuntimeFile, } from "./paths.js";
+import { resolveMilestoneFile, resolveSliceFile, resolveSlicePath, resolveTasksDir, resolveTaskFiles, resolveTaskFile, relMilestoneFile, relSliceFile, relSlicePath, relMilestonePath, relTaskFile, resolveGsdRootFile, relGsdRootFile, resolveRuntimeFile, } from "./paths.js";
 import { resolveInlineLevel, loadEffectiveGSDPreferences } from "./preferences.js";
 import { isContextModeEnabled } from "./preferences-types.js";
 import { parseRoadmap } from "./parsers-legacy.js";
@@ -23,6 +23,7 @@ import { formatDecisionsCompact, formatRequirementsCompact } from "./structured-
 import { readPhaseAnchor, formatAnchorForPrompt } from "./phase-anchor.js";
 import { composeContextModeInstructions, composeContractedUnitContext, composeInlinedContext, composeToolSurfaceInstructions, composeUnitContext, } from "./unit-context-composer.js";
 import { resolveManifest } from "./unit-context-manifest.js";
+import { resolveSliceResearchLocation } from "./auto-artifact-paths.js";
 import { compileUnitContextContract } from "./tool-contract.js";
 import { readCompactionSnapshot } from "./compaction-snapshot.js";
 import { logWarning } from "./workflow-logger.js";
@@ -247,14 +248,21 @@ function requireComposedArtifactBlock(blocks, unitType, key) {
     return block.body;
 }
 function renderExecuteTaskOnDemandContext(base, mid, sid, artifacts) {
-    if (!artifacts.includes("slice-research"))
-        return "";
-    const researchPath = relSliceFile(base, mid, sid, "RESEARCH");
-    return [
-        "## On-demand Context",
-        "",
-        `Slice research is available at \`${researchPath}\`. Read it only if the inlined task plan, slice plan excerpt, and carry-forward context do not explain a required implementation detail.`,
-    ].join("\n");
+    if (!artifacts.includes("slice-research")) {
+        return { text: "", skipReason: "not declared by contract" };
+    }
+    const research = resolveSliceResearchLocation(base, mid, sid);
+    if (!research.absolutePath || !research.relativePath) {
+        return { text: "", skipReason: "missing" };
+    }
+    return {
+        text: [
+            "## On-demand Context",
+            "",
+            `Slice research is available at \`${research.relativePath}\`. Read it only if the inlined task plan, slice plan excerpt, and carry-forward context do not explain a required implementation detail.`,
+        ].join("\n"),
+        skipReason: null,
+    };
 }
 // ─── Executor Constraints ─────────────────────────────────────────────────────
 /**
@@ -1167,19 +1175,29 @@ export function extractSliceExecutionExcerpt(content, relPath) {
     return parts.join("\n");
 }
 // ─── Prior Task Summaries ──────────────────────────────────────────────────
-export async function getPriorTaskSummaryPaths(mid, sid, currentTid, base) {
+/** Resolve where task SUMMARY files live (legacy tasks/ subdir or flat-phase phase dir). */
+function resolveTaskSummariesLocation(base, mid, sid) {
+    const slicePath = resolveSlicePath(base, mid, sid);
+    if (!slicePath)
+        return null;
     const tDir = resolveTasksDir(base, mid, sid);
-    if (!tDir)
+    const sRel = relSlicePath(base, mid, sid);
+    if (tDir)
+        return { dir: tDir, relPrefix: `${sRel}/tasks` };
+    return { dir: slicePath, relPrefix: sRel };
+}
+export async function getPriorTaskSummaryPaths(mid, sid, currentTid, base) {
+    const loc = resolveTaskSummariesLocation(base, mid, sid);
+    if (!loc)
         return [];
-    const summaryFiles = resolveTaskFiles(tDir, "SUMMARY");
+    const summaryFiles = resolveTaskFiles(loc.dir, "SUMMARY");
     const currentNum = parseInt(currentTid.replace(/^T/, ""), 10);
-    const sRel = relSlicePath(base, mid, sid);
     return summaryFiles
         .filter(f => {
         const num = parseInt(f.replace(/^T/, ""), 10);
         return num < currentNum;
     })
-        .map(f => `${sRel}/tasks/${f}`);
+        .map(f => `${loc.relPrefix}/${f}`);
 }
 /**
  * Get carry-forward summary paths scoped to a task's derived dependencies.
@@ -1196,11 +1214,10 @@ export async function getDependencyTaskSummaryPaths(mid, sid, currentTid, depend
     if (dependsOn.length === 0) {
         return getPriorTaskSummaryPaths(mid, sid, currentTid, base);
     }
-    const tDir = resolveTasksDir(base, mid, sid);
-    if (!tDir)
+    const loc = resolveTaskSummariesLocation(base, mid, sid);
+    if (!loc)
         return [];
-    const summaryFiles = resolveTaskFiles(tDir, "SUMMARY");
-    const sRel = relSlicePath(base, mid, sid);
+    const summaryFiles = resolveTaskFiles(loc.dir, "SUMMARY");
     const depSet = new Set(dependsOn.map((d) => d.toUpperCase()));
     return summaryFiles
         .filter((f) => {
@@ -1208,7 +1225,7 @@ export async function getDependencyTaskSummaryPaths(mid, sid, currentTid, depend
         const tid = f.replace(/-SUMMARY\.md$/i, "").toUpperCase();
         return depSet.has(tid);
     })
-        .map((f) => `${sRel}/tasks/${f}`);
+        .map((f) => `${loc.relPrefix}/${f}`);
 }
 // ─── Adaptive Replanning Checks ────────────────────────────────────────────
 /**
@@ -1450,7 +1467,13 @@ export async function buildDiscussMilestonePrompt(mid, midTitle, base, structure
     const draftPath = resolveMilestoneFile(base, mid, "CONTEXT-DRAFT");
     const draftContent = draftPath ? await loadFile(draftPath) : null;
     if (includeDraftSeed && draftContent) {
-        return `${promptWithContextMode}\n\n## Prior Discussion (Draft Seed)\n\nThe following draft was captured from a prior multi-milestone discussion. Use it as seed material — the user has already provided this context. Start with a brief reflection on what the draft covers, then probe for any gaps or open questions before writing the full CONTEXT.md.\n\n${draftContent}`;
+        const draftRelPath = relMilestoneFile(base, mid, "CONTEXT-DRAFT");
+        const draftSeed = `### Prior Discussion Draft\nSource: \`${draftRelPath}\`\n\n${draftContent.trim()}`;
+        const cappedDraftSeed = capPreamble(draftSeed);
+        const truncationNote = cappedDraftSeed !== draftSeed
+            ? `\n\n_(Draft seed truncated; read the full draft at \`${draftRelPath}\` if needed.)_`
+            : "";
+        return `${promptWithContextMode}\n\n## Prior Discussion (Draft Seed)\n\nThe following draft was captured from a prior multi-milestone discussion. Use it as seed material — the user has already provided this context. Start with a brief reflection on what the draft covers, then probe for any gaps or open questions before writing the full CONTEXT.md.\n\n${cappedDraftSeed}${truncationNote}`;
     }
     return promptWithContextMode;
 }
@@ -1696,7 +1719,7 @@ export async function buildResearchMilestonePrompt(mid, midTitle, base) {
     return loadPrompt("research-milestone", {
         workingDirectory: base,
         milestoneId: mid, milestoneTitle: midTitle,
-        milestonePath: relMilestonePath(base, mid),
+        milestonePath: relMilestonePath(base, mid, midTitle),
         contextPath: relMilestoneFile(base, mid, "CONTEXT"),
         outputPath: join(base, outputRelPath),
         inlinedContext,
@@ -1844,7 +1867,7 @@ export async function buildPlanMilestonePrompt(mid, midTitle, base, level) {
     return loadPrompt("plan-milestone", {
         workingDirectory: base,
         milestoneId: mid, milestoneTitle: midTitle,
-        milestonePath: relMilestonePath(base, mid),
+        milestonePath: relMilestonePath(base, mid, midTitle),
         contextPath: contextRel,
         researchPath: researchRel,
         researchOutputPath,
@@ -2259,7 +2282,7 @@ export async function buildExecuteTaskPrompt(mid, sid, sTitle, tid, tTitle, base
         : "- (no prior tasks)";
     const taskPlanPath = resolveTaskFile(base, mid, sid, tid, "PLAN");
     const taskPlanContent = taskPlanPath ? await loadFile(taskPlanPath) : null;
-    const taskPlanRelPath = relSlicePath(base, mid, sid) + `/tasks/${tid}-PLAN.md`;
+    const taskPlanRelPath = relTaskFile(base, mid, sid, tid, "PLAN");
     const taskPlanContext = taskPlanContent
         ? [
             "## Inlined Task Plan (authoritative local execution contract)",
@@ -2396,8 +2419,9 @@ export async function buildExecuteTaskPrompt(mid, sid, sTitle, tid, tTitle, base
     const slicePlanExcerpt = requireComposedArtifactBlock(contractedContext.blocks, "execute-task", "slice-plan");
     const contractedCarryForward = requireComposedArtifactBlock(contractedContext.blocks, "execute-task", "prior-task-summaries");
     const contractedTemplates = requireComposedArtifactBlock(contractedContext.blocks, "execute-task", "templates");
-    const onDemandContext = renderExecuteTaskOnDemandContext(base, mid, sid, contractedContext.onDemand);
-    trackPromptContext(contextTelemetry, "slice-research", onDemandContext ? "on-demand" : "skipped", onDemandContext, onDemandContext ? undefined : "not declared by contract");
+    const onDemandResult = renderExecuteTaskOnDemandContext(base, mid, sid, contractedContext.onDemand);
+    const onDemandContext = onDemandResult.text;
+    trackPromptContext(contextTelemetry, "slice-research", onDemandContext ? "on-demand" : "skipped", onDemandContext, onDemandContext ? undefined : onDemandResult.skipReason ?? undefined);
     const prompt = loadPrompt("execute-task", {
         overridesSection,
         runtimeContext,
@@ -2474,17 +2498,16 @@ export async function buildCompleteSlicePrompt(mid, midTitle, sid, sTitle, base,
                     return body;
                 }
             case "prior-task-summaries": {
-                const tDir = resolveTasksDir(base, mid, sid);
-                if (!tDir) {
+                const loc = resolveTaskSummariesLocation(base, mid, sid);
+                if (!loc) {
                     trackPromptContext(contextTelemetry, "prior-task-summaries", "skipped", null, "missing tasks dir");
                     return null;
                 }
-                const summaryFiles = resolveTaskFiles(tDir, "SUMMARY").sort();
-                const sRel = relSlicePath(base, mid, sid);
+                const summaryFiles = resolveTaskFiles(loc.dir, "SUMMARY").sort();
                 const blocks = [];
                 for (const file of summaryFiles) {
-                    const absPath = join(tDir, file);
-                    const relPath = `${sRel}/tasks/${file}`;
+                    const absPath = join(loc.dir, file);
+                    const relPath = `${loc.relPrefix}/${file}`;
                     const taskId = file.replace(/-SUMMARY\.md$/i, "");
                     blocks.push(await buildTaskSummaryExcerpt(absPath, relPath, taskId));
                 }
@@ -2734,8 +2757,10 @@ export async function buildCompleteMilestonePrompt(mid, midTitle, base, level) {
     trackPromptContext(contextTelemetry, "cap", cappedInlinedContext.length < rawInlinedContext.length ? "skipped" : "inline", null, cappedInlinedContext.length < rawInlinedContext.length ? `dropped ${rawInlinedContext.length - cappedInlinedContext.length} chars` : "within budget");
     const inlinedContext = prependContextModeToBlock("complete-milestone", base, cappedInlinedContext);
     emitPromptContextTelemetry("complete-milestone", contextTelemetry, inlinedContext);
-    const milestoneSummaryPath = join(base, `${relMilestonePath(base, mid)}/${mid}-SUMMARY.md`);
-    const learningsRelPath = join(relMilestonePath(base, mid), `${mid}-LEARNINGS.md`);
+    // Use relMilestoneFile to get the layout-aware filename (NN-SUFFIX.md for flat-phase,
+    // M001-SUFFIX.md for legacy) rather than manually appending the raw milestone id.
+    const milestoneSummaryPath = join(base, relMilestoneFile(base, mid, "SUMMARY"));
+    const learningsRelPath = relMilestoneFile(base, mid, "LEARNINGS");
     const learningsAbsPath = join(base, learningsRelPath);
     const extractLearningsSteps = buildExtractionStepsBlock({
         milestoneId: mid,
@@ -2965,8 +2990,9 @@ export async function buildValidateMilestonePrompt(mid, midTitle, base, level) {
     trackPromptContext(contextTelemetry, "cap", cappedInlinedContext.length < rawInlinedContext.length ? "skipped" : "inline", null, cappedInlinedContext.length < rawInlinedContext.length ? `dropped ${rawInlinedContext.length - cappedInlinedContext.length} chars` : "within budget");
     const inlinedContext = prependContextModeToBlock("validate-milestone", base, cappedInlinedContext);
     emitPromptContextTelemetry("validate-milestone", contextTelemetry, inlinedContext);
-    const validationOutputPath = join(base, `${relMilestonePath(base, mid)}/${mid}-VALIDATION.md`);
-    const roadmapOutputPath = `${relMilestonePath(base, mid)}/${mid}-ROADMAP.md`;
+    // Use relMilestoneFile for layout-aware filenames (NN-SUFFIX.md flat-phase, M001-SUFFIX.md legacy).
+    const validationOutputPath = join(base, relMilestoneFile(base, mid, "VALIDATION"));
+    const roadmapOutputPath = relMilestoneFile(base, mid, "ROADMAP");
     // Every milestone validation turn owns MV01–MV04 unconditionally: the
     // registry is the source of truth for which gates the validator must
     // address, and the block below is what the template renders so the
@@ -3009,17 +3035,16 @@ export async function buildReplanSlicePrompt(mid, midTitle, sid, sTitle, base) {
     inlined.push(await inlineFile(slicePlanPath, slicePlanRel, "Current Slice Plan"));
     // Find the blocker task summary — the completed task with blocker_discovered: true
     let blockerTaskId = "";
-    const tDir = resolveTasksDir(base, mid, sid);
-    if (tDir) {
-        const summaryFiles = resolveTaskFiles(tDir, "SUMMARY").sort();
+    const summaryLoc = resolveTaskSummariesLocation(base, mid, sid);
+    if (summaryLoc) {
+        const summaryFiles = resolveTaskFiles(summaryLoc.dir, "SUMMARY").sort();
         for (const file of summaryFiles) {
-            const absPath = join(tDir, file);
+            const absPath = join(summaryLoc.dir, file);
             const content = await loadFile(absPath);
             if (!content)
                 continue;
             const summary = parseSummary(content);
-            const sRel = relSlicePath(base, mid, sid);
-            const relPath = `${sRel}/tasks/${file}`;
+            const relPath = `${summaryLoc.relPrefix}/${file}`;
             if (summary.frontmatter.blocker_discovered) {
                 blockerTaskId = summary.frontmatter.id || file.replace(/-SUMMARY\.md$/i, "");
                 inlined.push(await buildTaskSummaryExcerpt(absPath, relPath, blockerTaskId, { blocker: true }));
@@ -3035,7 +3060,7 @@ export async function buildReplanSlicePrompt(mid, midTitle, sid, sTitle, base) {
     if (replanOverridesInline)
         inlined.unshift(replanOverridesInline);
     const inlinedContext = prependContextModeToBlock("replan-slice", base, capPreamble(`## Inlined Context (preloaded — do not re-read these files)\n\n${inlined.join("\n\n---\n\n")}`));
-    const replanPath = join(base, `${relSlicePath(base, mid, sid)}/${sid}-REPLAN.md`);
+    const replanPath = join(base, relSliceFile(base, mid, sid, "REPLAN"));
     // Build capture context for replan prompt (captures that triggered this replan)
     let captureContext = "(none)";
     try {
@@ -3351,7 +3376,7 @@ opts) {
         const depPaths = await getDependencyTaskSummaryPaths(mid, sid, tid, node?.dependsOn ?? [], base);
         const taskPlanPath = resolveTaskFile(base, mid, sid, tid, "PLAN");
         const taskPlanContent = taskPlanPath ? await loadFile(taskPlanPath) : null;
-        const taskPlanRelPath = `${relSlicePath(base, mid, sid)}/tasks/${tid}-PLAN.md`;
+        const taskPlanRelPath = relTaskFile(base, mid, sid, tid, "PLAN");
         const taskPlanInline = taskPlanContent
             ? [
                 "## Inlined Task Plan (authoritative local execution contract)",