@herbertgao/pi-extensions 2026.9.0 → 2026.9.2

package/node_modules/pi-lens/dist/mcp/analyze-cli.js

@@ -48,7 +48,9 @@ async function readStdin() {
 function formatReport(result, cwd) {
     const rel = path.relative(cwd, result.filePath) || result.filePath;
     const lines = [
-        `🔎 pi-lens: ${rel} — ${result.counts.blockers} blocking, ${result.counts.warnings} warning(s)`,
+        // #2420: advisories (hint/info-tier style opinions) reported under their
+        // own label — no longer folded into the warning count on this surface.
+        `🔎 pi-lens: ${rel} — ${result.counts.blockers} blocking, ${result.counts.warnings} warning(s), ${result.counts.advisories} advisory(ies)`,
     ];
     for (const d of result.diagnostics.slice(0, 30)) {
         const marker = d.semantic === "blocking" ? "🔴" : "⚠";

package/node_modules/pi-lens/dist/mcp/build-staleness.js

@@ -31,7 +31,7 @@
  * per-call I/O" shape) and reuses the last verdict in between.
  */
 import * as fs from "node:fs";
-export const realStat = (targetPath) => {
+const realStat = (targetPath) => {
     try {
         return fs.statSync(targetPath);
     }

package/node_modules/pi-lens/dist/mcp/server.js

@@ -25,7 +25,7 @@ import { fileURLToPath } from "node:url";
 import { AstGrepClient } from "../clients/ast-grep-client.js";
 import { CacheManager } from "../clients/cache-manager.js";
 import { getDegradationSummary, renderDegradationLines, } from "../clients/degradation-ledger.js";
-import { acknowledgeTurnEnd, analyzeFile, analyzeFileFresh, canRebuildPiLens, createMcpHost, createWarmIpcLineReader, createWarmIpcRequestQueue, diagnosticStats, ensureLspConfig, generatedSkipNotice, ipcPathForCwd, lspStatus, moduleReport, projectReport, projectScan, readEnclosing, readSymbol, readTurnEndStatus, recentLatency, renderCompactModuleReport, renderCompactProjectReport, renderLspBrokenStatusLines, resolveRebuildScript, resourceFootprint, runRebuild, runSessionStart, runTurnEnd, runTurnEndForIpc, scanTruncationNotice, summarizeScan, symbolSearch, treeSitterRuntimeStatus, WARM_TURN_END_SCHEMA_VERSION, } from "../clients/lens-engine.js";
+import { acknowledgeTurnEnd, analyzeFile, analyzeFileFresh, canRebuildPiLens, createMcpHost, createWarmIpcLineReader, createWarmIpcRequestQueue, diagnosticStats, effectiveConfig, ensureLspConfig, generatedSkipNotice, ipcPathForCwd, isEffectiveFileViewError, lspStatus, moduleReport, projectReport, projectScan, readEnclosing, readSymbol, readTurnEndStatus, recentLatency, renderCompactModuleReport, renderCompactProjectReport, renderLspBrokenStatusLines, resolveRebuildScript, resourceFootprint, runRebuild, runSessionStart, runTurnEnd, runTurnEndForIpc, scanTruncationNotice, summarizeScan, symbolSearch, treeSitterRuntimeStatus, WARM_TURN_END_SCHEMA_VERSION, } from "../clients/lens-engine.js";
 import { createAstGrepReplaceTool } from "../tools/ast-grep-replace.js";
 import { createAstGrepSearchTool } from "../tools/ast-grep-search.js";
 import { createLensDiagnosticsTool } from "../tools/lens-diagnostics.js";
@@ -443,7 +443,7 @@ function schemaWithCwd(parameters) {
     return {
         type: "object",
         properties: {
-            ...(p.properties ?? {}),
+            ...p.properties,
             cwd: {
                 type: "string",
                 description: "Project root (defaults to the server workspace).",
@@ -748,11 +748,39 @@ const ALL_TOOLS = [
     {
         name: "pilens_health",
         description: "pi-lens runtime health for THIS server: alive LSP servers, last dispatch " +
-            "summary, session diagnostic counts, and the total CPU/RAM footprint " +
-            "attributable to pi-lens across every process it owns (host + LSP " +
-            "children) machine-wide, from the shared instance registry.",
+            "summary, session diagnostic counts, how many resolved config leaves each " +
+            "source tier decided, and the total CPU/RAM footprint attributable to " +
+            "pi-lens across every process it owns (host + LSP children) machine-wide, " +
+            "from the shared instance registry.",
         inputSchema: { type: "object", properties: {} },
     },
+    {
+        name: "pilens_effective_config",
+        description: "The resolved pi-lens configuration with the provenance of every " +
+            "decision: which file and source tier each setting came from, the trust " +
+            "decision that applied, and which config files contributed. Pass `file` " +
+            "to also get, for that path, its canonical language, EVERY LSP server " +
+            "with the reason it was selected or denied — including which tier's " +
+            "config denied it, which a nearer file cannot lift — and the lint/format " +
+            "runners that would dispatch. This is the answer to 'why is X running / " +
+            "why is X not running' without reading logs. Redacted by construction: " +
+            "it reports sources, never values — no environment values, no command " +
+            "arguments beyond the binary itself, and config paths are home-relative.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                cwd: { type: "string" },
+                file: {
+                    type: "string",
+                    description: "Path to explain: adds the resolved language plus the per-server and " +
+                        "per-runner selection decisions for it. Must resolve inside `cwd` " +
+                        "(a sibling package in the same monorepo does not count) — a file " +
+                        "outside it is rejected with the `cwd` it was measured against; " +
+                        "re-query with `cwd` set to that file's own workspace instead.",
+                },
+            },
+        },
+    },
     {
         name: "pilens_session_start",
         description: "Run pi-lens's real session_start lifecycle: warm the dominant-language " +
@@ -836,6 +864,9 @@ function formatAnalyze(result, cwd, mode, servedBy) {
         : "";
     const summary = `${path.relative(cwd, result.filePath) || result.filePath} [${mode}] — ` +
         `${result.counts.blockers} blocking, ${result.counts.warnings} warning(s), ` +
+        // #2420: advisories (hint/info-tier style opinions) are reported under
+        // their own label so they are no longer folded into the warning count.
+        `${result.counts.advisories} advisory(ies), ` +
         `${result.counts.diagnostics} total` +
         (result.latency ? ` · ${result.latency.totalDurationMs}ms` : "") +
         lspNote +
@@ -843,6 +874,51 @@ function formatAnalyze(result, cwd, mode, servedBy) {
         (servedBy ? `\n\nservedBy: ${servedBy}` : "");
     return toolText(summary, servedBy ? { ...result, servedBy } : result);
 }
+/**
+ * The `Config:` line of `pilens_health` (#2427).
+ *
+ * A named formatter rather than a ternary nested inside a template inside a
+ * ternary in the `lines` array (review round 2, F3). Counts only — the
+ * whole point of embedding provenance in a health report is that it names WHICH
+ * files contributed without carrying WHAT they said.
+ */
+function healthConfigLine(provenance) {
+    if (!provenance)
+        return "Config: unavailable (resolution failed)";
+    const tiers = Object.entries(provenance.tiers)
+        .filter(([, count]) => count > 0)
+        .map(([tier, count]) => `${tier} ${count}`)
+        .join(" · ");
+    const codes = Object.entries(provenance.codes)
+        .map(([code, count]) => `${code}×${count}`)
+        .join(", ");
+    const notices = codes.length > 0 ? ` · notices ${codes}` : "";
+    const documents = `${provenance.documents} file(s)`;
+    return `Config: ${documents} · ${tiers} leaf/leaves${notices}`;
+}
+/** One `pilens_effective_config` server row, with the tier that decided it. */
+function effectiveServerLine(server) {
+    const mark = server.selected ? "✓" : "✗";
+    const decided = server.decidedBy;
+    if (!decided)
+        return `  ${mark} ${server.id} — ${server.reason}`;
+    const file = decided.file === undefined ? "" : ` ${decided.file}`;
+    const source = `${decided.tier}${file} → ${decided.key}`;
+    return `  ${mark} ${server.id} — ${server.reason} (${source})`;
+}
+/** The `File:` heading of `pilens_effective_config`. */
+function effectiveFileHeading(file) {
+    const language = file.language === undefined ? "" : ` — ${file.language}`;
+    const kind = file.kind === undefined ? "" : ` (kind ${file.kind})`;
+    return `File: ${file.path}${language}${kind}`;
+}
+/** One `pilens_effective_config` config-document row. */
+function effectiveDocumentLine(document) {
+    const legacy = document.legacy
+        ? " (legacy location — scheduled for removal)"
+        : "";
+    return `  ${document.tier}: ${document.file}${legacy}`;
+}
 async function callTool(name, args) {
     if (name === "pilens_analyze") {
         const file = args.file;
@@ -1252,6 +1328,17 @@ async function callTool(name, args) {
         // hook records its outcome per workspace; this is where it becomes
         // visible without `claude --debug` log spelunking.
         const turnEnd = readTurnEndStatus(DEFAULT_CWD) ?? null;
+        // #2427: per-tier COUNTS only, never values — #2415's "provenance
+        // surfaces in health output without leaking values". Best-effort for the
+        // same reason the footprint below is: a config resolution failure must
+        // not take down the older, more load-bearing half of this report.
+        const configProvenance = await effectiveConfig({ cwd: DEFAULT_CWD })
+            .then((view) => ({
+            documents: view.documents.length,
+            tiers: view.provenanceCounts,
+            codes: view.recordCounts,
+        }))
+            .catch(() => null);
         // #620: best-effort — a footprint read failure must never break the rest
         // of pilens_health's (much older, more load-bearing) reporting.
         const footprint = await resourceFootprint().catch(() => null);
@@ -1280,6 +1367,7 @@ async function callTool(name, args) {
                     `${turnEnd.lastRunAt ? ` (last ran ${turnEnd.lastRunAt})` : ""}` +
                     `${turnEnd.lastSkipReason ? ` — last skip: ${turnEnd.lastSkipReason}${turnEnd.lastSkipAt ? ` at ${turnEnd.lastSkipAt}` : ""}` : ""}`
                 : "Stop-hook turn-end: no activity recorded (hook not installed, or no Stop yet)",
+            healthConfigLine(configProvenance),
             footprint
                 ? `Resource footprint: ${footprint.instanceCount} pi-lens instance(s) · ` +
                     `${(footprint.totalRssBytes / 1024 / 1024).toFixed(0)}MB RSS · ` +
@@ -1308,7 +1396,36 @@ async function callTool(name, args) {
             turnEnd,
             resourceFootprint: footprint,
             degradations,
+            configProvenance,
+        });
+    }
+    if (name === "pilens_effective_config") {
+        const cwd = typeof args.cwd === "string" ? args.cwd : DEFAULT_CWD;
+        const view = await effectiveConfig({
+            cwd,
+            ...(typeof args.file === "string" ? { file: args.file } : {}),
+            redact: true,
         });
+        const fileLines = view.file === undefined
+            ? []
+            : isEffectiveFileViewError(view.file)
+                ? [`File: ${view.file.error}`]
+                : [
+                    effectiveFileHeading(view.file),
+                    ...view.file.servers
+                        // Only the servers with something to say: every registry entry
+                        // whose extension simply does not match this file would be ~40
+                        // lines of "not applicable" ahead of the answer.
+                        .filter((server) => server.reason !== "extension-mismatch")
+                        .map(effectiveServerLine),
+                    ...view.file.tools.map((tool) => `  ${tool.selected ? "✓" : "✗"} ${tool.id} — ${tool.reason}`),
+                ];
+        const lines = [
+            `Config: ${view.documents.length} file(s) contributing, ${view.provenance.length} resolved leaf/leaves`,
+            ...view.documents.map(effectiveDocumentLine),
+            ...fileLines,
+        ];
+        return toolText(lines.join("\n"), view);
     }
     if (name === "pilens_diagnostics") {
         const cwd = typeof args.cwd === "string" ? args.cwd : DEFAULT_CWD;
@@ -1411,6 +1528,10 @@ async function callTool(name, args) {
 //   pilens_read_symbol, pilens_read_enclosing  — stateless file reads, but no
 //     existing fresh-fork path either; warn rather than silently answer with
 //     however this stale build's tree-sitter/read-symbol logic behaves.
+//   pilens_effective_config                    — derives (never reads) the
+//     LSP config for the file's directory to say which servers it selects,
+//     and the merge/deny semantics it explains are compiled code; a stale
+//     build would explain a resolution the running one no longer performs.
 //
 // `pilens_rebuild` is deliberately excluded: it doesn't answer with analysis
 // at all (it shells out to `npm run build`/`build:dist`), and it's the very
@@ -1431,6 +1552,7 @@ const WARN_ONLY_STALE_TOOLS = new Set([
     "pilens_lsp_diagnostics",
     "pilens_read_symbol",
     "pilens_read_enclosing",
+    "pilens_effective_config",
 ]);
 /**
  * Appends the warm-code-stale advisory to a tool result's text (and a