@emiliosp/pi-maestro 0.6.1 → 0.6.3

package/README.md

@@ -70,11 +70,14 @@ The owner follows this workflow:
 5. Reviews verifier findings and chooses an action for every finding.
 6. Reads Maestro's summary at `candidate-ready` and performs the final review.
 
-Maestro starts the verifier after a successful builder run. Both agents run in the foreground: Pi waits for each run to finish. Maestro shows the current phase in Pi's status. `pi-subagents` FleetView and `/subagents-fleet` show agent activity and transcripts.
+Maestro starts the verifier after a successful builder run. 
+Both agents run in the foreground: Pi waits for each run to finish and Maestro shows the current phase in Pi's status, while `pi-subagents` FleetView and `/subagents-fleet` show agent activity and transcripts.
 
-The owner must not edit product files while the workflow runs. Maestro can perform temporary experiments with owner agreement during spec preparation and permitted revisions. See [Workflow](docs/workflow.md#spec-approval).
+The owner must not edit product files while the workflow runs. 
+Maestro can perform temporary experiments with owner agreement during spec preparation and permitted revisions. See [Workflow](docs/workflow.md#spec-approval).
 
-If a contract change is needed during an escalation or finding decision, the owner reviews the revised spec and replies `GREEN FLAG` again. Maestro then starts another builder run. A recorded builder failure stops the workflow and requires manual owner follow-up.
+An escalation asks the owner to decide an implementation question, and a finding records a technical issue reported by the verifier. The owner discusses both with Maestro, never directly with the builder or verifier.
+If a contract change is needed during an escalation or finding decision, the owner reviews the revised spec and replies `GREEN FLAG` again. Maestro then starts another builder run.
 
 The workflow ends at `candidate-ready`. Rejected findings retain their reasons. Later changes are outside the completed verification. The owner controls any later Git use, pull request, and merge.
 

package/extensions/maestro.ts

@@ -4,6 +4,7 @@
  */
 
 import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
+import { loadConfiguration } from '#config/loadConfiguration.ts';
 import { assertEnvironment } from '#maestro/checks/assertEnvironment.ts';
 import { getMaestroInstructions } from '#maestro/instructions/getMaestroInstructions.ts';
 import maestroSessionState from '#maestro/session/MaestroSessionState.ts';
@@ -83,6 +84,27 @@ export default (pi: ExtensionAPI): void => {
 
       try {
         await assertEnvironment(context);
+        const config = await loadConfiguration(context.cwd);
+
+        maestroSessionState.activate();
+        syncMainTools();
+        await refreshMaestroStatus(context);
+
+        // show builder/verifier model on activation
+        context.ui.notify(
+          [
+            context.ui.theme.fg('muted', 'Maestro active'),
+            context.ui.theme.fg(
+              'accent',
+              `Builder (${config.builder.model} ${config.builder.thinking})`,
+            ),
+            context.ui.theme.fg(
+              'accent',
+              `Verifier (${config.verifier.model} ${config.verifier.thinking})`,
+            ),
+          ].join('\n'),
+          'info',
+        );
       } catch (error) {
         context.ui.notify(
           error instanceof Error ? error.message : String(error),
@@ -91,10 +113,6 @@ export default (pi: ExtensionAPI): void => {
 
         return;
       }
-
-      maestroSessionState.activate();
-      syncMainTools();
-      await refreshMaestroStatus(context);
     },
   });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@emiliosp/pi-maestro",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "A spec-driven multiagent development workflow for Pi.",
   "license": "MIT",
   "type": "module",

package/src/maestro/status/formatMaestroStatus.ts

@@ -3,9 +3,14 @@
  * Used: When the main extension refreshes Maestro status.
  */
 
-import { WORKFLOW_PHASES, type WorkflowState } from '#workflow/state/schema.ts';
+import type { Theme, ThemeColor } from '@earendil-works/pi-coding-agent';
+import {
+  WORKFLOW_PHASES,
+  type WorkflowPhase,
+  type WorkflowState,
+} from '#workflow/state/schema.ts';
 
-const WORKFLOW_PHASE_LABELS = {
+export const WORKFLOW_PHASE_LABELS = {
   [WORKFLOW_PHASES.DRAFTING_SPEC]: 'Preparing specification',
   [WORKFLOW_PHASES.READY_FOR_BUILDER]: 'Ready for builder',
   [WORKFLOW_PHASES.BUILDER_RUNNING]: 'Builder running',
@@ -17,6 +22,40 @@ const WORKFLOW_PHASE_LABELS = {
   [WORKFLOW_PHASES.CANDIDATE_READY]: 'Completed',
 } as const;
 
+export const WORKFLOW_PHASE_ICONS = {
+  [WORKFLOW_PHASES.DRAFTING_SPEC]: '📝',
+  [WORKFLOW_PHASES.READY_FOR_BUILDER]: '🚧',
+  [WORKFLOW_PHASES.BUILDER_RUNNING]: '🛠️',
+  [WORKFLOW_PHASES.ESCALATION_DECISION]: '✋',
+  [WORKFLOW_PHASES.BUILDER_FAILED]: '❌',
+  [WORKFLOW_PHASES.READY_FOR_VERIFIER]: '📋',
+  [WORKFLOW_PHASES.VERIFIER_RUNNING]: '🔍',
+  [WORKFLOW_PHASES.FINDINGS_DECISION]: '💬',
+  [WORKFLOW_PHASES.CANDIDATE_READY]: '✅',
+} as const;
+
+export const WORKFLOW_PHASE_COLORS = {
+  [WORKFLOW_PHASES.DRAFTING_SPEC]: 'accent',
+  [WORKFLOW_PHASES.READY_FOR_BUILDER]: 'accent',
+  [WORKFLOW_PHASES.BUILDER_RUNNING]: 'accent',
+  [WORKFLOW_PHASES.ESCALATION_DECISION]: 'warning',
+  [WORKFLOW_PHASES.BUILDER_FAILED]: 'error',
+  [WORKFLOW_PHASES.READY_FOR_VERIFIER]: 'accent',
+  [WORKFLOW_PHASES.VERIFIER_RUNNING]: 'accent',
+  [WORKFLOW_PHASES.FINDINGS_DECISION]: 'warning',
+  [WORKFLOW_PHASES.CANDIDATE_READY]: 'success',
+} as const satisfies Record<WorkflowPhase, ThemeColor>;
+
+const shortenSpecId = (specId: string): string => {
+  const title = specId.slice(16);
+
+  if (title.length <= 10) {
+    return specId;
+  }
+
+  return `${specId.slice(0, 26)}...`;
+};
+
 type MaestroStatusWorkflow = {
   state: WorkflowState;
 };
@@ -24,21 +63,34 @@ type MaestroStatusWorkflow = {
 type FormatMaestroStatusInput = {
   active: boolean;
   workflow: MaestroStatusWorkflow | null;
+  theme: Pick<Theme, 'fg'>; // foreground theme
 };
 
 export const formatMaestroStatus = ({
   active,
   workflow,
+  theme,
 }: FormatMaestroStatusInput): string | undefined => {
   if (!active) {
     return undefined;
   }
 
   if (workflow === null) {
-    return 'Maestro active · No active spec';
+    // muted is a theme color for less prominent text, usually a dim gray.
+    return `${theme.fg('muted', 'Maestro active · ')}${theme.fg('accent', 'No active spec')}`;
   }
 
-  const phase = WORKFLOW_PHASE_LABELS[workflow.state.phase];
+  const { phase, specId } = workflow.state;
+
+  const prefix = theme.fg(
+    'muted',
+    `Maestro active · ${shortenSpecId(specId)} · `,
+  );
+
+  const label = theme.fg(
+    WORKFLOW_PHASE_COLORS[phase],
+    WORKFLOW_PHASE_LABELS[phase],
+  );
 
-  return `Maestro active · ${workflow.state.specId} · ${phase}`;
+  return `${prefix}${WORKFLOW_PHASE_ICONS[phase]} ${label}`;
 };

package/src/maestro/status/refreshMaestroStatus.ts

@@ -25,7 +25,11 @@ export const refreshMaestroStatus = async (
   if (specId === null) {
     context.ui.setStatus(
       MAESTRO_STATUS_KEY,
-      formatMaestroStatus({ active: true, workflow: null }),
+      formatMaestroStatus({
+        active: true,
+        workflow: null,
+        theme: context.ui.theme,
+      }),
     );
 
     return;
@@ -38,7 +42,11 @@ export const refreshMaestroStatus = async (
 
     context.ui.setStatus(
       MAESTRO_STATUS_KEY,
-      formatMaestroStatus({ active: true, workflow: { state } }),
+      formatMaestroStatus({
+        active: true,
+        workflow: { state },
+        theme: context.ui.theme,
+      }),
     );
   } catch (error) {
     context.ui.setStatus(MAESTRO_STATUS_KEY, undefined);