Remove flag and experiment scaffolding, dead comment utilities and barrel-only exports

[yattias]

Fourth chunk of the dead-code removal plan (docs/dead-code-report.md, kept local), following #1166.

What goes

• Feature-flag scaffolding. utils/featureFlags.ts had one flag, hard-coded to true, a localStorage reader nobody called, a console helper installed on window, and DEBUG branches that were always off. The two isFeatureEnabled(...) checks in NoteEditorLayout were always true and are inlined. Only isProduction() had readers (the two legacy banners), so that is all that remains, without the debug branches.
• Homepage experiment helper. utils/experiment.ts was reachable only through an unused import in contexts/UserContext.tsx.
• Dead comment utilities. The commentUtils barrel re-exported seven names nothing imports, and the functions behind them were dead too: createOptimisticComment, updateCommentWithApiResponse, mapOptimisticToRealId, getParentId, clearIdMappings, removeCommentDeep, clearAllCommentDrafts, isContentEmpty, getFormattedCommentContent, feedEntryToPostCard, and the useComments hook (useCreateComment in the same file stays).
• Barrels trimmed to what their importers take: components/Funding and Funding/lib, Nonprofit, Referral, ui/icons, FeatureNotifications. The components only reachable through the removed lines go with them: FundingProposalCard, GrantCard, ProposalCount, FundraiseProgressBar, NonprofitInfoPopover, EndaomentInfoPopover, IllustratedGuide, PaperIcon, and one layer down ProposalPeekView, GrantPreviewTooltip, GrantBadge and usePostContent, which only FundingProposalCard used. RSC_PAYMENT_METHODS and PAYMENT_METHOD_LABELS in Funding/lib/constants.ts had no readers.

32 files changed, 1,848 lines removed, 48 added (the slimmed isProduction, trimmed import lines, and the id renames above). No behaviour change intended.

Second commit: the optimistic-id map

SonarCloud's quality gate on the first commit flagged that optimisticToRealIdMap in commentIdUtils.ts could only ever be empty once mapOptimisticToRealId was gone: getRealId had become an identity function and recordParentChildRelationship filled a map nobody read. Rather than ship a known no-op, the second commit removes commentIdUtils.ts and has the callers in commentTreeUtils, commentOptimisticUtils and CommentContext use the ids they were given. Every removed call either returned its argument or wrote to state nothing consulted, so behaviour is unchanged. The diff there is worth a read since it touches live comment code.

Checks

• npm run lint: 0 errors (2 pre-existing warnings in untouched files)
• npm run type-check: clean
• knip re-run twice (after the first pass and after the second-layer deletions): no unused files, no new unused exports or dependency issues
• Dev server on the branch: main routes return 200; the funding contribute flow's imports (PaymentStep, StripeProvider, FundingImpactPreview, QuickAmountSelector, the two hooks) are exactly what the trimmed barrel still exports

Next

Chunk 5 (cleanup/5-dead-exports) follows 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 2:17am 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