Skip to content

Commit 5f782e8

Browse files
xxshubhamxxclaude
andcommitted
fix(observability): mark BuildUpdate as sent so it emits once per run, not per test/hook [SDK-6574]
The one-shot guard `if(!this.haveSentBuildUpdate && ...)` never flipped `haveSentBuildUpdate`, so the BuildUpdate event (a byte-identical, build-level `observability_version` payload) re-fired on every test/hook event — hundreds to thousands of identical events per build. Set the flag right after the event is sent so it is emitted once per reporter instance. Complements consumer-side dedupe TRAP-4033. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 0ff7157 commit 5f782e8

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎bin/testObservability/reporter/index.js‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -577,6 +577,7 @@ class MyReporter {
577577
}
578578
};
579579
await uploadEventData(buildUpdateData);
580+
this.haveSentBuildUpdate = true;
580581
}
581582

582583
// Add started hooks to the hash

0 commit comments

Comments
 (0)