feat: incorporate lint rule recommendations and update agent routing for ESLint rule handling

Author: mubaidr

agents/gem-debugger.agent.md

@@ -131,6 +131,11 @@ Note: These skills complement workflow. Constitutional: NEVER implement — only
 - Estimate fix complexity: small | medium | large.
 - Prove-It Pattern: Recommend writing failing reproduction test FIRST, confirm it fails, THEN apply fix.
 
+### 5.2.1 ESLint Rule Recommendations
+IF root cause is recurrence-prone (common mistake, easy to repeat, no existing rule): recommend ESLint rule in `lint_rule_recommendations`.
+- Recommend custom only if no built-in covers pattern.
+- Skip: one-off errors, business logic bugs, environment-specific issues.
+
 ### 5.3 Prevention Recommendations
 - Suggest tests that would have caught this.
 - Identify patterns to avoid.
@@ -186,6 +191,7 @@ Note: These skills complement workflow. Constitutional: NEVER implement — only
     "root_cause": {"description": "string", "location": "string", "error_type": "runtime|logic|integration|configuration|dependency", "causal_chain": ["string"]},
     "reproduction": {"confirmed": "boolean", "steps": ["string"], "environment": "string"},
     "fix_recommendations": [{"approach": "string", "location": "string", "complexity": "small|medium|large", "trade_offs": "string"}],
+    "lint_rule_recommendations": [{"rule_name": "string", "rule_type": "built-in|custom", "eslint_config": "object", "rationale": "string", "affected_files": ["string"]}],
     "prevention": {"suggested_tests": ["string"], "patterns_to_avoid": ["string"]},
     "confidence": "number (0-1)"
   }

agents/gem-orchestrator.agent.md

@@ -382,7 +382,7 @@ After each agent completes, the orchestrator routes based on status AND extra fi
 | completed | gem-critic | verdict=pass | Aggregate findings, present to user |
 | completed | gem-critic | verdict=needs_changes | Include findings in status summary, proceed |
 | completed | gem-critic | verdict=blocking | Route findings to gem-planner for fixes (check extra.verdict, NOT status) |
-| completed | gem-debugger | - | IF code fix: delegate to gem-implementer. IF config/test/infra: delegate to original agent. |
+| completed | gem-debugger | - | IF code fix: delegate to gem-implementer. IF config/test/infra: delegate to original agent. IF lint_rule_recommendations: delegate to gem-implementer to update ESLint config. |
 | needs_revision | gem-browser-tester | - | gem-debugger → gem-implementer (if code bug) → gem-browser-tester re-verify. |
 | needs_revision | gem-devops | - | gem-debugger → gem-implementer (if code) or gem-devops retry (if infra) → re-verify. |
 | needs_revision | gem-implementer | - | gem-debugger → gem-implementer (with diagnosis) → re-verify. |
@@ -536,6 +536,7 @@ Blocked tasks (if any): task_id, why blocked (missing dep), how long waiting.
 - Handle Failure: If agent returns status=failed, evaluate failure_type field:
   - Transient: Retry task (up to 3 times).
   - Fixable: Delegate to `gem-debugger` for root-cause analysis. Validate confidence (≥0.7). Inject diagnosis. IF code fix → `gem-implementer`. IF infra/config → original agent. After fix → original agent re-verifies. Same wave, max 3 retries.
+  - IF debugger returns `lint_rule_recommendations`: Delegate to `gem-implementer` to add/update ESLint config with recommended rules. This prevents recurrence across the codebase.
   - Needs_replan: Delegate to gem-planner for replanning (include diagnosis if available).
   - Escalate: Mark task as blocked. Escalate to user (include diagnosis if available).
   - Flaky: (from gem-browser-tester) Test passed on retry. Log for investigation. Mark task as completed with flaky flag in plan.yaml. Do NOT count against retry budget.

plugins/gem-team/README.md

@@ -285,6 +285,7 @@ All agents share: Execution rules, Constitutional rules, Anti-Patterns, and Dire
 | 🔄 **Codebase Patterns** | Avoids reinventing the wheel |
 | 🪞 **Self-Critique** | Reflection step before output (0.85 confidence threshold) |
 | 🔬 **Root-Cause Diagnosis** | Stack trace analysis, regression bisection |
+| 🛡️ **Auto-Generated Lint Rules** | Debugger recommends ESLint rules for recurring error patterns to prevent recurrence |
 | 💬 **Constructive Critique** | Challenges assumptions, finds edge cases |
 | ⚡ **Magic Keywords** | Fast-track routing: agent names in input trigger direct delegation (e.g., "simplify this" → gem-code-simplifier, "critique" → gem-critic, "debug" → gem-debugger) |
 | 📚 **Docs-Code Parity** | Documentation auto-included for new features |
@@ -381,7 +382,7 @@ Agents consult only the sources relevant to their role. Trust levels apply:
 | Agent | Verification |
 |:------|:-------------|
 | Implementer | get_errors → typecheck → unit tests → contract tests (if applicable) |
-| Debugger | reproduce → stack trace → root cause → fix recommendations |
+| Debugger | reproduce → stack trace → root cause → fix recommendations → lint rules (if recurring pattern) |
 | Critic | assumption audit → edge case discovery → over-engineering detection → logic gap analysis |
 | Browser Tester | validation matrix → console → network → accessibility |
 | Reviewer (task) | OWASP scan → code quality → logic → task_completion_check → coverage_status |