supipowers 1.5.2 → 1.5.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supipowers",
-  "version": "1.5.2",
+  "version": "1.5.3",
   "description": "Workflow extension for OMP coding agents.",
   "type": "module",
   "scripts": {

package/src/commands/agents.ts

@@ -8,8 +8,8 @@ import {
   getGlobalReviewAgentsDir,
   getGlobalReviewAgentsConfigPath,
 } from "../review/agent-loader.js";
-import { selectModelFromList } from "./model.js";
-import type { ConfiguredReviewAgent } from "../types.js";
+import { selectModelFromList, THINKING_LEVELS } from "./model.js";
+import type { ConfiguredReviewAgent, ThinkingLevel } from "../types.js";
 import creatingAgentsSkill from "../../skills/creating-supi-agents/SKILL.md" with { type: "text" };
 
 // ── List View ──────────────────────────────────────────────────
@@ -17,23 +17,25 @@ import creatingAgentsSkill from "../../skills/creating-supi-agents/SKILL.md" wit
 function buildAgentDashboard(agents: ConfiguredReviewAgent[]): string {
   const nameCol = 18;
   const modelCol = 24;
+  const thinkingCol = 12;
   const scopeCol = 10;
 
   const lines: string[] = [
     "\n  Review Agents\n",
-    `  ${"name".padEnd(nameCol)} ${"model".padEnd(modelCol)} ${"scope".padEnd(scopeCol)} focus`,
+    `  ${"name".padEnd(nameCol)} ${"model".padEnd(modelCol)} ${"thinking".padEnd(thinkingCol)} ${"scope".padEnd(scopeCol)} focus`,
   ];
 
   for (const agent of agents) {
     const name = agent.name.padEnd(nameCol);
-    const model = (agent.model ?? "—").padEnd(modelCol);
+    const model = (agent.model ?? "\u2014").padEnd(modelCol);
+    const thinking = (agent.thinkingLevel ?? "\u2014").padEnd(thinkingCol);
     const scope = (agent.scope ?? "project").padEnd(scopeCol);
     const focus = agent.focus
       ? agent.focus.length > 40
         ? agent.focus.slice(0, 37) + "..."
         : agent.focus
-      : "—";
-    lines.push(`  ${name} ${model} ${scope} ${focus}`);
+      : "\u2014";
+    lines.push(`  ${name} ${model} ${thinking} ${scope} ${focus}`);
   }
 
   const globalCount = agents.filter((a) => a.scope === "global").length;
@@ -95,7 +97,16 @@ export async function runAgentCreateFlow(platform: Platform, ctx: any): Promise<
   const modelInput = await selectModelFromList(ctx);
   // null means "inherit default" — that's fine
 
-  // Step 4: Prompt source
+  // Step 4: Thinking level
+  const thinkingChoice = await ctx.ui.select(
+    "Thinking level",
+    THINKING_LEVELS.map((t) => t.label),
+    { helpText: "Select thinking level · Esc to cancel" },
+  );
+  if (!thinkingChoice) return;
+  const thinkingLevel: ThinkingLevel | null = THINKING_LEVELS.find((t) => t.label === thinkingChoice)?.value ?? null;
+
+  // Step 5: Prompt source
   const promptChoice = await ctx.ui.select("Agent prompt", [
     "Send a prompt",
     "Create from zero",
@@ -105,7 +116,7 @@ export async function runAgentCreateFlow(platform: Platform, ctx: any): Promise<
 
   if (promptChoice === "Create from zero") {
     // Load the creating-supi-agents skill via steer
-    loadSkillAndSteer(platform, ctx, scope, agentName, modelInput);
+    loadSkillAndSteer(platform, ctx, scope, agentName, modelInput, thinkingLevel);
     return;
   }
 
@@ -119,7 +130,7 @@ export async function runAgentCreateFlow(platform: Platform, ctx: any): Promise<
     return;
   }
 
-  // Step 5: Save
+  // Step 6: Save
   const agentsDir = scope === "global"
     ? getGlobalReviewAgentsDir(platform.paths)
     : getReviewAgentsDir(platform.paths, ctx.cwd);
@@ -138,6 +149,7 @@ export async function runAgentCreateFlow(platform: Platform, ctx: any): Promise<
     enabled: true,
     data: fileName,
     model: modelInput,
+    thinkingLevel,
   });
 
   ctx.ui.notify(`Agent "${agentName}" created (${scope})`, "info");
@@ -151,6 +163,7 @@ function loadSkillAndSteer(
   scope: "global" | "project",
   agentName: string,
   model: string | null,
+  thinkingLevel: ThinkingLevel | null,
 ): void {
   const agentsDir = scope === "global"
     ? getGlobalReviewAgentsDir(platform.paths)
@@ -159,7 +172,7 @@ function loadSkillAndSteer(
     ? getGlobalReviewAgentsConfigPath(platform.paths)
     : getReviewAgentsConfigPath(platform.paths, ctx.cwd);
 
-  const prompt = buildSkillSteerPrompt(agentName, scope, agentsDir, configPath, model);
+  const prompt = buildSkillSteerPrompt(agentName, scope, agentsDir, configPath, model, thinkingLevel);
 
   platform.sendMessage(
     {
@@ -177,6 +190,7 @@ function buildSkillSteerPrompt(
   agentsDir: string,
   configPath: string,
   model: string | null,
+  thinkingLevel: ThinkingLevel | null,
 ): string {
   return `You are guiding the user through creating a new AI review agent for supipowers' multi-agent code review pipeline.
 
@@ -184,6 +198,7 @@ function buildSkillSteerPrompt(
 - **Name**: ${agentName}
 - **Scope**: ${scope}
 - **Model**: ${model ?? "inherit default"}
+- **Thinking Level**: ${thinkingLevel ?? "inherit default"}
 - **Target directory**: ${agentsDir}
 - **Config path**: ${configPath}
 
@@ -197,7 +212,7 @@ Once the user approves the agent design, save it:
 1. Write the markdown file to: ${agentsDir}/${agentName}.md
    - The file MUST have YAML frontmatter (name, description, focus) and a prompt body ending with {output_instructions}
 2. Update config at: ${configPath}
-   - Add entry: { name: "${agentName}", enabled: true, data: "${agentName}.md", model: ${model ? `"${model}"` : "null"} }
+   - Add entry: { name: "${agentName}", enabled: true, data: "${agentName}.md", model: ${model ? `"${model}"` : "null"}, thinkingLevel: ${thinkingLevel ? `"${thinkingLevel}"` : "null"} }
 
 Use the \`writeAgentFile\` and \`addAgentToConfig\` functions from \`src/review/agent-loader.ts\` if you have tool access, or write the files directly.
 

package/src/commands/model.ts

@@ -11,7 +11,7 @@ import type { ModelAction, ModelAssignment, ThinkingLevel } from "../types.js";
 import { getBundledProviders, getBundledModels, type GeneratedProvider } from "@oh-my-pi/pi-ai";
 import { createModelPicker, type AvailableModelSet } from "./model-picker.js";
 
-const THINKING_LEVELS: Array<{ label: string; value: ThinkingLevel | null }> = [
+export const THINKING_LEVELS: Array<{ label: string; value: ThinkingLevel | null }> = [
   { label: "Inherit (model default)", value: null },
   { label: "Off", value: "off" },
   { label: "Minimal", value: "minimal" },

package/src/review/agent-loader.ts

@@ -29,9 +29,9 @@ const DEFAULT_AGENT_TEMPLATES: Record<string, string> = {
 };
 
 const DEFAULT_REVIEW_AGENTS_CONFIG: ReviewAgentConfig[] = [
-  { name: "security", enabled: true, data: "security.md", model: null },
-  { name: "correctness", enabled: true, data: "correctness.md", model: null },
-  { name: "maintainability", enabled: true, data: "maintainability.md", model: null },
+  { name: "security", enabled: true, data: "security.md", model: null, thinkingLevel: "low" },
+  { name: "correctness", enabled: true, data: "correctness.md", model: null, thinkingLevel: "low" },
+  { name: "maintainability", enabled: true, data: "maintainability.md", model: null, thinkingLevel: "low" },
 ];
 
 export interface LoadedReviewAgents {
@@ -41,19 +41,39 @@ export interface LoadedReviewAgents {
   agents: ConfiguredReviewAgent[];
 }
 
-function buildDefaultConfigText(): string {
+const CONFIG_HEADER = [
+  "# Review Agents Configuration",
+  "#",
+  "# Options:",
+  "#   name:          string   - agent identifier (kebab-case)",
+  "#   enabled:       boolean  - true | false",
+  "#   data:          string   - markdown file name in the agents directory",
+  "#   model:         string   - model id (e.g. \"anthropic/claude-sonnet-4-20250514\") or null to inherit",
+  "#   thinkingLevel: string   - off | minimal | low | medium | high | xhigh | null to inherit",
+  "#",
+].join("\n");
+
+function serializeConfigYaml(agents: ReviewAgentConfig[]): string {
   return [
+    CONFIG_HEADER,
+    "",
     "agents:",
-    ...DEFAULT_REVIEW_AGENTS_CONFIG.flatMap((agent) => [
-      `  - name: ${agent.name}`,
-      `    enabled: ${agent.enabled}`,
-      `    data: ${agent.data}`,
-      "    model: null",
+    ...agents.flatMap((a, i) => [
+      ...(i > 0 ? [""] : []),
+      `  - name: ${a.name}`,
+      `    enabled: ${a.enabled}`,
+      `    data: ${a.data}`,
+      `    model: ${a.model ?? "null"}`,
+      `    thinkingLevel: ${a.thinkingLevel ?? "null"}`,
     ]),
     "",
   ].join("\n");
 }
 
+function buildDefaultConfigText(): string {
+  return serializeConfigYaml(DEFAULT_REVIEW_AGENTS_CONFIG);
+}
+
 function writeIfMissing(filePath: string, content: string): void {
   if (fs.existsSync(filePath)) {
     return;
@@ -62,6 +82,51 @@ function writeIfMissing(filePath: string, content: string): void {
   fs.writeFileSync(filePath, content);
 }
 
+/**
+ * Migrate pre-existing config.yml files:
+ *  - adds the comment header if missing
+ *  - backfills thinkingLevel on agents that lack it
+ */
+function migrateConfigIfNeeded(configPath: string): void {
+  if (!fs.existsSync(configPath)) return;
+
+  const raw = fs.readFileSync(configPath, "utf-8");
+  if (raw.startsWith("# Review Agents Configuration")) return;
+
+  // Parse the bare YAML by hand — we only need name/enabled/data/model/thinkingLevel.
+  // The file is small and always has the same shape.
+  const agents: ReviewAgentConfig[] = [];
+  let current: Partial<ReviewAgentConfig> | null = null;
+
+  for (const line of raw.split("\n")) {
+    const trimmed = line.trim();
+    if (trimmed.startsWith("- name:")) {
+      if (current?.name) agents.push(current as ReviewAgentConfig);
+      current = { name: trimmed.slice("- name:".length).trim() };
+    } else if (current && trimmed.startsWith("enabled:")) {
+      current.enabled = trimmed.slice("enabled:".length).trim() === "true";
+    } else if (current && trimmed.startsWith("data:")) {
+      current.data = trimmed.slice("data:".length).trim();
+    } else if (current && trimmed.startsWith("model:")) {
+      const val = trimmed.slice("model:".length).trim();
+      current.model = val === "null" ? null : val;
+    } else if (current && trimmed.startsWith("thinkingLevel:")) {
+      const val = trimmed.slice("thinkingLevel:".length).trim();
+      current.thinkingLevel = val === "null" ? null : (val as any);
+    }
+  }
+  if (current?.name) agents.push(current as ReviewAgentConfig);
+
+  // Backfill thinkingLevel for agents that didn't have it
+  for (const agent of agents) {
+    if (agent.thinkingLevel === undefined) {
+      agent.thinkingLevel = null;
+    }
+  }
+
+  fs.writeFileSync(configPath, serializeConfigYaml(agents));
+}
+
 function validateReviewAgentsConfig(data: unknown): ReviewAgentsConfig {
   const errors = formatReviewValidationErrors(collectReviewValidationErrors(ReviewAgentsConfigSchema, data));
   if (errors.length > 0) {
@@ -145,6 +210,7 @@ export function ensureDefaultReviewAgents(paths: PlatformPaths, cwd: string): vo
 
   // Default agent markdown files are installed globally only.
   writeIfMissing(getReviewAgentsConfigPath(paths, cwd), buildDefaultConfigText());
+  migrateConfigIfNeeded(getReviewAgentsConfigPath(paths, cwd));
 }
 
 export async function loadReviewAgentsConfig(paths: PlatformPaths, cwd: string): Promise<ReviewAgentsConfig> {
@@ -183,6 +249,7 @@ export async function loadReviewAgents(paths: PlatformPaths, cwd: string): Promi
         enabled: agent.enabled,
         data: agent.data,
         model: agent.model,
+        thinkingLevel: agent.thinkingLevel ?? null,
       } satisfies ConfiguredReviewAgent;
     });
 
@@ -213,6 +280,7 @@ export function ensureGlobalDefaultReviewAgents(paths: PlatformPaths): void {
   }
 
   writeIfMissing(getGlobalReviewAgentsConfigPath(paths), buildDefaultConfigText());
+  migrateConfigIfNeeded(getGlobalReviewAgentsConfigPath(paths));
 }
 
 export async function loadGlobalReviewAgentsConfig(paths: PlatformPaths): Promise<ReviewAgentsConfig> {
@@ -245,6 +313,7 @@ export async function loadGlobalReviewAgents(paths: PlatformPaths): Promise<Load
         enabled: agent.enabled,
         data: agent.data,
         model: agent.model,
+        thinkingLevel: agent.thinkingLevel ?? null,
         scope: "global" as const,
       } satisfies ConfiguredReviewAgent;
     });
@@ -321,15 +390,5 @@ export async function addAgentToConfig(
     config.agents.push(agent);
   }
 
-  const text = [
-    "agents:",
-    ...config.agents.flatMap((a) => [
-      `  - name: ${a.name}`,
-      `    enabled: ${a.enabled}`,
-      `    data: ${a.data}`,
-      `    model: ${a.model ?? "null"}`,
-    ]),
-    "",
-  ].join("\n");
-  fs.writeFileSync(configPath, text);
+  fs.writeFileSync(configPath, serializeConfigYaml(config.agents));
 }

package/src/review/multi-agent-runner.ts

@@ -88,7 +88,7 @@ async function runConfiguredAgent(
       };
     },
     model: agent.model ?? input.model,
-    thinkingLevel: input.thinkingLevel ?? null,
+    thinkingLevel: agent.thinkingLevel ?? input.thinkingLevel ?? null,
     timeoutMs: input.timeoutMs ?? 120_000,
   });
 

package/src/review/types.ts

@@ -17,6 +17,7 @@ import type {
   ReviewScopeStats,
   ReviewSession,
   ReviewSessionArtifacts,
+  ThinkingLevel,
 } from "../types.js";
 export type {
   ConfiguredReviewAgent,
@@ -122,6 +123,15 @@ export const ReviewAgentConfigSchema = Type.Object(
     enabled: Type.Boolean(),
     data: Type.String({ minLength: 1 }),
     model: Type.Union([Type.String({ minLength: 1 }), Type.Null()]),
+    thinkingLevel: Type.Optional(Type.Union([
+      Type.Literal("off"),
+      Type.Literal("minimal"),
+      Type.Literal("low"),
+      Type.Literal("medium"),
+      Type.Literal("high"),
+      Type.Literal("xhigh"),
+      Type.Null(),
+    ])),
   },
   { additionalProperties: false },
 );

package/src/types.ts

@@ -174,6 +174,7 @@ export interface ReviewAgentConfig {
   enabled: boolean;
   data: string;
   model: string | null;
+  thinkingLevel: ThinkingLevel | null;
 }
 
 /** Top-level review agent pipeline config */
@@ -194,6 +195,7 @@ export interface ConfiguredReviewAgent extends ReviewAgentDefinition {
   enabled: boolean;
   data: string;
   model: string | null;
+  thinkingLevel: ThinkingLevel | null;
   scope?: "global" | "project";
 }