@gempack/squad-mcp 0.7.0 → 0.8.0

package/dist/tools/mode/exec-mode.d.ts

@@ -0,0 +1,124 @@
+import type { AgentName, WorkType } from "../../config/ownership-matrix.js";
+/**
+ * Execution depth resolution and squad shaping.
+ *
+ * `selectMode` picks one of `quick` / `normal` / `deep` from classify+risk
+ * signals (or honours the user's flag). `shapeSquadForMode` adjusts the
+ * agent list per the resolved mode.
+ *
+ * Lives in its own module so the pipeline orchestrator
+ * (`compose-squad-workflow.ts`) stays focused on its single job (detect →
+ * classify → score → select → shape). A future `.squad.yaml`-configurable
+ * threshold lands here, not in the pipeline.
+ */
+/**
+ * Auto-detect threshold for `mode: "quick"`. Surfaced as a named constant so
+ * a future `.squad.yaml` override (Sprint 2 / v0.8.1) has a single hook.
+ *
+ * Note: an earlier draft also gated on a `loc_changed` heuristic, but the
+ * heuristic was tautological with this file-count cap and could be foot-gunned
+ * by single-file giant rewrites. Removed in favour of a real `git diff --numstat`
+ * once we need finer granularity — track in the v0.8.1 follow-up.
+ */
+export declare const QUICK_AUTO_MAX_FILES = 5;
+/**
+ * Execution depth. PUBLIC STABLE CONTRACT from v0.8.0 — downstream
+ * `.squad.yaml` consumers and CI integrations may key on this field.
+ *
+ *  - `quick`: cap squad to 2 agents (top-2 from the ranked `selectSquad`
+ *    output, with `senior-developer` always force-included as a tiebreaker
+ *    for code-touching work types). Skip tech-lead-planner and
+ *    tech-lead-consolidator personas. `apply_consolidation_rules` still runs.
+ *  - `normal`: zero behavioural change vs. pre-v0.8.0. The implicit default.
+ *  - `deep`: force-include `senior-architect` + `senior-dev-security`.
+ *    Reject-loop cap raised from 2 to 3 iterations. Codex round suggested
+ *    (still gated on `--codex` consent).
+ *
+ * Auto-detection fires only when neither `--quick` nor `--deep` is passed.
+ * User flag always wins. See `selectMode` for the rule.
+ */
+export type ExecMode = "quick" | "normal" | "deep";
+export declare const EXEC_MODES: readonly ["quick", "normal", "deep"];
+/**
+ * Named constants for the agents the mode logic explicitly cares about.
+ * Exported so tests can assert against them by name instead of by string
+ * literal — if the role names ever change, the tests fail in the right
+ * place rather than silently passing on the new literal.
+ */
+export declare const TIEBREAKER_AGENT: AgentName;
+export declare const FALLBACK_SECONDARY: AgentName;
+export declare const DEEP_REQUIRED: readonly AgentName[];
+/**
+ * `mode_warning` codes. Stable from v0.8.0. CI integrations can differentiate
+ * warning types without regex-parsing the human message.
+ */
+export type ModeWarningCode = "forced_quick_on_high_risk" | "force_agents_truncated";
+export interface ModeWarning {
+    code: ModeWarningCode;
+    message: string;
+}
+interface RiskSignals {
+    touches_auth: boolean;
+    touches_money: boolean;
+    touches_migration: boolean;
+    files_count: number;
+    new_module?: boolean;
+    api_contract_change?: boolean;
+}
+/**
+ * Decide execution depth from classify+risk signals. User-supplied `mode`
+ * always wins (returned with `source: "user"`). Auto-detect rules:
+ *
+ *   deep  ← riskLevel == High || workType == Security || touches_migration
+ *   quick ← riskLevel == Low
+ *           && files_count <= QUICK_AUTO_MAX_FILES
+ *           && !touches_auth && !touches_money && !touches_migration
+ *           && workType != Security
+ *   normal← everything else
+ */
+export declare function selectMode(args: {
+    userMode: ExecMode | undefined;
+    riskLevel: "Low" | "Medium" | "High";
+    workType: WorkType;
+    signals: RiskSignals;
+}): {
+    mode: ExecMode;
+    source: "user" | "auto";
+    warning?: ModeWarning;
+};
+/**
+ * Cap or expand the agent list according to execution depth.
+ *
+ *  - `quick`: take the top-2 agents from `selectedAgents` (which arrives
+ *    rank-ordered from selectSquad: core matrix → signals → user
+ *    force_agents). `senior-developer` is force-included as a tiebreaker
+ *    when the work_type is code-touching (anything other than
+ *    `Business Rule`) — covers the Refactor/Performance path where
+ *    `selectSquad` may not pick `senior-developer` from the matrix but
+ *    the user still wants code-review eyes.
+ *    When the user forced `--quick` on a high-risk diff (signalled via
+ *    `signals.touches_auth/money/migration`), `senior-dev-security` is
+ *    force-included as one of the two regardless.
+ *  - `normal`: pass through unchanged.
+ *  - `deep`: force-include `DEEP_REQUIRED` (architect + security) even if
+ *    `selectSquad` did not pick them.
+ *
+ * Precedence inside the 2-agent cap for `quick` (highest first):
+ *   1. Safety override (`senior-dev-security` on forced-quick-on-high-risk).
+ *   2. `userForcedAgents` (caller intent).
+ *   3. Tiebreaker `senior-developer` (code-touching work types).
+ *   4. Top of `selectedAgents` ranked order.
+ *   5. `FALLBACK_SECONDARY` last resort to guarantee 2 agents.
+ *
+ * Returns the (potentially) reshaped list and a `truncationWarning` when
+ * the user's `force_agents` exceeded the cap-to-2 and slots were dropped.
+ */
+export declare function shapeSquadForMode(selectedAgents: AgentName[], mode: ExecMode, ctx: {
+    workType: WorkType;
+    signals: RiskSignals;
+    userForcedAgents: readonly AgentName[];
+}): {
+    agents: AgentName[];
+    warning?: ModeWarning;
+};
+export {};

