@entelligentsia/forgecli 0.6.6 → 0.7.6
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +88 -0
- package/README.md +28 -1
- package/dist/bin/forge.js +20 -0
- package/dist/bin/forge.js.map +1 -1
- package/dist/extensions/forgecli/approve.d.ts +24 -0
- package/dist/extensions/forgecli/approve.js +202 -0
- package/dist/extensions/forgecli/approve.js.map +1 -0
- package/dist/extensions/forgecli/audience-gate.d.ts +4 -0
- package/dist/extensions/forgecli/audience-gate.js +8 -5
- package/dist/extensions/forgecli/audience-gate.js.map +1 -1
- package/dist/extensions/forgecli/collate.d.ts +24 -0
- package/dist/extensions/forgecli/collate.js +199 -0
- package/dist/extensions/forgecli/collate.js.map +1 -0
- package/dist/extensions/forgecli/commit.d.ts +24 -0
- package/dist/extensions/forgecli/commit.js +202 -0
- package/dist/extensions/forgecli/commit.js.map +1 -0
- package/dist/extensions/forgecli/fix-bug.d.ts +75 -0
- package/dist/extensions/forgecli/fix-bug.js +1133 -0
- package/dist/extensions/forgecli/fix-bug.js.map +1 -0
- package/dist/extensions/forgecli/forge-commands.js +7 -0
- package/dist/extensions/forgecli/forge-commands.js.map +1 -1
- package/dist/extensions/forgecli/forge-init.js +16 -8
- package/dist/extensions/forgecli/forge-init.js.map +1 -1
- package/dist/extensions/forgecli/forge-subagent.d.ts +29 -0
- package/dist/extensions/forgecli/forge-subagent.js +14 -1
- package/dist/extensions/forgecli/forge-subagent.js.map +1 -1
- package/dist/extensions/forgecli/hook-dispatcher.d.ts +53 -1
- package/dist/extensions/forgecli/hook-dispatcher.js +47 -1
- package/dist/extensions/forgecli/hook-dispatcher.js.map +1 -1
- package/dist/extensions/forgecli/hooks/post-init-hook.d.ts +15 -0
- package/dist/extensions/forgecli/hooks/post-init-hook.js +127 -0
- package/dist/extensions/forgecli/hooks/post-init-hook.js.map +1 -0
- package/dist/extensions/forgecli/hooks/post-sprint-hook.d.ts +37 -0
- package/dist/extensions/forgecli/hooks/post-sprint-hook.js +166 -0
- package/dist/extensions/forgecli/hooks/post-sprint-hook.js.map +1 -0
- package/dist/extensions/forgecli/index.js +47 -0
- package/dist/extensions/forgecli/index.js.map +1 -1
- package/dist/extensions/forgecli/review-code.d.ts +24 -0
- package/dist/extensions/forgecli/review-code.js +202 -0
- package/dist/extensions/forgecli/review-code.js.map +1 -0
- package/dist/extensions/forgecli/review-plan.d.ts +24 -0
- package/dist/extensions/forgecli/review-plan.js +203 -0
- package/dist/extensions/forgecli/review-plan.js.map +1 -0
- package/dist/extensions/forgecli/run-sprint.d.ts +18 -0
- package/dist/extensions/forgecli/run-sprint.js +33 -1
- package/dist/extensions/forgecli/run-sprint.js.map +1 -1
- package/dist/extensions/forgecli/run-task.d.ts +21 -2
- package/dist/extensions/forgecli/run-task.js +33 -9
- package/dist/extensions/forgecli/run-task.js.map +1 -1
- package/dist/extensions/forgecli/session-registry.d.ts +10 -0
- package/dist/extensions/forgecli/session-registry.js +9 -0
- package/dist/extensions/forgecli/session-registry.js.map +1 -1
- package/dist/extensions/forgecli/validate.d.ts +24 -0
- package/dist/extensions/forgecli/validate.js +202 -0
- package/dist/extensions/forgecli/validate.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/engine.d.ts +23 -0
- package/dist/extensions/forgecli/wf-engine/engine.js +384 -0
- package/dist/extensions/forgecli/wf-engine/engine.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/event-parser.d.ts +6 -0
- package/dist/extensions/forgecli/wf-engine/event-parser.js +29 -0
- package/dist/extensions/forgecli/wf-engine/event-parser.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/id-gen.d.ts +6 -0
- package/dist/extensions/forgecli/wf-engine/id-gen.js +17 -0
- package/dist/extensions/forgecli/wf-engine/id-gen.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/loader.d.ts +2 -0
- package/dist/extensions/forgecli/wf-engine/loader.js +100 -0
- package/dist/extensions/forgecli/wf-engine/loader.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/predicate.d.ts +7 -0
- package/dist/extensions/forgecli/wf-engine/predicate.js +36 -0
- package/dist/extensions/forgecli/wf-engine/predicate.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/prompt-compiler.d.ts +15 -0
- package/dist/extensions/forgecli/wf-engine/prompt-compiler.js +23 -0
- package/dist/extensions/forgecli/wf-engine/prompt-compiler.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/register.d.ts +9 -0
- package/dist/extensions/forgecli/wf-engine/register.js +59 -0
- package/dist/extensions/forgecli/wf-engine/register.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/remit-check.d.ts +6 -0
- package/dist/extensions/forgecli/wf-engine/remit-check.js +42 -0
- package/dist/extensions/forgecli/wf-engine/remit-check.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/state-store.d.ts +13 -0
- package/dist/extensions/forgecli/wf-engine/state-store.js +43 -0
- package/dist/extensions/forgecli/wf-engine/state-store.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/types.d.ts +66 -0
- package/dist/extensions/forgecli/wf-engine/types.js +2 -0
- package/dist/extensions/forgecli/wf-engine/types.js.map +1 -0
- package/dist/extensions/forgecli/wf-engine/worker.d.ts +11 -0
- package/dist/extensions/forgecli/wf-engine/worker.js +50 -0
- package/dist/extensions/forgecli/wf-engine/worker.js.map +1 -0
- package/dist/forge-payload/.base-pack/workflows/_fragments/context-injection.md +10 -4
- package/dist/forge-payload/.base-pack/workflows/fix_bug.md +12 -0
- package/dist/forge-payload/.schemas/bug.schema.json +4 -2
- package/dist/forge-payload/.schemas/event.schema.json +22 -3
- package/dist/forge-payload/commands/add-pipeline.md +342 -0
- package/dist/forge-payload/commands/add-task.md +269 -0
- package/dist/forge-payload/commands/ask.md +43 -0
- package/dist/forge-payload/commands/calibrate.md +356 -0
- package/dist/forge-payload/commands/config.md +202 -0
- package/dist/forge-payload/commands/enhance.md +38 -0
- package/dist/forge-payload/commands/health.md +225 -0
- package/dist/forge-payload/commands/init.md +165 -0
- package/dist/forge-payload/commands/materialize.md +119 -0
- package/dist/forge-payload/commands/migrate.md +160 -0
- package/dist/forge-payload/commands/quiz-agent.md +38 -0
- package/dist/forge-payload/commands/regenerate.md +673 -0
- package/dist/forge-payload/commands/remove.md +174 -0
- package/dist/forge-payload/commands/report-bug.md +191 -0
- package/dist/forge-payload/commands/store-query.md +73 -0
- package/dist/forge-payload/commands/store-repair.md +187 -0
- package/dist/forge-payload/commands/update-tools.md +56 -0
- package/dist/forge-payload/commands/update.md +1376 -0
- package/dist/forge-payload/tools/preflight-gate.cjs +2 -1
- package/dist/forge-payload/tools/read-verdict.cjs +41 -8
- package/dist/forge-payload/tools/store-cli.cjs +4 -3
- package/node_modules/argparse/CHANGELOG.md +216 -0
- package/node_modules/argparse/LICENSE +254 -0
- package/node_modules/argparse/README.md +84 -0
- package/node_modules/argparse/argparse.js +3707 -0
- package/node_modules/argparse/lib/sub.js +67 -0
- package/node_modules/argparse/lib/textwrap.js +440 -0
- package/node_modules/argparse/package.json +31 -0
- package/node_modules/cliui/CHANGELOG.md +121 -0
- package/node_modules/color-convert/CHANGELOG.md +54 -0
- package/node_modules/esprima/ChangeLog +235 -0
- package/node_modules/js-yaml/LICENSE +21 -0
- package/node_modules/js-yaml/README.md +247 -0
- package/node_modules/js-yaml/bin/js-yaml.js +126 -0
- package/node_modules/js-yaml/dist/js-yaml.js +3880 -0
- package/node_modules/js-yaml/dist/js-yaml.min.js +2 -0
- package/node_modules/js-yaml/dist/js-yaml.mjs +3856 -0
- package/node_modules/js-yaml/index.js +47 -0
- package/node_modules/js-yaml/lib/common.js +59 -0
- package/node_modules/js-yaml/lib/dumper.js +965 -0
- package/node_modules/js-yaml/lib/exception.js +55 -0
- package/node_modules/js-yaml/lib/loader.js +1733 -0
- package/node_modules/js-yaml/lib/schema/core.js +11 -0
- package/node_modules/js-yaml/lib/schema/default.js +22 -0
- package/node_modules/js-yaml/lib/schema/failsafe.js +17 -0
- package/node_modules/js-yaml/lib/schema/json.js +19 -0
- package/node_modules/js-yaml/lib/schema.js +121 -0
- package/node_modules/js-yaml/lib/snippet.js +101 -0
- package/node_modules/js-yaml/lib/type/binary.js +125 -0
- package/node_modules/js-yaml/lib/type/bool.js +35 -0
- package/node_modules/js-yaml/lib/type/float.js +97 -0
- package/node_modules/js-yaml/lib/type/int.js +156 -0
- package/node_modules/js-yaml/lib/type/map.js +8 -0
- package/node_modules/js-yaml/lib/type/merge.js +12 -0
- package/node_modules/js-yaml/lib/type/null.js +35 -0
- package/node_modules/js-yaml/lib/type/omap.js +44 -0
- package/node_modules/js-yaml/lib/type/pairs.js +53 -0
- package/node_modules/js-yaml/lib/type/seq.js +8 -0
- package/node_modules/js-yaml/lib/type/set.js +29 -0
- package/node_modules/js-yaml/lib/type/str.js +8 -0
- package/node_modules/js-yaml/lib/type/timestamp.js +88 -0
- package/node_modules/js-yaml/lib/type.js +66 -0
- package/node_modules/js-yaml/package.json +66 -0
- package/node_modules/mz/HISTORY.md +66 -0
- package/node_modules/proper-lockfile/CHANGELOG.md +108 -0
- package/node_modules/source-map/CHANGELOG.md +301 -0
- package/node_modules/thenify/History.md +11 -0
- package/node_modules/thenify-all/History.md +11 -0
- package/node_modules/y18n/CHANGELOG.md +100 -0
- package/node_modules/yargs/CHANGELOG.md +88 -0
- package/node_modules/yargs-parser/CHANGELOG.md +263 -0
- package/package.json +6 -2
- package/workflows/lead-qualifier/prompts/digest.md +44 -0
- package/workflows/lead-qualifier/prompts/draft-outreach.md +44 -0
- package/workflows/lead-qualifier/prompts/enrich.md +52 -0
- package/workflows/lead-qualifier/prompts/intake.md +48 -0
- package/workflows/lead-qualifier/prompts/mark-cold.md +38 -0
- package/workflows/lead-qualifier/prompts/score.md +45 -0
- package/workflows/lead-qualifier/workflow.yaml +95 -0
- package/workflows/research-brief/prompts/brief-synthesize.md +43 -0
- package/workflows/research-brief/prompts/intake.md +51 -0
- package/workflows/research-brief/prompts/source-critique.md +38 -0
- package/workflows/research-brief/prompts/source-score.md +38 -0
- package/workflows/research-brief/prompts/source-summarize.md +54 -0
- package/workflows/research-brief/workflow.yaml +66 -0
- package/dist/extensions/forgecli/session-monitor-widget.d.ts +0 -37
- package/dist/extensions/forgecli/session-monitor-widget.js +0 -320
- package/dist/extensions/forgecli/session-monitor-widget.js.map +0 -1
- package/dist/extensions/forgecli/session-monitor.d.ts +0 -2
- package/dist/extensions/forgecli/session-monitor.js +0 -135
- package/dist/extensions/forgecli/session-monitor.js.map +0 -1
|
@@ -0,0 +1,202 @@
|
|
|
1
|
+
// forge:review-code — native kickoff handler (FORGE-S21-T10).
|
|
2
|
+
//
|
|
3
|
+
// Replaces the auto-generated stub previously installed by
|
|
4
|
+
// registerAllForgeCommands (forge-commands.ts). Kickoff Shim archetype
|
|
5
|
+
// (Pack-04 + Pack-06): single LLM handoff in current context, no fork.
|
|
6
|
+
//
|
|
7
|
+
// Note: The materialized workflow (review_code.md) declares
|
|
8
|
+
// `audience: subagent` — advisory only. Users may invoke this command
|
|
9
|
+
// manually from the CLI; assertAudience never refuses subagent-audience
|
|
10
|
+
// workflows. Orchestrator chains still dispatch via runForgeSubagent
|
|
11
|
+
// directly and do NOT route through this handler.
|
|
12
|
+
//
|
|
13
|
+
// Iron Laws:
|
|
14
|
+
// IL1 — code only under forge-cli/src/extensions/forgecli/.
|
|
15
|
+
// IL4 — no JSON.stringify-into-subagent dispatch.
|
|
16
|
+
// IL6 — no shell-string interpolation; no spawn calls here.
|
|
17
|
+
// IL7 — every failure path emits ctx.ui.notify and returns; no silent
|
|
18
|
+
// continuation.
|
|
19
|
+
import * as fs from "node:fs";
|
|
20
|
+
import * as path from "node:path";
|
|
21
|
+
import { assertAudience } from "./audience-gate.js";
|
|
22
|
+
import { sendKickoff } from "./kickoff.js";
|
|
23
|
+
import { loadPersona, PersonaSkillLoaderError } from "./loaders/persona-skill-loader.js";
|
|
24
|
+
import { loadWorkflow, WorkflowLoaderError } from "./loaders/workflow-loader.js";
|
|
25
|
+
export function parseReviewCodeArgs(rawArgs, cwd) {
|
|
26
|
+
const trimmed = (rawArgs ?? "").trim();
|
|
27
|
+
if (trimmed === "") {
|
|
28
|
+
return { mode: "empty", taskRef: "", sourceLabel: "(no input — supervisor infers task from store/context)" };
|
|
29
|
+
}
|
|
30
|
+
if (trimmed.startsWith("@")) {
|
|
31
|
+
const ref = trimmed.slice(1).trim();
|
|
32
|
+
const filePath = path.isAbsolute(ref) ? ref : path.resolve(cwd, ref);
|
|
33
|
+
const seed = fs.readFileSync(filePath, "utf8");
|
|
34
|
+
return { mode: "file", taskRef: seed, sourceLabel: `(seed from file: ${ref})` };
|
|
35
|
+
}
|
|
36
|
+
return { mode: "text", taskRef: trimmed, sourceLabel: "(seed from inline text)" };
|
|
37
|
+
}
|
|
38
|
+
// Frontmatter persona extraction (permissive) ------------------------------
|
|
39
|
+
export function extractPersonaNames(workflowMd) {
|
|
40
|
+
const lines = workflowMd.split(/\r?\n/);
|
|
41
|
+
if (lines.length === 0 || lines[0] !== "---")
|
|
42
|
+
return [];
|
|
43
|
+
let inside = false;
|
|
44
|
+
let depsBlock = false;
|
|
45
|
+
for (let i = 0; i < lines.length; i++) {
|
|
46
|
+
const line = lines[i];
|
|
47
|
+
if (i === 0 && line === "---") {
|
|
48
|
+
inside = true;
|
|
49
|
+
continue;
|
|
50
|
+
}
|
|
51
|
+
if (!inside)
|
|
52
|
+
break;
|
|
53
|
+
if (line === "---")
|
|
54
|
+
break;
|
|
55
|
+
if (/^deps\s*:\s*$/.test(line)) {
|
|
56
|
+
depsBlock = true;
|
|
57
|
+
continue;
|
|
58
|
+
}
|
|
59
|
+
const m = /^\s*personas\s*:\s*\[([^\]]*)\]\s*$/.exec(line);
|
|
60
|
+
if (m && (depsBlock || /^\s/.test(line))) {
|
|
61
|
+
return m[1]
|
|
62
|
+
.split(",")
|
|
63
|
+
.map((s) => s.trim().replace(/^["']|["']$/g, ""))
|
|
64
|
+
.filter((s) => s.length > 0);
|
|
65
|
+
}
|
|
66
|
+
if (depsBlock && /^[A-Za-z0-9_-]+\s*:/.test(line) && !/^\s/.test(line)) {
|
|
67
|
+
depsBlock = false;
|
|
68
|
+
}
|
|
69
|
+
}
|
|
70
|
+
return [];
|
|
71
|
+
}
|
|
72
|
+
export function checkMaterialization(workflowPath, workflowMd) {
|
|
73
|
+
const missing = [];
|
|
74
|
+
if (!workflowMd.includes("Store-Write Verification")) {
|
|
75
|
+
missing.push("Store-Write Verification");
|
|
76
|
+
}
|
|
77
|
+
if (!workflowMd.includes("Iron Laws")) {
|
|
78
|
+
missing.push("Iron Laws");
|
|
79
|
+
}
|
|
80
|
+
if (!workflowMd.includes("forge_store")) {
|
|
81
|
+
missing.push("forge_store");
|
|
82
|
+
}
|
|
83
|
+
const personas = extractPersonaNames(workflowMd);
|
|
84
|
+
if (personas.length === 0) {
|
|
85
|
+
missing.push("deps.personas: declaration");
|
|
86
|
+
}
|
|
87
|
+
else {
|
|
88
|
+
const bodyStart = (() => {
|
|
89
|
+
if (!workflowMd.startsWith("---\n") && !workflowMd.startsWith("---\r\n"))
|
|
90
|
+
return 0;
|
|
91
|
+
const re = /\r?\n---\r?\n/;
|
|
92
|
+
const m = re.exec(workflowMd);
|
|
93
|
+
return m ? m.index + m[0].length : 0;
|
|
94
|
+
})();
|
|
95
|
+
const body = workflowMd.slice(bodyStart);
|
|
96
|
+
const anyHit = personas.some((name) => {
|
|
97
|
+
if (!name)
|
|
98
|
+
return false;
|
|
99
|
+
const tokenRegex = new RegExp(`\\b${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`);
|
|
100
|
+
return body.includes(`${name}.md`) || tokenRegex.test(body);
|
|
101
|
+
});
|
|
102
|
+
if (!anyHit) {
|
|
103
|
+
missing.push(`persona file path (${personas.join(", ")})`);
|
|
104
|
+
}
|
|
105
|
+
}
|
|
106
|
+
void workflowPath;
|
|
107
|
+
return { ok: missing.length === 0, missing };
|
|
108
|
+
}
|
|
109
|
+
export function composeKickoff(opts) {
|
|
110
|
+
const { workflowMd, personaIdentity, parsed } = opts;
|
|
111
|
+
const sections = ["# /forge:review-code", ""];
|
|
112
|
+
if (personaIdentity.trim().length > 0) {
|
|
113
|
+
sections.push(personaIdentity.trim(), "");
|
|
114
|
+
}
|
|
115
|
+
sections.push("## Dispatch", "", "Run the workflow below. Specifically:", "", "1. Read the implementation diff and PROGRESS.md for `engineering/sprints/<SPRINT_ID>/<TASK_ID>/` (the source of truth).", "2. Query the store for the task and its sprint/feature context via `forge_store_query` — do NOT raw-read `.forge/store/`.", "3. Follow the workflow Algorithm verbatim: review code for correctness, test coverage, security, and architecture compliance.", "4. Write `CODE_REVIEW.md` and `CODE-REVIEW-SUMMARY.json` to the task directory using the `write` tool.", "5. Update task status by calling the `forge_store` MCP tool: `{command:'update-status', args:['task','<TASK_ID>','status','<new-status>']}`. Never raw-write `.forge/store/`. Do NOT bash-shell `forge store ...`.", "6. Honour Pack-06 Read/Write/Ask/Store discipline: writes go via the `forge_store` MCP tool; in-conversation clarifications use `forge_ask_user`.");
|
|
116
|
+
sections.push("", "---", "", "## Workflow", "", workflowMd.trim(), "", "---");
|
|
117
|
+
if (parsed.mode === "empty") {
|
|
118
|
+
sections.push("", "## Input", "", "(no seed — infer task from sprint context and store)");
|
|
119
|
+
}
|
|
120
|
+
else {
|
|
121
|
+
sections.push("", `## Input — ${parsed.sourceLabel}`, "", parsed.taskRef.trim());
|
|
122
|
+
}
|
|
123
|
+
return sections.join("\n");
|
|
124
|
+
}
|
|
125
|
+
// Registration -------------------------------------------------------------
|
|
126
|
+
const WORKFLOW_REL_PATH = path.join(".forge", "workflows", "review_code.md");
|
|
127
|
+
export function registerReviewCode(pi, options = {}) {
|
|
128
|
+
pi.registerCommand("forge:review-code", {
|
|
129
|
+
description: "Run the review-code workflow for a Forge task. " +
|
|
130
|
+
"Usage: /forge:review-code [@<file> | <free-form text>]. " +
|
|
131
|
+
"Note: this workflow is subagent-only; standalone invocations are refused. " +
|
|
132
|
+
"Orchestrator chains dispatch directly via runForgeSubagent.",
|
|
133
|
+
async handler(args, ctx) {
|
|
134
|
+
const cwd = options.cwd ?? process.cwd();
|
|
135
|
+
const workflowPath = path.join(cwd, WORKFLOW_REL_PATH);
|
|
136
|
+
let workflowMd;
|
|
137
|
+
let workflowAudience;
|
|
138
|
+
try {
|
|
139
|
+
const loaded = loadWorkflow(workflowPath);
|
|
140
|
+
workflowMd = loaded.rawMarkdown;
|
|
141
|
+
workflowAudience = loaded.audience;
|
|
142
|
+
}
|
|
143
|
+
catch (err) {
|
|
144
|
+
if (err instanceof WorkflowLoaderError) {
|
|
145
|
+
if (err.code === "missing_file") {
|
|
146
|
+
ctx.ui.notify(`× forge:review-code — workflow not found at ${WORKFLOW_REL_PATH}; run /forge:init or /forge:regenerate first.`, "error");
|
|
147
|
+
}
|
|
148
|
+
else {
|
|
149
|
+
ctx.ui.notify(`× forge:review-code — workflow load failed (${err.code}): ${err.message}`, "error");
|
|
150
|
+
}
|
|
151
|
+
return;
|
|
152
|
+
}
|
|
153
|
+
const e = err;
|
|
154
|
+
ctx.ui.notify(`× forge:review-code — failed to read workflow: ${e.message ?? "unknown"}`, "error");
|
|
155
|
+
return;
|
|
156
|
+
}
|
|
157
|
+
let parsed;
|
|
158
|
+
try {
|
|
159
|
+
parsed = parseReviewCodeArgs(args, cwd);
|
|
160
|
+
}
|
|
161
|
+
catch (err) {
|
|
162
|
+
const e = err;
|
|
163
|
+
ctx.ui.notify(`× forge:review-code — failed to read seed: ${e.message ?? "unknown"}`, "error");
|
|
164
|
+
return;
|
|
165
|
+
}
|
|
166
|
+
const check = checkMaterialization(workflowPath, workflowMd);
|
|
167
|
+
if (!check.ok) {
|
|
168
|
+
for (const marker of check.missing) {
|
|
169
|
+
ctx.ui.notify(`× workflow regression: ${marker} not found in ${workflowPath}`, "error");
|
|
170
|
+
}
|
|
171
|
+
return;
|
|
172
|
+
}
|
|
173
|
+
const personas = extractPersonaNames(workflowMd);
|
|
174
|
+
let personaIdentity = "";
|
|
175
|
+
if (personas.length > 0) {
|
|
176
|
+
try {
|
|
177
|
+
const persona = loadPersona(personas[0], { cwd });
|
|
178
|
+
personaIdentity = persona.identity;
|
|
179
|
+
}
|
|
180
|
+
catch (err) {
|
|
181
|
+
if (err instanceof PersonaSkillLoaderError) {
|
|
182
|
+
ctx.ui.notify(`× forge:review-code — persona '${personas[0]}' load failed (${err.code}): ${err.message}`, "error");
|
|
183
|
+
return;
|
|
184
|
+
}
|
|
185
|
+
const e = err;
|
|
186
|
+
ctx.ui.notify(`× forge:review-code — persona load error: ${e.message ?? "unknown"}`, "error");
|
|
187
|
+
return;
|
|
188
|
+
}
|
|
189
|
+
}
|
|
190
|
+
if (!assertAudience({ workflowName: "review_code", audience: workflowAudience }, ctx)) {
|
|
191
|
+
return;
|
|
192
|
+
}
|
|
193
|
+
const kickoff = composeKickoff({
|
|
194
|
+
workflowMd,
|
|
195
|
+
personaIdentity,
|
|
196
|
+
parsed,
|
|
197
|
+
});
|
|
198
|
+
sendKickoff(pi, kickoff);
|
|
199
|
+
},
|
|
200
|
+
});
|
|
201
|
+
}
|
|
202
|
+
//# sourceMappingURL=review-code.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-code.js","sourceRoot":"","sources":["../../../src/extensions/forgecli/review-code.ts"],"names":[],"mappings":"AAAA,8DAA8D;AAC9D,EAAE;AACF,2DAA2D;AAC3D,uEAAuE;AACvE,uEAAuE;AACvE,EAAE;AACF,4DAA4D;AAC5D,sEAAsE;AACtE,wEAAwE;AACxE,qEAAqE;AACrE,kDAAkD;AAClD,EAAE;AACF,aAAa;AACb,8DAA8D;AAC9D,oDAAoD;AACpD,8DAA8D;AAC9D,wEAAwE;AACxE,wBAAwB;AAExB,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAGlC,OAAO,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AACpD,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAC3C,OAAO,EAAE,WAAW,EAAE,uBAAuB,EAAE,MAAM,mCAAmC,CAAC;AACzF,OAAO,EAAE,YAAY,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAC;AAYjF,MAAM,UAAU,mBAAmB,CAAC,OAAe,EAAE,GAAW;IAC/D,MAAM,OAAO,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACvC,IAAI,OAAO,KAAK,EAAE,EAAE,CAAC;QACpB,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,wDAAwD,EAAE,CAAC;IAC9G,CAAC;IACD,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QAC7B,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACpC,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;QACrE,MAAM,IAAI,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/C,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,oBAAoB,GAAG,GAAG,EAAE,CAAC;IACjF,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,yBAAyB,EAAE,CAAC;AACnF,CAAC;AAED,6EAA6E;AAE7E,MAAM,UAAU,mBAAmB,CAAC,UAAkB;IACrD,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACxC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC,KAAK,KAAK;QAAE,OAAO,EAAE,CAAC;IACxD,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACtB,IAAI,CAAC,KAAK,CAAC,IAAI,IAAI,KAAK,KAAK,EAAE,CAAC;YAC/B,MAAM,GAAG,IAAI,CAAC;YACd,SAAS;QACV,CAAC;QACD,IAAI,CAAC,MAAM;YAAE,MAAM;QACnB,IAAI,IAAI,KAAK,KAAK;YAAE,MAAM;QAC1B,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YAChC,SAAS,GAAG,IAAI,CAAC;YACjB,SAAS;QACV,CAAC;QACD,MAAM,CAAC,GAAG,qCAAqC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3D,IAAI,CAAC,IAAI,CAAC,SAAS,IAAI,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;YAC1C,OAAO,CAAC,CAAC,CAAC,CAAC;iBACT,KAAK,CAAC,GAAG,CAAC;iBACV,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC;iBAChD,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAC/B,CAAC;QACD,IAAI,SAAS,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACxE,SAAS,GAAG,KAAK,CAAC;QACnB,CAAC;IACF,CAAC;IACD,OAAO,EAAE,CAAC;AACX,CAAC;AASD,MAAM,UAAU,oBAAoB,CAAC,YAAoB,EAAE,UAAkB;IAC5E,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,0BAA0B,CAAC,EAAE,CAAC;QACtD,OAAO,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;IAC1C,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QACvC,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC3B,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,CAAC;QACzC,OAAO,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC7B,CAAC;IAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,UAAU,CAAC,CAAC;IACjD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC3B,OAAO,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;IAC5C,CAAC;SAAM,CAAC;QACP,MAAM,SAAS,GAAG,CAAC,GAAG,EAAE;YACvB,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,SAAS,CAAC;gBAAE,OAAO,CAAC,CAAC;YACnF,MAAM,EAAE,GAAG,eAAe,CAAC;YAC3B,MAAM,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC9B,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;QACtC,CAAC,CAAC,EAAE,CAAC;QACL,MAAM,IAAI,GAAG,UAAU,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QAEzC,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;YACrC,IAAI,CAAC,IAAI;gBAAE,OAAO,KAAK,CAAC;YACxB,MAAM,UAAU,GAAG,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;YACtF,OAAO,IAAI,CAAC,QAAQ,CAAC,GAAG,IAAI,KAAK,CAAC,IAAI,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,EAAE,CAAC;YACb,OAAO,CAAC,IAAI,CAAC,sBAAsB,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC5D,CAAC;IACF,CAAC;IAED,KAAK,YAAY,CAAC;IAClB,OAAO,EAAE,EAAE,EAAE,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,OAAO,EAAE,CAAC;AAC9C,CAAC;AAUD,MAAM,UAAU,cAAc,CAAC,IAAwB;IACtD,MAAM,EAAE,UAAU,EAAE,eAAe,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC;IAErD,MAAM,QAAQ,GAAa,CAAC,sBAAsB,EAAE,EAAE,CAAC,CAAC;IACxD,IAAI,eAAe,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvC,QAAQ,CAAC,IAAI,CAAC,eAAe,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;IAC3C,CAAC;IAED,QAAQ,CAAC,IAAI,CACZ,aAAa,EACb,EAAE,EACF,uCAAuC,EACvC,EAAE,EACF,yHAAyH,EACzH,2HAA2H,EAC3H,+HAA+H,EAC/H,wGAAwG,EACxG,oNAAoN,EACpN,mJAAmJ,CACnJ,CAAC;IAEF,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,EAAE,UAAU,CAAC,IAAI,EAAE,EAAE,EAAE,EAAE,KAAK,CAAC,CAAC;IAE9E,IAAI,MAAM,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;QAC7B,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,UAAU,EAAE,EAAE,EAAE,sDAAsD,CAAC,CAAC;IAC3F,CAAC;SAAM,CAAC;QACP,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,MAAM,CAAC,WAAW,EAAE,EAAE,EAAE,EAAE,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IAClF,CAAC;IAED,OAAO,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5B,CAAC;AAED,6EAA6E;AAE7E,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,WAAW,EAAE,gBAAgB,CAAC,CAAC;AAM7E,MAAM,UAAU,kBAAkB,CAAC,EAAgB,EAAE,UAAqC,EAAE;IAC3F,EAAE,CAAC,eAAe,CAAC,mBAAmB,EAAE;QACvC,WAAW,EACV,iDAAiD;YACjD,0DAA0D;YAC1D,4EAA4E;YAC5E,6DAA6D;QAC9D,KAAK,CAAC,OAAO,CAAC,IAAY,EAAE,GAA4B;YACvD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;YACzC,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,iBAAiB,CAAC,CAAC;YAEvD,IAAI,UAAkB,CAAC;YACvB,IAAI,gBAAsE,CAAC;YAC3E,IAAI,CAAC;gBACJ,MAAM,MAAM,GAAG,YAAY,CAAC,YAAY,CAAC,CAAC;gBAC1C,UAAU,GAAG,MAAM,CAAC,WAAW,CAAC;gBAChC,gBAAgB,GAAG,MAAM,CAAC,QAAQ,CAAC;YACpC,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACvB,IAAI,GAAG,YAAY,mBAAmB,EAAE,CAAC;oBACxC,IAAI,GAAG,CAAC,IAAI,KAAK,cAAc,EAAE,CAAC;wBACjC,GAAG,CAAC,EAAE,CAAC,MAAM,CACZ,+CAA+C,iBAAiB,+CAA+C,EAC/G,OAAO,CACP,CAAC;oBACH,CAAC;yBAAM,CAAC;wBACP,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,+CAA+C,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,OAAO,EAAE,EAAE,OAAO,CAAC,CAAC;oBACpG,CAAC;oBACD,OAAO;gBACR,CAAC;gBACD,MAAM,CAAC,GAAG,GAA2B,CAAC;gBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,kDAAkD,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;gBACnG,OAAO;YACR,CAAC;YAED,IAAI,MAAkB,CAAC;YACvB,IAAI,CAAC;gBACJ,MAAM,GAAG,mBAAmB,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;YACzC,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACvB,MAAM,CAAC,GAAG,GAA2B,CAAC;gBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,8CAA8C,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;gBAC/F,OAAO;YACR,CAAC;YAED,MAAM,KAAK,GAAG,oBAAoB,CAAC,YAAY,EAAE,UAAU,CAAC,CAAC;YAC7D,IAAI,CAAC,KAAK,CAAC,EAAE,EAAE,CAAC;gBACf,KAAK,MAAM,MAAM,IAAI,KAAK,CAAC,OAAO,EAAE,CAAC;oBACpC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,0BAA0B,MAAM,iBAAiB,YAAY,EAAE,EAAE,OAAO,CAAC,CAAC;gBACzF,CAAC;gBACD,OAAO;YACR,CAAC;YAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,UAAU,CAAC,CAAC;YACjD,IAAI,eAAe,GAAG,EAAE,CAAC;YACzB,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBACzB,IAAI,CAAC;oBACJ,MAAM,OAAO,GAAG,WAAW,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC;oBAClD,eAAe,GAAG,OAAO,CAAC,QAAQ,CAAC;gBACpC,CAAC;gBAAC,OAAO,GAAY,EAAE,CAAC;oBACvB,IAAI,GAAG,YAAY,uBAAuB,EAAE,CAAC;wBAC5C,GAAG,CAAC,EAAE,CAAC,MAAM,CACZ,kCAAkC,QAAQ,CAAC,CAAC,CAAC,kBAAkB,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,OAAO,EAAE,EAC1F,OAAO,CACP,CAAC;wBACF,OAAO;oBACR,CAAC;oBACD,MAAM,CAAC,GAAG,GAA2B,CAAC;oBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,6CAA6C,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;oBAC9F,OAAO;gBACR,CAAC;YACF,CAAC;YAED,IAAI,CAAC,cAAc,CAAC,EAAE,YAAY,EAAE,aAAa,EAAE,QAAQ,EAAE,gBAAgB,EAAE,EAAE,GAAG,CAAC,EAAE,CAAC;gBACvF,OAAO;YACR,CAAC;YAED,MAAM,OAAO,GAAG,cAAc,CAAC;gBAC9B,UAAU;gBACV,eAAe;gBACf,MAAM;aACN,CAAC,CAAC;YAEH,WAAW,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;QAC1B,CAAC;KACD,CAAC,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
import type { ExtensionAPI } from "@entelligentsia/pi-coding-agent";
|
|
2
|
+
export type ArgMode = "empty" | "file" | "text";
|
|
3
|
+
export interface ParsedArgs {
|
|
4
|
+
mode: ArgMode;
|
|
5
|
+
taskRef: string;
|
|
6
|
+
sourceLabel: string;
|
|
7
|
+
}
|
|
8
|
+
export declare function parseReviewPlanArgs(rawArgs: string, cwd: string): ParsedArgs;
|
|
9
|
+
export declare function extractPersonaNames(workflowMd: string): string[];
|
|
10
|
+
export interface MaterializationCheck {
|
|
11
|
+
ok: boolean;
|
|
12
|
+
missing: string[];
|
|
13
|
+
}
|
|
14
|
+
export declare function checkMaterialization(workflowPath: string, workflowMd: string): MaterializationCheck;
|
|
15
|
+
export interface ComposeKickoffOpts {
|
|
16
|
+
workflowMd: string;
|
|
17
|
+
personaIdentity: string;
|
|
18
|
+
parsed: ParsedArgs;
|
|
19
|
+
}
|
|
20
|
+
export declare function composeKickoff(opts: ComposeKickoffOpts): string;
|
|
21
|
+
export interface RegisterReviewPlanOptions {
|
|
22
|
+
cwd?: string;
|
|
23
|
+
}
|
|
24
|
+
export declare function registerReviewPlan(pi: ExtensionAPI, options?: RegisterReviewPlanOptions): void;
|
|
@@ -0,0 +1,203 @@
|
|
|
1
|
+
// forge:review-plan — native kickoff handler (FORGE-S21-T10).
|
|
2
|
+
//
|
|
3
|
+
// Replaces the auto-generated stub previously installed by
|
|
4
|
+
// registerAllForgeCommands (forge-commands.ts). Kickoff Shim archetype
|
|
5
|
+
// (Pack-04 + Pack-06): single LLM handoff in current context, no fork.
|
|
6
|
+
//
|
|
7
|
+
// Note: The materialized workflow (review_plan.md) declares
|
|
8
|
+
// `audience: subagent` — advisory only. Users may invoke this command
|
|
9
|
+
// manually from the CLI; assertAudience never refuses subagent-audience
|
|
10
|
+
// workflows. Orchestrator chains (run-task, run-sprint, fix-bug) still
|
|
11
|
+
// dispatch via runForgeSubagent directly and do NOT route through this
|
|
12
|
+
// handler.
|
|
13
|
+
//
|
|
14
|
+
// Iron Laws:
|
|
15
|
+
// IL1 — code only under forge-cli/src/extensions/forgecli/.
|
|
16
|
+
// IL4 — no JSON.stringify-into-subagent dispatch.
|
|
17
|
+
// IL6 — no shell-string interpolation; no spawn calls here.
|
|
18
|
+
// IL7 — every failure path emits ctx.ui.notify and returns; no silent
|
|
19
|
+
// continuation.
|
|
20
|
+
import * as fs from "node:fs";
|
|
21
|
+
import * as path from "node:path";
|
|
22
|
+
import { assertAudience } from "./audience-gate.js";
|
|
23
|
+
import { sendKickoff } from "./kickoff.js";
|
|
24
|
+
import { loadPersona, PersonaSkillLoaderError } from "./loaders/persona-skill-loader.js";
|
|
25
|
+
import { loadWorkflow, WorkflowLoaderError } from "./loaders/workflow-loader.js";
|
|
26
|
+
export function parseReviewPlanArgs(rawArgs, cwd) {
|
|
27
|
+
const trimmed = (rawArgs ?? "").trim();
|
|
28
|
+
if (trimmed === "") {
|
|
29
|
+
return { mode: "empty", taskRef: "", sourceLabel: "(no input — supervisor infers task from store/context)" };
|
|
30
|
+
}
|
|
31
|
+
if (trimmed.startsWith("@")) {
|
|
32
|
+
const ref = trimmed.slice(1).trim();
|
|
33
|
+
const filePath = path.isAbsolute(ref) ? ref : path.resolve(cwd, ref);
|
|
34
|
+
const seed = fs.readFileSync(filePath, "utf8");
|
|
35
|
+
return { mode: "file", taskRef: seed, sourceLabel: `(seed from file: ${ref})` };
|
|
36
|
+
}
|
|
37
|
+
return { mode: "text", taskRef: trimmed, sourceLabel: "(seed from inline text)" };
|
|
38
|
+
}
|
|
39
|
+
// Frontmatter persona extraction (permissive) ------------------------------
|
|
40
|
+
export function extractPersonaNames(workflowMd) {
|
|
41
|
+
const lines = workflowMd.split(/\r?\n/);
|
|
42
|
+
if (lines.length === 0 || lines[0] !== "---")
|
|
43
|
+
return [];
|
|
44
|
+
let inside = false;
|
|
45
|
+
let depsBlock = false;
|
|
46
|
+
for (let i = 0; i < lines.length; i++) {
|
|
47
|
+
const line = lines[i];
|
|
48
|
+
if (i === 0 && line === "---") {
|
|
49
|
+
inside = true;
|
|
50
|
+
continue;
|
|
51
|
+
}
|
|
52
|
+
if (!inside)
|
|
53
|
+
break;
|
|
54
|
+
if (line === "---")
|
|
55
|
+
break;
|
|
56
|
+
if (/^deps\s*:\s*$/.test(line)) {
|
|
57
|
+
depsBlock = true;
|
|
58
|
+
continue;
|
|
59
|
+
}
|
|
60
|
+
const m = /^\s*personas\s*:\s*\[([^\]]*)\]\s*$/.exec(line);
|
|
61
|
+
if (m && (depsBlock || /^\s/.test(line))) {
|
|
62
|
+
return m[1]
|
|
63
|
+
.split(",")
|
|
64
|
+
.map((s) => s.trim().replace(/^["']|["']$/g, ""))
|
|
65
|
+
.filter((s) => s.length > 0);
|
|
66
|
+
}
|
|
67
|
+
if (depsBlock && /^[A-Za-z0-9_-]+\s*:/.test(line) && !/^\s/.test(line)) {
|
|
68
|
+
depsBlock = false;
|
|
69
|
+
}
|
|
70
|
+
}
|
|
71
|
+
return [];
|
|
72
|
+
}
|
|
73
|
+
export function checkMaterialization(workflowPath, workflowMd) {
|
|
74
|
+
const missing = [];
|
|
75
|
+
if (!workflowMd.includes("Store-Write Verification")) {
|
|
76
|
+
missing.push("Store-Write Verification");
|
|
77
|
+
}
|
|
78
|
+
if (!workflowMd.includes("Iron Laws")) {
|
|
79
|
+
missing.push("Iron Laws");
|
|
80
|
+
}
|
|
81
|
+
if (!workflowMd.includes("forge_store")) {
|
|
82
|
+
missing.push("forge_store");
|
|
83
|
+
}
|
|
84
|
+
const personas = extractPersonaNames(workflowMd);
|
|
85
|
+
if (personas.length === 0) {
|
|
86
|
+
missing.push("deps.personas: declaration");
|
|
87
|
+
}
|
|
88
|
+
else {
|
|
89
|
+
const bodyStart = (() => {
|
|
90
|
+
if (!workflowMd.startsWith("---\n") && !workflowMd.startsWith("---\r\n"))
|
|
91
|
+
return 0;
|
|
92
|
+
const re = /\r?\n---\r?\n/;
|
|
93
|
+
const m = re.exec(workflowMd);
|
|
94
|
+
return m ? m.index + m[0].length : 0;
|
|
95
|
+
})();
|
|
96
|
+
const body = workflowMd.slice(bodyStart);
|
|
97
|
+
const anyHit = personas.some((name) => {
|
|
98
|
+
if (!name)
|
|
99
|
+
return false;
|
|
100
|
+
const tokenRegex = new RegExp(`\\b${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`);
|
|
101
|
+
return body.includes(`${name}.md`) || tokenRegex.test(body);
|
|
102
|
+
});
|
|
103
|
+
if (!anyHit) {
|
|
104
|
+
missing.push(`persona file path (${personas.join(", ")})`);
|
|
105
|
+
}
|
|
106
|
+
}
|
|
107
|
+
void workflowPath;
|
|
108
|
+
return { ok: missing.length === 0, missing };
|
|
109
|
+
}
|
|
110
|
+
export function composeKickoff(opts) {
|
|
111
|
+
const { workflowMd, personaIdentity, parsed } = opts;
|
|
112
|
+
const sections = ["# /forge:review-plan", ""];
|
|
113
|
+
if (personaIdentity.trim().length > 0) {
|
|
114
|
+
sections.push(personaIdentity.trim(), "");
|
|
115
|
+
}
|
|
116
|
+
sections.push("## Dispatch", "", "Run the workflow below. Specifically:", "", "1. Read the plan at `engineering/sprints/<SPRINT_ID>/<TASK_ID>/PLAN.md` (the source of truth).", "2. Query the store for the task and its sprint/feature context via `forge_store_query` — do NOT raw-read `.forge/store/`.", "3. Follow the workflow Algorithm verbatim: review the plan for correctness, completeness, and risk.", "4. Write `PLAN_REVIEW.md` and `PLAN-REVIEW-SUMMARY.json` to the task directory using the `write` tool.", "5. Update task status by calling the `forge_store` MCP tool: `{command:'update-status', args:['task','<TASK_ID>','status','<new-status>']}`. Never raw-write `.forge/store/`. Do NOT bash-shell `forge store ...`.", "6. Honour Pack-06 Read/Write/Ask/Store discipline: writes go via the `forge_store` MCP tool; in-conversation clarifications use `forge_ask_user`.");
|
|
117
|
+
sections.push("", "---", "", "## Workflow", "", workflowMd.trim(), "", "---");
|
|
118
|
+
if (parsed.mode === "empty") {
|
|
119
|
+
sections.push("", "## Input", "", "(no seed — infer task from sprint context and store)");
|
|
120
|
+
}
|
|
121
|
+
else {
|
|
122
|
+
sections.push("", `## Input — ${parsed.sourceLabel}`, "", parsed.taskRef.trim());
|
|
123
|
+
}
|
|
124
|
+
return sections.join("\n");
|
|
125
|
+
}
|
|
126
|
+
// Registration -------------------------------------------------------------
|
|
127
|
+
const WORKFLOW_REL_PATH = path.join(".forge", "workflows", "review_plan.md");
|
|
128
|
+
export function registerReviewPlan(pi, options = {}) {
|
|
129
|
+
pi.registerCommand("forge:review-plan", {
|
|
130
|
+
description: "Run the review-plan workflow for a Forge task. " +
|
|
131
|
+
"Usage: /forge:review-plan [@<file> | <free-form text>]. " +
|
|
132
|
+
"Note: this workflow is subagent-only; standalone invocations are refused. " +
|
|
133
|
+
"Orchestrator chains dispatch directly via runForgeSubagent.",
|
|
134
|
+
async handler(args, ctx) {
|
|
135
|
+
const cwd = options.cwd ?? process.cwd();
|
|
136
|
+
const workflowPath = path.join(cwd, WORKFLOW_REL_PATH);
|
|
137
|
+
let workflowMd;
|
|
138
|
+
let workflowAudience;
|
|
139
|
+
try {
|
|
140
|
+
const loaded = loadWorkflow(workflowPath);
|
|
141
|
+
workflowMd = loaded.rawMarkdown;
|
|
142
|
+
workflowAudience = loaded.audience;
|
|
143
|
+
}
|
|
144
|
+
catch (err) {
|
|
145
|
+
if (err instanceof WorkflowLoaderError) {
|
|
146
|
+
if (err.code === "missing_file") {
|
|
147
|
+
ctx.ui.notify(`× forge:review-plan — workflow not found at ${WORKFLOW_REL_PATH}; run /forge:init or /forge:regenerate first.`, "error");
|
|
148
|
+
}
|
|
149
|
+
else {
|
|
150
|
+
ctx.ui.notify(`× forge:review-plan — workflow load failed (${err.code}): ${err.message}`, "error");
|
|
151
|
+
}
|
|
152
|
+
return;
|
|
153
|
+
}
|
|
154
|
+
const e = err;
|
|
155
|
+
ctx.ui.notify(`× forge:review-plan — failed to read workflow: ${e.message ?? "unknown"}`, "error");
|
|
156
|
+
return;
|
|
157
|
+
}
|
|
158
|
+
let parsed;
|
|
159
|
+
try {
|
|
160
|
+
parsed = parseReviewPlanArgs(args, cwd);
|
|
161
|
+
}
|
|
162
|
+
catch (err) {
|
|
163
|
+
const e = err;
|
|
164
|
+
ctx.ui.notify(`× forge:review-plan — failed to read seed: ${e.message ?? "unknown"}`, "error");
|
|
165
|
+
return;
|
|
166
|
+
}
|
|
167
|
+
const check = checkMaterialization(workflowPath, workflowMd);
|
|
168
|
+
if (!check.ok) {
|
|
169
|
+
for (const marker of check.missing) {
|
|
170
|
+
ctx.ui.notify(`× workflow regression: ${marker} not found in ${workflowPath}`, "error");
|
|
171
|
+
}
|
|
172
|
+
return;
|
|
173
|
+
}
|
|
174
|
+
const personas = extractPersonaNames(workflowMd);
|
|
175
|
+
let personaIdentity = "";
|
|
176
|
+
if (personas.length > 0) {
|
|
177
|
+
try {
|
|
178
|
+
const persona = loadPersona(personas[0], { cwd });
|
|
179
|
+
personaIdentity = persona.identity;
|
|
180
|
+
}
|
|
181
|
+
catch (err) {
|
|
182
|
+
if (err instanceof PersonaSkillLoaderError) {
|
|
183
|
+
ctx.ui.notify(`× forge:review-plan — persona '${personas[0]}' load failed (${err.code}): ${err.message}`, "error");
|
|
184
|
+
return;
|
|
185
|
+
}
|
|
186
|
+
const e = err;
|
|
187
|
+
ctx.ui.notify(`× forge:review-plan — persona load error: ${e.message ?? "unknown"}`, "error");
|
|
188
|
+
return;
|
|
189
|
+
}
|
|
190
|
+
}
|
|
191
|
+
if (!assertAudience({ workflowName: "review_plan", audience: workflowAudience }, ctx)) {
|
|
192
|
+
return;
|
|
193
|
+
}
|
|
194
|
+
const kickoff = composeKickoff({
|
|
195
|
+
workflowMd,
|
|
196
|
+
personaIdentity,
|
|
197
|
+
parsed,
|
|
198
|
+
});
|
|
199
|
+
sendKickoff(pi, kickoff);
|
|
200
|
+
},
|
|
201
|
+
});
|
|
202
|
+
}
|
|
203
|
+
//# sourceMappingURL=review-plan.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-plan.js","sourceRoot":"","sources":["../../../src/extensions/forgecli/review-plan.ts"],"names":[],"mappings":"AAAA,8DAA8D;AAC9D,EAAE;AACF,2DAA2D;AAC3D,uEAAuE;AACvE,uEAAuE;AACvE,EAAE;AACF,4DAA4D;AAC5D,sEAAsE;AACtE,wEAAwE;AACxE,uEAAuE;AACvE,uEAAuE;AACvE,WAAW;AACX,EAAE;AACF,aAAa;AACb,8DAA8D;AAC9D,oDAAoD;AACpD,8DAA8D;AAC9D,wEAAwE;AACxE,wBAAwB;AAExB,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAGlC,OAAO,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AACpD,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAC3C,OAAO,EAAE,WAAW,EAAE,uBAAuB,EAAE,MAAM,mCAAmC,CAAC;AACzF,OAAO,EAAE,YAAY,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAC;AAYjF,MAAM,UAAU,mBAAmB,CAAC,OAAe,EAAE,GAAW;IAC/D,MAAM,OAAO,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACvC,IAAI,OAAO,KAAK,EAAE,EAAE,CAAC;QACpB,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,wDAAwD,EAAE,CAAC;IAC9G,CAAC;IACD,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QAC7B,MAAM,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACpC,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;QACrE,MAAM,IAAI,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/C,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,oBAAoB,GAAG,GAAG,EAAE,CAAC;IACjF,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,yBAAyB,EAAE,CAAC;AACnF,CAAC;AAED,6EAA6E;AAE7E,MAAM,UAAU,mBAAmB,CAAC,UAAkB;IACrD,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACxC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC,KAAK,KAAK;QAAE,OAAO,EAAE,CAAC;IACxD,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACtB,IAAI,CAAC,KAAK,CAAC,IAAI,IAAI,KAAK,KAAK,EAAE,CAAC;YAC/B,MAAM,GAAG,IAAI,CAAC;YACd,SAAS;QACV,CAAC;QACD,IAAI,CAAC,MAAM;YAAE,MAAM;QACnB,IAAI,IAAI,KAAK,KAAK;YAAE,MAAM;QAC1B,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YAChC,SAAS,GAAG,IAAI,CAAC;YACjB,SAAS;QACV,CAAC;QACD,MAAM,CAAC,GAAG,qCAAqC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3D,IAAI,CAAC,IAAI,CAAC,SAAS,IAAI,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;YAC1C,OAAO,CAAC,CAAC,CAAC,CAAC;iBACT,KAAK,CAAC,GAAG,CAAC;iBACV,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC;iBAChD,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAC/B,CAAC;QACD,IAAI,SAAS,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACxE,SAAS,GAAG,KAAK,CAAC;QACnB,CAAC;IACF,CAAC;IACD,OAAO,EAAE,CAAC;AACX,CAAC;AASD,MAAM,UAAU,oBAAoB,CAAC,YAAoB,EAAE,UAAkB;IAC5E,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,0BAA0B,CAAC,EAAE,CAAC;QACtD,OAAO,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;IAC1C,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QACvC,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC3B,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,CAAC;QACzC,OAAO,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC7B,CAAC;IAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,UAAU,CAAC,CAAC;IACjD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC3B,OAAO,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;IAC5C,CAAC;SAAM,CAAC;QACP,MAAM,SAAS,GAAG,CAAC,GAAG,EAAE;YACvB,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,SAAS,CAAC;gBAAE,OAAO,CAAC,CAAC;YACnF,MAAM,EAAE,GAAG,eAAe,CAAC;YAC3B,MAAM,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC9B,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;QACtC,CAAC,CAAC,EAAE,CAAC;QACL,MAAM,IAAI,GAAG,UAAU,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QAEzC,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;YACrC,IAAI,CAAC,IAAI;gBAAE,OAAO,KAAK,CAAC;YACxB,MAAM,UAAU,GAAG,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;YACtF,OAAO,IAAI,CAAC,QAAQ,CAAC,GAAG,IAAI,KAAK,CAAC,IAAI,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,EAAE,CAAC;YACb,OAAO,CAAC,IAAI,CAAC,sBAAsB,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC5D,CAAC;IACF,CAAC;IAED,KAAK,YAAY,CAAC;IAClB,OAAO,EAAE,EAAE,EAAE,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,OAAO,EAAE,CAAC;AAC9C,CAAC;AAUD,MAAM,UAAU,cAAc,CAAC,IAAwB;IACtD,MAAM,EAAE,UAAU,EAAE,eAAe,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC;IAErD,MAAM,QAAQ,GAAa,CAAC,sBAAsB,EAAE,EAAE,CAAC,CAAC;IACxD,IAAI,eAAe,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvC,QAAQ,CAAC,IAAI,CAAC,eAAe,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;IAC3C,CAAC;IAED,QAAQ,CAAC,IAAI,CACZ,aAAa,EACb,EAAE,EACF,uCAAuC,EACvC,EAAE,EACF,gGAAgG,EAChG,2HAA2H,EAC3H,qGAAqG,EACrG,wGAAwG,EACxG,oNAAoN,EACpN,mJAAmJ,CACnJ,CAAC;IAEF,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,EAAE,UAAU,CAAC,IAAI,EAAE,EAAE,EAAE,EAAE,KAAK,CAAC,CAAC;IAE9E,IAAI,MAAM,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;QAC7B,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,UAAU,EAAE,EAAE,EAAE,sDAAsD,CAAC,CAAC;IAC3F,CAAC;SAAM,CAAC;QACP,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,MAAM,CAAC,WAAW,EAAE,EAAE,EAAE,EAAE,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IAClF,CAAC;IAED,OAAO,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5B,CAAC;AAED,6EAA6E;AAE7E,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,WAAW,EAAE,gBAAgB,CAAC,CAAC;AAM7E,MAAM,UAAU,kBAAkB,CAAC,EAAgB,EAAE,UAAqC,EAAE;IAC3F,EAAE,CAAC,eAAe,CAAC,mBAAmB,EAAE;QACvC,WAAW,EACV,iDAAiD;YACjD,0DAA0D;YAC1D,4EAA4E;YAC5E,6DAA6D;QAC9D,KAAK,CAAC,OAAO,CAAC,IAAY,EAAE,GAA4B;YACvD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;YACzC,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,iBAAiB,CAAC,CAAC;YAEvD,IAAI,UAAkB,CAAC;YACvB,IAAI,gBAAsE,CAAC;YAC3E,IAAI,CAAC;gBACJ,MAAM,MAAM,GAAG,YAAY,CAAC,YAAY,CAAC,CAAC;gBAC1C,UAAU,GAAG,MAAM,CAAC,WAAW,CAAC;gBAChC,gBAAgB,GAAG,MAAM,CAAC,QAAQ,CAAC;YACpC,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACvB,IAAI,GAAG,YAAY,mBAAmB,EAAE,CAAC;oBACxC,IAAI,GAAG,CAAC,IAAI,KAAK,cAAc,EAAE,CAAC;wBACjC,GAAG,CAAC,EAAE,CAAC,MAAM,CACZ,+CAA+C,iBAAiB,+CAA+C,EAC/G,OAAO,CACP,CAAC;oBACH,CAAC;yBAAM,CAAC;wBACP,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,+CAA+C,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,OAAO,EAAE,EAAE,OAAO,CAAC,CAAC;oBACpG,CAAC;oBACD,OAAO;gBACR,CAAC;gBACD,MAAM,CAAC,GAAG,GAA2B,CAAC;gBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,kDAAkD,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;gBACnG,OAAO;YACR,CAAC;YAED,IAAI,MAAkB,CAAC;YACvB,IAAI,CAAC;gBACJ,MAAM,GAAG,mBAAmB,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;YACzC,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACvB,MAAM,CAAC,GAAG,GAA2B,CAAC;gBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,8CAA8C,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;gBAC/F,OAAO;YACR,CAAC;YAED,MAAM,KAAK,GAAG,oBAAoB,CAAC,YAAY,EAAE,UAAU,CAAC,CAAC;YAC7D,IAAI,CAAC,KAAK,CAAC,EAAE,EAAE,CAAC;gBACf,KAAK,MAAM,MAAM,IAAI,KAAK,CAAC,OAAO,EAAE,CAAC;oBACpC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,0BAA0B,MAAM,iBAAiB,YAAY,EAAE,EAAE,OAAO,CAAC,CAAC;gBACzF,CAAC;gBACD,OAAO;YACR,CAAC;YAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,UAAU,CAAC,CAAC;YACjD,IAAI,eAAe,GAAG,EAAE,CAAC;YACzB,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBACzB,IAAI,CAAC;oBACJ,MAAM,OAAO,GAAG,WAAW,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC;oBAClD,eAAe,GAAG,OAAO,CAAC,QAAQ,CAAC;gBACpC,CAAC;gBAAC,OAAO,GAAY,EAAE,CAAC;oBACvB,IAAI,GAAG,YAAY,uBAAuB,EAAE,CAAC;wBAC5C,GAAG,CAAC,EAAE,CAAC,MAAM,CACZ,kCAAkC,QAAQ,CAAC,CAAC,CAAC,kBAAkB,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,OAAO,EAAE,EAC1F,OAAO,CACP,CAAC;wBACF,OAAO;oBACR,CAAC;oBACD,MAAM,CAAC,GAAG,GAA2B,CAAC;oBACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,6CAA6C,CAAC,CAAC,OAAO,IAAI,SAAS,EAAE,EAAE,OAAO,CAAC,CAAC;oBAC9F,OAAO;gBACR,CAAC;YACF,CAAC;YAED,IAAI,CAAC,cAAc,CAAC,EAAE,YAAY,EAAE,aAAa,EAAE,QAAQ,EAAE,gBAAgB,EAAE,EAAE,GAAG,CAAC,EAAE,CAAC;gBACvF,OAAO;YACR,CAAC;YAED,MAAM,OAAO,GAAG,cAAc,CAAC;gBAC9B,UAAU;gBACV,eAAe;gBACf,MAAM;aACN,CAAC,CAAC;YAEH,WAAW,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;QAC1B,CAAC;KACD,CAAC,CAAC;AACJ,CAAC"}
|
|
@@ -1,5 +1,23 @@
|
|
|
1
1
|
import type { ExtensionAPI } from "@entelligentsia/pi-coding-agent";
|
|
2
|
+
import type { StreamFn } from "@entelligentsia/pi-agent-core";
|
|
3
|
+
/**
|
|
4
|
+
* Test-only seam (forge-cli#17). Resolves a StreamFn for a given dispatch
|
|
5
|
+
* context — ceremony or per-phase. Production callers leave this undefined.
|
|
6
|
+
*/
|
|
7
|
+
export type SprintStreamFnFactory = (ctx: {
|
|
8
|
+
kind: "task-phase" | "ceremony";
|
|
9
|
+
persona: string;
|
|
10
|
+
phase?: string;
|
|
11
|
+
taskId?: string;
|
|
12
|
+
}) => StreamFn | undefined;
|
|
2
13
|
export interface RegisterRunSprintOptions {
|
|
3
14
|
cwd?: string;
|
|
15
|
+
/**
|
|
16
|
+
* Test-only seam (forge-cli#17). When set, each `runForgeSubagent` call
|
|
17
|
+
* spawned by the sprint handler (ceremony) and by the per-task pipeline
|
|
18
|
+
* (each phase) is dispatched with `streamFn = factory({...})`. Production
|
|
19
|
+
* callers leave this undefined.
|
|
20
|
+
*/
|
|
21
|
+
streamFnFactory?: SprintStreamFnFactory;
|
|
4
22
|
}
|
|
5
23
|
export declare function registerRunSprint(pi: ExtensionAPI, options?: RegisterRunSprintOptions): void;
|
|
@@ -30,6 +30,7 @@ import { loadWorkflow } from "./loaders/workflow-loader.js";
|
|
|
30
30
|
import { discoverForgeConfig } from "./forge-root.js";
|
|
31
31
|
import { getSessionRegistry } from "./session-registry.js";
|
|
32
32
|
import { loadForgePersona, runForgeSubagent } from "./forge-subagent.js";
|
|
33
|
+
import { emitSyntheticEvent } from "./hook-dispatcher.js";
|
|
33
34
|
import { runTaskPipeline, isNonInteractive, formatLocalTime, emitEvent, isoCompact, validateId, readState as readTaskState, isStateStale, } from "./run-task.js";
|
|
34
35
|
function sprintStateFilePath(cwd, sprintId) {
|
|
35
36
|
if (!validateId(sprintId)) {
|
|
@@ -90,7 +91,7 @@ function readSprintRecord(sprintId, storeCli, cwd) {
|
|
|
90
91
|
}
|
|
91
92
|
}
|
|
92
93
|
async function dispatchSprintCeremony(params) {
|
|
93
|
-
const { sprintId, mode, completedTaskIds, pausedAfterIndex, cwd, forgeRoot, ctx, registry } = params;
|
|
94
|
+
const { sprintId, mode, completedTaskIds, pausedAfterIndex, cwd, forgeRoot, ctx, registry, streamFnFactory } = params;
|
|
94
95
|
const startMs = Date.now();
|
|
95
96
|
// Materialized workflow path — already shipped from base pack.
|
|
96
97
|
const workflowName = "architect_review_sprint_completion";
|
|
@@ -131,6 +132,11 @@ async function dispatchSprintCeremony(params) {
|
|
|
131
132
|
cwd,
|
|
132
133
|
exportTag: `${sprintId}__ceremony`,
|
|
133
134
|
forgeRoot,
|
|
135
|
+
streamFn: streamFnFactory?.({ kind: "ceremony", persona: personaName }),
|
|
136
|
+
// Sprint-scoped prompt-cache key — every subagent spawned across
|
|
137
|
+
// the sprint (ceremonies + per-task phases) shares this namespace
|
|
138
|
+
// so the system-prompt + persona prefix stays warm.
|
|
139
|
+
cacheSessionId: `forge:${sprintId}`,
|
|
134
140
|
});
|
|
135
141
|
model = result.model;
|
|
136
142
|
provider = result.provider;
|
|
@@ -373,6 +379,14 @@ export function registerRunSprint(pi, options = {}) {
|
|
|
373
379
|
preflightGate,
|
|
374
380
|
registry,
|
|
375
381
|
resumeFromState,
|
|
382
|
+
streamFnFactory: options.streamFnFactory
|
|
383
|
+
? (c) => options.streamFnFactory?.({
|
|
384
|
+
kind: "task-phase",
|
|
385
|
+
persona: c.persona,
|
|
386
|
+
phase: c.phase,
|
|
387
|
+
taskId: c.taskId,
|
|
388
|
+
})
|
|
389
|
+
: undefined,
|
|
376
390
|
});
|
|
377
391
|
// Capture model/provider from last task result (REVIEW FIX #1)
|
|
378
392
|
if (taskResult.model)
|
|
@@ -442,6 +456,7 @@ export function registerRunSprint(pi, options = {}) {
|
|
|
442
456
|
forgeRoot,
|
|
443
457
|
ctx,
|
|
444
458
|
registry,
|
|
459
|
+
streamFnFactory: options.streamFnFactory,
|
|
445
460
|
});
|
|
446
461
|
}
|
|
447
462
|
const pausedEvent = {
|
|
@@ -489,6 +504,7 @@ export function registerRunSprint(pi, options = {}) {
|
|
|
489
504
|
forgeRoot,
|
|
490
505
|
ctx,
|
|
491
506
|
registry,
|
|
507
|
+
streamFnFactory: options.streamFnFactory,
|
|
492
508
|
});
|
|
493
509
|
const sprintEvent = {
|
|
494
510
|
eventId: `${isoCompact(sprintStartMs)}_${sprintId}_sprint_complete`,
|
|
@@ -511,6 +527,22 @@ export function registerRunSprint(pi, options = {}) {
|
|
|
511
527
|
if (!emitResult.ok) {
|
|
512
528
|
ctx.ui.notify(`⚠ forge:run-sprint — sprint-complete event emit failed: ${emitResult.stderr.trim()}`, "warning");
|
|
513
529
|
}
|
|
530
|
+
// ── Emit synthetic sprint-collate-complete event (FORGE-S21-T05) ──
|
|
531
|
+
// Fires the in-process hook for post-sprint-hook.ts to consume.
|
|
532
|
+
// Best-effort: failure-to-emit notifies but does NOT halt — sprint is
|
|
533
|
+
// already complete at this point.
|
|
534
|
+
try {
|
|
535
|
+
const collateEvent = {
|
|
536
|
+
type: "sprint-collate-complete",
|
|
537
|
+
sprintId,
|
|
538
|
+
cwd,
|
|
539
|
+
};
|
|
540
|
+
await emitSyntheticEvent(collateEvent, ctx);
|
|
541
|
+
}
|
|
542
|
+
catch (err) {
|
|
543
|
+
const e = err;
|
|
544
|
+
ctx.ui.notify(`⚠ forge:run-sprint — sprint-collate-complete synthetic event emit failed: ${e.message ?? "unknown"}`, "warning");
|
|
545
|
+
}
|
|
514
546
|
ctx.ui.setStatus?.(SPRINT_STATUS_KEY, undefined);
|
|
515
547
|
if (ceremony.verdict === "complete") {
|
|
516
548
|
ctx.ui.notify(`〇 forge:run-sprint — sprint ${sprintId} complete (${completedTaskIds.length}/${taskIds.length} tasks).`, "info");
|