arrayview 0.29.0__tar.gz → 0.29.2__tar.gz

{arrayview-0.29.0 → arrayview-0.29.2}/.mex/context/lifecycle.md

@@ -60,7 +60,7 @@ This contract describes who owns the backend, when it starts, and what closes it
 
 - Remote or tunnel launches may persist when `--serve` or tunnel display ownership requires it.
 - VS Code tunnel display uses forwarded localhost URLs; the extension should configure the port, promote privacy when available, and resolve the URL with `asExternalUri`.
-- With multiple registered tunnel windows, a missing or stale `ARRAYVIEW_WINDOW_ID` must fail closed with a diagnostic rather than broadcasting to whichever window is focused.
+- With multiple registered tunnel windows, a missing or stale `ARRAYVIEW_WINDOW_ID` uses the shared broadcast signal; only the focused VS Code window may claim it.
 - An exact registered `ARRAYVIEW_WINDOW_ID` wins; do not redirect it to a newer same-parent registration because live tunnel windows can share ancestry.
 - Plain SSH should use `localhost` forwarding guidance and stay transient unless a shared server was explicitly requested.
 

{arrayview-0.29.0 → arrayview-0.29.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arrayview
-Version: 0.29.0
+Version: 0.29.2
 Summary: Fast multi-dimensional array viewer
 Project-URL: Home, https://github.com/oscarvanderheide/arrayview
 Project-URL: Source, https://github.com/oscarvanderheide/arrayview

arrayview-0.29.2/docs/vscode-window-routing-plan.md

@@ -0,0 +1,37 @@
+# VS Code Window Routing Plan
+
+## Problem
+
+ArrayView can open the viewer tab in the wrong VS Code window when multiple
+local windows and multiple remote/tunnel windows are active. The unsafe cases
+are the ones where Python cannot prove the originating window and falls back to
+writing multiple signal files or a shared signal file.
+
+## Contract
+
+- An exact `ARRAYVIEW_WINDOW_ID` wins when its registration exists.
+- A missing or stale `ARRAYVIEW_WINDOW_ID` with multiple active registrations
+  fails closed with a diagnostic.
+- PID ancestry and IPC hook matching are allowed only when they identify a
+  single window.
+- tmux client fallback is allowed only when all attached clients for the current
+  tmux session report the same `ARRAYVIEW_WINDOW_ID`.
+- File-explorer opens launched by the extension keep using the extension's own
+  `ARRAYVIEW_WINDOW_ID`.
+
+## Implementation Slice
+
+- Tighten `src/arrayview/_vscode_signal.py` window selection.
+- Add focused lifecycle tests for local ambiguity, stale exact IDs, exact-ID
+  ownership, and tmux multi-client ambiguity.
+- Avoid changing `_app.py` or the viewer frontend.
+
+## Manual Verification Targets
+
+- Local VS Code terminal, one window: opens in that window.
+- Local VS Code terminal, several windows: stale/missing exact ID prints a
+  diagnostic instead of opening elsewhere.
+- Remote/tunnel terminal, several windows: exact ID opens the matching window;
+  stale/missing ID fails closed.
+- File explorer custom editor: spawned Python receives the extension window ID
+  and opens back into the same window.

{arrayview-0.29.0 → arrayview-0.29.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "arrayview"
-version = "0.29.0"
+version = "0.29.2"
 description = "Fast multi-dimensional array viewer"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.12"

{arrayview-0.29.0 → arrayview-0.29.2}/scripts/release.sh

@@ -8,7 +8,7 @@ set -euo pipefail
 BUMP="minor"
 DRY_RUN=true
 NO_AI=false
-AI_TOOL="claude"
+AI_TOOL="codex"
 
 usage() {
     cat <<EOF
@@ -16,7 +16,7 @@ Usage: $(basename "$0") [OPTIONS]
 
 Options:
   --bump {major,minor,patch}   Version bump type (default: minor)
-  --ai {claude,codex}          AI tool for release notes (default: claude)
+  --ai {claude,codex}          AI tool for release notes (default: codex)
   --execute                    Actually run (default is dry-run)
   --no-ai                      Skip AI release notes, use GitHub's --generate-notes
   -h, --help                   Show this help
@@ -116,9 +116,13 @@ Rules:
             if command -v "$CODEX_BIN" &>/dev/null; then
                 echo "Generating release notes with Codex..."
                 tmpfile=$(mktemp)
-                trap 'rm -f "$tmpfile"' EXIT
-                if printf '%s\n' "$PROMPT" | "$CODEX_BIN" exec --output-last-message "$tmpfile" - >/dev/null 2>/dev/null; then
+                errfile=$(mktemp)
+                trap 'rm -f "$tmpfile" "$errfile"' EXIT
+                if printf '%s\n' "$PROMPT" | "$CODEX_BIN" exec --output-last-message "$tmpfile" - >/dev/null 2>"$errfile"; then
                     NOTES=$(<"$tmpfile")
+                elif [[ -s "$errfile" ]]; then
+                    echo "Codex release notes failed:"
+                    sed 's/^/  /' "$errfile"
                 fi
             fi
             ;;
@@ -127,7 +131,19 @@ fi
 
 if [[ -z "$NOTES" ]]; then
     if [[ "$NO_AI" == false ]]; then
-        echo "AI notes unavailable, falling back to --generate-notes"
+        echo "AI notes unavailable, falling back to commit-based notes"
+    fi
+    NOTES="## What's new in $TAG"
+    if [[ -n "$COMMITS" ]]; then
+        while IFS= read -r commit; do
+            [[ -z "$commit" ]] && continue
+            message="${commit#* }"
+            NOTES+=$'\n'
+            NOTES+="- ${message}"
+        done <<< "$COMMITS"
+    else
+        NOTES+=$'\n'
+        NOTES+="- Maintenance release."
     fi
 fi
 
@@ -146,17 +162,10 @@ run git push origin main
 run git tag "$TAG"
 run git push origin "$TAG"
 
-if [[ -n "$NOTES" ]]; then
-    run gh release create "$TAG" \
-        --title "$TAG" \
-        --notes "$NOTES" \
-        --prerelease
-else
-    run gh release create "$TAG" \
-        --title "$TAG" \
-        --generate-notes \
-        --prerelease
-fi
+run gh release create "$TAG" \
+    --title "$TAG" \
+    --notes "$NOTES" \
+    --prerelease
 
 if [[ "$DRY_RUN" == true ]]; then
     echo ""

{arrayview-0.29.0 → arrayview-0.29.2}/src/arrayview/_launcher.py

@@ -677,9 +677,7 @@ def _activate_early_cli_native_shell(
         notified = bool(notify_result.get("notified"))
     except Exception:
         notified = False
-    if notified and _wait_for_native_shell_or_viewer_connection(
-        port, viewer_before=viewer_before
-    ):
+    if notified and _wait_for_viewer_connection(port, before=viewer_before):
         return True
     _vprint(
         "[ArrayView] Native window did not connect to the backend; falling back to browser",
@@ -1216,8 +1214,12 @@ def _handle_cli_spawned_daemon(
         and not is_remote
         and not overlay_files
         and not compare_files
+        and not sys.platform.startswith("linux")
     ):
-        url_shell_early = _shell_url(port, sid, name)
+        # Linux QtWebEngine denies sessionStorage to the viewer iframe when its
+        # parent shell was created from inline data. Let Linux open the normal
+        # HTTP shell after the daemon is listening instead.
+        url_shell_early = f"http://{_LOOPBACK_HOST}:{port}/shell"
         early_native_shell_opened, early_native_shell_proc = _open_webview_cli_tracked(
             url_shell_early, 1400, 900, shell_port=port
         )

{arrayview-0.29.0 → arrayview-0.29.2}/src/arrayview/_viewer.html

@@ -439,7 +439,6 @@
         #mv-cb { border: none; }
         .highlight { color: var(--highlight); font-weight: bold; }
         .dim-size { opacity: 0.28; font-size: 0.68em; font-weight: normal; }
-        .dim-hover-idx { font-size: 0.68em; font-weight: normal; color: var(--dim-hover, #777); opacity: 1; }
         .active-dim { font-weight: bold; }
         .playing-dim { font-weight: bold; }
         .vfield-t-dim { color: var(--vfield-t-dim); }
@@ -3338,6 +3337,7 @@
         #mv-panes.mv-crosshair-active .mv-pane::after {
             content: ''; position: absolute; inset: 0; z-index: 4;
             box-shadow: inset 0 0 0 var(--mv-plane-border-width, 1px) var(--mv-plane-border, var(--active-dim));
+            opacity: 0.45;
             border-radius: inherit; box-sizing: border-box;
             pointer-events: none;
         }
@@ -13885,7 +13885,6 @@
             if (!['circle', 'rect', 'freehand', 'floodfill'].includes(_roiShape)) _roiShape = 'circle';
             _roiVisible = true;
             hideAllPixelInfo();
-            _resetDimbarHover();
             canvas.style.cursor = 'crosshair';
             if (qmriActive) qmriViews.forEach(v => { v.canvas.style.cursor = 'crosshair'; });
             if (multiViewActive) mvViews.forEach(v => { v.canvas.style.cursor = 'crosshair'; });
@@ -16481,38 +16480,6 @@
             _updateMvPaneFill();
         }
 
-        function _dimHoverPadded(val, maxVal) {
-            const maxStr = String(maxVal);
-            const valStr = String(val);
-            const padLen = maxStr.length - valStr.length;
-            if (padLen <= 0) return '/' + valStr;
-            return '/<span style="opacity:0.28">' + '0'.repeat(padLen) + '</span>' + valStr;
-        }
-        function _updateDimbarHover(px, py, xLabel, yLabel, pz) {
-            // xLabel/yLabel: 'x','y','z' — which dimbar label to update (default x,y)
-            // pz: optional z index (for mosaic mode)
-            const xl = xLabel || 'x', yl = yLabel || 'y';
-            const xEl = document.getElementById('dim-' + xl + '-size');
-            const yEl = document.getElementById('dim-' + yl + '-size');
-            const zEl = document.getElementById('dim-z-size');
-            // Resolve label to shape dim index for padding width
-            const labelToDim = (multiViewActive || compareMvActive)
-                ? { x: (multiViewActive ? mvDims : compareMvDims)[0], y: (multiViewActive ? mvDims : compareMvDims)[1], z: (multiViewActive ? mvDims : compareMvDims)[2] }
-                : { x: dim_x, y: dim_y, z: dim_z };
-            const xMax = labelToDim[xl] >= 0 ? shape[labelToDim[xl]] : 1;
-            const yMax = labelToDim[yl] >= 0 ? shape[labelToDim[yl]] : 1;
-            if (xEl) { xEl.innerHTML = _dimHoverPadded(px, xMax); xEl.className = 'dim-hover-idx'; }
-            if (yEl) { yEl.innerHTML = _dimHoverPadded(py, yMax); yEl.className = 'dim-hover-idx'; }
-            if (zEl && pz != null) {
-                const zMax = dim_z >= 0 ? shape[dim_z] : 1;
-                zEl.innerHTML = _dimHoverPadded(pz + 1, zMax); zEl.className = 'dim-hover-idx';
-            }
-        }
-        function _resetDimbarHover() {
-            // Re-render dimbar to restore correct max sizes for all modes
-            renderInfo();
-        }
-
         // Gnomon visibility: shown while a dim-bar token is hovered, or flashed
         // briefly when activeDim changes via h/l/←/→. Hidden otherwise.
         // In oblique mode the x/y labels no longer name axis-aligned directions,
@@ -17711,13 +17678,18 @@
 
         function saveState() {
             if (!sid) return;
-            sessionStorage.setItem('av_' + sid, JSON.stringify(collectStateSnapshot()));
+            try {
+                sessionStorage.setItem('av_' + sid, JSON.stringify(collectStateSnapshot()));
+            } catch (e) {
+                // Some embedded webviews deny storage access. Viewing must keep
+                // working even when this optional convenience is unavailable.
+            }
         }
 
         function restoreStateFromSessionStorage() {
-            const raw = sessionStorage.getItem('av_' + sid);
-            if (!raw) return false;
             try {
+                const raw = sessionStorage.getItem('av_' + sid);
+                if (!raw) return false;
                 return applyStateSnapshot(JSON.parse(raw));
             } catch (e) {
                 return false;
@@ -20117,7 +20089,6 @@
                             showPixelInfo(document.getElementById('main-pixel-info'), { valueText: _pixelFmt(d.value, _activeNumericContext()), rawX, rawY }, e.clientX, e.clientY);
                         }).catch(() => {});
                 }
-                _updateDimbarHover(localX, localY, null, null, zIdx);
                 return;
             }
 
@@ -20131,11 +20102,6 @@
                 imgW: canvas.width,
                 imgH: canvas.height,
                 displayEl: document.getElementById('main-pixel-info'),
-                onResult: (d, px, py) => {
-                    if (_pixelInfoActive() && !multiViewActive && !compareActive) {
-                        _updateDimbarHover(px, py);
-                    }
-                },
             });
         });
         canvas.addEventListener('mouseleave', () => {
@@ -20154,8 +20120,7 @@
             if (_pixelInfoHeld) {
                 _pixelInfoHeld = false;
                 if (!_pixelInfoVisible) {
-                    const piEl = document.getElementById('main-pixel-info');
-                    if (piEl) piEl.style.display = 'none';
+                    hideAllPixelInfo();
                 }
             }
         });