package/dist/tools/mode/exec-mode.js

@@ -0,0 +1,153 @@
+/**
+ * Execution depth resolution and squad shaping.
+ *
+ * `selectMode` picks one of `quick` / `normal` / `deep` from classify+risk
+ * signals (or honours the user's flag). `shapeSquadForMode` adjusts the
+ * agent list per the resolved mode.
+ *
+ * Lives in its own module so the pipeline orchestrator
+ * (`compose-squad-workflow.ts`) stays focused on its single job (detect →
+ * classify → score → select → shape). A future `.squad.yaml`-configurable
+ * threshold lands here, not in the pipeline.
+ */
+/**
+ * Auto-detect threshold for `mode: "quick"`. Surfaced as a named constant so
+ * a future `.squad.yaml` override (Sprint 2 / v0.8.1) has a single hook.
+ *
+ * Note: an earlier draft also gated on a `loc_changed` heuristic, but the
+ * heuristic was tautological with this file-count cap and could be foot-gunned
+ * by single-file giant rewrites. Removed in favour of a real `git diff --numstat`
+ * once we need finer granularity — track in the v0.8.1 follow-up.
+ */
+export const QUICK_AUTO_MAX_FILES = 5;
+export const EXEC_MODES = ["quick", "normal", "deep"];
+/**
+ * Named constants for the agents the mode logic explicitly cares about.
+ * Exported so tests can assert against them by name instead of by string
+ * literal — if the role names ever change, the tests fail in the right
+ * place rather than silently passing on the new literal.
+ */
+export const TIEBREAKER_AGENT = "senior-developer";
+export const FALLBACK_SECONDARY = "senior-qa";
+export const DEEP_REQUIRED = ["senior-architect", "senior-dev-security"];
+/**
+ * Decide execution depth from classify+risk signals. User-supplied `mode`
+ * always wins (returned with `source: "user"`). Auto-detect rules:
+ *
+ *   deep  ← riskLevel == High || workType == Security || touches_migration
+ *   quick ← riskLevel == Low
+ *           && files_count <= QUICK_AUTO_MAX_FILES
+ *           && !touches_auth && !touches_money && !touches_migration
+ *           && workType != Security
+ *   normal← everything else
+ */
+export function selectMode(args) {
+    const isHighRiskShape = args.riskLevel === "High" ||
+        args.workType === "Security" ||
+        args.signals.touches_auth ||
+        args.signals.touches_money ||
+        args.signals.touches_migration;
+    if (args.userMode !== undefined) {
+        if (args.userMode === "quick" && isHighRiskShape) {
+            return {
+                mode: "quick",
+                source: "user",
+                warning: {
+                    code: "forced_quick_on_high_risk",
+                    message: "user forced --quick on a high-risk diff; senior-dev-security force-included in the 2-agent cap as a safety override",
+                },
+            };
+        }
+        return { mode: args.userMode, source: "user" };
+    }
+    if (isHighRiskShape)
+        return { mode: "deep", source: "auto" };
+    if (args.riskLevel === "Low" &&
+        args.signals.files_count <= QUICK_AUTO_MAX_FILES &&
+        args.workType !== "Security") {
+        return { mode: "quick", source: "auto" };
+    }
+    return { mode: "normal", source: "auto" };
+}
+/**
+ * Cap or expand the agent list according to execution depth.
+ *
+ *  - `quick`: take the top-2 agents from `selectedAgents` (which arrives
+ *    rank-ordered from selectSquad: core matrix → signals → user
+ *    force_agents). `senior-developer` is force-included as a tiebreaker
+ *    when the work_type is code-touching (anything other than
+ *    `Business Rule`) — covers the Refactor/Performance path where
+ *    `selectSquad` may not pick `senior-developer` from the matrix but
+ *    the user still wants code-review eyes.
+ *    When the user forced `--quick` on a high-risk diff (signalled via
+ *    `signals.touches_auth/money/migration`), `senior-dev-security` is
+ *    force-included as one of the two regardless.
+ *  - `normal`: pass through unchanged.
+ *  - `deep`: force-include `DEEP_REQUIRED` (architect + security) even if
+ *    `selectSquad` did not pick them.
+ *
+ * Precedence inside the 2-agent cap for `quick` (highest first):
+ *   1. Safety override (`senior-dev-security` on forced-quick-on-high-risk).
+ *   2. `userForcedAgents` (caller intent).
+ *   3. Tiebreaker `senior-developer` (code-touching work types).
+ *   4. Top of `selectedAgents` ranked order.
+ *   5. `FALLBACK_SECONDARY` last resort to guarantee 2 agents.
+ *
+ * Returns the (potentially) reshaped list and a `truncationWarning` when
+ * the user's `force_agents` exceeded the cap-to-2 and slots were dropped.
+ */
+export function shapeSquadForMode(selectedAgents, mode, ctx) {
+    if (mode === "normal")
+        return { agents: selectedAgents };
+    if (mode === "deep") {
+        const out = [...selectedAgents];
+        for (const required of DEEP_REQUIRED) {
+            if (!out.includes(required))
+                out.push(required);
+        }
+        return { agents: out };
+    }
+    // mode === "quick"
+    const highRiskShape = ctx.signals.touches_auth || ctx.signals.touches_money || ctx.signals.touches_migration;
+    const seen = new Set();
+    const picked = [];
+    const push = (a) => {
+        if (picked.length >= 2)
+            return false;
+        if (seen.has(a))
+            return false;
+        seen.add(a);
+        picked.push(a);
+        return true;
+    };
+    // (1) Safety override.
+    if (highRiskShape)
+        push("senior-dev-security");
+    // (2) User-forced agents — preserve their intent within the cap.
+    let droppedForced = 0;
+    for (const a of ctx.userForcedAgents) {
+        if (!push(a) && !seen.has(a))
+            droppedForced += 1;
+    }
+    // (3) Tiebreaker for code-touching work types: senior-developer is the most
+    // general reviewer; force-include even if the matrix did not pick it.
+    // Skip only for Business Rule (PO-owned, not code-centric).
+    if (ctx.workType !== "Business Rule")
+        push(TIEBREAKER_AGENT);
+    // (4) Fill remaining slot(s) from the ranked list.
+    for (const a of selectedAgents)
+        push(a);
+    // (5) Last-resort fallback to guarantee 2 agents even with pathological
+    // upstream (e.g. selectSquad returned an empty list).
+    if (picked.length < 2)
+        push(FALLBACK_SECONDARY);
+    const result = { agents: picked };
+    if (droppedForced > 0) {
+        result.warning = {
+            code: "force_agents_truncated",
+            message: `quick mode caps the squad at 2 agents; ${droppedForced} of the user's force_agents were dropped`,
+        };
+    }
+    return result;
+}
+//# sourceMappingURL=exec-mode.js.map

