fix(azure): add azure-anthropic support to router, evaluator, copilot, and tokenization#3158
fix(azure): add azure-anthropic support to router, evaluator, copilot, and tokenization#3158waleedlatif1 merged 7 commits intostagingfrom
Conversation
…, and tokenization
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile OverviewGreptile Summary
Confidence Score: 3/5
Important Files Changed
Sequence DiagramsequenceDiagram
autonumber
participant UI as Workflow UI/Executor
participant RBH as RouterBlockHandler
participant EBH as EvaluatorBlockHandler
participant GR as /api/guardrails/validate
participant VH as validateHallucination()
participant PR as /api/providers
participant AZA as azure-anthropic provider
UI->>RBH: Execute Router block
RBH->>PR: POST providerRequest (providerId + creds)
PR-->>RBH: Provider response (route decision)
UI->>EBH: Execute Evaluator block
EBH->>PR: POST providerRequest (providerId + creds)
PR-->>EBH: Provider response (scores)
UI->>GR: POST guardrails_validate (input + model + providerCredentials)
GR->>VH: validateHallucination(input)
VH->>PR: executeProviderRequest(providerId, request)
PR->>AZA: If providerId=azure-anthropic, call Azure Anthropic SDK
AZA-->>PR: LLM response
PR-->>VH: content/tokens
VH-->>GR: score/reasoning
GR-->>UI: validation result
|
…anslate missing bedrockAccessKeyId
|
@greptile |
|
@cursor review |
Additional Comments (2)
This needs to guard the spread (e.g.,
In Update the description to a provider-agnostic Azure endpoint (matching the new shared usage). |
|
@greptile |
|
@cursor review |
- Pass all provider credentials unconditionally in router, evaluator (matching agent pattern) - Remove conditional if-branching on providerId for credential fields - Thread workspaceId through guardrails → hallucination validator for BYOK key resolution - Remove getApiKey() from hallucination validator, let executeProviderRequest handle it - Resolve vertex OAuth credentials in hallucination validator matching agent handler pattern Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@greptile |
|
@cursor review |
Additional Comments (1)
Fix by only attaching the credential fields for the active provider (as the code did before this PR), or by ensuring the |
Summary
getProviderCredentialSubBlocks()so router and evaluator blocks show the Azure endpoint fieldazureEndpointin router and evaluator handlers for azure-anthropic providerType of Change
Testing
Tested manually
Checklist