Skip to content

Conversation

@Teages
Copy link
Contributor

@Teages Teages commented Apr 8, 2025

resolve #249
resolve nuxt/module-builder#597

This pr generate .d.vue.ts for all .vue inputs. more detail.

The new build-in loader will skip dts generation for .vue files which is no need to transfrom, this caused a regression in issue #249. This pr also fixed that by emit type declarations for every .vue files.

@Teages Teages changed the title fix(vue): force emit dts for build-in loader fix(dts): force emit dts for build-in loader Apr 8, 2025
@Teages Teages changed the title fix(dts): force emit dts for build-in loader fix(dts): force emit dts for build-in vue loader Apr 8, 2025
@codecov
Copy link

codecov bot commented Apr 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.87%. Comparing base (9000888) to head (b0b56b5).
⚠️ Report is 107 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #301      +/-   ##
==========================================
+ Coverage   82.86%   84.87%   +2.01%     
==========================================
  Files          12       12              
  Lines         852      992     +140     
  Branches      133      204      +71     
==========================================
+ Hits          706      842     +136     
- Misses        144      149       +5     
+ Partials        2        1       -1     

☔ View full report in Codecov by Sentry.
📢 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.

@Teages Teages changed the title fix(dts): force emit dts for build-in vue loader feat(dts): emit .d.vue.ts for .vue inputs Jul 19, 2025
@Teages Teages changed the title feat(dts): emit .d.vue.ts for .vue inputs feat(dts): emit .d.vue.ts as type declaration of .vue files Jul 19, 2025
@danielroe danielroe requested a review from pi0 August 11, 2025 11:56
@danielroe danielroe merged commit 585793c into unjs:main Sep 11, 2025
4 checks passed
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.

Typescript is not recognized well extensions are wrongly added with addRelativeDeclarationExtensions and vue + setup

2 participants