Skip to content

feat(fireworks): add FireworksImageModelOptions type with providerOptions validation#12558

Open
saakshigupta2002 wants to merge 1 commit intovercel:mainfrom
saakshigupta2002:feat/fireworks-image-model-options
Open

feat(fireworks): add FireworksImageModelOptions type with providerOptions validation#12558
saakshigupta2002 wants to merge 1 commit intovercel:mainfrom
saakshigupta2002:feat/fireworks-image-model-options

Conversation

@saakshigupta2002
Copy link

Summary

Adds a proper FireworksImageModelOptions type definition for Fireworks image model provider options, backed by a Zod schema and validated at runtime using parseProviderOptions.

This addresses the lack of type safety for Fireworks image generation provider options. Previously, providerOptions.fireworks was spread directly into the request body without any validation.

Changes

  • packages/fireworks/src/fireworks-image-options.ts: Added Zod schema (fireworksImageModelOptions) defining all supported Fireworks image generation options (cfg_scale, steps, negative_prompt, strength, scheduler, safety_checker, output_format, safety_tolerance). Exported inferred FireworksImageModelOptions type.
  • packages/fireworks/src/fireworks-image-model.ts: Integrated parseProviderOptions to validate provider options against the Zod schema at runtime, replacing unvalidated ...(providerOptions.fireworks ?? {}) spread. Provider options are now consumed via individual property assignments.
  • packages/fireworks/src/fireworks-image-model.test.ts: Updated tests to use schema-defined options instead of arbitrary pass-through properties.
  • packages/fireworks/src/index.ts: Exported FireworksImageModelOptions type.
  • examples/ai-functions/src/generate-image/fireworks.ts: Updated example to import and use satisfies FireworksImageModelOptions for type-safe provider options.

Verification

  • Build passes (pnpm turbo build --filter=@ai-sdk/fireworks)
  • Lint passes (pnpm turbo lint --filter=@ai-sdk/fireworks)
  • All existing tests pass (33/33)

Closes #12439

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.

Add image model options types for Fireworks provider

1 participant