fix(review): riferimento corretto alla soglia e test fedele al client

- il commento sulla query compatta rimandava a SENATO_MAX_OR_TERMS, che
  riguarda il chunking: il vincolo qui è SENATO_MAX_REQUEST_URI
- l'asserzione sulla dimensione usava encodeURIComponent su un frammento;
  ora sta accanto agli helper, misura con senatoRequestUriLength (stesso
  encoding del client) su una query intera con i suoi prefissi

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: aborruso

src/core/chunk.test.ts

@@ -19,18 +19,6 @@ describe("chunk", () => {
     expect(chunk([], 25)).toEqual([]);
   });
 
-  it("tiene la query sotto la soglia di ~2 KB del Senato", () => {
-    // 53 fiducie in leg. 19: la OR-chain intera supera i 2 KB di query string
-    // e l'endpoint risponde 403. A blocchi di 25 ogni query resta ben sotto.
-    const nums = Array.from({ length: 53 }, (_, i) => String(1000 + i));
-    const orChain = (batch: string[]) =>
-      batch.map((n) => `STR(?f) = "S.${n}"`).join(" || ");
-    expect(encodeURIComponent(orChain(nums)).length).toBeGreaterThan(2000);
-    for (const batch of chunk(nums, 25)) {
-      expect(encodeURIComponent(orChain(batch)).length).toBeLessThan(1300);
-    }
-  });
-
   it("rifiuta una dimensione non valida", () => {
     expect(() => chunk([1, 2], 0)).toThrow();
   });

src/core/senato-query-size.test.ts

@@ -1,9 +1,12 @@
 import { describe, it, expect } from "vitest";
 import {
+  SENATO_MAX_OR_TERMS,
   SENATO_MAX_REQUEST_URI,
   senatoRequestUriLength,
   assertQueryFits,
 } from "./senato-query-size.js";
+import { chunk } from "./chunk.js";
+import { OSR_PREFIXES } from "./prefixes.js";
 
 describe("senato-query-size", () => {
   it("misura la request-URI con l'encoding usato dal client", () => {
@@ -23,6 +26,22 @@ describe("senato-query-size", () => {
     );
   });
 
+  it("la OR-chain delle fiducie sfora intera, a blocchi di SENATO_MAX_OR_TERMS no", () => {
+    // 53 fiducie in leg. 19: è il caso reale che mandava --keyword in 403.
+    const nums = Array.from({ length: 53 }, (_, i) => String(1000 + i));
+    const query = (batch: string[]) => `${OSR_PREFIXES}
+SELECT ?ddl ?f WHERE {
+  ?ddl a osr:Ddl ; osr:legislatura 19 ; osr:fase ?f .
+  FILTER(${batch.map((n) => `STR(?f) = "S.${n}"`).join(" || ")})
+}`;
+    expect(senatoRequestUriLength(query(nums))).toBeGreaterThan(
+      SENATO_MAX_REQUEST_URI,
+    );
+    for (const batch of chunk(nums, SENATO_MAX_OR_TERMS)) {
+      expect(() => assertQueryFits(query(batch))).not.toThrow();
+    }
+  });
+
   it("il confine è esattamente 2047 accettata / 2048 rifiutata", () => {
     // padding calibrato sulla lunghezza reale, non stimato
     const fill = (target: number) => {

src/tools/senato-votes.ts

@@ -465,8 +465,8 @@ SELECT DISTINCT ?date ?label WHERE {
     // post-filtro (altrimenti il LIMIT taglierebbe prima del filtro).
     const paginate = input.ddlUri ? "" : `LIMIT ${input.limit}\nOFFSET ${input.offset}`;
     // Query scritta compatta di proposito: indentazione e a capo finiscono
-    // nella request-URI, che oltre 2047 byte viene respinta con 403 (vedi
-    // SENATO_MAX_OR_TERMS). La keyword entra tre volte — label, titolo e
+    // nella request-URI, che oltre SENATO_MAX_REQUEST_URI byte viene respinta
+    // con 403 (di qui assertQueryFits, sotto). La keyword entra tre volte — label, titolo e
     // titoloBreve del DDL — quindi la lunghezza cresce di ~3 volte la keyword:
     // il margine recuperato qui è ciò che permette le keyword multi-parola.
     const coreSelect = `SELECT DISTINCT ?v ?date ?numero ?tipo ?label ?esito ?favorevoli ?contrari ?astenuti ?presenti ?votanti ?maggioranza ?ddl ?oggetto