transcript-viewer 0.6.1__tar.gz → 0.6.2__tar.gz

{transcript_viewer-0.6.1 → transcript_viewer-0.6.2}/PKG-INFO

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

{transcript_viewer-0.6.1 → transcript_viewer-0.6.2}/pyproject.toml

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

{transcript_viewer-0.6.1 → transcript_viewer-0.6.2}/src/transcript_viewer/page.html

@@ -838,6 +838,8 @@ function pre(content) {
 
 const esc=s=>String(s??"").replace(/[&<>"]/g,c=>({"&":"&amp;","<":"&lt;",">":"&gt;",'"':"&quot;"}[c]));
 const num=n=>(n??0).toLocaleString();
+/* How many sessions are in the library, said the same way everywhere. */
+const showCount=()=>{count.textContent=`${num(INDEX.length)} session${INDEX.length===1?"":"s"}`};
 /* One size formatter. Everything used to divide by 1048576 and say MB, so a
    900-byte log read as "0.0 MB" and a 144 GB bucket as "147456.0 MB". Each step
    keeps three significant figures, which is as much as anyone reads. */
@@ -870,7 +872,7 @@ function reveal(a){
 async function openFiles(files){
   if(!files||!files.length)return;
   const names=[...files].map(f=>f.name);
-  count.textContent=`opening ${names.length} file${names.length>1?"s":""}…`;
+  count.textContent=`opening ${num(names.length)} file${names.length>1?"s":""}…`;
   const problems=[];
   let first=null;
   for(const file of files){
@@ -884,7 +886,7 @@ async function openFiles(files){
   }
   const fresh=await fetch("/api/index").then(r=>r.json());
   INDEX=fresh.sessions||[];GROUPS=fresh.groups||[];TAGS=fresh.tags||[];AI=fresh.ai||{};DOWNLOADS=fresh.downloads||DOWNLOADS;
-  count.textContent=INDEX.length+" sessions";drawList();
+  showCount();drawList();
   if(first!==null)pick(first);
   if(problems.length)note(problems.join("\n"));
   picker.value="";
@@ -1015,10 +1017,10 @@ async function openLibrary(){
   try{
     const d=await readIndex();
     INDEX=d.sessions||[];GROUPS=d.groups||[];TAGS=d.tags||[];AI=d.ai||{};DOWNLOADS=d.downloads||DOWNLOADS;
-    bootSay("Drawing "+INDEX.length+" sessions…",0,0);
+    bootSay(`Drawing ${num(INDEX.length)} sessions…`,0,0);
     // Let that line paint before the work that blocks the thread.
     await new Promise(r=>requestAnimationFrame(()=>requestAnimationFrame(r)));
-    count.textContent=INDEX.length+" sessions";drawList();showLibrary();
+    showCount();drawList();showLibrary();
   }catch(err){
     // Leave it on screen: a blank window would say nothing at all.
     bootSay("Could not open your library — "+((err&&err.message)||err),0,0);
@@ -1117,13 +1119,13 @@ function drawList(){
           <button title="Remove these sessions from the library" class="dang"
             onclick="event.stopPropagation();dropGroup('${esc(r.path)}')">✕</button>
         </span>`}
-        <span class="rc">${r.count}</span></div>`;
+        <span class="rc">${num(r.count)}</span></div>`;
   }).join("");
 
   tagbar.innerHTML=TAGS.map(t=>{
     const on=!!ACTIVE_TAGS[t.name];
     return `<span class="pill tag${on?" on":""}" onclick="toggleTag('${esc(t.name)}')">
-      ${esc(t.name)}<b>${t.count}</b></span>`;
+      ${esc(t.name)}<b>${num(t.count)}</b></span>`;
   }).join("");
 }
 
@@ -1685,7 +1687,7 @@ async function planUrl(){
     showLibrary();
     closeUrl();
     showProgress(0,0);
-    if(result)note(`Added ${result.added} session${result.added===1?"":"s"} — ${result.into}`);
+    if(result)note(`Added ${num(result.added)} session${result.added===1?"":"s"} — ${result.into}`);
     return;
   }
 
@@ -1712,8 +1714,8 @@ async function planUrl(){
     const size=p.bytes?` · ${bytes(p.bytes)}`:"";
     const sample=p.names.length?` — ${p.names.slice(0,3).join(", ")}${
       p.count>3?", …":""}`:"";
-    setUrlState(`${p.count} file${p.count===1?"":"s"} in ${p.label}${size}${sample}\n`
-      +`→ ${p.into}`, `Download ${p.count}`);
+    setUrlState(`${num(p.count)} file${p.count===1?"":"s"} in ${p.label}${size}${sample}\n`
+      +`→ ${p.into}`, `Download ${num(p.count)}`);
   }catch(e){
     err.textContent=e.message;err.hidden=false;
     setUrlState("","Look");PLANNED=null;
@@ -1874,7 +1876,7 @@ async function annotate(key,fields){
 async function refreshIndex(){
   const d=await fetch("/api/index").then(r=>r.json());
   INDEX=d.sessions||[];GROUPS=d.groups||[];TAGS=d.tags||[];AI=d.ai||{};DOWNLOADS=d.downloads||DOWNLOADS;
-  count.textContent=INDEX.length+" sessions";
+  showCount();
   drawList();
   if(!cur)showLibrary();
 }
@@ -2277,7 +2279,7 @@ function branchNav(branches){
         <span class="sid">${b.stepId}</span>
         <b>${esc(label)}</b>
         <span>${esc(x.description||"")||"—"}</span>
-        <em>${b.sub?b.sub.steps.length+" steps":"external"}</em>
+        <em>${b.sub?num(b.sub.steps.length)+" steps":"external"}</em>
       </button>`;
     }).join("")+`</div></div>`;
 }
@@ -2305,7 +2307,7 @@ function step(s,ctx,depth,idx,prefix){
   const on=starredSteps().includes(key);
   let h=`<div class="step ${s.source}" id="step-${key}">
     <div class="gut">
-      <a class="sid" href="#step-${key}" title="step ${s.step_id}">${s.step_id}</a>
+      <a class="sid" href="#step-${key}" title="step ${num(s.step_id)}">${num(s.step_id)}</a>
       <span class="sstar${on?" on":""}" onclick="toggleStep(event,'${key}')"
         title="${on?"Remove from favourites":"Add to favourites"}">
         <svg width="11" height="11" viewBox="0 0 24 24" fill="${on?"currentColor":"none"}"
@@ -2349,7 +2351,7 @@ function branch(ref,ctx,depth){
   return `<details class="branch"${OPEN_ALL?" open":""}><summary>
       <b>${esc(label)}</b>
       <span class="desc">${esc(x.description||"")}</span>
-      <span class="pill br">${sub.steps.length} steps</span>
+      <span class="pill br">${num(sub.steps.length)} steps</span>
       ${depth?`<span class="pill">depth ${depth+1}</span>`:""}
     </summary><div class="inner">
       ${sub.steps.map(s=>step(s,sub,depth+1,null,esc(sub.trajectory_id||"sub")+"-")).join("")}

{transcript_viewer-0.6.1 → transcript_viewer-0.6.2}/tests/test_style.py

@@ -110,3 +110,40 @@ def test_sizes_go_through_one_formatter():
     )
     assert "1048576" not in code, "a size is still being divided by hand"
     assert code.count("function bytes(") == 1
+
+
+# Numbers a reader sees are formatted in one place, so a corpus of thousands
+# reads as "2,258 sessions" rather than "2258 sessions". These are the shapes
+# that reached the screen raw before anyone looked.
+RAW_NUMBERS = (
+    (r"\.length\s*\+\s*[\"']", "a count concatenated onto a string"),
+    (r"\$\{[^}]*\.length\}\s*(?:session|step|file|result|match)", "${…length} beside a noun"),
+    (r"\$\{\s*[a-zA-Z_$][\w.$]*\.count\s*\}", "${…count}"),
+    (r"\$\{\s*[a-zA-Z_$][\w.$]*\.added\s*\}", "${…added}"),
+    (r"\$\{\s*s\.step_id\s*\}", "${s.step_id}"),
+)
+
+
+@pytest.mark.parametrize(("pattern", "what"), RAW_NUMBERS, ids=[w for _, w in RAW_NUMBERS])
+def test_a_number_on_screen_goes_through_the_formatter(pattern, what):
+    script = PAGE[PAGE.index("<script>") : PAGE.rindex("</script>")]
+    hits = []
+    for i, line in enumerate(script.splitlines(), start=1):
+        stripped = line.strip()
+        if stripped.startswith("//") or stripped.startswith("*"):
+            continue
+        for match in re.finditer(pattern, line):
+            around = line[max(0, match.start() - 14) : match.end() + 3]
+            if "num(" in around or "bytes(" in around:
+                continue
+            hits.append(f"line {i}: {stripped[:90]}")
+    assert not hits, f"{what} without num(): " + "; ".join(hits)
+
+
+def test_the_session_count_is_said_in_one_place():
+    """It was written out three times, and one of them was already wrong."""
+    script = PAGE[PAGE.index("<script>") : PAGE.rindex("</script>")]
+    assert "const showCount=" in script, "the count should have a single home"
+    assert script.count('count.textContent=') <= 2, (
+        "the session count is being written in more than one place again"
+    )

{transcript_viewer-0.6.1 → transcript_viewer-0.6.2}/uv.lock

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