pi-lens 4.2.0 → 4.3.0

package/dist/clients/runtime-tool-result.js

@@ -484,7 +484,7 @@ function refreshCachedExports(runtime, filePath) {
  * pipeline AND record" block could not express.
  */
 async function dispatchPipelineAnalysis(args) {
-    const { deps, runtime, filePath, dispatchCwd, turnStateCwd, autofixMode, modifiedRanges, writeIndex, initialStateHash, readGuardCorrelationId, requestedEditIndexes, requestedEditTotal, isPartialApplyResult, toolResultStart, nativeAppliedPairs, allowReadGuardWrites, } = args;
+    const { deps, runtime, filePath, dispatchCwd, turnStateCwd, autofixMode, modifiedRanges, writeIndex, initialStateHash, readGuardCorrelationId, requestedEditIndexes, requestedEditTotal, isPartialApplyResult, toolResultStart, nativeAppliedPairs, allowAutonomousWriters, } = args;
     const { event, getFlag, getFlagSource, dbg, biomeClient, ruffClient, metricsClient, resetLSPService, } = deps;
     const pipelinePromise = runPipeline({
         signal: deps.signal,
@@ -493,6 +493,7 @@ async function dispatchPipelineAnalysis(args) {
         projectRoot: turnStateCwd,
         toolName: event.toolName,
         autofixMode,
+        allowAutonomousWriters,
         modifiedRanges,
         telemetry: {
             model: runtime.telemetryModel,
@@ -693,7 +694,7 @@ async function dispatchPipelineAnalysis(args) {
     // The model's write/edit and pi-lens' own immediate format/autofix are now
     // reflected on disk. Refresh read-guard staleness stamps so a follow-up edit
     // is judged by read-range coverage, not by our own previous write.
-    if (!getFlag("no-read-guard") && allowReadGuardWrites) {
+    if (!getFlag("no-read-guard") && allowAutonomousWriters) {
         const changedForReadGuard = new Set([
             path.resolve(filePath),
             ...(result.changedFiles ?? []).map((changedFile) => path.resolve(changedFile)),
@@ -710,7 +711,9 @@ export async function handleToolResult(deps) {
     const { event, getFlag, dbg, runtime, cacheManager, agentBehaviorRecord, formatBehaviorWarnings, } = deps;
     const rawFilePath = event.input.path;
     const workspaceRoot = runtime.projectRoot || process.cwd();
-    let bashAuthorshipConfirmed = deps._readGuardAuthorship ?? event.toolName !== "bash";
+    let bashAuthorshipConfirmed = deps._allowAutonomousWriters ??
+        deps._readGuardAuthorship ??
+        event.toolName !== "bash";
     // #1642: a gitignored worktree edit got re-attributed onto a
     // same-relative-path file in the parent checkout because this handler
     // used to always resolve a relative path against `workspaceRoot`
@@ -975,11 +978,9 @@ export async function handleToolResult(deps) {
         const opaqueSet = new Set(opaquePaths);
         const written = [...recognizedWritten, ...opaquePaths];
         const recognizedAuthoredSet = new Set(recognizedAuthored);
-        bashAuthorshipConfirmed =
-            recognizedAuthored.length > 0 || opaquePaths.length > 0;
+        bashAuthorshipConfirmed = recognizedAuthored.length > 0;
         for (const wp of written) {
-            if (!getFlag("no-read-guard") &&
-                (opaqueSet.has(wp) || recognizedAuthoredSet.has(wp)))
+            if (!getFlag("no-read-guard") && recognizedAuthoredSet.has(wp))
                 deps.readGuard?.recordWritten(wp);
             else if (!getFlag("no-read-guard") && recognizedWritten.includes(wp))
                 deps.readGuard?.recordUnchanged?.(wp);
@@ -1013,7 +1014,11 @@ export async function handleToolResult(deps) {
                 _autofixMode: autofixMode,
                 _attachmentBudget: syntheticAttachmentBudget,
                 _mutationSourceOverride: isOpaque ? "opaque-script" : undefined,
-                _readGuardAuthorship: opaqueSet.has(wp) || recognizedAuthoredSet.has(wp),
+                _readGuardAuthorship: recognizedAuthoredSet.has(wp),
+                // Opaque recovery is mutation evidence only. The synthetic call still
+                // records freshness and runs diagnostics, but it cannot format/autofix
+                // or issue an edit-directed blocker/actionable instruction.
+                _allowAutonomousWriters: recognizedAuthoredSet.has(wp),
             });
             if (syntheticResult) {
                 // #1590: forward verbatim. The synthetic call already charged the
@@ -1476,7 +1481,7 @@ export async function handleToolResult(deps) {
                         // bridge, so the dispatch's read-guard write refresh
                         // carries evidence and stays enabled on every
                         // per-path dispatch.
-                        allowReadGuardWrites: true,
+                        allowAutonomousWriters: true,
                     }), {
                         ms: HOOK_WALL_BUDGET_MS.tool_result_edit,
                         signal: observedDispatchSignal,
@@ -1787,7 +1792,7 @@ export async function handleToolResult(deps) {
         participantTotal,
         toolResultStart,
         nativeAppliedPairs,
-        allowReadGuardWrites: bashAuthorshipConfirmed,
+        allowAutonomousWriters: bashAuthorshipConfirmed,
     }), {
         ms: HOOK_WALL_BUDGET_MS.tool_result_edit,
         signal: deps.signal || undefined,
@@ -1808,6 +1813,7 @@ export async function handleToolResult(deps) {
     const result = dispatchOutcome.result;
     let autofixNewlyQueued = false;
     if (!result.isError &&
+        bashAuthorshipConfirmed &&
         autofixMode === "deferred" &&
         nodeFs.existsSync(filePath)) {
         autofixNewlyQueued =
@@ -1816,6 +1822,7 @@ export async function handleToolResult(deps) {
     }
     let formatQueued = false;
     if (!result.isError &&
+        bashAuthorshipConfirmed &&
         !getFlag("no-autoformat", filePath) &&
         (autofixMode === "deferred" || !getFlag("immediate-format")) &&
         nodeFs.existsSync(filePath)) {
@@ -1869,7 +1876,11 @@ export async function handleToolResult(deps) {
             source: "autofix",
             dbg,
         });
-        if (resolvedChanged === path.resolve(filePath))
+        // Workspace-edit paths come from fileURLToPath and are normally absolute,
+        // so the turn-state cwd is inert for those values. Keep it in the resolve
+        // call because the producer's contract is cwd-relative URI resolution, and
+        // let pathsEqual ask the filesystem-aware path identity question (#3294).
+        if (pathsEqual(path.resolve(turnStateCwd, changedFile), path.resolve(filePath)))
             continue;
         try {
             const content = nodeFs.readFileSync(resolvedChanged, "utf-8");
@@ -1929,7 +1940,11 @@ export async function handleToolResult(deps) {
         // #1561: stamp the verdict with THIS dispatch's write token — the same
         // counter `lsp_diagnostics`' reconciliation seam draws from — so a later
         // confirmed-clean result can be ordered against it instead of racing it.
-        runtime.recordInlineBlockers(filePath, result.inlineBlockerSummary, writeIndex, result.inlineBlockerSources, result.inlineBlockerLines, result.inlineBlockerFileContent);
+        runtime.recordInlineBlockers(filePath, result.inlineBlockerSummary, writeIndex, result.inlineBlockerSources, result.inlineBlockerLines, result.inlineBlockerFileContent, 
+        // #3246: the structured blockers the summary was rendered from, so a
+        // later `lens_diagnostic_mark` can be applied to this record at turn
+        // end instead of replaying pre-mark text.
+        result.inlineBlockerDiagnostics);
     }
     else {
         runtime.clearInlineBlockers(filePath);