package/dist/tools/mode/exec-mode.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"exec-mode.js","sourceRoot":"","sources":["../../../src/tools/mode/exec-mode.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;GAWG;AAEH;;;;;;;;GAQG;AACH,MAAM,CAAC,MAAM,oBAAoB,GAAG,CAAC,CAAC;AAmBtC,MAAM,CAAC,MAAM,UAAU,GAAG,CAAC,OAAO,EAAE,QAAQ,EAAE,MAAM,CAAU,CAAC;AAE/D;;;;;GAKG;AACH,MAAM,CAAC,MAAM,gBAAgB,GAAc,kBAAkB,CAAC;AAC9D,MAAM,CAAC,MAAM,kBAAkB,GAAc,WAAW,CAAC;AACzD,MAAM,CAAC,MAAM,aAAa,GAAyB,CAAC,kBAAkB,EAAE,qBAAqB,CAAC,CAAC;AAsB/F;;;;;;;;;;GAUG;AACH,MAAM,UAAU,UAAU,CAAC,IAK1B;IACC,MAAM,eAAe,GACnB,IAAI,CAAC,SAAS,KAAK,MAAM;QACzB,IAAI,CAAC,QAAQ,KAAK,UAAU;QAC5B,IAAI,CAAC,OAAO,CAAC,YAAY;QACzB,IAAI,CAAC,OAAO,CAAC,aAAa;QAC1B,IAAI,CAAC,OAAO,CAAC,iBAAiB,CAAC;IAEjC,IAAI,IAAI,CAAC,QAAQ,KAAK,SAAS,EAAE,CAAC;QAChC,IAAI,IAAI,CAAC,QAAQ,KAAK,OAAO,IAAI,eAAe,EAAE,CAAC;YACjD,OAAO;gBACL,IAAI,EAAE,OAAO;gBACb,MAAM,EAAE,MAAM;gBACd,OAAO,EAAE;oBACP,IAAI,EAAE,2BAA2B;oBACjC,OAAO,EACL,qHAAqH;iBACxH;aACF,CAAC;QACJ,CAAC;QACD,OAAO,EAAE,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IACjD,CAAC;IAED,IAAI,eAAe;QAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAE7D,IACE,IAAI,CAAC,SAAS,KAAK,KAAK;QACxB,IAAI,CAAC,OAAO,CAAC,WAAW,IAAI,oBAAoB;QAChD,IAAI,CAAC,QAAQ,KAAK,UAAU,EAC5B,CAAC;QACD,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAC3C,CAAC;IAED,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;AAC5C,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AACH,MAAM,UAAU,iBAAiB,CAC/B,cAA2B,EAC3B,IAAc,EACd,GAIC;IAED,IAAI,IAAI,KAAK,QAAQ;QAAE,OAAO,EAAE,MAAM,EAAE,cAAc,EAAE,CAAC;IAEzD,IAAI,IAAI,KAAK,MAAM,EAAE,CAAC;QACpB,MAAM,GAAG,GAAG,CAAC,GAAG,cAAc,CAAC,CAAC;QAChC,KAAK,MAAM,QAAQ,IAAI,aAAa,EAAE,CAAC;YACrC,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,QAAQ,CAAC;gBAAE,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAClD,CAAC;QACD,OAAO,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC;IACzB,CAAC;IAED,mBAAmB;IACnB,MAAM,aAAa,GACjB,GAAG,CAAC,OAAO,CAAC,YAAY,IAAI,GAAG,CAAC,OAAO,CAAC,aAAa,IAAI,GAAG,CAAC,OAAO,CAAC,iBAAiB,CAAC;IAEzF,MAAM,IAAI,GAAG,IAAI,GAAG,EAAa,CAAC;IAClC,MAAM,MAAM,GAAgB,EAAE,CAAC;IAC/B,MAAM,IAAI,GAAG,CAAC,CAAY,EAAW,EAAE;QACrC,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC;YAAE,OAAO,KAAK,CAAC;QACrC,IAAI,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;YAAE,OAAO,KAAK,CAAC;QAC9B,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACf,OAAO,IAAI,CAAC;IACd,CAAC,CAAC;IAEF,uBAAuB;IACvB,IAAI,aAAa;QAAE,IAAI,CAAC,qBAAqB,CAAC,CAAC;IAE/C,iEAAiE;IACjE,IAAI,aAAa,GAAG,CAAC,CAAC;IACtB,KAAK,MAAM,CAAC,IAAI,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACrC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;YAAE,aAAa,IAAI,CAAC,CAAC;IACnD,CAAC;IAED,4EAA4E;IAC5E,sEAAsE;IACtE,4DAA4D;IAC5D,IAAI,GAAG,CAAC,QAAQ,KAAK,eAAe;QAAE,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAE7D,mDAAmD;IACnD,KAAK,MAAM,CAAC,IAAI,cAAc;QAAE,IAAI,CAAC,CAAC,CAAC,CAAC;IAExC,wEAAwE;IACxE,sDAAsD;IACtD,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC;QAAE,IAAI,CAAC,kBAAkB,CAAC,CAAC;IAEhD,MAAM,MAAM,GAAmD,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAClF,IAAI,aAAa,GAAG,CAAC,EAAE,CAAC;QACtB,MAAM,CAAC,OAAO,GAAG;YACf,IAAI,EAAE,wBAAwB;YAC9B,OAAO,EAAE,0CAA0C,aAAa,0CAA0C;SAC3G,CAAC;IACJ,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/tools/read-learnings.js

@@ -63,7 +63,7 @@ export const readLearningsToolDef = {
     description: "Read recent team decisions from `.squad/learnings.jsonl` (path overridable via .squad.yaml.learnings.path). " +
         "Returns the filtered entries plus a pre-rendered markdown block ready to inject into agent / consolidator prompts. " +
         "Filters: agent, decision (accept|reject), changed_files (matches scoped entries against these paths). " +
-        "Used by the /squad-review skill in Phase 5 (advisory) and Phase 10 (consolidation) to make the squad less repetitive over time.",
+        "Used by the /squad:review skill in Phase 5 (advisory) and Phase 10 (consolidation) to make the squad less repetitive over time.",
     schema,
     handler: readLearningsTool,
 };

package/dist/tools/record-learning.d.ts

@@ -12,7 +12,7 @@ declare const schema: z.ZodObject<{
     severity: z.ZodOptional<z.ZodEnum<["Blocker", "Major", "Minor", "Suggestion"]>>;
     /** Free-form rationale. Surfaces in the consolidator prompt. */
     reason: z.ZodOptional<z.ZodString>;
-    /** PR number when recorded from `/squad-review #N`. */
+    /** PR number when recorded from `/squad:review #N`. */
     pr: z.ZodOptional<z.ZodNumber>;
     /** Branch name when recorded from a local review. */
     branch: z.ZodOptional<z.ZodString>;

package/dist/tools/record-learning.js

@@ -15,7 +15,7 @@ const schema = z.object({
     severity: z.enum(["Blocker", "Major", "Minor", "Suggestion"]).optional(),
     /** Free-form rationale. Surfaces in the consolidator prompt. */
     reason: z.string().max(4096).optional(),
-    /** PR number when recorded from `/squad-review #N`. */
+    /** PR number when recorded from `/squad:review #N`. */
     pr: z.number().int().positive().optional(),
     /** Branch name when recorded from a local review. */
     branch: z.string().min(1).max(255).optional(),

package/dist/tools/select-squad.js

@@ -99,9 +99,15 @@ export async function selectSquad(input) {
             rationale.push({ agent: forced, reason: "forced by caller" });
         }
     }
-    const sortedAgents = Array.from(selected).sort();
+    // Preserve insertion order: core agents from the matrix come first, then
+    // content/path signals, then user force_agents last. This is the "ranked"
+    // order downstream consumers (notably `shapeSquadForMode` in
+    // compose-squad-workflow.ts when mode === "quick") rely on. Prior versions
+    // sorted alphabetically and silently shipped a top-2 that didn't match
+    // the docstring contract.
+    const rankedAgents = Array.from(selected);
     return {
-        agents: sortedAgents,
+        agents: rankedAgents,
         rationale,
         evidence,
         low_confidence_files: lowConfidence,

package/dist/tools/select-squad.js.map

@@ -1 +1 @@
-{"version":3,"file":"select-squad.js","sourceRoot":"","sources":["../../src/tools/select-squad.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EACL,MAAM,EACN,iBAAiB,EACjB,eAAe,EACf,UAAU,EACV,aAAa,EACb,aAAa,GAGd,MAAM,+BAA+B,CAAC;AACvC,OAAO,EACL,qBAAqB,EACrB,SAAS,EACT,WAAW,EACX,eAAe,GAChB,MAAM,wBAAwB,CAAC;AAChC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,UAAU,IAAI,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAEhE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,SAAS,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,aAAa,EAAE,UAAU,EAAE,eAAe,CAAC,CAAC;IACjG,KAAK,EAAE,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC;IACxD,YAAY,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC;IAClD,YAAY,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACvE,cAAc,EAAE,UAAU,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE;CAC5C,CAAC,CAAC;AAoBH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,KAAY;IAC5C,MAAM,WAAW,GAAG,aAAa,CAAC,KAAK,CAAC,SAAqB,CAAC,CAAC;IAC/D,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAY,WAAW,CAAC,IAAI,CAAC,CAAC;IACtD,MAAM,SAAS,GAA2C,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QACrF,KAAK,EAAE,CAAC;QACR,MAAM,EAAE,kBAAkB,KAAK,CAAC,SAAS,EAAE;KAC5C,CAAC,CAAC,CAAC;IAEJ,MAAM,QAAQ,GAAe,EAAE,CAAC;IAChC,MAAM,aAAa,GAAuC,EAAE,CAAC;IAE7D,MAAM,GAAG,GAAG,qBAAqB,EAAE,CAAC;IACpC,MAAM,cAAc,GAAG,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,cAAc,KAAK,SAAS,CAAC;IAEhF,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;QAC/B,MAAM,OAAO,GAKP,EAAE,CAAC;QAET,KAAK,MAAM,IAAI,IAAI,UAAU,EAAE,CAAC;YAC9B,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC5B,OAAO,CAAC,IAAI,CAAC;oBACX,KAAK,EAAE,IAAI,CAAC,KAAK;oBACjB,MAAM,EAAE,IAAI,CAAC,WAAW;oBACxB,MAAM,EAAE,MAAM;iBACf,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,IAAI,cAAc,GAAG,KAAK,CAAC;QAC3B,IAAI,gBAAgB,GAAG,KAAK,CAAC;QAC7B,IAAI,cAAc,EAAE,CAAC;YACnB,IAAI,CAAC;gBACH,MAAM,GAAG,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,cAAc,EAAE,IAAI,EAAE,GAAG,CAAC,CAAC;gBACnE,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC,GAAG,CAAC,CAAC;gBACvC,IAAI,OAAO,EAAE,CAAC;oBACZ,gBAAgB,GAAG,OAAO,CAAC,SAAS,CAAC;oBACrC,KAAK,MAAM,GAAG,IAAI,eAAe,EAAE,CAAC;wBAClC,IAAI,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC;4BAAE,SAAS;wBACxC,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;4BACtC,OAAO,CAAC,IAAI,CAAC;gCACX,KAAK,EAAE,GAAG,CAAC,KAAK;gCAChB,MAAM,EAAE,GAAG,CAAC,WAAW;gCACvB,MAAM,EAAE,SAAS;gCACjB,SAAS,EAAE,OAAO,CAAC,SAAS;6BAC7B,CAAC,CAAC;4BACH,cAAc,GAAG,IAAI,CAAC;wBACxB,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,IAAI,YAAY,CAAC,GAAG,CAAC,EAAE,CAAC;oBACtB,aAAa,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,kBAAkB,GAAG,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;oBACnE,SAAS;gBACX,CAAC;gBACD,MAAM,GAAG,CAAC;YACZ,CAAC;QACH,CAAC;QAED,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzB,aAAa,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,mCAAmC,EAAE,CAAC,CAAC;YAC1E,SAAS;QACX,CAAC;QAED,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;QAC/B,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;YACxB,MAAM,GAAG,GAAG,GAAG,CAAC,CAAC,KAAK,IAAI,CAAC,CAAC,MAAM,EAAE,CAAC;YACrC,IAAI,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC5B,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;YACd,MAAM,UAAU,GACd,CAAC,CAAC,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC;YACzE,MAAM,EAAE,GAAa;gBACnB,IAAI;gBACJ,KAAK,EAAE,CAAC,CAAC,KAAK;gBACd,MAAM,EAAE,CAAC,CAAC,MAAM;gBAChB,MAAM,EAAE,CAAC,CAAC,MAAM;gBAChB,UAAU;aACX,CAAC;YACF,IAAI,CAAC,CAAC,SAAS,IAAI,CAAC,CAAC,CAAC,MAAM,KAAK,SAAS,IAAI,gBAAgB,CAAC;gBAAE,EAAE,CAAC,SAAS,GAAG,IAAI,CAAC;YACrF,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YAClB,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC3B,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;gBACtB,SAAS,CAAC,IAAI,CAAC;oBACb,KAAK,EAAE,CAAC,CAAC,KAAK;oBACd,MAAM,EAAE,GAAG,CAAC,CAAC,MAAM,cAAc,IAAI,KAAK,CAAC,CAAC,MAAM,EAAE;iBACrD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAED,KAAK,MAAM,MAAM,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;QACxC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;YAC1B,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;YACrB,SAAS,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC,CAAC;QAChE,CAAC;IACH,CAAC;IAED,MAAM,YAAY,GAAG,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,IAAI,EAAE,CAAC;IACjD,OAAO;QACL,MAAM,EAAE,YAAY;QACpB,SAAS;QACT,QAAQ;QACR,oBAAoB,EAAE,aAAa;KACpC,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,eAAe,GAA2B;IACrD,IAAI,EAAE,cAAc;IACpB,WAAW,EACT,6GAA6G;QAC7G,eAAe,SAAS,4GAA4G;IACtI,MAAM;IACN,OAAO,EAAE,WAAW;CACrB,CAAC;AAEF,KAAK,MAAM,CAAC"}
+{"version":3,"file":"select-squad.js","sourceRoot":"","sources":["../../src/tools/select-squad.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EACL,MAAM,EACN,iBAAiB,EACjB,eAAe,EACf,UAAU,EACV,aAAa,EACb,aAAa,GAGd,MAAM,+BAA+B,CAAC;AACvC,OAAO,EACL,qBAAqB,EACrB,SAAS,EACT,WAAW,EACX,eAAe,GAChB,MAAM,wBAAwB,CAAC;AAChC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,UAAU,IAAI,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAEhE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,SAAS,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,aAAa,EAAE,UAAU,EAAE,eAAe,CAAC,CAAC;IACjG,KAAK,EAAE,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC;IACxD,YAAY,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC;IAClD,YAAY,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACvE,cAAc,EAAE,UAAU,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE;CAC5C,CAAC,CAAC;AAoBH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,KAAY;IAC5C,MAAM,WAAW,GAAG,aAAa,CAAC,KAAK,CAAC,SAAqB,CAAC,CAAC;IAC/D,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAY,WAAW,CAAC,IAAI,CAAC,CAAC;IACtD,MAAM,SAAS,GAA2C,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QACrF,KAAK,EAAE,CAAC;QACR,MAAM,EAAE,kBAAkB,KAAK,CAAC,SAAS,EAAE;KAC5C,CAAC,CAAC,CAAC;IAEJ,MAAM,QAAQ,GAAe,EAAE,CAAC;IAChC,MAAM,aAAa,GAAuC,EAAE,CAAC;IAE7D,MAAM,GAAG,GAAG,qBAAqB,EAAE,CAAC;IACpC,MAAM,cAAc,GAAG,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,cAAc,KAAK,SAAS,CAAC;IAEhF,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;QAC/B,MAAM,OAAO,GAKP,EAAE,CAAC;QAET,KAAK,MAAM,IAAI,IAAI,UAAU,EAAE,CAAC;YAC9B,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC5B,OAAO,CAAC,IAAI,CAAC;oBACX,KAAK,EAAE,IAAI,CAAC,KAAK;oBACjB,MAAM,EAAE,IAAI,CAAC,WAAW;oBACxB,MAAM,EAAE,MAAM;iBACf,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,IAAI,cAAc,GAAG,KAAK,CAAC;QAC3B,IAAI,gBAAgB,GAAG,KAAK,CAAC;QAC7B,IAAI,cAAc,EAAE,CAAC;YACnB,IAAI,CAAC;gBACH,MAAM,GAAG,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,cAAc,EAAE,IAAI,EAAE,GAAG,CAAC,CAAC;gBACnE,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC,GAAG,CAAC,CAAC;gBACvC,IAAI,OAAO,EAAE,CAAC;oBACZ,gBAAgB,GAAG,OAAO,CAAC,SAAS,CAAC;oBACrC,KAAK,MAAM,GAAG,IAAI,eAAe,EAAE,CAAC;wBAClC,IAAI,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC;4BAAE,SAAS;wBACxC,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;4BACtC,OAAO,CAAC,IAAI,CAAC;gCACX,KAAK,EAAE,GAAG,CAAC,KAAK;gCAChB,MAAM,EAAE,GAAG,CAAC,WAAW;gCACvB,MAAM,EAAE,SAAS;gCACjB,SAAS,EAAE,OAAO,CAAC,SAAS;6BAC7B,CAAC,CAAC;4BACH,cAAc,GAAG,IAAI,CAAC;wBACxB,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,IAAI,YAAY,CAAC,GAAG,CAAC,EAAE,CAAC;oBACtB,aAAa,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,kBAAkB,GAAG,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;oBACnE,SAAS;gBACX,CAAC;gBACD,MAAM,GAAG,CAAC;YACZ,CAAC;QACH,CAAC;QAED,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzB,aAAa,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,mCAAmC,EAAE,CAAC,CAAC;YAC1E,SAAS;QACX,CAAC;QAED,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;QAC/B,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;YACxB,MAAM,GAAG,GAAG,GAAG,CAAC,CAAC,KAAK,IAAI,CAAC,CAAC,MAAM,EAAE,CAAC;YACrC,IAAI,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC5B,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;YACd,MAAM,UAAU,GACd,CAAC,CAAC,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC;YACzE,MAAM,EAAE,GAAa;gBACnB,IAAI;gBACJ,KAAK,EAAE,CAAC,CAAC,KAAK;gBACd,MAAM,EAAE,CAAC,CAAC,MAAM;gBAChB,MAAM,EAAE,CAAC,CAAC,MAAM;gBAChB,UAAU;aACX,CAAC;YACF,IAAI,CAAC,CAAC,SAAS,IAAI,CAAC,CAAC,CAAC,MAAM,KAAK,SAAS,IAAI,gBAAgB,CAAC;gBAAE,EAAE,CAAC,SAAS,GAAG,IAAI,CAAC;YACrF,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YAClB,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC3B,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;gBACtB,SAAS,CAAC,IAAI,CAAC;oBACb,KAAK,EAAE,CAAC,CAAC,KAAK;oBACd,MAAM,EAAE,GAAG,CAAC,CAAC,MAAM,cAAc,IAAI,KAAK,CAAC,CAAC,MAAM,EAAE;iBACrD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAED,KAAK,MAAM,MAAM,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;QACxC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;YAC1B,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;YACrB,SAAS,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC,CAAC;QAChE,CAAC;IACH,CAAC;IAED,yEAAyE;IACzE,0EAA0E;IAC1E,6DAA6D;IAC7D,2EAA2E;IAC3E,uEAAuE;IACvE,0BAA0B;IAC1B,MAAM,YAAY,GAAG,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC1C,OAAO;QACL,MAAM,EAAE,YAAY;QACpB,SAAS;QACT,QAAQ;QACR,oBAAoB,EAAE,aAAa;KACpC,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,eAAe,GAA2B;IACrD,IAAI,EAAE,cAAc;IACpB,WAAW,EACT,6GAA6G;QAC7G,eAAe,SAAS,4GAA4G;IACtI,MAAM;IACN,OAAO,EAAE,WAAW;CACrB,CAAC;AAEF,KAAK,MAAM,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gempack/squad-mcp",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "MCP server for the squad-dev workflow: classification, risk scoring, agent selection, advisory orchestration",
   "type": "module",
   "license": "Apache-2.0",

package/shared/Skill-Squad-Dev.md

@@ -2,11 +2,11 @@
 
 ## Objective
 
-Development skill that takes a user prompt, builds an implementation plan, runs gated advisory with specialized agents, implements, and consolidates via TechLead-Consolidator. Codex is optional (`/squad --codex`) and may be auto-suggested when the plan is high-risk.
+Development skill that takes a user prompt, builds an implementation plan, runs gated advisory with specialized agents, implements, and consolidates via TechLead-Consolidator. Codex is optional (`/squad:implement --codex`) and may be auto-suggested when the plan is high-risk.
 
 ## Skill Name
 
-`/squad`
+`/squad:implement`
 
 ## Agent Registry
 
@@ -25,7 +25,7 @@ Development skill that takes a user prompt, builds an implementation plan, runs
 ## General Flow
 
 ```
-User -> /squad {prompt}
+User -> /squad:implement {prompt}
        |
        v
 [0. Pre-Check]
@@ -378,21 +378,21 @@ Critical-change auto-fallback: if scope touches `auth`, `crypto`, `permissions`,
 ## Usage Examples
 
 ```
-/squad implement dollar balance endpoint
+/squad:implement implement dollar balance endpoint
 -> No Codex
 -> Plan + Planner -> User approves -> Advisory -> Implement -> Consolidator
 
-/squad --codex fix cache bug in ParameterService
+/squad:implement --codex fix cache bug in ParameterService
 -> With Codex
 -> Plan + Planner -> Codex validates -> User approves -> Advisory -> Implement -> Codex reviews -> Consolidator
 
-/squad refactor ExchangeUsdService to split responsibilities
+/squad:implement refactor ExchangeUsdService to split responsibilities
 -> Plan + Planner -> User approves -> Advisory -> Implement -> Consolidator
 
-/squad --quick rename a misspelled local variable in BalanceController
+/squad:implement --quick rename a misspelled local variable in BalanceController
 -> Quick mode: condensed plan, 1 specialist + tech-lead, terse prompts, condensed delivery
 
-/squad --quick --codex anything
+/squad:implement --quick --codex anything
 -> Error: --quick is mutually exclusive with --codex
 ```
 

package/shared/Skill-Squad-Review.md

@@ -6,14 +6,14 @@ Skill that takes a user prompt, interprets intent, selects the relevant agents,
 
 ## Skill Name
 
-`/squad-review`
+`/squad:review`
 
 ## How It Works
 
 ### General Flow
 
 ```
-User -> /squad-review {prompt}
+User -> /squad:review {prompt}
                 |
                 v
       [1. Prompt Analysis]
@@ -62,42 +62,42 @@ The user can request a generic review or target a focus area. The skill maps the
 All specialized agents + TechLead-Consolidator.
 
 - **When**: Complete PR review, branch ready to merge
-- **Triggers**: `/squad-review`, `/squad-review PR`, `/squad-review branch`
+- **Triggers**: `/squad:review`, `/squad:review PR`, `/squad:review branch`
 
 #### Code Squad
 
 `senior-dev-reviewer` + `senior-developer` + `senior-qa` + `tech-lead-consolidator`
 
 - **When**: Focused review on code quality and correctness
-- **Triggers**: `/squad-review code`
+- **Triggers**: `/squad:review code`
 
 #### Data Squad
 
 `senior-dba` + `senior-developer` + `tech-lead-consolidator`
 
 - **When**: Changes to queries, migrations, cache, EF
-- **Triggers**: `/squad-review data`
+- **Triggers**: `/squad:review data`
 
 #### Security Squad
 
 `senior-dev-security` + `senior-developer` + `senior-dev-reviewer` + `tech-lead-consolidator`
 
 - **When**: Focused security review
-- **Triggers**: `/squad-review security`
+- **Triggers**: `/squad:review security`
 
 #### Architecture Squad
 
 `senior-architect` + `senior-developer` + `senior-dba` + `tech-lead-consolidator`
 
 - **When**: Structural changes, new modules, large refactors
-- **Triggers**: `/squad-review arch`
+- **Triggers**: `/squad:review arch`
 
 #### Business Squad
 
 `po` + `senior-developer` + `senior-qa` + `tech-lead-consolidator`
 
 - **When**: New feature, business-rule change
-- **Triggers**: `/squad-review business`
+- **Triggers**: `/squad:review business`
 
 ### Automatic Squad Detection
 
@@ -317,25 +317,25 @@ Verdict: APPROVED — no Blocker or Major findings.
 ## Usage Examples
 
 ```
-/squad-review
+/squad:review
 -> Auto-detects the squad; reviews diff of current branch vs. master
 
-/squad-review security
+/squad:review security
 -> Security squad; reviews current branch diff
 
-/squad-review arch Services/ParameterService.cs
+/squad:review arch Services/ParameterService.cs
 -> Architecture squad; focused on the specified file
 
-/squad-review full
+/squad:review full
 -> Every agent; complete review
 
-/squad-review --quick
+/squad:review --quick
 -> Quick mode: 1 specialist + tech-lead, terse prompts, condensed output
 
-/squad-review --quick code
+/squad:review --quick code
 -> Quick code-quality review (senior-dev-reviewer + tech-lead)
 
-/squad-review --quick --codex
+/squad:review --quick --codex
 -> Error: --quick is mutually exclusive with --codex
 ```