@fredericboyer/dev-team 0.4.1 → 0.6.0

package/dist/parallel.js

@@ -0,0 +1,323 @@
+"use strict";
+/**
+ * parallel.ts — Parallel state management for multi-issue orchestration (ADR-019).
+ *
+ * Manages the `.claude/dev-team-parallel.json` state file that tracks
+ * parallel implementation phases, sync barriers, and review waves.
+ *
+ * Zero runtime dependencies (project constraint from ADR-002).
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DEFAULT_MAX_ITERATIONS = exports.STATE_FILENAME = void 0;
+exports.statePath = statePath;
+exports.readState = readState;
+exports.writeState = writeState;
+exports.stateExists = stateExists;
+exports.deleteState = deleteState;
+exports.createState = createState;
+exports.transitionPhase = transitionPhase;
+exports.isValidTransition = isValidTransition;
+exports.allowedTransitions = allowedTransitions;
+exports.updateIssueStatus = updateIssueStatus;
+exports.checkSyncBarrier = checkSyncBarrier;
+exports.startReviewWave = startReviewWave;
+exports.recordFindings = recordFindings;
+exports.isReviewWaveComplete = isReviewWaveComplete;
+exports.getDefectRoutes = getDefectRoutes;
+exports.checkConvergence = checkConvergence;
+exports.getIndependentIssues = getIndependentIssues;
+exports.getConflictGroup = getConflictGroup;
+exports.summarize = summarize;
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
+const files_1 = require("./files");
+// ─── Constants ───────────────────────────────────────────────────────────────
+exports.STATE_FILENAME = "dev-team-parallel.json";
+exports.DEFAULT_MAX_ITERATIONS = 10;
+// ─── State file path ─────────────────────────────────────────────────────────
+function statePath(projectRoot) {
+    return path_1.default.join(projectRoot, ".claude", exports.STATE_FILENAME);
+}
+// ─── Read / Write ────────────────────────────────────────────────────────────
+function readState(projectRoot) {
+    const fp = statePath(projectRoot);
+    const content = (0, files_1.readFile)(fp);
+    if (content === null)
+        return null;
+    try {
+        return JSON.parse(content);
+    }
+    catch {
+        return null;
+    }
+}
+function writeState(projectRoot, state) {
+    state.updatedAt = new Date().toISOString();
+    const fp = statePath(projectRoot);
+    (0, files_1.writeFile)(fp, JSON.stringify(state, null, 2) + "\n");
+}
+function stateExists(projectRoot) {
+    return (0, files_1.fileExists)(statePath(projectRoot));
+}
+function deleteState(projectRoot) {
+    const fp = statePath(projectRoot);
+    if (!(0, files_1.fileExists)(fp))
+        return false;
+    fs_1.default.unlinkSync(fp);
+    return true;
+}
+function createState(projectRoot, options) {
+    if (options.issues.length === 0) {
+        throw new Error("Cannot create parallel state with zero issues");
+    }
+    // Validate no duplicate issue numbers
+    const issueNumbers = options.issues.map((i) => i.issue);
+    const unique = new Set(issueNumbers);
+    if (unique.size !== issueNumbers.length) {
+        throw new Error("Duplicate issue numbers in parallel state");
+    }
+    // Validate conflict groups reference existing issues
+    for (const group of options.conflictGroups || []) {
+        for (const issueNum of group) {
+            if (!unique.has(issueNum)) {
+                throw new Error(`Conflict group references unknown issue #${issueNum}`);
+            }
+        }
+    }
+    const now = new Date().toISOString();
+    const state = {
+        mode: "parallel",
+        issues: options.issues.map((i) => ({
+            issue: i.issue,
+            branch: i.branch,
+            agent: i.agent,
+            status: "pending",
+            reviewIteration: 0,
+        })),
+        phase: "pre-assessment",
+        conflictGroups: options.conflictGroups || [],
+        reviewWave: null,
+        maxIterations: options.maxIterations ?? exports.DEFAULT_MAX_ITERATIONS,
+        createdAt: now,
+        updatedAt: now,
+        phaseLog: [],
+    };
+    writeState(projectRoot, state);
+    return state;
+}
+// ─── Phase transitions ──────────────────────────────────────────────────────
+function transitionPhase(state, to, reason) {
+    const from = state.phase;
+    // Validate transition is legal
+    if (!isValidTransition(from, to)) {
+        throw new Error(`Invalid phase transition: ${from} -> ${to}. Allowed from ${from}: ${allowedTransitions(from).join(", ")}`);
+    }
+    state.phaseLog.push({
+        from,
+        to,
+        timestamp: new Date().toISOString(),
+        reason,
+    });
+    state.phase = to;
+    return state;
+}
+const TRANSITION_MAP = {
+    "pre-assessment": ["implementation"],
+    implementation: ["sync-barrier"],
+    "sync-barrier": ["review-wave"],
+    "review-wave": ["defect-routing", "borges-completion"],
+    "defect-routing": ["review-wave", "borges-completion"],
+    "borges-completion": ["done"],
+    done: [],
+};
+function isValidTransition(from, to) {
+    return TRANSITION_MAP[from]?.includes(to) ?? false;
+}
+function allowedTransitions(from) {
+    return TRANSITION_MAP[from] || [];
+}
+// ─── Issue status updates ────────────────────────────────────────────────────
+function updateIssueStatus(state, issueNumber, status) {
+    const entry = state.issues.find((i) => i.issue === issueNumber);
+    if (!entry) {
+        throw new Error(`Issue #${issueNumber} not found in parallel state`);
+    }
+    entry.status = status;
+    return state;
+}
+// ─── Sync barrier ────────────────────────────────────────────────────────────
+/**
+ * Checks whether ALL implementation agents have completed.
+ * Returns true when every issue has status "implemented" or later.
+ */
+function checkSyncBarrier(state) {
+    const completedStatuses = [
+        "implemented",
+        "reviewing",
+        "defects-found",
+        "fixing",
+        "approved",
+    ];
+    const completed = [];
+    const pending = [];
+    for (const issue of state.issues) {
+        if (completedStatuses.includes(issue.status)) {
+            completed.push(issue.issue);
+        }
+        else {
+            pending.push(issue.issue);
+        }
+    }
+    return {
+        ready: pending.length === 0,
+        pending,
+        completed,
+    };
+}
+// ─── Review wave management ──────────────────────────────────────────────────
+function startReviewWave(state) {
+    const barrier = checkSyncBarrier(state);
+    if (!barrier.ready) {
+        throw new Error(`Cannot start review wave: issues still pending: ${barrier.pending.join(", ")}`);
+    }
+    const waveNumber = state.reviewWave ? state.reviewWave.wave + 1 : 1;
+    const branches = state.issues.map((i) => i.branch);
+    state.reviewWave = {
+        wave: waveNumber,
+        startedAt: new Date().toISOString(),
+        branches,
+        findings: {},
+    };
+    // Update all issue statuses to reviewing
+    for (const issue of state.issues) {
+        if (issue.status !== "approved") {
+            issue.status = "reviewing";
+        }
+    }
+    return state;
+}
+function recordFindings(state, branch, findings) {
+    if (!state.reviewWave) {
+        throw new Error("No active review wave to record findings for");
+    }
+    const issue = state.issues.find((i) => i.branch === branch);
+    if (!issue) {
+        throw new Error(`Branch "${branch}" not found in parallel state`);
+    }
+    state.reviewWave.findings[branch] = findings;
+    // Update issue status based on findings
+    if (findings.defects.length > 0) {
+        issue.status = "defects-found";
+        issue.defects = findings.defects;
+        issue.reviewIteration = (issue.reviewIteration || 0) + 1;
+    }
+    else {
+        issue.status = "approved";
+        issue.defects = [];
+    }
+    return state;
+}
+/**
+ * Checks if ALL branches in the current review wave have reported findings.
+ */
+function isReviewWaveComplete(state) {
+    if (!state.reviewWave)
+        return false;
+    const reported = new Set(Object.keys(state.reviewWave.findings));
+    return state.reviewWave.branches.every((b) => reported.has(b));
+}
+/**
+ * Returns defect routes: which issues need fixes routed back to their implementing agent.
+ * Respects per-branch iteration limits.
+ */
+function getDefectRoutes(state) {
+    const routes = [];
+    const exhausted = [];
+    for (const issue of state.issues) {
+        if (issue.status !== "defects-found")
+            continue;
+        const route = {
+            issue: issue.issue,
+            branch: issue.branch,
+            agent: issue.agent,
+            defects: issue.defects || [],
+            iteration: issue.reviewIteration || 1,
+        };
+        if ((issue.reviewIteration || 0) >= state.maxIterations) {
+            exhausted.push(route);
+        }
+        else {
+            routes.push(route);
+            issue.status = "fixing";
+        }
+    }
+    return { routes, exhausted };
+}
+/**
+ * Checks if the parallel execution has converged (all approved or exhausted).
+ */
+function checkConvergence(state) {
+    const approved = [];
+    const exhausted = [];
+    const pendingDefects = [];
+    for (const issue of state.issues) {
+        if (issue.status === "approved") {
+            approved.push(issue.issue);
+        }
+        else if ((issue.reviewIteration || 0) >= state.maxIterations) {
+            exhausted.push(issue.issue);
+        }
+        else {
+            pendingDefects.push(issue.issue);
+        }
+    }
+    return {
+        converged: approved.length + exhausted.length === state.issues.length,
+        allApproved: approved.length === state.issues.length,
+        exhaustedBranches: exhausted,
+        pendingDefects,
+    };
+}
+// ─── Conflict group helpers ──────────────────────────────────────────────────
+/**
+ * Returns issues that can execute in parallel (not in any conflict group with
+ * currently-running issues).
+ */
+function getIndependentIssues(state) {
+    const inConflict = new Set();
+    for (const group of state.conflictGroups) {
+        for (const num of group) {
+            inConflict.add(num);
+        }
+    }
+    return state.issues.filter((i) => !inConflict.has(i.issue)).map((i) => i.issue);
+}
+/**
+ * Returns the conflict group that contains a given issue, or null.
+ */
+function getConflictGroup(state, issueNumber) {
+    for (const group of state.conflictGroups) {
+        if (group.includes(issueNumber)) {
+            return group;
+        }
+    }
+    return null;
+}
+function summarize(state) {
+    const byStatus = {};
+    for (const issue of state.issues) {
+        byStatus[issue.status] = (byStatus[issue.status] || 0) + 1;
+    }
+    return {
+        phase: state.phase,
+        totalIssues: state.issues.length,
+        byStatus: byStatus,
+        currentWave: state.reviewWave?.wave ?? null,
+        phaseTransitions: state.phaseLog.length,
+        conflictGroupCount: state.conflictGroups.length,
+    };
+}
+//# sourceMappingURL=parallel.js.map

