Skip to content

fix(eve): ignore framework marker packages - #1272

Open
glucas1103 wants to merge 1 commit into
vercel:mainfrom
glucas1103:fix/server-only-pseudo-packages
Open

fix(eve): ignore framework marker packages#1272
glucas1103 wants to merge 1 commit into
vercel:mainfrom
glucas1103:fix/server-only-pseudo-packages

Conversation

@glucas1103

Copy link
Copy Markdown

Description

Closes #1235.

Eve's authored-module, immutable-generation, Workflow, and Nitro host bundles all execute on the server, but some paths currently preserve server-only, client-only, and Next.js's compiled variants as runtime imports. The marker package can then throw while the generated Node function is loading.

This change introduces one shared pseudo-package bundler plugin that resolves those four framework-only markers to empty virtual modules and wires it into every Eve-owned server bundling path. It also replaces the Workflow-only special case with the shared implementation, documents the behavior, and adds a patch changeset.

How did you test your changes?

  • Confirmed two focused regression scenarios fail before the fix and pass after it.
  • pnpm test:unit — 513 files, 5,460 tests passed.
  • pnpm typecheck
  • pnpm docs:check
  • pnpm lint

PR Checklist

  • I linked an issue with prior discussion confirming this change is wanted
  • I ran the relevant checks from CONTRIBUTING.md
  • I added tests and documentation where relevant
  • I added a changeset if this touches the published eve package
  • DCO sign-off passes for every commit (git commit --signoff)

Signed-off-by: Lucas Gaillard <lucasgaillard@outlook.com>
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

@glucas1103 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@glucas1103
glucas1103 marked this pull request as ready for review July 28, 2026 08:35
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.

Treat server-only/client-only as pseudo-packages in every bundling path, as the workflow bundler already does

1 participant