@gempack/squad-mcp 0.5.0 → 0.6.1

package/dist/config/squad-yaml.js

@@ -0,0 +1,403 @@
+import { promises as fs } from "node:fs";
+import path from "node:path";
+import yaml from "js-yaml";
+import { z } from "zod";
+import { AGENT_NAMES_TUPLE, DEFAULT_RUBRIC_WEIGHTS, } from "./ownership-matrix.js";
+import { SquadError } from "../errors.js";
+import { logger } from "../observability/logger.js";
+/**
+ * `.squad.yaml` schema. All fields optional; absent keys fall back to package defaults.
+ *
+ * Repo-versioned config. Lives at workspace_root. Loaded by composers
+ * (compose_squad_workflow, compose_advisory_bundle) automatically when they
+ * receive a workspace_root. Pure tools (apply_consolidation_rules, score_rubric,
+ * select_squad) do NOT read it directly — that would make them stateful and
+ * couple their inputs to the filesystem. Composers do the read; pure tools
+ * receive the resolved values via parameters.
+ */
+const squadYamlSchema = z.object({
+    /**
+     * Override default rubric weights. Keys are agent names; values 0-100. The set
+     * of supplied keys MUST sum to 100 across the agents you list. Agents not
+     * listed fall back to the package default. Use weight 0 to ignore a dimension.
+     */
+    weights: z
+        .record(z.enum(AGENT_NAMES_TUPLE), z.number().min(0).max(100))
+        .optional(),
+    /**
+     * Per-dimension threshold (0-100). Below this, the dimension is flagged in the
+     * scorecard. Default 75 (in code, not duplicated here).
+     */
+    threshold: z.number().min(0).max(100).optional(),
+    /**
+     * Quality floor. If supplied AND the weighted score is below it AND verdict
+     * would otherwise be APPROVED, the consolidator downgrades to CHANGES_REQUIRED.
+     */
+    min_score: z.number().min(0).max(100).optional(),
+    /**
+     * Glob patterns (relative to workspace_root) of files to exclude from advisory.
+     * Matched against `changed_files` BEFORE squad selection. Use to silence
+     * docs-only changes, generated code, etc.
+     */
+    skip_paths: z.array(z.string().min(1).max(512)).max(200).optional(),
+    /**
+     * Agents to disable for this repo. Removed from the selected squad in
+     * compose_squad_workflow. Useful for repos that don't have, say, a database
+     * (disable senior-dba) or aren't user-facing (disable product-owner).
+     */
+    disable_agents: z.array(z.enum(AGENT_NAMES_TUPLE)).max(20).optional(),
+    /**
+     * Optional GitHub PR posting policy used by `tools/post-review.mjs` and the
+     * `/squad-review` skill. Default behaviour without this section: dry-run.
+     * The skill prepares the post and shows the user — never posts unsolicited.
+     *
+     * - `auto_post`: when true, the skill posts without asking for confirmation
+     *   each time. The user opted in by editing this YAML — no AI action without
+     *   prior authorisation.
+     * - `request_changes_below_score`: when set, an APPROVED verdict with
+     *   weighted score below this floor is posted as `--request-changes` instead
+     *   of `--approve`. Independent of `min_score` (which downgrades the verdict
+     *   itself).
+     * - `omit_attribution_footer`: omits the trailing "Generated by squad-mcp"
+     *   line from the PR body. Useful for orgs that prefer a leaner PR.
+     */
+    pr_posting: z
+        .object({
+        auto_post: z.boolean().optional(),
+        request_changes_below_score: z.number().min(0).max(100).optional(),
+        omit_attribution_footer: z.boolean().optional(),
+    })
+        .optional(),
+    /**
+     * Optional `.squad/learnings.jsonl` configuration. The store records
+     * accept/reject decisions on findings; future advisory runs read the
+     * recent tail and inject it into agent / consolidator prompts so the
+     * squad stops re-suggesting things the team has already declined.
+     *
+     * - `path`: relative location of the JSONL file (default
+     *   `.squad/learnings.jsonl`). Override only if the repo already uses
+     *   `.squad/` for something else.
+     * - `max_recent`: how many entries to inject per advisory run (default 50,
+     *   hard cap 200). Larger = more context but more prompt cost.
+     * - `enabled`: master switch. Default true. Set false to disable the
+     *   read/inject side without deleting the file.
+     */
+    learnings: z
+        .object({
+        path: z.string().min(1).max(512).optional(),
+        max_recent: z.number().int().positive().max(200).optional(),
+        enabled: z.boolean().optional(),
+    })
+        .optional(),
+    /**
+     * Optional `.squad/tasks.json` configuration. The store holds the task list
+     * decomposed from a PRD (or seeded manually); the squad workflow runs one
+     * task at a time with scope-narrowed context to avoid prompt bloat.
+     *
+     * - `path`: relative location of the JSON file (default `.squad/tasks.json`).
+     *   Override only if the repo already uses `.squad/` for something else.
+     * - `enabled`: master switch. Default true. Set false to silence task tools
+     *   without deleting the file.
+     */
+    tasks: z
+        .object({
+        path: z.string().min(1).max(512).optional(),
+        enabled: z.boolean().optional(),
+    })
+        .optional(),
+});
+const CANDIDATE_FILENAMES = [".squad.yaml", ".squad.yml"];
+const cache = new Map();
+/**
+ * Test-only: clear the cache. Production code MUST NOT call this.
+ */
+export function __resetSquadYamlCacheForTests() {
+    cache.clear();
+}
+async function locate(workspaceRoot) {
+    for (const name of CANDIDATE_FILENAMES) {
+        const candidate = path.join(workspaceRoot, name);
+        try {
+            const s = await fs.stat(candidate);
+            if (s.isFile()) {
+                return { filePath: candidate, mtimeMs: s.mtimeMs };
+            }
+        }
+        catch {
+            // ENOENT or stat failure — try next.
+        }
+    }
+    return null;
+}
+function applyDefaults(parsed, source) {
+    // Merge weights: parsed override sums to 100 across its keys (validated below);
+    // missing keys take the default. Returning a fully-populated map keeps the
+    // downstream score_rubric simple (no fallback chain in math).
+    const weights = { ...DEFAULT_RUBRIC_WEIGHTS };
+    if (parsed.weights) {
+        const supplied = Object.entries(parsed.weights);
+        const sum = supplied.reduce((acc, [, v]) => acc + v, 0);
+        if (Math.abs(sum - 100) > 0.01) {
+            throw new SquadError("INVALID_INPUT", `.squad.yaml weights must sum to 100 across the agents listed; got ${sum}`, { source: source ?? "<inline>", supplied: parsed.weights });
+        }
+        for (const [name, w] of supplied) {
+            weights[name] = w;
+        }
+        // Zero-out the agents NOT supplied — the YAML user explicitly chose a
+        // partial set of weights, so anything missing is "this dimension does not
+        // count for this repo". Keeping defaults would let ungated dimensions sneak
+        // back into the rollup.
+        const overridden = new Set(supplied.map(([name]) => name));
+        for (const name of AGENT_NAMES_TUPLE) {
+            if (!overridden.has(name))
+                weights[name] = 0;
+        }
+    }
+    const pr_posting = {
+        auto_post: parsed.pr_posting?.auto_post ?? false,
+        request_changes_below_score: parsed.pr_posting?.request_changes_below_score,
+        omit_attribution_footer: parsed.pr_posting?.omit_attribution_footer ?? false,
+    };
+    const learnings = {
+        path: parsed.learnings?.path ?? ".squad/learnings.jsonl",
+        max_recent: parsed.learnings?.max_recent ?? 50,
+        enabled: parsed.learnings?.enabled ?? true,
+    };
+    const tasks = {
+        path: parsed.tasks?.path ?? ".squad/tasks.json",
+        enabled: parsed.tasks?.enabled ?? true,
+    };
+    return {
+        weights,
+        threshold: parsed.threshold ?? 75,
+        min_score: parsed.min_score,
+        skip_paths: parsed.skip_paths ?? [],
+        disable_agents: parsed.disable_agents ?? [],
+        pr_posting,
+        learnings,
+        tasks,
+        source,
+    };
+}
+/**
+ * Read `.squad.yaml` (or `.squad.yml`) from workspace_root. Returns the resolved
+ * config with defaults filled in. If no file is present, returns defaults.
+ * Caches by absolute path + mtimeMs.
+ *
+ * Throws `SquadError(INVALID_INPUT)` only on malformed YAML or schema violations
+ * (e.g. weights not summing to 100). Missing-file and partial config are NOT
+ * errors — they fall back gracefully.
+ */
+export async function readSquadYaml(workspaceRoot) {
+    const absRoot = path.resolve(workspaceRoot);
+    const located = await locate(absRoot);
+    if (!located) {
+        // No file. Cache the "no file" sentinel keyed by mtimeMs=0.
+        const cached = cache.get(absRoot);
+        if (cached && cached.filePath === null && cached.mtimeMs === 0) {
+            return cached.resolved;
+        }
+        const resolved = applyDefaults({}, null);
+        cache.set(absRoot, { mtimeMs: 0, resolved, filePath: null });
+        return resolved;
+    }
+    const cached = cache.get(absRoot);
+    if (cached &&
+        cached.filePath === located.filePath &&
+        cached.mtimeMs === located.mtimeMs) {
+        return cached.resolved;
+    }
+    let raw;
+    try {
+        raw = await fs.readFile(located.filePath, "utf8");
+    }
+    catch (err) {
+        throw new SquadError("CONFIG_READ_FAILED", `failed to read ${located.filePath}: ${err.message}`, {
+            source: located.filePath,
+        });
+    }
+    let doc;
+    try {
+        doc = yaml.load(raw, {
+            filename: located.filePath,
+            schema: yaml.FAILSAFE_SCHEMA,
+        });
+    }
+    catch (err) {
+        throw new SquadError("INVALID_INPUT", `invalid YAML in ${located.filePath}: ${err.message}`, {
+            source: located.filePath,
+        });
+    }
+    // FAILSAFE_SCHEMA returns strings for everything; coerce numbers in known fields.
+    // Doing this here keeps the zod schema strict (numbers stay numbers) while
+    // tolerating YAML's "1" vs 1 ambiguity.
+    if (typeof doc === "object" && doc !== null) {
+        const o = doc;
+        if (o.threshold !== undefined)
+            o.threshold = coerceNumber(o.threshold, "threshold", located.filePath);
+        if (o.min_score !== undefined)
+            o.min_score = coerceNumber(o.min_score, "min_score", located.filePath);
+        if (o.weights && typeof o.weights === "object") {
+            const w = o.weights;
+            for (const [k, v] of Object.entries(w)) {
+                w[k] = coerceNumber(v, `weights.${k}`, located.filePath);
+            }
+        }
+        if (o.pr_posting && typeof o.pr_posting === "object") {
+            const p = o.pr_posting;
+            if (p.request_changes_below_score !== undefined) {
+                p.request_changes_below_score = coerceNumber(p.request_changes_below_score, "pr_posting.request_changes_below_score", located.filePath);
+            }
+            // FAILSAFE_SCHEMA returns booleans as the strings "true"/"false". Coerce.
+            if (typeof p.auto_post === "string") {
+                p.auto_post = p.auto_post === "true";
+            }
+            if (typeof p.omit_attribution_footer === "string") {
+                p.omit_attribution_footer = p.omit_attribution_footer === "true";
+            }
+        }
+        if (o.learnings && typeof o.learnings === "object") {
+            const l = o.learnings;
+            if (l.max_recent !== undefined) {
+                l.max_recent = coerceNumber(l.max_recent, "learnings.max_recent", located.filePath);
+            }
+            if (typeof l.enabled === "string") {
+                l.enabled = l.enabled === "true";
+            }
+        }
+    }
+    else if (doc === null || doc === undefined) {
+        // Empty YAML file — treat as empty object.
+        doc = {};
+    }
+    const parsed = squadYamlSchema.safeParse(doc);
+    if (!parsed.success) {
+        throw new SquadError("INVALID_INPUT", `${located.filePath}: ${parsed.error.message}`, {
+            source: located.filePath,
+            issues: parsed.error.issues.length,
+        });
+    }
+    const resolved = applyDefaults(parsed.data, located.filePath);
+    cache.set(absRoot, {
+        mtimeMs: located.mtimeMs,
+        resolved,
+        filePath: located.filePath,
+    });
+    logger.info("squad-yaml loaded", {
+        details: {
+            source: located.filePath,
+            has_weights: Boolean(parsed.data.weights),
+            threshold: resolved.threshold,
+            skip_paths_count: resolved.skip_paths.length,
+            disabled_agents_count: resolved.disable_agents.length,
+        },
+    });
+    return resolved;
+}
+function coerceNumber(value, field, source) {
+    if (typeof value === "number")
+        return value;
+    if (typeof value === "string") {
+        const n = Number(value);
+        if (!Number.isFinite(n)) {
+            throw new SquadError("INVALID_INPUT", `${source}: ${field} must be a number, got "${value}"`, { source });
+        }
+        return n;
+    }
+    throw new SquadError("INVALID_INPUT", `${source}: ${field} must be a number`, { source });
+}
+/**
+ * Glob-ish path matching used for skip_paths. Supports:
+ *   - double-star matches any number of segments (zero or more)
+ *   - single star matches anything within a single segment
+ *   - `?` matches one character within a segment
+ *   - exact strings
+ *
+ * Patterns and paths are matched after normalising path separators to `/`.
+ * NOT a full minimatch — kept tiny on purpose to avoid pulling minimatch as a
+ * dep. Sufficient for the canonical use cases (docs/double-star, double-star
+ * slash *.md, double-star slash generated slash *.ts, vendor/double-star).
+ */
+export function matchesGlob(pattern, p) {
+    const normPattern = pattern.replace(/\\/g, "/");
+    const normPath = p.replace(/\\/g, "/");
+    const re = globToRegExp(normPattern);
+    return re.test(normPath);
+}
+function globToRegExp(pattern) {
+    let i = 0;
+    let out = "^";
+    while (i < pattern.length) {
+        const c = pattern[i];
+        if (c === "*" && pattern[i + 1] === "*") {
+            // `**` — zero or more segments
+            // Handle `**/`, `/**`, `/**/`, or bare `**`.
+            if (pattern[i + 2] === "/") {
+                // `**/` — zero or more segments followed by /
+                out += "(?:.*/)?";
+                i += 3;
+                continue;
+            }
+            out += ".*";
+            i += 2;
+            continue;
+        }
+        if (c === "*") {
+            // single segment
+            out += "[^/]*";
+            i += 1;
+            continue;
+        }
+        if (c === "?") {
+            out += "[^/]";
+            i += 1;
+            continue;
+        }
+        if (".+^${}()|[]\\".includes(c)) {
+            out += "\\" + c;
+            i += 1;
+            continue;
+        }
+        out += c;
+        i += 1;
+    }
+    out += "$";
+    return new RegExp(out);
+}
+/**
+ * Apply `skip_paths` filter to a list of files. Returns { kept, skipped }.
+ * Skipped files are excluded from advisory; they still appear in the diff but
+ * agents don't see them.
+ */
+export function applySkipPaths(files, skipPaths) {
+    if (skipPaths.length === 0)
+        return { kept: files, skipped: [] };
+    const kept = [];
+    const skipped = [];
+    for (const f of files) {
+        if (skipPaths.some((p) => matchesGlob(p, f))) {
+            skipped.push(f);
+        }
+        else {
+            kept.push(f);
+        }
+    }
+    return { kept, skipped };
+}
+/**
+ * Filter a selected squad against `disable_agents`. Returns the agents that
+ * remain. Logs a warning if every agent was disabled (likely a misconfig).
+ */
+export function applyDisableAgents(squad, disableAgents) {
+    if (disableAgents.length === 0)
+        return squad;
+    const disabled = new Set(disableAgents);
+    const remaining = squad.filter((a) => !disabled.has(a));
+    if (remaining.length === 0 && squad.length > 0) {
+        logger.warn("squad-yaml disable_agents removed every selected agent", {
+            details: { selected: squad, disabled: disableAgents },
+        });
+    }
+    return remaining;
+}
+//# sourceMappingURL=squad-yaml.js.map