@@ -20924,7 +20889,6 @@
                     if (!_pixelInfoVisible) {
                         _clearHoverPins();
                         hideAllPixelInfo();
-                        _resetDimbarHover();
                         const axEl = document.getElementById('main-axes-svg');
                         if (axEl) {
                             axEl.querySelector('.axes-lbl-x').textContent = 'x';
@@ -26047,21 +26011,6 @@
                         imgH: view.lastH,
                         view,
                         displayEl: view.pixelInfoEl,
-                        onResult: (d, px, py) => {
-                            if (_pixelInfoActive()) {
-                                // Map pane dims to dimbar x/y/z labels
-                                const labels = ['x','y','z'];
-                                const activeDims = multiViewActive ? mvDims : (compareMvActive ? compareMvDims : []);
-                                const posX = activeDims.indexOf(view.dimX);
-                                const posY = activeDims.indexOf(view.dimY);
-                                const lblX = labels[posX] || 'x';
-                                const lblY = labels[posY] || 'y';
-                                const elX = document.getElementById('dim-' + lblX + '-size');
-                                const elY = document.getElementById('dim-' + lblY + '-size');
-                                if (elX) { elX.textContent = '/' + px; elX.className = 'dim-hover-idx'; }
-                                if (elY) { elY.textContent = '/' + py; elY.className = 'dim-hover-idx'; }
-                            }
-                        },
                     });
                 });
                 cv.addEventListener('mouseleave', () => {
@@ -26072,7 +26021,6 @@
                     if (tq) tq.classList.remove('visible');
                     _roiSuppressPixelInfo = false;
                     if (!mvDraggingView && view.pixelInfoEl) view.pixelInfoEl.style.display = 'none';
-                    if (_pixelInfoActive()) _resetDimbarHover();
                 });
                 // Click-to-copy: multiview canvases
                 let _mvClickDown = null;
