Skip to content

Extract shared validateLoadOptions helper#28

Merged
corasan merged 1 commit into
mainfrom
t3code/edb5037f
Apr 24, 2026
Merged

Extract shared validateLoadOptions helper#28
corasan merged 1 commit into
mainfrom
t3code/edb5037f

Conversation

@corasan
Copy link
Copy Markdown
Owner

@corasan corasan commented Apr 24, 2026

Summary

  • Add a generic validateLoadOptions<T> helper in package/src/runtime.ts that guards on undefined and wraps onProgress with createSafeCallback.
  • Collapse validateSTTLoadOptions, validateTTSLoadOptions, and validateEmbeddingsLoadOptions into one-line delegations to the helper.
  • Route validateLLMLoadOptions through the helper, layering the existing tool-definition validation on top.
  • Public API (signatures and return types) unchanged — no caller updates needed.

Test plan

  • bun typescript passes
  • bun test package/src/runtime.test.ts — 6 pass, 0 fail

- Deduplicate onProgress callback wrapping across LLM, STT, Embeddings, and TTS load option validators
@corasan corasan changed the title PR content Extract shared validateLoadOptions helper Apr 24, 2026
@corasan corasan merged commit 582b2ee into main Apr 24, 2026
3 checks passed
@corasan corasan deleted the t3code/edb5037f branch April 24, 2026 12:22
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