@mrclrchtr/supi-code-intelligence 3.2.0 → 4.0.0

package/src/analysis/brief/summarize.ts

@@ -1,208 +0,0 @@
-/**
- * Recursive directory summary — source file discovery, import/export counting,
- * and named-export extraction.
- *
- * Extracted from brief-focused.ts to keep the focus-generation modules
- * cohesive and avoid a single 800+ line file.
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-
-// ── Types ─────────────────────────────────────────────────────────────
-
-export interface RecursiveDirectorySummary {
-  directFiles: string[];
-  allFiles: string[];
-  totalSourceFiles: number;
-  subdirs: Array<{ name: string; fileCount: number }>;
-  publicSurfaces: string[];
-  importCount: number;
-  exportCount: number;
-}
-
-// ── Recognized source extensions ──────────────────────────────────────
-
-export const SOURCE_EXTENSIONS = new Set([
-  // JS/TS
-  ".ts",
-  ".tsx",
-  ".js",
-  ".jsx",
-  ".mts",
-  ".cts",
-  ".mjs",
-  ".cjs",
-  // Python
-  ".py",
-  ".pyi",
-  // Rust
-  ".rs",
-  // Go
-  ".go",
-  ".mod",
-  // C/C++
-  ".c",
-  ".h",
-  ".cpp",
-  ".hpp",
-  ".cc",
-  ".cxx",
-  ".hxx",
-  ".c++",
-  ".h++",
-  // Java / Kotlin
-  ".java",
-  ".kt",
-  ".kts",
-  // Ruby
-  ".rb",
-  // Shell
-  ".sh",
-  ".bash",
-  ".zsh",
-  // Web
-  ".html",
-  ".htm",
-  ".xhtml",
-  ".css",
-  ".scss",
-  ".less",
-  // Data / Config
-  ".json",
-  ".yaml",
-  ".yml",
-  ".toml",
-  ".md",
-  // Other languages
-  ".php",
-  ".swift",
-  ".cs",
-  ".r",
-  ".sql",
-]);
-
-// ── Public API ────────────────────────────────────────────────────────
-
-/** List source files (by extension) in a flat directory. */
-export function listSourceFiles(dir: string): string[] {
-  const files: string[] = [];
-  try {
-    const entries = fs.readdirSync(dir, { withFileTypes: true });
-    for (const entry of entries) {
-      if (!entry.isFile()) continue;
-      if (entry.name.startsWith(".")) continue;
-      if (SOURCE_EXTENSIONS.has(path.extname(entry.name))) {
-        files.push(entry.name);
-      }
-    }
-  } catch {
-    // Directory not readable
-  }
-  return files.sort((a, b) => a.localeCompare(b));
-}
-
-/** Recursively summarize a directory tree: source files, imports, exports, subdirectories. */
-export function summarizeDirectoryRecursively(dir: string): RecursiveDirectorySummary {
-  const directFiles = listSourceFiles(dir);
-  const summary: RecursiveDirectorySummary = {
-    directFiles,
-    allFiles: directFiles.map((file) => path.join(dir, file)),
-    totalSourceFiles: directFiles.length,
-    subdirs: [],
-    publicSurfaces: [],
-    importCount: 0,
-    exportCount: 0,
-  };
-
-  accumulateJsTsSignals(summary, dir, directFiles, "");
-
-  try {
-    const entries = fs.readdirSync(dir, { withFileTypes: true });
-    for (const entry of entries) {
-      if (!entry.isDirectory() || entry.name.startsWith(".")) continue;
-      const childDir = path.join(dir, entry.name);
-      const childSummary = summarizeDirectoryRecursively(childDir);
-      if (childSummary.totalSourceFiles === 0) continue;
-      summary.totalSourceFiles += childSummary.totalSourceFiles;
-      summary.allFiles.push(...childSummary.allFiles);
-      summary.subdirs.push({ name: entry.name, fileCount: childSummary.totalSourceFiles });
-      summary.publicSurfaces.push(
-        ...childSummary.publicSurfaces.map((surface) => `${entry.name}/${surface}`),
-      );
-      summary.importCount += childSummary.importCount;
-      summary.exportCount += childSummary.exportCount;
-    }
-  } catch {
-    // Directory not readable
-  }
-
-  summary.subdirs.sort((a, b) => a.name.localeCompare(b.name));
-  return summary;
-}
-
-// ── Internal helpers ──────────────────────────────────────────────────
-
-function accumulateJsTsSignals(
-  summary: RecursiveDirectorySummary,
-  dir: string,
-  files: string[],
-  prefix: string,
-): void {
-  for (const file of files) {
-    if (!isJsTsFile(file)) continue;
-    try {
-      const relFile = prefix ? `${prefix}/${file}` : file;
-      const content = fs.readFileSync(path.join(dir, file), "utf-8");
-      summary.importCount += countMatches(content, /^import\s/gm);
-      const exports = extractNamedExports(content);
-      summary.exportCount += exports.length;
-      for (const exportedName of exports) {
-        summary.publicSurfaces.push(`\`${exportedName}\` — \`${relFile}\``);
-      }
-    } catch {
-      // File not readable
-    }
-  }
-}
-
-function isJsTsFile(file: string): boolean {
-  return [".ts", ".tsx", ".js", ".jsx", ".mts", ".cts", ".mjs", ".cjs"].includes(
-    path.extname(file),
-  );
-}
-
-/**
- * Lightweight exported-name extraction for recursive directory briefs.
- *
- * Best-effort only: handles exported declarations and simple named export lists,
- * but intentionally does not try to cover default exports, export-all forms,
- * re-exported defaults, or every multiline formatting variant.
- */
-function extractNamedExports(content: string): string[] {
-  const names = new Set<string>();
-  const declarationRegex =
-    /export\s+(?:async\s+)?(?:function|class|interface|type|const|let|var|enum)\s+([A-Za-z_$][\w$]*)/g;
-  const namedExportRegex = /export\s*\{\s*([^}]+)\s*\}/g;
-
-  for (const match of content.matchAll(declarationRegex)) {
-    if (match[1]) names.add(match[1]);
-  }
-
-  for (const match of content.matchAll(namedExportRegex)) {
-    const parts = match[1]?.split(",") ?? [];
-    for (const part of parts) {
-      const candidate = part
-        .trim()
-        .split(/\s+as\s+/i)[0]
-        ?.trim();
-      if (candidate) names.add(candidate);
-    }
-  }
-
-  return [...names].sort((a, b) => a.localeCompare(b));
-}
-
-function countMatches(content: string, regex: RegExp): number {
-  return [...content.matchAll(regex)].length;
-}

