Skip to content

Execute dynamic tools through llm-providers#61

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

Execute dynamic tools through llm-providers#61
stackbilt-admin merged 1 commit into
mainfrom
fix/issue-24-dynamic-tools-provider

Conversation

@stackbilt-admin
Copy link
Copy Markdown
Member

Summary

  • routes runtime dynamic-tool execution through buildLLMProviderFactory(env)
  • removes direct env.ai.run() and raw Groq fetch inference from dynamic-tools.ts
  • preserves dynamic-tool executor choices by mapping gpt_oss, workers_ai, and groq to their configured provider models
  • adds focused provider-boundary tests for dynamic-tool execution and availability errors

Refs #24. This removes another public raw-provider inference surface; the broader epic remains open for router/probe/composite/scheduled cleanup.

Validation

  • npm run typecheck
  • npx vitest run tests/dynamic-tools-provider.test.ts tests/claude-provider.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 85bf810 into main Jun 2, 2026
2 checks passed
@stackbilt-admin stackbilt-admin deleted the fix/issue-24-dynamic-tools-provider branch June 2, 2026 14:58
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