René's URL Explorer Experiment

Title: refactor(*): use 'Icon'-suffixed names for all phosphor imports by sukvvon · Pull Request #1124 · TanStack/tanstack.com · GitHub

Open Graph Title: refactor(*): use 'Icon'-suffixed names for all phosphor imports by sukvvon · Pull Request #1124 · TanStack/tanstack.com

X Title: refactor(*): use 'Icon'-suffixed names for all phosphor imports by sukvvon · Pull Request #1124 · TanStack/tanstack.com

Description: Every phosphor icon is now imported under its supported Icon-suffixed export. The bare names (Star, CaretDown, …) are deprecated by the package — each icon module carries @deprecated Use Icon on them. Why these were not caught #1094 renamed all icons to the Icon suffix, and #1099 added a generated declare module '@phosphor-icons/react' so that the deprecated spelling became a compile error rather than a strikethrough. That declaration only covers the barrel. @phosphor-icons/react/CaretDown is a separate module, so subpath imports resolve against the package's own types and the deprecated names are merely struck through again. When #1099 was written, src/ had no subpath imports at all — the barrel was the only entry point, so locking it was sufficient. Subpath imports were introduced later, which opened a second path the guard does not cover. The guard is unchanged here; this PR only fixes the call sites. Extending it to subpaths is a separate change. Changes 100 imports across 24 files, in three shapes: Shape Count Result { Name as NameIcon } 92 { NameIcon } — local binding is identical, so no usage sites change { Name as CustomName } 3 { NameIcon as CustomName } — the alias is kept, only the imported name moves { Name } 5 { NameIcon } — no alias, so the JSX usages are renamed too The three custom aliases are all in Navbar: List as Menu, Sparkle as Sparkles, and TrendUp as TrendingUp. NavbarAuthControls imported the same module twice as { SignIn, SignIn as SignInIcon }, with both bindings in use; they are collapsed to SignInIcon. ds/ui/index.tsx was internally inconsistent — CaretDownIcon, CircleNotchIcon, and UserIcon were already suffixed while MagnifyingGlass and X were not. Verification tsc, lint, and the unit tests pass. No deprecated phosphor name remains in the repo. This was checked across barrel imports, subpath imports, multi-line import blocks, namespace and default imports, and dynamic import(). The generated registry and module declaration are untouched — regenerating them produces no diff.

Open Graph Description: Every phosphor icon is now imported under its supported Icon-suffixed export. The bare names (Star, CaretDown, …) are deprecated by the package — each icon module carries @deprecated Use <Name&g...

X Description: Every phosphor icon is now imported under its supported Icon-suffixed export. The bare names (Star, CaretDown, …) are deprecated by the package — each icon module carries @deprecated Use &lt;Na...

Opengraph URL: https://github.com/TanStack/tanstack.com/pull/1124

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width