fix(vote-detail): escape del letterale SPARQL + test sul path execute

Da review PR #80:
- la sigla risolta finiva interpolata cruda nel FILTER: ora passa per
  sparqlStringLiteral, estratto in src/core (era duplicato in bill-progress)
  con i suoi test. Il valore viene dal grafo, non dall'utente, ma una
  virgoletta nel dato romperebbe comunque la query.
- l'hint sul caso "nessuna sigla registrata" citava il flag --group-acronym,
  che per un client MCP non significa nulla: formulazione neutra.
- due test di integrazione su voteDetailTool.execute: match risolto (sigla
  minuscola → righe, nessun hint) e sigla di groups list → vuoto con hint.

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

Author: aborruso

src/core/sparql-literal.test.ts

@@ -0,0 +1,24 @@
+import { describe, it, expect } from "vitest";
+import { sparqlStringLiteral } from "./sparql-literal.js";
+
+describe("sparqlStringLiteral", () => {
+  it("avvolge il valore tra virgolette", () => {
+    expect(sparqlStringLiteral("FDI")).toBe('"FDI"');
+  });
+
+  it("scappa le virgolette interne", () => {
+    expect(sparqlStringLiteral('AL"T')).toBe('"AL\\"T"');
+  });
+
+  it("scappa i backslash", () => {
+    expect(sparqlStringLiteral("A\\B")).toBe('"A\\\\B"');
+  });
+
+  it("scappa i ritorni a capo", () => {
+    expect(sparqlStringLiteral("a\nb\rc")).toBe('"a\\nb\\rc"');
+  });
+
+  it("lascia intatti gli altri caratteri, accenti compresi", () => {
+    expect(sparqlStringLiteral("M-+EUR è così")).toBe('"M-+EUR è così"');
+  });
+});

src/core/sparql-literal.ts

@@ -0,0 +1,20 @@
+// Escapa solo i caratteri che la grammatica STRING_LITERAL2 vieta (\, ",
+// newline, CR). Non usare JSON.stringify: emette \uXXXX per i caratteri di
+// controllo, sequenza che alcuni parser SPARQL rifiutano nel corpo del
+// letterale (o cercano un valore diverso), rompendo il filtro invece di fare
+// il match atteso.
+const SPARQL_ESC: Record<string, string> = {
+  "\\": "\\\\",
+  '"': '\\"',
+  "\n": "\\n",
+  "\r": "\\r",
+};
+
+/**
+ * Serializza un valore come string literal SPARQL, virgolette comprese.
+ * Serve ovunque un valore finisca dentro una query: anche i valori che
+ * arrivano dal grafo (etichette, sigle) possono contenere virgolette o
+ * backslash e renderebbero la query invalida.
+ */
+export const sparqlStringLiteral = (value: string): string =>
+  `"${value.replace(/[\\"\n\r]/g, (c) => SPARQL_ESC[c] ?? c)}"`;

src/tools/bill-progress.ts

@@ -6,22 +6,9 @@ import { decodeHtml } from "../core/decode-html.js";
 import { htmlEntityKeywordVariants } from "../core/html-entity-variants.js";
 import { ddlRssUrl } from "../core/html-url.js";
 import { currentLegislature } from "../core/current-legislature.js";
+import { sparqlStringLiteral } from "../core/sparql-literal.js";
 import type { Tool } from "./types.js";
 
-// Costruisce un letterale-stringa SPARQL valido escapando solo i caratteri che
-// la grammatica STRING_LITERAL2 vieta (\, ", newline, CR). Non usare
-// JSON.stringify: emette \uXXXX per i caratteri di controllo, sequenza che
-// alcuni parser SPARQL rifiutano nel corpo del letterale (o cercano un valore
-// diverso), rompendo il filtro keyword invece di fare il match atteso.
-const SPARQL_ESC: Record<string, string> = {
-  "\\": "\\\\",
-  '"': '\\"',
-  "\n": "\\n",
-  "\r": "\\r",
-};
-const sparqlStringLiteral = (value: string): string =>
-  `"${value.replace(/[\\"\n\r]/g, (c) => SPARQL_ESC[c] ?? c)}"`;
-
 const inputSchema = z.object({
   ddlUri: z
     .string()

src/tools/tools.test.ts

@@ -452,6 +452,30 @@ describe("Camera tools", () => {
     expect(result.rows[0]).toHaveProperty("group_acronym");
   }, 30000);
 
+  it("vote-detail: --group-acronym resolves against the sigle present in the vote (#77)", async () => {
+    // Sigla scritta in minuscolo: deve risolvere e filtrare, non restare vuota.
+    const result = await voteDetailTool.execute({
+      voteUri: "http://dati.camera.it/ocd/votazione.rdf/vs19_456_018",
+      groupAcronym: "fdi",
+      limit: 700,
+    });
+    expect(result.rows.length).toBeGreaterThan(0);
+    expect(new Set(result.rows.map((r) => r.group_acronym))).toEqual(new Set(["FDI"]));
+    expect(result.hint).toBeUndefined();
+  }, 30000);
+
+  it("vote-detail: sigla di groups list non presente nel voto → vuoto con hint, non silenzio (#77)", async () => {
+    const result = await voteDetailTool.execute({
+      voteUri: "http://dati.camera.it/ocd/votazione.rdf/vs19_456_018",
+      groupAcronym: "AZ-PER-RE",
+      limit: 700,
+    });
+    expect(result.rows.length).toBe(0);
+    expect(result.hint).toContain("AZ-PER-RE");
+    // L'elenco viene dalla query, non da una lista scritta nel codice.
+    expect(result.hint).toContain("APERRE");
+  }, 30000);
+
   it("group-members: returns members for legislature 19", async () => {
     const result = await groupMembersTool.execute({ legislature: 19, limit: 3, offset: 0 });
     expect(result.rows.length).toBe(3);

src/tools/vote-detail.ts

@@ -3,6 +3,7 @@ import { cdQuery } from "../core/client.js";
 import { OCD_PREFIXES } from "../core/prefixes.js";
 import { flattenBindings } from "../core/flatten.js";
 import { personHtmlUrl } from "../core/html-url.js";
+import { sparqlStringLiteral } from "../core/sparql-literal.js";
 import type { Tool } from "./types.js";
 
 const inputSchema = z.object({
@@ -76,7 +77,7 @@ function similarity(a: string, b: string): number {
 /** Nota mostrata quando la sigla richiesta non esiste in quella votazione. */
 export function buildGroupAcronymHint(requested: string, available: string[]): string {
   if (available.length === 0) {
-    return `Nessuna sigla di gruppo registrata in questa votazione: il filtro --group-acronym non è applicabile.`;
+    return `Nessuna sigla di gruppo registrata in questa votazione: il filtro per gruppo non è applicabile.`;
   }
   const norm = normalizeAcronym(requested);
   const ranked = [...available].sort(
@@ -135,7 +136,7 @@ export const voteDetailTool: Tool<typeof inputSchema> = {
       if (!resolved) {
         return { rows: [], columns, hint: buildGroupAcronymHint(input.groupAcronym, available) };
       }
-      filters.push(`?v ocd:siglaGruppo ?_sg . FILTER(STR(?_sg) = "${resolved}")`);
+      filters.push(`?v ocd:siglaGruppo ?_sg . FILTER(STR(?_sg) = ${sparqlStringLiteral(resolved)})`);
     }
     if (input.voteType) filters.push(`FILTER(?type = "${input.voteType}")`);