Skip to content

Route Groq helpers through llm-providers#62

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

Route Groq helpers through llm-providers#62
stackbilt-admin merged 1 commit into
mainfrom
fix/issue-24-groq-provider-helpers

Conversation

@stackbilt-admin
Copy link
Copy Markdown
Member

Summary

  • routes askGroq and askGroqJson through @stackbilt/llm-providers
  • keeps the existing helper API for router, titles, memory/cognition, scheduled helpers, and composite planner calls
  • updates Groq tests to assert provider-factory requests instead of raw chat-completion fetches
  • leaves askGroqWithLogprobs on the raw Groq endpoint because llm-providers does not expose logprob request options yet

Refs #24. This removes the common raw Groq text/JSON inference helper path; remaining raw-provider cleanup is narrowed to logprobs, composite direct Anthropic/Workers calls, and a few scheduled Workers AI helpers.

Validation

  • npm run typecheck
  • npx vitest run tests/groq.test.ts tests/router.test.ts tests/dispatch.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 d6807bf into main Jun 2, 2026
2 checks passed
@stackbilt-admin stackbilt-admin deleted the fix/issue-24-groq-provider-helpers branch June 2, 2026 15:07
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