Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

feat(review): expand grain-key not_null detector to fire on any contracted-model column change

Aperta
  • Qualcosa non va?
#1,034 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
48/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
typescript
Ambito
data-engineering

Direzione di ricerca

Inizia in packages/opencode/src/altimate/review/dbt-patterns.ts, in extractGrainKeyGaps e nel filtro grainDeclChangedEntities. Ricostruisci come newMap e oldMap identificano le aggiunte, quindi esamina come vengono rappresentate le modifiche alle colonne di contracted-model schema.yml. Il lavoro è completato quando il rilevatore conserva il percorso esistente della dichiarazione di grain e valuta anche le colonne aggiunte o modificate che si sovrappongono alla copertura esistente delle grain key, senza ampliare inutilmente le modifiche non correlate.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Deferred from the v0.9.3 pre-release review.

Current behavior

extractGrainKeyGaps in packages/opencode/src/altimate/review/dbt-patterns.ts fires only when the grain declaration itself changes, filtered by grainDeclChangedEntities — which iterates newMap and compares against oldMap for additions.

Gap

A PR that adds a new column to an existing contracted model without touching unique_combination_of_columns will not surface uncovered grain columns, even when new columns increase the exposure surface. The scoping is conservative and correct for the initial ship, but leaves recall on the table.

Proposal

Extend the change-scoping filter to also fire when any column added/modified in a contracted model's schema.yml overlaps with existing grain-key coverage. Keep the current "grain declaration changed" path as one of two entry conditions.

Why deferred

Scope expansion touches filter semantics and could shift the false-positive rate; the initial detector is intentionally conservative and documented, so a follow-up release is the right cadence for widening.

Lingua principale
TypeScript
Stelle
815
Fork
135
Merge medio
1g 12h
PR unite (30g)
53

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di AltimateAI/altimate-code

Tutte le issue di AltimateAI/altimate-code

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.