fix(lint): Fix remaining ESLint errors in agents package

- scan-fix-executor.ts: Remove unnecessary escape chars in regex
- python-tool-parser.ts: Add braces to case block, use import instead of require
- v9-analysis-pipeline.ts: Add braces to default case, add comment to empty function

Lint: ✅ 0 errors (2077 warnings pre-existing)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: alpsla

packages/agents/src/fix-agent/infrastructure/supabase/framework-pattern-storage.ts

@@ -404,7 +404,7 @@ export class FrameworkPatternStorage {
   async recordPatternApplication(
     patternId: string,
     success: boolean,
-    reverted: boolean = false
+    reverted = false
   ): Promise<void> {
     await this.initialize();
 

packages/agents/src/fix-agent/scan-fix-executor.ts

@@ -241,7 +241,7 @@ const AI_ERROR_PATTERNS: RegExp[] = [
  */
 const VALID_CODE_INDICATORS: RegExp[] = [
   /^(?:import|from|const|let|var|function|class|def|async|export|return)\s/m,  // Code keywords at line start
-  /[{}\[\]();]/,  // Contains common code syntax
+  /[{}[\]();]/,  // Contains common code syntax
   /=\s*[^=]/,  // Assignment (not comparison)
   /\.\w+\(/,  // Method calls
 ];
@@ -283,7 +283,7 @@ export function validatePatternTemplate(
   if (codeToCheck.length > 50 && !hasCodeIndicators) {
     // Check if it's mostly natural language (high letter-to-symbol ratio)
     const letters = (codeToCheck.match(/[a-zA-Z]/g) || []).length;
-    const symbols = (codeToCheck.match(/[{}\[\]();=<>]/g) || []).length;
+    const symbols = (codeToCheck.match(/[{}[\]();=<>]/g) || []).length;
     const ratio = symbols > 0 ? letters / symbols : letters;
 
     if (ratio > 20) {  // Very high letter-to-symbol ratio = likely prose

packages/agents/src/fix-agent/tool-fixers/dependency-fixer.ts

@@ -433,7 +433,7 @@ export class DependencyFixerExecutor extends ToolExecutorBase {
   parseVulnerabilityFromMessage(
     message: string,
     rule: string,
-    severity: string = 'medium'
+    severity = 'medium'
   ): DependencyVulnerability | null {
     // Try to extract package name from various formats
     let packageName: string | null = null;

packages/agents/src/fix-agent/tool-fixers/python-fixer.ts

@@ -504,7 +504,7 @@ export function isPythonVulnerabilityAutoFixable(tool: string, rule: string): bo
 export function parsePythonVulnerabilityFromMessage(
   message: string,
   rule: string,
-  severity: string = 'medium'
+  severity = 'medium'
 ): PipAuditVulnerability | null {
   // Try to extract package name from message
   // Format: "package-name has vulnerability CVE-2023-xxxx"

packages/agents/src/two-branch/parsers/python-tool-parser.ts

@@ -9,6 +9,7 @@
  */
 
 import { exec as execCallback } from 'child_process';
+import { existsSync } from 'fs';
 import { promisify } from 'util';
 
 const exec = promisify(execCallback);
@@ -793,12 +794,13 @@ export class PythonToolParser {
 
     const prefix = code.charAt(0);
     switch (prefix) {
-      case 'S':  // Security rules - always high/critical
+      case 'S': {  // Security rules - always high/critical
         // S1xx = low, S2xx = medium, S3xx-S7xx = high
         const secNum = parseInt(code.substring(1));
         if (secNum >= 300) return 'high';
         if (secNum >= 200) return 'medium';
         return 'low';
+      }
       case 'E':  // Errors
         return 'high';
       case 'F':  // Pyflakes (undefined names, etc.)
@@ -834,8 +836,7 @@ export class PythonToolParser {
       // When vulnerabilities found, exits with code 1 but stdout still contains JSON
       // Try requirements.txt first, fallback to environment scan
       const requirementsTxt = `${repoPath}/requirements.txt`;
-      const fs = require('fs');
-      const command = fs.existsSync(requirementsTxt)
+      const command = existsSync(requirementsTxt)
         ? `cd ${repoPath} && pip-audit --format json -r requirements.txt`
         : `cd ${repoPath} && pip-audit --format json`;
 

packages/agents/src/two-branch/report/business-impact.ts

@@ -161,7 +161,7 @@ function getPreCommitHookRecommendation(language: string): string {
  * Includes financial impact, risk assessment, and recommendations
  * SESSION 50 FIX: Added language parameter for language-specific recommendations
  */
-export function generateBusinessImpact(issues: EnrichedIssue[], groups: IssueGroup[], language: string = 'java'): string {
+export function generateBusinessImpact(issues: EnrichedIssue[], groups: IssueGroup[], language = 'java'): string {
   // BLOCKERS ONLY: NEW/EXISTING_MODIFIED + critical/high
   const blocking = issues.filter(i =>
     (i.category === 'NEW' || i.category === 'EXISTING_MODIFIED') &&

packages/agents/src/two-branch/services/v9-analysis-pipeline.ts

@@ -181,14 +181,15 @@ async function getToolOrchestrator(language: SupportedLanguage): Promise<ToolOrc
     case 'rust':
     case 'ruby':
     case 'php':
-    default:
+    default: {
       // For languages without specialized orchestrator or patterns,
       // warn and use TypeScript orchestrator as fallback
       // TODO: Create language-specific orchestrators and calibrate patterns
       console.warn(`[Pipeline] Language '${language}' not yet calibrated. Using TypeScript orchestrator as fallback.`);
       console.warn(`[Pipeline] To add support: 1) Create orchestrator 2) Run pattern calibration`);
       const { TypeScriptToolOrchestrator } = await import('../tools/typescript/typescript-tool-orchestrator');
       return new TypeScriptToolOrchestrator();
+    }
   }
 }
 
@@ -359,7 +360,7 @@ export class V9AnalysisPipeline {
       // maxIssuesToFix is only set explicitly for testing
       mainBranchPath: '',
       prMetadata: {},
-      onProgress: () => {},
+      onProgress: () => { /* Default no-op progress handler */ },
       verbose: false,
       ...config,
     };

packages/agents/src/two-branch/tools/universal/codeql-runner.ts

@@ -272,7 +272,7 @@ export class CodeQLRunner extends UniversalToolBase {
   private sarifPath: string;
   private codeqlConfig: Required<CodeQLConfig>;
   private sourceHash: string | null = null;
-  private useDocker: boolean = false;  // ARM64 Docker execution mode
+  private useDocker = false;  // ARM64 Docker execution mode
 
   constructor(workspacePath: string, language: string, config: CodeQLConfig = {}) {
     // Merge with defaults (user config overrides defaults)

packages/agents/src/two-branch/tools/universal/dependency-check-runner.ts

@@ -79,7 +79,7 @@ export class UniversalDependencyCheckRunner extends UniversalToolBase {
   private pgDatabase: string;
   private pgUser: string;
   private pgPassword: string;
-  private dependencyCheckPath: string = 'dependency-check.sh'; // Default to PATH lookup
+  private dependencyCheckPath = 'dependency-check.sh'; // Default to PATH lookup
 
   /**
    * Find dependency-check.sh in common installation locations