transcript-viewer 0.7.1__tar.gz → 0.8.0__tar.gz

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/PKG-INFO

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

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/pyproject.toml

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

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/src/transcript_viewer/page.html

@@ -412,20 +412,45 @@ pre{background:var(--sunk);border-radius:7px;padding:10px;overflow:auto;font-siz
 /* JSON syntax colours */
 .j-key{color:var(--j-key)}
 /* A command reads as a command: one gutter mark, then the script as written. */
+/* Wrapped rather than scrolled: a command is read, and a line that runs off the
+   side is a line nobody reads.
+
+   No hanging indent. A negative text-indent pulls the *first* line out of the
+   content box to the left, and `pre` carries overflow:auto, so the box scrolled
+   itself sideways to chase it — which hid the first word of every command and
+   left the prompt mark stranded mid-block. Wrapped lines simply align with the
+   rest; which lines the shell sees as separate is legible from the text. */
 .sh{background:var(--bg);border-radius:7px;padding:10px 12px 10px 26px;margin:0;
   position:relative;font:12px/1.6 var(--font-mono);color:var(--ink);
-  /* Wrapped rather than scrolled: a command is read, and a line that runs off
-     the side is a line nobody reads. A continued line is indented so it stays
-     obvious which lines the shell would actually see as separate. */
-  white-space:pre-wrap;overflow-wrap:anywhere;text-indent:-1.4em;padding-left:calc(26px + 1.4em)}
+  white-space:pre-wrap;overflow-wrap:anywhere;overflow-x:hidden}
 .sh::before{content:"$";position:absolute;left:11px;top:10px;color:var(--faint);
-  user-select:none;text-indent:0}
+  user-select:none}
 .sh-cmd{color:var(--sh-cmd)}
 .sh-str{color:var(--sh-str)}
 .sh-flag{color:var(--sh-flag)}
 .sh-op{color:var(--sh-op);font-weight:500}
 .sh-var{color:var(--sh-var)}
 .sh-com{color:var(--sh-com);font-style:italic}
+
+/* A file, shown as a file. The numbers sit in a gutter and are not selectable,
+   so copying the block gives back the file rather than the listing of it. */
+.fpath{font:11.5px/1.5 var(--font-mono);color:var(--muted);margin-bottom:6px;
+  overflow-wrap:anywhere}
+.fmeta{font:11px/1.5 var(--font-mono);color:var(--faint);margin-top:6px}
+.filelines,.filebody{background:var(--bg);border-radius:7px;padding:10px 12px;
+  margin:0;max-height:340px;overflow:auto;font:12px/1.6 var(--font-mono);
+  color:var(--ink);white-space:pre-wrap;overflow-wrap:anywhere}
+.ln{display:inline-block;width:3.2em;margin-right:.8em;text-align:right;
+  color:var(--faint);user-select:none;-webkit-user-select:none}
+/* What a change replaced, and what it put there. */
+.epair{display:grid;gap:8px}
+.eside{border-radius:7px;padding:8px 10px;background:var(--bg);
+  font:12px/1.6 var(--font-mono);white-space:pre-wrap;overflow-wrap:anywhere;
+  max-height:260px;overflow:auto}
+.eside b{display:block;font:11px/1.5 var(--font-mono);font-weight:500;
+  margin-bottom:4px;letter-spacing:.04em;text-transform:uppercase}
+.eside.out b{color:var(--danger)}
+.eside.in b{color:var(--ok)}
 /* What the call was for, beside what it was called. */
 .tdesc{color:var(--muted);font-size:12px;overflow:hidden;text-overflow:ellipsis;
   white-space:nowrap;flex:1;min-width:0}
