chore(router, controlplane): add engine config hash mapping to node proto v1 - #2816
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feat/split-configs #2816 +/- ##
=======================================================
- Coverage 64.32% 40.89% -23.43%
=======================================================
Files 311 990 +679
Lines 44308 124191 +79883
Branches 4767 5666 +899
=======================================================
+ Hits 28502 50793 +22291
- Misses 15783 71700 +55917
- Partials 23 1698 +1675
🚀 New features to boost your workflow:
|
Router-nonroot image scan passed✅ No security vulnerabilities found in image: |
Adds the message type
ActiveGraphsto the node proto v1. It can be used to serialize / deserialize the upcomingmapper.jsonCDN files, so the router knows what to pull when using the new split config design.@coderabbitai summary
Checklist
Open Source AI Manifesto
This project follows the principles of the Open Source AI Manifesto. Please ensure your contribution aligns with its principles.