package/dist/parallel.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parallel.js","sourceRoot":"","sources":["../src/parallel.ts"],"names":[],"mappings":";AAAA;;;;;;;GAOG;;;;;;AA4EH,8BAEC;AAID,8BASC;AAED,gCAIC;AAED,kCAEC;AAED,kCAKC;AAUD,kCA0CC;AAID,0CAkBC;AAYD,8CAEC;AAED,gDAEC;AAID,8CAWC;AAQD,4CA4BC;AAID,0CA0BC;AAED,wCA2BC;AAKD,oDAIC;AAgBD,0CA2BC;AAcD,4CAqBC;AAQD,oDASC;AAKD,4CAOC;AAaD,8BAcC;AAncD,4CAAoB;AACpB,gDAAwB;AACxB,mCAA0D;AAiE1D,gFAAgF;AAEnE,QAAA,cAAc,GAAG,wBAAwB,CAAC;AAC1C,QAAA,sBAAsB,GAAG,EAAE,CAAC;AAEzC,gFAAgF;AAEhF,SAAgB,SAAS,CAAC,WAAmB;IAC3C,OAAO,cAAI,CAAC,IAAI,CAAC,WAAW,EAAE,SAAS,EAAE,sBAAc,CAAC,CAAC;AAC3D,CAAC;AAED,gFAAgF;AAEhF,SAAgB,SAAS,CAAC,WAAmB;IAC3C,MAAM,EAAE,GAAG,SAAS,CAAC,WAAW,CAAC,CAAC;IAClC,MAAM,OAAO,GAAG,IAAA,gBAAQ,EAAC,EAAE,CAAC,CAAC;IAC7B,IAAI,OAAO,KAAK,IAAI;QAAE,OAAO,IAAI,CAAC;IAClC,IAAI,CAAC;QACH,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAkB,CAAC;IAC9C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,SAAgB,UAAU,CAAC,WAAmB,EAAE,KAAoB;IAClE,KAAK,CAAC,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC3C,MAAM,EAAE,GAAG,SAAS,CAAC,WAAW,CAAC,CAAC;IAClC,IAAA,iBAAS,EAAC,EAAE,EAAE,IAAI,CAAC,SAAS,CAAC,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,CAAC,CAAC;AACvD,CAAC;AAED,SAAgB,WAAW,CAAC,WAAmB;IAC7C,OAAO,IAAA,kBAAU,EAAC,SAAS,CAAC,WAAW,CAAC,CAAC,CAAC;AAC5C,CAAC;AAED,SAAgB,WAAW,CAAC,WAAmB;IAC7C,MAAM,EAAE,GAAG,SAAS,CAAC,WAAW,CAAC,CAAC;IAClC,IAAI,CAAC,IAAA,kBAAU,EAAC,EAAE,CAAC;QAAE,OAAO,KAAK,CAAC;IAClC,YAAE,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC;IAClB,OAAO,IAAI,CAAC;AACd,CAAC;AAUD,SAAgB,WAAW,CAAC,WAAmB,EAAE,OAAsB;IACrE,IAAI,OAAO,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,+CAA+C,CAAC,CAAC;IACnE,CAAC;IAED,sCAAsC;IACtC,MAAM,YAAY,GAAG,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;IACxD,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,YAAY,CAAC,CAAC;IACrC,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,CAAC,MAAM,EAAE,CAAC;QACxC,MAAM,IAAI,KAAK,CAAC,2CAA2C,CAAC,CAAC;IAC/D,CAAC;IAED,qDAAqD;IACrD,KAAK,MAAM,KAAK,IAAI,OAAO,CAAC,cAAc,IAAI,EAAE,EAAE,CAAC;QACjD,KAAK,MAAM,QAAQ,IAAI,KAAK,EAAE,CAAC;YAC7B,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC1B,MAAM,IAAI,KAAK,CAAC,4CAA4C,QAAQ,EAAE,CAAC,CAAC;YAC1E,CAAC;QACH,CAAC;IACH,CAAC;IAED,MAAM,GAAG,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IACrC,MAAM,KAAK,GAAkB;QAC3B,IAAI,EAAE,UAAU;QAChB,MAAM,EAAE,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACjC,KAAK,EAAE,CAAC,CAAC,KAAK;YACd,MAAM,EAAE,CAAC,CAAC,MAAM;YAChB,KAAK,EAAE,CAAC,CAAC,KAAK;YACd,MAAM,EAAE,SAAwB;YAChC,eAAe,EAAE,CAAC;SACnB,CAAC,CAAC;QACH,KAAK,EAAE,gBAAgB;QACvB,cAAc,EAAE,OAAO,CAAC,cAAc,IAAI,EAAE;QAC5C,UAAU,EAAE,IAAI;QAChB,aAAa,EAAE,OAAO,CAAC,aAAa,IAAI,8BAAsB;QAC9D,SAAS,EAAE,GAAG;QACd,SAAS,EAAE,GAAG;QACd,QAAQ,EAAE,EAAE;KACb,CAAC;IAEF,UAAU,CAAC,WAAW,EAAE,KAAK,CAAC,CAAC;IAC/B,OAAO,KAAK,CAAC;AACf,CAAC;AAED,+EAA+E;AAE/E,SAAgB,eAAe,CAAC,KAAoB,EAAE,EAAS,EAAE,MAAc;IAC7E,MAAM,IAAI,GAAG,KAAK,CAAC,KAAK,CAAC;IAEzB,+BAA+B;IAC/B,IAAI,CAAC,iBAAiB,CAAC,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC;QACjC,MAAM,IAAI,KAAK,CACb,6BAA6B,IAAI,OAAO,EAAE,kBAAkB,IAAI,KAAK,kBAAkB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAC3G,CAAC;IACJ,CAAC;IAED,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC;QAClB,IAAI;QACJ,EAAE;QACF,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;QACnC,MAAM;KACP,CAAC,CAAC;IACH,KAAK,CAAC,KAAK,GAAG,EAAE,CAAC;IACjB,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,cAAc,GAA2B;IAC7C,gBAAgB,EAAE,CAAC,gBAAgB,CAAC;IACpC,cAAc,EAAE,CAAC,cAAc,CAAC;IAChC,cAAc,EAAE,CAAC,aAAa,CAAC;IAC/B,aAAa,EAAE,CAAC,gBAAgB,EAAE,mBAAmB,CAAC;IACtD,gBAAgB,EAAE,CAAC,aAAa,EAAE,mBAAmB,CAAC;IACtD,mBAAmB,EAAE,CAAC,MAAM,CAAC;IAC7B,IAAI,EAAE,EAAE;CACT,CAAC;AAEF,SAAgB,iBAAiB,CAAC,IAAW,EAAE,EAAS;IACtD,OAAO,cAAc,CAAC,IAAI,CAAC,EAAE,QAAQ,CAAC,EAAE,CAAC,IAAI,KAAK,CAAC;AACrD,CAAC;AAED,SAAgB,kBAAkB,CAAC,IAAW;IAC5C,OAAO,cAAc,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;AACpC,CAAC;AAED,gFAAgF;AAEhF,SAAgB,iBAAiB,CAC/B,KAAoB,EACpB,WAAmB,EACnB,MAAmB;IAEnB,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,WAAW,CAAC,CAAC;IAChE,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,UAAU,WAAW,8BAA8B,CAAC,CAAC;IACvE,CAAC;IACD,KAAK,CAAC,MAAM,GAAG,MAAM,CAAC;IACtB,OAAO,KAAK,CAAC;AACf,CAAC;AAED,gFAAgF;AAEhF;;;GAGG;AACH,SAAgB,gBAAgB,CAAC,KAAoB;IAKnD,MAAM,iBAAiB,GAAkB;QACvC,aAAa;QACb,WAAW;QACX,eAAe;QACf,QAAQ;QACR,UAAU;KACX,CAAC;IACF,MAAM,SAAS,GAAa,EAAE,CAAC;IAC/B,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjC,IAAI,iBAAiB,CAAC,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC;YAC7C,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC9B,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC5B,CAAC;IACH,CAAC;IAED,OAAO;QACL,KAAK,EAAE,OAAO,CAAC,MAAM,KAAK,CAAC;QAC3B,OAAO;QACP,SAAS;KACV,CAAC;AACJ,CAAC;AAED,gFAAgF;AAEhF,SAAgB,eAAe,CAAC,KAAoB;IAClD,MAAM,OAAO,GAAG,gBAAgB,CAAC,KAAK,CAAC,CAAC;IACxC,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,CAAC;QACnB,MAAM,IAAI,KAAK,CACb,mDAAmD,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAChF,CAAC;IACJ,CAAC;IAED,MAAM,UAAU,GAAG,KAAK,CAAC,UAAU,CAAC,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACpE,MAAM,QAAQ,GAAG,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAEnD,KAAK,CAAC,UAAU,GAAG;QACjB,IAAI,EAAE,UAAU;QAChB,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;QACnC,QAAQ;QACR,QAAQ,EAAE,EAAE;KACb,CAAC;IAEF,yCAAyC;IACzC,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjC,IAAI,KAAK,CAAC,MAAM,KAAK,UAAU,EAAE,CAAC;YAChC,KAAK,CAAC,MAAM,GAAG,WAAW,CAAC;QAC7B,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAgB,cAAc,CAC5B,KAAoB,EACpB,MAAc,EACd,QAAwB;IAExB,IAAI,CAAC,KAAK,CAAC,UAAU,EAAE,CAAC;QACtB,MAAM,IAAI,KAAK,CAAC,8CAA8C,CAAC,CAAC;IAClE,CAAC;IAED,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC;IAC5D,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,WAAW,MAAM,+BAA+B,CAAC,CAAC;IACpE,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,QAAQ,CAAC,MAAM,CAAC,GAAG,QAAQ,CAAC;IAE7C,wCAAwC;IACxC,IAAI,QAAQ,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChC,KAAK,CAAC,MAAM,GAAG,eAAe,CAAC;QAC/B,KAAK,CAAC,OAAO,GAAG,QAAQ,CAAC,OAAO,CAAC;QACjC,KAAK,CAAC,eAAe,GAAG,CAAC,KAAK,CAAC,eAAe,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;IAC3D,CAAC;SAAM,CAAC;QACN,KAAK,CAAC,MAAM,GAAG,UAAU,CAAC;QAC1B,KAAK,CAAC,OAAO,GAAG,EAAE,CAAC;IACrB,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;GAEG;AACH,SAAgB,oBAAoB,CAAC,KAAoB;IACvD,IAAI,CAAC,KAAK,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IACpC,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC,CAAC;IACjE,OAAO,KAAK,CAAC,UAAU,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;AACjE,CAAC;AAYD;;;GAGG;AACH,SAAgB,eAAe,CAAC,KAAoB;IAIlD,MAAM,MAAM,GAAkB,EAAE,CAAC;IACjC,MAAM,SAAS,GAAkB,EAAE,CAAC;IAEpC,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjC,IAAI,KAAK,CAAC,MAAM,KAAK,eAAe;YAAE,SAAS;QAE/C,MAAM,KAAK,GAAgB;YACzB,KAAK,EAAE,KAAK,CAAC,KAAK;YAClB,MAAM,EAAE,KAAK,CAAC,MAAM;YACpB,KAAK,EAAE,KAAK,CAAC,KAAK;YAClB,OAAO,EAAE,KAAK,CAAC,OAAO,IAAI,EAAE;YAC5B,SAAS,EAAE,KAAK,CAAC,eAAe,IAAI,CAAC;SACtC,CAAC;QAEF,IAAI,CAAC,KAAK,CAAC,eAAe,IAAI,CAAC,CAAC,IAAI,KAAK,CAAC,aAAa,EAAE,CAAC;YACxD,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACxB,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACnB,KAAK,CAAC,MAAM,GAAG,QAAQ,CAAC;QAC1B,CAAC;IACH,CAAC;IAED,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;AAC/B,CAAC;AAWD;;GAEG;AACH,SAAgB,gBAAgB,CAAC,KAAoB;IACnD,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,SAAS,GAAa,EAAE,CAAC;IAC/B,MAAM,cAAc,GAAa,EAAE,CAAC;IAEpC,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjC,IAAI,KAAK,CAAC,MAAM,KAAK,UAAU,EAAE,CAAC;YAChC,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC7B,CAAC;aAAM,IAAI,CAAC,KAAK,CAAC,eAAe,IAAI,CAAC,CAAC,IAAI,KAAK,CAAC,aAAa,EAAE,CAAC;YAC/D,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC9B,CAAC;aAAM,CAAC;YACN,cAAc,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QACnC,CAAC;IACH,CAAC;IAED,OAAO;QACL,SAAS,EAAE,QAAQ,CAAC,MAAM,GAAG,SAAS,CAAC,MAAM,KAAK,KAAK,CAAC,MAAM,CAAC,MAAM;QACrE,WAAW,EAAE,QAAQ,CAAC,MAAM,KAAK,KAAK,CAAC,MAAM,CAAC,MAAM;QACpD,iBAAiB,EAAE,SAAS;QAC5B,cAAc;KACf,CAAC;AACJ,CAAC;AAED,gFAAgF;AAEhF;;;GAGG;AACH,SAAgB,oBAAoB,CAAC,KAAoB;IACvD,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,cAAc,EAAE,CAAC;QACzC,KAAK,MAAM,GAAG,IAAI,KAAK,EAAE,CAAC;YACxB,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACtB,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;AAClF,CAAC;AAED;;GAEG;AACH,SAAgB,gBAAgB,CAAC,KAAoB,EAAE,WAAmB;IACxE,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,cAAc,EAAE,CAAC;QACzC,IAAI,KAAK,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;YAChC,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAaD,SAAgB,SAAS,CAAC,KAAoB;IAC5C,MAAM,QAAQ,GAA2B,EAAE,CAAC;IAC5C,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjC,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;IAC7D,CAAC;IAED,OAAO;QACL,KAAK,EAAE,KAAK,CAAC,KAAK;QAClB,WAAW,EAAE,KAAK,CAAC,MAAM,CAAC,MAAM;QAChC,QAAQ,EAAE,QAAuC;QACjD,WAAW,EAAE,KAAK,CAAC,UAAU,EAAE,IAAI,IAAI,IAAI;QAC3C,gBAAgB,EAAE,KAAK,CAAC,QAAQ,CAAC,MAAM;QACvC,kBAAkB,EAAE,KAAK,CAAC,cAAc,CAAC,MAAM;KAChD,CAAC;AACJ,CAAC"}

