Skip to content

fix(deepinfra): reconcile reasoning controls#2134

Open
rekram1-node wants to merge 1 commit into
devfrom
audit/deepinfra-reasoning
Open

fix(deepinfra): reconcile reasoning controls#2134
rekram1-node wants to merge 1 commit into
devfrom
audit/deepinfra-reasoning

Conversation

@rekram1-node

Copy link
Copy Markdown
Collaborator

Summary

  • audit every configured DeepInfra model against the live catalog, official reasoning documentation, OpenAPI schema, and model API pages
  • expose DeepInfra's documented reasoning toggle and low/medium/high effort controls on all reasoning models
  • remove reasoning options from non-reasoning models and remove undocumented xhigh values

Validation

  • bun validate
  • git diff --check
  • resolved DeepInfra invariant audit: 25 models, 21 reasoning, 4 non-reasoning, no invalid option combinations
  • bun test packages/core/test/generate.test.ts: 9 pass, 1 unrelated repository-wide failure for missing weights links in NVIDIA/Sarvam/Cohere metadata

Audit notes

  • all 25 configured models remain present in DeepInfra's public model catalog
  • no DeepInfra API credential was available, so authenticated behavior testing was not performed
  • no token-budget control is documented by DeepInfra
  • prior DeepInfra PR feat(deepinfra): add reasoning options #2083 is merged; no open DeepInfra PR existed before this PR

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