npm - xtralab - Versions diffs - 0.15.0 → 0.15.1 - Mend

xtralab 0.15.0 → 0.15.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (228) hide show

package/README.md +10 -0
package/lib/about/icons.d.ts +0 -5
package/lib/about/icons.js +0 -5
package/lib/about/index.d.ts +4 -8
package/lib/about/index.js +6 -17
package/lib/agentSessions.d.ts +9 -29
package/lib/agentSessions.js +2 -7
package/lib/askAgent/editorSelection.d.ts +20 -26
package/lib/askAgent/editorSelection.js +11 -26
package/lib/askAgent/icons.d.ts +2 -3
package/lib/askAgent/icons.js +2 -3
package/lib/askAgent/index.d.ts +4 -26
package/lib/askAgent/index.js +54 -158
package/lib/askAgent/popup.d.ts +12 -16
package/lib/askAgent/popup.js +18 -37
package/lib/askAgent/prompt.d.ts +3 -5
package/lib/askAgent/prompt.js +6 -19
package/lib/askAgent/queue.d.ts +11 -21
package/lib/askAgent/queue.js +9 -19
package/lib/askAgent/queuePanel.d.ts +13 -18
package/lib/askAgent/queuePanel.js +15 -37
package/lib/askAgent/targetPicker.d.ts +2 -9
package/lib/askAgent/targetPicker.js +6 -11
package/lib/askAgent/tokens.d.ts +32 -46
package/lib/askAgent/tokens.js +4 -6
package/lib/commandBar/index.d.ts +4 -13
package/lib/commandBar/index.js +21 -39
package/lib/customPanel/index.js +15 -0
package/lib/editorBreadcrumbs/index.d.ts +3 -6
package/lib/editorBreadcrumbs/index.js +10 -6
package/lib/editorBreadcrumbs/widget.d.ts +17 -4
package/lib/editorBreadcrumbs/widget.js +11 -17
package/lib/editorIndent/index.d.ts +4 -12
package/lib/editorIndent/index.js +8 -23
package/lib/fileBrowser/commands.d.ts +28 -19
package/lib/fileBrowser/commands.js +39 -98
package/lib/fileBrowser/contents.d.ts +3 -0
package/lib/fileBrowser/dragAndDrop.d.ts +17 -4
package/lib/fileBrowser/dragAndDrop.js +2 -7
package/lib/fileBrowser/fileBrowser.d.ts +19 -9
package/lib/fileBrowser/fileBrowser.js +68 -216
package/lib/fileBrowser/gitStatus.d.ts +4 -9
package/lib/fileBrowser/gitStatus.js +7 -17
package/lib/fileBrowser/gitignore.d.ts +4 -17
package/lib/fileBrowser/gitignore.js +4 -25
package/lib/fileBrowser/icons.d.ts +7 -17
package/lib/fileBrowser/icons.js +25 -80
package/lib/fileBrowser/index.js +2 -4
package/lib/fileBrowser/widget.d.ts +126 -48
package/lib/fileBrowser/widget.js +89 -4
package/lib/fileTypeIcons/index.js +0 -10
package/lib/git/api.d.ts +5 -13
package/lib/git/api.js +10 -36
package/lib/git/askRequest.d.ts +4 -8
package/lib/git/askRequest.js +6 -20
package/lib/git/commands.d.ts +40 -5
package/lib/git/commands.js +16 -10
package/lib/git/diffModel.js +4 -0
package/lib/git/diffSurface.d.ts +41 -55
package/lib/git/diffSurface.js +42 -117
package/lib/git/diffWidget.d.ts +81 -2
package/lib/git/diffWidget.js +62 -12
package/lib/git/imageDiff.d.ts +5 -6
package/lib/git/imageDiff.js +18 -48
package/lib/git/index.d.ts +0 -3
package/lib/git/index.js +3 -8
package/lib/git/notebookDiff.d.ts +110 -33
package/lib/git/notebookDiff.js +52 -195
package/lib/git/tokens.d.ts +73 -31
package/lib/git/tokens.js +2 -3
package/lib/highlight/index.d.ts +4 -10
package/lib/highlight/index.js +10 -41
package/lib/index.d.ts +0 -9
package/lib/index.js +0 -9
package/lib/launcher/agents.d.ts +55 -51
package/lib/launcher/agents.js +15 -57
package/lib/launcher/availability.d.ts +3 -9
package/lib/launcher/availability.js +3 -9
package/lib/launcher/commands.d.ts +7 -32
package/lib/launcher/commands.js +14 -45
package/lib/launcher/dashboard.d.ts +19 -24
package/lib/launcher/dashboard.js +21 -67
package/lib/launcher/editorRegistry.d.ts +10 -22
package/lib/launcher/editorRegistry.js +14 -18
package/lib/launcher/editors.d.ts +39 -45
package/lib/launcher/editors.js +8 -34
package/lib/launcher/icons.d.ts +0 -12
package/lib/launcher/icons.js +5 -39
package/lib/launcher/index.js +12 -55
package/lib/launcher/invocation.d.ts +4 -10
package/lib/launcher/invocation.js +7 -22
package/lib/launcher/registry.d.ts +9 -7
package/lib/launcher/registry.js +9 -7
package/lib/launcher/tokens.d.ts +10 -22
package/lib/launcher/tokens.js +5 -8
package/lib/menuBar/index.d.ts +3 -10
package/lib/menuBar/index.js +11 -28
package/lib/menus/index.d.ts +3 -9
package/lib/menus/index.js +14 -30
package/lib/omnibox/files.d.ts +2 -4
package/lib/omnibox/files.js +5 -11
package/lib/omnibox/index.d.ts +4 -14
package/lib/omnibox/index.js +9 -27
package/lib/omnibox/model.d.ts +38 -10
package/lib/omnibox/model.js +15 -33
package/lib/omnibox/recents.d.ts +15 -17
package/lib/omnibox/recents.js +12 -17
package/lib/omnibox/tokens.d.ts +5 -10
package/lib/omnibox/tokens.js +2 -6
package/lib/omnibox/widget.d.ts +11 -7
package/lib/omnibox/widget.js +8 -16
package/lib/searchReplace/index.d.ts +4 -20
package/lib/searchReplace/index.js +7 -27
package/lib/showOutput/index.d.ts +3 -12
package/lib/showOutput/index.js +6 -22
package/lib/sidebar/index.d.ts +4 -7
package/lib/sidebar/index.js +5 -12
package/lib/terminalNotifications/index.d.ts +4 -10
package/lib/terminalNotifications/index.js +6 -20
package/lib/terminals/agentTerminals.d.ts +12 -5
package/lib/terminals/agentTerminals.js +25 -44
package/lib/terminals/detection.d.ts +4 -8
package/lib/terminals/detection.js +4 -8
package/lib/terminals/index.d.ts +0 -35
package/lib/terminals/index.js +12 -101
package/lib/terminals/model.d.ts +64 -139
package/lib/terminals/model.js +80 -220
package/lib/terminals/tokens.d.ts +11 -22
package/lib/terminals/tokens.js +2 -2
package/lib/terminals/widget.d.ts +43 -13
package/lib/terminals/widget.js +33 -18
package/lib/topBar/icons.d.ts +3 -8
package/lib/topBar/icons.js +3 -8
package/lib/topBar/index.d.ts +4 -11
package/lib/topBar/index.js +10 -38
package/lib/walkthrough/index.d.ts +4 -9
package/lib/walkthrough/index.js +6 -20
package/lib/walkthrough/panel.d.ts +40 -9
package/lib/walkthrough/panel.js +4 -10
package/package.json +2 -2
package/src/about/icons.ts +0 -5
package/src/about/index.tsx +6 -17
package/src/agentSessions.ts +9 -29
package/src/askAgent/editorSelection.ts +22 -29
package/src/askAgent/icons.ts +2 -3
package/src/askAgent/index.ts +54 -158
package/src/askAgent/popup.tsx +23 -45
package/src/askAgent/prompt.ts +6 -19
package/src/askAgent/queue.ts +13 -24
package/src/askAgent/queuePanel.tsx +21 -46
package/src/askAgent/targetPicker.tsx +5 -15
package/src/askAgent/tokens.ts +32 -46
package/src/commandBar/index.ts +24 -39
package/src/customPanel/index.ts +15 -0
package/src/editorBreadcrumbs/index.ts +10 -6
package/src/editorBreadcrumbs/widget.ts +23 -17
package/src/editorIndent/index.ts +17 -23
package/src/fileBrowser/commands.ts +57 -98
package/src/fileBrowser/contents.ts +3 -0
package/src/fileBrowser/dragAndDrop.ts +17 -7
package/src/fileBrowser/fileBrowser.tsx +84 -223
package/src/fileBrowser/gitStatus.ts +7 -17
package/src/fileBrowser/gitignore.ts +4 -25
package/src/fileBrowser/icons.ts +25 -80
package/src/fileBrowser/index.ts +2 -4
package/src/fileBrowser/widget.tsx +149 -50
package/src/fileTypeIcons/index.ts +0 -10
package/src/git/api.ts +10 -36
package/src/git/askRequest.ts +6 -20
package/src/git/commands.ts +70 -10
package/src/git/diffModel.ts +25 -0
package/src/git/diffSurface.tsx +71 -159
package/src/git/diffWidget.tsx +104 -14
package/src/git/imageDiff.tsx +45 -51
package/src/git/index.ts +3 -8
package/src/git/notebookDiff.tsx +211 -238
package/src/git/tokens.ts +73 -31
package/src/highlight/index.ts +10 -41
package/src/index.ts +0 -9
package/src/launcher/agents.ts +64 -95
package/src/launcher/availability.ts +3 -9
package/src/launcher/commands.ts +14 -51
package/src/launcher/dashboard.tsx +49 -88
package/src/launcher/editorRegistry.ts +21 -29
package/src/launcher/editors.ts +39 -58
package/src/launcher/icons.ts +5 -39
package/src/launcher/index.ts +12 -55
package/src/launcher/invocation.ts +7 -22
package/src/launcher/registry.ts +9 -7
package/src/launcher/tokens.ts +10 -22
package/src/menuBar/index.ts +11 -28
package/src/menus/index.ts +14 -30
package/src/omnibox/files.ts +10 -14
package/src/omnibox/index.ts +9 -27
package/src/omnibox/model.ts +50 -38
package/src/omnibox/recents.ts +17 -22
package/src/omnibox/tokens.ts +5 -10
package/src/omnibox/widget.tsx +14 -19
package/src/searchReplace/index.ts +7 -27
package/src/showOutput/index.ts +6 -22
package/src/sidebar/index.ts +20 -12
package/src/terminalNotifications/index.ts +42 -25
package/src/terminals/agentTerminals.ts +25 -49
package/src/terminals/detection.ts +4 -8
package/src/terminals/index.ts +12 -101
package/src/terminals/model.ts +130 -246
package/src/terminals/tokens.ts +11 -22
package/src/terminals/widget.tsx +48 -31
package/src/topBar/icons.ts +3 -8
package/src/topBar/index.ts +21 -44
package/src/walkthrough/index.ts +6 -20
package/src/walkthrough/panel.ts +40 -10
package/style/about.css +2 -7
package/style/askAgent.css +8 -40
package/style/base.css +9 -45
package/style/chrome.css +108 -308
package/style/commandBar.css +4 -20
package/style/customPanel.css +1 -4
package/style/git.css +22 -184
package/style/highlight.css +1 -3
package/style/launcher.css +6 -70
package/style/omnibox.css +2 -14
package/style/showOutput.css +0 -6
package/style/sidebar.css +29 -9
package/style/tabs.css +5 -13
package/style/terminals.css +6 -53
package/style/topBar.css +2 -22
package/style/walkthrough.css +0 -8

