@alexeiled/pi-fusion 0.5.2 → 0.6.1

package/src/orchestrator.ts

@@ -1,4 +1,6 @@
+import { applyClaudeAliasShorthand } from "./claude-aliases.js";
 import {
+  buildInlinePanelProfile,
   loadFusionConfig,
   resolveProfile as resolveFusionProfile,
   type ResolvedFusionProfile,
@@ -30,12 +32,14 @@ import {
   readSubagentResultArtifact,
   readSubagentStatusArtifact,
 } from "./subagent-artifacts.js";
-import type {
-  FailedPanelSummary,
-  FusionProfile,
-  FusionRun,
-  PanelOutput,
-  ParsedFusionArgs,
+import {
+  memberLabel,
+  resolveSynthesisMode,
+  type FailedPanelSummary,
+  type FusionProfile,
+  type FusionRun,
+  type PanelOutput,
+  type ParsedFusionArgs,
 } from "./types.js";
 import {
   extractRunObservation,
@@ -47,6 +51,7 @@ import type { SubagentsTargetParams } from "./subagents-rpc.js";
 export const SUBAGENT_ASYNC_COMPLETE_EVENT = "subagent:async-complete";
 
 const RECONCILE_INTERVAL_MS = 2_000;
+const WORKFLOW_RESULT_ARTIFACT_GRACE_MS = 5_000;
 
 export type FusionNotifyType = "info" | "warning" | "error";
 
@@ -101,6 +106,7 @@ interface RunLifecycleSnapshot {
   statusPayload?: unknown;
   resultPayload?: unknown;
   resultIsTerminal: boolean;
+  resultArtifactPending?: boolean;
 }
 
 export class FusionOrchestrator {
@@ -150,9 +156,30 @@ export class FusionOrchestrator {
     }
 
     let resolved: ResolvedFusionProfile;
+    let baseProfileName: string | undefined;
     try {
       const config = await this.loadConfig(ctx);
       resolved = this.resolveProfile(config, args.profile);
+      baseProfileName = resolved.name;
+      if (args.panel?.length) {
+        // The named profile still supplies the judge and every other setting;
+        // only the panel is replaced. Inline models skip the alias pass that
+        // runs at config load, so re-run it over the assembled profile.
+        const inlineName = `${resolved.name} (inline panel)`;
+        const aliased = await applyClaudeAliasShorthand(
+          {
+            defaultProfile: inlineName,
+            profiles: {
+              [inlineName]: buildInlinePanelProfile(
+                resolved.profile,
+                args.panel,
+              ),
+            },
+          },
+          ctx,
+        );
+        resolved = this.resolveProfile(aliased, inlineName);
+      }
       this.configWarning = undefined;
     } catch (error: unknown) {
       const message = errorMessage(error);
@@ -166,6 +193,9 @@ export class FusionOrchestrator {
       run = this.runStore.startRun({
         prompt: args.prompt,
         profileName: resolved.name,
+        ...(args.panel?.length
+          ? { inlinePanel: args.panel, baseProfileName: baseProfileName }
+          : {}),
         ...(args.operationId !== undefined
           ? { operationId: args.operationId }
           : {}),
@@ -350,6 +380,12 @@ export class FusionOrchestrator {
           ? configuredJudgeModel(this.activeProfile)
           : undefined,
       ),
+      ...(this.activeProfile
+        ? {
+            synthesis: resolveSynthesisMode(this.activeProfile),
+            panel: this.activeProfile.panel,
+          }
+        : {}),
     });
     const cancelled = this.runStore.cancelRun(active.id, {
       ...(active.chainRunId ? { chainRunId: active.chainRunId } : {}),
@@ -381,10 +417,19 @@ export class FusionOrchestrator {
 
     try {
       const config = await this.loadConfig(ctx);
-      this.activeProfile = this.resolveProfile(
+      // An inline run's profileName is a display name that no config defines,
+      // so rebuild it from the base profile plus the persisted entries. Looking
+      // the display name up throws, which used to leave activeProfile undefined
+      // and fail the run the moment its panel completed.
+      const base = this.resolveProfile(
         config,
-        active.profileName,
+        active.inlinePanel?.length
+          ? active.baseProfileName
+          : active.profileName,
       ).profile;
+      this.activeProfile = active.inlinePanel?.length
+        ? buildInlinePanelProfile(base, active.inlinePanel)
+        : base;
       this.configWarning = undefined;
     } catch (error: unknown) {
       const message = `Could not restore fusion profile "${active.profileName}": ${errorMessage(error)}`;
@@ -513,6 +558,7 @@ export class FusionOrchestrator {
       ...(active.chainAsyncDir ? { asyncDir: active.chainAsyncDir } : {}),
       eventPayload: payload,
     });
+    if (snapshot.resultArtifactPending) return { status: "ignored" };
     const terminalPayload =
       snapshot.resultPayload ?? snapshot.statusPayload ?? payload;
     if (
@@ -576,6 +622,8 @@ export class FusionOrchestrator {
         failures: storedPanelFailures(observed),
         ...withJudgeModel(configuredJudgeModel(profile)),
         judgeObservation,
+        ...(profile.blindPanelLabels ? { blindPanelLabels: true } : {}),
+        synthesis: resolveSynthesisMode(profile),
       });
       return this.completeActiveRun(report);
     }
@@ -610,8 +658,12 @@ export class FusionOrchestrator {
           : {}),
       eventPayload: payload,
     });
+    if (snapshot.resultArtifactPending) return { status: "ignored" };
 
-    if (!active.panelStopReason) {
+    const panelIsTerminal =
+      snapshot.resultIsTerminal ||
+      isTerminalSubagentState(extractSubagentState(snapshot.statusPayload));
+    if (!active.panelStopReason && !panelIsTerminal) {
       const partial = extractPanelResults(snapshot.statusPayload, {
         panel: profile.panel,
         completedOnly: true,
@@ -644,12 +696,16 @@ export class FusionOrchestrator {
       return this.failActiveRun(lifecycleError);
     }
 
+    const workflowStoppedIndices = extractWorkflowStoppedPanelIndices(
+      lifecyclePayload,
+    );
+    const stoppedPanelIndices =
+      active.panelStoppedIndices ??
+      (workflowStoppedIndices.length > 0 ? workflowStoppedIndices : undefined);
     const extracted = extractPanelResults(lifecyclePayload, {
       panel: profile.panel,
       limit: profile.panel.length,
-      ...(active.panelStoppedIndices
-        ? { stoppedPanelIndices: active.panelStoppedIndices }
-        : {}),
+      ...(stoppedPanelIndices ? { stoppedPanelIndices } : {}),
     });
     if (!extracted.ok) {
       return this.failActiveRun(
@@ -662,11 +718,18 @@ export class FusionOrchestrator {
       snapshot.statusPayload,
       profile,
     );
-    const updated = this.storePanelResults(
+    const stored = this.storePanelResults(
       active.id,
       observedPanels.outputs,
       observedPanels.failures,
     );
+    const updated =
+      workflowStoppedIndices.length > 0 && !active.panelStopReason
+        ? this.runStore.updateRun(stored.id, {
+            panelStopReason: "agreement",
+            panelStoppedIndices: workflowStoppedIndices,
+          })
+        : stored;
 
     return this.finishPanelCompletion(
       updated,
@@ -795,6 +858,7 @@ export class FusionOrchestrator {
       ...(active.judgeAsyncDir ? { asyncDir: active.judgeAsyncDir } : {}),
       eventPayload: payload,
     });
+    if (snapshot.resultArtifactPending) return { status: "ignored" };
     const terminalPayload =
       snapshot.resultPayload ?? snapshot.statusPayload ?? payload;
     if (
@@ -814,9 +878,17 @@ export class FusionOrchestrator {
     const output = extractJudgeOutput(lifecyclePayload);
     if (!output.ok) return this.failActiveRun(output.error);
 
-    const judgeModel = this.activeProfile
-      ? configuredJudgeModel(this.activeProfile)
-      : undefined;
+    // The panel and chain handlers already treat a missing profile as fatal.
+    // Without the same guard here the run "succeeds" with a degraded report:
+    // blind labels are never restored, and the judge model is dropped.
+    const profile = this.activeProfile;
+    if (!profile) {
+      return this.failActiveRun(
+        "Fusion judge completed, but the active profile was not available.",
+      );
+    }
+
+    const judgeModel = configuredJudgeModel(profile);
     const judgeObservation = mergeRunObservations(
       extractRunObservation(
         findStepsArray(snapshot.statusPayload)[0] ?? snapshot.statusPayload,
@@ -837,6 +909,8 @@ export class FusionOrchestrator {
       failures: storedPanelFailures(observed),
       ...withJudgeModel(judgeModel),
       judgeObservation,
+      ...(profile.blindPanelLabels ? { blindPanelLabels: true } : {}),
+      synthesis: resolveSynthesisMode(profile),
     });
     return this.completeActiveRun(report);
   }
@@ -897,6 +971,17 @@ export class FusionOrchestrator {
       };
     }
 
+    if (
+      statusIsTerminal &&
+      isWorkflowResultArtifactPending(statusPayload)
+    ) {
+      return {
+        statusPayload,
+        resultIsTerminal: false,
+        resultArtifactPending: true,
+      };
+    }
+
     if (hasResultsArray(statusPayload)) {
       return {
         statusPayload,
@@ -989,6 +1074,12 @@ export class FusionOrchestrator {
           ? configuredJudgeModel(this.activeProfile)
           : undefined,
       ),
+      ...(this.activeProfile
+        ? {
+            synthesis: resolveSynthesisMode(this.activeProfile),
+            panel: this.activeProfile.panel,
+          }
+        : {}),
     });
   }
 
@@ -1269,7 +1360,7 @@ function buildFusionStatusDetails(
     const activity = describeStepActivity(step);
     const metrics = describeStepMetrics(step);
     return {
-      label: member.label,
+      label: memberLabel(member),
       ...(member.role ? { role: member.role } : {}),
       ...(model ? { model } : {}),
       status: describePanelStatus(step),
@@ -1334,7 +1425,7 @@ function buildCompletedPanelStatusLines(
     );
     if (output) {
       return {
-        label: member.label,
+        label: memberLabel(member),
         ...(member.role ? { role: member.role } : {}),
         ...(model ? { model } : {}),
         status: "completed",
@@ -1344,7 +1435,7 @@ function buildCompletedPanelStatusLines(
       (item) => item.id === member.id || item.index === index,
     );
     return {
-      label: member.label,
+      label: memberLabel(member),
       ...(member.role ? { role: member.role } : {}),
       ...(model ? { model } : {}),
       status: failure ? "failed" : "unknown",
@@ -1547,10 +1638,46 @@ function isTerminalSubagentState(state: string | undefined): boolean {
   );
 }
 
+function isWorkflowResultArtifactPending(payload: unknown): boolean {
+  if (!isRecord(payload)) return false;
+  const details = isRecord(payload.details) ? payload.details : undefined;
+  const mode = firstString(payload.mode, details?.mode);
+  const endedAtValue = payload.endedAt ?? details?.endedAt;
+  if (mode === "workflow" && typeof endedAtValue === "number") {
+    return Date.now() - endedAtValue < WORKFLOW_RESULT_ARTIFACT_GRACE_MS;
+  }
+  return isRecord(payload.data)
+    ? isWorkflowResultArtifactPending(payload.data)
+    : false;
+}
+
 function hasLifecycleResults(payload: unknown): boolean {
   return hasResultsArray(payload) || findStepsArray(payload).length > 0;
 }
 
+function extractWorkflowStoppedPanelIndices(payload: unknown): number[] {
+  if (!isRecord(payload)) return [];
+  const emits = isRecord(payload.workflow)
+    ? unknownArray(payload.workflow.emits)
+    : undefined;
+  const indices = new Set<number>();
+  for (const emit of emits ?? []) {
+    if (!isRecord(emit) || emit.type !== "pi-fusion-panel-stop") continue;
+    for (const index of unknownArray(emit.indices) ?? []) {
+      if (typeof index === "number" && Number.isInteger(index) && index >= 0) {
+        indices.add(index);
+      }
+    }
+  }
+  if (indices.size > 0) return [...indices].sort((left, right) => left - right);
+  if (isRecord(payload.details)) {
+    const nested = extractWorkflowStoppedPanelIndices(payload.details);
+    if (nested.length > 0) return nested;
+  }
+  if (isRecord(payload.data)) return extractWorkflowStoppedPanelIndices(payload.data);
+  return [];
+}
+
 function extractSubagentFailure(payload: unknown): string | undefined {
   if (!isRecord(payload)) return undefined;
   const direct = firstNonBlankString(payload.error, payload.errorMessage);

package/src/panel-completion.ts

@@ -4,11 +4,12 @@ import {
   buildJudgeSpawnParams,
   type JudgeSpawnParams,
 } from "./run-builder.js";
-import type {
-  FailedPanelSummary,
-  FusionProfile,
-  FusionRun,
-  PanelOutput,
+import {
+  resolveSynthesisMode,
+  type FailedPanelSummary,
+  type FusionProfile,
+  type FusionRun,
+  type PanelOutput,
 } from "./types.js";
 
 export type PanelCompletionDecision =
@@ -39,6 +40,8 @@ export function decidePanelCompletion(
       run: input.run,
       failures: input.panelFailures,
       ...withJudgeModel(judgeModel),
+      synthesis: resolveSynthesisMode(input.profile),
+      panel: input.profile.panel,
     });
     return {
       kind: "fail",
@@ -47,7 +50,14 @@ export function decidePanelCompletion(
     };
   }
 
-  if (input.panelOutputs.length === 1) {
+  // Under `select` every panelist answered the whole question, so a lone
+  // survivor is a complete if thin answer. Under `merge` it answered ONE facet:
+  // returning it as the answer would be wrong, not thin. Run the composer so
+  // the report names the facets nobody covered.
+  if (
+    input.panelOutputs.length === 1 &&
+    resolveSynthesisMode(input.profile) !== "merge"
+  ) {
     const report = renderSinglePanelReport({
       run: input.run,
       output: input.panelOutputs[0]!,
@@ -64,6 +74,7 @@ export function decidePanelCompletion(
       prompt: input.run.prompt,
       panelOutputs: input.panelOutputs,
       failedPanelists: input.panelFailures,
+      runId: input.run.id,
     }),
     missingRunIdError: input.fallbackJudge
       ? "pi-subagents spawn did not return a fallback judge run ID."