Skip to content

Remove final raw Groq inference path#65

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

Remove final raw Groq inference path#65
stackbilt-admin merged 1 commit into
mainfrom
issue-24-logprobs-provider

Conversation

@stackbilt-admin
Copy link
Copy Markdown
Member

Summary

  • removes the final raw Groq chat-completions endpoint from groq.ts
  • keeps askGroqWithLogprobs API-compatible for router callers by using provider-backed JSON classification
  • maps tokenConfidence to self-reported confidence until llm-providers grows explicit logprob request support
  • updates Groq tests to cover provider-backed confidence classification

Fixes #24.

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 8177c42 into main Jun 2, 2026
2 checks passed
@stackbilt-admin stackbilt-admin deleted the issue-24-logprobs-provider branch June 2, 2026 15: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.

Phase D — migrate aegis-oss to @stackbilt/llm-providers (remove all bolted-in LLM logic)

1 participant