mstro-app 0.4.44 → 0.4.45

package/server/services/websocket/quality-complexity.ts

@@ -2,9 +2,12 @@
 // Licensed under the MIT License. See LICENSE file for details.
 
 import { extname, relative } from 'node:path';
-import { runCommand, type SourceFile } from './quality-tools.js';
+import { chunkFileList, filesByExt, runCommand, type SourceFile } from './quality-tools.js';
 import { biomeDiagToFinding, type Ecosystem, FUNCTION_LENGTH_THRESHOLD, isBiomeComplexityDiagnostic, isEslintComplexityRule, type QualityFinding } from './quality-types.js';
 
+const NODE_COMPLEXITY_EXTS = ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs'];
+const PY_COMPLEXITY_EXTS = ['.py', '.pyi'];
+
 // ============================================================================
 // Function Length Analysis
 // ============================================================================
@@ -160,43 +163,62 @@ function computeComplexityScore(findings: QualityFinding[]): number {
   return Math.max(0, 100 - penalty);
 }
 
-async function complexityFromBiome(dirPath: string): Promise<QualityFinding[] | null> {
-  const result = await runCommand('npx', ['@biomejs/biome', 'lint', '--reporter=json', '.'], dirPath);
-  if (result.exitCode > 1) return null;
-
-  try {
-    const parsed = JSON.parse(result.stdout);
-    if (!parsed.diagnostics) return [];
-    return parsed.diagnostics
-      .filter(isBiomeComplexityDiagnostic)
-      .map((d: Record<string, unknown>) => biomeDiagToFinding(d, 'complexity'));
-  } catch {
-    return null;
+async function complexityFromBiome(dirPath: string, files: SourceFile[]): Promise<QualityFinding[] | null> {
+  const targets = filesByExt(files, NODE_COMPLEXITY_EXTS);
+  if (targets.length === 0) return [];
+
+  const findings: QualityFinding[] = [];
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('npx', ['@biomejs/biome', 'lint', '--reporter=json', ...chunk], dirPath);
+    if (result.exitCode > 1) return null;
+
+    try {
+      const parsed = JSON.parse(result.stdout);
+      if (!parsed.diagnostics) continue;
+      for (const d of parsed.diagnostics) {
+        if (isBiomeComplexityDiagnostic(d)) findings.push(biomeDiagToFinding(d, 'complexity'));
+      }
+    } catch {
+      return null;
+    }
+  }
+  return findings;
+}
+
+function eslintFileToComplexityFindings(
+  file: { filePath: string; messages?: Array<Record<string, unknown>> },
+  dirPath: string,
+): QualityFinding[] {
+  const out: QualityFinding[] = [];
+  for (const msg of file.messages || []) {
+    if (!isEslintComplexityRule(msg.ruleId as string)) continue;
+    out.push({
+      severity: msg.severity === 2 ? 'high' : 'medium',
+      category: 'complexity',
+      file: relative(dirPath, file.filePath),
+      line: (msg.line as number) ?? null,
+      title: (msg.ruleId as string) || 'complexity',
+      description: msg.message as string,
+    });
   }
+  return out;
 }
 
-async function complexityFromEslint(dirPath: string): Promise<QualityFinding[] | null> {
-  const result = await runCommand('npx', ['eslint', '--format=json', '.'], dirPath);
-  if (result.exitCode > 1 && !result.stdout.trim().startsWith('[')) return null;
+async function complexityFromEslint(dirPath: string, files: SourceFile[]): Promise<QualityFinding[] | null> {
+  const targets = filesByExt(files, NODE_COMPLEXITY_EXTS);
+  if (targets.length === 0) return [];
 
   const findings: QualityFinding[] = [];
-  try {
-    const parsed = JSON.parse(result.stdout);
-    for (const file of parsed) {
-      for (const msg of file.messages || []) {
-        if (!isEslintComplexityRule(msg.ruleId)) continue;
-        findings.push({
-          severity: msg.severity === 2 ? 'high' : 'medium',
-          category: 'complexity',
-          file: relative(dirPath, file.filePath),
-          line: msg.line ?? null,
-          title: msg.ruleId || 'complexity',
-          description: msg.message,
-        });
-      }
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('npx', ['eslint', '--format=json', ...chunk], dirPath);
+    if (result.exitCode > 1 && !result.stdout.trim().startsWith('[')) return null;
+
+    try {
+      const parsed = JSON.parse(result.stdout);
+      for (const file of parsed) findings.push(...eslintFileToComplexityFindings(file, dirPath));
+    } catch {
+      return null;
     }
-  } catch {
-    return null;
   }
 
   return findings;
@@ -215,28 +237,40 @@ function radonFuncToFinding(filePath: string, func: Record<string, unknown>): Qu
   };
 }
 
-async function complexityFromRadon(dirPath: string): Promise<QualityFinding[] | null> {
-  const result = await runCommand('radon', ['cc', '--json', '.'], dirPath);
-  if (result.exitCode !== 0 && !result.stdout.trim().startsWith('{')) return null;
-
-  try {
-    const parsed = JSON.parse(result.stdout) as Record<string, Array<Record<string, unknown>>>;
-    const findings: QualityFinding[] = [];
-    for (const [filePath, functions] of Object.entries(parsed)) {
-      for (const func of functions) {
-        const finding = radonFuncToFinding(filePath, func);
-        if (finding) findings.push(finding);
-      }
+function radonPayloadToFindings(payload: Record<string, Array<Record<string, unknown>>>): QualityFinding[] {
+  const out: QualityFinding[] = [];
+  for (const [filePath, functions] of Object.entries(payload)) {
+    for (const func of functions) {
+      const finding = radonFuncToFinding(filePath, func);
+      if (finding) out.push(finding);
+    }
+  }
+  return out;
+}
+
+async function complexityFromRadon(dirPath: string, files: SourceFile[]): Promise<QualityFinding[] | null> {
+  const targets = filesByExt(files, PY_COMPLEXITY_EXTS);
+  if (targets.length === 0) return [];
+
+  const findings: QualityFinding[] = [];
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('radon', ['cc', '--json', ...chunk], dirPath);
+    if (result.exitCode !== 0 && !result.stdout.trim().startsWith('{')) return null;
+
+    try {
+      const parsed = JSON.parse(result.stdout) as Record<string, Array<Record<string, unknown>>>;
+      findings.push(...radonPayloadToFindings(parsed));
+    } catch {
+      return null;
     }
-    return findings;
-  } catch {
-    return null;
   }
+  return findings;
 }
 
 async function analyzeNodeComplexity(
   dirPath: string,
   installed: Set<string> | null,
+  files: SourceFile[],
 ): Promise<QualityFinding[] | null> {
   const hasCapableTool = !installed || installed.has('biome') || installed.has('eslint');
   if (!hasCapableTool) return null;
@@ -245,24 +279,26 @@ async function analyzeNodeComplexity(
   // This fixes monorepo scenarios where biome.json is in a subdirectory.
   const hasBiome = !installed || installed.has('biome');
   if (hasBiome) {
-    const findings = await complexityFromBiome(dirPath);
+    const findings = await complexityFromBiome(dirPath, files);
     if (findings) return findings;
   }
-  return complexityFromEslint(dirPath);
+  return complexityFromEslint(dirPath, files);
 }
 
 async function analyzePythonComplexity(
   dirPath: string,
   installed: Set<string> | null,
+  files: SourceFile[],
 ): Promise<QualityFinding[] | null> {
   const hasRadon = !installed || installed.has('radon');
   if (!hasRadon) return null;
-  return complexityFromRadon(dirPath);
+  return complexityFromRadon(dirPath, files);
 }
 
 export async function analyzeComplexity(
   dirPath: string,
   ecosystems: Ecosystem[],
+  files: SourceFile[],
   installedToolNames?: string[],
 ): Promise<{ score: number; findings: QualityFinding[]; issueCount: number; available: boolean }> {
   const allFindings: QualityFinding[] = [];
@@ -272,7 +308,7 @@ export async function analyzeComplexity(
   for (const ecosystem of ecosystems) {
     const analyze = ecosystem === 'node' ? analyzeNodeComplexity : ecosystem === 'python' ? analyzePythonComplexity : null;
     if (!analyze) continue;
-    const findings = await analyze(dirPath, installed);
+    const findings = await analyze(dirPath, installed, files);
     if (findings) {
       canAnalyze = true;
       allFindings.push(...findings);

package/server/services/websocket/quality-handlers.ts

@@ -2,19 +2,18 @@
 // Licensed under the MIT License. See LICENSE file for details.
 
 /**
- * Quality Handlers — WebSocket message router for quality scanning,
- * code review, and fix operations.
+ * Quality Handlers — WebSocket message router for quality scanning
+ * and code review operations. Fixes are handled via chat tabs from the
+ * web client (see web/src/components/views/QualityView/qualityUtils.ts)
+ * so there is no server-side fix message anymore.
  *
  * Agent logic lives in focused modules:
  * - quality-review-agent.ts  — AI code review prompt, parsing, handler
- * - quality-fix-agent.ts     — AI fix prompt, progress tracking, handler
  */
 
 import { join, resolve } from 'node:path';
 import { validatePathWithinWorkingDir } from '../pathUtils.js';
 import type { HandlerContext } from './handler-context.js';
-import type { FindingForFix } from './quality-fix-agent.js';
-import { handleFixIssues } from './quality-fix-agent.js';
 import { QualityPersistence } from './quality-persistence.js';
 import { handleCodeReview } from './quality-review-agent.js';
 import { detectTools, installTools, runQualityScan } from './quality-service.js';
@@ -92,17 +91,6 @@ export function handleQualityMessage(
       handleCodeReview(ctx, ws, reportPath, dirPath, workingDir, activeReviews, getPersistence)
         .finally(() => persistence.clearActiveOperation(reportPath));
     },
-    qualityFixIssues: () => {
-      const { resolved: dirPath, error } = resolveAndValidatePath(workingDir, msg.data?.path, isSandboxed);
-      if (error) { sendPathError(msg.data?.path || '.', error); return; }
-      const reportPath = msg.data?.path || '.';
-      const section: string | undefined = msg.data?.section;
-      const findings: FindingForFix[] = msg.data?.findings || [];
-      const persistence = getPersistence(workingDir);
-      persistence.setActiveOperation(reportPath, 'fixing');
-      handleFixIssues(ctx, ws, reportPath, dirPath, workingDir, section, findings, getPersistence)
-        .finally(() => persistence.clearActiveOperation(reportPath));
-    },
     qualityLoadState: () => handleLoadState(ctx, ws, workingDir),
     qualityClearPending: () => {
       const persistence = getPersistence(workingDir);
@@ -145,8 +133,6 @@ async function handleLoadState(
           ctx.send(ws, { type: 'qualityScanResults', data: pending.data });
         } else if (pending.type === 'codeReview') {
           ctx.send(ws, { type: 'qualityCodeReview', data: pending.data });
-        } else if (pending.type === 'fixComplete') {
-          ctx.send(ws, { type: 'qualityFixComplete', data: pending.data });
         }
       }
       persistence.clearPendingResults();

package/server/services/websocket/quality-linting.ts

@@ -2,9 +2,12 @@
 // Licensed under the MIT License. See LICENSE file for details.
 
 import { relative } from 'node:path';
-import { runCommand, type SourceFile } from './quality-tools.js';
+import { chunkFileList, filesByExt, runCommand, type SourceFile } from './quality-tools.js';
 import { biomeDiagToFinding, type Ecosystem, isBiomeComplexityDiagnostic, isEslintComplexityRule, type QualityFinding } from './quality-types.js';
 
+const NODE_LINT_EXTS = ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs'];
+const PY_LINT_EXTS = ['.py', '.pyi'];
+
 interface LintAccumulator {
   errors: number;
   warnings: number;
@@ -31,17 +34,22 @@ function parseBiomeDiagnostics(stdout: string, acc: LintAccumulator): void {
   }
 }
 
-async function lintWithBiome(dirPath: string, acc: LintAccumulator): Promise<void> {
-  const result = await runCommand('npx', ['@biomejs/biome', 'lint', '--reporter=json', '.'], dirPath);
-  if (result.exitCode > 1) return;
+async function lintWithBiome(dirPath: string, acc: LintAccumulator, files: SourceFile[]): Promise<void> {
+  const targets = filesByExt(files, NODE_LINT_EXTS);
+  if (targets.length === 0) return;
 
-  acc.ran = true;
-  try {
-    parseBiomeDiagnostics(result.stdout, acc);
-  } catch {
-    acc.errors += (result.stdout.match(/error/gi) || []).length;
-    acc.warnings += (result.stdout.match(/warning/gi) || []).length;
-    acc.ran = acc.errors > 0 || acc.warnings > 0 || result.exitCode === 0;
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('npx', ['@biomejs/biome', 'lint', '--reporter=json', ...chunk], dirPath);
+    if (result.exitCode > 1) return;
+
+    acc.ran = true;
+    try {
+      parseBiomeDiagnostics(result.stdout, acc);
+    } catch {
+      acc.errors += (result.stdout.match(/error/gi) || []).length;
+      acc.warnings += (result.stdout.match(/warning/gi) || []).length;
+      acc.ran = acc.errors > 0 || acc.warnings > 0 || result.exitCode === 0;
+    }
   }
 }
 
@@ -64,58 +72,77 @@ function processEslintMessage(
   });
 }
 
-async function lintWithEslint(dirPath: string, acc: LintAccumulator): Promise<void> {
-  const result = await runCommand('npx', ['eslint', '--format=json', '.'], dirPath);
-  acc.ran = true;
-  try {
-    const parsed = JSON.parse(result.stdout);
-    for (const file of parsed) {
-      for (const msg of file.messages || []) {
-        processEslintMessage(msg, file.filePath, dirPath, acc);
+async function lintWithEslint(dirPath: string, acc: LintAccumulator, files: SourceFile[]): Promise<void> {
+  const targets = filesByExt(files, NODE_LINT_EXTS);
+  if (targets.length === 0) return;
+
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('npx', ['eslint', '--format=json', ...chunk], dirPath);
+    acc.ran = true;
+    try {
+      const parsed = JSON.parse(result.stdout);
+      for (const file of parsed) {
+        for (const msg of file.messages || []) {
+          processEslintMessage(msg, file.filePath, dirPath, acc);
+        }
       }
+    } catch {
+      acc.errors += (result.stderr.match(/error/gi) || []).length;
+      acc.warnings += (result.stderr.match(/warning/gi) || []).length;
     }
-  } catch {
-    acc.errors += (result.stderr.match(/error/gi) || []).length;
-    acc.warnings += (result.stderr.match(/warning/gi) || []).length;
   }
 }
 
-async function lintNode(dirPath: string, acc: LintAccumulator, installed: Set<string> | null): Promise<void> {
+async function lintNode(
+  dirPath: string,
+  acc: LintAccumulator,
+  installed: Set<string> | null,
+  files: SourceFile[],
+): Promise<void> {
   // Use installed tools list to decide which linter to run, not config file presence.
   // This fixes monorepo scenarios where the config is in a subdirectory.
   const hasBiome = !installed || installed.has('biome');
   const hasEslint = !installed || installed.has('eslint');
 
   if (hasBiome) {
-    await lintWithBiome(dirPath, acc);
+    await lintWithBiome(dirPath, acc, files);
     if (acc.ran) return;
   }
   if (hasEslint) {
-    await lintWithEslint(dirPath, acc);
+    await lintWithEslint(dirPath, acc, files);
   }
 }
 
-async function lintPython(dirPath: string, acc: LintAccumulator): Promise<void> {
-  const result = await runCommand('ruff', ['check', '--output-format=json', '.'], dirPath);
-  if (result.exitCode !== 0 && !result.stdout.trim().startsWith('[')) return;
+function processRuffItem(item: Record<string, unknown>, dirPath: string, acc: LintAccumulator): void {
+  const code = item.code as string | undefined;
+  const sev = code?.startsWith('E') ? 'high' : 'medium';
+  if (sev === 'high') acc.errors++;
+  else acc.warnings++;
+  const location = item.location as Record<string, unknown> | undefined;
+  acc.findings.push({
+    severity: sev,
+    category: 'lint',
+    file: item.filename ? relative(dirPath, item.filename as string) : '',
+    line: (location?.row as number) ?? null,
+    title: code || 'Lint issue',
+    description: (item.message as string) || '',
+  });
+}
 
-  acc.ran = true;
-  try {
-    const parsed = JSON.parse(result.stdout);
-    for (const item of parsed) {
-      const sev = item.code?.startsWith('E') ? 'high' : 'medium';
-      if (sev === 'high') acc.errors++;
-      else acc.warnings++;
-      acc.findings.push({
-        severity: sev,
-        category: 'lint',
-        file: item.filename ? relative(dirPath, item.filename) : '',
-        line: item.location?.row ?? null,
-        title: item.code || 'Lint issue',
-        description: item.message || '',
-      });
-    }
-  } catch { /* ignore */ }
+async function lintPython(dirPath: string, acc: LintAccumulator, files: SourceFile[]): Promise<void> {
+  const targets = filesByExt(files, PY_LINT_EXTS);
+  if (targets.length === 0) return;
+
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('ruff', ['check', '--output-format=json', ...chunk], dirPath);
+    if (result.exitCode !== 0 && !result.stdout.trim().startsWith('[')) continue;
+
+    acc.ran = true;
+    try {
+      const parsed = JSON.parse(result.stdout);
+      for (const item of parsed) processRuffItem(item, dirPath, acc);
+    } catch { /* ignore */ }
+  }
 }
 
 function processClippyMessage(msg: Record<string, unknown>, acc: LintAccumulator): void {
@@ -174,8 +201,8 @@ export async function analyzeLinting(
   const acc = newLintAccumulator();
   const installed = installedToolNames ? new Set(installedToolNames) : null;
 
-  if (ecosystems.includes('node')) await lintNode(dirPath, acc, installed);
-  if (ecosystems.includes('python')) await lintPython(dirPath, acc);
+  if (ecosystems.includes('node')) await lintNode(dirPath, acc, installed, files);
+  if (ecosystems.includes('python')) await lintPython(dirPath, acc, files);
   if (ecosystems.includes('rust')) await lintRust(dirPath, acc);
 
   if (!acc.ran) {

package/server/services/websocket/quality-persistence.ts

@@ -62,7 +62,7 @@ export interface ActiveOperation {
 }
 
 export interface PendingResult {
-  type: 'scanResults' | 'codeReview' | 'fixComplete';
+  type: 'scanResults' | 'codeReview';
   path: string;
   data: Record<string, unknown>;
   completedAt: string;

package/server/services/websocket/quality-service.ts

@@ -4,9 +4,12 @@
 import { extname } from 'node:path';
 import { analyzeComplexity, analyzeFunctionLength } from './quality-complexity.js';
 import { analyzeLinting } from './quality-linting.js';
-import { collectSourceFiles, detectEcosystem, runCommand, type SourceFile } from './quality-tools.js';
+import { chunkFileList, collectSourceFiles, detectEcosystem, filesByExt, runCommand, type SourceFile } from './quality-tools.js';
 import { type CategoryPenalty, type CategoryScore, type Ecosystem, FILE_LENGTH_THRESHOLD, hasInstalledToolInCategory, type QualityFinding, type QualityResults, type ScanProgress, type ScoreBreakdown, TOTAL_STEPS } from './quality-types.js';
 
+const NODE_FMT_EXTS = ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs'];
+const PY_FMT_EXTS = ['.py', '.pyi'];
+
 export { detectEcosystem, detectTools, installTools } from './quality-tools.js';
 // Re-export public API for backward compatibility
 export type { CategoryPenalty, CategoryScore, QualityFinding, QualityResults, QualityTool, ScanProgress, ScoreBreakdown } from './quality-types.js';
@@ -27,34 +30,51 @@ function newFmtAccumulator(): FmtAccumulator {
 }
 
 async function fmtNode(dirPath: string, files: SourceFile[], acc: FmtAccumulator): Promise<void> {
-  const result = await runCommand('npx', ['prettier', '--check', '.'], dirPath);
-  acc.ran = true;
-  const unformatted = result.stdout.split('\n').filter((l) => l.trim() && !l.startsWith('Checking'));
-  const nodeFiles = files.filter((f) => ['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs'].includes(extname(f.path)));
-  acc.totalFiles += nodeFiles.length;
-  acc.passingFiles += Math.max(0, nodeFiles.length - unformatted.length);
-  for (const filePath of unformatted) {
-    if (!filePath.trim()) continue;
-    const rel = filePath.startsWith('/') ? filePath.replace(`${dirPath}/`, '') : filePath;
+  const targets = filesByExt(files, NODE_FMT_EXTS);
+  if (targets.length === 0) return;
+
+  acc.totalFiles += targets.length;
+  const unformattedSet = new Set<string>();
+
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('npx', ['prettier', '--check', ...chunk], dirPath);
+    acc.ran = true;
+    for (const line of result.stdout.split('\n')) {
+      if (!line.trim() || line.startsWith('Checking')) continue;
+      const rel = line.startsWith('/') ? line.replace(`${dirPath}/`, '') : line;
+      unformattedSet.add(rel.trim());
+    }
+  }
+
+  acc.passingFiles += Math.max(0, targets.length - unformattedSet.size);
+  for (const rel of unformattedSet) {
     acc.findings.push({ severity: 'low', category: 'format', file: rel, line: null, title: 'File not formatted', description: 'Does not match Prettier formatting rules.' });
   }
 }
 
 async function fmtPython(dirPath: string, files: SourceFile[], acc: FmtAccumulator): Promise<void> {
-  const result = await runCommand('black', ['--check', '--quiet', '.'], dirPath);
-  acc.ran = true;
-  const pyFiles = files.filter((f) => ['.py', '.pyi'].includes(extname(f.path)));
-  acc.totalFiles += pyFiles.length;
-  if (result.exitCode === 0) {
-    acc.passingFiles += pyFiles.length;
-    return;
-  }
-  const reformatLines = result.stderr.split('\n').filter((l) => l.includes('would reformat'));
-  acc.passingFiles += Math.max(0, pyFiles.length - reformatLines.length);
-  for (const line of reformatLines) {
-    const match = line.match(/would reformat (.+)/);
-    if (match) acc.findings.push({ severity: 'low', category: 'format', file: match[1].trim(), line: null, title: 'File not formatted', description: 'Does not match Black formatting rules.' });
+  const targets = filesByExt(files, PY_FMT_EXTS);
+  if (targets.length === 0) return;
+
+  acc.totalFiles += targets.length;
+  let reformatCount = 0;
+  const findings: typeof acc.findings = [];
+
+  for (const chunk of chunkFileList(targets)) {
+    const result = await runCommand('black', ['--check', '--quiet', ...chunk], dirPath);
+    acc.ran = true;
+    if (result.exitCode === 0) continue;
+
+    const reformatLines = result.stderr.split('\n').filter((l) => l.includes('would reformat'));
+    reformatCount += reformatLines.length;
+    for (const line of reformatLines) {
+      const match = line.match(/would reformat (.+)/);
+      if (match) findings.push({ severity: 'low', category: 'format', file: match[1].trim(), line: null, title: 'File not formatted', description: 'Does not match Black formatting rules.' });
+    }
   }
+
+  acc.passingFiles += Math.max(0, targets.length - reformatCount);
+  acc.findings.push(...findings);
 }
 
 async function fmtRust(dirPath: string, files: SourceFile[], acc: FmtAccumulator): Promise<void> {
@@ -223,21 +243,6 @@ export function computeFormulaScore(
   };
 }
 
-/** @deprecated — use computeFormulaScore instead */
-export function computeAiReviewScore(
-  findings: Array<{ severity: string }>,
-  totalLines: number,
-): number {
-  if (findings.length === 0) return 100;
-  const effectiveKloc = Math.max(totalLines / 1000, 1.0);
-  const totalPenalty = findings.reduce(
-    (sum, f) => sum + (SEVERITY_WEIGHT[f.severity] ?? 2.0),
-    0,
-  );
-  const penaltyDensity = totalPenalty / effectiveKloc;
-  return Math.round(100 * Math.exp(-0.10 * penaltyDensity));
-}
-
 // ============================================================================
 // Main Scan
 // ============================================================================
@@ -278,7 +283,7 @@ export async function runQualityScan(
 
   // Step 4: Analyze complexity (using real tools: Biome, ESLint, radon)
   progress('Analyzing complexity', 4);
-  const complexityResult = await analyzeComplexity(dirPath, ecosystems, installedToolNames);
+  const complexityResult = await analyzeComplexity(dirPath, ecosystems, files, installedToolNames);
 
   // Step 5: Check file lengths
   progress('Checking file lengths', 5);

package/server/services/websocket/quality-tools.ts

@@ -239,6 +239,39 @@ export async function collectSourceFiles(dirPath: string, rootPath: string): Pro
   return files;
 }
 
+// ============================================================================
+// File List Helpers
+// ============================================================================
+
+/**
+ * Filter a SourceFile list by extension and return relative paths.
+ * Used to pass an explicit, git-ignored-filtered file list to external tools
+ * (prettier, biome, eslint, ...) so they don't walk ignored directories like
+ * web/dist or nested build outputs in monorepos.
+ */
+export function filesByExt(files: SourceFile[], exts: string[]): string[] {
+  const set = new Set(exts.map((e) => e.toLowerCase()));
+  const out: string[] = [];
+  for (const f of files) {
+    if (set.has(extname(f.path).toLowerCase())) out.push(f.relativePath);
+  }
+  return out;
+}
+
+/**
+ * Split a file list into chunks so a single command invocation doesn't
+ * blow past ARG_MAX. macOS ARG_MAX is ~256KB; 400 paths at ~200 chars each
+ * leaves plenty of headroom.
+ */
+export function chunkFileList(paths: string[], size = 400): string[][] {
+  if (paths.length === 0) return [];
+  const chunks: string[][] = [];
+  for (let i = 0; i < paths.length; i += size) {
+    chunks.push(paths.slice(i, i + size));
+  }
+  return chunks;
+}
+
 // ============================================================================
 // Command Runner
 // ============================================================================

package/server/services/websocket/types.ts

@@ -42,7 +42,7 @@ const SessionSyncMessages = ['getActiveTabs', 'createTab', 'reorderTabs', 'syncT
 
 const SettingsMessages = ['getSettings', 'updateSettings'] as const;
 
-const QualityMessages = ['qualityDetectTools', 'qualityScan', 'qualityInstallTools', 'qualityCodeReview', 'qualityFixIssues', 'qualityLoadState', 'qualitySaveDirectories'] as const;
+const QualityMessages = ['qualityDetectTools', 'qualityScan', 'qualityInstallTools', 'qualityCodeReview', 'qualityLoadState', 'qualitySaveDirectories'] as const;
 
 const FileUploadMessages = ['fileUploadStart', 'fileUploadChunk', 'fileUploadComplete', 'fileUploadCancel'] as const;
 
@@ -105,7 +105,7 @@ const SessionSyncResponseMessages = ['activeTabs', 'tabCreated', 'tabRemoved', '
 
 const SettingsResponseMessages = ['settings', 'settingsUpdated'] as const;
 
-const QualityResponseMessages = ['qualityToolsDetected', 'qualityScanProgress', 'qualityScanResults', 'qualityInstallProgress', 'qualityInstallComplete', 'qualityCodeReview', 'qualityCodeReviewProgress', 'qualityPostSession', 'qualityFixProgress', 'qualityFixComplete', 'qualityError', 'qualityStateLoaded'] as const;
+const QualityResponseMessages = ['qualityToolsDetected', 'qualityScanProgress', 'qualityScanResults', 'qualityInstallProgress', 'qualityInstallComplete', 'qualityCodeReview', 'qualityCodeReviewProgress', 'qualityPostSession', 'qualityError', 'qualityStateLoaded'] as const;
 
 const FileUploadResponseMessages = ['fileUploadAck', 'fileUploadReady', 'fileUploadError'] as const;
 

package/dist/server/services/websocket/quality-fix-agent.d.ts

@@ -1,16 +0,0 @@
-import type { ToolUseEvent } from '../../cli/headless/types.js';
-import type { HandlerContext } from './handler-context.js';
-import type { QualityPersistence } from './quality-persistence.js';
-import type { WSContext } from './types.js';
-export interface FindingForFix {
-    severity: string;
-    category: string;
-    file: string;
-    line: number | null;
-    title: string;
-    description: string;
-    suggestion?: string;
-}
-export declare function createToolProgressCallback(ctx: HandlerContext, ws: WSContext, reportPath: string): (event: ToolUseEvent) => void;
-export declare function handleFixIssues(ctx: HandlerContext, ws: WSContext, reportPath: string, dirPath: string, workingDir: string, section: string | undefined, findings: FindingForFix[], getPersistence: (dir: string) => QualityPersistence): Promise<void>;
-//# sourceMappingURL=quality-fix-agent.d.ts.map

package/dist/server/services/websocket/quality-fix-agent.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"quality-fix-agent.d.ts","sourceRoot":"","sources":["../../../../server/services/websocket/quality-fix-agent.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,6BAA6B,CAAC;AAEhE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAC3D,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AAEnE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAI5C,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAYD,wBAAgB,0BAA0B,CAAC,GAAG,EAAE,cAAc,EAAE,EAAE,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,IAEvF,OAAO,YAAY,UAmB5B;AAiDD,wBAAsB,eAAe,CACnC,GAAG,EAAE,cAAc,EACnB,EAAE,EAAE,SAAS,EACb,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,MAAM,EACf,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,MAAM,GAAG,SAAS,EAC3B,QAAQ,EAAE,aAAa,EAAE,EACzB,cAAc,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,kBAAkB,GAClD,OAAO,CAAC,IAAI,CAAC,CAiGf"}