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

node-integration-tests: drop static pins from non-tracing suites

Aperta
#24,141 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
74/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
node.js, typescript
Ambito
testing-qa

Direzione di ricerca

Inizia con la ricetta di porting della issue padre, quindi cerca traceLifecycle: 'static' nelle directory delle suite indicate di node-integration-tests. Rimuovi il pin dalle suite non-tracing indicate sotto “pin removal only”; lascia i sette gruppi di riscrittura delle assertion per un lavoro separato. Il lavoro è concluso quando queste suite non contengono più il pin statico non necessario e le suite indicate che verificano l’envelope rimangono invariate.

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

Descrizione

javascript Task

Follow the porting recipe in the parent issue.

Largest group by file count and the smallest by effort. Roughly 90 suite directories carry the traceLifecycle: 'static' pin without ever asserting on a transaction envelope, so the pin just needs to go.

Suites (pin removal only)

  • public-api/* except the span surface dirs owned by the trace semantics sub-issue: addBreadcrumb, captureException, captureMessage, configureScope, LocalVariables, logger, metrics, OnUncaughtException, setContext, setExtra, setExtras, setTag, setTags, setUser, withMonitor, withScope
  • sessions, cron/*, client-reports/periodic-send
  • Logging: pino, winston, consola, integrations/console/filter
  • Runtime: anr, system-error, thread-blocked-native, child-process, modules, esm/*, contextLines/*, breadcrumbs/process-thread, node-runtime-metrics, bun-runtime-metrics
  • vercel/sigterm-flush

Suites that still need an assertion rewrite

These 7 do assert on transaction envelopes:

  • public-api/onUnhandledRejectionIntegration
  • client-reports/drop-reasons/* (4 dirs)
  • featureFlags/featureFlagsIntegration/onSpan and featureFlags/growthbook/onSpan
  • fs-instrumentation
  • aws-serverless/aws-integration (fold aws-serverless/aws-integration-streamed back into it)

Splitting this issue into a mechanical pin-removal PR and a smaller rewrite PR is fine.

Lingua principale
TypeScript
Stelle
8.7k
Fork
1.9k
Merge medio
1g 18h
PR unite (30g)
562

Guida per i contributori

Apri la guida per i contributori

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 getsentry/sentry-javascript

Tutte le issue di getsentry/sentry-javascript

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.