1 files changed, 36 insertions, 21 deletions
diff --git a/packages/meshbay-hub/tests/test_asset_versioning.py b/packages/meshbay-hub/tests/test_asset_versioning.py
index 3d84d99..0c93f5a 100644
--- a/packages/meshbay-hub/tests/test_asset_versioning.py
+++ b/packages/meshbay-hub/tests/test_asset_versioning.py
@@ -23,7 +23,7 @@ import re
 import pytest
 from fastapi.testclient import TestClient
 
-from meshbay_hub.api.webapp import _ASSETS, ASSET_V, STATIC_DIR, _asset_version
+from meshbay_hub.api.webapp import ASSET_V, STATIC_DIR, _asset_version
 from meshbay_hub.app import create_app
 
 
@@ -103,26 +103,41 @@ def test_the_fingerprint_follows_the_content(tmp_path, monkeypatch):
     assert _asset_version() == before, "the fingerprint is not reproducible"
 
 
-def test_every_static_script_participates_in_the_fingerprint():
+@pytest.mark.parametrize("relative", ["locales/en.js", "vendor/htm-preact.js", "style.css"])
+def test_a_change_anywhere_under_the_prefix_moves_the_fingerprint(relative):
     """
-    `_ASSETS` is hand-maintained, and forgetting an entry fails silently: the
-    file is imported by the page, so the browser fetches it, but it does not
-    feed the content hash — so a change confined to that one file ships at the
-    URL a cache already holds. Nothing errors, and the symptom is a fix that
-    "doesn't work" on exactly the machines that visited before.
+    Everything under the static directory is served at `/a/<hash>/` and cached
+    as immutable, so everything has to feed the hash — subdirectories included.
 
-    Found by `source-merge.js`, added to the Search view and left out of the
-    list. It happened to be harmless that day because `search-page.js` changed
-    in the same commit and *is* listed — which is the worst way for this to go
-    unnoticed. The checklist in docs/apps.md §4 step 5 names this trap; this
-    enforces it instead of relying on remembering.
-
-    `sw.js` is the one deliberate exclusion — the service worker is served
-    unversioned on purpose (`test_the_service_worker_is_not_versioned`).
+    The fingerprint used to cover a hand-kept list of top-level modules, and
+    the check that guarded the list globbed `*.js` at the top level only. The
+    catalogues and `vendor/` were on neither: a change confined to the English
+    catalogue shipped at the URL a phone already held, and it went on showing a
+    heading that had been rewritten and deployed. Pull-to-refresh fetches the
+    shell, which is `no-store` and was current; it does not refetch an
+    immutable file whose URL has not moved.
     """
-    on_disk = {p.name for p in STATIC_DIR.glob("*.js")} - {"sw.js"}
-    missing = sorted(on_disk - set(_ASSETS))
-    assert not missing, (
-        f"static scripts missing from webapp._ASSETS: {missing}. A change to "
-        "one of these will not move the asset URL, so a browser that cached "
-        "the page keeps running the old copy")
+    before = _asset_version()
+    target = STATIC_DIR / relative
+    original = target.read_bytes()
+    try:
+        target.write_bytes(original + b"\n/* touched */\n")
+        assert _asset_version() != before, (
+            f"the fingerprint did not move when {relative} did, so a browser "
+            "that cached it keeps the old copy for a year")
+    finally:
+        target.write_bytes(original)
+    assert _asset_version() == before
+
+
+def test_a_new_file_moves_the_fingerprint():
+    """A rename or an addition changes what is served even when no existing
+    file's bytes do."""
+    before = _asset_version()
+    extra = STATIC_DIR / "locales" / "zz-test-only.js"
+    try:
+        extra.write_text("export default {};\n")
+        assert _asset_version() != before
+    finally:
+        extra.unlink()
+    assert _asset_version() == before