Skip to content

chore(prompt-registry): Use mini zod - #2244

Open
davidkna-sap wants to merge 1 commit into
mainfrom
davidkna-sap/mini-zod
Open

chore(prompt-registry): Use mini zod#2244
davidkna-sap wants to merge 1 commit into
mainfrom
davidkna-sap/mini-zod

Conversation

@davidkna-sap

Copy link
Copy Markdown
Member

Context

Closes SAP/ai-sdk-js-backlog#ISSUENUMBER.
This should reduce bundled-application size for consumers and increase startup speed:

What this PR does and why it is needed

@hyperspace-pr-bot

Copy link
Copy Markdown
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Chore: Migrate Prompt Registry to Zod Mini

What this PR does and why it is needed

This PR updates the Prompt Registry package to use Zod Mini (zod/mini) instead of the standard Zod library for generated validation schemas.

The change is made in the orval.config.js configuration by setting the zod output variant to mini with version 4:

zod: {
  variant: 'mini',
  version: 4
}

The corresponding generated file prompt-registry.zod.ts has been regenerated to use the Zod Mini API.

Benefits:

  • 🌳 Better tree-shaking support, reducing bundled application size for consumers
  • ⚡ Faster startup speed due to a smaller runtime footprint

References:


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.31.9

  • Output Template: Repository PR Template
  • Correlation ID: 1c727980-a6e2-11f1-8e09-e48c334fa6d4
  • File Content Strategy: Full file content
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened
  • Summary Prompt: Default Prompt

@davidkna-sap davidkna-sap left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The main draw would be that error messages are worse (by default, unless the English langpack its loaded).

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