feat: metadata#27
Conversation
WalkthroughThis PR bumps the package version to 1.6.0 and adds metadata propagation and optional topic filtering to the evaluation system. Two new optional fields— Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/index.ts (1)
269-279: Backward-compat path doesn’t forwardallowedTopics.
EvaluationProxyAPIRequestSchemanow supportsallowedTopics, but the backward-compat payload omits it, so the feature won’t work for direct-message callers. Add the mapped field to the request body.✅ Suggested fix
topic_scoping_target: evaluationProxyAPIRequest.topicScopingTarget ?? evaluationProxyAPIRequest.topic_scoping_target, + allowed_topics: evaluationProxyAPIRequest.allowedTopics, metadata: evaluationProxyAPIRequest.metadata,
Description of change
Pull-Request Checklist
mainbranchnpm run lintpasses with this changenpm run testpasses with this changeFixes #0000Summary by CodeRabbit
New Features
Chores
✏️ Tip: You can customize this high-level summary in your review settings.