supipowers 1.2.6 → 1.5.0

package/src/commands/status.ts

@@ -1,24 +1,51 @@
 import type { Platform, PlatformContext } from "../platform/types.js";
+import { DEFAULT_CONFIG } from "../config/defaults.js";
+import { formatConfigErrors, inspectConfig } from "../config/loader.js";
 import { listPlans } from "../storage/plans.js";
-import { loadConfig } from "../config/loader.js";
+import { summarizeEnabledGates } from "../quality/setup.js";
+
+export interface StatusCommandDependencies {
+  inspectConfig: typeof inspectConfig;
+  listPlans: typeof listPlans;
+}
+
+const STATUS_COMMAND_DEPENDENCIES: StatusCommandDependencies = {
+  inspectConfig,
+  listPlans,
+};
+
+function formatStatusSummary(platform: Platform, ctx: PlatformContext, deps: StatusCommandDependencies): string {
+  const inspection = deps.inspectConfig(platform.paths, ctx.cwd);
+  const config = inspection.effectiveConfig ?? DEFAULT_CONFIG;
+
+  if (inspection.parseErrors.length > 0 || inspection.validationErrors.length > 0) {
+    return `Config error: ${formatConfigErrors(inspection).split("\n")[0]}`;
+  }
+
+  return `Gates: ${summarizeEnabledGates(config.quality.gates)}`;
+}
+
+export async function showStatusDialog(
+  platform: Platform,
+  ctx: PlatformContext,
+  deps: StatusCommandDependencies = STATUS_COMMAND_DEPENDENCIES,
+): Promise<void> {
+  const plans = deps.listPlans(platform.paths, ctx.cwd);
+  const options = [
+    formatStatusSummary(platform, ctx, deps),
+    `Plans: ${plans.length === 0 ? "none" : ""}`,
+    ...plans.map((plan) => `  · ${plan}`),
+    "",
+    "Close",
+  ];
+
+  await ctx.ui.select("Supipowers Status", options, {
+    helpText: "Esc to close",
+  });
+}
 
 export function handleStatus(platform: Platform, ctx: PlatformContext): void {
-  void (async () => {
-    const plans = listPlans(platform.paths, ctx.cwd);
-    const config = loadConfig(platform.paths, ctx.cwd);
-
-    const options = [
-      `Profile: ${config.defaultProfile}`,
-      `Plans: ${plans.length === 0 ? "none" : ""}`,
-      ...plans.map((p) => `  · ${p}`),
-      "",
-      "Close",
-    ];
-
-    await ctx.ui.select("Supipowers Status", options, {
-      helpText: "Esc to close",
-    });
-  })();
+  void showStatusDialog(platform, ctx, STATUS_COMMAND_DEPENDENCIES);
 }
 
 export function registerStatusCommand(platform: Platform): void {

package/src/commands/supi.ts

@@ -1,48 +1,77 @@
 import type { Platform, PlatformContext } from "../platform/types.js";
-import { loadConfig } from "../config/loader.js";
+import { DEFAULT_CONFIG } from "../config/defaults.js";
+import { formatConfigErrors, inspectConfig } from "../config/loader.js";
 import { loadLatestReport } from "../storage/reports.js";
 import { listPlans } from "../storage/plans.js";
+import { summarizeEnabledGates } from "../quality/setup.js";
 
-export function handleSupi(platform: Platform, ctx: PlatformContext): void {
-  void (async () => {
-    const config = loadConfig(platform.paths, ctx.cwd);
-    const latestReport = loadLatestReport(platform.paths, ctx.cwd);
-    const plans = listPlans(platform.paths, ctx.cwd);
-
-    const commands = [
-      "/supi:plan     — Start collaborative planning",
-      "/supi:review   — Run quality gates",
-      "/supi:qa       — E2E product testing with Playwright",
-      "/supi:fix-pr   — Fix PR review comments",
-      "/supi:release  — Release automation",
-      "/supi:config   — Manage configuration",
-      "/supi:status   — Show project status",
-      "/supi:doctor   — Run health checks",
-      "/supi:update   — Update to latest version",
-      "/supi:context  — Show context breakdown",
-    ];
-
-    const status = [
-      `Profile: ${config.defaultProfile}`,
-      `Plans: ${plans.length}`,
-      `Last review: ${latestReport ? `${latestReport.timestamp.slice(0, 10)} (${latestReport.passed ? "passed" : "failed"})` : "none"}`,
-    ];
-
-    const choice = await ctx.ui.select(
-      "Supipowers",
-      [...commands, "", ...status, "", "Close"],
-      { helpText: "Select a command to run · Esc to close" },
-    );
-
-    if (choice && choice.startsWith("/supi:")) {
-      const cmdName = choice.split(" ")[0].slice(1); // remove leading /
-      const cmd = platform.getCommands().find((c) => c.name === cmdName);
-      if (cmd?.handler) {
-        await cmd.handler("", ctx as any);
-      }
+export interface SupiCommandDependencies {
+  inspectConfig: typeof inspectConfig;
+  loadLatestReport: typeof loadLatestReport;
+  listPlans: typeof listPlans;
+}
+
+const SUPI_COMMAND_DEPENDENCIES: SupiCommandDependencies = {
+  inspectConfig,
+  loadLatestReport,
+  listPlans,
+};
+
+function formatOverviewStatus(platform: Platform, ctx: PlatformContext, deps: SupiCommandDependencies): string[] {
+  const inspection = deps.inspectConfig(platform.paths, ctx.cwd);
+  const config = inspection.effectiveConfig ?? DEFAULT_CONFIG;
+  const latestReport = deps.loadLatestReport(platform.paths, ctx.cwd);
+  const plans = deps.listPlans(platform.paths, ctx.cwd);
+
+  return [
+    inspection.parseErrors.length > 0 || inspection.validationErrors.length > 0
+      ? `Config error: ${formatConfigErrors(inspection).split("\n")[0]}`
+      : `Gates: ${summarizeEnabledGates(config.quality.gates)}`,
+    `Plans: ${plans.length}`,
+    `Last checks: ${latestReport ? `${latestReport.timestamp.slice(0, 10)} (${latestReport.overallStatus})` : "none"}`,
+  ];
+}
+
+export async function showSupiDialog(
+  platform: Platform,
+  ctx: PlatformContext,
+  deps: SupiCommandDependencies = SUPI_COMMAND_DEPENDENCIES,
+): Promise<void> {
+  const commands = [
+    "/supi:plan     — Start collaborative planning",
+    "/supi:review   — Run AI code review pipeline",
+    "/supi:agents  — List and manage review agents",
+    "/supi:checks   — Run quality gates",
+    "/supi:qa       — E2E product testing with Playwright",
+    "/supi:fix-pr   — Fix PR review comments",
+    "/supi:release  — Release automation",
+    "/supi:config   — Manage configuration",
+    "/supi:status   — Show project status",
+    "/supi:doctor   — Run health checks",
+    "/supi:update   — Update to latest version",
+    "/supi:context  — Show context breakdown",
+    "/supi:optimize-context — Optimize context to save tokens",
+  ];
+  const status = formatOverviewStatus(platform, ctx, deps);
+
+  const choice = await ctx.ui.select(
+    "Supipowers",
+    [...commands, "", ...status, "", "Close"],
+    { helpText: "Select a command to run · Esc to close" },
+  );
+
+  if (choice && choice.startsWith("/supi:")) {
+    const commandName = choice.split(" ")[0].slice(1);
+    const command = platform.getCommands().find((candidate) => candidate.name === commandName);
+    if (command?.handler) {
+      await command.handler("", ctx as any);
     }
-  })().catch((err) => {
-    ctx.ui.notify(`Supipowers error: ${(err as Error).message}`, "error");
+  }
+}
+
+export function handleSupi(platform: Platform, ctx: PlatformContext): void {
+  void showSupiDialog(platform, ctx, SUPI_COMMAND_DEPENDENCIES).catch((error) => {
+    ctx.ui.notify(`Supipowers error: ${(error as Error).message}`, "error");
   });
 }
 
@@ -50,7 +79,6 @@ export function registerSupiCommand(platform: Platform): void {
   platform.registerCommand("supi", {
     description: "Supipowers overview — show available commands and project status",
     async handler(_args: string | undefined, ctx: any) {
-      // Handled via input event interception — this is a fallback for non-interactive contexts
       handleSupi(platform, ctx);
     },
   });

package/src/commands/update.ts

@@ -1,8 +1,8 @@
 import type { Platform, PlatformContext } from "../platform/types.js";
 import type { DependencyStatus, InstallResult } from "../deps/registry.js";
 import { scanAll, installAll, formatReport } from "../deps/registry.js";
-import { readFileSync, existsSync, mkdirSync, cpSync, rmSync, readdirSync } from "node:fs";
-import { join } from "node:path";
+import { readFileSync, writeFileSync, existsSync, mkdirSync, cpSync, rmSync, readdirSync } from "node:fs";
+import { join, dirname } from "node:path";
 import { tmpdir } from "node:os";
 
 // ── Options builder ──────────────────────────────────────
@@ -94,6 +94,10 @@ async function updateSupipowers(
     // Copy extension files
     mkdirSync(extDir, { recursive: true });
     cpSync(join(downloadedRoot, "src"), join(extDir, "src"), { recursive: true });
+    const binSource = join(downloadedRoot, "bin");
+    if (existsSync(binSource)) {
+      cpSync(binSource, join(extDir, "bin"), { recursive: true });
+    }
     cpSync(join(downloadedRoot, "package.json"), join(extDir, "package.json"));
 
     // Copy skills
@@ -110,6 +114,9 @@ async function updateSupipowers(
       }
     }
 
+    // Clean up legacy MCP entries and re-register with current name
+    cleanupLegacyMcp(agentDir);
+
     ctx.ui.notify(`supipowers updated to v${latestVersion}`, "info");
     return { updated: true, fromVersion: currentVersion, toVersion: latestVersion };
   } finally {
@@ -174,6 +181,54 @@ export function handleUpdate(platform: Platform, ctx: PlatformContext): void {
   })();
 }
 
+// ── Legacy cleanup ───────────────────────────────────────
+
+/**
+ * Remove stale MCP artifacts from pre-v0.5.x installs.
+ * Context-mode tools are now native — remove any MCP server entries.
+ *
+ * Handles:
+ *  1. Old "context-mode" key in agent/mcp.json
+ *  2. "supi-context-mode" key in agent/mcp.json (no longer needed)
+ *  3. Old settings/mcp.json file (wrong path — should be agent/mcp.json)
+ */
+function cleanupLegacyMcp(agentDir: string): void {
+  const mcpConfigPath = join(agentDir, "mcp.json");
+
+  // Remove context-mode MCP entries (no longer needed — tools are native)
+  if (existsSync(mcpConfigPath)) {
+    try {
+      const config = JSON.parse(readFileSync(mcpConfigPath, "utf8"));
+      if (config.mcpServers) {
+        delete config.mcpServers["context-mode"];
+        delete config.mcpServers["supi-context-mode"];
+        writeFileSync(mcpConfigPath, JSON.stringify(config, null, 2));
+      }
+    } catch {
+      // Corrupted mcp.json — leave it for the user
+    }
+  }
+
+  // Remove old settings/mcp.json from pre-v0.5.x installs (wrong path)
+  const platformRoot = dirname(agentDir);
+  const oldMcpPath = join(platformRoot, "settings", "mcp.json");
+  if (!existsSync(oldMcpPath)) return;
+
+  try {
+    const config = JSON.parse(readFileSync(oldMcpPath, "utf8"));
+    delete config.mcpServers?.["context-mode"];
+    delete config.mcpServers?.["supi-context-mode"];
+
+    if (!config.mcpServers || Object.keys(config.mcpServers).length === 0) {
+      rmSync(oldMcpPath);
+    } else {
+      writeFileSync(oldMcpPath, JSON.stringify(config, null, 2));
+    }
+  } catch {
+    try { rmSync(oldMcpPath); } catch { /* best effort */ }
+  }
+}
+
 // ── Command registration ─────────────────────────────────
 
 export function registerUpdateCommand(platform: Platform): void {

package/src/config/defaults.ts

@@ -1,22 +1,22 @@
 // src/config/defaults.ts
-import type { SupipowersConfig, Profile } from "../types.js";
+import type { SupipowersConfig } from "../types.js";
 
 export const DEFAULT_CONFIG: SupipowersConfig = {
   version: "1.0.0",
-  defaultProfile: "thorough",
+  quality: {
+    gates: {},
+  },
   lsp: {
     setupGuide: true,
   },
-  notifications: {
-    verbosity: "normal",
-  },
   qa: {
     framework: null,
-    command: null,
     e2e: false,
   },
   release: {
     channels: [],
+    tagFormat: "v${version}",
+    customChannels: {},
   },
   contextMode: {
     enabled: true,
@@ -33,36 +33,3 @@ export const DEFAULT_CONFIG: SupipowersConfig = {
     closeSessionsOnExit: false,
   },
 };
-
-export const BUILTIN_PROFILES: Record<string, Profile> = {
-  quick: {
-    name: "quick",
-    gates: {
-      lspDiagnostics: true,
-      aiReview: { enabled: true, depth: "quick" },
-      codeQuality: false,
-      testSuite: false,
-      e2e: false,
-    },
-  },
-  thorough: {
-    name: "thorough",
-    gates: {
-      lspDiagnostics: true,
-      aiReview: { enabled: true, depth: "deep" },
-      codeQuality: true,
-      testSuite: false,
-      e2e: false,
-    },
-  },
-  "full-regression": {
-    name: "full-regression",
-    gates: {
-      lspDiagnostics: true,
-      aiReview: { enabled: true, depth: "deep" },
-      codeQuality: true,
-      testSuite: true,
-      e2e: true,
-    },
-  },
-};