@@ -884,6 +909,44 @@ function sh(src) {
 
 const shBlock = (command) => `<pre class="sh">${autolink(sh(command))}</pre>`;
 
+/* Where a file is, as something you can open.
+
+   Linked whole rather than through autolink: this value is a path in its
+   entirety, where autolink has to guess boundaries in prose and stops at the
+   first space. A real one here — "/Users/…/Redwood Research/…" — linked only
+   as far as "Redwood", which pointed the link at a directory that is not the
+   file. PATH_EXACT is the same test the JSON view uses for a quoted value. */
+const fileRef = (path) => {
+  const text = String(path ?? "");
+  if (!text) return "";
+  const shown = PATH_EXACT.test(text) ? pathAnchor(text, esc(text)) : esc(text);
+  return `<div class="fpath">${shown}</div>`;
+};
+
+const countOf = (text) => {
+  const s = String(text ?? "");
+  const lines = s ? s.split("\n").length : 0;
+  return `${num(lines)} line${lines === 1 ? "" : "s"} · ${bytes(s.length)}`;
+};
+
+/* A file read back arrives numbered, one "12\tline" per line. Put the numbers
+   in a gutter so the file reads as a file, and leave them unselectable so that
+   copying gives the contents rather than the listing. */
+function numbered(text) {
+  const rows = String(text ?? "")
+    .split("\n")
+    .map((line) => {
+      const m = /^(\s*\d+)\t([\s\S]*)$/.exec(line);
+      return m
+        ? `<span class="ln">${esc(m[1].trim())}</span>${esc(m[2])}`
+        : `<span class="ln"></span>${esc(line)}`;
+    });
+  return `<pre class="filelines">${autolink(rows.join("\n"))}</pre>`;
+}
+
+const fileBlock = (text) =>
+  `<pre class="filebody">${autolink(esc(String(text ?? "")))}</pre>`;
+
 /* What a call is worth showing as, when it is not worth showing as JSON. Only
    the shapes that carry a command or an address get a body of their own; the
    rest gain a title, which is the part that was missing everywhere. */
@@ -905,9 +968,37 @@ const TOOLVIEW = {
   },
   WebSearch: { title: (a) => a.query || "" },
   // The end of a path is the part worth reading; the full one is on hover.
-  Read: { title: (a) => short(a.file_path) },
-  Write: { title: (a) => short(a.file_path) },
-  Edit: { title: (a) => short(a.file_path) },
+  Read: {
+    title: (a) => short(a.file_path),
+    body: (a) =>
+      fileRef(a.file_path) +
+      (a.offset || a.limit || a.pages
+        ? `<div class="fmeta">${
+            a.pages
+              ? `pages ${esc(String(a.pages))}`
+              : `from line ${num(a.offset || 1)}${a.limit ? `, ${num(a.limit)} lines` : ""}`
+          }</div>`
+        : ""),
+    // What came back is the file, numbered as the reader saw it.
+    output: (content) => numbered(content),
+  },
+  Write: {
+    title: (a) => short(a.file_path),
+    body: (a) =>
+      fileRef(a.file_path) +
+      fileBlock(a.content) +
+      `<div class="fmeta">${countOf(a.content)}</div>`,
+  },
+  Edit: {
+    title: (a) => short(a.file_path),
+    body: (a) =>
+      fileRef(a.file_path) +
+      `<div class="epair">
+         <div class="eside out"><b>replaced</b>${autolink(esc(a.old_string ?? ""))}</div>
+         <div class="eside in"><b>with</b>${autolink(esc(a.new_string ?? ""))}</div>
+       </div>` +
+      (a.replace_all ? `<div class="fmeta">every occurrence</div>` : ""),
+  },
   NotebookEdit: { title: (a) => short(a.notebook_path) },
   Skill: { title: (a) => a.skill || "" },
   Task: { title: (a) => a.description || "" },
@@ -938,7 +1029,11 @@ function toolBody(id) {
     `<div class="io in"><span class="iol">call</span>${shown}</div>` +
     (r && r.content
       ? `<div class="io out"><span class="iol">output</span>${
-          typeof r.content === "string" ? pre(r.content) : body(r.content)
+          typeof r.content !== "string"
+            ? body(r.content)
+            : !toolIsRaw(id) && view && view.output
+              ? view.output(r.content, c.arguments || {})
+              : pre(r.content)
         }</div>`
       : "") +
     (aiOn() ? callSummary(c.tool_call_id) : "")

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/tests/page.test.js

@@ -1673,3 +1673,103 @@ test("a real 20,000-character command draws, and quickly", () => {
   assert.ok(html.includes("class=\"sh\""), "it should render");
   assert.ok(took < 2000, `took ${took}ms, which is a page nobody waits for`);
 });
+
+// ---- a file read, written or changed -----------------------------------------
+
+test("a file read back keeps its line numbers in a gutter", () => {
+  const html = run(`return TOOLVIEW.Read.output('1\\tfirst\\n2\\tsecond')`);
+  assert.match(html, /class="filelines"/);
+  assert.match(html, /<span class="ln">1<\/span>first/);
+  assert.match(html, /<span class="ln">2<\/span>second/);
+  assert.ok(!html.includes("\t"), "the tab becomes the gutter, not whitespace");
+});
+
+test("a line that is not numbered still shows", () => {
+  const html = run(`return TOOLVIEW.Read.output('no number here')`);
+  assert.match(html, /no number here/);
+});
+
+test("a read of part of a file says which part", () => {
+  const html = run(`return TOOLVIEW.Read.body({file_path:"/a/b.py",offset:40,limit:120})`);
+  assert.match(html, /from line 40/);
+  assert.match(html, /120 lines/);
+});
+
+test("a whole-file read says nothing about ranges", () => {
+  const html = run(`return TOOLVIEW.Read.body({file_path:"/a/b.py"})`);
+  assert.ok(!/from line/.test(html));
+});
+
+test("a path in a file view still opens in Finder", () => {
+  const html = run(`return TOOLVIEW.Read.body({file_path:"/Users/someone/a.py"})`);
+  assert.match(html, /api\/reveal\?path=/);
+});
+
+test("a write shows what it wrote, and how much", () => {
+  const html = run(
+    `return TOOLVIEW.Write.body({file_path:"/a/b.js",content:"one\\ntwo\\nthree"})`,
+  );
+  assert.match(html, /class="filebody"/);
+  assert.match(html, /three/);
+  assert.match(html, /3 lines/);
+});
+
+test("an edit shows what it replaced and what it put there", () => {
+  const html = run(
+    `return TOOLVIEW.Edit.body({file_path:"/a/b.js",old_string:"before",new_string:"after",replace_all:true})`,
+  );
+  assert.match(html, /class="eside out"[^>]*>.*replaced.*before/s);
+  assert.match(html, /class="eside in"[^>]*>.*with.*after/s);
+  assert.match(html, /every occurrence/);
+});
+
+test("an edit that replaces once does not claim otherwise", () => {
+  const html = run(
+    `return TOOLVIEW.Edit.body({file_path:"/a/b.js",old_string:"x",new_string:"y",replace_all:false})`,
+  );
+  assert.ok(!/every occurrence/.test(html));
+});
+
+test("file contents cannot reach the page as markup", () => {
+  for (const expr of [
+    `TOOLVIEW.Write.body({file_path:"/a",content:"<img src=x onerror=alert(1)>"})`,
+    `TOOLVIEW.Edit.body({file_path:"/a",old_string:"<script>",new_string:"<b>"})`,
+    `TOOLVIEW.Read.output("1\\t<img src=x onerror=alert(1)>")`,
+  ]) {
+    const html = run(`return ${expr}`);
+    assert.ok(!/<img|<script>/.test(html), `escaped badly: ${expr}`);
+  }
+});
+
+test("asking for the recording gives the recording, not the file view", () => {
+  const html = run(`
+    raw = false;
+    CALLS["r1"] = {
+      c:{function_name:"Read",tool_call_id:"r1",arguments:{file_path:"/a/b.py"}},
+      r:{source_call_id:"r1",content:"1\\thello"},
+    };
+    TOOLRAW["r1"] = true;
+    return toolBody("r1");
+  `);
+  assert.match(html, /class="json"/, "the call shows as recorded");
+  assert.ok(!/class="filelines"/.test(html), "and so does the output");
+});
+
+test("a path with a space in it links to the whole path", () => {
+  // A real one: "/Users/girish/Documents/Redwood Research/…" linked only as far
+  // as "Redwood", so the link opened a directory that was not the file.
+  const html = run(
+    `return TOOLVIEW.Read.body({file_path:"/Users/girish/Documents/Redwood Research/app/mentions.py"})`,
+  );
+  assert.match(html, /Redwood%20Research/, "the space must be inside the link");
+  assert.ok(
+    !/<\/a>\s*Research/.test(html),
+    "the link must not stop at the space",
+  );
+});
+
+test("a relative path is shown but not offered as a link", () => {
+  const html = run(`return TOOLVIEW.Read.body({file_path:"src/thing.py"})`);
+  assert.match(html, /src\/thing\.py/);
+  assert.ok(!/api\/reveal/.test(html), "revealing a relative path would guess");
+});

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/tests/test_style.py

@@ -147,3 +147,34 @@ def test_the_session_count_is_said_in_one_place():
     assert script.count('count.textContent=') <= 2, (
         "the session count is being written in more than one place again"
     )
+
+
+def _rule(selector: str) -> str:
+    """The declarations of one rule, as written."""
+    start = STYLE.index(selector + "{")
+    return STYLE[start + len(selector) + 1 : STYLE.index("}", start)]
+
+
+def test_a_command_block_keeps_its_first_line_inside_itself():
+    """A negative text-indent pulls the first line out of the box.
+
+    `pre` carries overflow:auto, so the box then scrolls sideways to chase it:
+    the first word of every command disappeared and the prompt mark was left
+    stranded mid-block. Both symptoms, one declaration.
+    """
+    rule = _rule(".sh")
+    assert "text-indent:-" not in rule.replace(" ", ""), (
+        "a negative text-indent on .sh hides the start of the command"
+    )
+
+
+def test_a_command_wraps_rather_than_scrolling():
+    rule = _rule(".sh").replace(" ", "").replace("\n", "")
+    assert "white-space:pre-wrap" in rule, "a command that runs off the side is not read"
+    assert "overflow-x:auto" not in rule, "it should wrap, not scroll"
+
+
+def test_the_prompt_mark_stays_at_the_top_left():
+    rule = _rule(".sh::before").replace(" ", "").replace("\n", "")
+    assert "position:absolute" in rule
+    assert "left:11px" in rule and "top:10px" in rule

{transcript_viewer-0.7.1 → transcript_viewer-0.8.0}/uv.lock

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