@brainforge/core 3.1.7 → 3.1.8

package/dist/index.d.ts

@@ -8,7 +8,7 @@ type TaskStatus = 'todo' | 'in-progress' | 'done' | 'skipped';
 type TaskPriority = 'low' | 'medium' | 'high' | 'critical';
 type TaskType = 'backend' | 'frontend' | 'database' | 'devops' | 'security' | 'test' | 'docs' | 'general';
 type BugSeverity = 'blocker' | 'high' | 'medium' | 'low';
-type AgentId = 'orchestrator' | 'researcher' | 'codebase-mapper' | 'requirements-analyst' | 'architect' | 'planner' | 'backend-engineer' | 'frontend-engineer' | 'fullstack-engineer' | 'database-engineer' | 'devops-engineer' | 'security-reviewer' | 'test-engineer' | 'ui-reviewer' | 'debugger' | 'verifier' | 'docs-writer' | 'professor' | 'student-explainer' | 'ship-manager';
+type AgentId = 'orchestrator' | 'researcher' | 'codebase-mapper' | 'requirements-analyst' | 'architect' | 'planner' | 'backend-engineer' | 'frontend-engineer' | 'fullstack-engineer' | 'database-engineer' | 'devops-engineer' | 'security-reviewer' | 'test-engineer' | 'ui-reviewer' | 'debugger' | 'verifier' | 'docs-writer' | 'professor' | 'student-explainer' | 'ship-manager' | 'code-reviewer' | 'code-fixer' | 'pattern-mapper' | 'integration-checker' | 'nyquist-auditor' | 'ui-researcher' | 'plan-checker';
 interface ProjectConfig {
     name: string;
     description: string;

package/dist/index.js

@@ -1621,6 +1621,76 @@ var AGENT_REGISTRY = [
     forbidden: ["shipping with open critical bugs", "skipping memory update"],
     qualityChecks: ["no critical bugs in BUGS.md", "CHANGELOG entry accurate", "memory entry written"],
     taskTypes: ["general"]
+  },
+  {
+    id: "code-reviewer",
+    name: "Code Reviewer",
+    purpose: "Adversarially reviews source code for bugs, security issues, and quality defects. Assumes all code contains defects until proven otherwise.",
+    inputs: ["changed source files", "PLAN.md", "CONVENTIONS.md", "task context"],
+    outputs: ["REVIEW.md with severity-classified findings (critical / warning / info)"],
+    forbidden: ["modifying source code", "approving code with critical findings", "reviewing undeclared scope"],
+    qualityChecks: ["every critical finding has a concrete fix suggestion", "OWASP patterns checked", "no false positives from existence-equals-correctness assumption"],
+    taskTypes: ["backend", "frontend", "security", "general"]
+  },
+  {
+    id: "code-fixer",
+    name: "Code Fixer",
+    purpose: "Reads REVIEW.md and applies the minimal fix for each finding. One focused change per finding \u2014 no refactors, no scope creep.",
+    inputs: ["REVIEW.md", "source files referenced in findings", "CONVENTIONS.md"],
+    outputs: ["fixed source files", "FIX_SUMMARY.md listing each finding and resolution"],
+    forbidden: ["refactoring code not in a finding", "changing behaviour beyond the fix", "skipping regression tests"],
+    qualityChecks: ["every critical finding addressed", "regression test added for each fix", "build passes after all changes"],
+    taskTypes: ["backend", "frontend", "general"]
+  },
+  {
+    id: "pattern-mapper",
+    name: "Pattern Mapper",
+    purpose: "Before planning, maps each new file to the closest existing code analog. Gives the planner concrete templates instead of starting from scratch.",
+    inputs: ["list of files to create or modify", "RESEARCH.md", "codebase source files"],
+    outputs: ["PATTERNS.md \u2014 per-file role, data-flow classification, analog path, extracted code excerpts"],
+    forbidden: ["modifying source code", "inventing patterns not present in codebase", "reading more than 5 analogs per file"],
+    qualityChecks: ["every new file has an analog or explicit no-analog note", "code excerpts include line numbers", "cross-cutting patterns listed"],
+    taskTypes: ["general"]
+  },
+  {
+    id: "integration-checker",
+    name: "Integration Checker",
+    purpose: "After multiple phases complete, verifies all cross-phase wiring is intact. Assumes connections are broken until grep evidence proves otherwise.",
+    inputs: ["phase SUMMARY.md files", "source files", "API routes", "component tree", "REQUIREMENTS.md"],
+    outputs: ["INTEGRATION.md \u2014 wiring status, orphaned exports, broken E2E flows, auth coverage gaps"],
+    forbidden: ["treating file existence as proof of integration", "approving broken E2E flows", "skipping auth coverage check"],
+    qualityChecks: ["every public export has a consumer or explicit orphan note", "all user-facing flows traced end-to-end", "findings mapped to requirement IDs"],
+    taskTypes: ["backend", "frontend", "general"]
+  },
+  {
+    id: "nyquist-auditor",
+    name: "Nyquist Auditor",
+    purpose: "Fills test coverage gaps by generating behavioral tests that can genuinely fail. Adversarially assumes each uncovered path is broken until a passing test proves otherwise.",
+    inputs: ["coverage gaps list", "source files", "existing test files", "PLAN.md acceptance criteria"],
+    outputs: ["new test files", "COVERAGE.md with gap-fill status (FILLED / ESCALATED / SKIPPED)"],
+    forbidden: ["testing implementation internals instead of behaviour", "writing tests that cannot fail", "patching source code to make tests pass"],
+    qualityChecks: ["every gap reaches a terminal state", "tests assert observable outcomes", "no more than 3 debug iterations per failing test before escalation"],
+    taskTypes: ["test", "general"]
+  },
+  {
+    id: "ui-researcher",
+    name: "UI Researcher",
+    purpose: "Produces the UI-SPEC.md design contract before any UI is built: spacing scale, typography, colour allocation, component inventory, and copywriting templates.",
+    inputs: ["REQUIREMENTS.md", "RESEARCH.md", "existing design tokens", "component library config"],
+    outputs: ["UI-SPEC.md \u2014 prescriptive values (not recommendations) for every visual dimension"],
+    forbidden: ['vague guidance ("use consistent spacing")', "recommending options without selecting one", "skipping accessibility contrast ratios"],
+    qualityChecks: ["spacing scale has \u22654 steps", "typography has body + heading + code sizes defined", "colour palette includes accessible contrast pairs", "all key screens enumerated"],
+    taskTypes: ["frontend", "general"]
+  },
+  {
+    id: "plan-checker",
+    name: "Plan Checker",
+    purpose: "Goal-backward validates a PLAN.md before execution begins. Confirms the plan will actually achieve the phase goal \u2014 not just complete tasks.",
+    inputs: ["PLAN.md", "REQUIREMENTS.md", "ROADMAP.md phase goal", "prior phase SUMMARYs"],
+    outputs: ["PLAN_CHECK.md \u2014 PASS / BLOCK verdict with specific gap list and fix suggestions"],
+    forbidden: ["rubber-stamping a plan with gaps", "blocking on style preferences", "inventing requirements not in REQUIREMENTS.md"],
+    qualityChecks: ["every must-have requirement has a covering task", "no task modifies files outside declared scope", "dependency graph is acyclic", "acceptance criteria are verifiable"],
+    taskTypes: ["general"]
   }
 ];
 var REGISTRY_MAP = new Map(
@@ -6681,6 +6751,13 @@ function inferAgent(task) {
   if (task.type === "docs" || /readme|doc|changelog|comment|jsdoc/.test(text)) return "docs-writer";
   if (task.type === "frontend" || /component|ui|page|layout|style|css|form|modal|button|nav/.test(text)) return "frontend-engineer";
   if (task.type === "backend" || /api|endpoint|route|controller|service|handler|webhook|cron/.test(text)) return "backend-engineer";
+  if (/review|audit|inspect|analyse|analyze/.test(text) && !/security|auth/.test(text)) return "code-reviewer";
+  if (/fix|patch|remediat/.test(text) && /review|finding|bug/.test(text)) return "code-fixer";
+  if (/pattern|analog|template|scaffold/.test(text)) return "pattern-mapper";
+  if (/integrat|wiring|cross.phase|end.to.end|e2e/.test(text)) return "integration-checker";
+  if (/coverage|gap|nyquist|missing test/.test(text)) return "nyquist-auditor";
+  if (/ui.spec|design.contract|spacing|typography|colour|color/.test(text)) return "ui-researcher";
+  if (/plan.check|validate.plan|verify.plan/.test(text)) return "plan-checker";
   return "fullstack-engineer";
 }
 function topologicalSort(tasks) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainforge/core",
-  "version": "3.1.7",
+  "version": "3.1.8",
   "description": "Core engine for BrainForge AI — state management, AST mapper, skills, and AI adapters",
   "type": "module",
   "license": "MIT",