supipowers 1.5.1 → 1.5.3

package/README.md

@@ -23,7 +23,7 @@ Run the interactive installer:
 bunx supipowers
 ```
 
-The installer detects your agent, registers the extension, removes legacy external context-mode MCP registrations, and can install missing optional tooling such as LSP servers, `mcpc`, and Playwright CLI.
+The installer detects Pi (`~/.pi`) and OMP (`~/.omp`) — when both are present it offers a multiselect to install to one or both. It registers the extension, removes legacy external context-mode MCP registrations from `agent/mcp.json` and cleans up the old `settings/mcp.json` if present, and can install missing optional tooling such as LSP servers, `mcpc`, and Playwright CLI.
 
 > [!TIP]
 > Run `/supi:update` at any time to upgrade to the latest version, or `/supi:doctor` to check your setup.
@@ -159,6 +159,15 @@ Configuration is a three-layer deep-merge (lowest to highest priority):
 
 Three built-in channels are available: `github` (GitHub Release via `gh` CLI), `gitlab` (GitLab Release via `glab` CLI), and `gitea` (Gitea Release via `tea` CLI). Channels are selected per-project in `release.channels`.
 
+Release notes are generated from conventional commits scoped to the paths listed in `package.json`'s `files` field — commits touching files outside those paths are excluded.
+
+`/supi:release` accepts two optional flags:
+
+| Flag | Effect |
+| ----------- | ------ |
+| `--raw` | Skip AI polish of release notes; use raw conventional-commit output |
+| `--dry-run` | Run the full release flow without publishing |
+
 Custom channels can be defined in `release.customChannels`:
 
 ```json

package/bin/install.ts

@@ -229,6 +229,12 @@ function installToPlatform(platformDir: string, packageRoot: string): string {
     cpSync(join(packageRoot, "src"), join(extDir, "src"), { recursive: true });
     cpSync(join(packageRoot, "bin"), join(extDir, "bin"), { recursive: true });
     cpSync(join(packageRoot, "package.json"), join(extDir, "package.json"));
+    // skills/ must live inside the extension dir — src/commands/agents.ts
+    // uses a static `import from "../../skills/..."` resolved relative to src/.
+    const skillsSrc = join(packageRoot, "skills");
+    if (existsSync(skillsSrc)) {
+      cpSync(skillsSrc, join(extDir, "skills"), { recursive: true });
+    }
     log(`  files copied to ${extDir}`);
 
     // Rewrite package.json for the installed extension.
@@ -242,13 +248,12 @@ function installToPlatform(platformDir: string, packageRoot: string): string {
       type: sourcePkg.type,
       omp: sourcePkg.omp,
       dependencies: {
-        // Only packages imported at runtime by src/ code:
-        // - config/schema.ts → @sinclair/typebox
-        // - commands/model.ts, model-picker.ts → @oh-my-pi/pi-ai
-        // - commands/model-picker.ts → @oh-my-pi/pi-tui
-        "@sinclair/typebox": "*",
-        "@oh-my-pi/pi-ai": "*",
-        "@oh-my-pi/pi-tui": "*",
+        // Runtime deps from the published package.json (handlebars, etc.)
+        ...(sourcePkg.dependencies ?? {}),
+        // Peer deps that OMP provides at its global level. On macOS Bun resolves
+        // these from the global install, but on Windows the extension's own
+        // node_modules must contain them or Bun's import fails.
+        ...(sourcePkg.peerDependencies ?? {}),
       },
     };
     writeFileSync(join(extDir, "package.json"), JSON.stringify(runtimePkg, null, 2));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supipowers",
-  "version": "1.5.1",
+  "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/commands/update.ts

@@ -98,7 +98,44 @@ async function updateSupipowers(
     if (existsSync(binSource)) {
       cpSync(binSource, join(extDir, "bin"), { recursive: true });
     }
-    cpSync(join(downloadedRoot, "package.json"), join(extDir, "package.json"));
+    // skills/ must live inside the extension dir — src/commands/agents.ts
+    // uses a static `import from "../../skills/..."` resolved relative to src/.
+    const skillsDirSource = join(downloadedRoot, "skills");
+    if (existsSync(skillsDirSource)) {
+      cpSync(skillsDirSource, join(extDir, "skills"), { recursive: true });
+    }
+
+    // Rewrite package.json: merge runtime deps + peer deps so Bun on Windows
+    // can resolve all imports from the extension's own node_modules.
+    const sourcePkg = JSON.parse(readFileSync(join(downloadedRoot, "package.json"), "utf8"));
+    const runtimePkg = {
+      name: sourcePkg.name,
+      version: sourcePkg.version,
+      type: sourcePkg.type,
+      omp: sourcePkg.omp,
+      dependencies: {
+        ...(sourcePkg.dependencies ?? {}),
+        ...(sourcePkg.peerDependencies ?? {}),
+      },
+    };
+    writeFileSync(join(extDir, "package.json"), JSON.stringify(runtimePkg, null, 2));
+
+    // Install runtime dependencies (handlebars, etc.)
+    // Without this, the extension fails to load because node_modules/ was deleted above.
+    ctx.ui.notify("Installing dependencies...", "info");
+    const bunInstall = await platform.exec("bun", ["install", "--production"], { cwd: extDir });
+    if (bunInstall.code !== 0) {
+      // Fallback to npm if bun is not available (e.g. Windows without global bun)
+      const npmInstall = await platform.exec("npm", ["install", "--omit=dev"], { cwd: extDir });
+      if (npmInstall.code !== 0) {
+        ctx.ui.notify(
+          "Could not install extension dependencies.\n" +
+            "Commands may not appear. Run manually:\n" +
+            `  cd ${extDir} && bun install`,
+          "warning",
+        );
+      }
+    }
 
     // Copy skills
     const skillsSource = join(downloadedRoot, "skills");

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";
 }