@@ -26935,7 +26883,6 @@
                             view,
                             onResult: (d, px, py) => {
                                 if (_pixelInfoActive()) {
-                                    _updateDimbarHover(px, py);
                                     return; // hover mode: only show hovered pane's value
                                 }
                                 // Non-hover: fetch for all other qmri views at same pixel
@@ -26962,10 +26909,9 @@
                         });
                     });
                     cv.addEventListener('mouseleave', () => {
-                        if (_qmriLoupeDrag && _qmriLoupeDrag.canvas === cv) _endQmriLoupe();
-                        qmriViews.forEach(v => { if (v.pixelInfoEl) v.pixelInfoEl.style.display = 'none'; });
-                        if (_pixelInfoActive()) _resetDimbarHover();
-                    });
+                    if (_qmriLoupeDrag && _qmriLoupeDrag.canvas === cv) _endQmriLoupe();
+                    qmriViews.forEach(v => { if (v.pixelInfoEl) v.pixelInfoEl.style.display = 'none'; });
+                });
                     // ROI drawing on qMRI canvases
                     if (rectRoiMode) cv.style.cursor = 'crosshair';
                     cv.addEventListener('mousemove', (e) => {
@@ -27029,6 +26975,11 @@
                         }
                     });
                     cv.addEventListener('mousedown', (e) => {
+                        if (!rectRoiMode && e.button === 0 && !e.shiftKey && !e.metaKey && !e.ctrlKey && !e.altKey
+                            && !_cropActive && !_segMode && !_rulerMode) {
+                            _pixelInfoHeld = true;
+                            cv.dispatchEvent(new MouseEvent('mousemove', { clientX: e.clientX, clientY: e.clientY, bubbles: true }));
+                        }
                         if (!rectRoiMode || e.button !== 0 || e.shiftKey || e.metaKey) return;
                         e.preventDefault(); sink.focus();
                         const rect = cv.getBoundingClientRect();
@@ -27449,15 +27400,17 @@
                             view,
                             displayEl: view.pixelInfoEl,
                             qmriRole: view.qmriRole,
-                            onResult: (d, px, py) => {
-                                if (_pixelInfoActive()) _updateDimbarHover(px, py);
-                            },
                         });
                     });
