fix: PARSE_ERROR 500 on stats/export; tighten CLI and state tests

Author: clean6378-max-it

api/export_api.py

@@ -293,7 +293,7 @@ def export_session(project_name, session_id):
         return error_response(
             ErrorCode.PARSE_ERROR,
             "Failed to parse session",
-            400,
+            500,
         )
 
     rules = current_app.config.get("EXCLUSION_RULES") or []

api/sessions.py

@@ -69,7 +69,7 @@ def get_session_stats(project_name, session_id):
         return error_response(
             ErrorCode.PARSE_ERROR,
             "Failed to parse session",
-            400,
+            500,
         )
 
     try:

static/js/shared/state.test.js

@@ -1,12 +1,20 @@
-import { describe, it, expect, afterEach } from 'vitest';
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { state } from './state.js';
 
+function resetState() {
+    state.currentProject = null;
+    state.cachedSessions = [];
+    state.projectDisplayNames = {};
+    state.navInProgress = false;
+}
+
 describe('state', () => {
+    beforeEach(() => {
+        resetState();
+    });
+
     afterEach(() => {
-        state.currentProject = null;
-        state.cachedSessions = [];
-        state.projectDisplayNames = {};
-        state.navInProgress = false;
+        resetState();
     });
 
     it('initializes with null current project', () => {

tests/test_cli_e2e.py

@@ -66,11 +66,19 @@ def test_cli_stats_exits_zero(tmp_path):
     assert proc.returncode == 0
 
 
-def test_cli_invalid_since_exits_nonzero():
-    proc = _run_cli(["--since", "yesterday"])
+def test_cli_invalid_since_exits_nonzero(tmp_path):
+    base = _seed_base_dir(tmp_path)
+    proc = _run_cli([
+        "export",
+        "--since",
+        "yesterday",
+        "--base-dir",
+        str(base),
+    ])
     assert proc.returncode != 0
     assert "--since" in proc.stderr
     assert "invalid choice" in proc.stderr.lower()
+    assert "export" in proc.stderr
 
 
 def test_cli_export_creates_output(tmp_path):

tests/test_error_propagation.py

@@ -154,7 +154,7 @@ def _boom(*args, **kwargs):
         monkeypatch.setattr("api.sessions.parse_session", _boom)
 
         resp = client.get("/api/sessions/proj/abc/stats")
-        assert resp.status_code == 400
+        assert resp.status_code == 500
         body = resp.get_json()
         assert body.get("error") == "Failed to parse session"
         assert body.get("code") == "PARSE_ERROR"