Upload 29 files

tests/test_v52_ui_fixes.py

@@ -39,6 +39,6 @@ def test_runs_explorer_is_limited_to_five_rows_with_pagination():
 
 def test_progress_endpoint_uses_view_model_links_for_job_url():
     app_py = Path("app.py").read_text(encoding="utf-8")
-    assert '"version": "v60-cancel-persist-delete-run"' in app_py
+    assert '"version": "v62-progress-perf-clean-delete"' in app_py
     assert 'view = build_run_view_model(run_id, bundle, bucket_source=bucket_source)' in app_py
     assert '"links": view.get("links")' in app_py

tests/test_v57_live_pi_traces.py

@@ -10,24 +10,26 @@ def test_worker_streams_pi_output_while_running():
     assert "run_cmd_streaming(" in worker
 
 
-def test_bucket_and_progress_endpoint_expose_pi_live_log():
+def test_progress_endpoint_no_longer_exposes_pi_stdout_payload():
     bucket = Path("src/bucket.py").read_text(encoding="utf-8")
     app = Path("app.py").read_text(encoding="utf-8")
-    assert '"pi_live_log": _safe_read_text' in bucket
-    assert '"pi_output_log": _safe_read_text' in bucket
-    assert '"pi_live_log": pi_stdout[-12000:]' in app
+    assert '"pi_live_log": _safe_read_text' not in bucket
+    assert '"pi_output_log": _safe_read_text' not in bucket
+    assert "include_heavy: bool = True" in bucket
+    assert "include_job_logs: bool = False" in app
+    assert '"pi_live_log"' not in app
 
 
 def test_ui_has_pi_live_trace_panel_and_renderer():
     html = Path("web/index.html").read_text(encoding="utf-8")
     js = Path("web/static/progress.js").read_text(encoding="utf-8")
     css = Path("web/static/app.css").read_text(encoding="utf-8")
-    assert 'id="piTracePanel"' in html
-    assert "Pi process stdout" in html
-    assert "function renderPiLiveTrace" in js
-    assert "p.pi_live_log" in js
-    assert ".pi-trace-output" in css
+    assert 'id="piTracePanel"' not in html
+    assert "Pi process stdout" not in html
+    assert "function renderPiLiveTrace" not in js
+    assert "p.pi_live_log" not in js
+    assert ".pi-trace-output" not in css
 
 
 def test_app_version_v57():
-    assert '"version": "v60-cancel-persist-delete-run"' in Path("app.py").read_text(encoding="utf-8")
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")

tests/test_v58_stable_refresh_pi_stdout.py

@@ -1,11 +1,18 @@
 from pathlib import Path
 
 
+def test_pi_stdout_panel_removed_from_active_run():
+    html = Path("web/index.html").read_text(encoding="utf-8")
+    js = Path("web/static/progress.js").read_text(encoding="utf-8")
+    assert 'id="piTracePanel"' not in html
+    assert "Pi process stdout" not in html
+    assert "renderPiLiveTrace" not in js
+
+
 def test_progress_poll_does_not_refresh_runs_every_tick():
     js = Path("web/static/app.js").read_text(encoding="utf-8")
     assert "lastRunsRefreshFromPoll" in js
     assert ">15000" in js
-    assert "loadRuns(false)}}catch" in js
 
 
 def test_elapsed_ticks_client_side_each_second():
@@ -13,19 +20,7 @@ def test_elapsed_ticks_client_side_each_second():
     assert "const elapsedClock" in progress
     assert "window.setInterval" in progress
     assert "elapsedClock.baseSeconds+Math.floor" in progress
-    assert "setElapsedClock(p.elapsed_seconds??header.elapsed_seconds" in progress
-
-
-def test_pi_live_panel_is_stdout_not_trace_jsonl():
-    html = Path("web/index.html").read_text(encoding="utf-8")
-    js = Path("web/static/progress.js").read_text(encoding="utf-8")
-    app = Path("app.py").read_text(encoding="utf-8")
-    assert "Pi process stdout" in html
-    assert "logs/pi_live_output.txt" in html
-    assert "pi_live_log_source" in app
-    assert "No Pi stdout yet" in js
-
 
 
-def test_app_version_v58():
-    assert '"version": "v60-cancel-persist-delete-run"' in Path("app.py").read_text(encoding="utf-8")
+def test_app_version_v62():
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")

tests/test_v59_job_log_progress_clean.py

@@ -9,7 +9,7 @@ def test_progress_endpoint_polls_hf_job_logs_and_extracts_worker_events():
     assert "fetch_recent_logs_safe(job_id" in app_py
     assert "_events_from_job_logs(job_logs)" in app_py
     assert "_merge_events(bundle.get" in app_py
-    assert '"job_logs": job_logs[-20000:]' in app_py
+    assert '"job_logs_polled": bool(include_job_logs)' in app_py
     assert '"job_log_events_count": len(job_log_events)' in app_py
 
 
@@ -51,4 +51,4 @@ def test_progress_renderer_no_longer_sets_active_run_link_buttons():
 
 
 def test_app_version_v59():
-    assert '"version": "v60-cancel-persist-delete-run"' in Path("app.py").read_text(encoding="utf-8")
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")

tests/test_v60_cancel_persist_delete_run.py

@@ -5,7 +5,7 @@ def test_cancel_uses_hf_jobs_cancel_api_and_persists_bucket_state():
     jobs = Path("src/jobs.py").read_text(encoding="utf-8")
     app = Path("app.py").read_text(encoding="utf-8")
     bucket = Path("src/bucket.py").read_text(encoding="utf-8")
-    assert "from huggingface_hub import Volume, cancel_job" in jobs
+    assert "from huggingface_hub import cancel_job" in jobs
     assert "cancel_job(job_id=job_id, namespace=namespace, token=token)" in jobs
     assert '"cancel_requested": True' in app
     assert '"cancelled_at": cancelled_at' in app
@@ -44,4 +44,4 @@ def test_active_run_still_only_has_cancel_button():
 
 
 def test_app_version_v60():
-    assert '"version": "v60-cancel-persist-delete-run"' in Path("app.py").read_text(encoding="utf-8")
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")

tests/test_v61_stabilize_buttons_runtime.py

@@ -0,0 +1,48 @@
+from pathlib import Path
+
+
+def test_cancel_job_import_is_lazy_to_avoid_app_crash_on_cached_hub_versions():
+    jobs = Path("src/jobs.py").read_text(encoding="utf-8")
+    first_lines = "\n".join(jobs.splitlines()[:12])
+    assert "cancel_job" not in first_lines
+    assert "from huggingface_hub import cancel_job" in jobs
+    assert "huggingface_hub.cancel_job is unavailable" in jobs
+
+
+def test_custom_static_scripts_have_no_syntax_errors():
+    import subprocess
+    for path in Path("web/static").glob("*.js"):
+        result = subprocess.run(["node", "--check", str(path)], capture_output=True, text=True)
+        assert result.returncode == 0, result.stderr
+
+
+def test_frontend_loads_with_minimal_dom_without_throwing():
+    import subprocess
+    script = r"""
+const fs=require('fs');
+class El{
+  constructor(id){this.id=id;this.value='';this.textContent='';this.innerHTML='';this.dataset={};this.classList={add(){},remove(){},toggle(){}};this.style={setProperty(){}};this.attributes={};this.disabled=false;this.hidden=false;this.href='';this.open=false;this.clientHeight=100;this.scrollHeight=100;this.scrollTop=0;}
+  addEventListener(){}
+  setAttribute(k,v){this.attributes[k]=v}
+  removeAttribute(k){delete this.attributes[k]}
+  querySelectorAll(){return[]}
+  querySelector(){return new El('child')}
+  scrollIntoView(){}
+}
+const ids = Array.from(fs.readFileSync('web/index.html','utf8').matchAll(/id="([^"]+)"/g)).map(m=>m[1]);
+const map={};
+for(const id of ids) map[id]=new El(id);
+global.document={getElementById(id){return map[id]||null},querySelectorAll(sel){if(sel.includes('filter')) return ['all','running','failed'].map(s=>{let e=new El('filter');e.dataset={status:s};return e}); return [];}};
+global.window={setTimeout,setInterval,clearInterval};
+global.localStorage={getItem(){return null},setItem(){},removeItem(){}};
+global.confirm=()=>true;
+global.fetch=async()=>({ok:false,text:async()=>'forced error',json:async()=>({})});
+for(const f of ['web/static/components.js','web/static/api.js','web/static/progress.js','web/static/runs.js','web/static/app.js']) global.eval(fs.readFileSync(f,'utf8'));
+setTimeout(()=>process.exit(0), 25);
+"""
+    result = subprocess.run(["node", "-e", script], capture_output=True, text=True, timeout=5)
+    assert result.returncode == 0, result.stderr
+
+
+def test_app_version_v61():
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")

tests/test_v62_progress_perf_clean_delete.py

@@ -0,0 +1,44 @@
+from pathlib import Path
+
+
+def test_progress_endpoint_uses_light_bucket_bundle_and_optional_job_logs():
+    app = Path("app.py").read_text(encoding="utf-8")
+    bucket = Path("src/bucket.py").read_text(encoding="utf-8")
+    assert "include_heavy: bool = True" in bucket
+    assert "include_heavy=False" in app
+    assert "include_job_logs: bool = False" in app
+    assert "if include_job_logs:" in app
+    assert '"job_logs_polled": bool(include_job_logs)' in app
+
+
+def test_progress_polls_job_logs_less_often_from_frontend():
+    js = Path("web/static/app.js").read_text(encoding="utf-8")
+    assert "lastJobLogPoll" in js
+    assert ">12000" in js
+    assert "include_job_logs=1" in js
+    assert "state.lastJobLogPoll=0" in js
+
+
+def test_pi_stdout_display_and_api_payload_are_removed():
+    html = Path("web/index.html").read_text(encoding="utf-8")
+    progress = Path("web/static/progress.js").read_text(encoding="utf-8")
+    app = Path("app.py").read_text(encoding="utf-8")
+    assert "piTracePanel" not in html
+    assert "Pi process stdout" not in html
+    assert "renderPiLiveTrace" not in progress
+    assert "pi_live_log" not in progress
+    assert '"pi_live_log"' not in app
+
+
+def test_delete_selected_run_clears_ui_and_refreshes_runs():
+    runs = Path("web/static/runs.js").read_text(encoding="utf-8")
+    assert "async function deleteSelectedRun" in runs
+    assert "await apiDelete" in runs
+    assert "await loadRuns(true)" in runs
+    assert "No report selected." in runs
+    assert "No artifacts indexed yet." in runs
+    assert "Delete run from bucket" in runs
+
+
+def test_app_version_v62():
+    assert '"version": "v62-progress-perf-clean-delete"' in Path("app.py").read_text(encoding="utf-8")