package/src/analysis/brief/types.ts

@@ -1,27 +0,0 @@
-// Brief use-case types — moved from shared/context/types.ts (Phase 2.5)
-
-import type { SessionLspServiceState } from "@mrclrchtr/supi-lsp/api";
-import type { BriefDetails } from "../../types/details.ts";
-import type { ArchitectureModel } from "../architecture/model.ts";
-import type { CodeProvider } from "../provider.ts";
-
-export type BriefInput =
-  | { kind: "project"; maxResults?: number }
-  | { kind: "path"; path: string; maxResults?: number }
-  | { kind: "file"; file: string; maxResults?: number }
-  | { kind: "symbol"; symbol: string; path?: string; maxResults?: number };
-
-export interface BriefDeps {
-  model: ArchitectureModel | null;
-  provider: CodeProvider | null;
-  cwd: string;
-  /** Show git context in orientation output. Defaults to true. */
-  showGitContext?: boolean;
-  /** LSP service state for diagnostic access. */
-  lspService: SessionLspServiceState;
-}
-
-export interface BriefUseCaseResult {
-  content: string;
-  details: BriefDetails;
-}

package/src/analysis/brief/use-case.ts

@@ -1,264 +0,0 @@
-import * as path from "node:path";
-import { renderSymbolBrief } from "../../ui/markdown/brief.ts";
-import { gatherTreeSitterContext } from "../../ui/markdown/gather.ts";
-import type { ArchitectureModel } from "../architecture/model.ts";
-import { findModuleForPath } from "../architecture/model.ts";
-import { normalizePath } from "../search/ripgrep.ts";
-import { resolveSymbolTarget } from "../target/symbol.ts";
-import type { TargetOutcome } from "../target/types.ts";
-import { generateFocusedBrief, generateProjectBrief } from "./public-brief.ts";
-import type { BriefDeps, BriefInput, BriefUseCaseResult } from "./types.ts";
-
-// ── Public entrypoint ─────────────────────────────────────────────────
-
-export async function executeBrief(
-  input: BriefInput,
-  deps: BriefDeps,
-): Promise<BriefUseCaseResult> {
-  switch (input.kind) {
-    case "project":
-      if (deps.model) {
-        return executeProjectBrief(deps.model, deps.showGitContext);
-      }
-      return noModelResult();
-    case "path":
-      return executePathBrief(input.path, deps, input);
-    case "file":
-      return executeFileBrief(input.file, deps, input);
-    case "symbol":
-      return executeSymbolBrief(input.symbol, input.path, deps);
-  }
-}
-
-// ── Project brief ─────────────────────────────────────────────────────
-
-function executeProjectBrief(
-  model: ArchitectureModel,
-  showGitContext?: boolean,
-): BriefUseCaseResult {
-  if (!model) {
-    return noModelResult();
-  }
-  const result = generateProjectBrief(model, { showGitContext });
-  return { content: result.content, details: result.details };
-}
-
-// ── Path / file briefs ────────────────────────────────────────────────
-
-async function executePathBrief(
-  requestPath: string,
-  deps: BriefDeps,
-  input: BriefInput,
-): Promise<BriefUseCaseResult> {
-  if (!deps.model) {
-    return noModelResult();
-  }
-  const resolved = normalizePath(requestPath, deps.cwd);
-  const maxResults =
-    "maxResults" in input ? (input as { maxResults?: number }).maxResults : undefined;
-  const result = await generateFocusedBrief(deps.model, resolved, {
-    provider: deps.provider,
-    cwd: deps.cwd,
-    maxResults,
-    showGitContext: deps.showGitContext ?? true,
-    lspService: deps.lspService,
-  });
-  return { content: result.content, details: result.details };
-}
-
-async function executeFileBrief(
-  file: string,
-  deps: BriefDeps,
-  input: BriefInput,
-): Promise<BriefUseCaseResult> {
-  if (!deps.model) {
-    return noModelResult();
-  }
-  const resolved = normalizePath(file, deps.cwd);
-  const maxResults =
-    "maxResults" in input ? (input as { maxResults?: number }).maxResults : undefined;
-  const result = await generateFocusedBrief(deps.model, resolved, {
-    provider: deps.provider,
-    cwd: deps.cwd,
-    maxResults,
-    showGitContext: deps.showGitContext ?? true,
-    lspService: deps.lspService,
-  });
-  return { content: result.content, details: result.details };
-}
-
-// ── Symbol brief ──────────────────────────────────────────────────────
-
-async function executeSymbolBrief(
-  symbol: string,
-  scopePath: string | undefined,
-  deps: BriefDeps,
-): Promise<BriefUseCaseResult> {
-  if (!deps.model || deps.model.modules.length === 0) {
-    return {
-      content: `**Error:** Symbol brief requires a project model, which is not available.`,
-      details: {
-        confidence: "unavailable",
-        focusTarget: symbol,
-        startHere: [],
-        publicSurfaces: [],
-        dependencySummary: null,
-        omittedCount: 0,
-        nextQueries: ["Add a package.json or workspace manifest to enable architecture analysis"],
-      },
-    };
-  }
-
-  const provider = deps.provider;
-  if (!provider) {
-    return {
-      content:
-        "**Error:** Symbol discovery requires an active code provider. Use `code_orientation` with `focus` for file orientation, or use `code_inspect` with `file` + coordinates for point facts.",
-      details: {
-        confidence: "unavailable",
-        focusTarget: symbol,
-        startHere: [],
-        publicSurfaces: [],
-        dependencySummary: null,
-        omittedCount: 0,
-        nextQueries: [
-          "Use `code_orientation` with `focus` for file orientation, or `code_inspect` with file + coordinates for point facts",
-        ],
-      },
-    };
-  }
-  const resolved = await resolveSymbolTarget(symbol, deps.cwd, provider, {
-    path: scopePath,
-  });
-
-  if (resolved.kind === "error") {
-    return {
-      content: resolved.message,
-      details: {
-        confidence: "unavailable",
-        focusTarget: symbol,
-        startHere: [],
-        publicSurfaces: [],
-        dependencySummary: null,
-        omittedCount: 0,
-        nextQueries: [
-          "Use `code_orientation` with `focus` for file orientation, or `code_inspect` with file + coordinates for point facts",
-        ],
-      },
-    };
-  }
-
-  if (resolved.kind === "disambiguation") {
-    const disambiguationContent = formatDisambiguation(symbol, resolved);
-    return {
-      content: disambiguationContent,
-      details: {
-        confidence: "unavailable",
-        focusTarget: symbol,
-        startHere: [],
-        publicSurfaces: [],
-        dependencySummary: null,
-        omittedCount: resolved.omittedCount,
-        nextQueries: [
-          "Use `code_orientation` with `focus` for file orientation, or `code_inspect` with file + coordinates for point facts",
-        ],
-      },
-    };
-  }
-
-  if (resolved.kind === "group") {
-    return {
-      content: `**Error:** Unexpected group result from symbol resolution for \`${symbol}\`. Use \`code_orientation\` with a file focus.`,
-      details: {
-        confidence: "unavailable",
-        focusTarget: symbol,
-        startHere: [],
-        publicSurfaces: [],
-        dependencySummary: null,
-        omittedCount: 0,
-        nextQueries: ["Use `code_orientation` with `focus` for file or directory orientation"],
-      },
-    };
-  }
-
-  const target = resolved.target;
-  const relPath = path.relative(deps.cwd, target.file);
-
-  const mod = findModuleForPath(deps.model, target.file);
-  const context = await gatherTreeSitterContext(
-    deps.provider,
-    relPath,
-    target.displayLine,
-    target.displayCharacter,
-  );
-
-  const details = {
-    confidence: target.confidence,
-    focusTarget: `${target.name ?? relPath}:${target.displayLine}:${target.displayCharacter}`,
-    startHere: [] as Array<{ target: string; reason: string }>,
-    publicSurfaces: [] as string[],
-    dependencySummary: mod ? { moduleCount: 1, edgeCount: mod.internalDeps.length } : null,
-    omittedCount: 0,
-    nextQueries: [
-      `\`code_inspect\` with \`file: "${relPath}"\`, \`line: ${target.displayLine}\`, and \`character: ${target.displayCharacter}\` for point facts`,
-      `\`code_graph\`, \`file: "${relPath}"\`, \`line: ${target.displayLine}\`, and \`character: ${target.displayCharacter}\` for reference sites`,
-      `\`code_impact\` with \`file: "${relPath}"\`, \`line: ${target.displayLine}\`, and \`character: ${target.displayCharacter}\` for impact analysis`,
-    ],
-  };
-
-  const rendered = renderSymbolBrief({
-    relPath,
-    symbolName: target.name ?? "",
-    targetLine: target.displayLine,
-    targetCharacter: target.displayCharacter,
-    targetKind: target.kind,
-    context,
-    model: deps.model ?? null,
-    details,
-    cwd: deps.cwd,
-  });
-
-  return rendered;
-}
-
-// ── Helpers ───────────────────────────────────────────────────────────
-
-function noModelResult(): BriefUseCaseResult {
-  return {
-    content:
-      "No project structure detected. This directory has no recognizable project metadata or source files.",
-    details: {
-      confidence: "unavailable",
-      focusTarget: null,
-      startHere: [],
-      publicSurfaces: [],
-      dependencySummary: null,
-      omittedCount: 0,
-      nextQueries: ["Add a package.json or pnpm-workspace.yaml to enable architecture analysis"],
-    },
-  };
-}
-
-function formatDisambiguation(
-  symbol: string,
-  result: Extract<TargetOutcome, { kind: "disambiguation" }>,
-): string {
-  const lines: string[] = [];
-  lines.push(`# Disambiguation needed for \`${symbol}\``);
-  lines.push("");
-  const omitNote = result.omittedCount > 0 ? ` (+${result.omittedCount} more)` : "";
-  lines.push(
-    `Found ${result.candidates.length} candidates${omitNote}. Rerun with anchored coordinates:`,
-  );
-  lines.push("");
-
-  for (const candidate of result.candidates) {
-    const kind = candidate.kind ? ` (${candidate.kind})` : "";
-    const container = candidate.container ? ` in ${candidate.container}` : "";
-    lines.push(
-      `${candidate.rank}. **${candidate.name}**${kind}${container} — \`${candidate.file}\`:${candidate.line}:${candidate.character}`,
-    );
-  }
-
-  return lines.join("\n");
-}