Skip to content

fix dotenv import for vercel-deno compat - #469

Open
Tony405-spec wants to merge 1 commit into
ryo-ma:masterfrom
Tony405-spec:master
Open

Tony405-spec wants to merge 1 commit into
ryo-ma:masterfrom
Tony405-spec:master

Conversation

@Tony405-spec

Copy link
Copy Markdown

No description provided.

Copilot AI lite review requested due to automatic review settings September 12, 2026 11:54

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@diogocouto18 diogocouto18 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Confirmed independently — I hit the exact same root cause (JSR subpath resolution failing on Vercel's read-only filesystem at request time via the vercel-deno runtime) and tested this fix on a personal fork deployed to Vercel: replacing the bare @std/dotenv/load specifier with a direct deno.land/std URL resolves the 500 on every request.

One thing worth calling out for whoever merges this: this approach keeps local .env loading working (useful for deno task start), unlike just removing the import outright, which is also a valid fix but drops that convenience for local dev. Good tradeoff for a project that's meant to run both locally and on Vercel.

This branch has not been deployed

No deployments
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.

3 participants