+                    cv.addEventListener('mousedown', (e) => {
+                        if (e.button !== 0 || e.shiftKey || e.metaKey || e.ctrlKey || e.altKey
+                            || _cropActive || _segMode || _rulerMode) return;
+                        _pixelInfoHeld = true;
+                        cv.dispatchEvent(new MouseEvent('mousemove', { clientX: e.clientX, clientY: e.clientY, bubbles: true }));
+                    });
                     cv.addEventListener('mouseleave', () => {
                         if (_qmriLoupeDrag && _qmriLoupeDrag.canvas === cv) _endQmriLoupe();
                         compareQmriViews.forEach(v => { if (v.pixelInfoEl) v.pixelInfoEl.style.display = 'none'; });
-                        if (_pixelInfoActive()) _resetDimbarHover();
                     });
                     if (!httpTransport) {
                         view.ws = createTransport(sidVal,
@@ -27936,20 +27889,6 @@
                             imgH: view.lastH,
                             view,
                             displayEl: view.pixelInfoEl,
-                            onResult: (d, px, py) => {
-                                if (_pixelInfoActive()) {
-                                    const labels = ['x','y','z'];
-                                    const activeDims = compareMvDims || [];
-                                    const posX = activeDims.indexOf(view.dimX);
-                                    const posY = activeDims.indexOf(view.dimY);
-                                    const lblX = labels[posX] || 'x';
-                                    const lblY = labels[posY] || 'y';
-                                    const elX = document.getElementById('dim-' + lblX + '-size');
-                                    const elY = document.getElementById('dim-' + lblY + '-size');
-                                    if (elX) { elX.textContent = '/' + px; elX.className = 'dim-hover-idx'; }
-                                    if (elY) { elY.textContent = '/' + py; elY.className = 'dim-hover-idx'; }
-                                }
-                            },
                         });
                     });
                     cv.addEventListener('mouseleave', () => {
@@ -27957,7 +27896,6 @@
                         if (!cmvDraggingView) {
                             compareMvViews.forEach(v => { if (v.pixelInfoEl) v.pixelInfoEl.style.display = 'none'; });
                         }
-                        if (_pixelInfoActive()) _resetDimbarHover();
                     });
                     // Click-to-copy: compare-multiview canvases
                     let _cmvClickDown = null;
