parallel-codex-tui 0.1.3 → 0.1.4

package/dist/orchestrator/prompts.js

@@ -1,3 +1,20 @@
+export function buildMainPrompt(input) {
+    const role = roleConfig(input.role, "Main", [
+        "Answer the user directly for simple chat and explanation requests."
+    ]);
+    return [
+        `# Role: ${role.title}`,
+        "",
+        ...instructionLines(role.instructions),
+        ...(input.context?.trim()
+            ? ["", "# Active task context", "", input.context.trim()]
+            : []),
+        "",
+        "User request:",
+        input.request,
+        ""
+    ].join("\n");
+}
 export function buildJudgePrompt(input) {
     const role = roleConfig(input.role, "Judge", [
         "You clarify requirements and write task files. Do not implement code."
@@ -9,6 +26,12 @@ export function buildJudgePrompt(input) {
         "",
         `Task directory: ${input.taskDir}`,
         ...(input.workerDir ? [`Worker directory: ${input.workerDir}`] : []),
+        ...(input.workspaceDir ? [
+            `Project workspace (read-only): ${input.workspaceDir}`,
+            "Inspect the project workspace when needed, but never modify it. Write only the Judge artifacts listed below.",
+            "Actors execute in isolated feature workspaces. In every artifact, use logical project root to mean the Actor's assigned feature workspace/current working directory.",
+            "Never put the absolute live workspace path into implementation instructions or acceptance paths."
+        ] : []),
         ...turnLines(input.turn),
         "",
         "Write these files in the worker directory above:",
@@ -17,6 +40,19 @@ export function buildJudgePrompt(input) {
         "- acceptance.md",
         "- actor-brief.md",
         "- critic-brief.md",
+        "- features.json",
+        "",
+        "Markdown artifact contract:",
+        "- requirements.md must use list items with stable requirement ids, for example: - [R-001] one actionable requirement",
+        "- plan.md must use ordered steps with stable plan ids, for example: 1. [P-001] one concrete implementation step",
+        "- acceptance.md must use list items with stable acceptance ids and related requirement ids, for example: - [A-001] [R-001] one observable check or command",
+        "- actor-brief.md and critic-brief.md must contain concrete role guidance below their headings",
+        "- Do not leave TODO, TBD, 待定, or placeholder-only content in any artifact",
+        "",
+        "features.json must contain version 1 and at most 8 features.",
+        "Use safe lowercase ids made from letters, numbers, and hyphens.",
+        "List dependencies in \"depends_on\"; independent features can run in parallel.",
+        "Example: {\"version\":1,\"features\":[{\"id\":\"ui\",\"title\":\"UI\",\"description\":\"Build the interface\",\"depends_on\":[]}]}",
         "",
         "User request:",
         input.request,
@@ -34,6 +70,13 @@ export function buildActorPrompt(input) {
         "",
         `Task directory: ${input.taskDir}`,
         `Judge directory: ${input.judgeDir}`,
+        ...(input.workspaceDir ? [
+            `Feature workspace: ${input.workspaceDir}`,
+            "The feature workspace above is the logical project root for this run.",
+            "Resolve every project-root, repository-root, or current-project path to this exact feature workspace.",
+            "Never write implementation files to the shared live workspace or any parent of the task directory.",
+            "Keep all implementation changes inside this feature workspace. Use task and feature directories only for coordination files."
+        ] : []),
         ...turnLines(input.turn),
         ...featureLines(input.feature),
         "",
@@ -49,7 +92,7 @@ export function buildActorPrompt(input) {
         "",
         "Feature mailbox writes:",
         "- actor-worklog.md with implementation notes for this feature.",
-        "- actor-replies.jsonl with one JSON object per Critic finding you fixed.",
+        '- actor-replies.jsonl with one JSON object per Critic finding you fixed: {"finding_id":"C-001","status":"fixed","notes":"what changed"}.',
         "",
         input.revision ? `Revision request:\n${input.revision}` : "No Critic revision request is active.",
         "",
@@ -70,6 +113,12 @@ export function buildCriticPrompt(input) {
         `Task directory: ${input.taskDir}`,
         `Judge directory: ${input.judgeDir}`,
         `Actor directory: ${input.actorDir ?? ""}`,
+        ...(input.workspaceDir ? [
+            `Review workspace: ${input.workspaceDir}`,
+            "This is a disposable review copy of the Actor feature workspace and is the logical project root for review.",
+            "Do not modify implementation files. Any implementation changes made in this review copy are discarded.",
+            "Do not modify the Actor feature workspace or live workspace."
+        ] : []),
         ...turnLines(input.turn),
         ...featureLines(input.feature),
         "",
@@ -78,7 +127,71 @@ export function buildCriticPrompt(input) {
         "",
         "Write review.md in your worker directory. Include APPROVED when no blocking findings remain.",
         "If revision is required, include REVISION_REQUIRED and a concise fix list.",
-        "Write critic-findings.jsonl in the feature mailbox with one JSON object per blocking issue.",
+        'Write critic-findings.jsonl in the feature mailbox with one JSON object per blocking issue: {"id":"C-001","severity":"blocker","summary":"what must change"}.',
+        "",
+        "User request:",
+        input.request,
+        ""
+    ].join("\n");
+}
+export function buildWaveCriticPrompt(input) {
+    const role = waveRoleConfig(input.role, "Wave Critic", [
+        "Verify the combined feature result against all Judge requirements before it reaches the live workspace."
+    ]);
+    return [
+        `# Role: ${role.title}`,
+        "",
+        ...instructionLines(role.instructions),
+        "",
+        `Task directory: ${input.taskDir}`,
+        `Judge directory: ${input.judgeDir}`,
+        `Worker directory: ${input.workerDir}`,
+        `Combined verification workspace: ${input.workspaceDir}`,
+        `Wave: ${input.wave}/${input.waves}`,
+        `Features in this wave: ${input.featureIds.join(", ")}`,
+        ...turnLines(input.turn),
+        "",
+        "Live workspace has not been updated. Review only the combined verification workspace.",
+        "Treat the combined verification workspace as the logical project root for this review.",
+        "Read Judge requirements.md, plan.md, acceptance.md, critic-brief.md, and every feature decisions.md.",
+        "Run relevant tests, builds, and cross-feature checks in the combined verification workspace.",
+        "Do not modify implementation files.",
+        "",
+        "Write review.md in the worker directory.",
+        "Include APPROVED only when the combined result satisfies the full request and Judge acceptance.md.",
+        "Otherwise include REVISION_REQUIRED with a concise, actionable fix list.",
+        "Do not omit the decision marker.",
+        "",
+        "User request:",
+        input.request,
+        ""
+    ].join("\n");
+}
+export function buildWaveActorPrompt(input) {
+    const role = waveRoleConfig(input.role, "Wave Actor", [
+        "Resolve combined integration findings without reopening approved feature scope unnecessarily."
+    ]);
+    return [
+        `# Role: ${role.title}`,
+        "",
+        ...instructionLines(role.instructions),
+        "",
+        `Task directory: ${input.taskDir}`,
+        `Judge directory: ${input.judgeDir}`,
+        `Worker directory: ${input.workerDir}`,
+        `Combined integration workspace: ${input.workspaceDir}`,
+        `Wave: ${input.wave}/${input.waves}`,
+        `Features in this wave: ${input.featureIds.join(", ")}`,
+        ...turnLines(input.turn),
+        "",
+        "Modify only the combined integration workspace. Do not modify the live workspace or isolated feature workspaces.",
+        "Treat the combined integration workspace as the logical project root for this revision.",
+        "Read Judge requirements and acceptance, feature decisions, and the Wave Critic review below.",
+        "Run relevant verification after fixing the combined result.",
+        "Write worklog.md and patch.diff in the worker directory.",
+        "",
+        "Wave Critic review:",
+        input.review?.trim() || "REVISION_REQUIRED\nNo review details were provided.",
         "",
         "User request:",
         input.request,
@@ -91,6 +204,12 @@ function roleConfig(role, title, instructions) {
         instructions: role?.instructions?.length ? role.instructions : instructions
     };
 }
+function waveRoleConfig(role, title, instructions) {
+    return {
+        title: role ? `${role.title} · Wave` : title,
+        instructions: role?.instructions?.length ? role.instructions : instructions
+    };
+}
 function instructionLines(instructions) {
     if (instructions.length === 1) {
         return [instructions[0]];
@@ -103,6 +222,11 @@ function featureLines(feature) {
     }
     return [
         `Feature id: ${feature.featureId}`,
+        `Feature title: ${feature.featureTitle}`,
+        `Feature description: ${feature.featureDescription}`,
+        `Feature specification: ${feature.featureSpecPath}`,
+        `Feature dependencies: ${feature.featureDependencies.length > 0 ? feature.featureDependencies.join(", ") : "(none)"}`,
+        "Work only on this feature scope and honor completed dependency outputs.",
         `Feature directory: ${feature.featureDir}`,
         `Actor/Critic dialogue log: ${feature.dialoguePath}`,
         `Actor feature worklog: ${feature.actorWorklogPath}`,

package/dist/orchestrator/supervisor-summary.js

@@ -13,6 +13,8 @@ export async function buildSupervisorSummary(dirs) {
     const findings = dirs.featureCriticFindingsPath
         ? await readTextIfExists(dirs.featureCriticFindingsPath)
         : "";
+    const changedFiles = changedFilesSummary(dirs.changedPaths ?? []);
+    const verification = dirs.verification?.trim() || verificationEvidence(review);
     return [
         "Complex task completed.",
         "",
@@ -22,17 +24,69 @@ export async function buildSupervisorSummary(dirs) {
         "Actor work:",
         excerpt(worklog),
         "",
+        "Changed files:",
+        excerpt(changedFiles),
+        "",
         "Critic review:",
         excerpt(review),
         "",
+        "Verification:",
+        excerpt(verification),
+        "",
         "Critic findings:",
         excerpt(findings)
     ].join("\n");
 }
+function changedFilesSummary(paths) {
+    const unique = [...new Set(paths.map(sanitizeSummaryPath).filter(Boolean))].sort();
+    if (unique.length === 0) {
+        return "";
+    }
+    const visible = unique.slice(0, 50);
+    return [
+        ...visible.map((path) => `- ${path}`),
+        ...(unique.length > visible.length ? [`- ... and ${unique.length - visible.length} more`] : [])
+    ].join("\n");
+}
+function verificationEvidence(review) {
+    const lines = review.split(/\r?\n/).map((line) => line.trim()).filter(Boolean);
+    const decision = lines
+        .map(cleanReviewDecisionLine)
+        .find((line) => /^(?:APPROVED|REVISION_REQUIRED|REJECTED|FAILED)\b/i.test(line));
+    const evidence = lines.filter((line) => (!/^#{1,6}\s*(?:review|verification)\b/i.test(line)
+        && !/^(?:verification|tests?)\s*:$/i.test(line)
+        && /(?:`[^`]*(?:test|build|lint|typecheck|check)[^`]*`|\b(?:passed|verified|verification|tests?|build|lint|typecheck|smoke)\b)/i.test(line)
+        && cleanReviewDecisionLine(line) !== decision));
+    const uniqueEvidence = [...new Set(evidence)].slice(0, 12);
+    return [
+        ...(decision ? [`Critic decision: ${decision.match(/^(?:APPROVED|REVISION_REQUIRED|REJECTED|FAILED)/i)?.[0]?.toUpperCase() ?? decision}`] : []),
+        ...uniqueEvidence
+    ].join("\n");
+}
+function cleanReviewDecisionLine(line) {
+    return line
+        .trim()
+        .replace(/^#{1,6}\s+/, "")
+        .replace(/^[-*]\s+/, "")
+        .replace(/^\*\*([^*]+)\*\*$/, "$1")
+        .trim();
+}
+function sanitizeSummaryPath(path) {
+    return path
+        .replace(/\x1b\[[0-?]*[ -/]*[@-~]/g, "")
+        .replace(/[\u0000-\u001f\u007f]/g, "")
+        .trim();
+}
 function excerpt(text) {
-    const trimmed = text.trim();
+    const trimmed = escapeSupervisorSectionDelimiters(text).trim();
     if (!trimmed) {
         return "(empty)";
     }
-    return trimmed.length > 800 ? `${trimmed.slice(0, 797)}...` : trimmed;
+    const codePoints = Array.from(trimmed);
+    return codePoints.length > 800 ? `${codePoints.slice(0, 797).join("")}...` : trimmed;
+}
+function escapeSupervisorSectionDelimiters(text) {
+    return text.split(/\r?\n/).map((line) => (/^(?:Requirements|Actor work|Changed files|Critic review|Verification|Critic findings):\s*$/i.test(line.trim())
+        ? `> ${line.trim()}`
+        : line)).join("\n");
 }