fix(lint): allow the v225 GOA snapshot (#227)

The reranker-token linter forbids bare 'vN' tokens in publishable prose to stop the
historical reranker shorthand creeping back in. Its GOA allowlist holds v160, v200,
v210, v215, v220, v226, v227, v229 and v230, and skips v225.

v225 is a real, loaded annotation_set (GOA 225, source_version '225' in the live
database) and it is the start of the validation window of the sealed frame: the model
is selected on v225 to v227 and measured on v227 to v230. Excluding it forces every
document that states the frame honestly to hide half of it. PROTEA's rewritten results
chapter tripped on exactly this line: 'validation window v225 to v227'.

The two guard tests that pin the allowlist to nine exact tokens are updated to ten.
v228 stays excluded, as its comment explains: it is a real release but outside the
slice spec, and the spec is the authority.

Verified: patched linter passes PROTEA's docs branch (was 1 offence) and still passes
PROTEA develop; agent-farm's own 7 offences are pre-existing under both versions.

Author: frapercan

scripts/lint-reranker-tokens.py

@@ -17,8 +17,8 @@
 bare `vN` tokens (schema versions, build numbers, semver-without-dot)
 unrelated to reranker shorthands. Six token classes are allowed:
 
-1. GOA snapshot identifiers: v160, v200, v210, v215, v220, v226, v227,
-   v229, v230
+1. GOA snapshot identifiers: v160, v200, v210, v215, v220, v225, v226,
+   v227, v229, v230
 2. Semver prefixes: any token immediately followed by ``.digit``
    (e.g. ``v0.3.0``, ``v2.1.13``).  These are package/tool release
    strings, not reranker shorthands.
@@ -62,7 +62,7 @@
 # via the reusable workflow; do not duplicate elsewhere.
 GOA_ALLOWLIST: frozenset[str] = frozenset({
     "v160", "v200", "v210", "v215", "v220",
-    "v226", "v227", "v229", "v230",
+    "v225", "v226", "v227", "v229", "v230",
 })
 
 # Method/scoring identifier that is NOT a reranker-version shorthand.

tests/test_lint_reranker_tokens.py

@@ -41,13 +41,13 @@ def _load_module():
 
 
 class TestAllowlist:
-    def test_allowlist_has_exactly_nine_tokens(self) -> None:
-        assert len(linter.GOA_ALLOWLIST) == 9
+    def test_allowlist_has_exactly_ten_tokens(self) -> None:
+        assert len(linter.GOA_ALLOWLIST) == 10
 
     def test_allowlist_exact_membership(self) -> None:
         expected = {
             "v160", "v200", "v210", "v215", "v220",
-            "v226", "v227", "v229", "v230",
+            "v225", "v226", "v227", "v229", "v230",
         }
         assert set(linter.GOA_ALLOWLIST) == expected