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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: transcript-viewer
-Version: 0.11.1
+Version: 0.11.2
 Summary: Browse agent transcripts in a local, dependency-free web viewer.
 License: MIT
 Requires-Python: >=3.12
@@ -182,8 +182,14 @@ so. Nothing is left behind and nothing is copied twice.
 A run that delegated shows a **timeline** above its steps: one lane per
 subagent, laid out on the clock. It exists for the one thing a nested box
 cannot say — that two subagents were running at the same moment, which 25 pairs
-in the corpus this was built against were. Clicking a lane opens the box the
-transcript already has for it rather than showing the same thing twice.
+in the corpus this was built against were. Two bars in the same column of the
+chart is what says it; overlap gets no colour of its own, since a second
+encoding of the same fact is noise.
+
+Clicking a lane opens the box the transcript already has for it rather than
+showing the same thing twice, drawing the run out far enough to reach it first:
+a long run paints 250 steps at a time, and in one real session every delegation
+happens after step 1,231.
 
 Where a session spans days but holds hours of work — one runs 10.3 days and
 contains 5.3 hours, with 31 gaps over an hour — the quiet stretches are skipped

{transcript_viewer-0.11.1 → transcript_viewer-0.11.2}/README.md

@@ -169,8 +169,14 @@ so. Nothing is left behind and nothing is copied twice.
 A run that delegated shows a **timeline** above its steps: one lane per
 subagent, laid out on the clock. It exists for the one thing a nested box
 cannot say — that two subagents were running at the same moment, which 25 pairs
-in the corpus this was built against were. Clicking a lane opens the box the
-transcript already has for it rather than showing the same thing twice.
+in the corpus this was built against were. Two bars in the same column of the
+chart is what says it; overlap gets no colour of its own, since a second
+encoding of the same fact is noise.
+
+Clicking a lane opens the box the transcript already has for it rather than
+showing the same thing twice, drawing the run out far enough to reach it first:
+a long run paints 250 steps at a time, and in one real session every delegation
+happens after step 1,231.
 
 Where a session spans days but holds hours of work — one runs 10.3 days and
 contains 5.3 hours, with 31 gaps over an hour — the quiet stretches are skipped

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

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

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

@@ -371,6 +371,8 @@ pre{background:var(--sunk);border-radius:7px;padding:10px;overflow:auto;font-siz
 .branch .inner{padding:6px 14px 4px 22px;border-top:1px solid var(--line);background:var(--sunk)}
 .branch .inner .step{border-left-color:var(--line)}
 .miss{padding:9px 12px;color:var(--dim);font-size:12.5px}
+/* Where a lane just sent you. Long enough to notice, short enough to forget. */
+.branch.found{box-shadow:0 0 0 2px var(--accent)}
 
 /* ---- delegation timeline ----
    Nested boxes cannot show that two subagents ran at once, and 25 pairs in the
@@ -396,7 +398,6 @@ pre{background:var(--sunk);border-radius:7px;padding:10px;overflow:auto;font-siz
 .lane.tight .track{height:13px}
 .track .rule{position:absolute;left:0;right:0;top:50%;height:1px;background:var(--line)}
 .track .tick{position:absolute;top:0;bottom:0;width:1px;background:var(--line);opacity:.45}
-.track .band{position:absolute;top:0;bottom:0;background:var(--hl)}
 /* A floor, so a four-minute subagent in a thirty-hour run is still a target.
    Where a bar is at the floor its width is not its duration — the card and
    the tooltip carry the real one. */
@@ -2590,11 +2591,6 @@ function drawLanes(t) {
   const scale = timeScale(lo, hi, busy);
   const bands = together(timed);
 
-  const bandLayer = bands.map(([a, b]) => {
-    const left = scale.posOf(a);
-    return `<span class="band" style="left:${left}%;width:${Math.max(scale.posOf(b) - left, 0.3)}%"></span>`;
-  }).join("");
-
   const breaks = scale.gaps.map((g) =>
     `<span class="gapmark" style="left:${g.at}%" title="nothing running for ${esc(g.took)}"></span>`
   ).join("");
@@ -2637,7 +2633,7 @@ function drawLanes(t) {
     mainRow,
     marks: marks.map((m) => `<span style="left:${m.pos}%">${esc(m.label)}</span>`).join(""),
     note,
-    layer: bandLayer + breaks,
+    layer: breaks,
     overlaps: bands.length,
   });
 }
