Skip to content

Improve Ory env validation errors#366

Merged
huv1k merged 1 commit into
mainfrom
huv1k/fix-ory-env-errors
Jun 9, 2026
Merged

Improve Ory env validation errors#366
huv1k merged 1 commit into
mainfrom
huv1k/fix-ory-env-errors

Conversation

@huv1k

@huv1k huv1k commented Jun 9, 2026

Copy link
Copy Markdown
Member

Updates the app env check so AUTH_PROVIDER=ory reports only the Ory-related variables that are actually missing. The required Ory env variable names are defined once and the validator now emits a custom Zod issue with the missing subset. Verified with bun scripts/check-app-env.ts and bunx biome check scripts/check-app-env.ts.

Copilot AI review requested due to automatic review settings June 9, 2026 07:28
@cla-bot cla-bot Bot added the cla-signed label Jun 9, 2026
@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
web Building Building Preview, Comment Jun 9, 2026 7:28am
web-juliett Building Building Preview, Comment Jun 9, 2026 7:28am

Request Review

@huv1k huv1k merged commit c1cb3e6 into main Jun 9, 2026
12 of 14 checks passed
@huv1k huv1k deleted the huv1k/fix-ory-env-errors branch June 9, 2026 07:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves developer feedback from the app environment validation script when AUTH_PROVIDER=ory by reporting only the specific Ory-related environment variables that are missing.

Changes:

  • Adds a centralized oryRequiredEnvVars list for Ory-specific required variables.
  • Replaces the previous boolean .refine() with .superRefine() to emit a custom Zod issue listing only the missing subset.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants