@runfusion/fusion 0.75.0-beta.0 → 0.75.0-beta.2
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/dist/bin.js +2858 -1801
- package/dist/child-process-worker.js +2650 -1824
- package/dist/client/.vite/manifest.json +249 -249
- package/dist/client/assets/{AgentDetailView-lS4k3-0w.js → AgentDetailView-C4Xpag_i.js} +1 -1
- package/dist/client/assets/{AgentPermissionPolicyEditor-D7LB_f1m.js → AgentPermissionPolicyEditor-D-Fp7-sy.js} +1 -1
- package/dist/client/assets/{AgentsView-DwCNc9_x.js → AgentsView-BmFsoY1Q.js} +3 -3
- package/dist/client/assets/{ChatView-CVxDxKOu.js → ChatView-5KsB3BVO.js} +1 -1
- package/dist/client/assets/{CommandCenter-Da3AiDs-.js → CommandCenter-CacGI8AV.js} +1 -1
- package/dist/client/assets/{DevServerView-C0UWmEhO.js → DevServerView-DufjauVn.js} +1 -1
- package/dist/client/assets/{DirectoryPicker-JH2ahFj8.js → DirectoryPicker-DvwqfQDS.js} +1 -1
- package/dist/client/assets/{DocumentsView-BA4Pqnfr.js → DocumentsView-BbYJFIQ1.js} +1 -1
- package/dist/client/assets/{EvalsView-DW88MPta.js → EvalsView-NBC8kOc-.js} +1 -1
- package/dist/client/assets/{ExperimentalAgentOnboardingModal-DOih_5nN.js → ExperimentalAgentOnboardingModal-C809yXXY.js} +1 -1
- package/dist/client/assets/{GoalsView-DDIorLyM.js → GoalsView-CpNO9eI0.js} +1 -1
- package/dist/client/assets/{InsightsView-B5VUUsmJ.js → InsightsView-Cahy5FcR.js} +1 -1
- package/dist/client/assets/{MemoryView-sDMj6cEL.js → MemoryView-BNJAQjqn.js} +1 -1
- package/dist/client/assets/{PiExtensionsManager-SpcPEzpg.js → PiExtensionsManager-DCHTyOmJ.js} +1 -1
- package/dist/client/assets/{PluginManager-lIMMAJYt.js → PluginManager-CBt-CqUz.js} +1 -1
- package/dist/client/assets/{PullRequestView-BfAHOr3-.js → PullRequestView-Dsb9RwmX.js} +1 -1
- package/dist/client/assets/{ReportModal-6RDeUqQG.js → ReportModal-8FjZOcbC.js} +1 -1
- package/dist/client/assets/{ResearchView-BUGq2GsN.js → ResearchView-DlSG7dOd.js} +1 -1
- package/dist/client/assets/{SecretsView-CTwukpAq.js → SecretsView-BSajNE-f.js} +1 -1
- package/dist/client/assets/{SessionTerminal-Dspbb2NO.js → SessionTerminal-M2fIpHrL.js} +1 -1
- package/dist/client/assets/{SettingsModal-B6guaVe8.js → SettingsModal-DLcvlp44.js} +3 -3
- package/dist/client/assets/{SettingsModal-D2DahTWc.js → SettingsModal-z6gOf7dG.js} +1 -1
- package/dist/client/assets/{SettingsTextareaRow-BZZnVXGb.js → SettingsTextareaRow-DeC19053.js} +1 -1
- package/dist/client/assets/{SetupWizardModal-xTlxjulj.js → SetupWizardModal-BOjMePFh.js} +2 -2
- package/dist/client/assets/{SkillsView-CMG1fM0H.js → SkillsView-BSBEC-Xm.js} +1 -1
- package/dist/client/assets/{WorkflowNodeEditor-BBrY5D7F.js → WorkflowNodeEditor-DZ9-RBqo.js} +1 -1
- package/dist/client/assets/{agent-import-generation-DV3YE8Rc.js → agent-import-generation-COItiXMT.js} +1 -1
- package/dist/client/assets/{architectureDiagram-3BPJPVTR-gMbSnyDo.js → architectureDiagram-3BPJPVTR-BwlCePoB.js} +1 -1
- package/dist/client/assets/{blockDiagram-GPEHLZMM-BRPa9ADn.js → blockDiagram-GPEHLZMM-KKrLi9v4.js} +1 -1
- package/dist/client/assets/{c4Diagram-AAUBKEIU-BZhZDOqG.js → c4Diagram-AAUBKEIU-CkIDt_8M.js} +1 -1
- package/dist/client/assets/channel-CeYcnOtQ.js +1 -0
- package/dist/client/assets/{chunk-2J33WTMH-bcODs8jt.js → chunk-2J33WTMH-D9sbdaqY.js} +1 -1
- package/dist/client/assets/{chunk-4BX2VUAB-BRVq7S5r.js → chunk-4BX2VUAB-D3DtFoAY.js} +1 -1
- package/dist/client/assets/{chunk-55IACEB6-Cl7bhvca.js → chunk-55IACEB6-gxblxRIN.js} +1 -1
- package/dist/client/assets/{chunk-727SXJPM-B9loHJy3.js → chunk-727SXJPM-D0AEDoHN.js} +1 -1
- package/dist/client/assets/{chunk-AQP2D5EJ-CFA8hWsC.js → chunk-AQP2D5EJ-BweoM7qY.js} +1 -1
- package/dist/client/assets/{chunk-FMBD7UC4-Buu3h7wx.js → chunk-FMBD7UC4-sjblhrjI.js} +1 -1
- package/dist/client/assets/{chunk-ND2GUHAM-DN_GDbIs.js → chunk-ND2GUHAM-CYuRENSs.js} +1 -1
- package/dist/client/assets/{chunk-QZHKN3VN-1t7Gs0aF.js → chunk-QZHKN3VN-CFKJr4kG.js} +1 -1
- package/dist/client/assets/classDiagram-4FO5ZUOK-CZwh2k1X.js +1 -0
- package/dist/client/assets/classDiagram-v2-Q7XG4LA2-CZwh2k1X.js +1 -0
- package/dist/client/assets/{cose-bilkent-S5V4N54A-QqcFhiY1.js → cose-bilkent-S5V4N54A-6Jb6SgHM.js} +1 -1
- package/dist/client/assets/{dagre-BM42HDAG-DjGM9BIi.js → dagre-BM42HDAG-FQJ87Ol3.js} +1 -1
- package/dist/client/assets/{dashboard-view-BO_R8jyX.js → dashboard-view-B1Rq2-73.js} +1 -1
- package/dist/client/assets/{dashboard-view-CxXlYApd.js → dashboard-view-DH1Zo0m4.js} +1 -1
- package/dist/client/assets/{dashboard-view-UqmC6VTb.js → dashboard-view-DVMZNVoc.js} +1 -1
- package/dist/client/assets/{dashboard-view-C6rmfIOU.js → dashboard-view-DgLbkn5n.js} +1 -1
- package/dist/client/assets/{diagram-2AECGRRQ-BmCGpU8P.js → diagram-2AECGRRQ-CKYOu9to.js} +1 -1
- package/dist/client/assets/{diagram-5GNKFQAL-CKRL1dki.js → diagram-5GNKFQAL-DPwjFnde.js} +1 -1
- package/dist/client/assets/{diagram-KO2AKTUF-DE9deYyi.js → diagram-KO2AKTUF-DzZS9Nk3.js} +1 -1
- package/dist/client/assets/{diagram-LMA3HP47-BmUGF67J.js → diagram-LMA3HP47-DW9e5YSP.js} +1 -1
- package/dist/client/assets/{diagram-OG6HWLK6-DLUATSuY.js → diagram-OG6HWLK6-C7V3Mj63.js} +1 -1
- package/dist/client/assets/{erDiagram-TEJ5UH35-nl31ZyQ8.js → erDiagram-TEJ5UH35-DSQzBA0e.js} +1 -1
- package/dist/client/assets/{flowDiagram-I6XJVG4X-BZdpCw1-.js → flowDiagram-I6XJVG4X-BUuOq9tO.js} +1 -1
- package/dist/client/assets/{folder-open-DKvxGzrx.js → folder-open-DXNu4ED4.js} +1 -1
- package/dist/client/assets/{ganttDiagram-6RSMTGT7-DLsqztRx.js → ganttDiagram-6RSMTGT7-Bc32G_Ef.js} +1 -1
- package/dist/client/assets/{gitGraphDiagram-PVQCEYII-Dk_Sy9NG.js → gitGraphDiagram-PVQCEYII-Cvfa0W6a.js} +1 -1
- package/dist/client/assets/index-C5YdAqNQ.js +2767 -0
- package/dist/client/assets/index-DHk_s6rA.css +1 -0
- package/dist/client/assets/{infoDiagram-5YYISTIA-DoQ0ID21.js → infoDiagram-5YYISTIA-TqRUTfvy.js} +1 -1
- package/dist/client/assets/{ishikawaDiagram-YF4QCWOH-By5pgTzr.js → ishikawaDiagram-YF4QCWOH-D33iop1_.js} +1 -1
- package/dist/client/assets/{journeyDiagram-JHISSGLW-296a-u4u.js → journeyDiagram-JHISSGLW-DH-TDR5Y.js} +1 -1
- package/dist/client/assets/{kanban-definition-UN3LZRKU-CgcRr3lQ.js → kanban-definition-UN3LZRKU-CAcebzyW.js} +1 -1
- package/dist/client/assets/{mermaid.core-BMcqO-oY.js → mermaid.core-C3Nh1Sak.js} +4 -4
- package/dist/client/assets/{mindmap-definition-RKZ34NQL-C6poaNyh.js → mindmap-definition-RKZ34NQL-CSIccWV8.js} +1 -1
- package/dist/client/assets/{pieDiagram-4H26LBE5-D5kkHa4c.js → pieDiagram-4H26LBE5-hXJ3G3MR.js} +1 -1
- package/dist/client/assets/{puzzle-Dn0zAogb.js → puzzle-DI3elldh.js} +1 -1
- package/dist/client/assets/{quadrantDiagram-W4KKPZXB-XYE61FOQ.js → quadrantDiagram-W4KKPZXB-B3wpvneC.js} +1 -1
- package/dist/client/assets/{requirementDiagram-4Y6WPE33-ChSV14Ej.js → requirementDiagram-4Y6WPE33-DPpYL2ca.js} +1 -1
- package/dist/client/assets/{sankeyDiagram-5OEKKPKP-ToxhxAPY.js → sankeyDiagram-5OEKKPKP-BS9QXSEF.js} +1 -1
- package/dist/client/assets/{sequenceDiagram-3UESZ5HK-shNTGnMF.js → sequenceDiagram-3UESZ5HK-C_fqU0MW.js} +1 -1
- package/dist/client/assets/{shield-alert-DoqrZ4qS.js → shield-alert-CzZYSqmH.js} +1 -1
- package/dist/client/assets/{standing-instructions-template-CbBEfZ8s.js → standing-instructions-template-CsXA4WMZ.js} +1 -1
- package/dist/client/assets/{stateDiagram-AJRCARHV-CRNXfwiP.js → stateDiagram-AJRCARHV-DbSZrprS.js} +1 -1
- package/dist/client/assets/{stateDiagram-v2-BHNVJYJU-atxeIHh0.js → stateDiagram-v2-BHNVJYJU-Cf_ckwmM.js} +1 -1
- package/dist/client/assets/{timeline-definition-PNZ67QCA-CguHqKR6.js → timeline-definition-PNZ67QCA-B1ohvKjs.js} +1 -1
- package/dist/client/assets/{upload-BF3YtpBl.js → upload-D7nNe9wD.js} +1 -1
- package/dist/client/assets/{users-CN7wFV_l.js → users-B79Dvre4.js} +1 -1
- package/dist/client/assets/{vennDiagram-CIIHVFJN-BpgXesnN.js → vennDiagram-CIIHVFJN-CxZYCQ72.js} +1 -1
- package/dist/client/assets/{wardley-L42UT6IY-4QN1B18l.js → wardley-L42UT6IY-clJem8VU.js} +1 -1
- package/dist/client/assets/{wardleyDiagram-YWT4CUSO-BuYjCimN.js → wardleyDiagram-YWT4CUSO-zE8lyaWe.js} +1 -1
- package/dist/client/assets/{xychartDiagram-2RQKCTM6-DL8viWGe.js → xychartDiagram-2RQKCTM6-DsdjuSxQ.js} +1 -1
- package/dist/client/index.html +2 -2
- package/dist/client/version.json +1 -1
- package/dist/droid-cli/package.json +1 -1
- package/dist/extension.js +2594 -1746
- package/dist/migrations/0000_initial.sql +1 -0
- package/dist/migrations/0043_fn8768_dispatch_dedupe.sql +17 -0
- package/dist/migrations/0044_fn_8785_queued_episode_signature.sql +3 -0
- package/dist/pi-claude-cli/package.json +1 -1
- package/dist/plugin-sdk/index.d.ts +38 -1
- package/dist/plugin-sdk/index.js +71 -20
- package/dist/plugins/fusion-plugin-acp-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-acp-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-claude-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-claude-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-cli-printing-press/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-cli-printing-press/package.json +1 -1
- package/dist/plugins/fusion-plugin-compound-engineering/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-compound-engineering/package.json +1 -1
- package/dist/plugins/fusion-plugin-cursor-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-cursor-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-dependency-graph/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-dependency-graph/package.json +1 -1
- package/dist/plugins/fusion-plugin-droid-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-droid-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-grok-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-grok-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-hermes-runtime/.bundled.reload-3.js +19167 -0
- package/dist/plugins/fusion-plugin-hermes-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-hermes-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-linear-import/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-linear-import/package.json +1 -1
- package/dist/plugins/fusion-plugin-omp-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-omp-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-openclaw-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-openclaw-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-paperclip-runtime/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-paperclip-runtime/package.json +1 -1
- package/dist/plugins/fusion-plugin-quality/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-quality/package.json +1 -1
- package/dist/plugins/fusion-plugin-reports/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-reports/package.json +1 -1
- package/dist/plugins/fusion-plugin-roadmap/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-roadmap/package.json +1 -1
- package/dist/plugins/fusion-plugin-todos/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-todos/package.json +1 -1
- package/dist/plugins/fusion-plugin-whatsapp-chat/bundled.js +71 -20
- package/dist/plugins/fusion-plugin-whatsapp-chat/package.json +1 -1
- package/package.json +1 -1
- package/dist/client/assets/channel-CcVVD2V-.js +0 -1
- package/dist/client/assets/classDiagram-4FO5ZUOK-Bdeei39Q.js +0 -1
- package/dist/client/assets/classDiagram-v2-Q7XG4LA2-Bdeei39Q.js +0 -1
- package/dist/client/assets/index-DTimvpMD.js +0 -2685
- package/dist/client/assets/index-DjY0GTMz.css +0 -1
|
@@ -5787,6 +5787,7 @@ __export(project_exports, {
|
|
|
5787
5787
|
tasks: () => tasks,
|
|
5788
5788
|
todoItems: () => todoItems,
|
|
5789
5789
|
todoLists: () => todoLists,
|
|
5790
|
+
unplannedExecutionBlocks: () => unplannedExecutionBlocks,
|
|
5790
5791
|
usageEvents: () => usageEvents,
|
|
5791
5792
|
verificationCache: () => verificationCache,
|
|
5792
5793
|
workflowPromptOverrides: () => workflowPromptOverrides,
|
|
@@ -5826,6 +5827,7 @@ var tasks = projectSchema.table("tasks", {
|
|
|
5826
5827
|
worktree: text("worktree"),
|
|
5827
5828
|
blockedBy: text("blocked_by"),
|
|
5828
5829
|
overlapBlockedBy: text("overlap_blocked_by"),
|
|
5830
|
+
queuedLogEpisodeSignature: text("queued_log_episode_signature"),
|
|
5829
5831
|
paused: integer("paused").default(0),
|
|
5830
5832
|
userPaused: integer("user_paused").default(0),
|
|
5831
5833
|
pausedReason: text("paused_reason"),
|
|
@@ -6631,6 +6633,14 @@ var workflowWorkItems = projectSchema.table("workflow_work_items", {
|
|
|
6631
6633
|
index("idx_workflow_work_items_task_run").on(t.taskId, t.runId),
|
|
6632
6634
|
uniqueIndex("idx_workflow_work_items_one_active_task_continuation").on(t.projectId, t.taskId).where(sql`${t.kind} = 'task' AND ${t.state} IN ('runnable', 'running', 'held', 'retrying')`)
|
|
6633
6635
|
]);
|
|
6636
|
+
var unplannedExecutionBlocks = projectSchema.table("unplanned_execution_blocks", {
|
|
6637
|
+
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6638
|
+
taskId: text("task_id").notNull(),
|
|
6639
|
+
episode: text("episode").notNull(),
|
|
6640
|
+
createdAt: text("created_at").notNull()
|
|
6641
|
+
}, (t) => [
|
|
6642
|
+
primaryKey({ columns: [t.projectId, t.taskId, t.episode] })
|
|
6643
|
+
]);
|
|
6634
6644
|
var workflowRunBranches = projectSchema.table("workflow_run_branches", {
|
|
6635
6645
|
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6636
6646
|
taskId: text("task_id").notNull(),
|
|
@@ -12248,6 +12258,37 @@ var TRIAGE_POLICY_DEFAULTS = new Map(
|
|
|
12248
12258
|
BUILTIN_TRIAGE_POLICY_SETTINGS.map((setting) => [setting.id, setting.default])
|
|
12249
12259
|
);
|
|
12250
12260
|
|
|
12261
|
+
// ../core/src/agents/planning-review-policy.ts
|
|
12262
|
+
var PLANNING_COMPLETENESS_POLICY = `## Mandatory Planning Completeness Procedure
|
|
12263
|
+
|
|
12264
|
+
Before writing the final PROMPT.md:
|
|
12265
|
+
|
|
12266
|
+
1. Build an internal **planning ledger** from the Original Description, user comments, project instructions, cited issue/report, and any saved planning document. Preserve every requirement, settled decision, explicit non-goal, and acceptance outcome; do not silently replace the reporter's contract with a narrower reproduction.
|
|
12267
|
+
2. Research before structuring the plan. Trace the affected behavior through its real production entry points, callers, writers/readers, shared consumers, persistence boundaries, recovery paths, and existing tests. Resolve planning-time facts from the repository now; defer only details that genuinely require implementation-time discovery, and label those explicitly.
|
|
12268
|
+
3. For stateful, lifecycle, persistence, or concurrency work, enumerate the participant graph and both relevant orderings: stale-work cancellation/fencing, lock order and reentrancy, transaction boundaries, pool/transport limits, competing recovery paths, configuration/deployment identities, bypass/force paths, and failure cleanup. Every stated invariant must name the surfaces that preserve it.
|
|
12269
|
+
4. Map every ledger row to concrete File Scope entries, dependency-ordered implementation steps, and verification. Each feature-bearing step must name specific automated test scenarios with the input/state, action or ordering, and expected observable result; helper-only tests do not prove production reachability.
|
|
12270
|
+
5. Keep scope disciplined. Reuse current architecture and documented patterns unless the reporter contract requires a redesign. Put optional cleanup and adjacent ideas outside the active steps.
|
|
12271
|
+
6. Before persisting PROMPT.md, perform a fresh holistic completeness pass across the whole ledger. Try to disprove the plan by checking referenced paths and patterns, missing consumers, contradictory steps, uncovered failure orderings, and acceptance criteria without tests. Fix all gaps in one pass.
|
|
12272
|
+
|
|
12273
|
+
On a revision, treat the cumulative revision ledger as durable decisions unless a later entry explicitly supersedes one: preserve resolved items, address every unresolved item surgically, and rerun the complete procedure instead of checking only the latest comment.`;
|
|
12274
|
+
var FAST_PLANNING_COMPLETENESS_POLICY = `## Fast Planning Completeness Check
|
|
12275
|
+
|
|
12276
|
+
Before writing PROMPT.md, build a compact internal ledger from the Original Description, user comments, project rules, and acceptance outcomes. Inspect the real production callers/consumers, persistence or recovery paths, and nearby tests before choosing File Scope. For stateful or concurrent work, enumerate every participant plus both relevant orderings and failure cleanup. Map each requirement to a concrete step, file, and automated test scenario, then reread the whole plan once for missing surfaces, contradictions, and unproved outcomes. On revision, preserve prior decisions and address the complete cumulative ledger, not only the latest note.`;
|
|
12277
|
+
var PLAN_REVIEW_COMPLETENESS_POLICY = `## Mandatory Plan Review Procedure
|
|
12278
|
+
|
|
12279
|
+
Apply this procedure to plan/spec reviews only. Code reviews use their dedicated procedure.
|
|
12280
|
+
|
|
12281
|
+
Before choosing a verdict:
|
|
12282
|
+
|
|
12283
|
+
1. Build one **review ledger** for the entire PROMPT.md: Original Description and user comments; Mission and Completion Criteria; Surface Enumeration and Symptom Verification when required; every implementation step, File Scope entry, dependency, risk, and test/verification promise.
|
|
12284
|
+
2. Complete the full review before reporting. Check coherence and requirement traceability, feasibility against the current repository, scope discipline, execution ordering, and verification quality. When relevant, also inspect security/data integrity, state transitions, concurrency orderings, deployment/configuration boundaries, recovery competitors, and force/bypass paths.
|
|
12285
|
+
3. Review at specification altitude. Block when a required behavior, surface, ordering, safety constraint, or proof is missing or the stated approach cannot work. Keep optional implementation detail, wording polish, and nonessential improvements advisory.
|
|
12286
|
+
4. If REVISE is necessary, batch **all independently discoverable blocking findings** into this one verdict; do not stop after the first defect. Give each blocker a stable ID, cite the affected section or repository evidence, and state the concrete PROMPT.md correction. Put advisory observations in a separate list.
|
|
12287
|
+
5. On re-review, use the supplied prior-review ledger as a decision primer. Verify every prior blocker, do not re-raise resolved or rejected semantic duplicates, and preserve accepted decisions. A newly blocking finding must say whether the revision introduced it, which prior blocker genuinely masked it, or why it is independently delivery-blocking for correctness, security, data safety, or executability. Record an earlier reviewer miss explicitly; never demote a critical defect merely because it was missed before.
|
|
12288
|
+
6. After any same-session PROMPT.md edit, distrust the edit: reread the complete artifact, rebuild the ledger, and perform a fresh holistic pass before APPROVE.
|
|
12289
|
+
|
|
12290
|
+
APPROVE when the plan is executable and verifiable, not when it is cosmetically perfect. If returning REVISE, the verdict notes must contain the complete blocking checklist because those notes are the durable input to the next planning round.`;
|
|
12291
|
+
|
|
12251
12292
|
// ../core/src/agents/agent-prompts.ts
|
|
12252
12293
|
var FUSION_RUNTIME_SELF_AWARENESS = `## Runtime Self-Awareness
|
|
12253
12294
|
|
|
@@ -12454,6 +12495,8 @@ The tool prevents your session from being killed by the inactivity watchdog duri
|
|
|
12454
12495
|
- If a verification command times out, do NOT blindly retry \u2014 investigate. Check for hung subprocesses, infinite test loops, or tests waiting on missing dependencies. Use \`node_modules/.modules.yaml\` presence to confirm bootstrap.`;
|
|
12455
12496
|
var FAST_TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn". This task is running in **fast mode**.
|
|
12456
12497
|
|
|
12498
|
+
${FAST_PLANNING_COMPLETENESS_POLICY}
|
|
12499
|
+
|
|
12457
12500
|
Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heavyweight ceremony, review scoring, and proactive subtask analysis.
|
|
12458
12501
|
|
|
12459
12502
|
## Fast-mode priorities
|
|
@@ -12462,7 +12505,7 @@ Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heav
|
|
|
12462
12505
|
- Preserve required safety sections for bugs, workflow routing, forensic tasks, and decision-only work.
|
|
12463
12506
|
|
|
12464
12507
|
## Duplicate check
|
|
12465
|
-
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols.
|
|
12508
|
+
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols. Do not search completed or archived work for duplicate candidates. When an active match is a duplicate, do not write a spec \u2014 but still write PROMPT.md, with its entire contents being the single line \`DUPLICATE: {existing-task-id}\` and nothing else. That file is how the duplicate is recorded; announcing it only in your reply leaves no plan behind and re-plans the task in a loop.
|
|
12466
12509
|
|
|
12467
12510
|
## Required PROMPT.md shape
|
|
12468
12511
|
Write PROMPT.md with Original Description, Before \u2192 After Transformation, Mission, Dependencies, Context to Read First, File Scope, Steps, Documentation Requirements, Completion Criteria, Git Commit Convention, and Do NOT. Put \`## Original Description\` immediately after the title/\`Created\`/\`Size\` metadata with the operator's original task description copied **verbatim** (do not paraphrase). Put \`## Before \u2192 After Transformation\` next, before \`## Mission\`, with concise Before/After bullets: current state, target state, why it satisfies the user's request at a glance. In \`## Steps\`, every executable heading MUST use \`### Step N: <name>\` (e.g. \`### Step 1: Preflight\`). Do not write bare \`### Preflight\` / \`### Implementation\` headings, and do not add review-level, triage subtask, or proactive subtask headings.
|
|
@@ -12504,6 +12547,8 @@ If the requested outcome is only to decide, route, or coordinate work, include \
|
|
|
12504
12547
|
Write PROMPT.md directly and stop. Do not call \`fn_review_spec()\`; workflow Plan Review is the single optional plan review gate before execution.`;
|
|
12505
12548
|
var TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn", an AI-orchestrated task board.
|
|
12506
12549
|
|
|
12550
|
+
${PLANNING_COMPLETENESS_POLICY}
|
|
12551
|
+
|
|
12507
12552
|
## Your Role
|
|
12508
12553
|
You are the specification quality gate for implementation success.
|
|
12509
12554
|
Your job: take a rough task description and produce a fully specified PROMPT.md that another AI agent can execute autonomously in a fresh context with zero memory of this conversation.
|
|
@@ -12728,9 +12773,9 @@ Verified facts about this codebase's storage \u2014 cite these correctly so Plan
|
|
|
12728
12773
|
- New Postgres migrations must be **registered explicitly** in \`packages/core/src/postgres/schema-applier.ts\` (version constant + bookkeeping check); a \`.sql\` file dropped in the migrations dir that is not wired there silently never runs.
|
|
12729
12774
|
|
|
12730
12775
|
## Duplicate check
|
|
12731
|
-
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12732
|
-
|
|
12733
|
-
If
|
|
12776
|
+
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12777
|
+
Do not search completed or archived work for duplicate candidates.
|
|
12778
|
+
If an actionable task already covers the same work (even if worded differently), do not write a spec.
|
|
12734
12779
|
Instead you MUST still write PROMPT.md, with its ENTIRE contents being this one line and nothing else:
|
|
12735
12780
|
\`DUPLICATE: {existing-task-id}\`
|
|
12736
12781
|
Writing that file IS how you report the duplicate. Reporting it only in your reply is not recorded:
|
|
@@ -12761,7 +12806,7 @@ Workflow policy can disable proactive oversized-task splitting for operators who
|
|
|
12761
12806
|
## Triage tools
|
|
12762
12807
|
You have these extra tools during triage:
|
|
12763
12808
|
- \`fn_task_list\` \u2014 list existing active tasks
|
|
12764
|
-
- \`fn_task_search\` \u2014 keyword search across tasks
|
|
12809
|
+
- \`fn_task_search\` \u2014 keyword search across active tasks by default; history is opt-in for non-duplicate research
|
|
12765
12810
|
- \`fn_task_show\` \u2014 inspect a task and its PROMPT.md
|
|
12766
12811
|
- \`fn_task_create\` \u2014 create a child/follow-up task while triaging
|
|
12767
12812
|
- \`fn_task_document_write\` \u2014 save a planning document (e.g., key="plan")
|
|
@@ -12988,20 +13033,6 @@ Concrete examples:
|
|
|
12988
13033
|
- [Optional improvements, not blocking]
|
|
12989
13034
|
\`\`\`
|
|
12990
13035
|
|
|
12991
|
-
## Spec / Plan Review Convergence
|
|
12992
|
-
|
|
12993
|
-
Specs and pre-execution Plan Review share this gate. Prefer **APPROVE** / **APPROVE_WITH_NOTES** when the plan is executable enough for an agent to implement. Put optional polish only under **Suggestions**.
|
|
12994
|
-
|
|
12995
|
-
When you must **REVISE**:
|
|
12996
|
-
- List each blocking issue as a concrete PROMPT.md edit (which section, what to add/change/remove).
|
|
12997
|
-
- Do not demand a full rewrite unless the approach is fundamentally wrong (**RETHINK**).
|
|
12998
|
-
- Prefer fixing local PROMPT.md defects in-session when you have write tools, then **APPROVE**, instead of bouncing the task through another full replan cycle.
|
|
12999
|
-
|
|
13000
|
-
**Converging on re-review (when the request includes your prior feedback + a Plan Review attempt number):**
|
|
13001
|
-
- This is a spec you already reviewed. VERIFY each issue you previously raised was addressed. REVISE only for (a) a PRIOR blocking issue still unresolved, or (b) a genuinely NEW problem THIS revision introduced.
|
|
13002
|
-
- Do NOT introduce a new blocking issue that ALSO applied to the version you previously reviewed \u2014 that is your own earlier miss. Record it under **Suggestions**, not REVISE.
|
|
13003
|
-
- **Severity ratchet at attempt 3+:** gate ONLY on \`critical\` (delivery-blocking) issues. Downgrade lone \`important\`/\`minor\` spec-wording nits to **Suggestions** and APPROVE. Rationale: the executor and downstream code review are later gates \u2014 a spec need not be perfect to be executable, only executable.
|
|
13004
|
-
|
|
13005
13036
|
## Spec Review \u2014 Undersplit Task Detection
|
|
13006
13037
|
|
|
13007
13038
|
When reviewing specs, assess whether the task should have been broken into subtasks. The bar for splitting is high \u2014 most tasks should remain whole. Coordination overhead (worktrees, dependency wiring, merge sequencing) is real, so splitting must clearly pay for itself.
|
|
@@ -13649,6 +13680,22 @@ function browserVerificationOptionalGroupNode(column, options = {}) {
|
|
|
13649
13680
|
};
|
|
13650
13681
|
}
|
|
13651
13682
|
|
|
13683
|
+
// ../core/src/agents/code-review-policy.ts
|
|
13684
|
+
var CODE_REVIEW_COMPLETENESS_POLICY = `## Mandatory Code Review Procedure
|
|
13685
|
+
|
|
13686
|
+
Apply this procedure to code reviews only. Plan/spec reviews use their dedicated criteria below.
|
|
13687
|
+
|
|
13688
|
+
Before choosing a verdict:
|
|
13689
|
+
|
|
13690
|
+
1. Build a **requirements ledger** from PROMPT.md. For every Mission outcome, Completion Criterion, relevant \`## Surface Enumeration\` item, and \`## Symptom Verification\` assertion, identify both implementation evidence and test evidence. A missing or partial required row is REVISE.
|
|
13691
|
+
2. Trace each changed behavior from its **real production entry point** and selector through the changed helper to its consumers. A helper-only unit test does not prove that production can reach the new branch.
|
|
13692
|
+
3. For bug fixes, locate an automated test that reproduces the exact reported failure and asserts it is gone across the enumerated surfaces. Green builds, nearby unit tests, and synthetic state injection alone are insufficient.
|
|
13693
|
+
4. For persistence, lifecycle, and concurrency changes, verify that evidence belongs to the **current state, version, or planning episode**; every read-check-write path is atomic, locked, or CAS-protected; all mutation surfaces preserve the invariant; waits and locks are bounded and cleanup-safe; and a production-shaped integration test covers both relevant orderings.
|
|
13694
|
+
5. For route or state changes, inspect all affected UI, API, CLI, and agent consumers plus paired success/failure or approve/reject paths for contract parity.
|
|
13695
|
+
6. Before APPROVE, perform a fresh adversarial validation pass: try to disprove each completed ledger row by reading the real caller, guard, consumer, and test. Cite \`file:line\` for every blocker and name the exact missing proof.
|
|
13696
|
+
|
|
13697
|
+
APPROVE only when every required ledger row has implementation and test evidence.`;
|
|
13698
|
+
|
|
13652
13699
|
// ../core/src/workflows/builtin-code-review-group.ts
|
|
13653
13700
|
var CODE_REVIEW_GROUP_ID = "code-review";
|
|
13654
13701
|
var CODE_REVIEW_STEP_NODE_ID = "code-review-step";
|
|
@@ -13668,6 +13715,8 @@ var CODE_REVIEW_PROMPT = `You are a senior code reviewer. Review the task's diff
|
|
|
13668
13715
|
5. **Error handling** \u2014 swallowed errors, unhandled rejections/exceptions, missing validation at trust boundaries, misleading error messages.
|
|
13669
13716
|
6. **Contract / signature changes** \u2014 changed function/exported-type signatures, API request/response shapes, or serialization that breaks existing callers.
|
|
13670
13717
|
|
|
13718
|
+
${CODE_REVIEW_COMPLETENESS_POLICY}
|
|
13719
|
+
|
|
13671
13720
|
Be specific: cite \`file:line\` for every finding and explain the concrete failure it causes.
|
|
13672
13721
|
|
|
13673
13722
|
## Output Requirements
|
|
@@ -13787,12 +13836,14 @@ var PLAN_REVIEW_PROMPT = `You are a senior plan reviewer. Review the task's PROM
|
|
|
13787
13836
|
4. **Verification quality** \u2014 absent or weak tests/checks for the behavior being changed.
|
|
13788
13837
|
5. **Risk callouts** \u2014 migrations, data-loss paths, external integrations, secrets, or plugin/runtime dependencies that need explicit handling.
|
|
13789
13838
|
|
|
13839
|
+
${PLAN_REVIEW_COMPLETENESS_POLICY}
|
|
13840
|
+
|
|
13790
13841
|
Be specific: cite the plan section or file path for every finding and explain the concrete correction.
|
|
13791
13842
|
|
|
13792
13843
|
## Output Requirements
|
|
13793
13844
|
- APPROVE: the plan is ready for execution.
|
|
13794
13845
|
- APPROVE_WITH_NOTES: execution may proceed, but include non-blocking advisory notes.
|
|
13795
|
-
- REVISE: the plan should be corrected before execution; include
|
|
13846
|
+
- REVISE: the plan should be corrected before execution; include every blocking finding and needed change in the JSON notes, not only in preceding prose.
|
|
13796
13847
|
- Final output: output exactly one trailing JSON object on the final line (no markdown fences, no surrounding prose):
|
|
13797
13848
|
{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"..."}`;
|
|
13798
13849
|
function planReviewOptionalGroupNode(column, options = {}) {
|
|
@@ -5759,6 +5759,7 @@ __export(project_exports, {
|
|
|
5759
5759
|
tasks: () => tasks,
|
|
5760
5760
|
todoItems: () => todoItems,
|
|
5761
5761
|
todoLists: () => todoLists,
|
|
5762
|
+
unplannedExecutionBlocks: () => unplannedExecutionBlocks,
|
|
5762
5763
|
usageEvents: () => usageEvents,
|
|
5763
5764
|
verificationCache: () => verificationCache,
|
|
5764
5765
|
workflowPromptOverrides: () => workflowPromptOverrides,
|
|
@@ -5798,6 +5799,7 @@ var tasks = projectSchema.table("tasks", {
|
|
|
5798
5799
|
worktree: text("worktree"),
|
|
5799
5800
|
blockedBy: text("blocked_by"),
|
|
5800
5801
|
overlapBlockedBy: text("overlap_blocked_by"),
|
|
5802
|
+
queuedLogEpisodeSignature: text("queued_log_episode_signature"),
|
|
5801
5803
|
paused: integer("paused").default(0),
|
|
5802
5804
|
userPaused: integer("user_paused").default(0),
|
|
5803
5805
|
pausedReason: text("paused_reason"),
|
|
@@ -6603,6 +6605,14 @@ var workflowWorkItems = projectSchema.table("workflow_work_items", {
|
|
|
6603
6605
|
index("idx_workflow_work_items_task_run").on(t.taskId, t.runId),
|
|
6604
6606
|
uniqueIndex("idx_workflow_work_items_one_active_task_continuation").on(t.projectId, t.taskId).where(sql`${t.kind} = 'task' AND ${t.state} IN ('runnable', 'running', 'held', 'retrying')`)
|
|
6605
6607
|
]);
|
|
6608
|
+
var unplannedExecutionBlocks = projectSchema.table("unplanned_execution_blocks", {
|
|
6609
|
+
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6610
|
+
taskId: text("task_id").notNull(),
|
|
6611
|
+
episode: text("episode").notNull(),
|
|
6612
|
+
createdAt: text("created_at").notNull()
|
|
6613
|
+
}, (t) => [
|
|
6614
|
+
primaryKey({ columns: [t.projectId, t.taskId, t.episode] })
|
|
6615
|
+
]);
|
|
6606
6616
|
var workflowRunBranches = projectSchema.table("workflow_run_branches", {
|
|
6607
6617
|
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6608
6618
|
taskId: text("task_id").notNull(),
|
|
@@ -12128,6 +12138,37 @@ var TRIAGE_POLICY_DEFAULTS = new Map(
|
|
|
12128
12138
|
BUILTIN_TRIAGE_POLICY_SETTINGS.map((setting) => [setting.id, setting.default])
|
|
12129
12139
|
);
|
|
12130
12140
|
|
|
12141
|
+
// ../core/src/agents/planning-review-policy.ts
|
|
12142
|
+
var PLANNING_COMPLETENESS_POLICY = `## Mandatory Planning Completeness Procedure
|
|
12143
|
+
|
|
12144
|
+
Before writing the final PROMPT.md:
|
|
12145
|
+
|
|
12146
|
+
1. Build an internal **planning ledger** from the Original Description, user comments, project instructions, cited issue/report, and any saved planning document. Preserve every requirement, settled decision, explicit non-goal, and acceptance outcome; do not silently replace the reporter's contract with a narrower reproduction.
|
|
12147
|
+
2. Research before structuring the plan. Trace the affected behavior through its real production entry points, callers, writers/readers, shared consumers, persistence boundaries, recovery paths, and existing tests. Resolve planning-time facts from the repository now; defer only details that genuinely require implementation-time discovery, and label those explicitly.
|
|
12148
|
+
3. For stateful, lifecycle, persistence, or concurrency work, enumerate the participant graph and both relevant orderings: stale-work cancellation/fencing, lock order and reentrancy, transaction boundaries, pool/transport limits, competing recovery paths, configuration/deployment identities, bypass/force paths, and failure cleanup. Every stated invariant must name the surfaces that preserve it.
|
|
12149
|
+
4. Map every ledger row to concrete File Scope entries, dependency-ordered implementation steps, and verification. Each feature-bearing step must name specific automated test scenarios with the input/state, action or ordering, and expected observable result; helper-only tests do not prove production reachability.
|
|
12150
|
+
5. Keep scope disciplined. Reuse current architecture and documented patterns unless the reporter contract requires a redesign. Put optional cleanup and adjacent ideas outside the active steps.
|
|
12151
|
+
6. Before persisting PROMPT.md, perform a fresh holistic completeness pass across the whole ledger. Try to disprove the plan by checking referenced paths and patterns, missing consumers, contradictory steps, uncovered failure orderings, and acceptance criteria without tests. Fix all gaps in one pass.
|
|
12152
|
+
|
|
12153
|
+
On a revision, treat the cumulative revision ledger as durable decisions unless a later entry explicitly supersedes one: preserve resolved items, address every unresolved item surgically, and rerun the complete procedure instead of checking only the latest comment.`;
|
|
12154
|
+
var FAST_PLANNING_COMPLETENESS_POLICY = `## Fast Planning Completeness Check
|
|
12155
|
+
|
|
12156
|
+
Before writing PROMPT.md, build a compact internal ledger from the Original Description, user comments, project rules, and acceptance outcomes. Inspect the real production callers/consumers, persistence or recovery paths, and nearby tests before choosing File Scope. For stateful or concurrent work, enumerate every participant plus both relevant orderings and failure cleanup. Map each requirement to a concrete step, file, and automated test scenario, then reread the whole plan once for missing surfaces, contradictions, and unproved outcomes. On revision, preserve prior decisions and address the complete cumulative ledger, not only the latest note.`;
|
|
12157
|
+
var PLAN_REVIEW_COMPLETENESS_POLICY = `## Mandatory Plan Review Procedure
|
|
12158
|
+
|
|
12159
|
+
Apply this procedure to plan/spec reviews only. Code reviews use their dedicated procedure.
|
|
12160
|
+
|
|
12161
|
+
Before choosing a verdict:
|
|
12162
|
+
|
|
12163
|
+
1. Build one **review ledger** for the entire PROMPT.md: Original Description and user comments; Mission and Completion Criteria; Surface Enumeration and Symptom Verification when required; every implementation step, File Scope entry, dependency, risk, and test/verification promise.
|
|
12164
|
+
2. Complete the full review before reporting. Check coherence and requirement traceability, feasibility against the current repository, scope discipline, execution ordering, and verification quality. When relevant, also inspect security/data integrity, state transitions, concurrency orderings, deployment/configuration boundaries, recovery competitors, and force/bypass paths.
|
|
12165
|
+
3. Review at specification altitude. Block when a required behavior, surface, ordering, safety constraint, or proof is missing or the stated approach cannot work. Keep optional implementation detail, wording polish, and nonessential improvements advisory.
|
|
12166
|
+
4. If REVISE is necessary, batch **all independently discoverable blocking findings** into this one verdict; do not stop after the first defect. Give each blocker a stable ID, cite the affected section or repository evidence, and state the concrete PROMPT.md correction. Put advisory observations in a separate list.
|
|
12167
|
+
5. On re-review, use the supplied prior-review ledger as a decision primer. Verify every prior blocker, do not re-raise resolved or rejected semantic duplicates, and preserve accepted decisions. A newly blocking finding must say whether the revision introduced it, which prior blocker genuinely masked it, or why it is independently delivery-blocking for correctness, security, data safety, or executability. Record an earlier reviewer miss explicitly; never demote a critical defect merely because it was missed before.
|
|
12168
|
+
6. After any same-session PROMPT.md edit, distrust the edit: reread the complete artifact, rebuild the ledger, and perform a fresh holistic pass before APPROVE.
|
|
12169
|
+
|
|
12170
|
+
APPROVE when the plan is executable and verifiable, not when it is cosmetically perfect. If returning REVISE, the verdict notes must contain the complete blocking checklist because those notes are the durable input to the next planning round.`;
|
|
12171
|
+
|
|
12131
12172
|
// ../core/src/agents/agent-prompts.ts
|
|
12132
12173
|
var FUSION_RUNTIME_SELF_AWARENESS = `## Runtime Self-Awareness
|
|
12133
12174
|
|
|
@@ -12334,6 +12375,8 @@ The tool prevents your session from being killed by the inactivity watchdog duri
|
|
|
12334
12375
|
- If a verification command times out, do NOT blindly retry \u2014 investigate. Check for hung subprocesses, infinite test loops, or tests waiting on missing dependencies. Use \`node_modules/.modules.yaml\` presence to confirm bootstrap.`;
|
|
12335
12376
|
var FAST_TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn". This task is running in **fast mode**.
|
|
12336
12377
|
|
|
12378
|
+
${FAST_PLANNING_COMPLETENESS_POLICY}
|
|
12379
|
+
|
|
12337
12380
|
Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heavyweight ceremony, review scoring, and proactive subtask analysis.
|
|
12338
12381
|
|
|
12339
12382
|
## Fast-mode priorities
|
|
@@ -12342,7 +12385,7 @@ Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heav
|
|
|
12342
12385
|
- Preserve required safety sections for bugs, workflow routing, forensic tasks, and decision-only work.
|
|
12343
12386
|
|
|
12344
12387
|
## Duplicate check
|
|
12345
|
-
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols.
|
|
12388
|
+
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols. Do not search completed or archived work for duplicate candidates. When an active match is a duplicate, do not write a spec \u2014 but still write PROMPT.md, with its entire contents being the single line \`DUPLICATE: {existing-task-id}\` and nothing else. That file is how the duplicate is recorded; announcing it only in your reply leaves no plan behind and re-plans the task in a loop.
|
|
12346
12389
|
|
|
12347
12390
|
## Required PROMPT.md shape
|
|
12348
12391
|
Write PROMPT.md with Original Description, Before \u2192 After Transformation, Mission, Dependencies, Context to Read First, File Scope, Steps, Documentation Requirements, Completion Criteria, Git Commit Convention, and Do NOT. Put \`## Original Description\` immediately after the title/\`Created\`/\`Size\` metadata with the operator's original task description copied **verbatim** (do not paraphrase). Put \`## Before \u2192 After Transformation\` next, before \`## Mission\`, with concise Before/After bullets: current state, target state, why it satisfies the user's request at a glance. In \`## Steps\`, every executable heading MUST use \`### Step N: <name>\` (e.g. \`### Step 1: Preflight\`). Do not write bare \`### Preflight\` / \`### Implementation\` headings, and do not add review-level, triage subtask, or proactive subtask headings.
|
|
@@ -12384,6 +12427,8 @@ If the requested outcome is only to decide, route, or coordinate work, include \
|
|
|
12384
12427
|
Write PROMPT.md directly and stop. Do not call \`fn_review_spec()\`; workflow Plan Review is the single optional plan review gate before execution.`;
|
|
12385
12428
|
var TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn", an AI-orchestrated task board.
|
|
12386
12429
|
|
|
12430
|
+
${PLANNING_COMPLETENESS_POLICY}
|
|
12431
|
+
|
|
12387
12432
|
## Your Role
|
|
12388
12433
|
You are the specification quality gate for implementation success.
|
|
12389
12434
|
Your job: take a rough task description and produce a fully specified PROMPT.md that another AI agent can execute autonomously in a fresh context with zero memory of this conversation.
|
|
@@ -12608,9 +12653,9 @@ Verified facts about this codebase's storage \u2014 cite these correctly so Plan
|
|
|
12608
12653
|
- New Postgres migrations must be **registered explicitly** in \`packages/core/src/postgres/schema-applier.ts\` (version constant + bookkeeping check); a \`.sql\` file dropped in the migrations dir that is not wired there silently never runs.
|
|
12609
12654
|
|
|
12610
12655
|
## Duplicate check
|
|
12611
|
-
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12612
|
-
|
|
12613
|
-
If
|
|
12656
|
+
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12657
|
+
Do not search completed or archived work for duplicate candidates.
|
|
12658
|
+
If an actionable task already covers the same work (even if worded differently), do not write a spec.
|
|
12614
12659
|
Instead you MUST still write PROMPT.md, with its ENTIRE contents being this one line and nothing else:
|
|
12615
12660
|
\`DUPLICATE: {existing-task-id}\`
|
|
12616
12661
|
Writing that file IS how you report the duplicate. Reporting it only in your reply is not recorded:
|
|
@@ -12641,7 +12686,7 @@ Workflow policy can disable proactive oversized-task splitting for operators who
|
|
|
12641
12686
|
## Triage tools
|
|
12642
12687
|
You have these extra tools during triage:
|
|
12643
12688
|
- \`fn_task_list\` \u2014 list existing active tasks
|
|
12644
|
-
- \`fn_task_search\` \u2014 keyword search across tasks
|
|
12689
|
+
- \`fn_task_search\` \u2014 keyword search across active tasks by default; history is opt-in for non-duplicate research
|
|
12645
12690
|
- \`fn_task_show\` \u2014 inspect a task and its PROMPT.md
|
|
12646
12691
|
- \`fn_task_create\` \u2014 create a child/follow-up task while triaging
|
|
12647
12692
|
- \`fn_task_document_write\` \u2014 save a planning document (e.g., key="plan")
|
|
@@ -12868,20 +12913,6 @@ Concrete examples:
|
|
|
12868
12913
|
- [Optional improvements, not blocking]
|
|
12869
12914
|
\`\`\`
|
|
12870
12915
|
|
|
12871
|
-
## Spec / Plan Review Convergence
|
|
12872
|
-
|
|
12873
|
-
Specs and pre-execution Plan Review share this gate. Prefer **APPROVE** / **APPROVE_WITH_NOTES** when the plan is executable enough for an agent to implement. Put optional polish only under **Suggestions**.
|
|
12874
|
-
|
|
12875
|
-
When you must **REVISE**:
|
|
12876
|
-
- List each blocking issue as a concrete PROMPT.md edit (which section, what to add/change/remove).
|
|
12877
|
-
- Do not demand a full rewrite unless the approach is fundamentally wrong (**RETHINK**).
|
|
12878
|
-
- Prefer fixing local PROMPT.md defects in-session when you have write tools, then **APPROVE**, instead of bouncing the task through another full replan cycle.
|
|
12879
|
-
|
|
12880
|
-
**Converging on re-review (when the request includes your prior feedback + a Plan Review attempt number):**
|
|
12881
|
-
- This is a spec you already reviewed. VERIFY each issue you previously raised was addressed. REVISE only for (a) a PRIOR blocking issue still unresolved, or (b) a genuinely NEW problem THIS revision introduced.
|
|
12882
|
-
- Do NOT introduce a new blocking issue that ALSO applied to the version you previously reviewed \u2014 that is your own earlier miss. Record it under **Suggestions**, not REVISE.
|
|
12883
|
-
- **Severity ratchet at attempt 3+:** gate ONLY on \`critical\` (delivery-blocking) issues. Downgrade lone \`important\`/\`minor\` spec-wording nits to **Suggestions** and APPROVE. Rationale: the executor and downstream code review are later gates \u2014 a spec need not be perfect to be executable, only executable.
|
|
12884
|
-
|
|
12885
12916
|
## Spec Review \u2014 Undersplit Task Detection
|
|
12886
12917
|
|
|
12887
12918
|
When reviewing specs, assess whether the task should have been broken into subtasks. The bar for splitting is high \u2014 most tasks should remain whole. Coordination overhead (worktrees, dependency wiring, merge sequencing) is real, so splitting must clearly pay for itself.
|
|
@@ -13529,6 +13560,22 @@ function browserVerificationOptionalGroupNode(column, options = {}) {
|
|
|
13529
13560
|
};
|
|
13530
13561
|
}
|
|
13531
13562
|
|
|
13563
|
+
// ../core/src/agents/code-review-policy.ts
|
|
13564
|
+
var CODE_REVIEW_COMPLETENESS_POLICY = `## Mandatory Code Review Procedure
|
|
13565
|
+
|
|
13566
|
+
Apply this procedure to code reviews only. Plan/spec reviews use their dedicated criteria below.
|
|
13567
|
+
|
|
13568
|
+
Before choosing a verdict:
|
|
13569
|
+
|
|
13570
|
+
1. Build a **requirements ledger** from PROMPT.md. For every Mission outcome, Completion Criterion, relevant \`## Surface Enumeration\` item, and \`## Symptom Verification\` assertion, identify both implementation evidence and test evidence. A missing or partial required row is REVISE.
|
|
13571
|
+
2. Trace each changed behavior from its **real production entry point** and selector through the changed helper to its consumers. A helper-only unit test does not prove that production can reach the new branch.
|
|
13572
|
+
3. For bug fixes, locate an automated test that reproduces the exact reported failure and asserts it is gone across the enumerated surfaces. Green builds, nearby unit tests, and synthetic state injection alone are insufficient.
|
|
13573
|
+
4. For persistence, lifecycle, and concurrency changes, verify that evidence belongs to the **current state, version, or planning episode**; every read-check-write path is atomic, locked, or CAS-protected; all mutation surfaces preserve the invariant; waits and locks are bounded and cleanup-safe; and a production-shaped integration test covers both relevant orderings.
|
|
13574
|
+
5. For route or state changes, inspect all affected UI, API, CLI, and agent consumers plus paired success/failure or approve/reject paths for contract parity.
|
|
13575
|
+
6. Before APPROVE, perform a fresh adversarial validation pass: try to disprove each completed ledger row by reading the real caller, guard, consumer, and test. Cite \`file:line\` for every blocker and name the exact missing proof.
|
|
13576
|
+
|
|
13577
|
+
APPROVE only when every required ledger row has implementation and test evidence.`;
|
|
13578
|
+
|
|
13532
13579
|
// ../core/src/workflows/builtin-code-review-group.ts
|
|
13533
13580
|
var CODE_REVIEW_GROUP_ID = "code-review";
|
|
13534
13581
|
var CODE_REVIEW_STEP_NODE_ID = "code-review-step";
|
|
@@ -13548,6 +13595,8 @@ var CODE_REVIEW_PROMPT = `You are a senior code reviewer. Review the task's diff
|
|
|
13548
13595
|
5. **Error handling** \u2014 swallowed errors, unhandled rejections/exceptions, missing validation at trust boundaries, misleading error messages.
|
|
13549
13596
|
6. **Contract / signature changes** \u2014 changed function/exported-type signatures, API request/response shapes, or serialization that breaks existing callers.
|
|
13550
13597
|
|
|
13598
|
+
${CODE_REVIEW_COMPLETENESS_POLICY}
|
|
13599
|
+
|
|
13551
13600
|
Be specific: cite \`file:line\` for every finding and explain the concrete failure it causes.
|
|
13552
13601
|
|
|
13553
13602
|
## Output Requirements
|
|
@@ -13667,12 +13716,14 @@ var PLAN_REVIEW_PROMPT = `You are a senior plan reviewer. Review the task's PROM
|
|
|
13667
13716
|
4. **Verification quality** \u2014 absent or weak tests/checks for the behavior being changed.
|
|
13668
13717
|
5. **Risk callouts** \u2014 migrations, data-loss paths, external integrations, secrets, or plugin/runtime dependencies that need explicit handling.
|
|
13669
13718
|
|
|
13719
|
+
${PLAN_REVIEW_COMPLETENESS_POLICY}
|
|
13720
|
+
|
|
13670
13721
|
Be specific: cite the plan section or file path for every finding and explain the concrete correction.
|
|
13671
13722
|
|
|
13672
13723
|
## Output Requirements
|
|
13673
13724
|
- APPROVE: the plan is ready for execution.
|
|
13674
13725
|
- APPROVE_WITH_NOTES: execution may proceed, but include non-blocking advisory notes.
|
|
13675
|
-
- REVISE: the plan should be corrected before execution; include
|
|
13726
|
+
- REVISE: the plan should be corrected before execution; include every blocking finding and needed change in the JSON notes, not only in preceding prose.
|
|
13676
13727
|
- Final output: output exactly one trailing JSON object on the final line (no markdown fences, no surrounding prose):
|
|
13677
13728
|
{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"..."}`;
|
|
13678
13729
|
function planReviewOptionalGroupNode(column, options = {}) {
|
|
@@ -5759,6 +5759,7 @@ __export(project_exports, {
|
|
|
5759
5759
|
tasks: () => tasks,
|
|
5760
5760
|
todoItems: () => todoItems,
|
|
5761
5761
|
todoLists: () => todoLists,
|
|
5762
|
+
unplannedExecutionBlocks: () => unplannedExecutionBlocks,
|
|
5762
5763
|
usageEvents: () => usageEvents,
|
|
5763
5764
|
verificationCache: () => verificationCache,
|
|
5764
5765
|
workflowPromptOverrides: () => workflowPromptOverrides,
|
|
@@ -5798,6 +5799,7 @@ var tasks = projectSchema.table("tasks", {
|
|
|
5798
5799
|
worktree: text("worktree"),
|
|
5799
5800
|
blockedBy: text("blocked_by"),
|
|
5800
5801
|
overlapBlockedBy: text("overlap_blocked_by"),
|
|
5802
|
+
queuedLogEpisodeSignature: text("queued_log_episode_signature"),
|
|
5801
5803
|
paused: integer("paused").default(0),
|
|
5802
5804
|
userPaused: integer("user_paused").default(0),
|
|
5803
5805
|
pausedReason: text("paused_reason"),
|
|
@@ -6603,6 +6605,14 @@ var workflowWorkItems = projectSchema.table("workflow_work_items", {
|
|
|
6603
6605
|
index("idx_workflow_work_items_task_run").on(t.taskId, t.runId),
|
|
6604
6606
|
uniqueIndex("idx_workflow_work_items_one_active_task_continuation").on(t.projectId, t.taskId).where(sql`${t.kind} = 'task' AND ${t.state} IN ('runnable', 'running', 'held', 'retrying')`)
|
|
6605
6607
|
]);
|
|
6608
|
+
var unplannedExecutionBlocks = projectSchema.table("unplanned_execution_blocks", {
|
|
6609
|
+
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6610
|
+
taskId: text("task_id").notNull(),
|
|
6611
|
+
episode: text("episode").notNull(),
|
|
6612
|
+
createdAt: text("created_at").notNull()
|
|
6613
|
+
}, (t) => [
|
|
6614
|
+
primaryKey({ columns: [t.projectId, t.taskId, t.episode] })
|
|
6615
|
+
]);
|
|
6606
6616
|
var workflowRunBranches = projectSchema.table("workflow_run_branches", {
|
|
6607
6617
|
projectId: text("project_id").notNull().default(sql`current_setting('fusion.project_id', true)`),
|
|
6608
6618
|
taskId: text("task_id").notNull(),
|
|
@@ -12128,6 +12138,37 @@ var TRIAGE_POLICY_DEFAULTS = new Map(
|
|
|
12128
12138
|
BUILTIN_TRIAGE_POLICY_SETTINGS.map((setting) => [setting.id, setting.default])
|
|
12129
12139
|
);
|
|
12130
12140
|
|
|
12141
|
+
// ../core/src/agents/planning-review-policy.ts
|
|
12142
|
+
var PLANNING_COMPLETENESS_POLICY = `## Mandatory Planning Completeness Procedure
|
|
12143
|
+
|
|
12144
|
+
Before writing the final PROMPT.md:
|
|
12145
|
+
|
|
12146
|
+
1. Build an internal **planning ledger** from the Original Description, user comments, project instructions, cited issue/report, and any saved planning document. Preserve every requirement, settled decision, explicit non-goal, and acceptance outcome; do not silently replace the reporter's contract with a narrower reproduction.
|
|
12147
|
+
2. Research before structuring the plan. Trace the affected behavior through its real production entry points, callers, writers/readers, shared consumers, persistence boundaries, recovery paths, and existing tests. Resolve planning-time facts from the repository now; defer only details that genuinely require implementation-time discovery, and label those explicitly.
|
|
12148
|
+
3. For stateful, lifecycle, persistence, or concurrency work, enumerate the participant graph and both relevant orderings: stale-work cancellation/fencing, lock order and reentrancy, transaction boundaries, pool/transport limits, competing recovery paths, configuration/deployment identities, bypass/force paths, and failure cleanup. Every stated invariant must name the surfaces that preserve it.
|
|
12149
|
+
4. Map every ledger row to concrete File Scope entries, dependency-ordered implementation steps, and verification. Each feature-bearing step must name specific automated test scenarios with the input/state, action or ordering, and expected observable result; helper-only tests do not prove production reachability.
|
|
12150
|
+
5. Keep scope disciplined. Reuse current architecture and documented patterns unless the reporter contract requires a redesign. Put optional cleanup and adjacent ideas outside the active steps.
|
|
12151
|
+
6. Before persisting PROMPT.md, perform a fresh holistic completeness pass across the whole ledger. Try to disprove the plan by checking referenced paths and patterns, missing consumers, contradictory steps, uncovered failure orderings, and acceptance criteria without tests. Fix all gaps in one pass.
|
|
12152
|
+
|
|
12153
|
+
On a revision, treat the cumulative revision ledger as durable decisions unless a later entry explicitly supersedes one: preserve resolved items, address every unresolved item surgically, and rerun the complete procedure instead of checking only the latest comment.`;
|
|
12154
|
+
var FAST_PLANNING_COMPLETENESS_POLICY = `## Fast Planning Completeness Check
|
|
12155
|
+
|
|
12156
|
+
Before writing PROMPT.md, build a compact internal ledger from the Original Description, user comments, project rules, and acceptance outcomes. Inspect the real production callers/consumers, persistence or recovery paths, and nearby tests before choosing File Scope. For stateful or concurrent work, enumerate every participant plus both relevant orderings and failure cleanup. Map each requirement to a concrete step, file, and automated test scenario, then reread the whole plan once for missing surfaces, contradictions, and unproved outcomes. On revision, preserve prior decisions and address the complete cumulative ledger, not only the latest note.`;
|
|
12157
|
+
var PLAN_REVIEW_COMPLETENESS_POLICY = `## Mandatory Plan Review Procedure
|
|
12158
|
+
|
|
12159
|
+
Apply this procedure to plan/spec reviews only. Code reviews use their dedicated procedure.
|
|
12160
|
+
|
|
12161
|
+
Before choosing a verdict:
|
|
12162
|
+
|
|
12163
|
+
1. Build one **review ledger** for the entire PROMPT.md: Original Description and user comments; Mission and Completion Criteria; Surface Enumeration and Symptom Verification when required; every implementation step, File Scope entry, dependency, risk, and test/verification promise.
|
|
12164
|
+
2. Complete the full review before reporting. Check coherence and requirement traceability, feasibility against the current repository, scope discipline, execution ordering, and verification quality. When relevant, also inspect security/data integrity, state transitions, concurrency orderings, deployment/configuration boundaries, recovery competitors, and force/bypass paths.
|
|
12165
|
+
3. Review at specification altitude. Block when a required behavior, surface, ordering, safety constraint, or proof is missing or the stated approach cannot work. Keep optional implementation detail, wording polish, and nonessential improvements advisory.
|
|
12166
|
+
4. If REVISE is necessary, batch **all independently discoverable blocking findings** into this one verdict; do not stop after the first defect. Give each blocker a stable ID, cite the affected section or repository evidence, and state the concrete PROMPT.md correction. Put advisory observations in a separate list.
|
|
12167
|
+
5. On re-review, use the supplied prior-review ledger as a decision primer. Verify every prior blocker, do not re-raise resolved or rejected semantic duplicates, and preserve accepted decisions. A newly blocking finding must say whether the revision introduced it, which prior blocker genuinely masked it, or why it is independently delivery-blocking for correctness, security, data safety, or executability. Record an earlier reviewer miss explicitly; never demote a critical defect merely because it was missed before.
|
|
12168
|
+
6. After any same-session PROMPT.md edit, distrust the edit: reread the complete artifact, rebuild the ledger, and perform a fresh holistic pass before APPROVE.
|
|
12169
|
+
|
|
12170
|
+
APPROVE when the plan is executable and verifiable, not when it is cosmetically perfect. If returning REVISE, the verdict notes must contain the complete blocking checklist because those notes are the durable input to the next planning round.`;
|
|
12171
|
+
|
|
12131
12172
|
// ../core/src/agents/agent-prompts.ts
|
|
12132
12173
|
var FUSION_RUNTIME_SELF_AWARENESS = `## Runtime Self-Awareness
|
|
12133
12174
|
|
|
@@ -12334,6 +12375,8 @@ The tool prevents your session from being killed by the inactivity watchdog duri
|
|
|
12334
12375
|
- If a verification command times out, do NOT blindly retry \u2014 investigate. Check for hung subprocesses, infinite test loops, or tests waiting on missing dependencies. Use \`node_modules/.modules.yaml\` presence to confirm bootstrap.`;
|
|
12335
12376
|
var FAST_TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn". This task is running in **fast mode**.
|
|
12336
12377
|
|
|
12378
|
+
${FAST_PLANNING_COMPLETENESS_POLICY}
|
|
12379
|
+
|
|
12337
12380
|
Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heavyweight ceremony, review scoring, and proactive subtask analysis.
|
|
12338
12381
|
|
|
12339
12382
|
## Fast-mode priorities
|
|
@@ -12342,7 +12385,7 @@ Write a lean, executable PROMPT.md quickly. Preserve safety gates, but skip heav
|
|
|
12342
12385
|
- Preserve required safety sections for bugs, workflow routing, forensic tasks, and decision-only work.
|
|
12343
12386
|
|
|
12344
12387
|
## Duplicate check
|
|
12345
|
-
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols.
|
|
12388
|
+
Before writing a spec, call \`fn_task_list\` for active work, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 targeted keyword phrases from the title/description, such as file paths, symptoms, and symbols. Do not search completed or archived work for duplicate candidates. When an active match is a duplicate, do not write a spec \u2014 but still write PROMPT.md, with its entire contents being the single line \`DUPLICATE: {existing-task-id}\` and nothing else. That file is how the duplicate is recorded; announcing it only in your reply leaves no plan behind and re-plans the task in a loop.
|
|
12346
12389
|
|
|
12347
12390
|
## Required PROMPT.md shape
|
|
12348
12391
|
Write PROMPT.md with Original Description, Before \u2192 After Transformation, Mission, Dependencies, Context to Read First, File Scope, Steps, Documentation Requirements, Completion Criteria, Git Commit Convention, and Do NOT. Put \`## Original Description\` immediately after the title/\`Created\`/\`Size\` metadata with the operator's original task description copied **verbatim** (do not paraphrase). Put \`## Before \u2192 After Transformation\` next, before \`## Mission\`, with concise Before/After bullets: current state, target state, why it satisfies the user's request at a glance. In \`## Steps\`, every executable heading MUST use \`### Step N: <name>\` (e.g. \`### Step 1: Preflight\`). Do not write bare \`### Preflight\` / \`### Implementation\` headings, and do not add review-level, triage subtask, or proactive subtask headings.
|
|
@@ -12384,6 +12427,8 @@ If the requested outcome is only to decide, route, or coordinate work, include \
|
|
|
12384
12427
|
Write PROMPT.md directly and stop. Do not call \`fn_review_spec()\`; workflow Plan Review is the single optional plan review gate before execution.`;
|
|
12385
12428
|
var TRIAGE_PROMPT_TEXT = `You are a task specification agent for "fn", an AI-orchestrated task board.
|
|
12386
12429
|
|
|
12430
|
+
${PLANNING_COMPLETENESS_POLICY}
|
|
12431
|
+
|
|
12387
12432
|
## Your Role
|
|
12388
12433
|
You are the specification quality gate for implementation success.
|
|
12389
12434
|
Your job: take a rough task description and produce a fully specified PROMPT.md that another AI agent can execute autonomously in a fresh context with zero memory of this conversation.
|
|
@@ -12608,9 +12653,9 @@ Verified facts about this codebase's storage \u2014 cite these correctly so Plan
|
|
|
12608
12653
|
- New Postgres migrations must be **registered explicitly** in \`packages/core/src/postgres/schema-applier.ts\` (version constant + bookkeeping check); a \`.sql\` file dropped in the migrations dir that is not wired there silently never runs.
|
|
12609
12654
|
|
|
12610
12655
|
## Duplicate check
|
|
12611
|
-
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12612
|
-
|
|
12613
|
-
If
|
|
12656
|
+
Before writing a spec, first call \`fn_task_list\` to see active tasks, then call \`fn_task_search\` with \`includeDone: false\` and \`includeArchived: false\` for 2-4 distinct keyword phrases from the task title and description (for example file paths, error symptoms, and symbol names).
|
|
12657
|
+
Do not search completed or archived work for duplicate candidates.
|
|
12658
|
+
If an actionable task already covers the same work (even if worded differently), do not write a spec.
|
|
12614
12659
|
Instead you MUST still write PROMPT.md, with its ENTIRE contents being this one line and nothing else:
|
|
12615
12660
|
\`DUPLICATE: {existing-task-id}\`
|
|
12616
12661
|
Writing that file IS how you report the duplicate. Reporting it only in your reply is not recorded:
|
|
@@ -12641,7 +12686,7 @@ Workflow policy can disable proactive oversized-task splitting for operators who
|
|
|
12641
12686
|
## Triage tools
|
|
12642
12687
|
You have these extra tools during triage:
|
|
12643
12688
|
- \`fn_task_list\` \u2014 list existing active tasks
|
|
12644
|
-
- \`fn_task_search\` \u2014 keyword search across tasks
|
|
12689
|
+
- \`fn_task_search\` \u2014 keyword search across active tasks by default; history is opt-in for non-duplicate research
|
|
12645
12690
|
- \`fn_task_show\` \u2014 inspect a task and its PROMPT.md
|
|
12646
12691
|
- \`fn_task_create\` \u2014 create a child/follow-up task while triaging
|
|
12647
12692
|
- \`fn_task_document_write\` \u2014 save a planning document (e.g., key="plan")
|
|
@@ -12868,20 +12913,6 @@ Concrete examples:
|
|
|
12868
12913
|
- [Optional improvements, not blocking]
|
|
12869
12914
|
\`\`\`
|
|
12870
12915
|
|
|
12871
|
-
## Spec / Plan Review Convergence
|
|
12872
|
-
|
|
12873
|
-
Specs and pre-execution Plan Review share this gate. Prefer **APPROVE** / **APPROVE_WITH_NOTES** when the plan is executable enough for an agent to implement. Put optional polish only under **Suggestions**.
|
|
12874
|
-
|
|
12875
|
-
When you must **REVISE**:
|
|
12876
|
-
- List each blocking issue as a concrete PROMPT.md edit (which section, what to add/change/remove).
|
|
12877
|
-
- Do not demand a full rewrite unless the approach is fundamentally wrong (**RETHINK**).
|
|
12878
|
-
- Prefer fixing local PROMPT.md defects in-session when you have write tools, then **APPROVE**, instead of bouncing the task through another full replan cycle.
|
|
12879
|
-
|
|
12880
|
-
**Converging on re-review (when the request includes your prior feedback + a Plan Review attempt number):**
|
|
12881
|
-
- This is a spec you already reviewed. VERIFY each issue you previously raised was addressed. REVISE only for (a) a PRIOR blocking issue still unresolved, or (b) a genuinely NEW problem THIS revision introduced.
|
|
12882
|
-
- Do NOT introduce a new blocking issue that ALSO applied to the version you previously reviewed \u2014 that is your own earlier miss. Record it under **Suggestions**, not REVISE.
|
|
12883
|
-
- **Severity ratchet at attempt 3+:** gate ONLY on \`critical\` (delivery-blocking) issues. Downgrade lone \`important\`/\`minor\` spec-wording nits to **Suggestions** and APPROVE. Rationale: the executor and downstream code review are later gates \u2014 a spec need not be perfect to be executable, only executable.
|
|
12884
|
-
|
|
12885
12916
|
## Spec Review \u2014 Undersplit Task Detection
|
|
12886
12917
|
|
|
12887
12918
|
When reviewing specs, assess whether the task should have been broken into subtasks. The bar for splitting is high \u2014 most tasks should remain whole. Coordination overhead (worktrees, dependency wiring, merge sequencing) is real, so splitting must clearly pay for itself.
|
|
@@ -13529,6 +13560,22 @@ function browserVerificationOptionalGroupNode(column, options = {}) {
|
|
|
13529
13560
|
};
|
|
13530
13561
|
}
|
|
13531
13562
|
|
|
13563
|
+
// ../core/src/agents/code-review-policy.ts
|
|
13564
|
+
var CODE_REVIEW_COMPLETENESS_POLICY = `## Mandatory Code Review Procedure
|
|
13565
|
+
|
|
13566
|
+
Apply this procedure to code reviews only. Plan/spec reviews use their dedicated criteria below.
|
|
13567
|
+
|
|
13568
|
+
Before choosing a verdict:
|
|
13569
|
+
|
|
13570
|
+
1. Build a **requirements ledger** from PROMPT.md. For every Mission outcome, Completion Criterion, relevant \`## Surface Enumeration\` item, and \`## Symptom Verification\` assertion, identify both implementation evidence and test evidence. A missing or partial required row is REVISE.
|
|
13571
|
+
2. Trace each changed behavior from its **real production entry point** and selector through the changed helper to its consumers. A helper-only unit test does not prove that production can reach the new branch.
|
|
13572
|
+
3. For bug fixes, locate an automated test that reproduces the exact reported failure and asserts it is gone across the enumerated surfaces. Green builds, nearby unit tests, and synthetic state injection alone are insufficient.
|
|
13573
|
+
4. For persistence, lifecycle, and concurrency changes, verify that evidence belongs to the **current state, version, or planning episode**; every read-check-write path is atomic, locked, or CAS-protected; all mutation surfaces preserve the invariant; waits and locks are bounded and cleanup-safe; and a production-shaped integration test covers both relevant orderings.
|
|
13574
|
+
5. For route or state changes, inspect all affected UI, API, CLI, and agent consumers plus paired success/failure or approve/reject paths for contract parity.
|
|
13575
|
+
6. Before APPROVE, perform a fresh adversarial validation pass: try to disprove each completed ledger row by reading the real caller, guard, consumer, and test. Cite \`file:line\` for every blocker and name the exact missing proof.
|
|
13576
|
+
|
|
13577
|
+
APPROVE only when every required ledger row has implementation and test evidence.`;
|
|
13578
|
+
|
|
13532
13579
|
// ../core/src/workflows/builtin-code-review-group.ts
|
|
13533
13580
|
var CODE_REVIEW_GROUP_ID = "code-review";
|
|
13534
13581
|
var CODE_REVIEW_STEP_NODE_ID = "code-review-step";
|
|
@@ -13548,6 +13595,8 @@ var CODE_REVIEW_PROMPT = `You are a senior code reviewer. Review the task's diff
|
|
|
13548
13595
|
5. **Error handling** \u2014 swallowed errors, unhandled rejections/exceptions, missing validation at trust boundaries, misleading error messages.
|
|
13549
13596
|
6. **Contract / signature changes** \u2014 changed function/exported-type signatures, API request/response shapes, or serialization that breaks existing callers.
|
|
13550
13597
|
|
|
13598
|
+
${CODE_REVIEW_COMPLETENESS_POLICY}
|
|
13599
|
+
|
|
13551
13600
|
Be specific: cite \`file:line\` for every finding and explain the concrete failure it causes.
|
|
13552
13601
|
|
|
13553
13602
|
## Output Requirements
|
|
@@ -13667,12 +13716,14 @@ var PLAN_REVIEW_PROMPT = `You are a senior plan reviewer. Review the task's PROM
|
|
|
13667
13716
|
4. **Verification quality** \u2014 absent or weak tests/checks for the behavior being changed.
|
|
13668
13717
|
5. **Risk callouts** \u2014 migrations, data-loss paths, external integrations, secrets, or plugin/runtime dependencies that need explicit handling.
|
|
13669
13718
|
|
|
13719
|
+
${PLAN_REVIEW_COMPLETENESS_POLICY}
|
|
13720
|
+
|
|
13670
13721
|
Be specific: cite the plan section or file path for every finding and explain the concrete correction.
|
|
13671
13722
|
|
|
13672
13723
|
## Output Requirements
|
|
13673
13724
|
- APPROVE: the plan is ready for execution.
|
|
13674
13725
|
- APPROVE_WITH_NOTES: execution may proceed, but include non-blocking advisory notes.
|
|
13675
|
-
- REVISE: the plan should be corrected before execution; include
|
|
13726
|
+
- REVISE: the plan should be corrected before execution; include every blocking finding and needed change in the JSON notes, not only in preceding prose.
|
|
13676
13727
|
- Final output: output exactly one trailing JSON object on the final line (no markdown fences, no surrounding prose):
|
|
13677
13728
|
{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"..."}`;
|
|
13678
13729
|
function planReviewOptionalGroupNode(column, options = {}) {
|