fix(senato-votes): ddl_title + fiducia seduta diversa; fix(camera-amendments): fallback ostr#69
Merged
Merged
Conversation
…mendments): fallback indice ostr - senato-votes: nuova colonna ddl_title, risolve il titolo del provvedimento collegato anche quando ddl_uri punta a un osr:Documento (non solo osr:Ddl), es. risoluzioni di commissione su schemi di intesa autonomia differenziata - senato-votes: --ddl-uri include correttamente la fiducia quando cade su una seduta diversa dal voto forte collegato (link diretto osr:oggetto) - camera-amendments: fallback all'indice ostr per-atto sugli atti storici senza link diretto emendamenti (es. leg.18, AC 2402 decreto Covid 2020) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
| Filename | Overview |
|---|---|
| src/tools/senato-votes.ts | Adds linked-title output and expands Senato vote lookup paths for title keywords and confidence votes. |
| src/tools/camera-amendments.ts | Adds the historical Camera amendment index fallback and lets fallback fetch failures propagate. |
| src/tools/tools.test.ts | Adds coverage for Senato title lookup, confidence vote inclusion, and Camera historical amendment fallback. |
Reviews (3): Last reviewed commit: "fix(senato-votes): --keyword matcha anch..." | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Questo PR migliora l’affidabilità e l’utilità dei tool di interrogazione votazioni Senato ed emendamenti Camera, riducendo i “vuoti silenziosi” e aggiungendo contesto (titolo provvedimento) dove i label sono ambigui.
Changes:
senato-votes: aggiunge colonnaddl_title(daosr:titoloBrevecon fallbackosr:titolo) e aggiorna la logica--ddl-uriper includere fiducie su sedute/date diverse dal voto “fortemente” collegato.camera-amendments: aggiunge fallback all’indiceapps/emendamenti/ostr/{leg}quando la scheda atto storica non contiene più link diretti agli emendamenti.- Aggiorna test di integrazione e documentazione (LOG + lod-wiki) per coprire regressioni reali.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/tools/senato-votes.ts | Aggiunge ddl_title e amplia la risoluzione date per --ddl-uri includendo fiducie su sedute diverse. |
| src/tools/camera-amendments.ts | Introduce fallback su indice ostr per atti storici senza link diretto in scheda. |
| src/tools/tools.test.ts | Nuovi test di integrazione per ddl_title, fiducia su seduta diversa, e fallback ostr Camera. |
| LOG.md | Entry di changelog per le modifiche del 2026-07-17. |
| docs/lod-wiki/senato/fiducia-fallback.md | Documenta il bug e il fix per fiducia su seduta diversa. |
| docs/lod-wiki/camera/assenti.md | Documenta la perdita del link diretto in schede storiche e la strategia ostr. |
…docs: documenta residuo fiducia-suffisso Da review Greptile PR #69: - camera-amendments: il fallback all'indice ostr copriva un fetch fallito come "nessun emendamento" (.catch(() => "")); rimosso, ora propaga l'errore HTTP invece di restituire un vuoto ingannevole. - fiducia-fallback.md: documentato il residuo noto sul match per numero esatto (suffisso di lettura incluso) — deliberatamente non allargato al numero base per non rischiare falsi positivi su letture diverse. Il terzo rilievo (pagination) è un falso positivo: la paginazione avviene già prima della query ddl_title in entrambi i path (server-side LIMIT/OFFSET o slice TS post-filtro --ddl-uri), stesso ordine già usato dal backfill bill_number preesistente. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…sr:titolo Da review Copilot PR #69: il match tematico sul DDL/documento collegato guardava solo osr:titolo, incoerente con ddl_title (che usa titoloBreve come prima scelta) — una ricerca poteva mancare un voto il cui unico match era nel titolo breve. Aggiunto osr:titoloBreve come terza alternativa OR nella query principale e nel supplemento fiducie. +1 test dedicato; verificato via query SPARQL diretta (stesso pattern, risultati corretti) e via il test esistente sulle fiducie (stesso percorso di codice, passato). Suite live non rieseguibile: WAF Senato in throttle per burst di query dirette durante la verifica manuale. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
senato-votes: nuova colonnaddl_title—ddl_uriera già risolto anche quando puntava a unosr:Documento(es. risoluzioni di commissione), ma il titolo non era mai esposto in output. Emerso verificando la seduta 19-438 del 16/7/2026 (autonomia differenziata): il voto sul Veneto aveva la regione nellabel, gli altri tre (Liguria/Lombardia/Piemonte) erano identificabili solo con query SPARQL manuale sul documento collegato. Oraddl_titleè popolato daosr:titoloBreve(fallbackosr:titolo), senza richiederea osr:Ddlnel BGP (iDocumentorestavano esclusi altrimenti).senato-votes --ddl-uri: fiducia su seduta diversa dal voto forte — la risoluzione delle date del DDL si basava solo sul link direttoosr:oggetto/osr:relativoA; quando la fiducia cadeva su una seduta diversa (es. decreto sicurezza 2025 S.1509: pregiudiziale 3/6, fiducia 4/6), spariva senza errore. Ora include anche le fiducie della legislatura il cui numero nel label corrisponde allaosr:fasedel DDL cercato.camera-amendments: fallback all'indice ostr sugli atti storici — la scheda atto delle legislature vecchie (es. leg.18) spesso non incorpora il link diretto agli emendamenti; il tool restituiva 0 righe senza errore pur esistendo gli emendamenti alla fonte (verificato su AC 2402, decreto Covid 2020: 54+53 emendamenti). Ora prova l'indice per-atto prima di concludere "nessun emendamento".Test plan
tsc --noEmitpulitonpm test -- --run: 172/172 verdi (16 file)ddl_titleverificato anche via CLI reale sulla seduta 19-438 (Liguria/Lombardia/Piemonte/Veneto risolti correttamente)docs/lod-wiki/senato/fiducia-fallback.md,docs/lod-wiki/camera/assenti.md) aggiornati🤖 Generated with Claude Code