agents: add nsolid-version and add it to metadata#460
agents: add nsolid-version and add it to metadata#460santigimeno wants to merge 1 commit intonode-v24.x-nsolid-v6.xfrom
Conversation
Also, cache the metadata so it's not generated multiple times per different rpc's. Finally, resolve having both `nsolid-saas` and `nsolid-saas-token` header name and settle for the first.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (11)
WalkthroughThis PR refactors gRPC metadata handling by consolidating separate Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Review rate limit: 3/5 reviews remaining, refill in 20 minutes and 28 seconds. Comment |
Also, cache the metadata so it's not generated multiple times per different rpc's.
Finally, resolve having both
nsolid-saasandnsolid-saas-tokenheader name and settle for the first.Summary by CodeRabbit
Refactor
Tests