@celestia-island/hikari 0.55.93 → 0.55.95

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celestia-island/hikari",
-  "version": "0.55.93",
+  "version": "0.55.95",
   "private": false,
   "type": "module",
   "description": "Hikari Vue 3 component library — production-grade UI components based on shittim-chest design system",

package/src/components/HkJsonTree.contract.test.ts

@@ -0,0 +1,59 @@
+import { describe, expect, it } from "vitest";
+import { readFileSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
+
+/**
+ * Source contract: the JSON tree's spacing rhythm is token-based, not
+ * ad-hoc. The 2026-09-30 normalization replaced the off-scale gaps
+ * (2px toggle margin, 0.35em colon, 0.75em type hint) with design
+ * units and added the toggle's first-line centering formula — this
+ * contract keeps the file from drifting back to magic values, which is
+ * exactly the "crooked collapsed/expanded alignment" regression class
+ * the normalization fixed.
+ */
+const here = dirname(fileURLToPath(import.meta.url));
+const scss = readFileSync(join(here, "HkJsonTree.scss"), "utf-8");
+
+describe("HkJsonTree spacing contract", () => {
+  it("keeps the toggle exactly one spacing unit from the row text", () => {
+    const block = scss.slice(scss.indexOf(".s-jt-toggle {"), scss.indexOf(".s-jt-toggle[data-parent]"));
+    expect(block).toContain("margin-inline-end: var(--space-4");
+  });
+
+  it("centers the toggle box on the first text line by formula", () => {
+    const block = scss.slice(scss.indexOf(".s-jt-toggle {"), scss.indexOf(".s-jt-toggle[data-parent]"));
+    expect(block).toContain("align-self: flex-start");
+    expect(block).toContain("margin-block-start: max(0px, calc((1.55em - var(--jt-toggle, 14px)) / 2))");
+  });
+
+  it("keeps every --jt-toggle fallback at the 14px the formula assumes", () => {
+    // Regex sweep, not a blacklist: ANY fallback other than 14px (today
+    // or added later) would de-center the box by the delta.
+    const fallbacks = [...scss.matchAll(/var\(--jt-toggle,\s*([^)]+)\)/g)].map((m) => m[1].trim());
+    expect(fallbacks.length, "fallback sites were scanned").toBeGreaterThan(2);
+    expect(fallbacks.every((f) => f === "14px"), `fallbacks = ${fallbacks}`).toBe(true);
+  });
+
+  it("pins the depth step to exactly one toggle width", () => {
+    const tsx = readFileSync(join(here, "HkJsonTree.tsx"), "utf-8");
+    expect(tsx).toContain("const JT_INDENT = JT_TOGGLE;");
+  });
+
+  it("keeps the key colon gap on one spacing unit", () => {
+    const block = scss.slice(scss.indexOf(".s-jt-colon {"), scss.indexOf(".s-jv-null"));
+    expect(block).toContain("margin-inline-end: var(--space-4");
+    expect(block).not.toContain("0.35em");
+  });
+
+  it("keeps the type hint gap on two spacing units", () => {
+    const block = scss.slice(scss.indexOf(".s-jt-type {"), scss.indexOf(".s-jt-badge {"));
+    expect(block).toContain("margin-inline-start: var(--space-8");
+    expect(block).not.toContain("0.75em");
+  });
+
+  it("leaves no off-scale magic gaps anywhere in the tree stylesheet", () => {
+    expect(scss).not.toMatch(/margin-inline-end: 0\.\d+em/);
+    expect(scss).not.toMatch(/margin-inline-start: 0\.\d+em/);
+  });
+});

package/src/components/HkJsonTree.scss

@@ -69,14 +69,20 @@
   display: inline-flex;
   align-items: center;
   justify-content: center;
-  width: var(--jt-toggle, 1rem);
-  height: var(--jt-toggle, 1rem);
+  width: var(--jt-toggle, 14px);
+  height: var(--jt-toggle, 14px);
   flex-shrink: 0;
