fix(senato-votes): derive legislature from --ddl-uri, not default 19 (#61)

* fix(senato-votes): derive legislature from --ddl-uri, not default 19

L'URI Senato dati.senato.it/ddl/{N} non codifica la legislatura, ma il
DDL espone osr:legislatura. Prima il filtro osr:legislatura era cablato
al default 19: --ddl-uri di un DDL di leg. diversa, senza --legislature
esplicito, dava un falso negativo silenzioso ("Nessuna votazione
trovata"). Ora la legislatura effettiva è derivata dal DDL e usata in
tutte le query, con fallback all'input. Allinea il comportamento a
votes --bill-code (Camera). +1 test di integrazione.

Chiude #60

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* docs(log): headline accurata — legislatura derivata dal DDL, non dall'URI

L'URI dati.senato.it/ddl/{N} non codifica la legislatura (identifica solo
il DDL); la legislatura è letta da osr:legislatura del DDL. Allinea
l'headline al corpo della voce.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* ci: rimuovi job live_integration (sempre rosso, zero segnale)

I test live interrogano dati.camera.it/dati.senato.it, che bloccano in
modo sistematico i runner GitHub (anti-bot/IP CI): il job falliva sempre
con 'SPARQL request failed after 5 attempts', senza distinguere una vera
regressione dal blocco di rete. Rete di sicurezza: testing live in locale
prima di ogni release (RELEASING.md).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Author: aborruso

.github/workflows/ci.yml

@@ -35,24 +35,11 @@ jobs:
       - name: Deterministic tests (no live SPARQL)
         run: npm test -- --run --exclude src/tools/tools.test.ts --exclude src/tools/sparql.test.ts
 
-  live_integration:
-    if: github.event_name == 'pull_request'
-    needs: ci
-    continue-on-error: true
-    runs-on: ubuntu-latest
-    timeout-minutes: 35
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: "20"
-          cache: npm
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Live integration tests (Camera/Senato endpoints)
-        run: npm test -- --run src/tools/sparql.test.ts src/tools/tools.test.ts
+  # I test live (src/tools/tools.test.ts, src/tools/sparql.test.ts) interrogano
+  # gli endpoint LOD dati.camera.it / dati.senato.it, che rifiutano in modo
+  # sistematico le richieste dai runner GitHub Actions (anti-bot / IP CI): il
+  # job falliva SEMPRE con "SPARQL request failed after 5 attempts", senza mai
+  # distinguere una vera regressione dal blocco di rete. Rimosso perché rumore
+  # a zero segnale. La rete di sicurezza è il testing live in locale prima di
+  # ogni release (RELEASING.md impone `npm test -- --run`, dove gli endpoint
+  # rispondono).

LOG.md

@@ -2,6 +2,7 @@
 
 ## 2026-07-11
 
+- **fix `senato-votes --ddl-uri` — legislatura derivata dal DDL (non dal default 19)** (#60, branch `fix/senato-votes-ddl-legislature`). Prima `--ddl-uri` di un DDL di legislatura ≠ 19, senza `--legislature` esplicito, dava un **falso negativo silenzioso** ("Nessuna votazione trovata"): il filtro `osr:legislatura` cablato al default 19 escludeva le sedute del DDL. L'URI Senato `dati.senato.it/ddl/{N}` non codifica la legislatura, ma il DDL espone `osr:legislatura`: ora quando c'è `--ddl-uri` la legislatura effettiva è derivata dal DDL (una query `<ddlUri> osr:legislatura ?leg`) e usata in tutte le query, con fallback all'input. Allinea il comportamento a `votes --bill-code` (Camera) che di default non vincola la legislatura. Ripro: DDL 52988 (S.1811, decreto lockdown DL 19/2020, leg.18) ora torna il voto `18-219-1` anche senza `--legislature`, con `rss_url` corretto su `/18`. +1 test di integrazione. Emerso dal run news-driven `docs/news-agent/2026-07-11_18-11.md`.
 - **v0.25.0** — release minor. Raccoglie **tutte** le PR mergiate dopo la v0.24.2 (#54–#59). Due nuove capacità utente: (1) **`speeches` filtro temporale** `--date-from`/`--date-to` + colonna `date` su entrambe le camere (#54, chiude #37); (2) **geografia di nascita normalizzata** `birth_province`/`birth_country`/`birth_region` su `senators`/`deputies`, regione derivata da SITUAS/ISTAT via `opensituas`, copertura 100% (#59, chiude #24/#23; corretto l'assunto errato "Senato solo città"). Più fix: **`bill-progress --branch C`** ora dà la timeline Camera (#55, chiude #41) e i follow-up di robustezza SPARQL/paginazione (#56/#57/#58). 43 tool invariati, 158/158 test verdi. Dettaglio nelle voci sotto.
 - **feat geografia di nascita normalizzata — provincia, nazione, regione su entrambe le camere** (#24/#23, branch `feat/birth-geography-province-region`). Corretto un **assunto sbagliato** (era anche nel codice e nel wiki): il Senato **non** espone "solo la città". Ri-verificato con `COUNT` su tutta la leg.19 (254 senatori): `osr:cittaNascita` 254/254, `osr:provinciaNascita` 250/254 (i 4 mancanti sono i nati all'estero), `osr:nazioneNascita` 254/254 (`Italia` o stato estero). `senators list` ora restituisce `birth_province`, `birth_country`, `birth_region`. **Regione derivata** una volta sola in `src/core/province-region.ts` (modulo TS bundle-safe) da SITUAS/ISTAT report 64 via CLI `opensituas`, date-accurate (union snapshot 2010 + 2020 + più recente per le province soppresse/rinominate, + 3 alias per le forme brevi del Senato): copertura 100% (0 province non risolte). **Camera** (`deputies list`): la geografia è solo uno slug nell'URI (`comune_provincia_regione`), non triple — ora decodificata in `birth_city`/`birth_province`/`birth_country`/`birth_region` con disambiguazione dello slug a 2 parti (Valle d'Aosta / Trentino-Alto Adige `comune_regione` vs estero `comune_stato`) e `birth_region` portata alla **stessa** forma canonica del Senato (verificato: "Veneto" identico tra camere). `birth_province`/`birth_country` restano nella forma nativa di ciascuna fonte (nessun canonicalizzatore province cross-camera). Wiki `dati-anagrafici.md`/`index.md` corretti; nota ai gestori Camera (geografia non strutturata) in `docs/note-gestori-lod/`. +10 test unitari (risorsa provincia→regione + parser slug Camera), 158/158 verdi.
 - **follow-up residuale commenti PR #56/#57 (`bill-progress`)**: migliorato l'hint del ramo Camera su risultato vuoto (ora segnala che i filtri `keyword`/intervallo di date o la paginazione `limit`/`offset` possono escludere tutte le righe), ordinamento timeline Camera reso stabile per paginazione con `?st` come tie-breaker in `ORDER BY` (non proiettato in `SELECT DISTINCT`, che ne altererebbe il dedup), escape robusto dei literal SPARQL per `keyword` (Senato + Camera) con helper dedicato che escapa solo `\ " newline CR` per la grammatica STRING_LITERAL2 (non `JSON.stringify`, che emette `\uXXXX` per i controlli, rifiutato da alcuni parser), e descrizioni `dateFrom/dateTo` chiarite sui due significati (presentazione Senato vs data stato-iter Camera).

src/tools/senato-votes.ts

@@ -138,6 +138,24 @@ export const senatoVotesTool: Tool<typeof inputSchema> = {
       ? `FILTER(?date <= "${input.dateTo}"^^xsd:date)`
       : "";
 
+    // --ddl-uri identifica univocamente il provvedimento e quindi la sua
+    // legislatura. L'URI (dati.senato.it/ddl/{N}) non la codifica, ma la
+    // proprietà osr:legislatura del DDL sì. Senza risolverla, il filtro di
+    // legislatura di default (19) esclude in silenzio i DDL di altre
+    // legislature (falso negativo: "Nessuna votazione trovata" pur essendo
+    // l'URI già non ambiguo). La deriviamo dal DDL e la usiamo in tutte le
+    // query; fallback all'input se il DDL non la espone.
+    let effectiveLeg = input.legislature;
+    if (input.ddlUri) {
+      const legRows = flattenBindings(
+        await snQuery(
+          `${OSR_PREFIXES}\nSELECT ?leg WHERE { <${input.ddlUri}> osr:legislatura ?leg } LIMIT 1`,
+        ),
+      );
+      const resolved = Number(legRows[0]?.leg);
+      if (Number.isInteger(resolved) && resolved > 0) effectiveLeg = resolved;
+    }
+
     // Supplemento fiducie per --keyword: le fiducie non hanno osr:oggetto,
     // quindi ?ddlTitolo resta unbound e il filtro keyword in SPARQL le esclude
     // anche quando il tema cercato è nel titolo del DDL citato per numero nel
@@ -160,7 +178,7 @@ export const senatoVotesTool: Tool<typeof inputSchema> = {
 SELECT DISTINCT ?v ?date ?numero ?tipo ?label ?esito
                 ?favorevoli ?contrari ?astenuti ?presenti ?votanti ?maggioranza
 WHERE {
-  ?v a osr:Votazione ; osr:legislatura ${input.legislature} ; osr:seduta ?s .
+  ?v a osr:Votazione ; osr:legislatura ${effectiveLeg} ; osr:seduta ?s .
   OPTIONAL { ?s osr:dataSeduta ?date }
   ?v rdfs:label ?label .
   FILTER(CONTAINS(LCASE(STR(?label)), "fiducia") &&
@@ -202,7 +220,7 @@ WHERE {
       const filter = nums.map((n) => `STR(?f) = "S.${n}"`).join(" || ");
       const fq = `${OSR_PREFIXES}
 SELECT ?ddl ?f ?titolo WHERE {
-  ?ddl a osr:Ddl ; osr:legislatura ${input.legislature} ; osr:fase ?f .
+  ?ddl a osr:Ddl ; osr:legislatura ${effectiveLeg} ; osr:fase ?f .
   OPTIONAL { ?ddl osr:titolo ?titolo }
   FILTER(${filter})
 }`;
@@ -232,7 +250,7 @@ SELECT ?ddl ?f ?titolo WHERE {
     if (input.ddlUri) {
       const datesQuery = `${OSR_PREFIXES}
 SELECT DISTINCT ?date WHERE {
-  ?v a osr:Votazione ; osr:legislatura ${input.legislature} ; osr:seduta ?s .
+  ?v a osr:Votazione ; osr:legislatura ${effectiveLeg} ; osr:seduta ?s .
   ?s osr:dataSeduta ?date .
   ?v osr:oggetto ?o . ?o osr:relativoA <${input.ddlUri}> .
 }`;
@@ -258,7 +276,7 @@ SELECT DISTINCT ?date WHERE {
     // a --ddl-uri, che richiede la risoluzione via fallback: in quel caso il
     // conteggio è la cardinalità del result-set filtrato (calcolata sotto).
     if (input.countOnly && !input.ddlUri) {
-      const countWhere = [`?v a osr:Votazione ; osr:legislatura ${input.legislature} .`];
+      const countWhere = [`?v a osr:Votazione ; osr:legislatura ${effectiveLeg} .`];
       if (ddlTopicPattern) countWhere.push(ddlTopicPattern);
       if (needsLabel) countWhere.push(`?v rdfs:label ?label . ${labelFilter}`);
       if (input.dateFrom || input.dateTo)
@@ -281,7 +299,7 @@ SELECT DISTINCT ?date WHERE {
                 ?favorevoli ?contrari ?astenuti ?presenti ?votanti ?maggioranza
                 ?ddl ?oggetto
 WHERE {
-  ?v a osr:Votazione ; osr:legislatura ${input.legislature} ; osr:seduta ?s .
+  ?v a osr:Votazione ; osr:legislatura ${effectiveLeg} ; osr:seduta ?s .
   OPTIONAL { ?s osr:dataSeduta ?date }
   ${ddlTopicPattern}
   ${needsLabel ? `?v rdfs:label ?label . ${labelFilter}` : "OPTIONAL { ?v rdfs:label ?label }"}
@@ -354,7 +372,7 @@ WHERE {
       const filter = nums.map((n) => `STR(?f) = "S.${n}"`).join(" || ");
       const fbQuery = `${OSR_PREFIXES}
 SELECT ?ddl ?f WHERE {
-  ?ddl a osr:Ddl ; osr:legislatura ${input.legislature} ; osr:fase ?f .
+  ?ddl a osr:Ddl ; osr:legislatura ${effectiveLeg} ; osr:fase ?f .
   FILTER(${filter})
 }`;
       const byFase = new Map<string, string>();
@@ -478,7 +496,7 @@ SELECT ?ddl ?f WHERE {
     const rows = values.map((v) => {
       const ddlUris = splitMulti(v.ddl_uri);
       const ddlHtmlUrls = ddlUris.map((u) => actHtmlUrl(u)).filter(Boolean);
-      const rssUrls = ddlUris.map((u) => ddlRssUrl(u, input.legislature)).filter(Boolean);
+      const rssUrls = ddlUris.map((u) => ddlRssUrl(u, effectiveLeg)).filter(Boolean);
       return {
         ...v,
         ddl_count: String(ddlUris.length),

src/tools/tools.test.ts

@@ -829,6 +829,24 @@ describe("Senato tools", () => {
     expect(fiducia!.bill_number).toBe("1944");
   }, 30000);
 
+  it("senato-votes: --ddl-uri resolves the legislature from the DDL, not the default 19", async () => {
+    // DDL 52988 = S.1811 (decreto lockdown DL 19/2020), legislatura 18. L'URI
+    // dati.senato.it/ddl/{N} non codifica la legislatura: senza risolverla dal
+    // DDL, il filtro di default (19) darebbe un falso negativo silenzioso.
+    // Nessun --legislature passato: deve comunque trovare il voto di leg.18.
+    const result = await senatoVotesTool.execute({
+      legislature: 19,
+      ddlUri: "http://dati.senato.it/ddl/52988",
+      limit: 100,
+      offset: 0,
+    });
+    const voto = result.rows.find((r) => r.uri.endsWith("/18-219-1"));
+    expect(voto).toBeDefined();
+    expect(voto!.ddl_uri).toBe("http://dati.senato.it/ddl/52988");
+    // rss_url deve puntare alla legislatura del DDL (18), non al default 19.
+    expect(voto!.rss_url).toContain("/52988/18");
+  }, 30000);
+
   it("senato-votes: backfills bill_number from the resolved DDL on generic labels", async () => {
     // 19-376-2 (Corte dei Conti): label "Votazione finale" senza numero, ma
     // ddl_uri risolto via osr:relativoA → bill_number = fase S.1457.