feat(core): validazione dati e metriche di qualità

[ronzim]

Closes #17

Cosa aggiunge

Nuovo modulo packages/core/src/quality.ts, esportato da index.ts:

• validateDataset(ds, { maxIssues }): controlla punti, voci della timeline e punti dei path. Segnala coordinate non finite o fuori range (lat ±90, lng ±180), timestamp mancanti o non validi e intervalli con fine prima dell'inizio. Restituisce il conteggio per motivo e i primi N problemi con indice.
• sanitizeDataset(ds): restituisce un nuovo dataset senza punti e voci non validi; dai path toglie solo i punti non validi. Non modifica l'input, come i filtri.
• computeQualityMetrics(ds, { gapThresholdMinutes, topGaps }):
  • distribuzione dell'accuratezza (min, mediana, p90, max e fasce ≤10/50/100/500/>500 m)
  • buchi nella timeline oltre la soglia, che tiene conto delle sovrapposizioni; sui punti se la timeline è vuota
  • copertura dei path delle activity: è la metrica che avrebbe intercettato Loader timeline-semantic: i timelinePath vengono scartati e i path delle attività sono vuoti #22

CLI: --quality stampa validazione e metriche, --sanitize toglie gli elementi non validi prima di filtri ed export.

Test

• 16 nuovi test in quality.test.ts, tra cui il test di regressione per Loader timeline-semantic: i timelinePath vengono scartati e i path delle attività sono vuoti #22: copertura dei path al 100% sulla fixture con la struttura reale
• 2 test CLI per --quality e --sanitize
• In totale 175/175 ok; tsc ok; ESLint senza nuovi warning

Note

• Le metriche ignorano i timestamp non validi, che vengono comunque segnalati da validateDataset.
• Non c'è ancora integrazione nell'app desktop: si potrebbe mostrare la qualità nella sidebar, in una issue a parte.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CecUKcJiSVnehcZEiXLgtk

[Copilot]

🟡 Changes recommended

sanitizeDataset removes points/entries but keeps the original dateRange, deviating from the filters' convention of recomputing it and leaving a potentially stale range observable via computeSummary.

2 open findings

• Sanitize leaves stale dateRange after removing invalid entries · New
• Singular removal message uses incorrect plural form · New

What changed in this PR

This PR implements issue #17 by adding a new data-quality module to packages/core. It introduces validateDataset (flags out-of-range/non-finite coordinates, missing/invalid timestamps, reversed intervals, and invalid activity path points), sanitizeDataset (returns a new dataset with invalid items removed, mirroring the filters' immutability), and computeQualityMetrics (accuracy distribution, overlap-aware timeline gaps, and activity path coverage — the metric intended to surface loader bugs like #22). The functions are wired into the public API and the CLI via new --quality and --sanitize flags, and are covered by new unit and CLI tests.

Changes:

• New quality.ts module with validation, sanitization, and quality-metrics functions, re-exported from index.ts.
• CLI gains --quality (prints validation + metrics) and --sanitize (drops invalid items before filtering/export), with a dedicated printQuality formatter.
• 18 new tests (quality + CLI), including a #22 regression asserting 100% activity path coverage on the real-format fixture.

File	Description
packages/​core/​src/​quality.ts	New module: validateDataset, sanitizeDataset, computeQualityMetrics and supporting types/helpers.
packages/​core/​src/​cli.ts	Adds --quality and --sanitize flags, printQuality output, and sanitize step in the load pipeline.
packages/​core/​src/​index.ts	Re-exports the new quality API and types.
packages/​core/​tests/​quality.test.ts	Unit tests for validation, sanitization, and metrics incl. #22 regression.
packages/​core/​tests/​cli.test.ts	CLI tests for --quality and --sanitize.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.