package/lib/git/diffSurface.d.ts CHANGED Viewed

@@ -3,33 +3,28 @@ import { IThemeManager } from '@jupyterlab/apputils';
 import { IRenderMimeRegistry } from '@jupyterlab/rendermime';
 import type { TranslationBundle } from '@jupyterlab/translation';
 import { type SelectedLineRange } from '@pierre/diffs';
-/**
- * The CSS class added to the diff main-area widget. The selectors that
- * style the embedded `@pierre/diffs` viewer hang off this class.
- */
 export declare const DIFF_WIDGET_CSS_CLASS = "jp-xtralab-DiffWidget";
+export type DiffStyle = 'split' | 'unified';
 /**
- * Layout for textual/code diffs, forwarded to `@pierre/diffs` as its
- * `diffStyle` option. `split` is the side-by-side old|new view; `unified`
- * is the single-column inline view (what diffs.com calls "stacked").
+ * Read the persisted diff style, defaulting to `'split'`.
  */
-export type DiffStyle = 'split' | 'unified';
 export declare function readStoredDiffStyle(): DiffStyle;
-export declare function writeStoredDiffStyle(style: DiffStyle): void;
 /**
- * View modes available for `.ipynb` diffs. `notebook` is the cell-by-cell
- * rendered view; `json` is the raw nbformat JSON file diff, for inspecting
- * exactly which bytes changed.
+ * Persist the diff style to local storage (best-effort).
  */
