pi-lens 4.1.2 → 4.1.4

package/README.md

@@ -152,6 +152,9 @@ Or from git:
 pi install git:github.com/apmantza/pi-lens
 ```
 
+Requires Node.js >=22.19.0 (matches the pi host's own floor — see `engines`
+in `package.json`).
+
 > **npm v12 users:** dependency lifecycle scripts (e.g. `@ast-grep/cli`'s
 > `postinstall`) now require explicit approval — if `npm install` warns about
 > unreviewed install scripts, review and allow them with

package/config/biome/core.jsonc

@@ -12,8 +12,17 @@
 				"noUnusedImports": "error",
 				"noUndeclaredVariables": "warn"
 			},
-			"style": {
-				"useConst": "error",
+		"style": {
+			// Off (refs #2385): its "safe" fix rewrites value imports to
+			// `import type` for imports used only in type positions, which
+			// erases the runtime binding that experimental decorator metadata
+			// (`emitDecoratorMetadata` -> design:type) still needs — verified
+			// against real biome 2.5.9 + tsc: the emitted design:type identity
+			// changes from the imported class to `Function`. Biome's own rule
+			// docs recommend disabling this rule for such repos. Explicit
+			// project biome.json(c) configs remain authoritative and unaffected.
+			"useImportType": "off",
+			"useConst": "error",
 				"useTemplate": "error",
 				"noNegationElse": "error",
 				"noNestedTernary": "error",

package/config/dependency-cruiser-eager-allowlist.json

@@ -0,0 +1,91 @@
+[
+  "./clients/agent-nudge.js",
+  "./clients/ast-grep-client.js",
+  "./clients/bootstrap.js",
+  "./clients/bounded-cache.js",
+  "./clients/bounded-telemetry.js",
+  "./clients/build-identity.js",
+  "./clients/bus-events-logger.js",
+  "./clients/bus-publish.js",
+  "./clients/cache-manager.js",
+  "./clients/cache-observability.js",
+  "./clients/cascade-format.js",
+  "./clients/console-guard-install.js",
+  "./clients/debug-handles.js",
+  "./clients/debug-heap.js",
+  "./clients/degradation-ledger.js",
+  "./clients/diagnostic-tracker.js",
+  "./clients/diagnostics-publish.js",
+  "./clients/dispatch/lazy.js",
+  "./clients/dispatch/runner-context.js",
+  "./clients/disposition-publish.js",
+  "./clients/event-loop-monitor.js",
+  "./clients/extension-log.js",
+  "./clients/extension-mode.js",
+  "./clients/file-utils.js",
+  "./clients/format-events-publish.js",
+  "./clients/format-service.js",
+  "./clients/formatters-lazy.js",
+  "./clients/git-guard.js",
+  "./clients/host-ports.js",
+  "./clients/installer/index.js",
+  "./clients/instance-reaper.js",
+  "./clients/instance-registry.js",
+  "./clients/language-profile.js",
+  "./clients/latency-logger.js",
+  "./clients/lens-config.js",
+  "./clients/lens-events.js",
+  "./clients/lens-flag-registry.js",
+  "./clients/lsp-budget.js",
+  "./clients/lsp-lazy.js",
+  "./clients/lsp-status.js",
+  "./clients/lsp/cascade-tier.js",
+  "./clients/lsp/config.js",
+  "./clients/lsp/index.js",
+  "./clients/lsp/session-roots.js",
+  "./clients/memory-sampler.js",
+  "./clients/message-end-attribution.js",
+  "./clients/mutating-tool.js",
+  "./clients/mutation-bridge.js",
+  // #2430: the agent_settled sweep and its post-drain re-baseline. Eager
+  // because they run inside the settle handler, on the same path as the
+  // deferred drain they bracket.
+  "./clients/observed-mutation-sources.js",
+  "./clients/observed-mutation.js",
+  "./clients/path-attribution-telemetry.js",
+  "./clients/path-utils.js",
+  "./clients/project-lens-config.js",
+  "./clients/project-trust.js",
+  "./clients/quiet-window.js",
+  "./clients/read-bridge.js",
+  "./clients/read-guard-tool-lines.js",
+  // index.ts:56 is type-only, but dependency-cruiser parses it as an edge.
+  "./clients/read-guard.js",
+  "./clients/recent-touches.js",
+  "./clients/runtime-agent-end.js",
+  "./clients/runtime-context.js",
+  "./clients/runtime-coordinator.js",
+  "./clients/runtime-session.js",
+  "./clients/runtime-tool-call.js",
+  "./clients/runtime-tool-result.js",
+  "./clients/runtime-turn.js",
+  "./clients/safe-spawn.js",
+  "./clients/session-event-guard.js",
+  "./clients/session-lifecycle.js",
+  "./clients/session-start-observability.js",
+  "./clients/session-state-store.js",
+  "./clients/sessionstart-logger.js",
+  "./clients/skills-resolver.js",
+  "./clients/smells-rollup.js",
+  "./clients/startup-timing.js",
+  "./clients/test-runner-delivery.js",
+  "./clients/tool-definition.js",
+  "./clients/tool-render.js",
+  "./clients/tool-set-policy.js",
+  "./clients/turn-summary-render.js",
+  "./clients/turn-summary.js",
+  "./clients/user-notify.js",
+  "./clients/vanished-instance-marker.js",
+  "./clients/warm-attach.js",
+  "./clients/widget-state.js"
+]

package/dist/clients/actionable-warnings-logger.js

@@ -1,8 +1,9 @@
 import * as path from "node:path";
 import { isTestMode } from "./env-utils.js";
-import { getGlobalPiLensDir } from "./file-utils.js";
+import { getGlobalPiLensLogDir } from "./probe-home-state.js";
 import { createNdjsonLogger } from "./ndjson-logger.js";
-const AW_LOG_DIR = getGlobalPiLensDir();
+import { normalizeFilePath } from "./path-utils.js";
+const AW_LOG_DIR = getGlobalPiLensLogDir();
 const AW_LOG_FILE = path.join(AW_LOG_DIR, "actionable-warnings.log");
 const AW_LOG_BACKUP_FILE = path.join(AW_LOG_DIR, "actionable-warnings.log.1");
 const MAX_LOG_BYTES = Math.max(128 * 1024, Number.parseInt(process.env.PI_LENS_AW_LOG_MAX_BYTES ?? "1048576", 10) ||
@@ -12,11 +13,25 @@ const writer = createNdjsonLogger({
     maxBytes: MAX_LOG_BYTES,
     backupPath: AW_LOG_BACKUP_FILE,
 });
+/**
+ * #2219 (the #2141 class): `filePath` reaches here from
+ * `actionable-warnings.ts`'s raw `path.resolve(cwd, file)` — the file
+ * imports `normalizeMapKey` and uses it for map lookups on this same value,
+ * but never normalizes what gets logged. `filePath` is optional (several
+ * events, e.g. `report_started`/`report_complete`, carry none), so only
+ * normalize when present.
+ */
 export function logActionableWarningsEvent(entry) {
     if (isTestMode()) {
         return;
     }
-    writer.log({ ts: new Date().toISOString(), ...entry });
+    writer.log({
+        ts: new Date().toISOString(),
+        ...entry,
+        ...(entry.filePath !== undefined
+            ? { filePath: normalizeFilePath(entry.filePath) }
+            : {}),
+    });
 }
 export function getActionableWarningsLogPath() {
     return AW_LOG_FILE;