@loicngr/kobo 1.7.8 → 1.7.10

package/dist/server/routes/workspaces.js

@@ -4,7 +4,7 @@ const execFileAsync = promisify(execFileCb);
 import fs from 'node:fs';
 import path from 'node:path';
 import { Hono } from 'hono';
-import { AUTO_LOOP_HARD_RULES, buildAutoLoopGroomingSteps, PREP_AUTOLOOP_INTRO, } from '../../shared/auto-loop-prompts.js';
+import { AUTO_LOOP_HARD_RULES, buildAutoLoopGroomingSteps, buildGroomingIntro } from '../../shared/auto-loop-prompts.js';
 import { getDb } from '../db/index.js';
 import { migrationGuard } from '../middleware/migration-guard.js';
 import { listEngines } from '../services/agent/engines/registry.js';
@@ -15,9 +15,9 @@ import * as devServerService from '../services/dev-server-service.js';
 import { DEFAULT_NOTION_INITIAL_PROMPT, DEFAULT_SENTRY_INITIAL_PROMPT, renderNotionInitialPrompt, renderSentryInitialPrompt, } from '../services/initial-prompt-template-service.js';
 import * as notionService from '../services/notion-service.js';
 import { renderPrTemplate } from '../services/pr-template-service.js';
-import { getAllPrStates } from '../services/pr-watcher-service.js';
+import { getAllPrSnapshots, refreshPrSnapshot } from '../services/pr-watcher-service.js';
 import * as quotaBackoffService from '../services/quota-backoff-service.js';
-import { DEFAULT_REVIEW_PROMPT_TEMPLATE, renderReviewTemplate } from '../services/review-template-service.js';
+import { getActiveReviewTemplate, renderReviewTemplate } from '../services/review-template-service.js';
 import * as sentryService from '../services/sentry-service.js';
 import * as settingsService from '../services/settings-service.js';
 import { runSetupScript } from '../services/setup-script-service.js';