+export declare function writeStoredDiffStyle(style: DiffStyle): void;
 export type NotebookDiffViewMode = 'notebook' | 'json';
+/**
+ * Read the persisted notebook view mode, defaulting to `'notebook'`.
+ */
 export declare function readStoredNotebookViewMode(): NotebookDiffViewMode;
+/**
+ * Persist the notebook view mode to local storage (best-effort).
+ */
 export declare function writeStoredNotebookViewMode(mode: NotebookDiffViewMode): void;
 /**
- * Decide whether the active JupyterLab theme is a dark theme. Prefers the
- * `IThemeManager` token when available, and falls back to sniffing the
- * `data-jp-theme-light` attribute that JupyterLab sets on `<body>` so the
- * component still works in stripped-down hosts where the token isn't
- * provided (lite, certain notebook configurations, …).
+ * Dark-theme check via `IThemeManager`, falling back to the
+ * `data-jp-theme-light` body attribute for hosts without the token.
  */
 export declare function isDarkTheme(themeManager: IThemeManager | null): boolean;
 /**
@@ -37,18 +32,26 @@ export declare function isDarkTheme(themeManager: IThemeManager | null): boolean
  */
 export declare function isPierreTheme(themeManager: IThemeManager | null): boolean;
 /**
- * Optional per-hunk discard wiring. When supplied and `enabled`, each hunk
- * in a plain-text/code diff gets an inline "discard" button; clicking it
- * reverts that hunk and hands the rebuilt full-file text back to the host
- * via {@link IHunkDiscard.save}. The host owns the actual write (it knows
- * the on-disk path) and triggers any follow-up refresh in
- * {@link IHunkDiscard.onAfterSave}.
+ * Per-hunk discard wiring: discarding rebuilds the full file text and the
+ * host owns the write via `save`, then refreshes in `onAfterSave`.
  */
 interface IHunkDiscard {
+    /**
+     * Whether hunk discarding is available.
+     */
     enabled: boolean;
+    /**
+     * Persist the full post-discard file text.
+     */
     save: (fullText: string) => Promise<void>;
+    /**
+     * Called after a successful save so the host can refresh the diff.
+     */
     onAfterSave: () => void;
 }