@@ -30116,7 +30054,6 @@
                     displayEl: piEl,
                     onResult: (d, px, py) => {
                         if (!_pixelInfoActive()) return;
-                        _updateDimbarHover(px, py);
                         // Show value on other compare panes at the same pixel
                         const ver = indices.join(',') + ':' + dim_x + ':' + dim_y + ':' + complexMode + ':' + logScale;
                         compareCanvases.forEach((ocv, j) => {
@@ -30145,7 +30082,6 @@
             cv.addEventListener('mouseleave', () => {
                 _comparePiEls.forEach(p => { if (p) p.style.display = 'none'; });
                 hideCompareCrosshairs();
-                if (_pixelInfoActive()) _resetDimbarHover();
             });
          });
         // Click-to-copy: compare canvases

{arrayview-0.29.0 → arrayview-0.29.2}/src/arrayview/_vscode_extension.py

@@ -44,7 +44,7 @@ def _vscode_app_bundle() -> str | None:
 
 _VSCODE_EXT_INSTALLED = False  # cached so we only check once per process
 _VSCODE_EXT_FRESH_INSTALL = False  # True if we just installed it this session
-_VSCODE_EXT_VERSION = "0.14.33"  # current bundled extension version
+_VSCODE_EXT_VERSION = "0.14.35"  # current bundled extension version
 
 def _bundled_vscode_vsix_version(vsix_path: str) -> str | None:
     """Return the bundled opener extension version recorded inside the VSIX."""