package/dist/status.d.ts

@@ -0,0 +1 @@
+export declare function status(targetDir: string): void;

package/dist/status.js

@@ -0,0 +1,82 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.status = status;
+const path_1 = __importDefault(require("path"));
+const fs_1 = __importDefault(require("fs"));
+const files_1 = require("./files");
+function status(targetDir) {
+    const claudeDir = path_1.default.join(targetDir, ".claude");
+    const prefsPath = path_1.default.join(claudeDir, "dev-team.json");
+    const prefsContent = (0, files_1.readFile)(prefsPath);
+    if (!prefsContent) {
+        console.error("Not a dev-team project. Run `npx dev-team init` first.");
+        process.exit(1);
+    }
+    let prefs;
+    try {
+        prefs = JSON.parse(prefsContent);
+    }
+    catch {
+        console.error("dev-team.json is corrupted.");
+        process.exit(1);
+    }
+    console.log("\ndev-team status\n");
+    // Version and preset
+    console.log(`  Version:  v${prefs.version}`);
+    if (prefs.preset) {
+        console.log(`  Preset:   ${prefs.preset}`);
+    }
+    // Agents
+    const agents = prefs.agents || [];
+    console.log(`  Agents:   ${agents.join(", ")} (${agents.length})`);
+    // Hooks
+    const hooks = prefs.hooks || [];
+    console.log(`  Hooks:    ${hooks.join(", ")} (${hooks.length})`);
+    // Skills (auto-discovered)
+    const skillsDir = path_1.default.join(claudeDir, "skills");
+    const skills = (0, files_1.listSubdirectories)(skillsDir).map((s) => s.replace("dev-team-", ""));
+    console.log(`  Skills:   ${skills.length > 0 ? skills.join(", ") : "none"} (${skills.length})`);
+    // Workflow
+    if (prefs.issueTracker) {
+        console.log(`  Tracker:  ${prefs.issueTracker}`);
+    }
+    if (prefs.branchConvention && prefs.branchConvention !== "None") {
+        console.log(`  Branches: ${prefs.branchConvention}`);
+    }
+    // Memory status
+    console.log("\n  Memory:");
+    const memoryDir = path_1.default.join(claudeDir, "agent-memory");
+    for (const label of agents) {
+        const dirName = `dev-team-${label.toLowerCase()}`;
+        const memPath = path_1.default.join(memoryDir, dirName, "MEMORY.md");
+        if ((0, files_1.fileExists)(memPath)) {
+            try {
+                const stat = fs_1.default.statSync(memPath);
+                const hasContent = stat.size > 200; // Template is ~200 bytes
+                console.log(`    ${label}: ${hasContent ? "has learnings" : "empty (template only)"}`);
+            }
+            catch {
+                console.log(`    ${label}: unknown`);
+            }
+        }
+        else {
+            console.log(`    ${label}: no memory file`);
+        }
+    }
+    // Shared learnings
+    const learningsPath = path_1.default.join(claudeDir, "dev-team-learnings.md");
+    if ((0, files_1.fileExists)(learningsPath)) {
+        try {
+            const stat = fs_1.default.statSync(learningsPath);
+            console.log(`    Shared learnings: ${stat.size > 300 ? "has content" : "template only"}`);
+        }
+        catch {
+            console.log(`    Shared learnings: unknown`);
+        }
+    }
+    console.log("");
+}
+//# sourceMappingURL=status.js.map