package/dist/config/squad-yaml.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"squad-yaml.js","sourceRoot":"","sources":["../../src/config/squad-yaml.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,SAAS,CAAC;AACzC,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,IAAI,MAAM,SAAS,CAAC;AAC3B,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EACL,iBAAiB,EAEjB,sBAAsB,GACvB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC1C,OAAO,EAAE,MAAM,EAAE,MAAM,4BAA4B,CAAC;AAEpD;;;;;;;;;GASG;AACH,MAAM,eAAe,GAAG,CAAC,CAAC,MAAM,CAAC;IAC/B;;;;OAIG;IACH,OAAO,EAAE,CAAC;SACP,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;SAC7D,QAAQ,EAAE;IACb;;;OAGG;IACH,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;IAChD;;;OAGG;IACH,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;IAChD;;;;OAIG;IACH,UAAU,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;IACnE;;;;OAIG;IACH,cAAc,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,QAAQ,EAAE;IACrE;;;;;;;;;;;;;;OAcG;IACH,UAAU,EAAE,CAAC;SACV,MAAM,CAAC;QACN,SAAS,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;QACjC,2BAA2B,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAClE,uBAAuB,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;KAChD,CAAC;SACD,QAAQ,EAAE;IACb;;;;;;;;;;;;;OAaG;IACH,SAAS,EAAE,CAAC;SACT,MAAM,CAAC;QACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAC3C,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAC3D,OAAO,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;KAChC,CAAC;SACD,QAAQ,EAAE;IACb;;;;;;;;;OASG;IACH,KAAK,EAAE,CAAC;SACL,MAAM,CAAC;QACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAC3C,OAAO,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE;KAChC,CAAC;SACD,QAAQ,EAAE;CACd,CAAC,CAAC;AAkDH,MAAM,mBAAmB,GAAG,CAAC,aAAa,EAAE,YAAY,CAAC,CAAC;AAa1D,MAAM,KAAK,GAAG,IAAI,GAAG,EAAsB,CAAC;AAE5C;;GAEG;AACH,MAAM,UAAU,6BAA6B;IAC3C,KAAK,CAAC,KAAK,EAAE,CAAC;AAChB,CAAC;AAED,KAAK,UAAU,MAAM,CACnB,aAAqB;IAErB,KAAK,MAAM,IAAI,IAAI,mBAAmB,EAAE,CAAC;QACvC,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC;QACjD,IAAI,CAAC;YACH,MAAM,CAAC,GAAG,MAAM,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACnC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC;gBACf,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,OAAO,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC;YACrD,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,qCAAqC;QACvC,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,aAAa,CACpB,MAAuB,EACvB,MAAqB;IAErB,gFAAgF;IAChF,2EAA2E;IAC3E,8DAA8D;IAC9D,MAAM,OAAO,GAA8B,EAAE,GAAG,sBAAsB,EAAE,CAAC;IACzE,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACnB,MAAM,QAAQ,GAAG,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAA0B,CAAC;QACzE,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;QACxD,IAAI,IAAI,CAAC,GAAG,CAAC,GAAG,GAAG,GAAG,CAAC,GAAG,IAAI,EAAE,CAAC;YAC/B,MAAM,IAAI,UAAU,CAClB,eAAe,EACf,qEAAqE,GAAG,EAAE,EAC1E,EAAE,MAAM,EAAE,MAAM,IAAI,UAAU,EAAE,QAAQ,EAAE,MAAM,CAAC,OAAO,EAAE,CAC3D,CAAC;QACJ,CAAC;QACD,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,QAAQ,EAAE,CAAC;YACjC,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACpB,CAAC;QACD,sEAAsE;QACtE,0EAA0E;QAC1E,4EAA4E;QAC5E,wBAAwB;QACxB,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,KAAK,MAAM,IAAI,IAAI,iBAAiB,EAAE,CAAC;YACrC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC;gBAAE,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC/C,CAAC;IACH,CAAC;IAED,MAAM,UAAU,GAAoB;QAClC,SAAS,EAAE,MAAM,CAAC,UAAU,EAAE,SAAS,IAAI,KAAK;QAChD,2BAA2B,EAAE,MAAM,CAAC,UAAU,EAAE,2BAA2B;QAC3E,uBAAuB,EACrB,MAAM,CAAC,UAAU,EAAE,uBAAuB,IAAI,KAAK;KACtD,CAAC;IAEF,MAAM,SAAS,GAAoB;QACjC,IAAI,EAAE,MAAM,CAAC,SAAS,EAAE,IAAI,IAAI,wBAAwB;QACxD,UAAU,EAAE,MAAM,CAAC,SAAS,EAAE,UAAU,IAAI,EAAE;QAC9C,OAAO,EAAE,MAAM,CAAC,SAAS,EAAE,OAAO,IAAI,IAAI;KAC3C,CAAC;IAEF,MAAM,KAAK,GAAgB;QACzB,IAAI,EAAE,MAAM,CAAC,KAAK,EAAE,IAAI,IAAI,mBAAmB;QAC/C,OAAO,EAAE,MAAM,CAAC,KAAK,EAAE,OAAO,IAAI,IAAI;KACvC,CAAC;IAEF,OAAO;QACL,OAAO;QACP,SAAS,EAAE,MAAM,CAAC,SAAS,IAAI,EAAE;QACjC,SAAS,EAAE,MAAM,CAAC,SAAS;QAC3B,UAAU,EAAE,MAAM,CAAC,UAAU,IAAI,EAAE;QACnC,cAAc,EAAE,MAAM,CAAC,cAAc,IAAI,EAAE;QAC3C,UAAU;QACV,SAAS;QACT,KAAK;QACL,MAAM;KACP,CAAC;AACJ,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,aAAqB;IAErB,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC;IAC5C,MAAM,OAAO,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC,CAAC;IAEtC,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,4DAA4D;QAC5D,MAAM,MAAM,GAAG,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAClC,IAAI,MAAM,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,MAAM,CAAC,OAAO,KAAK,CAAC,EAAE,CAAC;YAC/D,OAAO,MAAM,CAAC,QAAQ,CAAC;QACzB,CAAC;QACD,MAAM,QAAQ,GAAG,aAAa,CAAC,EAAE,EAAE,IAAI,CAAC,CAAC;QACzC,KAAK,CAAC,GAAG,CAAC,OAAO,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,QAAQ,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QAC7D,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,MAAM,MAAM,GAAG,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAClC,IACE,MAAM;QACN,MAAM,CAAC,QAAQ,KAAK,OAAO,CAAC,QAAQ;QACpC,MAAM,CAAC,OAAO,KAAK,OAAO,CAAC,OAAO,EAClC,CAAC;QACD,OAAO,MAAM,CAAC,QAAQ,CAAC;IACzB,CAAC;IAED,IAAI,GAAW,CAAC;IAChB,IAAI,CAAC;QACH,GAAG,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,UAAU,CAClB,oBAAoB,EACpB,kBAAkB,OAAO,CAAC,QAAQ,KAAM,GAAa,CAAC,OAAO,EAAE,EAC/D;YACE,MAAM,EAAE,OAAO,CAAC,QAAQ;SACzB,CACF,CAAC;IACJ,CAAC;IAED,IAAI,GAAY,CAAC;IACjB,IAAI,CAAC;QACH,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE;YACnB,QAAQ,EAAE,OAAO,CAAC,QAAQ;YAC1B,MAAM,EAAE,IAAI,CAAC,eAAe;SAC7B,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,UAAU,CAClB,eAAe,EACf,mBAAmB,OAAO,CAAC,QAAQ,KAAM,GAAa,CAAC,OAAO,EAAE,EAChE;YACE,MAAM,EAAE,OAAO,CAAC,QAAQ;SACzB,CACF,CAAC;IACJ,CAAC;IAED,kFAAkF;IAClF,2EAA2E;IAC3E,wCAAwC;IACxC,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,GAAG,KAAK,IAAI,EAAE,CAAC;QAC5C,MAAM,CAAC,GAAG,GAA8B,CAAC;QACzC,IAAI,CAAC,CAAC,SAAS,KAAK,SAAS;YAC3B,CAAC,CAAC,SAAS,GAAG,YAAY,CAAC,CAAC,CAAC,SAAS,EAAE,WAAW,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;QACzE,IAAI,CAAC,CAAC,SAAS,KAAK,SAAS;YAC3B,CAAC,CAAC,SAAS,GAAG,YAAY,CAAC,CAAC,CAAC,SAAS,EAAE,WAAW,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;QACzE,IAAI,CAAC,CAAC,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;YAC/C,MAAM,CAAC,GAAG,CAAC,CAAC,OAAkC,CAAC;YAC/C,KAAK,MAAM,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;gBACvC,CAAC,CAAC,CAAC,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,WAAW,CAAC,EAAE,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;YAC3D,CAAC;QACH,CAAC;QACD,IAAI,CAAC,CAAC,UAAU,IAAI,OAAO,CAAC,CAAC,UAAU,KAAK,QAAQ,EAAE,CAAC;YACrD,MAAM,CAAC,GAAG,CAAC,CAAC,UAAqC,CAAC;YAClD,IAAI,CAAC,CAAC,2BAA2B,KAAK,SAAS,EAAE,CAAC;gBAChD,CAAC,CAAC,2BAA2B,GAAG,YAAY,CAC1C,CAAC,CAAC,2BAA2B,EAC7B,wCAAwC,EACxC,OAAO,CAAC,QAAQ,CACjB,CAAC;YACJ,CAAC;YACD,0EAA0E;YAC1E,IAAI,OAAO,CAAC,CAAC,SAAS,KAAK,QAAQ,EAAE,CAAC;gBACpC,CAAC,CAAC,SAAS,GAAG,CAAC,CAAC,SAAS,KAAK,MAAM,CAAC;YACvC,CAAC;YACD,IAAI,OAAO,CAAC,CAAC,uBAAuB,KAAK,QAAQ,EAAE,CAAC;gBAClD,CAAC,CAAC,uBAAuB,GAAG,CAAC,CAAC,uBAAuB,KAAK,MAAM,CAAC;YACnE,CAAC;QACH,CAAC;QACD,IAAI,CAAC,CAAC,SAAS,IAAI,OAAO,CAAC,CAAC,SAAS,KAAK,QAAQ,EAAE,CAAC;YACnD,MAAM,CAAC,GAAG,CAAC,CAAC,SAAoC,CAAC;YACjD,IAAI,CAAC,CAAC,UAAU,KAAK,SAAS,EAAE,CAAC;gBAC/B,CAAC,CAAC,UAAU,GAAG,YAAY,CACzB,CAAC,CAAC,UAAU,EACZ,sBAAsB,EACtB,OAAO,CAAC,QAAQ,CACjB,CAAC;YACJ,CAAC;YACD,IAAI,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;gBAClC,CAAC,CAAC,OAAO,GAAG,CAAC,CAAC,OAAO,KAAK,MAAM,CAAC;YACnC,CAAC;QACH,CAAC;IACH,CAAC;SAAM,IAAI,GAAG,KAAK,IAAI,IAAI,GAAG,KAAK,SAAS,EAAE,CAAC;QAC7C,2CAA2C;QAC3C,GAAG,GAAG,EAAE,CAAC;IACX,CAAC;IAED,MAAM,MAAM,GAAG,eAAe,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAC9C,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;QACpB,MAAM,IAAI,UAAU,CAClB,eAAe,EACf,GAAG,OAAO,CAAC,QAAQ,KAAK,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,EAC9C;YACE,MAAM,EAAE,OAAO,CAAC,QAAQ;YACxB,MAAM,EAAE,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM;SACnC,CACF,CAAC;IACJ,CAAC;IAED,MAAM,QAAQ,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC9D,KAAK,CAAC,GAAG,CAAC,OAAO,EAAE;QACjB,OAAO,EAAE,OAAO,CAAC,OAAO;QACxB,QAAQ;QACR,QAAQ,EAAE,OAAO,CAAC,QAAQ;KAC3B,CAAC,CAAC;IAEH,MAAM,CAAC,IAAI,CAAC,mBAAmB,EAAE;QAC/B,OAAO,EAAE;YACP,MAAM,EAAE,OAAO,CAAC,QAAQ;YACxB,WAAW,EAAE,OAAO,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC;YACzC,SAAS,EAAE,QAAQ,CAAC,SAAS;YAC7B,gBAAgB,EAAE,QAAQ,CAAC,UAAU,CAAC,MAAM;YAC5C,qBAAqB,EAAE,QAAQ,CAAC,cAAc,CAAC,MAAM;SACtD;KACF,CAAC,CAAC;IAEH,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,YAAY,CAAC,KAAc,EAAE,KAAa,EAAE,MAAc;IACjE,IAAI,OAAO,KAAK,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC5C,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,CAAC,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC;QACxB,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC;YACxB,MAAM,IAAI,UAAU,CAClB,eAAe,EACf,GAAG,MAAM,KAAK,KAAK,2BAA2B,KAAK,GAAG,EACtD,EAAE,MAAM,EAAE,CACX,CAAC;QACJ,CAAC;QACD,OAAO,CAAC,CAAC;IACX,CAAC;IACD,MAAM,IAAI,UAAU,CAClB,eAAe,EACf,GAAG,MAAM,KAAK,KAAK,mBAAmB,EACtC,EAAE,MAAM,EAAE,CACX,CAAC;AACJ,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,WAAW,CAAC,OAAe,EAAE,CAAS;IACpD,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;IAChD,MAAM,QAAQ,GAAG,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;IACvC,MAAM,EAAE,GAAG,YAAY,CAAC,WAAW,CAAC,CAAC;IACrC,OAAO,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;AAC3B,CAAC;AAED,SAAS,YAAY,CAAC,OAAe;IACnC,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,IAAI,GAAG,GAAG,GAAG,CAAC;IACd,OAAO,CAAC,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,OAAO,CAAC,CAAC,CAAW,CAAC;QAC/B,IAAI,CAAC,KAAK,GAAG,IAAI,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;YACxC,+BAA+B;YAC/B,6CAA6C;YAC7C,IAAI,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;gBAC3B,8CAA8C;gBAC9C,GAAG,IAAI,UAAU,CAAC;gBAClB,CAAC,IAAI,CAAC,CAAC;gBACP,SAAS;YACX,CAAC;YACD,GAAG,IAAI,IAAI,CAAC;YACZ,CAAC,IAAI,CAAC,CAAC;YACP,SAAS;QACX,CAAC;QACD,IAAI,CAAC,KAAK,GAAG,EAAE,CAAC;YACd,iBAAiB;YACjB,GAAG,IAAI,OAAO,CAAC;YACf,CAAC,IAAI,CAAC,CAAC;YACP,SAAS;QACX,CAAC;QACD,IAAI,CAAC,KAAK,GAAG,EAAE,CAAC;YACd,GAAG,IAAI,MAAM,CAAC;YACd,CAAC,IAAI,CAAC,CAAC;YACP,SAAS;QACX,CAAC;QACD,IAAI,eAAe,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC;YAChC,GAAG,IAAI,IAAI,GAAG,CAAC,CAAC;YAChB,CAAC,IAAI,CAAC,CAAC;YACP,SAAS;QACX,CAAC;QACD,GAAG,IAAI,CAAC,CAAC;QACT,CAAC,IAAI,CAAC,CAAC;IACT,CAAC;IACD,GAAG,IAAI,GAAG,CAAC;IACX,OAAO,IAAI,MAAM,CAAC,GAAG,CAAC,CAAC;AACzB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,cAAc,CAC5B,KAAe,EACf,SAAmB;IAEnB,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC;IAChE,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,IAAI,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YAC7C,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;aAAM,CAAC;YACN,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACf,CAAC;IACH,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC;AAC3B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,kBAAkB,CAChC,KAAkB,EAClB,aAA0B;IAE1B,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAC7C,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,aAAa,CAAC,CAAC;IACxC,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;IACxD,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC/C,MAAM,CAAC,IAAI,CAAC,wDAAwD,EAAE;YACpE,OAAO,EAAE,EAAE,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAE,aAAa,EAAE;SACtD,CAAC,CAAC;IACL,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC"}

