@vibedrift/cli 0.5.4 → 0.5.6

package/README.md

@@ -41,12 +41,15 @@ vibedrift . --deep       # uses one of your 3 monthly deep scans
 - **Dependency health** — phantom deps, missing deps, undocumented env vars
 - **Code quality** — cyclomatic complexity, dead code, TODO density, long functions, unclear naming
 
-### Layer 1.5 — Cross-File Drift Detection (5 detectors)
-- **Architectural consistency** — data access patterns, error handling, DI, config approaches
+### Layer 1.5 — Cross-File Drift Detection (8 detectors)
+- **Architectural consistency** — error handling, data access, DI, and config patterns across all source files
 - **Convention oscillation** — naming conventions, file naming across the whole project
 - **Security consistency** — auth middleware, input validation, rate limiting across routes
 - **Semantic duplication** — functions that do the same thing under different names in different files
-- **Phantom scaffolding** — CRUD endpoints the AI generated but never wired up
+- **Phantom scaffolding** — exported types and CRUD endpoints that are never used
+- **Import style** — relative paths vs path aliases, default vs named imports
+- **Export style** — default exports vs named-only exports per file
+- **Async patterns** — async/await vs .then() chains consistency
 
 ### Layer 1.7 — Code DNA Engine (5 modules, runs locally)
 - **Semantic fingerprinting** — normalizes function bodies and hashes them. Identical hashes = exact semantic duplicates at confidence 1.0

package/dist/index.js

@@ -5178,11 +5178,14 @@ init_esm_shims();
 // src/drift/types.ts
 init_esm_shims();
 var DRIFT_WEIGHTS = {
-  architectural_consistency: 25,
-  security_posture: 25,
-  semantic_duplication: 20,
-  naming_conventions: 15,
-  phantom_scaffolding: 15
+  architectural_consistency: 16,
+  security_posture: 14,
+  semantic_duplication: 14,
+  naming_conventions: 12,
+  phantom_scaffolding: 12,
+  import_style: 12,
+  export_style: 10,
+  async_patterns: 10
 };
 
 // src/drift/architectural-contradiction.ts
@@ -5200,9 +5203,6 @@ function extractEvidence(content, pattern, maxResults = 3) {
   }
   return evidence;
 }
