lucide-icons / lucide

Beautiful & consistent icon toolkit made by the community. Open-source project and a fork of Feather Icons.
https://lucide.dev
ISC License
11.43k stars 524 forks source link

Upgrade to 0.458.0 from 0.457.0 breaks: Could not resolve "../icons/alarm-check.ts" from .. aliases.js" #2614

Closed kyle-leonhard closed 6 days ago

kyle-leonhard commented 6 days ago

Package

Version

0.458.0

Can you reproduce this in the latest version?

Browser

Operating system

Description

After upgrading to 0.458.0, my build breaks with: Could not resolve "../icons/alarm-check.ts" from "node_modules/.pnpm/lucide-svelte@0.458.0_svelte@5.2.0/node_modules/lucide-svelte/dist/aliases/aliases.js". Despite not importing that icon anywhere.

https://github.com/lucide-icons/lucide/pull/2584 stands out as potentially related. @ericfennis, fyi, in case this rings a bell.

Steps to reproduce

TODO: I have my repo, but not a minimal repro.

Checklist

aunumever commented 6 days ago

Happening to me as well. It's referencing a .ts file, but there are only .js files.

dongumayagay commented 6 days ago

having this issue as well

wiesson commented 6 days ago

Same here, also

https://lucide.dev/guide/advanced/aliased-names#create-a-custom-module-declaration-file is missing svelte

ericfennis commented 6 days ago

Can you confirm this is still happening when removing node_modules en reinstalling it. Can be happening because of the build cache.

ericfennis commented 6 days ago

Working on a fix!

ericfennis commented 6 days ago

It is fixed in lucide-svelte@0.460.1. Sorry for the inconvenience.

kyle-leonhard commented 5 days ago

Thanks for the quick fix, @ericfennis

ericfennis commented 4 days ago

@kyle-leonhard Thank you for letting us know!