Skip to content

Conversation

jycouet
Copy link
Contributor

@jycouet jycouet commented Sep 9, 2025

This PR replaces the unmaintained pascal-case package with a recommended one, scule.

Why?

  • [email protected] is outdated, and 4.0.0 is deprecated.
  • scule has 0 dependencies and a smaller total footprint.

Comparison:

Package Size Dependencies Total Size
pascal-case 14KB 2 (no-case 25KB, lower-case 17KB) 56KB
scule 29KB 0 29KB

Net win: Fewer dependencies, lower size, and a more future-proof solution.


One adjustment, before Test3$$PropsProps was stripping $$ in the middle, not with scule. So I had to tweak the regex.

Playground to see;
https://svelte.dev/playground/96bd32583b4d45bf838699550bc795a7?version=5.38.7

@dummdidumm
Copy link
Member

Looks good, thank you! Could you resolve the conflict and add a changeset (I just merged a changeset PR, we now use it in this repo)? Thanks :)

Copy link

changeset-bot bot commented Sep 11, 2025

🦋 Changeset detected

Latest commit: c5b823a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
svelte2tsx Patch
svelte-language-server Patch
typescript-svelte-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jycouet
Copy link
Contributor Author

jycouet commented Sep 11, 2025

Nice to have changeset now 🚀
Should be good

Copy link
Member

@dummdidumm dummdidumm left a comment

Choose a reason for hiding this comment

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

thank you!

@dummdidumm dummdidumm merged commit f799839 into sveltejs:master Sep 11, 2025
3 checks passed
@github-actions github-actions bot mentioned this pull request Sep 11, 2025
axel7083 pushed a commit to axel7083/svelte-language-tools that referenced this pull request Sep 12, 2025
…ltejs#2842)

* (deps) Replace pascal-case with change-case (Maintained & Lighter)

* add changeset
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.

2 participants