-function isHandlerOrController(path2) {
-  return /(?:handler|controller|route|endpoint|view|api|resource|page|action)/i.test(path2) && !/(?:repository|repo|store|dal|model|query|migration|test|spec|mock)/i.test(path2);
-}
 function detectDataAccess(file) {
   const results = [];
   const c = file.content;
@@ -5220,7 +5220,7 @@ function detectDataAccess(file) {
     results.push({ pattern: "direct_db", evidence: dbEvidence });
   }
   const httpEvidence = extractEvidence(c, /\b(?:fetch|axios|http\.(?:Get|Post|Put)|requests\.(?:get|post))\s*\(/g);
-  if (httpEvidence.length > 0 && isHandlerOrController(file.path)) {
+  if (httpEvidence.length > 0) {
     results.push({ pattern: "http_client", evidence: httpEvidence });
   }
   return results;
@@ -5280,13 +5280,19 @@ function detectDIPattern(file) {
   }
   return results;
 }
+function isAnalyzableSource(path2) {
+  if (/(?:test|spec|mock|fixture|__test__|__mocks__|\.test\.|\.spec\.)/i.test(path2)) return false;
+  if (/(?:\.config\.|\.d\.ts$|node_modules|dist\/|build\/)/i.test(path2)) return false;
+  return true;
+}
 function buildProfile(file) {
-  if (!file.language || !isHandlerOrController(file.path)) return null;
+  if (!file.language) return null;
+  if (!isAnalyzableSource(file.path)) return null;
   const dataAccess = detectDataAccess(file);
   const errorHandling = detectErrorHandling(file);
   const config = detectConfigPattern(file);
   const di = detectDIPattern(file);
-  if (dataAccess.length === 0 && errorHandling.length === 0) return null;
+  if (dataAccess.length === 0 && errorHandling.length === 0 && config.length === 0) return null;
   return { file: file.path, language: file.language, dataAccess, errorHandling, config, di };
 }
 function detectFilePattern(patterns) {
@@ -6285,6 +6291,341 @@ var phantomScaffolding = {
   }
 };
 
+// src/drift/import-consistency.ts
+init_esm_shims();
+function isSourceFile(path2) {
+  if (/(?:test|spec|mock|fixture|__test__|__mocks__|\.test\.|\.spec\.)/i.test(path2)) return false;
+  if (/(?:\.config\.|\.d\.ts$|node_modules|dist\/|build\/)/i.test(path2)) return false;
+  return true;
+}
+function analyzeImports(file) {
+  if (!file.language || !["javascript", "typescript"].includes(file.language)) return null;
+  if (!isSourceFile(file.path)) return null;
+  const lines = file.content.split("\n");
+  let relativeCount = 0;
+  let aliasCount = 0;
+  let defaultCount = 0;
+  let namedCount = 0;
+  const evidence = [];
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i].trim();
+    const importMatch = line.match(/^import\s+(?!type\s)/);
+    if (!importMatch) continue;
+    const fromMatch = line.match(/from\s+["']([^"']+)["']/);
+    if (!fromMatch) continue;
+    const importPath = fromMatch[1];
+    if (!importPath.startsWith(".") && !importPath.startsWith("@/") && !importPath.startsWith("~/")) continue;
+    if (importPath.startsWith("./") || importPath.startsWith("../")) {
+      relativeCount++;
+    } else if (importPath.startsWith("@/") || importPath.startsWith("~/")) {
+      aliasCount++;
+    }
+    if (/^import\s+\{/.test(line)) {
+      namedCount++;
+    } else if (/^import\s+\w+\s+from/.test(line) || /^import\s+\w+,\s*\{/.test(line)) {
+      defaultCount++;
+    }
+    if (evidence.length < 3) {
+      evidence.push({ line: i + 1, code: line });
+    }
+  }
+  const totalLocalImports = relativeCount + aliasCount;
+  if (totalLocalImports < 3) return null;
+  const pathStyle = aliasCount === 0 ? "relative" : relativeCount === 0 ? "alias" : relativeCount >= aliasCount ? "relative" : "alias";
+  const totalBindings = defaultCount + namedCount;
+  let bindingStyle = null;
+  if (totalBindings >= 3) {
+    const defaultRatio = defaultCount / totalBindings;
+    bindingStyle = defaultRatio > 0.6 ? "default_heavy" : defaultRatio < 0.4 ? "named_heavy" : "mixed";
+  }
+  return {
+    file: file.path,
+    pathStyle,
+    bindingStyle,
+    relativeCount,
+    aliasCount,
+    defaultCount,
+    namedCount,
+    evidence
+  };
+}
+var PATH_STYLE_NAMES = {
+  relative: "relative paths (./)",
+  alias: "path aliases (@/)"
+};
+function buildFinding(profiles, getStyle, styleNames, subCategory, label) {
+  const withStyle = profiles.filter((p) => getStyle(p) !== null);
+  if (withStyle.length < 5) return null;
+  const counts = /* @__PURE__ */ new Map();
+  for (const p of withStyle) {
+    const style = getStyle(p);
+    if (!counts.has(style)) counts.set(style, { count: 0, files: [] });
+    const entry = counts.get(style);
+    entry.count++;
+    entry.files.push(p.file);
+  }
+  if (counts.size < 2) return null;
+  let dominant = null;
+  let dominantCount = 0;
+  for (const [style, data] of counts) {
+    if (data.count > dominantCount) {
+      dominantCount = data.count;
+      dominant = style;
+    }
+  }
+  if (!dominant) return null;
+  const totalFiles = withStyle.length;
+  const consistencyScore = Math.round(dominantCount / totalFiles * 100);
+  if (consistencyScore === 100) return null;
+  const deviating = [];
+  for (const [style, data] of counts) {
+    if (style === dominant) continue;
+    for (const filePath of data.files) {
+      const profile = withStyle.find((p) => p.file === filePath);
+      deviating.push({
+        path: filePath,
+        detectedPattern: styleNames[style] ?? style,
+        evidence: profile?.evidence.slice(0, 2) ?? []
+      });
+    }
+  }
+  return {
+    detector: "import_style",
+    subCategory,
+    driftCategory: "import_style",
+    severity: deviating.length >= 5 ? "warning" : "info",
+    confidence: 0.8,
+    finding: `${label}: ${dominantCount} files use ${styleNames[dominant] ?? dominant}, ${deviating.length} deviate`,
+    dominantPattern: styleNames[dominant] ?? dominant,
+    dominantCount,
+    totalRelevantFiles: totalFiles,
+    consistencyScore,
+    deviatingFiles: deviating,
+    recommendation: `Standardize on ${styleNames[dominant] ?? dominant} across all files for consistency.`
+  };
+}
+var importConsistency = {
+  id: "import-consistency",
+  name: "Import Style Consistency",
+  category: "import_style",
+  detect(ctx) {
+    const findings = [];
+    const profiles = [];
+    for (const file of ctx.files) {
+      const p = analyzeImports(file);
+      if (p) profiles.push(p);
+    }
+    if (profiles.length < 5) return findings;
+    const pathFinding = buildFinding(
+      profiles,
+      (p) => p.pathStyle,
+      PATH_STYLE_NAMES,
+      "path_style",
+      "Import path style"
+    );
+    if (pathFinding) findings.push(pathFinding);
+    return findings;
+  }
+};
+
+// src/drift/export-consistency.ts
+init_esm_shims();
+function isSourceFile2(path2) {
+  if (/(?:test|spec|mock|fixture|__test__|__mocks__|\.test\.|\.spec\.)/i.test(path2)) return false;
+  if (/(?:\.config\.|\.d\.ts$|node_modules|dist\/|build\/|index\.)/i.test(path2)) return false;
+  return true;
+}
+function analyzeExports(file) {
+  if (!file.language || !["javascript", "typescript"].includes(file.language)) return null;
+  if (!isSourceFile2(file.path)) return null;
+  const lines = file.content.split("\n");
+  let hasDefaultExport = false;
+  let hasNamedExport = false;
+  const evidence = [];
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i].trim();
+    if (/^export\s+default\b/.test(line) || /\bmodule\.exports\s*=/.test(line)) {
+      hasDefaultExport = true;
+      if (evidence.length < 2) evidence.push({ line: i + 1, code: line.slice(0, 100) });
+    } else if (/^export\s+(?:function|class|const|let|var|interface|type|enum|abstract)\b/.test(line)) {
+      hasNamedExport = true;
+      if (evidence.length < 2) evidence.push({ line: i + 1, code: line.slice(0, 100) });
+    } else if (/^export\s*\{/.test(line)) {
+      hasNamedExport = true;
+      if (evidence.length < 2) evidence.push({ line: i + 1, code: line.slice(0, 100) });
+    }
+  }
+  if (!hasDefaultExport && !hasNamedExport) return null;
+  const style = hasDefaultExport ? "default_export" : "named_only";
+  return { file: file.path, style, evidence };
+}
+var STYLE_NAMES = {
+  default_export: "default exports",
+  named_only: "named exports only"
+};
+var exportConsistency = {
+  id: "export-consistency",
+  name: "Export Style Consistency",
+  category: "export_style",
+  detect(ctx) {
+    const profiles = [];
+    for (const file of ctx.files) {
+      const p = analyzeExports(file);
+      if (p) profiles.push(p);
+    }
+    if (profiles.length < 5) return [];
+    const counts = /* @__PURE__ */ new Map();
+    for (const p of profiles) {
+      if (!counts.has(p.style)) counts.set(p.style, { count: 0, files: [] });
+      const entry = counts.get(p.style);
+      entry.count++;
+      entry.files.push(p.file);
+    }
+    if (counts.size < 2) return [];
+    let dominant = null;
+    let dominantCount = 0;
+    for (const [style, data] of counts) {
+      if (data.count > dominantCount) {
+        dominantCount = data.count;
+        dominant = style;
+      }
+    }
+    if (!dominant) return [];
+    const totalFiles = profiles.length;
+    const consistencyScore = Math.round(dominantCount / totalFiles * 100);
+    if (consistencyScore === 100) return [];
+    const deviating = [];
+    for (const [style, data] of counts) {
+      if (style === dominant) continue;
+      for (const filePath of data.files) {
+        const profile = profiles.find((p) => p.file === filePath);
+        deviating.push({
+          path: filePath,
+          detectedPattern: STYLE_NAMES[style],
+          evidence: profile?.evidence.slice(0, 2) ?? []
+        });
+      }
+    }
+    return [{
+      detector: "export_style",
+      subCategory: "export_style",
+      driftCategory: "export_style",
+      severity: deviating.length >= 5 ? "warning" : "info",
+      confidence: 0.75,
+      finding: `Export style: ${dominantCount} files use ${STYLE_NAMES[dominant]}, ${deviating.length} use ${STYLE_NAMES[dominant === "default_export" ? "named_only" : "default_export"]}`,
+      dominantPattern: STYLE_NAMES[dominant],
+      dominantCount,
+      totalRelevantFiles: totalFiles,
+      consistencyScore,
+      deviatingFiles: deviating,
+      recommendation: `Standardize on ${STYLE_NAMES[dominant]}. ${dominant === "named_only" ? "Named exports enable tree-shaking and explicit imports." : "Default exports simplify imports but disable tree-shaking."}`
+    }];
+  }
+};
+
+// src/drift/async-consistency.ts
+init_esm_shims();
+function isSourceFile3(path2) {
+  if (/(?:test|spec|mock|fixture|__test__|__mocks__|\.test\.|\.spec\.)/i.test(path2)) return false;
+  if (/(?:\.config\.|\.d\.ts$|node_modules|dist\/|build\/)/i.test(path2)) return false;
+  return true;
+}
+function analyzeAsync(file) {
+  if (!file.language || !["javascript", "typescript"].includes(file.language)) return null;
+  if (!isSourceFile3(file.path)) return null;
+  const lines = file.content.split("\n");
+  let awaitCount = 0;
+  let thenCount = 0;
+  const evidence = [];
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i];
+    if (/\bawait\s+/.test(line) && !line.trim().startsWith("//") && !line.trim().startsWith("*")) {
+      awaitCount++;
+      if (evidence.length < 3) {
+        evidence.push({ line: i + 1, code: line.trim().slice(0, 100) });
+      }
+    }
+    if (/\.\s*then\s*\(/.test(line) && !line.trim().startsWith("//") && !line.trim().startsWith("*") && !/type\s|interface\s/.test(line)) {
+      thenCount++;
+      if (evidence.length < 3) {
+        evidence.push({ line: i + 1, code: line.trim().slice(0, 100) });
+      }
+    }
+  }
+  const total = awaitCount + thenCount;
+  if (total < 2) return null;
+  let style;
+  const awaitRatio = awaitCount / total;
+  if (awaitRatio > 0.7) style = "async_await";
+  else if (awaitRatio < 0.3) style = "then_chains";
+  else style = "mixed";
+  return { file: file.path, style, awaitCount, thenCount, evidence };
+}
+var STYLE_NAMES2 = {
+  async_await: "async/await",
+  then_chains: ".then() chains",
+  mixed: "mixed async patterns"
+};
+var asyncConsistency = {
+  id: "async-consistency",
+  name: "Async Pattern Consistency",
+  category: "async_patterns",
+  detect(ctx) {
+    const profiles = [];
+    for (const file of ctx.files) {
+      const p = analyzeAsync(file);
+      if (p) profiles.push(p);
+    }
+    if (profiles.length < 5) return [];
+    const counts = /* @__PURE__ */ new Map();
+    for (const p of profiles) {
+      if (!counts.has(p.style)) counts.set(p.style, { count: 0, files: [] });
+      const entry = counts.get(p.style);
+      entry.count++;
+      entry.files.push(p.file);
+    }
+    if (counts.size < 2) return [];
+    let dominant = null;
+    let dominantCount = 0;
+    for (const [style, data] of counts) {
+      if (data.count > dominantCount) {
+        dominantCount = data.count;
+        dominant = style;
+      }
+    }
+    if (!dominant) return [];
+    const totalFiles = profiles.length;
+    const consistencyScore = Math.round(dominantCount / totalFiles * 100);
+    if (consistencyScore === 100) return [];
+    const deviating = [];
+    for (const [style, data] of counts) {
+      if (style === dominant) continue;
+      for (const filePath of data.files) {
+        const profile = profiles.find((p) => p.file === filePath);
+        deviating.push({
+          path: filePath,
+          detectedPattern: STYLE_NAMES2[style],
+          evidence: profile?.evidence.slice(0, 2) ?? []
+        });
+      }
+    }
+    return [{
+      detector: "async_patterns",
+      subCategory: "async_style",
+      driftCategory: "async_patterns",
+      severity: deviating.length >= 5 ? "warning" : "info",
+      confidence: 0.8,
+      finding: `Async style: ${dominantCount} files use ${STYLE_NAMES2[dominant]}, ${deviating.length} use different patterns`,
+      dominantPattern: STYLE_NAMES2[dominant],
+      dominantCount,
+      totalRelevantFiles: totalFiles,
+      consistencyScore,
+      deviatingFiles: deviating,
+      recommendation: `Standardize on ${STYLE_NAMES2[dominant]}. ${dominant === "async_await" ? "async/await is more readable and has clearer error handling with try/catch." : "Consider migrating to async/await for consistency and readability."}`
+    }];
+  }
+};
+
 // src/drift/index.ts
 function createDriftDetectors() {
   return [
@@ -6292,7 +6633,10 @@ function createDriftDetectors() {
     conventionOscillation,
     securityConsistency,
     semanticDuplication,
-    phantomScaffolding
+    phantomScaffolding,
+    importConsistency,
+    exportConsistency,
+    asyncConsistency
   ];
 }
 function buildDriftContext(ctx) {
@@ -6325,7 +6669,10 @@ function computeDriftScores(findings) {
     "security_posture",
     "semantic_duplication",
     "naming_conventions",
-    "phantom_scaffolding"
+    "phantom_scaffolding",
+    "import_style",
+    "export_style",
+    "async_patterns"
   ];
   const scores = {};
   for (const cat of categories) {
@@ -7067,8 +7414,12 @@ function buildCoherenceMatrix(files) {
   }
   const colCount = categories.length + 3;
   const alignedRows = aligned.slice(0, 3).map((f) => fileRow(f)).join("");
-  const collapsedAligned = aligned.length > 3 ? `<tr><td colspan="${colCount}" style="padding:6px 12px;font-size:12px;color:var(--text-tertiary);cursor:pointer" data-collapse="aligned-rest">&middot;&middot;&middot; ${aligned.length - 3} more at 100% &middot;&middot;&middot;</td></tr>
-       <tbody id="aligned-rest" style="display:none">${aligned.slice(3).map((f) => fileRow(f)).join("")}</tbody>` : "";
+  const collapsedAligned = aligned.length > 3 ? `<tr><td colspan="${colCount}" style="padding:0"><details style="margin:0">
+         <summary style="cursor:pointer;padding:8px 16px;font-size:12px;color:var(--text-tertiary);list-style:none;display:flex;align-items:center;gap:6px;background:rgba(255,255,255,0.02)">
+           <span class="chevron">&#9654;</span> ${aligned.length - 3} more files at 100% alignment
+         </summary>
+         <table style="width:100%;border-collapse:collapse">${aligned.slice(3).map((f) => fileRow(f)).join("")}</table>
+       </details></td></tr>` : "";
   const singleIssueDrifting = /* @__PURE__ */ new Map();
   const multiIssueDrifting = [];
   for (const f of drifting) {
@@ -7085,9 +7436,12 @@ function buildCoherenceMatrix(files) {
   for (const [devKey, files2] of singleIssueDrifting) {
     if (files2.length >= 4) {
       driftingRows += fileRow(files2[0]);
-      const collapseId = `single-dev-${devKey.replace(/[^a-zA-Z0-9]/g, "-")}`;
-      driftingRows += `<tr><td colspan="${colCount}" style="padding:4px 12px;font-size:12px;color:var(--text-tertiary);cursor:pointer" data-collapse="${collapseId}">&middot;&middot;&middot; ${files2.length - 1} more files with same issue (${esc(devKey)}) &middot;&middot;&middot;</td></tr>`;
-      driftingRows += `<tbody id="${collapseId}" style="display:none">${files2.slice(1).map((f) => fileRow(f)).join("")}</tbody>`;
+      driftingRows += `<tr><td colspan="${colCount}" style="padding:0"><details style="margin:0">
+        <summary style="cursor:pointer;padding:8px 16px;font-size:12px;color:var(--text-tertiary);list-style:none;display:flex;align-items:center;gap:6px;background:rgba(255,255,255,0.02)">
+          <span class="chevron">&#9654;</span> ${files2.length - 1} more files with same issue <span style="color:var(--drift-orange)">(${esc(devKey)})</span>
+        </summary>
+        <table style="width:100%;border-collapse:collapse">${files2.slice(1).map((f) => fileRow(f)).join("")}</table>
+      </details></td></tr>`;
     } else {
       driftingRows += files2.map((f) => fileRow(f)).join("");
     }
@@ -7246,7 +7600,7 @@ function buildRadarSection(result) {
   return `<section class="section">
   <div class="label">DRIFT FINGERPRINT</div>
   <p style="font-size:13px;color:var(--text-secondary);margin-bottom:20px;line-height:1.6">
-    This radar shows how well your codebase maintains consistency across 5 drift categories.
+    This radar shows how well your codebase maintains consistency across 8 drift categories.
     The outer edge represents a perfect score in each category. Points closer to the center indicate more drift.
     A balanced shape means consistent quality; an uneven shape reveals where drift concentrates.
   </p>
@@ -7330,23 +7684,32 @@ function buildDeviatingBlocks(d) {
   }
   if (isConvention && d.deviatingFiles.length > 4) {
     return [...devByPattern.entries()].map(([pattern, files]) => {
-      const fileList = files.map((df) => esc(df.path)).join("</div><div style='padding:1px 0'>");
-      const collapseId = `conv-${pattern.replace(/[^a-zA-Z0-9]/g, "-")}-${Math.random().toString(36).slice(2, 6)}`;
-      return `<div style="background:var(--tint-orange);border-left:3px solid var(--drift-orange);border-radius:0;padding:12px 16px;margin:8px 0">
-        <div class="label" style="color:var(--drift-orange);margin-bottom:4px">DRIFT &mdash; ${esc(pattern)} &mdash; ${files.length} files</div>
-        <div style="cursor:pointer;font-size:12px;color:var(--text-secondary);margin-top:6px" data-collapse="${collapseId}">&#9654; Show ${files.length} files</div>
-        <div id="${collapseId}" style="display:none;padding:6px 0" class="mono" style="font-size:12px;color:var(--text-secondary)"><div style="padding:1px 0">${fileList}</div></div>
+      const fileList = files.map((df) => `<div style="padding:2px 0;font-size:12px;color:var(--text-secondary)" class="mono">${esc(df.path)}</div>`).join("");
+      return `<div style="background:var(--tint-orange);border-left:3px solid var(--drift-orange);border-radius:0;margin:8px 0">
+        <details>
+          <summary style="cursor:pointer;padding:12px 16px;list-style:none;display:flex;align-items:center;gap:6px">
+            <span class="chevron">&#9654;</span>
+            <span class="label" style="color:var(--drift-orange);margin:0">DRIFT &mdash; ${esc(pattern)}</span>
+            <span style="font-size:12px;color:var(--text-tertiary);margin-left:auto">${files.length} files</span>
+          </summary>
+          <div style="padding:4px 16px 12px">${fileList}</div>
+        </details>
       </div>`;
     }).join("");
   }
-  return d.deviatingFiles.slice(0, 4).map((df) => {
+  return d.deviatingFiles.slice(0, 6).map((df) => {
     const evidence = df.evidence.slice(0, 3).map(
       (e) => `<div style="background:var(--bg-code);padding:6px 12px;border-radius:0;margin:4px 0;overflow-x:auto" class="mono"><span style="color:var(--text-tertiary);margin-right:12px;user-select:none">${e.line}</span>${esc(e.code.slice(0, 120))}</div>`
     ).join("");
-    return `<div style="background:var(--tint-orange);border-left:3px solid var(--drift-orange);border-radius:0;padding:12px 16px;margin:8px 0">
-      <div class="label" style="color:var(--drift-orange);margin-bottom:4px">DRIFT &mdash; ${esc(df.detectedPattern)}</div>
-      <div class="mono" style="font-size:12px;color:var(--text-secondary);margin-bottom:6px">${esc(df.path)}</div>
-      ${evidence}
+    return `<div style="background:var(--tint-orange);border-left:3px solid var(--drift-orange);border-radius:0;margin:8px 0">
+      <details>
+        <summary style="cursor:pointer;padding:10px 16px;list-style:none;display:flex;align-items:center;gap:6px">
+          <span class="chevron">&#9654;</span>
+          <span class="label" style="color:var(--drift-orange);margin:0">DRIFT &mdash; ${esc(df.detectedPattern)}</span>
+          <span class="mono" style="font-size:12px;color:var(--text-secondary);margin-left:8px">${esc(df.path)}</span>
+        </summary>
+        <div style="padding:4px 16px 12px">${evidence}</div>
+      </details>
     </div>`;
   }).join("");
 }
@@ -7381,13 +7744,16 @@ function buildDistributionBar(d) {
   </div>`;
 }
 function buildDriftFindings(result) {
-  const driftCats = ["architectural_consistency", "security_posture", "semantic_duplication", "naming_conventions", "phantom_scaffolding"];
+  const driftCats = ["architectural_consistency", "security_posture", "semantic_duplication", "naming_conventions", "phantom_scaffolding", "import_style", "export_style", "async_patterns"];
   const catLabels = {
     architectural_consistency: "Architectural contradictions",
     security_posture: "Security posture gaps",
     semantic_duplication: "Semantic duplication",
     naming_conventions: "Convention drift",
-    phantom_scaffolding: "Phantom scaffolding"
+    phantom_scaffolding: "Phantom scaffolding",
+    import_style: "Import style drift",
+    export_style: "Export style drift",
+    async_patterns: "Async pattern drift"
   };
   const groups = driftCats.map((cat) => ({
     cat,
@@ -7414,18 +7780,21 @@ function buildDriftFindings(result) {
       const rec = recText ? `<div style="margin-top:12px;padding:10px 16px;background:var(--tint-cyan);border-left:3px solid var(--border);border-radius:0;font-size:14px;line-height:1.6;color:var(--text-primary)">
         <span style="color:var(--text-secondary);font-weight:700;margin-right:4px">&rarr;</span> ${esc(recText)}
       </div>` : "";
-      return `<div style="background:var(--bg-surface);border-radius:0;padding:20px 24px;margin-bottom:10px;border:1px solid var(--border)">
-        <div style="display:flex;align-items:center;gap:8px;margin-bottom:10px;flex-wrap:wrap">
+      return `<details style="background:var(--bg-surface);border-radius:0;margin-bottom:10px;border:1px solid var(--border)">
+        <summary style="cursor:pointer;padding:16px 20px;display:flex;align-items:center;gap:8px;flex-wrap:wrap;list-style:none">
+          <span class="chevron">&#9654;</span>
           <span class="sev-badge" style="background:${sevColor(d.severity)}">${sevLabel(d.severity)}</span>
-          <span style="font-size:15px;font-weight:500;color:var(--text-primary);flex:1">${esc(d.finding)}</span>
+          <span style="font-size:14px;font-weight:500;color:var(--text-primary);flex:1">${esc(d.finding)}</span>
           <span class="mono" style="font-size:12px;color:var(--text-tertiary)">${d.dominantCount}/${d.totalRelevantFiles}</span>
+        </summary>
+        <div style="padding:4px 20px 20px">
+          ${domBlock}
+          ${devBlocks}
+          ${distBar}
+          ${rec}
+          ${closeSplitQualifier}
         </div>
-        ${domBlock}
-        ${devBlocks}
-        ${distBar}
-        ${rec}
-        ${closeSplitQualifier}
-      </div>`;
+      </details>`;
     }).join("");
     return `<details ${gi === 0 ? "open" : ""} style="margin-bottom:8px">
       <summary style="cursor:pointer;padding:12px 18px;background:var(--bg-surface);border-radius:0;display:flex;align-items:center;gap:10px;font-size:14px;font-weight:600;color:var(--text-primary);list-style:none;border:1px solid var(--border)">