@integrity-labs/agt-cli 0.28.972 → 0.28.974

package/dist/{chunk-J7J3JWZ3.js → chunk-PHPOHMSK.js}

@@ -20,7 +20,7 @@ import {
   rotateDailySession,
   sessionFileExists,
   todayLocalIso
-} from "./chunk-O3VCRRUF.js";
+} from "./chunk-HRHDATT5.js";
 import {
   classifyUnanswerablePane,
   findUsageLimitResetHint,
@@ -5575,4 +5575,4 @@ export {
   stopAllSessionsAndWait,
   getProjectDir
 };
-//# sourceMappingURL=chunk-J7J3JWZ3.js.map
+//# sourceMappingURL=chunk-PHPOHMSK.js.map

package/dist/{claude-pair-runtime-JT5WNC3G.js → claude-pair-runtime-U6GBR7KM.js}

@@ -236,7 +236,7 @@ async function spawnPairSession(session) {
     return { ok: true };
   } catch {
   }
-  const { resolveClaudeBinary } = await import("./persistent-session-24ZIVLH3.js");
+  const { resolveClaudeBinary } = await import("./persistent-session-YJJXZSUE.js");
   const claudeBin = resolveClaudeBinary();
   const pairEnv = {
     ...process.env,
@@ -558,4 +558,4 @@ export {
   startClaudePair,
   submitClaudePairCode
 };
-//# sourceMappingURL=claude-pair-runtime-JT5WNC3G.js.map
+//# sourceMappingURL=claude-pair-runtime-U6GBR7KM.js.map

package/dist/lib/manager-worker.js

@@ -63,7 +63,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-TJXGP5XV.js";
+} from "../chunk-3PYNSD4I.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -140,7 +140,7 @@ import {
   takeZombieDetection,
   toOpencodeModel,
   writeEgressAllowlist
-} from "../chunk-J7J3JWZ3.js";
+} from "../chunk-PHPOHMSK.js";
 import {
   ACCOUNT_ENFORCEMENT_MARKER_FILENAME,
   AnchorSessionClient,
@@ -238,7 +238,7 @@ import {
   subagentActivityAgeSeconds,
   sumTranscriptUsageInWindow,
   transcriptActivityAgeSeconds
-} from "../chunk-O3VCRRUF.js";
+} from "../chunk-HRHDATT5.js";
 import "../chunk-DHWNVVX4.js";
 import {
   reapOrphanChannelMcps
@@ -13182,7 +13182,7 @@ var pendingDayRolloverReset = /* @__PURE__ */ new Set();
 var dayRolloverInboundHold = /* @__PURE__ */ new Map();
 var INBOUND_HOLD_EPISODE_GAP_MS = 12e4;
 async function channelInboundActivityAgeSecondsFor(codeName) {
-  const { newestPendingInboundActivityMtimeMs } = await import("../responsiveness-probe-7KOUL2XK.js");
+  const { newestPendingInboundActivityMtimeMs } = await import("../responsiveness-probe-SQZGSNVI.js");
   const newest = newestPendingInboundActivityMtimeMs(dirname11(paneLogPath(codeName)));
   if (newest === null) return null;
   return Math.max(0, Math.floor((Date.now() - newest) / 1e3));
@@ -13867,7 +13867,7 @@ var agentRestartTimezoneInputs = /* @__PURE__ */ new Map();
 var lastVersionCheckAt = 0;
 var VERSION_CHECK_INTERVAL_MS = 5 * 60 * 1e3;
 var lastResponsivenessProbeAt = 0;
-var agtCliVersion = true ? "0.28.972" : "dev";
+var agtCliVersion = true ? "0.28.974" : "dev";
 function resolveBrewPath(execFileSync2) {
   try {
     const out = execFileSync2("which", ["brew"], { timeout: 5e3 }).toString().trim();
@@ -15469,7 +15469,7 @@ function flushRestartedAgentDiagnostics(hostId, codeNames) {
   if (codeNames.length === 0) return;
   void (async () => {
     try {
-      const { collectDiagnostics } = await import("../persistent-session-24ZIVLH3.js");
+      const { collectDiagnostics } = await import("../persistent-session-YJJXZSUE.js");
       await api.post("/host/heartbeat", {
         host_id: hostId,
         agent_diagnostics: collectDiagnostics(codeNames, quarantineEntriesFor, claudeMdSizeFor, spawnOutcomeForDiagnostics, pidPressureFor, forwardedToolsFor)
@@ -15611,7 +15611,7 @@ async function pollCycleInner() {
     }
     try {
       const { detectHostSecurity } = await import("../host-security-6PDFG7F5.js");
-      const { collectDiagnostics } = await import("../persistent-session-24ZIVLH3.js");
+      const { collectDiagnostics } = await import("../persistent-session-YJJXZSUE.js");
       const diagCodeNames = [...agentState.persistentSessionAgents];
       const agentDiagnostics = diagCodeNames.length > 0 ? collectDiagnostics(diagCodeNames, quarantineEntriesFor, claudeMdSizeFor, spawnOutcomeForDiagnostics, pidPressureFor, forwardedToolsFor) : void 0;
       let tailscaleHostname;
@@ -15787,7 +15787,7 @@ async function pollCycleInner() {
         collectPanelessActivityProbes,
         getResponsivenessIntervalMs,
         occupancyQualificationClassifications
-      } = await import("../responsiveness-probe-7KOUL2XK.js");
+      } = await import("../responsiveness-probe-SQZGSNVI.js");
       const probeIntervalMs = getResponsivenessIntervalMs();
       if (now - lastResponsivenessProbeAt > probeIntervalMs) {
         const probeCodeNames = [...agentState.persistentSessionAgents];
@@ -15904,7 +15904,7 @@ async function pollCycleInner() {
           collectResponsivenessProbes,
           livePendingInboundOldestAgeSeconds,
           parkPendingInbound
-        } = await import("../responsiveness-probe-7KOUL2XK.js");
+        } = await import("../responsiveness-probe-SQZGSNVI.js");
         const { getProjectDir: wedgeProjectDir } = await import("../scheduler-engine-NDP36U7O.js");
         const wedgeNow = /* @__PURE__ */ new Date();
         const liveAgents = agentState.persistentSessionAgents;
@@ -16060,7 +16060,7 @@ async function pollCycleInner() {
     }
     try {
       const { scrapeMcpFailedBannerCount } = await import("../pane-mcp-banner-scraper-JA437JIB.js");
-      const { probeSessionAuth } = await import("../session-auth-dead-BQVFVWZM.js");
+      const { probeSessionAuth } = await import("../session-auth-dead-7OLNDHJC.js");
       const observations = [];
       const pendingCacheCommits = [];
       const modelApiErrorReportingOn = hostFlagStore().getBoolean("model-api-error-reporting");
@@ -20336,7 +20336,7 @@ async function handleRestartDoorbell(agentId, requestedAt, restartReason) {
       try {
         const freshId = rotateSessionForWedge(codeName);
         consecutiveWedgeCycles.delete(codeName);
-        const { parkPendingInbound: parkForRestart } = await import("../responsiveness-probe-7KOUL2XK.js");
+        const { parkPendingInbound: parkForRestart } = await import("../responsiveness-probe-SQZGSNVI.js");
         const { parked, deadLettered } = parkForRestart(codeName, /* @__PURE__ */ new Date());
         const parkNote = parked > 0 ? `, ${parked} inbound parked` : "";
         const deadNote = deadLettered > 0 ? `, ${deadLettered} undeliverable dead-lettered` : "";
@@ -20379,7 +20379,7 @@ async function handleRestartDoorbell(agentId, requestedAt, restartReason) {
     void api.post("/host/restart-ack", { host_id: hostId, agent_id: agentId, restart_requested_at: requestedAt }).catch((err) => log(`[restart-lane] ack failed for '${codeName}': ${err.message}`));
     void (async () => {
       try {
-        const { collectDiagnostics } = await import("../persistent-session-24ZIVLH3.js");
+        const { collectDiagnostics } = await import("../persistent-session-YJJXZSUE.js");
         await api.post("/host/heartbeat", {
           host_id: hostId,
           agent_diagnostics: collectDiagnostics([codeName], quarantineEntriesFor, claudeMdSizeFor, spawnOutcomeForDiagnostics, pidPressureFor, forwardedToolsFor)
@@ -20430,7 +20430,7 @@ async function respawnAgentAfterMcpStop(codeName, reason) {
     }
     try {
       const hostId = await getHostId();
-      const { collectDiagnostics } = await import("../persistent-session-24ZIVLH3.js");
+      const { collectDiagnostics } = await import("../persistent-session-YJJXZSUE.js");
       await api.post("/host/heartbeat", {
         host_id: hostId,
         agent_diagnostics: collectDiagnostics([codeName], quarantineEntriesFor, claudeMdSizeFor, spawnOutcomeForDiagnostics, pidPressureFor, forwardedToolsFor)
@@ -21072,7 +21072,7 @@ async function processClaudePairSessions(agents) {
     killPairSession,
     pairTmuxSession,
     finalizeClaudePairOnboarding
-  } = await import("../claude-pair-runtime-JT5WNC3G.js");
+  } = await import("../claude-pair-runtime-U6GBR7KM.js");
   for (const pairId of pendingResp.cancelled_pair_ids ?? []) {
     log(`[claude-pair] sweeping orphan tmux session for pair ${pairId.slice(0, 8)}`);
     const killed = await killPairSession(pairTmuxSession(pairId));

package/dist/mcp/direct-chat-channel.js

@@ -32068,21 +32068,80 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // ancestors too - a wider style mutation than intended, for a probe whose
   // result the caller then rejects anyway.
   function textNodeAtPointThrough(x,y,target){
+    return throughOverlays(x,y,target,function(){ return textNodeAtPoint(x,y); });
+  }
+  // Run probe() with everything stacked IN FRONT of target muted for pointer
+  // hit-testing, then restore it. Shared by the text-node probe above and the
+  // caret placement below (ENG-10632), so there is one overlay-muting path.
+  // A target that is not in the hit stack (or no elementsFromPoint at all) runs
+  // the probe unmuted, for the reason given on textNodeAtPointThrough.
+  function throughOverlays(x,y,target,probe){
     var stack;
-    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return textNodeAtPoint(x,y); }
+    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return probe(); }
     var cut=-1, i;
-    for(i=0;i<stack.length;i++){ if(stack[i]===target){ cut=i; break; } }
-    if(cut<0) return textNodeAtPoint(x,y);
+    for(i=0;i<stack.length;i++){ if(stack[i]===target || (target && target.contains && stack[i] && stack[i].nodeType && target.contains(stack[i]))){ cut=i; break; } }
+    if(cut<0) return probe();
     var muted=[];
     for(i=0;i<cut;i++){
       muted.push([stack[i], stack[i].style ? stack[i].style.pointerEvents : '']);
       try{ stack[i].style.pointerEvents='none'; }catch(e){}
     }
-    try{ return textNodeAtPoint(x,y); }
+    try{ return probe(); }
     finally{
       for(i=0;i<muted.length;i++){ try{ muted[i][0].style.pointerEvents=muted[i][1]||''; }catch(e){} }
     }
   }
+  // ENG-10632: is this pointer event aimed INSIDE the open editor?
+  //
+  // 'direct' - the event's own target is the editor or inside it (no overlay).
+  // 'through' - the target is something laid over the editor (a deck click
+  //   zone), but the editor is in the hit stack under the pointer.
+  // null     - no edit is open, or the press is genuinely elsewhere.
+  //
+  // WHY NOT e.target ALONE. That is exactly the ENG-8477 blind spot: a pointer
+  // event targets the HIT-TEST result, and a deck's full-viewport zones win the
+  // hit test, so e.target is the zone for every press on slide text. ENG-8477
+  // fixed the click that OPENS an editor this way and left the guards that
+  // protect an OPEN editor keyed on e.target, so the second click - the one that
+  // places the caret - was handed to the zone and paged the slide.
+  //
+  // A keydown is different: it targets the FOCUSED element, so the ENG-8500 key
+  // guard's e.target test is correct and stays as it is.
+  function pointInEditor(e){
+    if(!editing || !e) return null;
+    var t=e.target;
+    if(t===editing || (editing.contains && t && t.nodeType && editing.contains(t))) return 'direct';
+    if(typeof e.clientX!=='number' || typeof e.clientY!=='number') return null;
+    var stack;
+    try{ stack=document.elementsFromPoint(e.clientX, e.clientY)||[]; }catch(err){ return null; }
+    for(var i=0;i<stack.length;i++){
+      var n=stack[i];
+      if(n===editing || (editing.contains && n && n.nodeType && editing.contains(n))) return 'through';
+    }
+    return null;
+  }
+  // ENG-10632: put the caret where the viewer pressed, as the browser would have
+  // if nothing covered the editor. The browser's own caret placement goes to the
+  // topmost element, which is the overlay, so it places nothing in the text.
+  // Returns true when a caret was placed inside the editor.
+  function placeCaretAtPointThrough(x,y,el){
+    return throughOverlays(x,y,el,function(){
+      try{
+        var node=null, offset=0, r=null;
+        if(document.caretRangeFromPoint){
+          r=document.caretRangeFromPoint(x,y);
+          if(r){ node=r.startContainer; offset=r.startOffset||0; }
+        } else if(document.caretPositionFromPoint){
+          var p=document.caretPositionFromPoint(x,y);
+          if(p){ node=p.offsetNode; offset=p.offset||0; }
+        }
+        if(!node || !(node===el || (el.contains && el.contains(node.nodeType===3 ? node.parentNode||node.parentElement : node)))) return false;
+        if(!r){ r=document.createRange(); r.setStart(node, offset); r.collapse(true); }
+        var s=document.getSelection(); s.removeAllRanges(); s.addRange(r);
+        return true;
+      }catch(err){ return false; }
+    });
+  }
   // Index of a text node among its parent's child TEXT nodes, in order - the same
   // counting the server uses to re-locate it in source_html (ENG-6856).
   function textIndexOf(container, tn){
@@ -32090,6 +32149,13 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
     for(var k=0;k<ns.length;k++){ if(ns[k].nodeType===3){ i++; if(ns[k]===tn) return i; } }
     return -1;
   }
+  function keepCaretIn(el){
+    try{
+      var s=document.getSelection(), n=s && s.rangeCount ? s.anchorNode : null;
+      if(n && (n===el || (el.contains && el.contains(n.nodeType===3 ? n.parentNode : n)))) return;
+    }catch(err){}
+    placeCaretEnd(el);
+  }
   function placeCaretEnd(el){ try{ var r=document.createRange(); r.selectNodeContents(el); r.collapse(false); var s=document.getSelection(); s.removeAllRanges(); s.addRange(r); }catch(e){} }
   // The Save/Cancel toolbar anchored above el. Shared by whole-element edits
   // (startEdit) and per-text-node edits (startEditTextNode).
@@ -32299,9 +32365,35 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // press that lands inside the editor. A press anywhere else is the viewer
   // leaving, and the artefact keeps it.
   window.addEventListener('pointerdown', function(e){
-    if(!editing) return;
-    var t=e.target;
-    if(t!==editing && !(editing.contains && t && t.nodeType && editing.contains(t))) return;
+    // ENG-10632: "inside the editor" is decided through the hit stack, not from
+    // e.target, which a deck's click zone always wins. See pointInEditor.
+    var where=pointInEditor(e);
+    if(!where) return;
+    e.stopImmediatePropagation();
+    if(where==='through'){
+      // The overlay is the browser's hit target, so its default action would
+      // move focus off the editor and place no caret in the text. Take both over.
+      try{ e.preventDefault(); }catch(err){}
+      try{ editing.focus(); }catch(err){}
+      // The press is suppressed either way, so a probe that cannot resolve a
+      // point inside the editor must not leave it without a caret: keep one the
+      // editor already holds, otherwise put it at the end of the text.
+      if(!placeCaretAtPointThrough(e.clientX, e.clientY, editing)) keepCaretIn(editing);
+    }
+  }, true);
+  // ENG-10632: the same press also arrives as mousedown and click, and the deck
+  // can page on either. Only a press that went THROUGH an overlay into the open
+  // editor is swallowed; a direct press keeps its native behaviour (caret,
+  // drag-select), and a press anywhere else is the viewer leaving and is left to
+  // the artefact.
+  window.addEventListener('mousedown', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
+    e.stopImmediatePropagation();
+  }, true);
+  window.addEventListener('click', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
     e.stopImmediatePropagation();
   }, true);
   // ENG-8477: the hover affordance has the same blind spot as the click did -

package/dist/mcp/index.js

@@ -24656,21 +24656,80 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // ancestors too - a wider style mutation than intended, for a probe whose
   // result the caller then rejects anyway.
   function textNodeAtPointThrough(x,y,target){
+    return throughOverlays(x,y,target,function(){ return textNodeAtPoint(x,y); });
+  }
+  // Run probe() with everything stacked IN FRONT of target muted for pointer
+  // hit-testing, then restore it. Shared by the text-node probe above and the
+  // caret placement below (ENG-10632), so there is one overlay-muting path.
+  // A target that is not in the hit stack (or no elementsFromPoint at all) runs
+  // the probe unmuted, for the reason given on textNodeAtPointThrough.
+  function throughOverlays(x,y,target,probe){
     var stack;
-    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return textNodeAtPoint(x,y); }
+    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return probe(); }
     var cut=-1, i;
-    for(i=0;i<stack.length;i++){ if(stack[i]===target){ cut=i; break; } }
-    if(cut<0) return textNodeAtPoint(x,y);
+    for(i=0;i<stack.length;i++){ if(stack[i]===target || (target && target.contains && stack[i] && stack[i].nodeType && target.contains(stack[i]))){ cut=i; break; } }
+    if(cut<0) return probe();
     var muted=[];
     for(i=0;i<cut;i++){
       muted.push([stack[i], stack[i].style ? stack[i].style.pointerEvents : '']);
       try{ stack[i].style.pointerEvents='none'; }catch(e){}
     }
-    try{ return textNodeAtPoint(x,y); }
+    try{ return probe(); }
     finally{
       for(i=0;i<muted.length;i++){ try{ muted[i][0].style.pointerEvents=muted[i][1]||''; }catch(e){} }
     }
   }
+  // ENG-10632: is this pointer event aimed INSIDE the open editor?
+  //
+  // 'direct' - the event's own target is the editor or inside it (no overlay).
+  // 'through' - the target is something laid over the editor (a deck click
+  //   zone), but the editor is in the hit stack under the pointer.
+  // null     - no edit is open, or the press is genuinely elsewhere.
+  //
+  // WHY NOT e.target ALONE. That is exactly the ENG-8477 blind spot: a pointer
+  // event targets the HIT-TEST result, and a deck's full-viewport zones win the
+  // hit test, so e.target is the zone for every press on slide text. ENG-8477
+  // fixed the click that OPENS an editor this way and left the guards that
+  // protect an OPEN editor keyed on e.target, so the second click - the one that
+  // places the caret - was handed to the zone and paged the slide.
+  //
+  // A keydown is different: it targets the FOCUSED element, so the ENG-8500 key
+  // guard's e.target test is correct and stays as it is.
+  function pointInEditor(e){
+    if(!editing || !e) return null;
+    var t=e.target;
+    if(t===editing || (editing.contains && t && t.nodeType && editing.contains(t))) return 'direct';
+    if(typeof e.clientX!=='number' || typeof e.clientY!=='number') return null;
+    var stack;
+    try{ stack=document.elementsFromPoint(e.clientX, e.clientY)||[]; }catch(err){ return null; }
+    for(var i=0;i<stack.length;i++){
+      var n=stack[i];
+      if(n===editing || (editing.contains && n && n.nodeType && editing.contains(n))) return 'through';
+    }
+    return null;
+  }
+  // ENG-10632: put the caret where the viewer pressed, as the browser would have
+  // if nothing covered the editor. The browser's own caret placement goes to the
+  // topmost element, which is the overlay, so it places nothing in the text.
+  // Returns true when a caret was placed inside the editor.
+  function placeCaretAtPointThrough(x,y,el){
+    return throughOverlays(x,y,el,function(){
+      try{
+        var node=null, offset=0, r=null;
+        if(document.caretRangeFromPoint){
+          r=document.caretRangeFromPoint(x,y);
+          if(r){ node=r.startContainer; offset=r.startOffset||0; }
+        } else if(document.caretPositionFromPoint){
+          var p=document.caretPositionFromPoint(x,y);
+          if(p){ node=p.offsetNode; offset=p.offset||0; }
+        }
+        if(!node || !(node===el || (el.contains && el.contains(node.nodeType===3 ? node.parentNode||node.parentElement : node)))) return false;
+        if(!r){ r=document.createRange(); r.setStart(node, offset); r.collapse(true); }
+        var s=document.getSelection(); s.removeAllRanges(); s.addRange(r);
+        return true;
+      }catch(err){ return false; }
+    });
+  }
   // Index of a text node among its parent's child TEXT nodes, in order - the same
   // counting the server uses to re-locate it in source_html (ENG-6856).
   function textIndexOf(container, tn){
@@ -24678,6 +24737,13 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
     for(var k=0;k<ns.length;k++){ if(ns[k].nodeType===3){ i++; if(ns[k]===tn) return i; } }
     return -1;
   }
+  function keepCaretIn(el){
+    try{
+      var s=document.getSelection(), n=s && s.rangeCount ? s.anchorNode : null;
+      if(n && (n===el || (el.contains && el.contains(n.nodeType===3 ? n.parentNode : n)))) return;
+    }catch(err){}
+    placeCaretEnd(el);
+  }
   function placeCaretEnd(el){ try{ var r=document.createRange(); r.selectNodeContents(el); r.collapse(false); var s=document.getSelection(); s.removeAllRanges(); s.addRange(r); }catch(e){} }
   // The Save/Cancel toolbar anchored above el. Shared by whole-element edits
   // (startEdit) and per-text-node edits (startEditTextNode).
@@ -24887,9 +24953,35 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // press that lands inside the editor. A press anywhere else is the viewer
   // leaving, and the artefact keeps it.
   window.addEventListener('pointerdown', function(e){
-    if(!editing) return;
-    var t=e.target;
-    if(t!==editing && !(editing.contains && t && t.nodeType && editing.contains(t))) return;
+    // ENG-10632: "inside the editor" is decided through the hit stack, not from
+    // e.target, which a deck's click zone always wins. See pointInEditor.
+    var where=pointInEditor(e);
+    if(!where) return;
+    e.stopImmediatePropagation();
+    if(where==='through'){
+      // The overlay is the browser's hit target, so its default action would
+      // move focus off the editor and place no caret in the text. Take both over.
+      try{ e.preventDefault(); }catch(err){}
+      try{ editing.focus(); }catch(err){}
+      // The press is suppressed either way, so a probe that cannot resolve a
+      // point inside the editor must not leave it without a caret: keep one the
+      // editor already holds, otherwise put it at the end of the text.
+      if(!placeCaretAtPointThrough(e.clientX, e.clientY, editing)) keepCaretIn(editing);
+    }
+  }, true);
+  // ENG-10632: the same press also arrives as mousedown and click, and the deck
+  // can page on either. Only a press that went THROUGH an overlay into the open
+  // editor is swallowed; a direct press keeps its native behaviour (caret,
+  // drag-select), and a press anywhere else is the viewer leaving and is left to
+  // the artefact.
+  window.addEventListener('mousedown', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
+    e.stopImmediatePropagation();
+  }, true);
+  window.addEventListener('click', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
     e.stopImmediatePropagation();
   }, true);
   // ENG-8477: the hover affordance has the same blind spot as the click did -

package/dist/mcp/origami.js

@@ -38807,21 +38807,80 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // ancestors too - a wider style mutation than intended, for a probe whose
   // result the caller then rejects anyway.
   function textNodeAtPointThrough(x,y,target){
+    return throughOverlays(x,y,target,function(){ return textNodeAtPoint(x,y); });
+  }
+  // Run probe() with everything stacked IN FRONT of target muted for pointer
+  // hit-testing, then restore it. Shared by the text-node probe above and the
+  // caret placement below (ENG-10632), so there is one overlay-muting path.
+  // A target that is not in the hit stack (or no elementsFromPoint at all) runs
+  // the probe unmuted, for the reason given on textNodeAtPointThrough.
+  function throughOverlays(x,y,target,probe){
     var stack;
-    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return textNodeAtPoint(x,y); }
+    try{ stack=document.elementsFromPoint(x,y)||[]; }catch(e){ return probe(); }
     var cut=-1, i;
-    for(i=0;i<stack.length;i++){ if(stack[i]===target){ cut=i; break; } }
-    if(cut<0) return textNodeAtPoint(x,y);
+    for(i=0;i<stack.length;i++){ if(stack[i]===target || (target && target.contains && stack[i] && stack[i].nodeType && target.contains(stack[i]))){ cut=i; break; } }
+    if(cut<0) return probe();
     var muted=[];
     for(i=0;i<cut;i++){
       muted.push([stack[i], stack[i].style ? stack[i].style.pointerEvents : '']);
       try{ stack[i].style.pointerEvents='none'; }catch(e){}
     }
-    try{ return textNodeAtPoint(x,y); }
+    try{ return probe(); }
     finally{
       for(i=0;i<muted.length;i++){ try{ muted[i][0].style.pointerEvents=muted[i][1]||''; }catch(e){} }
     }
   }
+  // ENG-10632: is this pointer event aimed INSIDE the open editor?
+  //
+  // 'direct' - the event's own target is the editor or inside it (no overlay).
+  // 'through' - the target is something laid over the editor (a deck click
+  //   zone), but the editor is in the hit stack under the pointer.
+  // null     - no edit is open, or the press is genuinely elsewhere.
+  //
+  // WHY NOT e.target ALONE. That is exactly the ENG-8477 blind spot: a pointer
+  // event targets the HIT-TEST result, and a deck's full-viewport zones win the
+  // hit test, so e.target is the zone for every press on slide text. ENG-8477
+  // fixed the click that OPENS an editor this way and left the guards that
+  // protect an OPEN editor keyed on e.target, so the second click - the one that
+  // places the caret - was handed to the zone and paged the slide.
+  //
+  // A keydown is different: it targets the FOCUSED element, so the ENG-8500 key
+  // guard's e.target test is correct and stays as it is.
+  function pointInEditor(e){
+    if(!editing || !e) return null;
+    var t=e.target;
+    if(t===editing || (editing.contains && t && t.nodeType && editing.contains(t))) return 'direct';
+    if(typeof e.clientX!=='number' || typeof e.clientY!=='number') return null;
+    var stack;
+    try{ stack=document.elementsFromPoint(e.clientX, e.clientY)||[]; }catch(err){ return null; }
+    for(var i=0;i<stack.length;i++){
+      var n=stack[i];
+      if(n===editing || (editing.contains && n && n.nodeType && editing.contains(n))) return 'through';
+    }
+    return null;
+  }
+  // ENG-10632: put the caret where the viewer pressed, as the browser would have
+  // if nothing covered the editor. The browser's own caret placement goes to the
+  // topmost element, which is the overlay, so it places nothing in the text.
+  // Returns true when a caret was placed inside the editor.
+  function placeCaretAtPointThrough(x,y,el){
+    return throughOverlays(x,y,el,function(){
+      try{
+        var node=null, offset=0, r=null;
+        if(document.caretRangeFromPoint){
+          r=document.caretRangeFromPoint(x,y);
+          if(r){ node=r.startContainer; offset=r.startOffset||0; }
+        } else if(document.caretPositionFromPoint){
+          var p=document.caretPositionFromPoint(x,y);
+          if(p){ node=p.offsetNode; offset=p.offset||0; }
+        }
+        if(!node || !(node===el || (el.contains && el.contains(node.nodeType===3 ? node.parentNode||node.parentElement : node)))) return false;
+        if(!r){ r=document.createRange(); r.setStart(node, offset); r.collapse(true); }
+        var s=document.getSelection(); s.removeAllRanges(); s.addRange(r);
+        return true;
+      }catch(err){ return false; }
+    });
+  }
   // Index of a text node among its parent's child TEXT nodes, in order - the same
   // counting the server uses to re-locate it in source_html (ENG-6856).
   function textIndexOf(container, tn){
@@ -38829,6 +38888,13 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
     for(var k=0;k<ns.length;k++){ if(ns[k].nodeType===3){ i++; if(ns[k]===tn) return i; } }
     return -1;
   }
+  function keepCaretIn(el){
+    try{
+      var s=document.getSelection(), n=s && s.rangeCount ? s.anchorNode : null;
+      if(n && (n===el || (el.contains && el.contains(n.nodeType===3 ? n.parentNode : n)))) return;
+    }catch(err){}
+    placeCaretEnd(el);
+  }
   function placeCaretEnd(el){ try{ var r=document.createRange(); r.selectNodeContents(el); r.collapse(false); var s=document.getSelection(); s.removeAllRanges(); s.addRange(r); }catch(e){} }
   // The Save/Cancel toolbar anchored above el. Shared by whole-element edits
   // (startEdit) and per-text-node edits (startEditTextNode).
@@ -39038,9 +39104,35 @@ var MARKUP_BRIDGE_SCRIPT = `<script>(function(){
   // press that lands inside the editor. A press anywhere else is the viewer
   // leaving, and the artefact keeps it.
   window.addEventListener('pointerdown', function(e){
-    if(!editing) return;
-    var t=e.target;
-    if(t!==editing && !(editing.contains && t && t.nodeType && editing.contains(t))) return;
+    // ENG-10632: "inside the editor" is decided through the hit stack, not from
+    // e.target, which a deck's click zone always wins. See pointInEditor.
+    var where=pointInEditor(e);
+    if(!where) return;
+    e.stopImmediatePropagation();
+    if(where==='through'){
+      // The overlay is the browser's hit target, so its default action would
+      // move focus off the editor and place no caret in the text. Take both over.
+      try{ e.preventDefault(); }catch(err){}
+      try{ editing.focus(); }catch(err){}
+      // The press is suppressed either way, so a probe that cannot resolve a
+      // point inside the editor must not leave it without a caret: keep one the
+      // editor already holds, otherwise put it at the end of the text.
+      if(!placeCaretAtPointThrough(e.clientX, e.clientY, editing)) keepCaretIn(editing);
+    }
+  }, true);
+  // ENG-10632: the same press also arrives as mousedown and click, and the deck
+  // can page on either. Only a press that went THROUGH an overlay into the open
+  // editor is swallowed; a direct press keeps its native behaviour (caret,
+  // drag-select), and a press anywhere else is the viewer leaving and is left to
+  // the artefact.
+  window.addEventListener('mousedown', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
+    e.stopImmediatePropagation();
+  }, true);
+  window.addEventListener('click', function(e){
+    if(pointInEditor(e)!=='through') return;
+    try{ e.preventDefault(); }catch(err){}
     e.stopImmediatePropagation();
   }, true);
   // ENG-8477: the hover affordance has the same blind spot as the click did -