package/dist/status.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"status.js","sourceRoot":"","sources":["../src/status.ts"],"names":[],"mappings":";;;;;AAIA,wBA8EC;AAlFD,gDAAwB;AACxB,4CAAoB;AACpB,mCAAmE;AAEnE,SAAgB,MAAM,CAAC,SAAiB;IACtC,MAAM,SAAS,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;IAClD,MAAM,SAAS,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,eAAe,CAAC,CAAC;IAExD,MAAM,YAAY,GAAG,IAAA,gBAAQ,EAAC,SAAS,CAAC,CAAC;IACzC,IAAI,CAAC,YAAY,EAAE,CAAC;QAClB,OAAO,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;QACxE,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,IAAI,KAA8B,CAAC;IACnC,IAAI,CAAC;QACH,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;IACnC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,CAAC,KAAK,CAAC,6BAA6B,CAAC,CAAC;QAC7C,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,qBAAqB,CAAC,CAAC;IAEnC,qBAAqB;IACrB,OAAO,CAAC,GAAG,CAAC,gBAAgB,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC;IAC7C,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;QACjB,OAAO,CAAC,GAAG,CAAC,eAAe,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC;IAC7C,CAAC;IAED,SAAS;IACT,MAAM,MAAM,GAAI,KAAK,CAAC,MAAmB,IAAI,EAAE,CAAC;IAChD,OAAO,CAAC,GAAG,CAAC,eAAe,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC;IAEnE,QAAQ;IACR,MAAM,KAAK,GAAI,KAAK,CAAC,KAAkB,IAAI,EAAE,CAAC;IAC9C,OAAO,CAAC,GAAG,CAAC,eAAe,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC;IAEjE,2BAA2B;IAC3B,MAAM,SAAS,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC;IACjD,MAAM,MAAM,GAAG,IAAA,0BAAkB,EAAC,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,WAAW,EAAE,EAAE,CAAC,CAAC,CAAC;IACpF,OAAO,CAAC,GAAG,CAAC,eAAe,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC;IAEhG,WAAW;IACX,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;QACvB,OAAO,CAAC,GAAG,CAAC,eAAe,KAAK,CAAC,YAAY,EAAE,CAAC,CAAC;IACnD,CAAC;IACD,IAAI,KAAK,CAAC,gBAAgB,IAAI,KAAK,CAAC,gBAAgB,KAAK,MAAM,EAAE,CAAC;QAChE,OAAO,CAAC,GAAG,CAAC,eAAe,KAAK,CAAC,gBAAgB,EAAE,CAAC,CAAC;IACvD,CAAC;IAED,gBAAgB;IAChB,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;IAC3B,MAAM,SAAS,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,cAAc,CAAC,CAAC;IACvD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,MAAM,OAAO,GAAG,YAAY,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;QAClD,MAAM,OAAO,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,OAAO,EAAE,WAAW,CAAC,CAAC;QAC3D,IAAI,IAAA,kBAAU,EAAC,OAAO,CAAC,EAAE,CAAC;YACxB,IAAI,CAAC;gBACH,MAAM,IAAI,GAAG,YAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;gBAClC,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,yBAAyB;gBAC7D,OAAO,CAAC,GAAG,CAAC,OAAO,KAAK,KAAK,UAAU,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,uBAAuB,EAAE,CAAC,CAAC;YACzF,CAAC;YAAC,MAAM,CAAC;gBACP,OAAO,CAAC,GAAG,CAAC,OAAO,KAAK,WAAW,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,OAAO,KAAK,kBAAkB,CAAC,CAAC;QAC9C,CAAC;IACH,CAAC;IAED,mBAAmB;IACnB,MAAM,aAAa,GAAG,cAAI,CAAC,IAAI,CAAC,SAAS,EAAE,uBAAuB,CAAC,CAAC;IACpE,IAAI,IAAA,kBAAU,EAAC,aAAa,CAAC,EAAE,CAAC;QAC9B,IAAI,CAAC;YACH,MAAM,IAAI,GAAG,YAAE,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC;YACxC,OAAO,CAAC,GAAG,CAAC,yBAAyB,IAAI,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,eAAe,EAAE,CAAC,CAAC;QAC5F,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,CAAC,GAAG,CAAC,+BAA+B,CAAC,CAAC;QAC/C,CAAC;IACH,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;AAClB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fredericboyer/dev-team",
-  "version": "0.4.1",
+  "version": "0.6.0",
   "description": "Adversarial AI agent team for any project — installs Claude Code agents, hooks, and skills that enforce quality through productive friction",
   "main": "dist/init.js",
   "types": "dist/init.d.ts",
@@ -15,7 +15,7 @@
   "scripts": {
     "build": "tsc",
     "pretest": "npm run build",
-    "test": "node --test tests/unit/files.test.js tests/unit/hooks.test.js tests/unit/scan.test.js tests/unit/create-agent.test.js tests/integration/fresh-project.test.js tests/integration/idempotency.test.js tests/integration/update.test.js tests/scenarios/node-project.test.js tests/scenarios/python-project.test.js tests/scenarios/upgrade-path.test.js",
+    "test": "node --test tests/unit/files.test.js tests/unit/hooks.test.js tests/unit/scan.test.js tests/unit/create-agent.test.js tests/unit/cli.test.js tests/unit/parallel.test.js tests/integration/fresh-project.test.js tests/integration/idempotency.test.js tests/integration/update.test.js tests/scenarios/node-project.test.js tests/scenarios/python-project.test.js tests/scenarios/upgrade-path.test.js",
     "test:unit": "node --test tests/unit/files.test.js tests/unit/hooks.test.js",
     "test:integration": "node --test tests/integration/fresh-project.test.js tests/integration/idempotency.test.js",
     "test:scenarios": "node --test tests/scenarios/node-project.test.js tests/scenarios/python-project.test.js tests/scenarios/upgrade-path.test.js",

package/templates/CLAUDE.md

@@ -9,16 +9,17 @@ This project uses [dev-team](https://github.com/dev-team) — adversarial AI age
 | Agent | Role | When to use |
 |-------|------|-------------|
 | `@dev-team-voss` | Backend Engineer | API design, data modeling, system architecture, error handling |
+| `@dev-team-hamilton` | Infrastructure Engineer | Dockerfiles, IaC, CI/CD, k8s, deployment, health checks, monitoring |
 | `@dev-team-mori` | Frontend/UI Engineer | Components, accessibility, UX patterns, state management |
 | `@dev-team-szabo` | Security Auditor | Vulnerability review, auth flows, attack surface analysis |
 | `@dev-team-knuth` | Quality Auditor | Coverage gaps, boundary conditions, correctness verification |
 | `@dev-team-beck` | Test Implementer | Writing tests, TDD cycles, translating audit findings into test cases |
 | `@dev-team-deming` | Tooling Optimizer | Linters, formatters, CI/CD, hooks, onboarding, automation |
 | `@dev-team-tufte` | Documentation Engineer | Doc accuracy, stale docs, README/API docs, doc-code sync |
-| `@dev-team-brooks` | Architect | Architectural review, coupling, dependency direction, ADR compliance |
+| `@dev-team-brooks` | Architect & Quality Reviewer | Architectural review, coupling, ADR compliance, quality attributes (performance, maintainability, scalability) |
 | `@dev-team-conway` | Release Manager | Versioning, changelog, release readiness, semver validation |
 | `@dev-team-drucker` | Team Lead / Orchestrator | Auto-delegates to specialists, manages review loops, resolves conflicts |
-| `@dev-team-borges` | Librarian | End-of-task memory review, cross-agent coherence, system improvement |
+| `@dev-team-borges` | Librarian | End-of-task/review/audit memory review, cross-agent coherence, system improvement |
 
 ### Workflow
 
@@ -26,10 +27,31 @@ For automatic delegation, use `@dev-team-drucker` — it analyzes the task and r
 
 For non-trivial work: explore the area first, then implement, then review.
 
+**Automatic invocation (hooks):**
+- **Szabo** — auto-flagged when security-sensitive files change (auth, token, session, crypto, etc.)
+- **Knuth** — auto-flagged when any non-test implementation code changes
+- **Mori** — auto-flagged when API contract files change (/api/, /routes/, schema, etc.)
+- **Hamilton** — auto-flagged when infrastructure/operations files change (Dockerfile, docker-compose, CI workflows, Terraform, Helm, k8s, health checks, monitoring config, .env templates, etc.)
+- **Voss** — auto-flagged when app config/data files change (.env, config, migrations, database, etc.)
+- **Deming** — auto-flagged when tooling files change (eslint, CI workflows, package.json, etc.)
+- **Tufte** — auto-flagged when documentation files change (.md, /docs/, README, etc.) AND when significant implementation files change (src/, templates/agents/, templates/skills/, templates/hooks/, bin/, package.json) to detect doc-code drift
+- **Brooks** — auto-flagged when any non-test implementation code changes (quality attributes) and when architectural boundaries are touched (/adr/, /core/, /domain/, /lib/, build config, etc.)
+- **Conway** — auto-flagged when release artifacts change (package.json, changelog, version files, release/publish/deploy workflows, etc.)
+
+**End-of-workflow agents:**
+- **Borges** — mandatory at end of every `/dev-team:task`, `/dev-team:review`, and `/dev-team:audit`. Reviews memory freshness, cross-agent coherence, and system improvement opportunities.
+
+**Orchestration:**
+- **Drucker** — delegates tasks to the right implementing agent and spawns reviewers. Szabo, Knuth, and Brooks review all code changes. Brooks covers both structural review and quality attribute assessment (performance, maintainability, scalability).
+
 Agents challenge each other using classified findings:
 - `[DEFECT]` blocks progress. `[RISK]`, `[QUESTION]`, `[SUGGESTION]` are advisory.
 - When agents disagree, they escalate to the human after one exchange each. Human decides.
 
+### Parallel execution
+
+When working on multiple independent issues, use parallel agents on separate branches. Drucker coordinates the review wave after all implementations complete. See ADR-019 for the full model: Brooks assesses file independence, implementations run concurrently, reviews are batched into a coordinated wave, defects route back per-branch, and Borges runs once across all branches at the end.
+
 ### Hook directives are MANDATORY
 
 When a dev-team hook outputs `ACTION REQUIRED — spawn these agents`, you MUST:

package/templates/agent-memory/dev-team-hamilton/MEMORY.md

@@ -0,0 +1,12 @@
+# Agent Memory: Hamilton (Infrastructure Engineer)
+<!-- First 200 lines are loaded into agent context. Keep concise. -->
+
+## Project Conventions
+
+
+## Patterns to Watch For
+
+
+## Calibration Log
+<!-- Challenges accepted/overruled — tunes adversarial intensity over time -->
+

package/templates/agents/dev-team-brooks.md

@@ -1,6 +1,6 @@
 ---
 name: dev-team-brooks
-description: Architect. Use to review architectural decisions, challenge coupling and dependency direction, validate changes against ADRs, and assess system design trade-offs. Read-only — does not modify code.
+description: Architect and quality attribute reviewer. Use to review architectural decisions, challenge coupling and dependency direction, validate changes against ADRs, and assess quality attributes (performance, maintainability, scalability). Always-on for all non-test code changes. Read-only — does not modify code.
 tools: Read, Grep, Glob, Bash, Agent
 model: opus
 memory: project
@@ -23,6 +23,8 @@ You are **read-only**. You analyze structure and identify architectural violatio
 
 ## Focus areas
 
+### Structural review
+
 You always check for:
 - **Coupling direction**: Dependencies must point inward — from unstable to stable, from concrete to abstract. A utility module importing a domain module is a dependency inversion.
 - **Layer violations**: Each architectural layer has a contract. Presentation should not query the database. Business logic should not know about HTTP status codes.
@@ -31,6 +33,37 @@ You always check for:
 - **Interface surface area**: Every public API, every exported function, every shared type is a commitment. Minimize the surface area — what is not exposed cannot be depended upon.
 - **Change propagation**: When this module changes, how many other modules must also change? High fan-out from a change is a design smell.
 
+### Quality attribute assessment
+
+In addition to structural review, you assess every code change against three quality dimensions. Every finding must cite a **measurable criterion**, **concrete threshold**, or **specific scenario** where the issue manifests. Not "this is complex" but "this function has cyclomatic complexity >10 with 4 levels of nesting."
+
+**Performance:**
+- Algorithm complexity appropriate for the data size and call frequency?
+- Hot path impact — is this code on a critical path that runs per-request or per-event?
+- Resource lifecycle — are allocations paired with releases? Are there leaks in error paths?
+- I/O patterns — blocking calls on async paths? Unbatched operations in loops?
+
+**Maintainability:**
+- Cognitive complexity reasonable? (Flag functions with cyclomatic complexity >10 or nesting depth >3)
+- Naming communicates intent? Can a reader understand the purpose without reading the implementation?
+- Abstraction level consistent within the module? Mixing high-level orchestration with low-level bit manipulation is a readability hazard.
+- Hidden coupling or side effects? Does calling this function change state that the caller cannot predict from the signature?
+- Future reader test: can someone understand this code six months from now without the surrounding PR context?
+
+**Scalability:**
+- Data growth assumptions — does this code assume small N? What happens at 10x, 100x current load?
+- Concurrency model appropriate? Shared mutable state without synchronization is a race condition.
+- Bottleneck introduction — does this create a single point of serialization (single lock, single queue, single connection)?
+
+### Explicitly out of scope
+
+These quality attributes are owned by other agents — do not assess them:
+- **Security** — owned by Szabo (threat modeling, attack surface, vulnerability patterns)
+- **Correctness/reliability** — owned by Knuth (edge cases, boundary conditions, coverage gaps)
+- **Usability/UX** — owned by Mori
+- **Availability** — owned by Hamilton (health checks, graceful degradation, deployment quality)
+- **Portability** — owned by Deming
+
 ## Challenge style
 
 You analyze structural consequences over time:
@@ -38,6 +71,8 @@ You analyze structural consequences over time:
 - "Module A imports Module B, but B also imports A through a transitive dependency via C. This circular dependency means you cannot deploy A without B. Was that intentional?"
 - "This handler reads from the database, applies business rules, formats the HTTP response, and sends an email — four responsibilities. When the email provider changes, you will be modifying request handler code."
 - "ADR-003 says hooks must be plain JavaScript for portability. This new hook imports a TypeScript-only utility. Either the hook or the ADR needs to change."
+- "This loop calls `fetchRecord()` once per ID without batching. With the current 50-record average that is 50 sequential network round-trips (~2.5s at 50ms each). At 500 records this becomes 25 seconds."
+- "This function has 6 parameters, 4 levels of nesting, and 3 early returns that mutate a shared accumulator. Cyclomatic complexity is approximately 14. A reader must hold all branches in working memory simultaneously."
 
 ## Challenge protocol
 
@@ -49,10 +84,11 @@ When reviewing another agent's work, classify each concern:
 
 Rules:
 1. Every challenge must include a concrete scenario, input, or code reference.
-2. Only `[DEFECT]` blocks progress.
-3. When challenged: address directly, concede when wrong, justify with a counter-scenario when you disagree.
-4. One exchange each before escalating to the human.
-5. Acknowledge good work when you see it.
+2. Every quality attribute finding must cite a measurable criterion, concrete threshold, or specific scenario — not subjective impressions.
+3. Only `[DEFECT]` blocks progress.
+4. When challenged: address directly, concede when wrong, justify with a counter-scenario when you disagree.
+5. One exchange each before escalating to the human.
+6. Acknowledge good work when you see it.
 
 ## Learning
 
@@ -61,4 +97,5 @@ After completing a review, write key learnings to your MEMORY.md:
 - ADRs and their current compliance status
 - Dependency directions that have been validated or corrected
 - Layer boundaries and where they are weakest
+- Quality attribute patterns observed (hot paths, complexity hotspots, scalability assumptions)
 - Challenges you raised that were accepted (reinforce) or overruled (calibrate)

package/templates/agents/dev-team-deming.md

@@ -34,6 +34,7 @@ You always check for:
 - **CI/CD pipeline speed**: Are independent steps running in parallel? Are there unnecessary rebuilds? Is caching configured?
 - **Onboarding friction**: How fast can a new developer go from clone to productive? Are there undocumented setup steps or missing scripts?
 - **Toolchain bloat**: Is every tool earning its keep? Remove tools that add more cognitive load than they remove.
+- **Portability**: Cross-platform CI coverage, platform-specific behavior detection, and environment portability. A build that only passes on the author's machine is not a build.
 
 ## Challenge style