@@ -2698,7 +2694,6 @@ function laneShell({ count, shown, whole, rows, mainRow, marks, note, layer, ove
         <div class="lanes-legend">
           <span><i style="background:var(--ink);opacity:.85"></i>main thread</span>
           <span><i style="background:var(--accent);opacity:.85"></i>subagent</span>
-          ${overlaps ? `<span><i style="background:var(--hl)"></i>running at the same time</span>` : ""}
           <span class="lanes-note">${esc(note)}</span>
         </div>
       </div>
@@ -2746,13 +2741,51 @@ function laneCard(ev, id) {
 /* Open the subagent a lane stands for: scroll to the step that called it and
    unfold the box that is already there, rather than inventing a second place
    for the same thing to live. */
+/* Open the subagent a lane stands for: reach the step that called it, unfold
+   the box already there, and go to it.
+
+   A long run is drawn 250 steps at a time. In one real session all thirty
+   delegations happen after step 1,231, so the box a lane pointed at had never
+   been painted and the click did nothing at all. The run is drawn far enough
+   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. */
 function openLane(id) {
   laneCard(null);
-  const box = document.querySelector(`details.branch[data-lane="${CSS.escape(id)}"]`);
-  if (!box) return;
+  const lane = LANES.find((l) => l.id === id);
+  if (!lane) return;
+
+  const find = () =>
+    document.querySelector(`details.branch[data-lane="${CSS.escape(id)}"]`);
+
+  let box = find();
+  if (!box) {
+    let repaint = false;
+    if (lane.fromStep === null && (lens !== "all" || query)) {
+      lens = "all";
+      query = "";
+      repaint = true;
+    }
+    if (lane.fromIndex >= limit) {
+      // Far enough to include it, and a little beyond so it is not the last row.
+      limit = lane.fromIndex + PAGE_SIZE;
+      repaint = true;
+    }
+    if (repaint) render();
+    box = find();
+  }
+  if (!box) {
+    note("That subagent is not in this view.");
+    return;
+  }
+
   box.open = true;
-  box.scrollIntoView({ behavior: "smooth", block: "center" });
   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" }),
+  );
   setTimeout(() => box.classList.remove("found"), 1400);
 }
 

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

@@ -66,8 +66,31 @@ for (const id of [
 const SAFE_FETCH = () =>
   Promise.resolve({ ok: true, json: () => Promise.resolve({}) });
 
+/* Lookup by id, as the page expects. Reset with fetch on every run: a test that
+   stubs it to watch one element used to leave the next test's render() holding
+   an object with no focus(), which fails somewhere else entirely. */
+const SAFE_BY_ID = () => null;
+
+/* Looking for an element by selector, against what was actually painted. The
+   stub DOM holds markup as a string rather than a tree, so this answers the one
+   question the page asks — is this box on the page — from that string. */
+const SAFE_QUERY = (sel) => {
+  const m = /\[data-lane="([^"]+)"\]/.exec(sel);
+  if (!m) return null;
+  const html = (global.main && global.main.innerHTML) || "";
+  return html.includes(`data-lane="${m[1]}"`)
+    ? { open: false, classList: { add() {}, remove() {} }, scrollIntoView() {} }
+    : null;
+};
+
+/* The page escapes an id before putting it in a selector; the stub DOM has no
+   CSS object, so provide the one method that is used. */
+global.CSS = { escape: (v) => String(v).replace(/["\\]/g, "\\$&") };
+
 const run = (body) => {
   global.fetch = SAFE_FETCH;
+  global.document.getElementById = SAFE_BY_ID;
+  global.document.querySelector = SAFE_QUERY;
   return eval(script + "\n;(() => {" + body + "})()");
 };
 
@@ -1841,7 +1864,9 @@ test("the timeline says how many ran at once", () => {
   assert.match(html, /id="lanes"/);
   assert.match(html, /2 subagents/);
   assert.match(html, /overlapped once/, "one overlap reads as once, not '1 times'");
-  assert.match(html, /class="band"/, "the overlap is drawn, not only counted");
+  // Overlap is not given a colour of its own: two bars in the same column of
+  // the chart already say it, and a second encoding of the same fact is noise.
+  assert.ok(!/class="band"/.test(html), "the lanes are the encoding");
 });
 
 test("the header counts, and counts in English", () => {
@@ -2042,3 +2067,60 @@ test("two durations read against each other share a unit", () => {
   const wide = run(`return durationPair(31*3600000, 10*86400000)`);
   assert.deepStrictEqual(wide, ["1.3d", "10d"]);
 });
+
+test("a lane reaches its box even when the run has not been drawn that far", () => {
+  // The bug: a long run paints 250 steps at a time, and in one real session all
+  // thirty delegations happen after step 1,231 — so the box a lane pointed at
+  // had never been painted and clicking did nothing whatever.
+  const steps = [];
+  for (let i = 0; i < 400; i++) {
+    steps.push({ step_id: i + 1, source: "agent", message: "working" });
+  }
+  steps[300] = {
+    step_id: 301, source: "agent",
+    tool_calls: [{ tool_call_id: "c1", function_name: "Task" }],
+    observation: { results: [{ source_call_id: "c1",
+      subagent_trajectory_ref: [{ trajectory_id: "far" }] }] },
+  };
+  const t = {
+    steps,
+    subagent_trajectories: [{ trajectory_id: "far", agent: { name: "claude-code" },
+      extra: { agentType: "Explore" }, steps: [] }],
+  };
+
+  const out = run(
+    setup({ ...ROW, key: "k" }) + `
+    traj = ${JSON.stringify(t)};
+    cur = "k"; lens = "all"; query = ""; limit = 250;
+    render();
+    const before = !!document.querySelector('details.branch[data-lane="far"]');
+    LANES = laneIndex(traj);
+    openLane("far");
+    const after = !!document.querySelector('details.branch[data-lane="far"]');
+    return { before, after, limit };
+  `);
+
+  assert.strictEqual(out.before, false, "the box starts undrawn — this is the bug");
+  assert.strictEqual(out.after, true, "and the click has to reach it");
+  assert.ok(out.limit > 300, `the run was drawn far enough (limit ${out.limit})`);
+});
+
+test("a lane whose box is filtered away brings the view back rather than failing", () => {
+  const t = {
+    steps: [{ step_id: 1, source: "user", message: "go" }],
+    subagent_trajectories: [{ trajectory_id: "loose", agent: { name: "metr-malt" },
+      extra: {}, steps: [] }],
+  };
+  const out = run(
+    setup({ ...ROW, key: "k" }) + `
+    traj = ${JSON.stringify(t)};
+    cur = "k"; lens = "tools"; query = "nothing matches this"; limit = 250;
+    render();
+    LANES = laneIndex(traj);
+    openLane("loose");
+    return { lens, query, found: !!document.querySelector('details.branch[data-lane="loose"]') };
+  `);
+  assert.strictEqual(out.lens, "all", "the lens is cleared to reach it");
+  assert.strictEqual(out.query, "");
+  assert.strictEqual(out.found, true);
+});

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

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