-  margin-inline-end: var(--space-2, 0.125rem);
+  /* Toggle → text is ONE spacing unit everywhere (leaf, collapsed and
+     expanded parents alike) — the collapsed row and its children read
+     on the same rhythm instead of ad-hoc gaps. */
+  margin-inline-end: var(--space-4, 0.25rem);
   /* The toggle box has no text baseline of its own — pin it to the line
-     top instead of letting it join the row's baseline group with a
-     synthesized border-box baseline. */
+     top and center it on the FIRST text line by formula (line-height
+     1.55 × font-size vs the fixed box), so the chevron/leaf sits on the
+     text's optical center in collapsed and expanded rows alike instead
+     of floating above the baseline group. */
   align-self: flex-start;
+  margin-block-start: max(0px, calc((1.55em - var(--jt-toggle, 14px)) / 2));
 }
 
 .s-jt-toggle[data-parent] {
@@ -117,7 +123,7 @@
 
 .s-jt-colon {
   color: rgb(var(--color-muted));
-  margin-inline-end: 0.35em;
+  margin-inline-end: var(--space-4, 0.25rem);
 }
 
 .s-jv-null {
@@ -152,7 +158,7 @@
   font-size: var(--text-3xs, 0.5rem);
   font-style: italic;
   font-weight: 400;
-  margin-inline-start: 0.75em;
+  margin-inline-start: var(--space-8, 0.5rem);
 }
 
 .s-jt-badge {

package/src/components/HkJsonTree.tsx

@@ -130,14 +130,24 @@ function initialExpandedSet(root: HJsonNode): Set<number> {
 /**
  * HkJsonTree — the interactive expandable JSON tree.
  *
- * This is the exact renderer the chat card flow uses for tool results
- * (extracted verbatim from HkToolBlock, same DOM and `s-jt-*` stylesheet, so
- * the two can never drift). Feed it either pre-parsed `value` or a raw JSON
- * `text`; non-container input renders nothing.
+ * A first-class family component (same tier as the rich-text surfaces):
+ * the chat tool blocks, the chest error landing's raw-details pane and
+ * any host pane render through THIS component — HkToolBlock consumes it
+ * directly, and the `s-jt-*` stylesheet single-sources the visual
+ * language, so the two can never drift. Feed it either pre-parsed
+ * `value` or a raw JSON `text`; non-container input renders nothing.
  *
  * The root keeps the legacy `s-tool-json-tree` class: hosts (including
  * HkToolBlock's deferred scrollbar pass) target it, and the pane chrome
  * (mono font, capped height, hidden native scrollbars) lives there.
+ *
+ * Spacing contract (pinned by HkJsonTree.contract.test.ts): the toggle
+ * box is 14px and sits ONE `--space-4` from the row text — identical for
+ * leaf dots, collapsed previews and expanded parents — and is centered
+ * on the first text line by formula, so collapsed and expanded states
+ * align with the same left rhythm; `key:` gaps are one unit, type hints
+ * two; the depth step is one toggle width (JT_INDENT = JT_TOGGLE,
+ * pinned alongside the stylesheet contract).
  */
 export const HkJsonTree = defineComponent({
   name: "HkJsonTree",

package/src/components/HkNodeCanvas.test.tsx

@@ -2,6 +2,7 @@ import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { createApp, defineComponent, h, isRef, nextTick, onUpdated, ref, type App } from "vue";
 
 import HkNodeCanvas, { NODE_CANVAS_DEFAULTS } from "./HkNodeCanvas";
+import { CANVAS_BOUNDS_MARGIN, clampCameraToBounds } from "./nodeCanvasTypes";
 
 /**
  * HkNodeCanvas tests — the camera protocol the three node surfaces share:
@@ -1050,4 +1051,142 @@ describe("HkNodeCanvas", () => {
     // Controlled: the internal camera did not move on its own.
     expect(instance.value!.camera).toEqual({ k: 1, x: 0, y: 0 });
   });
+
+  // ── Bounds clamp ─────────────────────────────────────────────────
+  // The camera may never push the content hull farther than half a
+  // viewport past the visible edge, per axis: at the limit the hull sits
+  // exactly CANVAS_BOUNDS_MARGIN of a viewport outside, and past it the
+  // outward drag is refused. Every camera write funnels through
+  // `setCamera`, so a drag, a wheel/pinch zoom, a tween and an imperative
+  // call all stop at the same edge.
+
+  describe("bounds clamp", () => {
+    // BOUNDS = 400×200 at (0,0), viewport 800×600, k = 1 ⇒
+    // x ∈ [-k·(x+w) - f·W, W - k·x + f·W] = [-800, 1200],
+    // y ∈ [-200 - 300, 600 + 300] = [-500, 900].
+    const X_LIMIT_LOW = -800;
+    const Y_LIMIT_LOW = -500;
+
+    it("clamps the pure camera to the expanded hull", () => {
+      const bounds = { x: 0, y: 0, width: 400, height: 200 };
+      const viewport = { width: 800, height: 600 };
+      // A camera already framing the content passes through untouched.
+      const inside = { k: 1, x: 100, y: 100 };
+      expect(clampCameraToBounds(inside, bounds, viewport)).toEqual(inside);
+      // Dragged far off-screen: stopped at the derived limit.
+      expect(clampCameraToBounds({ k: 1, x: -5000, y: -9000 }, bounds, viewport)).toEqual({
+        k: 1,
+        x: X_LIMIT_LOW,
+        y: Y_LIMIT_LOW,
+      });
+      // …and symmetrically for the far corners.
+      expect(clampCameraToBounds({ k: 1, x: 5000, y: 9000 }, bounds, viewport)).toEqual({
+        k: 1,
+        x: 1200,
+        y: 900,
+      });
+    });
+
+    it("holds the margin as a SCREEN distance at any zoom", () => {
+      // At k = 2 the world window halves, so the world-space margin halves
+      // with it — but the on-screen slack stays half a viewport.
+      const bounds = { x: 0, y: 0, width: 400, height: 200 };
+      const viewport = { width: 800, height: 600 };
+      const clamped = clampCameraToBounds({ k: 2, x: -99999, y: 0 }, bounds, viewport);
+      // xLow = -2·400 - 0.5·800 = -1200.
+      expect(clamped.x).toBe(-1200);
+      // At the limit the hull's right edge sits exactly half a viewport
+      // left of the screen's left edge.
+      const hullRightOnScreen = (bounds.width) * clamped.k + clamped.x;
+      expect(hullRightOnScreen).toBe(-viewport.width / 2);
+    });
+
+    it("passes degenerate inputs through unchanged", () => {
+      const cam = { k: 1, x: 123, y: 456 };
+      expect(clampCameraToBounds(cam, null, { width: 800, height: 600 })).toBe(cam);
+      expect(clampCameraToBounds(cam, { x: 0, y: 0, width: 0, height: 100 }, { width: 800, height: 600 })).toBe(cam);
+      expect(clampCameraToBounds(cam, { x: 0, y: 0, width: 100, height: 0 }, { width: 800, height: 600 })).toBe(cam);
+      expect(clampCameraToBounds(cam, { x: 0, y: 0, width: 100, height: 100 }, { width: 0, height: 0 })).toBe(cam);
+      expect(clampCameraToBounds({ k: 0, x: 1, y: 1 }, { x: 0, y: 0, width: 100, height: 100 }, { width: 800, height: 600 })).toEqual({ k: 0, x: 1, y: 1 });
+      // An astronomically placed hull overflows the clamp range (k·coord
+      // passes the double ceiling): refuse to clamp rather than write a
+      // non-finite translation (the browser would drop the transform).
+      const far = { x: 1e308, y: 1e308, width: 100, height: 100 };
+      expect(clampCameraToBounds({ k: 2, x: 123, y: 456 }, far, { width: 800, height: 600 })).toEqual({ k: 2, x: 123, y: 456 });
+      expect(clampCameraToBounds({ k: 1, x: NaN, y: 0 }, { x: 0, y: 0, width: 100, height: 100 }, { width: 800, height: 600 })).toEqual({ k: 1, x: NaN, y: 0 });
+    });
+
+    it("stops a pan that would drag the hull off-screen", async () => {
+      const { instance } = mount({ contentBounds: BOUNDS, fitOnLoad: false });
+      await nextTick();
+      instance.value?.panBy(-5000, -5000);
+      expect(instance.value!.camera).toEqual({ k: 1, x: X_LIMIT_LOW, y: Y_LIMIT_LOW });
+      // The far direction clamps at its own wall.
+      instance.value?.panBy(99999, 99999);
+      expect(instance.value!.camera).toEqual({ k: 1, x: 1200, y: 900 });
+      // Small moves apply on the free axis; the wall axis stays pinned.
+      instance.value?.panBy(-10, 5);
+      expect(instance.value!.camera).toEqual({ k: 1, x: 1190, y: 900 });
+    });
+
+    it("stops a pointer drag at the same edge", async () => {
+      const { instance, root } = mount({ contentBounds: BOUNDS, fitOnLoad: false });
+      await nextTick();
+      root.setPointerCapture = () => {};
+      root.dispatchEvent(pointerEvent("pointerdown", { pointerId: 21, clientX: 400, clientY: 300 }));
+      // 400 px of drag left/up, in two moves, is nowhere near the limit —
+      // then one enormous move rides into the wall.
+      root.dispatchEvent(pointerEvent("pointermove", { pointerId: 21, clientX: 300, clientY: 250 }));
+      root.dispatchEvent(pointerEvent("pointermove", { pointerId: 21, clientX: 200, clientY: 200 }));
+      root.dispatchEvent(pointerEvent("pointermove", { pointerId: 21, clientX: -100000, clientY: -100000 }));
+      document.body.dispatchEvent(pointerEvent("pointerup", { pointerId: 21, clientX: -100000, clientY: -100000, buttons: 0 }));
+      expect(instance.value!.camera).toEqual({ k: 1, x: X_LIMIT_LOW, y: Y_LIMIT_LOW });
+    });
+
+    it("keeps a wheel zoom anchored without escaping the walls", async () => {
+      const { instance } = mount({ contentBounds: BOUNDS, fitOnLoad: false });
+      await nextTick();
+      // Ride to the low-x wall first, then zoom out hard: the anchor keeps
+      // the world point under the cursor, but the translation must stay
+      // inside the clamp interval for the new k.
+      instance.value?.panBy(-5000, 0);
+      for (let step = 0; step < 12; step++) instance.value?.zoomBy(-1);
+      const cam = instance.value!.camera;
+      const xLow = -cam.k * (BOUNDS.x + BOUNDS.width) - CANVAS_BOUNDS_MARGIN * VIEWPORT.width;
+      const xHigh = VIEWPORT.width - cam.k * BOUNDS.x + CANVAS_BOUNDS_MARGIN * VIEWPORT.width;
+      expect(cam.x).toBeGreaterThanOrEqual(xLow - 1e-6);
+      expect(cam.x).toBeLessThanOrEqual(xHigh + 1e-6);
+    });
+
+    it("lets a controlled host's gestures clamp before the emit", async () => {
+      const onUpdate = vi.fn();
+      const { instance } = mount({
+        contentBounds: BOUNDS,
+        camera: { k: 1, x: 0, y: 0 },
+        "onUpdate:camera": onUpdate,
+        fitOnLoad: false,
+      });
+      await nextTick();
+      instance.value?.panBy(-5000, -5000);
+      const last = onUpdate.mock.calls.at(-1)?.[0] as { k: number; x: number; y: number };
+      expect(last).toEqual({ k: 1, x: X_LIMIT_LOW, y: Y_LIMIT_LOW });
+    });
+
+    it("does nothing the host did not ask for when the clamp is off", async () => {
+      const { instance } = mount({ contentBounds: BOUNDS, fitOnLoad: false, clampToBounds: false });
+      await nextTick();
+      instance.value?.panBy(-5000, -5000);
+      expect(instance.value!.camera).toEqual({ k: 1, x: -5000, y: -5000 });
+    });
+
+    it("never fights a fit: framing still centres the hull", async () => {
+      const { instance } = mount({ contentBounds: BOUNDS, fitOnLoad: false });
+      await nextTick();
+      instance.value?.panBy(-5000, -5000);
+      expect(instance.value!.fit()).toBe(true);
+      // 400×200 in 800×600 with 40 px padding → k = min(1.8, 2.6) capped at
+      // fitCap 1.25, centred: x = (800-500)/2, y = (600-250)/2.
+      expect(instance.value!.camera).toEqual({ k: 1.25, x: 150, y: 175 });
+    });
+  });
 });

package/src/components/HkNodeCanvas.tsx

@@ -13,6 +13,7 @@ import "./HkNodeCanvas.scss";
 
 import {
   computeLod,
+  clampCameraToBounds,
   edgeHitWidth,
   edgeMidpoint,
   edgePath,
@@ -39,7 +40,7 @@ export type {
   NodeCanvasPainter,
   PrintPaper,
 } from "./nodeCanvasTypes";
-export { computeLod, edgePath, edgeMidpoint, edgeHitWidth, sideOffset, oppositeSide, LOD_DEFAULTS, PAPER_SIZES } from "./nodeCanvasTypes";
+export { computeLod, edgePath, edgeMidpoint, edgeHitWidth, sideOffset, oppositeSide, LOD_DEFAULTS, PAPER_SIZES, clampCameraToBounds, CANVAS_BOUNDS_MARGIN } from "./nodeCanvasTypes";
 
 /** The pointer payload the edge events carry: a real MouseEvent for
  * hover/click/contextmenu, or a synthesized stand-in when a touch
@@ -143,6 +144,13 @@ interface PendingRequest {
  *     read to simplify their nodes at low zoom.
  *   - a **print mode**: disables gestures, fixes the camera, and exposes
  *     `exportSVG()` for vector output.
+ *   - a **bounds clamp** (default on): every camera write stops where the
+ *     content hull would leave half a viewport of itself past the visible
+ *     edge — a drag, a wheel or pinch zoom, a tween, an imperative call.
+ *     A graph can still be stranded only through the host's own channels:
+ *     opting out (`clampToBounds: false`), never supplying
+ *     `contentBounds`, or a controlled `camera` prop the host writes
+ *     verbatim (emitted gesture results are always pre-clamped).
  *
  * What it still does NOT own: what nodes look like, what data drives
  * them, or how they are laid out. The host draws node content through
@@ -165,7 +173,10 @@ export default defineComponent({
     /** Controlled camera (`v-model:camera`). Without it the component keeps
      *  its own, seeded by `fitOnLoad`. */
     camera: { type: Object as PropType<NodeCanvasCamera | undefined>, default: undefined },
-    /** Content rectangle, in world units, used for fit and clamping. */
+    /** Content rectangle, in world units — the hull every node on the
+     *  canvas forms. `fit` frames it, and the bounds clamp (`clampToBounds`,
+     *  on by default) refuses camera moves that would push it out of
+     *  reach. */
     contentBounds: { type: Object as PropType<NodeCanvasBounds | null>, default: null },
     /** Smallest zoom the camera will settle on. */
     minZoom: { type: Number, default: NODE_CANVAS_DEFAULTS.minZoom },
@@ -197,6 +208,15 @@ export default defineComponent({
     pannable: { type: Boolean, default: true },
     /** Whether the wheel zooms. */
     zoomable: { type: Boolean, default: true },
+    /** Keep the content within reach: the camera may never push the
+     *  content hull (the `contentBounds` envelope every node on the canvas
+     *  forms) farther than half a viewport past the visible edge, per
+     *  axis — the outward drag stops there instead of stranding the graph
+     *  off-screen. On by default; a canvas with a genuinely unbounded
+     *  surface (or one whose content the host moves independently) opts
+     *  out with `false`. Needs `contentBounds` to know the hull — without
+     *  it there is nothing to clamp against and this is a no-op. */
+    clampToBounds: { type: Boolean, default: true },
     /** Mount the minimap in the corner. */
     minimap: { type: Boolean, default: true },
     /** Which corner the minimap takes. */
@@ -558,12 +578,21 @@ export default defineComponent({
       // the surface freezes with no way back. Refuse to propagate it.
       if (!Number.isFinite(k) || k <= 0) return;
       if (!Number.isFinite(next.x) || !Number.isFinite(next.y)) return;
+      // Bounds clamp (default on): every write funnels through here, so a
+      // drag, a wheel/pinch zoom, a tween and an imperative `setCamera` all
+      // stop at the same edge — the content hull never leaves half a
+      // viewport of reach. `fit()` frames the hull inside the viewport and
+      // therefore always satisfies the clamp; it writes through
+      // `writeCamera` directly, so a host framing is never fought.
+      const bounded = props.clampToBounds
+        ? clampCameraToBounds({ k, x: next.x, y: next.y }, props.contentBounds, viewport.value)
+        : { k, x: next.x, y: next.y };
       // Any non-automatic camera move is the user's: a late `contentBounds`
       // must not throw that work away. It also means the host is somewhere
       // else, so a later `fit()` may ask for the same camera again.
       movedByHand = true;
       lastRequested = null;
-      writeCamera({ k, x: next.x, y: next.y });
+      writeCamera(bounded);
     }
 
     /** The camera that frames `contentBounds` inside the viewport. */

package/src/components/nodeCanvasTypes.ts

@@ -278,6 +278,74 @@ export function edgeMidpoint(
   return { x: (from.x + to.x) / 2, y: (from.y + to.y) / 2 };
 }
 
+// ── Camera bounds clamp ──────────────────────────────────────────────────
+
+/** How far past the content hull the camera may push it, as a fraction of
+ *  the viewport size per axis (0.5 = the hull may leave at most half a
+ *  viewport of itself past the visible edge — no farther). */
+export const CANVAS_BOUNDS_MARGIN = 0.5;
+
+/** Clamp a camera so the content cannot be dragged out of reach.
+ *
+ *  The binding geometry is the content hull — the axis-aligned envelope of
+ *  every component on the canvas (the hull's x/y extremes are exactly what
+ *  a pure translation can bind against). The camera is clamped so the
+ *  viewport's world window keeps intersecting that hull expanded by
+ *  `marginFactor` viewport sizes per axis: at the limit the hull itself
+ *  sits exactly `marginFactor` of a viewport outside the visible edge, and
+ *  past that the outward drag is refused.
+ *
+ *  Degenerate inputs pass the camera through unchanged: no bounds (the
+ *  host gave none), a non-finite or empty hull, an unmeasured viewport, a
+ *  non-finite zoom, or a coordinate combination whose clamp range is not
+ *  finite (an astronomically placed hull) — a clamped camera is only ever
+ *  returned finite, because the browser drops the whole transform
+ *  otherwise. */
+export function clampCameraToBounds(
+  camera: NodeCanvasCamera,
+  bounds: NodeCanvasBounds | null,
+  viewport: { width: number; height: number },
+  marginFactor: number = CANVAS_BOUNDS_MARGIN,
+): NodeCanvasCamera {
+  if (
+    !bounds ||
+    !(bounds.width > 0) ||
+    !(bounds.height > 0) ||
+    !Number.isFinite(bounds.x) ||
+    !Number.isFinite(bounds.y) ||
+    !(viewport.width > 0) ||
+    !(viewport.height > 0)
+  ) {
+    return camera;
+  }
+  const { k } = camera;
+  if (!Number.isFinite(k) || k <= 0) return camera;
+  if (!Number.isFinite(camera.x) || !Number.isFinite(camera.y)) return camera;
+  // The viewport's world window is [-x/k, (W-x)/k]; "keep intersecting the
+  // hull expanded by (f·W/k, f·H/k) world units" solves to these
+  // translation ranges. The viewport fraction cancels the zoom — the
+  // margin is a SCREEN distance (half a viewport) at any k.
+  const xLow = -k * (bounds.x + bounds.width) - marginFactor * viewport.width;
+  const xHigh = viewport.width - k * bounds.x + marginFactor * viewport.width;
+  const yLow = -k * (bounds.y + bounds.height) - marginFactor * viewport.height;
+  const yHigh = viewport.height - k * bounds.y + marginFactor * viewport.height;
+  // bounds.width/height > 0 keeps both intervals non-empty (dust aside);
+  // the finiteness guard refuses a hull so far out the range overflows —
+  // a non-finite translation would drop the whole transform.
+  if (!(xLow <= xHigh) || !(yLow <= yHigh)) return camera;
+  if (
+    !Number.isFinite(xLow) || !Number.isFinite(xHigh)
+    || !Number.isFinite(yLow) || !Number.isFinite(yHigh)
+  ) {
+    return camera;
+  }
+  return {
+    k,
+    x: Math.min(xHigh, Math.max(xLow, camera.x)),
+    y: Math.min(yHigh, Math.max(yLow, camera.y)),
+  };
+}
+
 // ── Print ────────────────────────────────────────────────────────────────
 
 export type PrintPaper = "a4-landscape" | "a3-landscape" | "auto";