@@ -720,13 +720,32 @@ app.get('/:id/sessions', (c) => {
 // exhaustive over the workspace list).
 app.get('/pr-states', (c) => {
     try {
-        return c.json(getAllPrStates());
+        return c.json(getAllPrSnapshots());
     }
     catch (err) {
         const message = err instanceof Error ? err.message : String(err);
         return c.json({ error: message }, 500);
     }
 });
+// POST /api/workspaces/pr-snapshot/refresh/:id — on-demand refresh of a single
+// workspace's PR snapshot, driven by the Git tab refresh button. Static prefix
+// keeps it ahead of `/:id` in the Hono matcher.
+app.post('/pr-snapshot/refresh/:id', async (c) => {
+    const id = c.req.param('id');
+    try {
+        const snapshot = await refreshPrSnapshot(id);
+        if (snapshot === null) {
+            return c.json({ error: 'No PR for this workspace' }, 404);
+        }
+        return c.json({ snapshot });
+    }
+    catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        if (/not found/i.test(message))
+            return c.json({ error: message }, 404);
+        return c.json({ error: message }, 500);
+    }
+});
 // GET /api/workspaces/auto-loop-states — batch snapshot keyed by workspace id.
 // Used by the drawer + Pinia store. Static path — must be BEFORE /:id.
 app.get('/auto-loop-states', (c) => {
@@ -1367,7 +1386,9 @@ app.get('/:id/prep-autoloop-prompt', (c) => {
         const projectSettings = settingsService.getProjectSettings(workspace.projectPath);
         const e2eSettings = projectSettings?.e2e ?? { framework: '', skill: '', prompt: '' };
         const finalizationSettings = projectSettings?.finalization ?? { prompt: '' };
-        const prompt = `${PREP_AUTOLOOP_INTRO}
+        const globalSettings = settingsService.getGlobalSettings();
+        const intro = buildGroomingIntro(globalSettings.skillSuite, globalSettings.customAutoLoopGroomingIntro);
+        const prompt = `${intro}
 
 ${buildAutoLoopGroomingSteps(e2eSettings, finalizationSettings)}
 
@@ -2477,8 +2498,7 @@ app.post('/:id/start-review', async (c) => {
         const diffStats = workingTreeStats.trim().length > 0
             ? `${committedStats}\n\n— Working tree (uncommitted) —\n${workingTreeStats}`
             : committedStats;
-        const effective = settingsService.getEffectiveSettings(workspace.projectPath);
-        const template = effective.reviewPromptTemplate || DEFAULT_REVIEW_PROMPT_TEMPLATE;
+        const template = getActiveReviewTemplate();
         const rendered = renderReviewTemplate(template, {
             workspace,
             commits,

package/dist/server/services/auto-loop-service.js

@@ -5,6 +5,7 @@ import { slugifyProjectName } from '../utils/project-slug.js';
 import { resolveWorkspaceWorktreePath } from '../utils/worktree-paths.js';
 import * as orchestrator from './agent/orchestrator.js';
 import * as settingsService from './settings-service.js';
+import { getSuitePrompts } from './skill-suite-prompts.js';
 import { emit, emitEphemeral } from './websocket-service.js';
 import { listTasks } from './workspace-service.js';
 const NO_PROGRESS_STALL_THRESHOLD = 3;
@@ -189,7 +190,7 @@ Your job this iteration:
 3. Run the project's quality checks (lint, typecheck, tests). Check \`.ai/.git-conventions.md\` for the exact commands if unclear.
 4. If checks fail, fix until they pass. If blocked, leave the task unchanged and explain in chat.
 5. Commit with a conventional message (\`feat: [short description]\` or similar per repo conventions).
-6. Code review gate — BEFORE marking the task done, dispatch an independent code-reviewer subagent via the Task tool with \`subagent_type: "code-reviewer"\` (or \`"superpowers:code-reviewer"\` / \`"pr-review-toolkit:code-reviewer"\` — use whichever exists in this environment; fall back to \`superpowers:requesting-code-review\` skill if none is available). Brief the reviewer with: what you just implemented, the task title, and the commit SHA (via \`git rev-parse HEAD\`). Ask specifically whether the change matches the task scope, whether edge cases are handled, and whether the commit is clean.
+6. {reviewGate}
 7. Act on the review:
    - If Critical/Important issues: fix them, amend or add a fix-up commit, re-run checks from step 3. Do NOT mark_task_done.
    - If only Minor issues: fix them if trivial (< 2 min), otherwise note them in the chat and proceed.
@@ -199,6 +200,18 @@ Your job this iteration:
 Do NOT modify other tasks' state. Do NOT create a PR. Do NOT skip the checks.
 Do NOT run \`kill\`, \`pkill\`, \`killall\`, \`pgrep -k\`, or any process-killing command — you may tear down the Kōbō server itself or sibling dev servers. If a dev server needs restarting, let the user do it.
 When you're done (success or blocked), end your turn cleanly.`;
+/**
+ * Resolve the active auto-loop review gate sentence (step 6 of the iteration
+ * prompt) for the current global skill suite, honouring any user override in
+ * `custom` mode. On the default `superpowers` suite this returns the same
+ * text that was historically inlined into PROMPT_TEMPLATE.
+ */
+function getActiveAutoLoopReviewGate() {
+    const global = settingsService.getGlobalSettings();
+    return getSuitePrompts(global.skillSuite, {
+        autoLoopReviewGate: global.customAutoLoopReviewGate,
+    }).autoLoopReviewGate;
+}
 function pickNextTask(workspaceId) {
     const pending = listTasks(workspaceId).filter((t) => t.status !== 'done');
     if (pending.length === 0)
@@ -257,7 +270,8 @@ function spawnNextIteration(workspaceId, opts = {}) {
         .replaceAll('{taskId}', task.id)
         .replaceAll('{taskTitle}', task.title)
         .replaceAll('{isAcceptanceCriterion}', String(task.isAcceptanceCriterion))
-        .replaceAll('{overrideBlock}', overrideBlock);
+        .replaceAll('{overrideBlock}', overrideBlock)
+        .replaceAll('{reviewGate}', getActiveAutoLoopReviewGate());
     const globalSettings = settingsService.getGlobalSettings();
     const projectSlug = globalSettings.worktreesPrefixByProject
         ? slugifyProjectName(projectSettings?.displayName ?? '', row.project_path)

package/dist/server/services/pr-watcher-service.js

@@ -2,7 +2,7 @@ import { getPrStatusAsync } from '../utils/git-ops.js';
 import { stopDevServer } from './dev-server-service.js';
 import { destroyTerminal } from './terminal-service.js';
 import { emitEphemeral } from './websocket-service.js';
-import { archiveWorkspace, listWorkspaces, updateWorkspaceSourceBranch } from './workspace-service.js';
+import { archiveWorkspace, getWorkspace, listWorkspaces, updateWorkspaceSourceBranch } from './workspace-service.js';
 // ── PR Watcher ────────────────────────────────────────────────────────────────
 // Polls GitHub every POLL_INTERVAL_MS to detect merged/closed PRs and
 // automatically archive the corresponding workspace.
@@ -14,18 +14,17 @@ import { archiveWorkspace, listWorkspaces, updateWorkspaceSourceBranch } from '.
 const POLL_INTERVAL_MS = 30 * 1000; // 30 seconds
 let timer = null;
 let checking = false;
+/** Tracks the last known PR snapshot per workspace, used to detect transitions
+ *  (state, base, reviewDecision). */
 const lastKnownPr = new Map();
 /**
- * Read-only snapshot of PR states known to the watcher, keyed by workspace id.
- * Used by the drawer to show a small PR-open indicator without N separate
- * `gh pr view` calls per workspace. Only contains entries where a PR has been
- * detected at least once by the watcher since boot; workspaces without a PR
- * are absent from the map.
+ * Read-only snapshot map, keyed by workspace id. Used by the drawer indicator
+ * AND the Git panel. Workspaces without a known PR are absent.
  */
-export function getAllPrStates() {
+export function getAllPrSnapshots() {
     const out = {};
-    for (const [id, known] of lastKnownPr) {
-        out[id] = known.state;
+    for (const [id, snap] of lastKnownPr) {
+        out[id] = snap;
     }
     return out;
 }
@@ -45,9 +44,6 @@ export async function checkPrStatuses() {
         }
     }
     for (const ws of workspaces) {
-        // Only check workspaces that are not actively running an agent
-        if (['extracting', 'brainstorming', 'executing'].includes(ws.status))
-            continue;
         try {
             const pr = await getPrStatusAsync(ws.projectPath, ws.workingBranch);
             if (!pr)
@@ -61,6 +57,13 @@ export async function checkPrStatuses() {
             // Archive on a transition FROM OPEN to CLOSED/MERGED. Skips the
             // base-change detection below — archiving wins.
             if (prev?.state === 'OPEN' && (pr.state === 'MERGED' || pr.state === 'CLOSED')) {
+                if (['extracting', 'brainstorming', 'executing'].includes(ws.status)) {
+                    // Agent is working — update the cache but skip auto-archive.
+                    // (The defensive base preservation from the no-base branch doesn't apply here
+                    // because we ARE in the OPEN→MERGED/CLOSED branch which always has a base.)
+                    lastKnownPr.set(ws.id, pr);
+                    continue;
+                }
                 console.log(`[pr-watcher] PR ${pr.state.toLowerCase()} for workspace '${ws.name}' — archiving`);
                 // Best-effort cleanup (same as manual archive): stop dev server + terminal.
                 // Agent is already not running here (guarded above).
@@ -84,13 +87,32 @@ export async function checkPrStatuses() {
                 });
                 continue; // do not run base-change detection on a workspace we just archived
             }
+            // Review-decision transitions (only on OPEN PRs; first-sight is silent).
+            // Reuses the baseline rule from base-change detection: emits only when we
+            // observe an actual transition between two known states.
+            if (pr.state === 'OPEN' && prev) {
+                if (prev.reviewDecision !== 'CHANGES_REQUESTED' && pr.reviewDecision === 'CHANGES_REQUESTED') {
+                    emitEphemeral(ws.id, 'pr:changes-requested', {
+                        prNumber: pr.number,
+                        prUrl: pr.url,
+                    });
+                }
+                else if (prev.reviewDecision === 'CHANGES_REQUESTED' && pr.reviewDecision === 'APPROVED') {
+                    emitEphemeral(ws.id, 'pr:approved', {
+                        prNumber: pr.number,
+                        prUrl: pr.url,
+                    });
+                }
+            }
             // Base-branch change detection. Only relevant for OPEN PRs — closed/
             // merged PRs don't accept base changes. Skip if the GitHub response
             // didn't include a baseRefName (defensive against malformed data).
             if (pr.state !== 'OPEN' || !pr.base) {
                 // Still update the cache for the state — keeps the OPEN→CLOSED/MERGED
-                // archiving logic working on the next tick.
-                lastKnownPr.set(ws.id, { state: pr.state, base: prev?.base });
+                // archiving logic working on the next tick. Preserve the previous
+                // `base` if the fresh snapshot is missing one (defensive).
+                const next = pr.base ? pr : { ...pr, base: prev?.base ?? pr.base };
+                lastKnownPr.set(ws.id, next);
                 continue;
             }
             // Comparison baseline:
@@ -100,8 +122,8 @@ export async function checkPrStatuses() {
             //    that happened while Kobo was offline.
             const previousBase = prev?.base ?? ws.sourceBranch;
             if (previousBase === pr.base) {
-                // No-op path: still record the base so subsequent ticks have a baseline.
-                lastKnownPr.set(ws.id, { state: pr.state, base: pr.base });
+                // No-op path: still record the snapshot so subsequent ticks have a baseline.
+                lastKnownPr.set(ws.id, pr);
                 continue;
             }
             console.log(`[pr-watcher] PR base changed for workspace '${ws.name}': ${previousBase} → ${pr.base}`);
@@ -116,7 +138,7 @@ export async function checkPrStatuses() {
             }
             // Both the persistence and the emit are part of "we successfully
             // observed a base change" — only NOW commit the new state to the cache.
-            lastKnownPr.set(ws.id, { state: pr.state, base: pr.base });
+            lastKnownPr.set(ws.id, pr);
             emitEphemeral(ws.id, 'pr:base-changed', {
                 oldBase: previousBase,
                 newBase: pr.base,
@@ -128,24 +150,49 @@ export async function checkPrStatuses() {
         }
     }
 }
+/**
+ * On-demand refresh of a single workspace's PR snapshot. Bypasses the 30s tick.
+ * No side effects beyond cache update — no archive, no transition emits. The
+ * user is watching the UI; we don't replay events for state they're already
+ * looking at.
+ *
+ * Returns the fresh snapshot, or null if the workspace has no PR (cache entry
+ * cleared in that case). Throws if the workspace doesn't exist.
+ */
+export async function refreshPrSnapshot(workspaceId) {
+    const ws = getWorkspace(workspaceId);
+    if (!ws)
+        throw new Error(`Workspace '${workspaceId}' not found`);
+    const snap = await getPrStatusAsync(ws.projectPath, ws.workingBranch);
+    if (snap === null) {
+        lastKnownPr.delete(workspaceId);
+        return null;
+    }
+    lastKnownPr.set(workspaceId, snap);
+    return snap;
+}
+/**
+ * Runs a single check while honouring the `checking` re-entrancy guard. Used
+ * both by the immediate boot-time kick-off and by the periodic timer tick.
+ */
+async function runOneCheck() {
+    if (checking)
+        return;
+    checking = true;
+    try {
+        await checkPrStatuses();
+    }
+    catch (err) {
+        console.error('[pr-watcher] Unexpected error in checkPrStatuses:', err);
+    }
+    finally {
+        checking = false;
+    }
+}
 function scheduleNext() {
     timer = setTimeout(async () => {
-        if (checking) {
-            // Previous run still in progress — skip and reschedule
-            scheduleNext();
-            return;
-        }
-        checking = true;
-        try {
-            await checkPrStatuses();
-        }
-        catch (err) {
-            console.error('[pr-watcher] Unexpected error in checkPrStatuses:', err);
-        }
-        finally {
-            checking = false;
-            scheduleNext();
-        }
+        await runOneCheck();
+        scheduleNext();
     }, POLL_INTERVAL_MS);
     timer.unref?.();
 }
@@ -153,6 +200,10 @@ function scheduleNext() {
 export function startPrWatcher() {
     if (timer)
         return;
+    // Kick off an immediate check so the front-end has fresh PR data on boot
+    // without waiting for the first 30s tick. Fire-and-forget; the recurring
+    // loop is scheduled independently and the `checking` guard prevents overlap.
+    void runOneCheck();
     scheduleNext();
 }
 /** Stop the PR watcher polling loop. */

package/dist/server/services/review-template-service.js

@@ -1,35 +1,28 @@
 import path from 'node:path';
-export const DEFAULT_REVIEW_PROMPT_TEMPLATE = `You are reviewing code changes on workspace "{{workspace_name}}" in project {{project_name}}.
-
-Branch: {{branch_name}}  (base: {{source_branch}})
-Base commit: {{base_commit}}
-
-If a code-review skill is available (e.g. superpowers:requesting-code-review), invoke it to drive this review. Otherwise follow the steps below directly.
-
-## Scope
-
-Review ALL changes — both committed and uncommitted in the working tree:
-- \`git diff {{base_commit}}..HEAD\` — committed changes on this branch
-- \`git status\` and \`git diff\` — uncommitted changes (staged + unstaged)
-
-## Diff summary
-{{diff_stats}}
-
-## Commits
-{{commits}}
-
-## Additional instructions
-{{additional_instructions}}
-
-## Output
-
-If no review skill is available, structure your reply as:
-1. Summary — what changed and why
-2. Issues — bugs, regressions, security or perf concerns (with file:line)
-3. Suggestions — refactor / improvement opportunities
-4. Tests — coverage gaps
-5. Verdict — ship / fix-then-ship / blocked
-`;
+import { getGlobalSettings } from './settings-service.js';
+import { getSuitePrompts, SUPERPOWERS_PROMPTS } from './skill-suite-prompts.js';
+/**
+ * Back-compat alias for the legacy review-template default. The canonical
+ * default now lives in `SUPERPOWERS_PROMPTS.reviewTemplate`; this export is
+ * kept so existing seed/migration paths in `settings-service.ts` and
+ * `routes/settings.ts` (the "reset to default" endpoint) keep working
+ * without being rewritten. Runtime readers should call
+ * `getActiveReviewTemplate()` instead, which respects the user's
+ * `skillSuite` choice.
+ */
+export const DEFAULT_REVIEW_PROMPT_TEMPLATE = SUPERPOWERS_PROMPTS.reviewTemplate;
+/**
+ * Resolve the review prompt template active for this user right now:
+ * read the global `skillSuite` and, in `custom` mode, the
+ * `customReviewTemplate` override. On the default `superpowers` suite this
+ * returns the same text as the legacy `DEFAULT_REVIEW_PROMPT_TEMPLATE`.
+ */
+export function getActiveReviewTemplate() {
+    const global = getGlobalSettings();
+    return getSuitePrompts(global.skillSuite, {
+        reviewTemplate: global.customReviewTemplate,
+    }).reviewTemplate;
+}
 function buildVariableMap(ctx) {
     return {
         project_name: path.basename(ctx.workspace.projectPath),

package/dist/server/services/settings-service.js

@@ -1,11 +1,14 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { WORKTREES_PATH } from '../../shared/consts.js';
+import { isValidProjectColor } from '../../shared/project-colors.js';
+import { isValidSkillSuite } from '../../shared/skill-suite-prompts.js';
 import { listClaudeMcpEntries } from '../utils/mcp-client.js';
 import { getSettingsPath } from '../utils/paths.js';
 import { InvalidWorktreesPathError, resolveGlobalWorktreesRoot, sanitizeWorktreesPath, validateWorktreesPath, } from '../utils/worktree-paths.js';
 import { DEFAULT_NOTION_INITIAL_PROMPT, DEFAULT_SENTRY_INITIAL_PROMPT } from './initial-prompt-template-service.js';
 import { DEFAULT_REVIEW_PROMPT_TEMPLATE } from './review-template-service.js';
+import { AGNOSTIC_PROMPTS } from './skill-suite-prompts.js';
 export const DEFAULT_GIT_CONVENTIONS = `# Git conventions
 
 ## Commits
@@ -354,6 +357,46 @@ const settingsMigrations = [
             delete global.defaultPermissionMode;
         },
     },
+    {
+        version: 21,
+        name: 'add-project-color-and-flatten',
+        migrate: ({ global, projects }) => {
+            if (typeof global.flattenWorkspaceList !== 'boolean') {
+                global.flattenWorkspaceList = false;
+            }
+            for (const p of projects) {
+                if (!('color' in p) || (p.color !== null && !isValidProjectColor(p.color))) {
+                    ;
+                    p.color = null;
+                }
+            }
+        },
+    },
+    {
+        version: 22,
+        name: 'add-skill-suite-selector',
+        // Auto-migrate every existing user to `superpowers` (the closest match to
+        // today's behaviour). New installs get the same default via
+        // `defaultSettings()`. The 4 `custom*` fields seed with the agnostic
+        // baseline so users switching to `custom` mode have a sane editable start.
+        migrate: ({ global }) => {
+            if (!isValidSkillSuite(global.skillSuite)) {
+                global.skillSuite = 'superpowers';
+            }
+            if (typeof global.customReviewTemplate !== 'string') {
+                global.customReviewTemplate = AGNOSTIC_PROMPTS.reviewTemplate;
+            }
+            if (typeof global.customAutoLoopReviewGate !== 'string') {
+                global.customAutoLoopReviewGate = AGNOSTIC_PROMPTS.autoLoopReviewGate;
+            }
+            if (typeof global.customAutoLoopGroomingIntro !== 'string') {
+                global.customAutoLoopGroomingIntro = AGNOSTIC_PROMPTS.autoLoopGroomingIntro;
+            }
+            if (typeof global.customQaPromptTemplate !== 'string') {
+                global.customQaPromptTemplate = AGNOSTIC_PROMPTS.qaPromptTemplate;
+            }
+        },
+    },
 ];
 /** Current settings schema version — always equals the highest migration version. */
 export const SETTINGS_SCHEMA_VERSION = settingsMigrations.length > 0 ? settingsMigrations[settingsMigrations.length - 1].version : 0;
@@ -411,6 +454,12 @@ function defaultSettings() {
             voicePrompt: '',
             voiceTranslateToEnglish: false,
             voiceSuppressNonSpeechTokens: true,
+            flattenWorkspaceList: false,
+            skillSuite: 'superpowers',
+            customReviewTemplate: AGNOSTIC_PROMPTS.reviewTemplate,
+            customAutoLoopReviewGate: AGNOSTIC_PROMPTS.autoLoopReviewGate,
+            customAutoLoopGroomingIntro: AGNOSTIC_PROMPTS.autoLoopGroomingIntro,
+            customQaPromptTemplate: AGNOSTIC_PROMPTS.qaPromptTemplate,
         },
         projects: [],
     };
@@ -440,6 +489,7 @@ function defaultProjectSettings(projectPath) {
         finalization: {
             prompt: DEFAULT_FINALIZATION_PROMPT,
         },
+        color: null,
     };
 }
 function pickKnownKeys(data, allowedKeys) {
@@ -635,6 +685,14 @@ export function getEffectiveSettings(projectPath) {
 /** Merge partial updates into global settings and persist. */
 export function updateGlobalSettings(data) {
     const settings = readSettings();
+    // Validate skillSuite before merging: drop invalid values so the previous
+    // value is preserved (same pattern as `upsertProject`'s color validation).
+    if ('skillSuite' in data) {
+        if (!isValidSkillSuite(data.skillSuite)) {
+            console.warn(`[settings] Invalid skillSuite value rejected: ${data.skillSuite}`);
+            delete data.skillSuite;
+        }
+    }
     const allowedGlobalKeys = [
         'defaultModelByEngine',
         'dangerouslySkipPermissions',
@@ -666,6 +724,12 @@ export function updateGlobalSettings(data) {
         'voicePrompt',
         'voiceTranslateToEnglish',
         'voiceSuppressNonSpeechTokens',
+        'flattenWorkspaceList',
+        'skillSuite',
+        'customReviewTemplate',
+        'customAutoLoopReviewGate',
+        'customAutoLoopGroomingIntro',
+        'customQaPromptTemplate',
     ];
     const filtered = pickKnownKeys(data, allowedGlobalKeys);
     if (filtered.tags !== undefined) {
@@ -709,6 +773,13 @@ function isNonNativeWindowsPath(value) {
 }
 /** Create or update project-specific settings. Merges devServer, e2e, and finalization fields on update. */
 export function upsertProject(projectPath, data) {
+    // Validate color: accept null or a valid palette entry; drop anything else.
+    if ('color' in data) {
+        if (data.color !== null && !isValidProjectColor(data.color)) {
+            console.warn(`[settings] Invalid color value rejected for project '${projectPath}': ${data.color}`);
+            delete data.color;
+        }
+    }
     const allowedProjectKeys = [
         'displayName',
         'defaultSourceBranch',
@@ -723,6 +794,7 @@ export function upsertProject(projectPath, data) {
         'devServer',
         'e2e',
         'finalization',
+        'color',
     ];
     const allowedDevServerKeys = ['startCommand', 'stopCommand'];
     const allowedE2eKeys = ['framework', 'skill', 'prompt'];

package/dist/server/services/skill-suite-prompts.js

@@ -0,0 +1,80 @@
+import { AGNOSTIC_AUTO_LOOP_GROOMING_INTRO, AGNOSTIC_AUTO_LOOP_REVIEW_GATE, AGNOSTIC_QA_PROMPT_TEMPLATE, AGNOSTIC_REVIEW_TEMPLATE, GROOMING_INTRO_GSTACK, GROOMING_INTRO_SUPERPOWERS, } from '../../shared/skill-suite-prompts.js';
+// Headers and body shared with the agnostic baseline. We reconstruct them
+// locally here so the suite-specific text can sit between header and body
+// in the same shape across all 3 suites.
+const REVIEW_HEADER = `You are reviewing code changes on workspace "{{workspace_name}}" in project {{project_name}}.
+
+Branch: {{branch_name}}  (base: {{source_branch}})
+Base commit: {{base_commit}}
+
+`;
+const REVIEW_BODY = `## Scope
+
+Review ALL changes — both committed and uncommitted in the working tree:
+- \`git diff {{base_commit}}..HEAD\` — committed changes on this branch
+- \`git status\` and \`git diff\` — uncommitted changes (staged + unstaged)
+
+## Diff summary
+{{diff_stats}}
+
+## Commits
+{{commits}}
+
+## Additional instructions
+{{additional_instructions}}
+
+## Output
+
+If no review skill is available, structure your reply as:
+1. Summary — what changed and why
+2. Issues — bugs, regressions, security or perf concerns (with file:line)
+3. Suggestions — refactor / improvement opportunities
+4. Tests — coverage gaps
+5. Verdict — ship / fix-then-ship / blocked
+`;
+export const SUPERPOWERS_PROMPTS = {
+    reviewTemplate: REVIEW_HEADER +
+        'If a code-review skill is available (e.g. superpowers:requesting-code-review), invoke it to drive this review. Otherwise follow the steps below directly.\n\n' +
+        REVIEW_BODY,
+    autoLoopReviewGate: 'Code review gate — BEFORE marking the task done, dispatch an independent code-reviewer subagent via the Task tool with `subagent_type: "code-reviewer"` (or `"superpowers:code-reviewer"` / `"pr-review-toolkit:code-reviewer"` — use whichever exists in this environment; fall back to `superpowers:requesting-code-review` skill if none is available). Brief the reviewer with: what you just implemented, the task title, and the commit SHA (via `git rev-parse HEAD`). Ask specifically whether the change matches the task scope, whether edge cases are handled, and whether the commit is clean.',
+    autoLoopGroomingIntro: GROOMING_INTRO_SUPERPOWERS,
+    qaPromptTemplate: 'QA pass for workspace "{{workspace_name}}" in project {{project_name}}.\n\nBranch: {{branch_name}}\nStaging URL: {{staging_url}}\n\nIf a QA-style skill that drives a real browser is available in this environment (e.g. via the superpowers-chrome browsing skill), use it to navigate the staging URL and exercise the changes. Otherwise, fall back to manually scripting the smoke checks and recording your findings as a bug report.',
+};
+export const GSTACK_PROMPTS = {
+    reviewTemplate: REVIEW_HEADER +
+        'Run /review to drive this audit (the gstack Staff Engineer skill — finds bugs that pass CI but blow up in production, auto-fixes the obvious ones, flags completeness gaps). If /review is unavailable in this environment, fall back to the manual checklist below.\n\n' +
+        REVIEW_BODY,
+    autoLoopReviewGate: 'Code review gate — BEFORE marking the task done, run /review (the gstack Staff Engineer skill). Brief it with: what you just implemented, the task title, and the commit SHA (via `git rev-parse HEAD`). Ask specifically whether the change matches the task scope, whether edge cases are handled, and whether the commit is clean. If /review auto-fixes minor issues, accept the fixes via an amend or fix-up commit, then re-run step 3 checks.',
+    autoLoopGroomingIntro: GROOMING_INTRO_GSTACK,
+    qaPromptTemplate: 'QA pass for workspace "{{workspace_name}}" in project {{project_name}}.\n\nBranch: {{branch_name}}\nStaging URL: {{staging_url}}\n\nRun /qa {{staging_url}} (gstack QA Lead skill — opens a real browser, clicks through flows, finds bugs, fixes them with atomic commits, generates regression tests). If you only want a bug report without code changes, use /qa-only {{staging_url}} instead.',
+};
+export const AGNOSTIC_PROMPTS = {
+    reviewTemplate: AGNOSTIC_REVIEW_TEMPLATE,
+    autoLoopReviewGate: AGNOSTIC_AUTO_LOOP_REVIEW_GATE,
+    autoLoopGroomingIntro: AGNOSTIC_AUTO_LOOP_GROOMING_INTRO,
+    qaPromptTemplate: AGNOSTIC_QA_PROMPT_TEMPLATE,
+};
+/**
+ * Resolve the suite prompts to use right now, given the global `skillSuite`
+ * and the four user-editable `custom*` fields (only consulted in `custom` mode).
+ * Empty-string or whitespace-only overrides fall back to AGNOSTIC defaults.
+ */
+export function getSuitePrompts(suite, overrides) {
+    if (suite === 'superpowers')
+        return SUPERPOWERS_PROMPTS;
+    if (suite === 'gstack')
+        return GSTACK_PROMPTS;
+    // custom mode: per-field fallback to AGNOSTIC when the override is missing/blank
+    const pick = (k) => {
+        const value = overrides[k];
+        if (typeof value !== 'string')
+            return AGNOSTIC_PROMPTS[k];
+        return value.trim() ? value : AGNOSTIC_PROMPTS[k];
+    };
+    return {
+        reviewTemplate: pick('reviewTemplate'),
+        autoLoopReviewGate: pick('autoLoopReviewGate'),
+        autoLoopGroomingIntro: pick('autoLoopGroomingIntro'),
+        qaPromptTemplate: pick('qaPromptTemplate'),
+    };
+}