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

e2e compare: setClassMetadata extraction regex never matches (ɵɵ vs ɵ) — classMetadataDiffs is dead code

Aperta
  • Qualcosa non va?
#561 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Brooooooklyn ci sta già lavorando.

Dal 8/10/2026.

  • #562 di @Brooooooklyn — aperta

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
38/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
testing-qa

Direzione di ricerca

Start in src/compare.ts at extractClassMetadataCalls (around line 2328) and normalizeMetadataString, then run the e2e compare harness to inspect the 79 surfaced fixture mismatches. Fixing the regex and normalization is only the start: classify residual diffs as real emit bugs or noise, and update KnownDifference per fixture, including edge-cases/aliased-injectable. Done means class metadata diffs are detected, allowlisted explicitly where appropriate, and the comparison passes without hiding real mismatches.

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

Descrizione

The e2e compare harness never actually compares setClassMetadata: extractClassMetadataCalls (src/compare.ts:2328) matches i\d+\.ɵɵsetClassMetadata but both compilers emit i0.ɵsetClassMetadata (single ɵ), so classMetadataDiffs is always empty and testFixture's known-difference gate never sees them.

Correcting the regex surfaces 79 fixture mismatches. Sampling shows most are normalization noise — same decorator payloads differing in quote style, indentation and trailing commas — so normalizeMetadataString needs to be made syntax-insensitive (e.g. parse and compare the AST or normalize quotes/commas) before the check can be enabled.

Steps: fix the regex, strengthen normalizeMetadataString, triage the residual diffs (real emit bugs vs. remaining noise), and extend KnownDifference to allowlist classMetadataDiffs per fixture — the new edge-cases/aliased-injectable entry should then declare its extra setClassMetadata explicitly.

Surfaced by a Codex review comment on #560.

Lingua principale
Rust
Stelle
232
Fork
21
Merge medio
1g 18h
PR unite (30g)
59

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 voidzero-dev/oxc-angular-compiler

Tutte le issue di voidzero-dev/oxc-angular-compiler

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.