Skip to content

fix: add estree-walker as a dependency - #583

Merged
harlan-zw merged 1 commit into
mainfrom
fix/estree
Jan 18, 2026
Merged

fix: add estree-walker as a dependency#583
harlan-zw merged 1 commit into
mainfrom
fix/estree

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

resolves#582

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

this adds a missing dependency and also fixes url -> email in author field

@vercel

vercelBot commented Jan 17, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentReviewUpdated (UTC)
scripts-docsErrorErrorJan 17, 2026 11:01pm
scripts-playgroundReadyReadyPreview, CommentJan 17, 2026 11:01pm

@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/nuxt/scripts/@nuxt/scripts@583

commit: 38bf1ec

@danielroe

Copy link
Copy Markdown
MemberAuthor

seems to be failing on og images - is there another package relying on an implicit estree-walker dep?

[error] [request error] [unhandled] [GET] http://localhost/__og-image__/font/Inter/700.ttf?font=%7B%22cacheKey%22:%22Inter:undefined:700%22,%22style%22:%22normal%22,%22weight%22:700,%22name%22:%22Inter%22,%22key%22:%22nuxt-og-image:fonts:Inter-normal-700.ttf.base64%22%7D
[GET] "https://fonts.googleapis.com/css2?family=Inter:wght@700": <no response> fetch failed
at processTicksAndRejections (node:internal/process/task_queues:105:5)
at runNextTicks (node:internal/process/task_queues:69:3)
at process.processTimers (node:internal/timers:520:9)
at async $fetchRaw2 (/vercel/path0/node_modules/.pnpm/ofetch@1.5.1/node_modules/ofetch/dist/shared/ofetch.CWycOUEr.mjs:286:14)
at async $fetch2 (/vercel/path0/node_modules/.pnpm/ofetch@1.5.1/node_modules/ofetch/dist/shared/ofetch.CWycOUEr.mjs:332:15)
at async fontEventHandler (.nuxt/prerender/chunks/_/eventHandlers.mjs:227:15)
at async Object.handler (.nuxt/prerender/chunks/routes/__og-image__/font/font.mjs:42:12)
at async Object.handler (/vercel/path0/node_modules/.pnpm/h3@1.15.4/node_modules/h3/dist/index.mjs:2004:19)
at async toNodeHandle (/vercel/path0/node_modules/.pnpm/h3@1.15.4/node_modules/h3/dist/index.mjs:2296:7)
at async b (/vercel/path0/node_modules/.pnpm/node-mock-http@1.0.4/node_modules/node-mock-http/dist/index.mjs:1:6876)
at async C (/vercel/path0/node_modules/.pnpm/node-mock-http@1.0.4/node_modules/node-mock-http/dist/index.mjs:1:7159)
at async $fetchRaw2 (/vercel/path0/node_modules/.pnpm/ofetch@1.5.1/node_modules/ofetch/dist/shared/ofetch.CWycOUEr.mjs:274:26)
at async $fetchRaw2 (/vercel/path0/node_modules/.pnpm/ofetch@1.5.1/node_modules/ofetch/dist/shared/ofetch.CWycOUEr.mjs:327:14)
at async $fetch2 (/vercel/path0/node_modules/.pnpm/ofetch@1.5.1/node_modules/ofetch/dist/shared/ofetch.CWycOUEr.mjs:332:15)
at async loadFont (.nuxt/prerender/chunks/_/emojis.mjs:26:12)

@harlan-zw

Copy link
Copy Markdown
Collaborator

Quite possibly I'll dig into it, thanks!

@harlan-zw
harlan-zw merged commit a8d20b0 into mainJan 18, 2026
9 of 10 checks passed
@harlan-zw
harlan-zw deleted the fix/estree branch January 18, 2026 01:14
@harlan-zwharlan-zw mentioned this pull request Jan 23, 2026
zizzfizzix pushed a commit to zizzfizzix/nuxt-scripts that referenced this pull request Mar 5, 2026
Sign up for freeto 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.

estree-walker should be a dependency

2 participants

@danielroe@harlan-zw