+/**
+ * The props for the {@link DiffSurface} component.
+ */
 interface IDiffSurfaceProps {
     /**
      * Whether the host is still resolving the file contents.
@@ -71,13 +74,11 @@ interface IDiffSurfaceProps {
      */
     newText: string;
     /**
-     * New-side path. Drives notebook detection and the `@pierre/diffs` file
-     * name on the additions side.
+     * New-side path; drives notebook detection and the highlighting filename.
      */
     newName: string;
     /**
-     * Old-side path. Differs from {@link newName} only for renames; defaults
-     * to {@link newName} when the host does not track a previous path.
+     * Old-side path; differs from {@link newName} only for renames.
      */
     oldName?: string;
     /**
@@ -89,35 +90,28 @@ interface IDiffSurfaceProps {
      */
     pierreTheme: boolean;
     /**
-     * Rendermime registry used by the notebook view to render outputs and
-     * markdown cells. May be `null` in stripped-down hosts; the notebook
-     * diff falls back to a textual representation in that case.
+     * Used by the notebook view; `null` in stripped-down hosts, which forces a textual fallback.
      */
     rendermime: IRenderMimeRegistry | null;
     /**
-     * Current rendered-vs-JSON choice for notebook diffs (host-controlled).
+     * Rendered-vs-JSON choice for notebook diffs (host-controlled).
      */
     notebookViewMode: NotebookDiffViewMode;
     /**
-     * Split vs unified layout for the textual/code file diff (host-controlled).
+     * Split vs unified layout for the textual file diff (host-controlled).
      */
     diffStyle: DiffStyle;
     /**
-     * Called whenever the availability of a rendered notebook view changes,
-     * so the host can show/hide its Notebook/JSON toolbar toggle.
+     * Fires when rendered-notebook availability changes so the host can toggle its Notebook/JSON control.
      */
     onNotebookAvailabilityChange?: (available: boolean) => void;
     /**
-     * Called whenever the textual/code file diff (the only view the split vs
-     * unified choice affects) becomes the active view, so the host can
-     * show/hide its Split/Unified toolbar toggle. False for image, binary,
-     * rendered-notebook, loading and error states.
+     * Fires when the textual file diff becomes (in)active — the only view the split/unified choice affects.
      */
     onFileDiffActiveChange?: (active: boolean) => void;
     /**
-     * Called after each (re)computation of the diff with the number of hunks
-     * (or `null` when no textual diff exists). Hosts use this to implement
-     * post-discard behaviors such as auto-closing an emptied diff.
+     * Fires after each diff computation with the hunk count (`null` when no
+     * textual diff); hosts use it to auto-close emptied diffs.
      */
     onMetadataChange?: (info: {
         hunkCount: number | null;
@@ -127,11 +121,8 @@ interface IDiffSurfaceProps {
      */
     hunkDiscard?: IHunkDiscard;
     /**
-     * When provided, the textual/code file diff gets line selection: clicking
-     * or dragging over the line numbers selects a range, and a gutter "+"
-     * button appears on the hovered/selected lines. Clicking that button
-     * invokes this callback with the selected range and the button's viewport
-     * rectangle (to anchor a popup to), `null` when it cannot be measured.
+     * Enables line selection and the gutter "+" button; called with the
+     * selected range and the button's viewport rect (`null` when unmeasurable).
      */
     onLineAsk?: (range: SelectedLineRange, anchor: DOMRect | null) => void;
     /**
@@ -144,10 +135,8 @@ interface IDiffSurfaceProps {
  */
 export declare function DiffSurface(props: IDiffSurfaceProps): React.ReactElement;
 /**
- * Segmented Notebook/JSON selector. Hosts mount this into whatever toolbar
- * they own (the launcher's `MainAreaWidget` toolbar, or the
- * `jupyterlab-git`-provided diff toolbar) and drive its value/visibility
- * from the same state they pass to {@link DiffSurface}.
+ * Segmented Notebook/JSON selector. Hosts mount it into their own toolbar
+ * and drive value/visibility from the state they pass to {@link DiffSurface}.
  */
 export declare function NotebookViewModeControl(props: {
     mode: NotebookDiffViewMode;
@@ -156,11 +145,8 @@ export declare function NotebookViewModeControl(props: {
     trans: TranslationBundle;
 }): React.ReactElement;
 /**
- * Segmented Split/Unified selector. Mirrors {@link NotebookViewModeControl}:
- * hosts mount it into their own toolbar and drive its value/visibility from
- * the same state they pass to {@link DiffSurface}. Only meaningful while the
- * textual/code file diff is the active view, so `available` mirrors the
- * surface's `onFileDiffActiveChange`.
+ * Segmented Split/Unified selector; mirrors {@link NotebookViewModeControl}.
+ * `available` mirrors the surface's `onFileDiffActiveChange`.
  */
 export declare function DiffStyleControl(props: {
     diffStyle: DiffStyle;

package/lib/git/diffSurface.js CHANGED Viewed

@@ -5,35 +5,16 @@ import { diffAcceptRejectHunk, parseDiffFromFile } from '@pierre/diffs';
 import { ImageDiffView, imageDataType } from './imageDiff';
 import { buildNotebookDiff, NotebookDiffView } from './notebookDiff';
 import { resolveDiffTheme } from './diffTheme';
-/**
- * The CSS class added to the diff main-area widget. The selectors that
- * style the embedded `@pierre/diffs` viewer hang off this class.
- */
 export const DIFF_WIDGET_CSS_CLASS = 'jp-xtralab-DiffWidget';
-/**
- * `localStorage` key for the user's last-chosen split ratio, persisted so
- * the preference survives reloads (the library has no built-in way to
- * remember it).
- */
 const SPLIT_RATIO_STORAGE_KEY = 'xtralab:diff-split-ratio';
-/**
- * Bounds on the left-pane fraction. Keeping a few percent on either side
- * guarantees both panes remain visible — dragging to 0/100 would hide a
- * column and is almost never what the user wants.
- */
 const MIN_SPLIT_RATIO = 0.1;
 const MAX_SPLIT_RATIO = 0.9;
 const DEFAULT_SPLIT_RATIO = 0.5;
 /**
- * CSS injected via the `@pierre/diffs` `unsafeCSS` option. The library
- * places it inside the shadow root under `@layer unsafe`, which beats
- * the library's own `@layer base` rule that hardcodes
- * `grid-template-columns: 1fr 1fr` on the split-mode container.
- *
- * The override reads from `--xtralab-split-cols`, a custom property we set
- * on the host element via the `style` prop. Custom properties inherit
- * through the shadow DOM boundary, so adjusting it on the host instantly
- * resizes the columns without re-rendering the diff.
+ * Injected via the `unsafeCSS` option into the shadow root's `@layer
+ * unsafe`, beating the library's base rule that hardcodes `1fr 1fr`. Reads
+ * `--xtralab-split-cols` set on the host — custom properties cross the
+ * shadow boundary, so resizing needs no re-render.
  */
 const SPLIT_RESIZE_CSS = `pre[data-diff-type="split"][data-overflow="scroll"] {
   grid-template-columns: var(--xtralab-split-cols, 1fr 1fr);
@@ -52,8 +33,7 @@ function readStoredSplitRatio() {
         }
     }
     catch (_a) {
-        // localStorage may throw in privacy mode or sandboxed contexts. Falling
-        // back to the default is fine — the user can resize again on this run.
+        // localStorage can throw in privacy mode or sandboxed contexts.
     }
     return DEFAULT_SPLIT_RATIO;
 }
@@ -66,6 +46,9 @@ function writeStoredSplitRatio(ratio) {
     }
 }
 const DIFF_STYLE_STORAGE_KEY = 'xtralab:diff-style';
+/**
+ * Read the persisted diff style, defaulting to `'split'`.
+ */
 export function readStoredDiffStyle() {
     try {
         const raw = window.localStorage.getItem(DIFF_STYLE_STORAGE_KEY);
@@ -74,10 +57,13 @@ export function readStoredDiffStyle() {
         }
     }
     catch (_a) {
-        // See readStoredSplitRatio — privacy-mode / sandboxed contexts can throw.
+        // See readStoredSplitRatio.
     }
     return 'split';
 }
+/**
+ * Persist the diff style to local storage (best-effort).
+ */
 export function writeStoredDiffStyle(style) {
     try {
         window.localStorage.setItem(DIFF_STYLE_STORAGE_KEY, style);
@@ -87,6 +73,9 @@ export function writeStoredDiffStyle(style) {
     }
 }
 const NOTEBOOK_DIFF_VIEW_MODE_STORAGE_KEY = 'xtralab:notebook-diff-view-mode';
+/**
+ * Read the persisted notebook view mode, defaulting to `'notebook'`.
+ */
 export function readStoredNotebookViewMode() {
     try {
         const raw = window.localStorage.getItem(NOTEBOOK_DIFF_VIEW_MODE_STORAGE_KEY);
@@ -95,11 +84,13 @@ export function readStoredNotebookViewMode() {
         }
     }
     catch (_a) {
-        // See readStoredSplitRatio — privacy-mode / sandboxed contexts can
-        // throw on access. The notebook view is the better default anyway.
+        // See readStoredSplitRatio.
     }
     return 'notebook';
 }
+/**
+ * Persist the notebook view mode to local storage (best-effort).
+ */
 export function writeStoredNotebookViewMode(mode) {
     try {
         window.localStorage.setItem(NOTEBOOK_DIFF_VIEW_MODE_STORAGE_KEY, mode);
@@ -109,21 +100,15 @@ export function writeStoredNotebookViewMode(mode) {
     }
 }
 /**
- * Detect notebook files by extension. We only look at the suffix so the
- * caller doesn't need to plumb a content-type field through; mistaking
- * another `.ipynb`-named file for a notebook is harmless because
- * {@link buildNotebookDiff} validates the JSON shape and we fall back to
- * the file diff if parsing fails.
+ * Extension-only check; a false positive is harmless because
+ * {@link buildNotebookDiff} validates the JSON and the file diff is the fallback.
  */
 function isNotebookPath(path) {
     return path.toLowerCase().endsWith('.ipynb');
 }
 /**
- * Decide whether the active JupyterLab theme is a dark theme. Prefers the
- * `IThemeManager` token when available, and falls back to sniffing the
- * `data-jp-theme-light` attribute that JupyterLab sets on `<body>` so the
- * component still works in stripped-down hosts where the token isn't
- * provided (lite, certain notebook configurations, …).
+ * Dark-theme check via `IThemeManager`, falling back to the
+ * `data-jp-theme-light` body attribute for hosts without the token.
  */
 export function isDarkTheme(themeManager) {
     if (themeManager !== null && themeManager.theme !== null) {
@@ -145,14 +130,10 @@ export function isPierreTheme(themeManager) {
 export function DiffSurface(props) {
     const { loading, error, isBinary, oldText, newText, newName, oldName, dark, pierreTheme, rendermime, notebookViewMode, diffStyle, onNotebookAvailabilityChange, onFileDiffActiveChange, onMetadataChange, hunkDiscard, onLineAsk, trans } = props;
     const hasContent = !loading && error === null && !isBinary;
-    // Raster images take a dedicated `<img>`-based view instead of a
-    // text/notebook diff; the host passes the two sides as base64 strings in
-    // `oldText`/`newText` (the git server base64-encodes binary content).
+    // For raster images the host passes both sides as base64 in oldText/newText.
     const imageType = React.useMemo(() => imageDataType(newName), [newName]);
-    // Pre-compute the line-oriented diff metadata. Owning it here lets us
-    // pass it to {@link FileDiff} *and* thread the same instance through
-    // {@link diffAcceptRejectHunk} when the user discards a hunk — so hunk
-    // indexes line up between what the user clicked and what we mutate.
+    // Owning the metadata here keeps hunk indexes aligned between FileDiff
+    // and diffAcceptRejectHunk on discard.
     const metadata = React.useMemo(() => {
         if (!hasContent || imageType !== null) {
             return null;
@@ -164,47 +145,31 @@ export function DiffSurface(props) {
         const newFile = { name: newName, contents: newText };
         return parseDiffFromFile(oldFile, newFile);
     }, [hasContent, imageType, oldText, newText, oldName, newName]);
-    // Cell-by-cell notebook diff for `.ipynb` files. When non-null the
-    // renderer uses a notebook-aware view instead of the line-oriented file
-    // diff — kept distinct from {@link metadata} so a notebook whose JSON we
-    // cannot parse can fall back to the file diff path.
     const notebookDiff = React.useMemo(() => {
         if (!hasContent || imageType !== null || !isNotebookPath(newName)) {
             return null;
         }
         return buildNotebookDiff({ oldText, newText });
     }, [hasContent, imageType, oldText, newText, newName]);
-    // Tell the host whether a rendered notebook view is currently available
-    // so it can show/hide its Notebook/JSON toggle.
     React.useEffect(() => {
         onNotebookAvailabilityChange === null || onNotebookAvailabilityChange === void 0 ? void 0 : onNotebookAvailabilityChange(notebookDiff !== null);
     }, [onNotebookAvailabilityChange, notebookDiff]);
-    // Report the diff shape so hosts can react (e.g. auto-close on empty).
     React.useEffect(() => {
         onMetadataChange === null || onMetadataChange === void 0 ? void 0 : onMetadataChange({
             hunkCount: metadata !== null ? metadata.hunks.length : null
         });
     }, [onMetadataChange, metadata]);
-    // Decide which view to render. The notebook view requires a successful
-    // nbformat parse; if that failed, we only show the raw JSON file diff.
-    // Computed before the early returns below so the file-diff-active effect
-    // can depend on it without violating the rules of hooks.
+    // Computed before the early returns so the hooks below can depend on
+    // them without breaking the rules of hooks.
     const hasNotebookView = notebookDiff !== null;
     const showNotebookView = hasNotebookView && notebookViewMode === 'notebook';
     const showFileDiff = !showNotebookView && metadata !== null;
-    // The split vs unified choice only affects the textual/code file diff, so
-    // tell the host whether that view is active to drive its Split/Unified
-    // toolbar toggle.
     React.useEffect(() => {
         onFileDiffActiveChange === null || onFileDiffActiveChange === void 0 ? void 0 : onFileDiffActiveChange(showFileDiff);
     }, [onFileDiffActiveChange, showFileDiff]);
-    // Split ratio for the diff columns: fraction of width given to the
-    // deletions (left) pane. Persisted across sessions so the user only
-    // dials in their layout once.
     const [leftRatio, setLeftRatio] = React.useState(() => readStoredSplitRatio());
     const wrapperRef = React.useRef(null);
-    // The drag handler reads the live ratio from a ref so the listeners we
-    // attach on pointer-down don't capture a stale value across renders.
+    // Drag listeners read the live ratio from a ref to avoid stale captures.
     const leftRatioRef = React.useRef(leftRatio);
     React.useEffect(() => {
         leftRatioRef.current = leftRatio;
@@ -216,8 +181,6 @@ export function DiffSurface(props) {
         }
         return metadata.hunks.map((hunk, hunkIndex) => ({
             side: 'additions',
-            // The first line of each hunk in the new file. We anchor the button
-            // there so it sits at the top of the hunk's body.
             lineNumber: hunk.additionStart,
             metadata: { hunkIndex }
         }));
@@ -227,12 +190,8 @@ export function DiffSurface(props) {
             return;
         }
         const updated = diffAcceptRejectHunk(metadata, hunkIndex, 'reject');
-        // `additionLines` after a 'reject' contains the full new file with
-        // the hunk reverted back to the old content. The library splits
-        // file contents with `/(?<=\n)/` (a lookbehind that keeps the
-        // trailing `\n` on each line), so the entries already carry their
-        // own line endings — joining with `''` rebuilds the original text
-        // verbatim, while joining with `'\n'` would double every newline.
+        // additionLines holds the full new file with the hunk reverted; lines keep
+        // their trailing `\n` (lookbehind split), so join('') rebuilds it verbatim.
         const text = updated.additionLines.join('');
         try {
             await hunkDiscard.save(text);
@@ -256,15 +215,11 @@ export function DiffSurface(props) {
         if (onLineAsk === undefined) {
             return;
         }
-        // Anchor to the gutter "+" button that was just clicked. It lives in
-        // the `@pierre/diffs` shadow root (which is open), parked on the
-        // hovered/selected line's number element.
+        // Anchor to the just-clicked gutter "+" button inside the library's open shadow root.
         const slot = (_c = (_b = (_a = wrapperRef.current) === null || _a === void 0 ? void 0 : _a.querySelector('diffs-container')) === null || _b === void 0 ? void 0 : _b.shadowRoot) === null || _c === void 0 ? void 0 : _c.querySelector('[data-gutter-utility-slot]');
         onLineAsk(range, slot instanceof Element ? slot.getBoundingClientRect() : null);
     }, [onLineAsk]);
     const handleResizerPointerDown = React.useCallback((event) => {
-        // Only react to primary-button drags. Right-click / middle-click on
-        // the handle should pass through to the browser default.
         if (event.button !== 0) {
             return;
         }
@@ -293,8 +248,6 @@ export function DiffSurface(props) {
             handle.removeEventListener('pointermove', onPointerMove);
             handle.removeEventListener('pointerup', onPointerEnd);
             handle.removeEventListener('pointercancel', onPointerEnd);
-            // Only persist on drag-end so a quick drag isn't followed by a
-            // burst of writes for every intermediate position.
             writeStoredSplitRatio(leftRatioRef.current);
         };
         handle.addEventListener('pointermove', onPointerMove);
@@ -340,19 +293,12 @@ export function DiffSurface(props) {
         return (React.createElement("div", { className: "jp-xtralab-DiffWidget-status" }, trans.__('Binary file — diff not supported.')));
     }
     if (imageType !== null) {
-        // `oldText`/`newText` carry the two revisions as base64 here. The
-        // image view owns its own layout and mode selector, so it does not
-        // use the split resizer or the notebook/JSON toggle.
         return (React.createElement("div", { className: "jp-xtralab-DiffWidget-content" },
             React.createElement(ImageDiffView, { reference: oldText, challenger: newText, fileType: imageType, trans: trans })));
     }
     if (!showNotebookView && !showFileDiff) {
         return (React.createElement("div", { className: "jp-xtralab-DiffWidget-status" }, trans.__('No content to diff.')));
     }
-    // Custom property set on the diffs-container host. The library's
-    // shadow-root rule consumes it via `var(--xtralab-split-cols, …)` (see
-    // SPLIT_RESIZE_CSS) and recomputes the column tracks instantly. Only
-    // relevant in the file-diff path; the notebook view doesn't use it.
     const leftPercent = leftRatio * 100;
     const hostStyle = {
         '--xtralab-split-cols': `${leftPercent}% ${100 - leftPercent}%`
@@ -360,29 +306,15 @@ export function DiffSurface(props) {
     return (React.createElement("div", { className: "jp-xtralab-DiffWidget-content" },
         React.createElement("div", { ref: wrapperRef, className: "jp-xtralab-DiffWidget-body" },
             React.createElement("div", { className: "jp-xtralab-DiffWidget-scroll" }, showNotebookView && notebookDiff !== null ? (React.createElement(NotebookDiffView, { diff: notebookDiff, dark: dark, pierreTheme: pierreTheme, rendermime: rendermime, trans: trans })) : showFileDiff && metadata !== null ? (React.createElement(FileDiff, { fileDiff: metadata, lineAnnotations: lineAnnotations, renderAnnotation: renderAnnotation, style: hostStyle,
-                // Disable the worker pool: JupyterLab's webpack federation
-                // pipeline doesn't ship the library's
-                // `@pierre/diffs/worker/worker.js` file at a URL the worker
-                // bootstrap can resolve from, so leaving the pool enabled
-                // crashes on instantiation. Running on the main thread is
-                // fine for the small files git diffs typically operate on.
+                // JupyterLab's federation pipeline can't serve the worker bundle
+                // at a resolvable URL, so the pool crashes; run on the main thread.
                 disableWorkerPool: true, options: {
                     diffStyle,
-                    // Drop the file header — the tab title already shows the
-                    // file name and the panel header carries the change
-                    // context.
                     disableFileHeader: true,
                     theme: resolveDiffTheme(dark, pierreTheme),
                     themeType: dark ? 'dark' : 'light',
-                    // Inject the column-resize override into the shadow root
-                    // via the library's `@layer unsafe` channel. Keeping the
-                    // string constant lets the library short-circuit the
-                    // re-render path it uses when this option actually
-                    // changes.
+                    // Constant string lets the library skip its unsafeCSS re-render path.
                     unsafeCSS: SPLIT_RESIZE_CSS,
-                    // Line selection + the gutter "+" button feed the
-                    // ask-agent popup; only wired up when a handler exists so
-                    // a plain diff keeps its passive gutter.
                     ...(onLineAsk !== undefined
                         ? {
                             enableLineSelection: true,
@@ -394,10 +326,8 @@ export function DiffSurface(props) {
             showFileDiff && diffStyle === 'split' ? (React.createElement("div", { className: "jp-xtralab-DiffWidget-resizer", style: { left: `${leftPercent}%` }, role: "separator", "aria-orientation": "vertical", "aria-valuenow": Math.round(leftPercent), "aria-valuemin": Math.round(MIN_SPLIT_RATIO * 100), "aria-valuemax": Math.round(MAX_SPLIT_RATIO * 100), "aria-label": trans.__('Resize the diff panes'), tabIndex: 0, title: trans.__('Drag to resize the diff panes (double-click to reset)'), onPointerDown: handleResizerPointerDown, onDoubleClick: handleResizerDoubleClick, onKeyDown: handleResizerKeyDown })) : null)));
 }
 /**
- * Segmented Notebook/JSON selector. Hosts mount this into whatever toolbar
- * they own (the launcher's `MainAreaWidget` toolbar, or the
- * `jupyterlab-git`-provided diff toolbar) and drive its value/visibility
- * from the same state they pass to {@link DiffSurface}.
+ * Segmented Notebook/JSON selector. Hosts mount it into their own toolbar
+ * and drive value/visibility from the state they pass to {@link DiffSurface}.
  */
 export function NotebookViewModeControl(props) {
     const { mode, available, onChange, trans } = props;
@@ -409,11 +339,9 @@ export function NotebookViewModeControl(props) {
         React.createElement("button", { type: "button", role: "tab", "aria-selected": mode === 'json', "data-active": mode === 'json', className: "jp-xtralab-DiffWidget-segmentedButton", onClick: () => onChange('json') }, trans.__('JSON'))));
 }
 /**
- * Split / unified glyphs, inlined from the `@pierre/diffs` icon sprite
- * (`diffs-icon-diff-split` / `diffs-icon-diff-unified`) so the toggle looks
- * like the one on the library's own docs site without depending on the
- * sprite sheet being injected into the document. `currentColor` lets the
- * segmented-button styling drive the fill (including the active state).
+ * Split / unified glyphs inlined from the `@pierre/diffs` icon sprite so
+ * the toggle matches the library's docs site without the sprite sheet;
+ * `currentColor` lets the button styling drive the fill.
  */
 function DiffSplitIcon() {
     return (React.createElement("svg", { viewBox: "0 0 16 16", fill: "currentColor", "aria-hidden": "true" },
@@ -426,11 +354,8 @@ function DiffUnifiedIcon() {
         React.createElement("path", { fillRule: "evenodd", d: "M14 0a2 2 0 0 1 2 2v5.5H0V2a2 2 0 0 1 2-2zM6.5 3.5a.5.5 0 0 0 0 1h3a.5.5 0 0 0 0-1z", clipRule: "evenodd", opacity: ".4" })));
 }
 /**
- * Segmented Split/Unified selector. Mirrors {@link NotebookViewModeControl}:
- * hosts mount it into their own toolbar and drive its value/visibility from
- * the same state they pass to {@link DiffSurface}. Only meaningful while the
- * textual/code file diff is the active view, so `available` mirrors the
- * surface's `onFileDiffActiveChange`.
+ * Segmented Split/Unified selector; mirrors {@link NotebookViewModeControl}.
+ * `available` mirrors the surface's `onFileDiffActiveChange`.
  */
 export function DiffStyleControl(props) {
     const { diffStyle, available, onChange, trans } = props;