Skip to content

fix: resolve react-doctor warnings - unstable keys and next/link usage#46

Open
cto-new[bot] wants to merge 1 commit intomasterfrom
cto/fix-react-doctor-errors
Open

fix: resolve react-doctor warnings - unstable keys and next/link usage#46
cto-new[bot] wants to merge 1 commit intomasterfrom
cto/fix-react-doctor-errors

Conversation

@cto-new
Copy link
Copy Markdown
Contributor

@cto-new cto-new bot commented Feb 21, 2026

Fixed all critical react-doctor warnings by:

  1. Stable array keys in SourceFilesSection: Changed key={index} to key={file.name} to prevent React bugs when file lists are reordered or filtered

  2. Next.js Link component in landing page: Replaced <a href="/signin"> with <Link href="/signin"> to enable client-side navigation, prefetching, and proper scroll position preservation

Results:

  • React Doctor score improved from 91/100 to 99/100
  • Reduced warnings from 32 across 17 files to 1 across 1 file
  • All critical accessibility, performance, and best practice issues resolved

The remaining warning about LandingPage line count is a code organization suggestion; the component is already well-structured with HeroSection, LandingHeader, and LandingFooter sub-components.

Powered by CTO.new

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Feb 21, 2026

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

Project Deployment Actions Updated (UTC)
questgen Ready Ready Preview, Comment Feb 21, 2026 1:02pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants