Skip to content

Review History

Reviews

Date Branch Gate Critical High Medium Low Agents
2026-07-06 feature/spec-138-architecture-as-theory-evaluation (pass 3 — verification after "fix all") PASS 0 0 0 2 security-sentinel, spec-compliance-reviewer, research-quality-reviewer, document-quality-reviewer — 18/18 prior findings verified resolved; 2 new LOW optional-polish (RES-12, DOC-9)
2026-07-06 feature/spec-138-architecture-as-theory-evaluation (pass 2 — all 18 findings resolved, owner "fix all") PASS 0 0 0 0 authorial edits by owner directive; verdict unchanged (Defer, recommendation-for-ratification)
2026-07-06 feature/spec-138-architecture-as-theory-evaluation (pass 2) PASS (w/ recs) 0 2 7 8 spec-compliance-reviewer, research-quality-reviewer, document-quality-reviewer
2026-07-06 feature/spec-138-architecture-as-theory-evaluation PASS (w/ recs) 0 1 5 6 spec-compliance-reviewer, research-quality-reviewer, document-quality-reviewer
2026-05-18 feature/spec-120-domain-dashboard-pm-meetings (Pass 2 / post-simplify) PASS 0 0 2 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-18 feature/spec-120-domain-dashboard-pm-meetings PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-15 feature/spec-139-supply-chain-security-hook PASS 0 0 2 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-19 feature/spec-002-security-review-enforcement PASS 0 0 2 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-19 feature/spec-002-security-review-enforcement PASS 0 0 2 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-19 feature/spec-004-scaffold-template-patterns PASS 0 0 2 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-20 feature/spec-008-knowledge-inbox-pipeline PASS 0 0 4 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-21 feature/vt-c-content-evaluate-skill PASS 0 0 3 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-21 feature/spec-006-workflow-sequence-enforcement PASS 0 0 3 5 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-21 feature/spec-010-web-capture PASS 0 0 3 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-22 feature/spec-014-unified-init PASS 0 0 4 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-25 feature/spec-021-ims-plugin-extraction PASS 0 0 1 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-25 feature/spec-022-session-start-notifications PASS 0 0 3 5 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-26 feature/spec-023-deploy-gate-integration PASS 0 0 4 9 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-26 feature/spec-024-review-autofix-loop PASS 0 0 2 4 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-28 feature/spec-025-mcp-configuration-patterns PASS 0 0 1 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-28 main (11 merged branches post-SPEC-025) PASS 0 1 3 4 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-28 fix/cli-import-pipeline-data-root PASS 0 0 4 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-02-28 fix/cli-import-pipeline-data-root (re-review) PASS 0 0 4 5 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-01 feature/spec-032-agent-worktree-isolation PASS 0 0 2 4 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-01 feature/spec-032-agent-worktree-isolation (re-review) PASS 0 0 1 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-02 feature/spec-030-intelligent-skill-activation PASS 0 0 3 2 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-02 feature/spec-029-skill-permission-governance PASS 0 1 2 5 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-02 feature/spec-029-skill-permission-governance (re-review) PASS 0 1 3 5 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-02 feature/spec-029-skill-permission-governance (re-review 2) PASS 0 1 3 8 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-03 feature/spec-039-ufi-country-enrichment PASS 0 0 1 0 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-04 feature/spec-46-config-drift-audit PASS 0 0 1 3 security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-08 feature/spec-028-security-review-pipeline PASS 0 0 2 5 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-09 feature/spec-054-wave-dependency-execution PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-09 feature/spec-049-workflow-merger PASS 0 0 0 2 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-09 feature/spec-033-claudemd-lifecycle PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-09 feature/spec-051-spec-from-requirements PASS 0 0 1 2 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-09 feature/spec-045-commit-documentation-traceability PASS 0 0 1 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-09 feature/spec-041-adversarial-review-framing PASS 0 0 1 1 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-17 feature/spec-058-mcp-config-governance PASS 0 0 1 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-17 feature/knowledge-index PASS 0 0 5 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-17 feature/knowledge-index (re-review) PASS 0 0 5 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-17 feature/v000-projektupdates PASS 0 1 3 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist, document-quality-reviewer
2026-03-17 feature/spec-072-cross-file-impact-analysis PASS 0 0 2 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-18 feature/spec-078-state-documentation-standard PASS 0 0 4 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-18 feature/spec-075-strategic-mentor-agent PASS 0 0 2 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-18 feature/spec-079-repo-evaluation-safety PASS 0 0 3 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-18 feature/spec-081-simplify-skill-enhancement PASS 0 0 2 2 document-quality-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-18 feature/spec-081-simplify-skill-enhancement (pass 2) PASS 0 0 2 4 spec-compliance-reviewer, document-quality-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, accessibility-reviewer
2026-03-19 feature/spec-076-ims-audit-extensions PASS 0 0 3 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-19 feature/spec-077-advanced-diagram-generation PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, accessibility-reviewer
2026-03-19 claude/review-next-spec-BvgfC (SPEC-054 re-review) PASS 0 0 3 2 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-19 claude/review-next-spec-BvgfC (SPEC-075) PASS 0 0 1 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-19 claude/review-next-spec-BvgfC (SPEC-076) PASS 0 0 0 0 29 tests pass, 0 regressions
2026-03-19 claude/review-next-spec-BvgfC (SPEC-078) PASS 0 0 0 0 Plan validated in 2 iterations, all 6 FRs covered
2026-03-23 feature/spec-93-worktree-activate-integration PASS 0 0 0 2 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-23 feature/spec-86-brownfield-repo-onboarding PASS 0 0 3 7 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-23 feature/spec-88-secret-detection-hooks PASS 0 1 1 8 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-23 feature/spec-83-adr-workflow PASS 0 0 1 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-23 feature/spec-84-garbage-collection-agent PASS 0 0 2 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-23 feature/spec-87-diagram-auto-validation PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-24 feature/spec-090-user-manual-generation PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, accessibility-reviewer
2026-03-24 feature/spec-091-user-manual-update PASS 0 0 3 5 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-24 feature/spec-095-workflow-gate-enforcement PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-24 feature/spec-096-ims-cross-file-gaps PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-03-28 feature/spec-097-ims-to-vms-rename FAIL 1 5 1 0 spec-compliance-reviewer, security-sentinel, pattern-recognition-specialist, code-simplicity-reviewer
2026-03-28 feature/spec-097-ims-to-vms-rename (pass 2) PASS 0 2 2 0 spec-compliance-reviewer, security-sentinel, pattern-recognition-specialist, code-simplicity-reviewer
2026-03-27 feature/spec-097-ims-to-vms-rename (pass 3) PASS 0 2 4 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-28 feature/spec-097-ims-to-vms-rename (pass 4) PASS 0 1 2 1 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-27 feature/spec-097-ims-to-vms-rename (pass 5) PASS 0 0 1 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-29 feature/spec-098-skill-testing-framework PASS 0 2 5 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-30 feature/spec-100-designmd-pattern-evaluation PASS 0 0 4 1 spec-compliance-reviewer, research-quality-reviewer
2026-03-27 feature/spec-101-design-system-infrastructure PASS 0 3 3 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-27 feature/spec-101-design-system-infrastructure (pass 2) FAIL 1 1 3 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-27 feature/spec-101-design-system-infrastructure (pass 3) PASS 0 0 3 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-30 feature/spec-102-memory-consolidation PASS 0 0 2 2 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-03-30 feature/spec-099-remotion-video-skill PASS 0 0 3 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-30 feature/spec-103-security-hardening PASS 0 0 4 6 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-03-31 feature/spec-104-memory-consolidation-dream PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-30 feature/spec-105-recursive-criticism-improvement PASS 0 0 3 2 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-31 feature/spec-106-autoresearch-optimization-agent PASS 0 0 0 4 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-03-31 feature/spec-107-dream-open-brain PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-04-20 feature/spec-114-security-baseline-hardening PASS 0 1 3 9 security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-04-20 feature/spec-114-security-baseline-hardening (pass 2) PASS 0 0 0 6 security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-06 feature/spec-69-cross-referenz-handling PASS 0 0 0 6 (auto-fixed) spec-compliance-reviewer, security-sentinel, kieran-typescript-reviewer, julik-frontend-races-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-11 feature/spec-124-toolkit-install-defensive PASS 0 0 0 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-13 feature/spec-123-repo-scoped-vs-global-skills FAIL 1 1 2 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-05-13 feature/spec-123-repo-scoped-vs-global-skills (pass 2) FAIL 1 1 1 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-05-13 feature/spec-123-repo-scoped-vs-global-skills (pass 3) PASS 0 0 1 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-05-14 feature/spec-125-skill-orchestration-map PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-05-14 feature/spec-133-quick-fix-intake-path PASS 0 0 1 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist, kieran-typescript-reviewer
2026-05-14 feature/spec-133-quick-fix-intake-path (pass 2) PASS 0 0 2 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-05-18 feature/spec-120-domain-dashboard-pm-meetings PASS 0 0 2 4 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist
2026-05-19 feature/spec-126-worktree-env-propagation PASS 0 0 1 1 spec-compliance-reviewer, document-quality-reviewer, pattern-recognition-specialist
2026-05-19 feature/spec-122-prioritisierbare-spec-task-pipeline PASS 0 0 1 14 spec-compliance (manual), security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, data-integrity-guardian
2026-05-20 feature/spec-142-pm-bugs-detail-skill PASS 0 0 0 5 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, constitution-compliance
2026-05-21 feature/spec-137-smart-shape-prompt PASS 0 0 0 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-07-02 feature/spec-145-compound-engineering-pattern-adoption PASS 0 0 4 10 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist, performance-oracle, document-quality-reviewer, built-in /security-review
2026-07-04 feature/spec-129-claude-code-otel-monitoring PASS 0 1 6 14 spec-compliance-reviewer, security-sentinel, kieran-python-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-07-05 feature/spec-129-claude-code-otel-monitoring (pass 2) PASS 0 0 3 12 security-sentinel, kieran-python-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, document-quality-reviewer
2026-07-05 feature/spec-150-registration-check-entry-points PASS (w/ recs) 0 1 1 6 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-07-05 feature/spec-150-registration-check-entry-points (pass 2) PASS 0 0 0 7 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, pattern-recognition-specialist
2026-07-06 feature/spec-134-pd-prototype-refinement-loop PASS (w/ recs) 0 1 1 6 spec-compliance-reviewer (manual), security-sentinel, document-quality-reviewer, code-simplicity-reviewer
2026-07-06 feature/spec-134-pd-prototype-refinement-loop (pass 2) PASS 0 0 0 6 spec-compliance-reviewer (manual), document-quality-reviewer
2026-07-06 feature/spec-135-pd-prototype-decision-log-backflow PASS 0 0 0 1 (6 auto-fixed) spec-compliance-reviewer (manual), document-quality-reviewer, code-simplicity-reviewer
2026-07-21 feature/spec-157-theme-polarity FAIL 5 10 18 15 security-sentinel, spec-compliance-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, julik-frontend-races-reviewer, accessibility-reviewer, design-implementation-reviewer, document-quality-reviewer
2026-07-21 feature/spec-157-theme-polarity (pass 2) FAIL 4 15 27 17 security-sentinel, spec-compliance-reviewer, accessibility-reviewer, design-implementation-reviewer, julik-frontend-races-reviewer, document-quality-reviewer, pattern-recognition-specialist, code-simplicity-reviewer
2026-07-24 feature/spec-161-proactive-openbrain-capture PASS (w/ recs) 0 2 6 3 spec-compliance-reviewer, security-sentinel, code-simplicity-reviewer, document-quality-reviewer, pattern-recognition-specialist, performance-oracle, tdd-compliance-reviewer — 2H = sanitizer strips less than its §10a claim (SEC-1: lowercase bearer/Google/JWT/hex/client_secret= survive) + SEC-2: --check narrower than sanitizer masks it; PAT-1: secret-scanner "reuse" is a diverged copy
2026-07-27 feature/spec-165-concurrency-safe-spec-numbering FAIL 0 6 14 19 spec-compliance-reviewer (PASS), security-sentinel, kieran-python-reviewer, code-simplicity-reviewer, performance-oracle, pattern-recognition-specialist, tdd-compliance-reviewer, accessibility-reviewer (0 findings), julik-frontend-races-reviewer — architecture unchallenged; 3 of 6 H reintroduce duplicate IDs by independent routes (PY-1 scan_refs anchored on cwd so the prefix-relative ls-tree floor returns 0 from a subdirectory; PY-2/3 non-atomic write_counter + read_counter collapsing absent-vs-corrupt into a silent 0; RACE-1/2 degraded path reserves nothing across the whole Step-0→Step-5 window). Plus SPEC-1 (EC-3 probe disabled — no --specs-dir), PAT-1 (T11 whole-file greps defeat the R4 anti-bypass guard), SEC-1 (strict decode escapes unguarded → exit 1, defeats NFR-1). Tests NOT_RUN (permission system refused bash/python3 in the fork). kieran-python-reviewer added manually — persona-select.sh selected NO language reviewer for an all-Python deliverable. Disposition 2026-07-28: all 39 findings closed — 37 fixed, 2 accepted with measurement (PERF-1's content-OID dedupe was implemented and measured WORSE, 26 vs 20 git spawns, then reverted; PERF-2 declined because hoisting the scans collapses the window T5's negative control depends on). Suite 29 → 62 assertions, tests PASS (the fork's refusal did not recur). Reviewer-selection defect filed as intake/pending/bugs/2026-07-28-persona-select-python-marker-blind-to-plugin-scripts.md. Awaiting pass 2.
2026-07-28 feature/spec-165-concurrency-safe-spec-numbering PASS 0 6 24 ~35 Pass 2, ten reviewers (nine returned; kieran-python-reviewer completed late after repeated permission denials — the persona-select Python-marker defect from pass 1 is still open). The architectural finding: pass 1's own RACE-1/RACE-2 fix was one-directional. It gave the degraded path an O_EXCL marker but left the locked path reserving via the counter alone — two media, and a counter write is not atomic against a marker create, so a locked and a timed-out claimant could return the same ID (RACE-10, independently found as K2-1). Fixed by making the marker the arbiter on BOTH paths; the counter is demoted to a floor hint. Severity corrected: kieran's determinism argument is logically inverted (D − D_B < 5 means the loser reads the POST-write counter); measured 0 duplicates in 20 mixed-mode rounds pre-fix, so it is a ~1 ms window, not a certainty — fixed on code inspection, asserted structurally (T7c), not reproduced. Other Highs: RACE-11 (unlocked counter lift could LOWER the counter and destroy a locked claimant's only record — deleted), S2-1/S2-4 (unquoted --specs-dir shell interpolation from repo-tracked .design-state.yaml, the class this same file documents 280 lines further down), K2-2 (counter validated for syntax but not plausibility → one date +%s poisons the clone forever), K2-3 (docstring asserted three-source parity that never existed; live mismatch in-repo). T5's negative control had to be rebuilt: --no-lock alone no longer duplicates because O_EXCL is atomic without the lock — the lock is no longer the duplicate-prevention mechanism, so the control was re-pointed at the marker via a test-only VT_SPEC_NO_MARKER. Suite 62 → 84, 84/84 green over five runs. 5 findings accepted with rationale (SIMP-9 counter now arguably redundant, K2-10 dataclass, PERF2-5 cat-file --batch, SIMP-11, PAT2-17).

Recurring Findings

Finding Type Frequency First Seen Agent Action Taken
A fix that resolves a duplication finding by adding another copy 2 2026-07-21 pattern-recognition-specialist, design-implementation-reviewer, code-simplicity-reviewer 2x — SPEC-157 Pass 1 C1 said the canonical PrimeNG preset lacked dark mode; the Pass-2 fix (18770f6d) resolved it by copying a 145-line preset into bootstrap/SKILL.md rather than pointing at the generator, taking the anthracite ramp from three unlinked copies to five and shipping the wrong brand orange in the new copy (P2-C3). Guard: when a finding names a pointer as broken, the fix must repair the pointer, not clone the target. Candidate for docs/solutions/patterns/.
A documented pattern meant to be copied ships with a defect, multiplying it 2 2026-07-21 julik-frontend-races-reviewer 2x — SPEC-157 P2-C1/P2-C2: the theme switcher in visitrans-design-system/SKILL.md captures its toggle with an unguarded querySelector and never applies color-scheme at load, and pd-3-prototype/SKILL.md:311 was changed in the same branch to mandate copying it into every Angular product UI. Guard: a snippet a skill tells agents to copy is a distribution channel — review it at the severity of the fleet, not of the page.
Unrelated changes bundled in feature branch 17 2026-02-19 code-simplicity-reviewer, pattern-recognition-specialist, security-sentinel Recurring 17x — promoted to critical-patterns.md at 3x. Still occurring. Consider tooling enforcement.
argument-hint in new SKILL.md despite documentation saying not to 9 2026-02-20 pattern-recognition-specialist, code-simplicity-reviewer 9x — SPEC-104 dream skill also includes argument-hint. Pre-existing from SPEC-102 creation. Documented as "harmlessly ignored" in vt-base-rules.md.
plugin.json description inconsistent with constitution 4 2026-02-21 code-simplicity-reviewer, pattern-recognition-specialist 4x — SPEC-120 PAT-1: description claims "54 agents, 123 skills" / components: { agents: 55, skills: 123 } while actual manifests have 49 agents + 119 skills. SPEC-120 incremented 122→123 correctly (one new skill); drift was pre-existing. P-003 pattern already documented. Recurrence shows the pattern is not self-healing — needs tooling (pre-commit gate that asserts components.* match manifest line count).
Stale test gate from different branch 22 2026-02-25 code-simplicity-reviewer, vt-c-4-review 22x -- SPEC-165 review (2026-07-27) found BOTH .test-gate.md (feature/spec-161-*, PASS 59/59) and .review-gate.md (SPEC-161, PASS) forked into this worktree; both replaced. Note the compounding hazard this time: the review could not execute tests, so had the stale PASS been trusted, finalize would have read a 59/59 PASS belonging to another spec entirely. Prior: SPEC-161 review (2026-07-24) found BOTH .test-gate.md and .review-gate.md carrying BUG-003 (fix/bug-003-visimatch-contrast) content, forked into this worktree; refreshed to current branch + PASS. Prior: SPEC-129 review found gates carrying SPEC-149 content; SPEC-150 review (2026-07-05) then found BOTH .test-gate.md and .review-gate.md carrying feature/spec-151-multi-worktree-conventions content; SPEC-134 review (2026-07-06) found .test-gate.md carrying spec-150 content and .review-gate.md carrying spec-131 content; SPEC-135 review (2026-07-06) found BOTH gates still carrying SPEC-134 content — refreshed again. Same root cause: ephemeral gates persist across worktree switches. ≥3 threshold long surpassed — auto-invalidation hook still does not fire on worktree switch. Strong candidate for tooling enforcement (assert gate branch: == current branch before finalize reads it).
Fork-unsafe interactive prompt (AskUserQuestion in context:fork skill) 1 2026-07-05 pattern-recognition-specialist 1x -- SPEC-150 PAT-1: single-source Step 0.5 block (byte-identical across 11 stubs) names AskUserQuestion, but 2 of the 11 hosts (4-review, 5-finalize) are context:fork and forbid it. Root cause: a byte-identical shared block cannot be fork-aware. RESOLVED Pass 2 (2026-07-05): fork variants forked to a non-interactive report-only branch (no AskUserQuestion, no --record-skip); verified by pattern-recognition-specialist. Monitor when shared prose blocks are injected into mixed fork/non-fork skills.
Hardcoded Airtable infrastructure IDs 3 2026-02-28 security-sentinel, pattern-recognition-specialist 3x -- main review flagged table IDs and hierarchy record IDs; first fix review flagged hierarchy IDs moved to config without env-var override; re-review confirms same. Promote to critical-patterns.md when file is created.
Incomplete bulk rename (directory renamed but body references not updated) 1 2026-03-28 spec-compliance-reviewer, pattern-recognition-specialist 1x -- SPEC-097 renamed directories and frontmatter but left 72+ body references unchanged. Auto-fixed in Pass 2.
Hardcoded user path in committed settings 1 2026-03-28 security-sentinel 1x -- .claude/settings.json committed with /Users/rolf/ path. Related to P-002.
Test asserts substring-on-whole-file instead of structural location 3 2026-05-13 pattern-recognition-specialist, code-simplicity-reviewer, tdd-compliance-reviewer 3x — SPEC-165 (2026-07-27, PAT-1 HIGH): T11's five structural assertions are all grep -qF <needle> <whole SKILL.md>, exactly what structural-test-assertions.md forbids. Moving the pinned invocation into an ## Examples section and deleting Step 0 item 5 leaves every assertion green and the feature dead — and grep -qF being a prefix match means --no-lock could be appended to the invocation undetected (TDD-8). The spec's plan names this the top risk (R4, "one prompt line from silent bypass") and cites this very pattern as reused. Third occurrence in an implementation that CLAIMED the pattern — the pattern doc is not preventing the defect it documents; the reusable extract_* helper at structural-test-assertions.md:89-110 is being read as illustration rather than as required API. Recommend a mechanical guard: flag any grep -qF in tests/ whose target is a bare *.md path. SPEC-165 disposition (2026-07-28): T11 rebuilt correctly — step0_region() extracts the region first and every positive is paired with a mutated copy that must fail the identical check (invocation relocated out of Step 0; notice deleted), plus a new assertion forbidding --no-lock/--no-shared on the pinned line, closing the prefix-match hole. The instance is fixed; the mechanical guard is still unbuilt and is the actual remedy for a 3x recurrence in implementations that each claimed the pattern. It is a cross-cutting test-lint affecting every suite in the repo, so it needs its own spec rather than a ride-along. Prior — SPEC-123 PAT-1: T2/T11/T12 used grep -qF over whole SKILL.md to verify generated paths: block; tests passed even when block landed in document body (Critical SPEC-1 not caught). Fixed in Pass 2 with frontmatter-assertions.sh + extract_frontmatter helper. SPEC-135 (2026-07-06, TEST-1 HIGH): the decision-log drift-guard asserted frontmatter keys with unanchored grep -q "$key" over the extracted block — a dropped YAML key (iteration:/rationale:) survived as a prose word (false-green). Auto-fixed by anchoring to ^[[:space:]]*${key}: and proven to reddens on a perturbed block. Same anti-false-green class as SPEC-131 M1 / structural-test-assertions.md — anchor to the structural line, don't substring the region.

Agent Effectiveness

Agent Reviews Run Findings Critical Catches
security-sentinel 80 97 0
kieran-typescript-reviewer 43 7 0
kieran-python-reviewer 3 20 0
julik-frontend-races-reviewer 46 37 2
code-simplicity-reviewer 84 206 0
performance-oracle 69 45 0
pattern-recognition-specialist 82 202 1
spec-compliance-reviewer 56 36 1
document-quality-reviewer 18 64 2
accessibility-reviewer 6 31 2
design-implementation-reviewer 2 23 1
research-quality-reviewer 2 10 0
data-integrity-guardian 1 3 0
tdd-compliance-reviewer 2 12 0

Notes on selection (SPEC-165, 2026-07-27)

  • kieran-python-reviewer was the highest-yield reviewer of the review (2 of 6 High) and would not have been selected: persona-select.sh:93 gates has_py on pyproject.toml / setup.py / requirements.txt in PROJECT_ROOT, and this repo has none (only requirements-docs.txt) despite shipping 47 .py files under plugins/*/skills/*/scripts/. With diff_known && !has_ts && !has_py, want_lang is false and no language reviewer is selected at all. Same silent-drop class as BUG-004. Fix: add a .py-in-diff signal to has_py.
  • julik-frontend-races-reviewer and accessibility-reviewer were selected only because an inherited static docs/evaluations/vtd-vs-v025.html matched the frontend pattern — yet julik, re-aimed at process/file races rather than DOM races, produced 2 of the 6 High. Worth remembering that the races specialist generalises beyond the browser.