feat(multiprovider): execute child provider hooks during evaluation - #2005
feat(multiprovider): execute child provider hooks during evaluation#2005jonathannorris wants to merge 1 commit into
Conversation
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2005 +/- ##
============================================
+ Coverage 92.52% 92.92% +0.40%
- Complexity 728 774 +46
============================================
Files 60 62 +2
Lines 1739 1908 +169
Branches 202 218 +16
============================================
+ Hits 1609 1773 +164
+ Misses 80 79 -1
- Partials 50 56 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|



Summary
MultiProviderHookExecutor, which runs the before/after/error/finally stages per child, mirroring the JS SDK'sHookExecutor.beforehooks run in registration order;after,error, andfinallyAfterrun in reverse, per spec.MultiProviderexposes a provider-level hook that captures theClientMetadataand hook hints from the SDK lifecycle so they can be passed into child hook contexts. This mirrors the JS SDK'sWeakMapapproach; here it's aThreadLocalsnapshot captured before the strategy runs, so parallel strategies work correctly.Related PRs
This is one of three independent PRs that together close the multi-provider gaps identified in #1882. They branch off
mainseparately and can be reviewed and merged in any order. Together they replace #1897.ComparisonStrategyRelates to #1882