chore: remove export_count legacy alias from export state API (#66)

* chore: remove export_count legacy alias from export state API

Complete the export_count migration by removing the duplicate field from GET /api/export/state and updating the bundled SPA, tests, and docs to use last_export_session_count only.

* test: assert export_count absent from export state response

Address PR review: add negative assertions so re-introducing the legacy alias fails the suite, and reference PR #60 in the CHANGELOG removal note.

* docs: clarify pre-first-tag bundled SPA removal in deprecation policy

Document path B: 0.1.0.dev0 has no git tag yet; export_count removal in [Unreleased] is acceptable for bundled SPA+API atomic deploy. Update example table and CHANGELOG wording per review.

Author: clean6378-max-it

CHANGELOG.md

@@ -17,8 +17,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 - README notes that the server enforces the debug + host safety rule at startup
 
-### Deprecated
+### Removed
 
-- `export_count` on `GET /api/export/state` (documented only; still returned). Use `last_export_session_count`. Removal planned in a follow-up release per [deprecation policy](docs/deprecation-policy.md).
+- `export_count` on `GET /api/export/state` — use `last_export_session_count` (deprecated in PR #60; removed before first `v0.1.0` tag per [deprecation policy](docs/deprecation-policy.md) bundled SPA path; SPA updated in same `[Unreleased]` cut)
 
 [Unreleased]: https://github.com/cppalliance/claude-code-chat-browser/compare/f70505982d435f8b1f754cb18c0c9f65609f11b4...HEAD

api/export_api.py

@@ -69,7 +69,6 @@ def get_export_state() -> FlaskReturn:
             "last_export_time": state.get("lastExportTime"),
             # Sessions exported in the last completed bulk export (not a lifetime total).
             "last_export_session_count": n,
-            "export_count": n,
         }
     )
 

docs/api-reference.md

@@ -326,13 +326,11 @@ Read-only snapshot of bulk-export state persisted under `~/.claude-code-chat-bro
 |-------|------|-----------|-------------|
 | `last_export_time` | string \| null | stable | ISO timestamp of last completed bulk export |
 | `last_export_session_count` | integer | stable | Sessions in last bulk export run |
-| `export_count` | integer | deprecated | Legacy alias of `last_export_session_count`; prefer `last_export_session_count` in new code (still returned for SPA compatibility; removal per [deprecation-policy.md](deprecation-policy.md)) |
 
 ```json
 {
   "last_export_time": "2026-05-20T18:42:11.123456",
-  "last_export_session_count": 17,
-  "export_count": 17
+  "last_export_session_count": 17
 }
 ```
 

docs/deprecation-policy.md

@@ -27,11 +27,17 @@ A deprecated field may be removed when:
 
 For fields actively read by the bundled SPA (which does not track an external API version), removal happens no earlier than **two tagged releases** after the release that documented the deprecation in CHANGELOG (for example, deprecated in `0.1.0`, removable from `0.3.0` at earliest when versions advance `0.1.0` → `0.2.0` → `0.3.0`), and no earlier than **14 calendar days** after that deprecation announcement.
 
-## Example (in progress)
+### Bundled SPA + pre-first-tag (path B)
 
-| Field | Endpoint | Status | Replacement |
-|-------|----------|--------|-------------|
-| `export_count` | `GET /api/export/state` | deprecated | `last_export_session_count` |
+Until the first git tag ships (for example while `app.__version__` is `0.1.0.dev0`), the CHANGELOG `[Unreleased]` section is the source of truth — see [Versioning](#versioning) below. The bundled SPA is deployed from the same repo and commit as the API; it does not consume a separately versioned HTTP contract.
+
+For fields **only read by the bundled SPA** (no external integrators on a tagged API yet), an atomic PR that (1) stops reading the field in `static/js/*.js` and (2) removes it from the JSON response is acceptable **before** the first tag, provided deprecation was announced in CHANGELOG and api-reference (PR #60) and removal is recorded under `[Unreleased]`. External integrators who pin a **tagged** release must follow the two-release + 14-day rule above.
+
+## Example (completed — pre-first-tag)
+
+| Field | Endpoint | Status | Replacement | Notes |
+|-------|----------|--------|-------------|-------|
+| `export_count` | `GET /api/export/state` | removed | `last_export_session_count` | Deprecated in PR #60 (`[Unreleased]`); removed in bundled SPA+API PR before first `v0.1.0` tag |
 
 ## Versioning
 

static/js/projects.js

@@ -61,7 +61,7 @@ export async function showProjects() {
                     const d = new Date(exportState.last_export_time);
                     if (!isNaN(d.getTime())) {
                         hasPreviousExport = true;
-                        const sessionCount = Math.max(0, parseInt(exportState.last_export_session_count ?? exportState.export_count ?? 0, 10) || 0);
+                        const sessionCount = Math.max(0, parseInt(exportState.last_export_session_count ?? 0, 10) || 0);
                         lastExportHtml = `<p class="text-muted text-sm">Last export: ${d.toLocaleString()} (${sessionCount} sessions in last export)</p>`;
                     }
                 }

tests/test_api_routes.py

@@ -70,7 +70,8 @@ def test_export_state_defaults(client_empty):
     resp = client_empty.get("/api/export/state")
     assert resp.status_code == 200
     body = resp.get_json()
-    assert "export_count" in body
+    assert "last_export_session_count" in body
+    assert "export_count" not in body
 
 
 def test_bulk_export_empty_projects_returns_422(client_empty):

tests/test_export_api_bulk.py

@@ -86,4 +86,4 @@ def test_export_state_json_fields(isolated_state):
     assert resp.status_code == 200
     body = resp.get_json()
     assert body["last_export_session_count"] == 5
-    assert body["export_count"] == 5
+    assert "export_count" not in body