fix(module): preload global fonts

[Meliceanu]

Resolves #834
resolves #860

What

Global fonts (global: true) are never preloaded, even with preload: true set.

Why

The normal font-preload path runs through shouldPreload, which feeds into fontMap and finally into build:manifest. The nuxt-fonts-global.css template, however, only generates the @font-face rule and skips this process entirely, so global fonts never enter the preload map.

How

I extracted shouldPreload into a shared function and collect the top-priority font into fontMap, using the font-family name as the key. This lets build:manifest correctly attach the font to the entry chunk. As a result, global fonts are preloaded as expected.

Testing

I verified the issue is fixed in a production build, and the existing snapshot now includes /font-global.woff2.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/fonts@856

commit: 5540069

[coderabbitai]

📝 Walkthrough

Walkthrough

Global font processing now determines preload eligibility through a shared shouldPreload helper, selects the highest-priority font per family, and records eligible font URLs in the shared fontMap. The helper is passed to FontFamilyInjectionPlugin, and the feature test snapshot now includes /font-global.woff2 among the generated preload links.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes address #834 by adding preload handling for global fonts so preload links can be generated.
Out of Scope Changes check	✅ Passed	The diff stays focused on global font preload handling and the related test snapshot update.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly and concisely describes the fix for preloading global fonts.
Description check	✅ Passed	The description explains the global font preload issue, its cause, implementation, and testing results.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Meliceanu]

The failing ci (ubuntu-latest) job looks unrelated to this PR. It's a flaky test download, not related to the changes.

test/base-url.test.ts fails during the fixture build while src/assets.ts downloads a stubbed Adobe font, and the font that 404s differs between runs (aleo-i4.woff2, then barlow-n4.woff2) with no code change in between. Windows passes. This PR only touches the preload map in src/module.ts and doesn't change which fonts get downloaded.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 69.40%. Comparing base (16e87f6) to head (5540069).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/module.ts	93.75%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #856      +/-   ##
==========================================
+ Coverage   68.01%   69.40%   +1.39%     
==========================================
  Files           9        9              
  Lines         297      304       +7     
  Branches       67       71       +4     
==========================================
+ Hits          202      211       +9     
+ Misses         78       77       -1     
+ Partials       17       16       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Meliceanu]

Additional checks needed or all good? @danielroe