refactor: delete unreferenced components and the unused hiIQ ABI - #77
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Nothing imports any of these — verified by path and by exported identifier across src, i18n and messages, then confirmed by a clean tsc pass. - src/abis/hiIQABI.abi.ts (1.5k lines): getLockOverview talks to Alchemy with a hardcoded `0x18160ddd` totalSupply selector and never loads an ABI, so this has been dead since it landed - icons/1inch.tsx: a duplicate of one-inch.tsx exporting the same OneInchIcon name; exchanges.ts imports the latter - icons for QuickSwap, Bitget, Bithumb, Crypto.com, Fraxswap, IQ.wiki and SushiSwap: venues the current design dropped - svgs/: all four logo and hero-background variants, superseded by layouts/brand-logo and hero-orbits - transitions/InViewAnimateBottom.tsx: also broken — it never attached useInView's ref to a node, so `inView` was permanently false and the useAnimation controls were never bound to a component. The visible animation came from `whileInView` on the motion.div all along - ui/card.tsx, ui/container-scroll-animation.tsx, ui/popover.tsx: unused shadcn scaffolding Drops @radix-ui/react-popover (only popover.tsx) and react-intersection-observer (only InViewAnimateBottom.tsx).
invisiblemask
force-pushed
the
chore/drop-dead-code
branch
from
August 20, 2026 13:42
087a47f to
c4d769c
Compare
invisiblemask
changed the base branch from
chore/biome-formatting
to
restructure/iq-hiiq-scope
August 21, 2026 09:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #76, which is stacked on #74. Review those first.
Pure deletion — 17 files, 1,942 lines, two dependencies. No behaviour changes.
How dead was verified
For each candidate: no import of the file by path anywhere in
src,i18n,messagesor the root configs, and no reference to any exported identifier. Thenpnpm buildwith a full TypeScript pass, which is the authoritative check for a broken import.This turned up more than the four files I flagged during #74. Two were mislabelled as live by a first pass and had to be re-checked by hand —
zod-enum-from-recordis used bymessages/_schema.ts, andui/button/ui/dropdown-menuare used bylocale-switcher. Both stay.What goes
src/abis/hiIQABI.abi.ts(786 lines)getLockOverviewcalls Alchemy with a hardcoded0x18160dddtotalSupply()selector and never loads an ABI. Dead since it landed.icons/1inch.tsxone-inch.tsx, exporting the sameOneInchIconname.data/exchanges.tsimports the latter.icons/{QuickSwapIcon,bitget,bithumb,cryptocom,fraxswap,iq-wiki,sushiswap}marketsnow shows Binance, Upbit, 1inch and Frax.svgs/(all 4)BraindaoLogo,BraindaoLogoDark,BrainLogoWhite,HeroBackground— superseded bylayouts/brand-logoandlayouts/hero-orbits. Directory removed.transitions/InViewAnimateBottom.tsxui/{card,container-scroll-animation,popover}.tsxInViewAnimateBottomis worth calling out: it destructuredconst { inView } = useInView()without ever attaching the observer'srefto a node, soinViewwas permanentlyfalse, and theuseAnimation()controls it drove were never bound to any motion component. The animation that actually ran came fromwhileInViewon themotion.div. So the component was doing nothing even if something had imported it.Dependencies dropped
@radix-ui/react-popover— sole consumer wasui/popover.tsxreact-intersection-observer— sole consumer wasInViewAnimateBottom.tsxVerification
pnpm buildclean; TypeScript pass clean; both routes generatedpnpm biome check .clean (scope drops 89 → 72 files)/enand/en/hiiq: 29 SVGs, 0 empty, 1 zero-box (the hidden mobile hamburger) — identical to the pre-deletion baseline#marketsrender with real geometry and child nodes, including 1inch at 18×18 from the survivingone-inch.tsx#treasuryanchor resolves; live data intact (0.0006 / 16.68M / 27B / 2.5B locked, and 8B HiIQ on/hiiq)Note on the stack
Ten of the files reformatted in #76 are deleted here, so most of that PR's churn evaporates at merge. Leaving the order as-is rather than rewriting two open PRs' history — but if you'd rather review a small #76, say so and I'll reorder the stack so the deletion lands first.
One non-deletion change
pnpm removesilently dropped thepnpm.commentkey documenting why theoverridesblock exists and that Tailwind 4 retires it — pnpm rewrites its ownpnpmfield and discards keys it doesn't recognise. Restored as a top-level"//pnpm", which survives anadd+removeround-trip (verified).Rebased on the updated base
The merge of #73's hiIQ page rewrite is propagated down the stack (#74 → #76 → here). No conflicts at this level. Re-verified after the merge: none of the 18 files the base added imports anything this PR deletes,
biome check .is clean over 84 files,pnpm buildand the TypeScript pass are clean, and all three deleted directories stay gone.