Skip to content

feat: size impact analysis #486

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mary-ext wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: size impact analysis #486

mary-ext wants to merge 3 commits into from
+5,675 −7

Conversation

@mary-ext
Copy link

@mary-ext mary-ext commented Jan 31, 2026 •
edited
Loading

this is still very rough, I would need help because I am very unfamiliar with Nuxt here and with how npmx project is set up

image

we can extend this page with additional options later (like a platform config so you can test bundle sizes with various import conditions applied)

@vercel
Copy link

vercel bot commented Jan 31, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
npmx.dev Error Error Feb 1, 2026 1:27pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 1, 2026 1:27pm
npmx-lunaria Ignored Ignored Feb 1, 2026 1:27pm

Request Review

mary-ext
mary-ext commented Jan 31, 2026
View reviewed changes
nuxt.config.ts
Comment on lines +90 to +107
'/impact/**': {
headers: {
'Cross-Origin-Opener-Policy': 'same-origin',
'Cross-Origin-Embedder-Policy': 'require-corp',
},
},
Copy link
Author

@mary-ext mary-ext Jan 31, 2026

Choose a reason for hiding this comment

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

I am not sure what the ideal approach is here, setting the COOP/COEP headers here means that it is not possible to transition to /impact/ route directly without a full page navigation

@github-actions
Copy link

github-actions bot commented Jan 31, 2026 •
edited
Loading

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
lunaria/files/en-US.json Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@mary-ext
Copy link
Author

mary-ext commented Jan 31, 2026

blocked on rolldown/rolldown#8145 for Safari support

@danielroe
Merge remote-tracking branch 'origin/main' into feat/teardown
0ed5d0c 
# Conflicts:
#	app/pages/[...package].vue
#	i18n/locales/en.json
#	lunaria/files/en-US.json
#	package.json
#	pnpm-lock.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mary-ext @danielroe