Keyword entità HTML: supporto input maiuscolo accentato + test hardening (v0.25.2)#65
Merged
Merged
Conversation
Recepiti due rilievi Copilot sulla PR #64: - htmlEntityKeywordVariants ora mappa anche À/È/É/Ì/Ò/Ù → la variante encoded viene generata pure con keyword maiuscola (es. CRIMINALITÀ) - test di regressione keyword: limit 10 → 100 (fragile all'ordinamento, 19 atti matchano) + nuovo caso maiuscolo Bump versione 0.25.2 nei 5 punti. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Questo PR è un follow-up al fix “keyword vs entità HTML” per i titoli Camera: estende la generazione delle varianti HTML-entity per gestire anche input maiuscolo accentato (es. CRIMINALITÀ) e irrobustisce il test di regressione, includendo anche il bump di versione a v0.25.2 nei punti di sync del progetto.
Changes:
- Estesa
htmlEntityKeywordVariantsper includere le vocali accentate maiuscole (À/È/É/Ì/Ò/Ù) sia nella mappa sia nel regex di sostituzione. - Harden del test di regressione per
bills --keyword:limitportato a 100 e aggiunto caso con input maiuscolo accentato. - Bump versione a 0.25.2 (package, manifest, server, worker, User-Agent) e aggiornamento
LOG.md.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/core/html-entity-variants.ts | Aggiunge supporto a entità HTML per vocali accentate maiuscole nella generazione varianti keyword. |
| src/tools/tools.test.ts | Rende il test meno fragile (limit 100) e aggiunge regressione per keyword maiuscola accentata. |
| src/core/client.ts | Aggiorna lo User-Agent alla nuova versione 0.25.2. |
| src/server.ts | Aggiorna la versione esposta dal server MCP a 0.25.2. |
| src/worker.ts | Aggiorna la versione esposta dal Worker a 0.25.2. |
| package.json | Bump versione npm package a 0.25.2. |
| manifest.json | Bump versione manifest a 0.25.2. |
| LOG.md | Aggiunge entry di release v0.25.2 con contesto del fix e test hardening. |
|
| Filename | Overview |
|---|---|
| src/core/html-entity-variants.ts | Adds uppercase accented vowel mappings to the HTML entity variant helper. |
| src/tools/tools.test.ts | Expands Camera bills keyword coverage for lowercase and uppercase accented input. |
| package.json | Updates the npm package version to 0.25.2. |
| manifest.json | Updates the DXT manifest version to 0.25.2. |
| src/core/client.ts | Updates the SPARQL request User-Agent version string. |
| src/server.ts | Updates the MCP server version metadata. |
| src/worker.ts | Updates the Worker metadata endpoint version. |
| LOG.md | Adds the v0.25.2 changelog entry. |
Reviews (1): Last reviewed commit: "fix(camera): keyword entità HTML anche c..." | Re-trigger Greptile
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.
Follow-up alla PR #64 (già mergiata), recepisce due rilievi Copilot arrivati dopo il merge.
Fix
Uppercase accentato —
htmlEntityKeywordVariantsmappava solo le vocali accentate minuscole. Con una keyword in maiuscolo (CRIMINALITÀ) la variante HTML-encoded non veniva generata (la À maiuscola non era nel regex): pur essendo il match SPARQLLCASE-insensitive, senza la varianteÀil confronto sul dato grezzocriminalitàfalliva. Aggiunte À/È/É/Ì/Ò/Ù a mappa e regex.Test hardening — il test di regressione usava
limit: 10, fragile: 19 atti matchanocriminalità, quindi al variare dell'ordinamentoac19_2696poteva uscire dalla finestra. Portato alimit: 100e aggiunto un caso con input maiuscolo.Verifica
tsc --noEmitOK, build CLI + Worker OKnpm test -- --run: 162/162 verdi (era 161 + 1 nuovo caso)Include il bump di versione 0.25.2 nei 5 punti di sync (release patch per questo bug ad alto impatto).
🤖 Generated with Claude Code