OEV-851 Send dualBroadcast params in TxMessage#378
Conversation
📊 API Diff Results
|
|
This PR is stale because it has been open 30 days with no activity. |
|
This PR has been automatically closed because it has been stale for > 30 days. |
57275e6 to
f7794ae
Compare
f7794ae to
6ef5bf5
Compare
📊 API Diff Results
|
There was a problem hiding this comment.
Pull request overview
This PR extends the TXM Beholder svr.v1.TxMessage emission to include dual-broadcast URL params (when the transaction is marked as dual-broadcast), enabling downstream analysis of those params.
Changes:
- Populate
DualBroadcastParamson emittedsvr.v1.TxMessagewhenTxMeta.DualBroadcast == true. - Add unit tests asserting
DualBroadcastParamsis set/unset appropriately. - Bump
github.com/smartcontractkit/chainlink-protos/svrdependency tov1.2.0.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/txm/metrics.go | Adds conditional wiring of DualBroadcastParams into emitted TxMessage. |
| pkg/txm/metrics_test.go | Adds coverage for presence/absence of DualBroadcastParams based on tx meta. |
| go.mod | Updates chainlink-protos/svr to a version that includes the new TxMessage field. |
| go.sum | Updates checksums for the chainlink-protos/svr version bump. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Requires
smartcontractkit/chainlink-protos#302
What
See title
Why
To store these params for analysis.