Remove dead declarations inside live files, barrel-only re-exports and unused default exports

[yattias]

Fifth chunk of the dead-code removal plan (docs/dead-code-report.md, kept local), following #1169. This one works inside live files: the exports knip reports as imported nowhere, limited to declarations that are not used in their own file either. Symbols that are only used in-file keep their export (that mechanical pass is optional and left out).

What goes

• 77 declarations deleted outright. The nine unused bounty helpers in bountyUtil.ts; BountyHubSelector (its Hub type moves next to useBounties, its only reader); the four unused BaseFeedItem sections; LabelBadge; the two collapsible wrappers; useNote, useMakeNotePrivate, useUpdateNotePermissions; useUpdateOrgDetails, useUpdateOrgCoverImage; createFeedEntryFromWork; transformHub, transformContent; getAuditContentPreview and the text helpers only it called; the formatCombinedBalance pair; isValidPassword / isValidName; toNumberOrNull; isPinchGesture; ALL_CONTENT_TYPES; the deprecated HotScoreTooltip alias; the grant template object only getInitialContent returned; and the one-line Transformed* / response type aliases nothing references.
• 43 barrel re-export lines dropped (Activity, Embed, Editor/extensions, ui/Table, WorkHeader, PublishingForm, AuthorPosts, BaseFeedItem, plus a few single type re-exports). Everything behind them is still imported directly, except TopicAndJournalBadge, FUNDING_KIND_LABEL and the FundingRequest interface, which had no other reader and go too.
• 38 trailing export default X; lines removed from files that also export X by name; nothing imported the default.
• Dependencies: @tiptap/extension-bullet-list, -ordered-list and -paragraph leave package.json. The barrel re-export was their only reference, and StarterKit already provides those nodes (they stay in node_modules through it).
• Imports and local types orphaned by the deletions are removed. Unused imports that already existed on main in untouched lines are left alone, so the diff stays attributable.

95 files changed, 1,867 lines removed, 22 added (the moved Hub type and trimmed import lines). No behaviour change intended.

Checks

• npm run lint: 0 errors (2 pre-existing warnings in untouched files)
• npm run type-check: clean
• ESLint's no-unused-vars run ad hoc on every touched file: only findings that already existed on main remain
• knip run three times (after the main pass, after the second-layer deletions, after the dependency removal): no unused files, no new unused exports
• Dev server on the branch: all main routes return 200; the block editor still renders on a registered report page

Not in this chunk

knip still lists ~400 exports that are only used inside their own file (dropping the export keyword) and the undeclared transitive imports (highlight.js, @radix-ui/react-portal, @tanstack/react-query, wagmi). Both are separate, mechanical changes if wanted.

Next

Chunk 6 (retire/6-topic-pages) is the first product retirement and starts once this is merged.

🤖 Generated with Claude Code

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
web	Ready	Preview	Oct 11, 2026 9:26pm UTC

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud