fix(export): address PR review — parity asserts, stderr warnings, JSONL newlinefix(export): address PR review — parity asserts, stderr warnings, JSONL newline

Author: clean6378-max-it

scripts/export.py

@@ -427,7 +427,7 @@ def cmd_export(args):
     skipped_mtime_unchanged = 0
 
     def _on_export_error(sid: str, exc: Exception) -> None:
-        print(f"  Warning: failed to export {sid}: {exc}")
+        print(f"  Warning: failed to export {sid}: {exc}", file=sys.stderr)
 
     collect_sink = ListSink()
     export_result = run_bulk_export(

tests/test_export_engine_parity.py

@@ -92,14 +92,15 @@ def test_http_post_export_matches_cli_no_zip(tmp_path: Path, monkeypatch) -> Non
     assert resp.status_code == 200
 
     zf = zipfile.ZipFile(io.BytesIO(resp.data))
-    http_md = zf.read(
-        next(n for n in zf.namelist() if n.endswith(".md"))
-    ).decode("utf-8")
+    http_md_names = [n for n in zf.namelist() if n.endswith(".md")]
+    assert len(http_md_names) == 1, f"expected one .md in zip, got {http_md_names}"
+    http_md = zf.read(http_md_names[0]).decode("utf-8")
     http_manifest = [
         json.loads(line)
         for line in zf.read("manifest.jsonl").decode("utf-8").splitlines()
         if line.strip()
     ]
+    assert len(http_manifest) == 1, f"expected one manifest row, got {len(http_manifest)}"
 
     out_dir = tmp_path / "cli_out"
     proc = _run_cli([

utils/export_engine.py

@@ -106,7 +106,7 @@ def add_session(
 
     def finalize(self, manifest: list[dict[str, Any]]) -> None:
         if manifest:
-            manifest_str = "\n".join(json.dumps(e, default=str) for e in manifest)
+            manifest_str = "\n".join(json.dumps(e, default=str) for e in manifest) + "\n"
             self._zf.writestr("manifest.jsonl", manifest_str)