transcript-viewer 0.11.2__tar.gz → 0.11.3__tar.gz

{transcript_viewer-0.11.2 → transcript_viewer-0.11.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: transcript-viewer
-Version: 0.11.2
+Version: 0.11.3
 Summary: Browse agent transcripts in a local, dependency-free web viewer.
 License: MIT
 Requires-Python: >=3.12

{transcript_viewer-0.11.2 → transcript_viewer-0.11.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "transcript-viewer"
-version = "0.11.2"
+version = "0.11.3"
 description = "Browse agent transcripts in a local, dependency-free web viewer."
 readme = "README.md"
 requires-python = ">=3.12"

{transcript_viewer-0.11.2 → transcript_viewer-0.11.3}/src/transcript_viewer/page.html

@@ -2750,6 +2750,18 @@ function laneCard(ev, id) {
    first. A lane from a record where nothing spawned it is found in the block
    after the run instead — which only exists in the plain view, so a lens or a
    search is cleared to get back to it rather than failing quietly. */
+/* Put an element at the top of the reading area, with room to breathe. */
+function laneScroll(el) {
+  const main = document.getElementById("main");
+  if (!main || !el.getBoundingClientRect || !main.getBoundingClientRect) {
+    if (el.scrollIntoView) el.scrollIntoView({ behavior: "smooth", block: "start" });
+    return;
+  }
+  const top =
+    main.scrollTop + el.getBoundingClientRect().top - main.getBoundingClientRect().top - 16;
+  main.scrollTo({ top: Math.max(0, top), behavior: "smooth" });
+}
+
 function openLane(id) {
   laneCard(null);
   const lane = LANES.find((l) => l.id === id);
@@ -2783,9 +2795,11 @@ function openLane(id) {
   box.classList.add("found");
   // After the fold, not before: opening changes the height, and scrolling to
   // where the box used to end leaves the reader above what they asked for.
-  requestAnimationFrame(() =>
-    box.scrollIntoView({ behavior: "smooth", block: "center" }),
-  );
+  //
+  // To the top of it, not the middle. Centring a box puts a subagent of any
+  // length halfway through itself, when what was asked for was the subagent —
+  // which starts at its first step.
+  requestAnimationFrame(() => laneScroll(box));
   setTimeout(() => box.classList.remove("found"), 1400);
 }
 

{transcript_viewer-0.11.2 → transcript_viewer-0.11.3}/tests/page.test.js

@@ -2124,3 +2124,54 @@ test("a lane whose box is filtered away brings the view back rather than failing
   assert.strictEqual(out.query, "");
   assert.strictEqual(out.found, true);
 });
+
+test("a lane lands on the top of what it opened, not the middle of it", () => {
+  // Centring put a long subagent halfway through itself. What was asked for is
+  // the subagent, which starts at its first step.
+  const moved = [];
+  const box = {
+    open: false,
+    classList: { add() {}, remove() {} },
+    getBoundingClientRect: () => ({ top: 900 }),
+  };
+  const main = {
+    scrollTop: 200,
+    getBoundingClientRect: () => ({ top: 60 }),
+    scrollTo: (opts) => moved.push(opts),
+  };
+  global.__el = { main, box };
+
+  run(`
+    document.getElementById = (id) => (id === "main" ? globalThis.__el.main : null);
+    laneScroll(globalThis.__el.box);
+  `);
+
+  assert.strictEqual(moved.length, 1, "it should scroll the reading area itself");
+  // 200 + 900 - 60 - 16 of breathing room.
+  assert.strictEqual(moved[0].top, 1024);
+  assert.strictEqual(moved[0].behavior, "smooth");
+});
+
+test("landing does not scroll above the top of the run", () => {
+  const moved = [];
+  global.__el2 = {
+    main: {
+      scrollTop: 0,
+      getBoundingClientRect: () => ({ top: 60 }),
+      scrollTo: (o) => moved.push(o),
+    },
+    box: { getBoundingClientRect: () => ({ top: 10 }) },
+  };
+  run(`
+    document.getElementById = (id) => (id === "main" ? globalThis.__el2.main : null);
+    laneScroll(globalThis.__el2.box);
+  `);
+  assert.strictEqual(moved[0].top, 0, "never a negative offset");
+});
+
+test("the box a lane opened is marked, and the mark is a real rule", () => {
+  const style = page.slice(page.indexOf("<style>"), page.indexOf("</style>"));
+  assert.match(style, /\.branch\.found\{/, "the class must style something");
+  const script = page.slice(page.indexOf("<script>"), page.lastIndexOf("</script>"));
+  assert.match(script, /classList\.add\("found"\)/, "and something must set it");
+});

{transcript_viewer-0.11.2 → transcript_viewer-0.11.3}/uv.lock

@@ -388,7 +388,7 @@ wheels = [
 
 [[package]]
 name = "transcript-viewer"
-version = "0.11.2"
+version = "0.11.3"
 source = { editable = "." }
 dependencies = [
     { name = "atif-make" },