feat(dts): transform vue script setup if no type-only macros

yusufkandemir commented 6 months ago

🔗 Linked issue

Resolves #209

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ 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

Some macros in <script setup> have a type-only variant:

defineProps
defineEmits
defineSlots
defineModel

As long as their type-only variant is NOT used, they can safely be transformed into JS. So, this PR improves the exclusion logic to exclude <script setup> only if the file contains some type-only macro usage.

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

danielroe commented 4 months ago

Would you be able to look at the failing test? 🙏

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 95.65217% with 2 lines in your changes missing coverage. Please review.

Project coverage is 81.68%. Comparing base (9000888) to head (d5b65ae). Report is 56 commits behind head on main.

Files with missing lines	Patch %	Lines
src/loaders/vue.ts	95.65%	2 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #210 +/- ## ========================================== - Coverage 82.86% 81.68% -1.19% ========================================== Files 12 12 Lines 852 868 +16 Branches 133 191 +58 ========================================== + Hits 706 709 +3 - Misses 144 157 +13 Partials 2 2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

unjs / mkdist