package/dist/errors.d.ts

@@ -1,4 +1,4 @@
-export type SquadErrorCode = 'PATH_TRAVERSAL_DENIED' | 'PATH_REQUIRES_WORKSPACE' | 'PATH_INVALID' | 'AGENT_DIR_MISSING' | 'UNKNOWN_AGENT' | 'OVERRIDE_REJECTED' | 'INVALID_INPUT' | 'INTERNAL_ERROR' | 'GIT_EXEC_DENIED' | 'GIT_EXEC_TIMEOUT' | 'GIT_NOT_FOUND' | 'GIT_OUTPUT_TOO_LARGE' | 'GIT_NOT_A_REPO';
+export type SquadErrorCode = "PATH_TRAVERSAL_DENIED" | "PATH_REQUIRES_WORKSPACE" | "PATH_INVALID" | "AGENT_DIR_MISSING" | "UNKNOWN_AGENT" | "OVERRIDE_REJECTED" | "INVALID_INPUT" | "INTERNAL_ERROR" | "GIT_EXEC_DENIED" | "GIT_EXEC_TIMEOUT" | "GIT_NOT_FOUND" | "GIT_OUTPUT_TOO_LARGE" | "GIT_NOT_A_REPO" | "CONFIG_READ_FAILED";
 export declare class SquadError extends Error {
     readonly code: SquadErrorCode;
     readonly details?: Record<string, unknown>;

package/dist/errors.js

@@ -3,7 +3,7 @@ export class SquadError extends Error {
     details;
     constructor(code, message, details) {
         super(message);
-        this.name = 'SquadError';
+        this.name = "SquadError";
         this.code = code;
         this.details = details;
     }

package/dist/errors.js.map

@@ -1 +1 @@
-{"version":3,"file":"errors.js","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAeA,MAAM,OAAO,UAAW,SAAQ,KAAK;IAC1B,IAAI,CAAiB;IACrB,OAAO,CAA2B;IAE3C,YAAY,IAAoB,EAAE,OAAe,EAAE,OAAiC;QAClF,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,YAAY,CAAC;QACzB,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC;QACjB,IAAI,CAAC,OAAO,GAAG,OAAO,CAAC;IACzB,CAAC;CACF;AAED,MAAM,UAAU,YAAY,CAAC,GAAY;IACvC,OAAO,GAAG,YAAY,UAAU,CAAC;AACnC,CAAC"}
+{"version":3,"file":"errors.js","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAgBA,MAAM,OAAO,UAAW,SAAQ,KAAK;IAC1B,IAAI,CAAiB;IACrB,OAAO,CAA2B;IAE3C,YACE,IAAoB,EACpB,OAAe,EACf,OAAiC;QAEjC,KAAK,CAAC,OAAO,CAAC,CAAC;QACf,IAAI,CAAC,IAAI,GAAG,YAAY,CAAC;QACzB,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC;QACjB,IAAI,CAAC,OAAO,GAAG,OAAO,CAAC;IACzB,CAAC;CACF;AAED,MAAM,UAAU,YAAY,CAAC,GAAY;IACvC,OAAO,GAAG,YAAY,UAAU,CAAC;AACnC,CAAC"}

package/dist/format/pr-review.d.ts

@@ -0,0 +1,61 @@
+import type { ConsolidationOutput } from "../tools/consolidate.js";
+/**
+ * Verdict → `gh pr review` action mapping. The CLI passes the chosen action as
+ * a flag; we compute it deterministically from verdict + score so the formatter
+ * stays pure (no env reads, no auth).
+ *
+ *   REJECTED                                 -> request-changes (blocks merge)
+ *   CHANGES_REQUIRED                         -> comment         (advisory, doesn't block)
+ *   APPROVED + downgraded_by_score=true      -> comment         (downgraded, treat like CR)
+ *   APPROVED + score < request_changes_below -> request-changes (project-policy gate)
+ *   APPROVED otherwise                       -> approve
+ *
+ * `request_changes_below_score` is opt-in via .squad.yaml.pr_posting; without
+ * it, APPROVED always maps to approve.
+ */
+export type GhReviewAction = "approve" | "comment" | "request-changes";
+export interface FormatPrReviewOptions {
+    /**
+     * Below this weighted score, force `request-changes` even when verdict is
+     * APPROVED. Lets a repo enforce a hard quality bar at PR review time.
+     * Default: undefined (no override — APPROVED stays APPROVED).
+     */
+    requestChangesBelowScore?: number;
+    /**
+     * Optional repo identifier shown in the review header (e.g. "ggemba/squad-mcp").
+     * Cosmetic — surface where the review came from when the PR aggregates many
+     * sources.
+     */
+    repoLabel?: string;
+}
+export interface PrReviewPayload {
+    /** Markdown body to pass to `gh pr review --body-file -`. */
+    body: string;
+    /** Which `gh pr review --<action>` flag to invoke. */
+    action: GhReviewAction;
+    /** Short one-line summary suitable for `gh pr review --body` if you want a fallback. */
+    summary: string;
+}
+/**
+ * Compute the `gh pr review` action for the given consolidation output and
+ * options. Pure function; deterministic.
+ *
+ * Promotion never happens — a low-severity verdict cannot become approval.
+ * Demotion (score-driven) only applies to APPROVED.
+ */
+export declare function chooseGhAction(consolidation: ConsolidationOutput, options: FormatPrReviewOptions): GhReviewAction;
+/**
+ * Build the PR review payload from a consolidation output. Pure, deterministic.
+ *
+ * Output shape mirrors the squad's terminal report, repurposed as a markdown PR
+ * review body:
+ *   1. Header with verdict + weighted score
+ *   2. Rubric scorecard (fenced code block — keeps the bars monospace)
+ *   3. Findings grouped by agent (Blockers + unjustified Majors expanded;
+ *      Minor/Suggestion totals only)
+ *   4. Footer with attribution + override hint
+ *
+ * The action is chosen via `chooseGhAction` — caller passes it to
+ * `gh pr review --<action>`.
+ */
+export declare function formatPrReview(consolidation: ConsolidationOutput, options?: FormatPrReviewOptions): PrReviewPayload;

package/dist/format/pr-review.js

@@ -0,0 +1,146 @@
+const SEVERITY_ORDER = ["Blocker", "Major", "Minor", "Suggestion"];
+/**
+ * Compute the `gh pr review` action for the given consolidation output and
+ * options. Pure function; deterministic.
+ *
+ * Promotion never happens — a low-severity verdict cannot become approval.
+ * Demotion (score-driven) only applies to APPROVED.
+ */
+export function chooseGhAction(consolidation, options) {
+    if (consolidation.verdict === "REJECTED")
+        return "request-changes";
+    if (consolidation.verdict === "CHANGES_REQUIRED")
+        return "comment";
+    // APPROVED. Check downgrade signals.
+    if (consolidation.downgraded_by_score)
+        return "comment";
+    if (consolidation.rubric &&
+        typeof options.requestChangesBelowScore === "number" &&
+        consolidation.rubric.weighted_score < options.requestChangesBelowScore) {
+        return "request-changes";
+    }
+    return "approve";
+}
+function verdictHeader(consolidation) {
+    const r = consolidation.rubric;
+    const v = consolidation.verdict;
+    if (r) {
+        const score = r.weighted_score.toFixed(1);
+        if (v === "APPROVED" &&
+            r.passes_threshold &&
+            !consolidation.downgraded_by_score) {
+            return `Squad Advisory: APPROVED (${score} / 100)`;
+        }
+        if (v === "APPROVED") {
+            return `Squad Advisory: APPROVED with attention (${score} / 100)`;
+        }
+        return `Squad Advisory: ${v} (${score} / 100)`;
+    }
+    return `Squad Advisory: ${v}`;
+}
+function groupFindingsByAgent(consolidation) {
+    // Consolidator output retains blocker/major lists with agent attribution; we
+    // need the full set including Minor/Suggestion. The current ConsolidationOutput
+    // shape only exposes blockers + majors_unjustified explicitly; severity_counts
+    // is aggregate. To produce a per-agent listing of all findings, we'd need the
+    // raw reports. The skill is responsible for passing that input — but post-review
+    // CLI receives the consolidator output. We work with what's exposed here, which
+    // is enough for the v1 PR comment: blockers + unjustified majors are the actionable
+    // items. Minor/Suggestion counts go in the summary.
+    const grouped = new Map();
+    for (const b of consolidation.blockers) {
+        const list = grouped.get(b.agent) ?? [];
+        list.push({ severity: "Blocker", title: b.title });
+        grouped.set(b.agent, list);
+    }
+    for (const m of consolidation.majors_unjustified) {
+        const list = grouped.get(m.agent) ?? [];
+        list.push({ severity: "Major", title: m.title });
+        grouped.set(m.agent, list);
+    }
+    return grouped;
+}
+function formatFindingsSection(consolidation) {
+    const grouped = groupFindingsByAgent(consolidation);
+    const c = consolidation.severity_counts;
+    const totalFindings = c.Blocker + c.Major + c.Minor + c.Suggestion;
+    // Clean review with zero of every severity — skip the section entirely.
+    if (totalFindings === 0)
+        return "";
+    const totals = `**Severity totals:** ${c.Blocker} blocker / ${c.Major} major / ${c.Minor} minor / ${c.Suggestion} suggestion`;
+    // No actionable findings (no Blockers, no unjustified Majors) but Minor/Suggestion
+    // counts exist. Emit just the totals — no per-agent expansion needed.
+    if (grouped.size === 0) {
+        return ["### Findings", "", totals].join("\n");
+    }
+    const sections = [];
+    // Sort agents alphabetically for stable output (snapshot-friendly).
+    const agentNames = Array.from(grouped.keys()).sort();
+    for (const agent of agentNames) {
+        const findings = grouped.get(agent) ?? [];
+        findings.sort((a, b) => SEVERITY_ORDER.indexOf(a.severity) - SEVERITY_ORDER.indexOf(b.severity));
+        const lines = [`#### ${agent}`];
+        for (const f of findings) {
+            lines.push(`- **${f.severity}** — ${f.title}`);
+        }
+        sections.push(lines.join("\n"));
+    }
+    return ["### Findings", "", sections.join("\n\n"), "", totals].join("\n");
+}
+function formatRubricBlock(rubric) {
+    if (!rubric)
+        return "";
+    // Wrap the existing scorecard_text in a fenced block so the bars and ⚠ flags
+    // render as monospace in the GitHub PR review body.
+    return ["```", rubric.scorecard_text, "```"].join("\n");
+}
+function formatFooter(consolidation, action, options) {
+    const lines = [];
+    lines.push("---");
+    if (consolidation.downgraded_by_score) {
+        lines.push(`_Verdict downgraded from APPROVED to CHANGES_REQUIRED because the weighted score is below the configured floor._`);
+    }
+    if (action === "request-changes" && consolidation.verdict === "APPROVED") {
+        lines.push(`_Posting as request-changes because score is below \`request_changes_below_score\` (${options.requestChangesBelowScore})._`);
+    }
+    const suffix = options.repoLabel ? ` (\`${options.repoLabel}\`)` : "";
+    lines.push(`_Generated by [@gempack/squad-mcp](https://github.com/ggemba/squad-mcp)${suffix}. Each agent reviewed only its sliced jurisdiction; the rubric is the weighted rollup. Tune \`.squad.yaml\` to override weights, threshold, or skip paths._`);
+    return lines.join("\n\n");
+}
+/**
+ * Build the PR review payload from a consolidation output. Pure, deterministic.
+ *
+ * Output shape mirrors the squad's terminal report, repurposed as a markdown PR
+ * review body:
+ *   1. Header with verdict + weighted score
+ *   2. Rubric scorecard (fenced code block — keeps the bars monospace)
+ *   3. Findings grouped by agent (Blockers + unjustified Majors expanded;
+ *      Minor/Suggestion totals only)
+ *   4. Footer with attribution + override hint
+ *
+ * The action is chosen via `chooseGhAction` — caller passes it to
+ * `gh pr review --<action>`.
+ */
+export function formatPrReview(consolidation, options = {}) {
+    const action = chooseGhAction(consolidation, options);
+    const sections = [];
+    sections.push(`## ${verdictHeader(consolidation)}`);
+    const rubricBlock = formatRubricBlock(consolidation.rubric);
+    if (rubricBlock)
+        sections.push(rubricBlock);
+    const findings = formatFindingsSection(consolidation);
+    if (findings)
+        sections.push(findings);
+    sections.push(formatFooter(consolidation, action, options));
+    const body = sections.join("\n\n") + "\n";
+    // Single-line summary for fallbacks. Capped at ~200 chars — gh accepts more,
+    // but PR review bodies that go in `--body` instead of `--body-file` get
+    // shell-quoted and long lines invite escaping bugs.
+    const c = consolidation.severity_counts;
+    const scoreSegment = consolidation.rubric
+        ? ` | score ${consolidation.rubric.weighted_score.toFixed(1)}/100`
+        : "";
+    const summary = `Squad: ${consolidation.verdict}${scoreSegment} | ${c.Blocker}B/${c.Major}M/${c.Minor}m/${c.Suggestion}s`;
+    return { body, action, summary };
+}
+//# sourceMappingURL=pr-review.js.map

package/dist/format/pr-review.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-review.js","sourceRoot":"","sources":["../../src/format/pr-review.ts"],"names":[],"mappings":"AA2CA,MAAM,cAAc,GAAe,CAAC,SAAS,EAAE,OAAO,EAAE,OAAO,EAAE,YAAY,CAAC,CAAC;AAE/E;;;;;;GAMG;AACH,MAAM,UAAU,cAAc,CAC5B,aAAkC,EAClC,OAA8B;IAE9B,IAAI,aAAa,CAAC,OAAO,KAAK,UAAU;QAAE,OAAO,iBAAiB,CAAC;IACnE,IAAI,aAAa,CAAC,OAAO,KAAK,kBAAkB;QAAE,OAAO,SAAS,CAAC;IAEnE,qCAAqC;IACrC,IAAI,aAAa,CAAC,mBAAmB;QAAE,OAAO,SAAS,CAAC;IAExD,IACE,aAAa,CAAC,MAAM;QACpB,OAAO,OAAO,CAAC,wBAAwB,KAAK,QAAQ;QACpD,aAAa,CAAC,MAAM,CAAC,cAAc,GAAG,OAAO,CAAC,wBAAwB,EACtE,CAAC;QACD,OAAO,iBAAiB,CAAC;IAC3B,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,aAAa,CAAC,aAAkC;IACvD,MAAM,CAAC,GAAG,aAAa,CAAC,MAAM,CAAC;IAC/B,MAAM,CAAC,GAAG,aAAa,CAAC,OAAO,CAAC;IAChC,IAAI,CAAC,EAAE,CAAC;QACN,MAAM,KAAK,GAAG,CAAC,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;QAC1C,IACE,CAAC,KAAK,UAAU;YAChB,CAAC,CAAC,gBAAgB;YAClB,CAAC,aAAa,CAAC,mBAAmB,EAClC,CAAC;YACD,OAAO,6BAA6B,KAAK,SAAS,CAAC;QACrD,CAAC;QACD,IAAI,CAAC,KAAK,UAAU,EAAE,CAAC;YACrB,OAAO,4CAA4C,KAAK,SAAS,CAAC;QACpE,CAAC;QACD,OAAO,mBAAmB,CAAC,KAAK,KAAK,SAAS,CAAC;IACjD,CAAC;IACD,OAAO,mBAAmB,CAAC,EAAE,CAAC;AAChC,CAAC;AAED,SAAS,oBAAoB,CAC3B,aAAkC;IAElC,6EAA6E;IAC7E,gFAAgF;IAChF,+EAA+E;IAC/E,8EAA8E;IAC9E,iFAAiF;IACjF,gFAAgF;IAChF,oFAAoF;IACpF,oDAAoD;IACpD,MAAM,OAAO,GAAG,IAAI,GAAG,EAAmD,CAAC;IAC3E,KAAK,MAAM,CAAC,IAAI,aAAa,CAAC,QAAQ,EAAE,CAAC;QACvC,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QACxC,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,CAAC,KAAK,EAAE,CAAC,CAAC;QACnD,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;IAC7B,CAAC;IACD,KAAK,MAAM,CAAC,IAAI,aAAa,CAAC,kBAAkB,EAAE,CAAC;QACjD,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QACxC,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC,KAAK,EAAE,CAAC,CAAC;QACjD,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;IAC7B,CAAC;IACD,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,qBAAqB,CAAC,aAAkC;IAC/D,MAAM,OAAO,GAAG,oBAAoB,CAAC,aAAa,CAAC,CAAC;IACpD,MAAM,CAAC,GAAG,aAAa,CAAC,eAAe,CAAC;IACxC,MAAM,aAAa,GAAG,CAAC,CAAC,OAAO,GAAG,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,UAAU,CAAC;IAEnE,wEAAwE;IACxE,IAAI,aAAa,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAEnC,MAAM,MAAM,GAAG,wBAAwB,CAAC,CAAC,OAAO,cAAc,CAAC,CAAC,KAAK,YAAY,CAAC,CAAC,KAAK,YAAY,CAAC,CAAC,UAAU,aAAa,CAAC;IAE9H,mFAAmF;IACnF,sEAAsE;IACtE,IAAI,OAAO,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;QACvB,OAAO,CAAC,cAAc,EAAE,EAAE,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACjD,CAAC;IAED,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,oEAAoE;IACpE,MAAM,UAAU,GAAG,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACrD,KAAK,MAAM,KAAK,IAAI,UAAU,EAAE,CAAC;QAC/B,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QAC1C,QAAQ,CAAC,IAAI,CACX,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CACP,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,CAC1E,CAAC;QACF,MAAM,KAAK,GAAG,CAAC,QAAQ,KAAK,EAAE,CAAC,CAAC;QAChC,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;YACzB,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,QAAQ,QAAQ,CAAC,CAAC,KAAK,EAAE,CAAC,CAAC;QACjD,CAAC;QACD,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAClC,CAAC;IAED,OAAO,CAAC,cAAc,EAAE,EAAE,EAAE,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5E,CAAC;AAED,SAAS,iBAAiB,CAAC,MAA2B;IACpD,IAAI,CAAC,MAAM;QAAE,OAAO,EAAE,CAAC;IACvB,6EAA6E;IAC7E,oDAAoD;IACpD,OAAO,CAAC,KAAK,EAAE,MAAM,CAAC,cAAc,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1D,CAAC;AAED,SAAS,YAAY,CACnB,aAAkC,EAClC,MAAsB,EACtB,OAA8B;IAE9B,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,IAAI,aAAa,CAAC,mBAAmB,EAAE,CAAC;QACtC,KAAK,CAAC,IAAI,CACR,kHAAkH,CACnH,CAAC;IACJ,CAAC;IACD,IAAI,MAAM,KAAK,iBAAiB,IAAI,aAAa,CAAC,OAAO,KAAK,UAAU,EAAE,CAAC;QACzE,KAAK,CAAC,IAAI,CACR,uFAAuF,OAAO,CAAC,wBAAwB,KAAK,CAC7H,CAAC;IACJ,CAAC;IACD,MAAM,MAAM,GAAG,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,OAAO,CAAC,SAAS,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC;IACtE,KAAK,CAAC,IAAI,CACR,0EAA0E,MAAM,6JAA6J,CAC9O,CAAC;IACF,OAAO,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAC5B,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,cAAc,CAC5B,aAAkC,EAClC,UAAiC,EAAE;IAEnC,MAAM,MAAM,GAAG,cAAc,CAAC,aAAa,EAAE,OAAO,CAAC,CAAC;IAEtD,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,QAAQ,CAAC,IAAI,CAAC,MAAM,aAAa,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC;IAEpD,MAAM,WAAW,GAAG,iBAAiB,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC;IAC5D,IAAI,WAAW;QAAE,QAAQ,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAE5C,MAAM,QAAQ,GAAG,qBAAqB,CAAC,aAAa,CAAC,CAAC;IACtD,IAAI,QAAQ;QAAE,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAEtC,QAAQ,CAAC,IAAI,CAAC,YAAY,CAAC,aAAa,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IAE5D,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,IAAI,CAAC;IAE1C,6EAA6E;IAC7E,wEAAwE;IACxE,oDAAoD;IACpD,MAAM,CAAC,GAAG,aAAa,CAAC,eAAe,CAAC;IACxC,MAAM,YAAY,GAAG,aAAa,CAAC,MAAM;QACvC,CAAC,CAAC,YAAY,aAAa,CAAC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM;QAClE,CAAC,CAAC,EAAE,CAAC;IACP,MAAM,OAAO,GAAG,UAAU,aAAa,CAAC,OAAO,GAAG,YAAY,MAAM,CAAC,CAAC,OAAO,KAAK,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,UAAU,GAAG,CAAC;IAE1H,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;AACnC,CAAC"}