cursor.decoder.released logs Attr.OPERATION (hyp_operation) while its cursor.decoder.started counterpart logs a literal operation key

Aperta
  • Qualcosa non va?
#2,099 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à
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
48/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
observability

Direzione di ricerca

Read hypaware-core/plugins-workspace/cursor/src/listener.js:76 and hypaware-core/plugins-workspace/cursor/src/native_decoder.js:110, then compare the event attributes with listener.js:134 and :143. Done means the started/released pair uses the same operation-key convention and released carries component and status; the issue says this is backlog work and needs explicit delegation.

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

Descrizione

Source: #2097 at cd0d42d4ff03dc3c4e6439f8cef9dc6f43d76212
Location: hypaware-core/plugins-workspace/cursor/src/listener.js:76; hypaware-core/plugins-workspace/cursor/src/native_decoder.js:110
Severity: cosmetic

Evidence

Verified at head: listener.js:76 emits cursor.decoder.released with { [Attr.OPERATION]: 'recovery.read' } and Attr.OPERATION is 'hyp_operation' (src/core/observability/attrs.js:102), while native_decoder.js:110 emits cursor.decoder.started with the literal { component: 'plugin.cursor', operation: 'recovery.read' }. The spawn/release pair cannot be filtered or correlated on one attribute key, and released carries no component and no status unlike its neighbours at listener.js:134 and :143.

Why deferral is safe

Cosmetic observability inconsistency on the isolated deflake commit; no behavior is wrong and the test that consumes the line matches on the event name, not the attributes. Each file follows its own local convention, so either side could be called the odd one out; aligning them is cleanup, not a fix this PR owes.

Acceptance condition

The cursor.decoder.started / cursor.decoder.released pair shares one attribute-key convention (both Attr.* or both literal), and released carries component and status like its neighbours; a grep of the two lines shows the same operation key.

Review preference; backlog only. A human may explicitly delegate it with neutral:fix.

Lingua principale
JavaScript
Stelle
11
Fork
0
Merge medio
2h 55m
PR unite (30g)
294

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 hyparam/hypaware

Tutte le issue di hyparam/hypaware

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.