fix(cli): capabilityScore si difende da query vuota, no-match su stderr

Da review Copilot su PR #48 (estensioni coerenti dei fix precedenti):
- capabilityScore e' esportata: normalizza e valida la query
  internamente (vuota -> 0, trim+lowercase, match case-insensitive
  sui terms) invece di affidarsi alla guardia del solo chiamante which
- il messaggio "nessun comando trovato" usciva su stdout: spostato su
  stderr come il messaggio d'uso (stdout riservato a risultati/JSON)

+2 test (query vuota/spazi -> 0, case-insensitive). 130/130 verdi,
tsc pulito. Verificato: which zzz-inesistente 2>/dev/null -> stdout
vuoto, exit 2.

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

Author: aborruso

src/cli.ts

@@ -1635,7 +1635,8 @@ const whichCmd = defineCommand({
     if (args.json) {
       process.stdout.write(JSON.stringify(ranked) + "\n");
     } else if (ranked.length === 0) {
-      process.stdout.write(
+      // Su stderr: stdout resta per risultati/JSON (pipeline-friendly).
+      process.stderr.write(
         `Nessun comando trovato per "${q}". Prova 'italianparliament guide' per il flusso completo, o '--help'.\n`,
       );
     } else {

src/core/capabilities.test.ts

@@ -41,4 +41,16 @@ describe("capabilityScore", () => {
     const cap = CAPABILITIES.find((c) => c.cmd === "search find")!;
     expect(capabilityScore(cap, "zzz-inesistente")).toBe(0);
   });
+
+  it("query vuota o di soli spazi dà 0 (la funzione si difende da sola, è esportata)", () => {
+    const cap = CAPABILITIES.find((c) => c.cmd === "search find")!;
+    expect(capabilityScore(cap, "")).toBe(0);
+    expect(capabilityScore(cap, "   ")).toBe(0);
+  });
+
+  it("match case-insensitive sui terms", () => {
+    const cap = CAPABILITIES.find((c) => c.cmd === "person-career show")!;
+    expect(capabilityScore(cap, "CARRIERA")).toBe(100);
+    expect(capabilityScore(cap, "  Carriera  ")).toBe(100);
+  });
 });

src/core/capabilities.ts

@@ -42,9 +42,15 @@ export const CAPABILITIES: Capability[] = [
 ];
 
 
-export function capabilityScore(cap: Capability, q: string): number {
+// Normalizza e valida la query internamente: la funzione è esportata e deve
+// difendersi da sola — con q vuota `t.includes("")` sarebbe sempre vero e
+// ogni capability otterrebbe punteggio. Match case-insensitive sui terms.
+export function capabilityScore(cap: Capability, query: string): number {
+  const q = query.toLowerCase().trim();
+  if (!q) return 0;
   let s = 0;
-  for (const t of cap.terms) {
+  for (const term of cap.terms) {
+    const t = term.toLowerCase();
     if (t === q) s = Math.max(s, 100);
     else if (t.includes(q)) s = Math.max(s, 70);
     else if (q.includes(t)) s = Math.max(s, 60);