Skip to content

Route Claude executor through llm-providers#60

Merged
stackbilt-admin merged 1 commit into
mainfrom
fix/issue-24-claude-provider
Jun 2, 2026
Merged

Route Claude executor through llm-providers#60
stackbilt-admin merged 1 commit into
mainfrom
fix/issue-24-claude-provider

Conversation

@stackbilt-admin

Copy link
Copy Markdown
Member

Summary

  • routes executeClaudeChat and executeClaudeChatStream through @stackbilt/llm-providers instead of direct Anthropic fetch/SSE parsing
  • preserves the existing AEGIS tool loop for in-process tools and MCP tools using provider-native toolCalls / toolResults
  • forwards edgeEnv through Claude and GPT-OSS tool execution so dynamic tools can execute under provider-backed paths
  • adds focused regression coverage for Claude provider tool rounds and streaming callback behavior

Refs #24. This completes the Anthropic/D.1 executor migration slice; the broader epic should remain open for legacy composite/scheduled/raw provider cleanup.

Validation

  • npm run typecheck
  • npx vitest run tests/claude-provider.test.ts tests/executors.test.ts
  • npm test
  • npx charter validate
  • git -c core.whitespace=blank-at-eol,blank-at-eof,space-before-tab,cr-at-eol diff --check
  • npm pack --dry-run

@stackbilt-admin stackbilt-admin merged commit 8fe4dc8 into main Jun 2, 2026
2 checks passed
@stackbilt-admin stackbilt-admin deleted the fix/issue-24-claude-provider branch June 2, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant