[donbr]

Summary

Removes /concepts. The page was generated in b03de6c (5 Nov 2025, a Claude Code session committed directly to main) and opened with a first-person anecdote — "A student once asked me: 'Should I refactor my FastAPI RAG system to use FastMCP?'" — that never happened. No earlier source for the page or the anecdote exists anywhere in the repo's history.

• Deletes src/pages/ConceptsPage.tsx
• Removes its route from src/App.tsx and the Concepts link from Layout.tsx
• Drops "Concepts" from the nav list in CLAUDE.md

/concepts now renders the app's 404 page. (Until the 404.html SPA fallback in #9 ships, a direct request on GitHub Pages gets GitHub's own 404 instead; either way the content is gone.) The page text remains recoverable from git history.

This is the first PR in a stack: this → #9 (fixes and performance) → follow-ups. It's independent and small so it can merge and deploy first.

Test plan

• npm run build passes; npx tsc -p tsconfig.app.json --noEmit passes; npm run lint 0 errors (3 pre-existing warnings)
• No remaining references to the page in src, public or CLAUDE.md
• vite preview: /concepts renders "404 - Page Not Found"; nav shows About, Expertise, Publications, Projects, Teaching, Certifications, Contact, Graph Demos
• After merge + npm run deploy: https://donbr.github.io/concepts no longer shows the page

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ppa5DyJCZ1CowKtz58QARN

[claude]

Review

Clean, minimal removal. I checked the diff and grepped src, public and CLAUDE.md. The only remaining match for "concepts" is unrelated prose in HomePage.tsx.

Code quality

• The page, route, import and nav link are all removed together. No dead imports or leftovers.
• The CLAUDE.md nav list is updated. The Routing section never listed /concepts, so nothing else needs changing there.
• isActive and cn are still used by the other nav links, so no lint fallout.

Bugs / behavior

• /concepts now falls through to * and renders NotFoundPage, as described. On GitHub Pages a direct hit shows GitHub's own 404 until the SPA fallback in Portfolio fixes: deep links, mobile nav, OG image, card links, code-splitting, page titles #9 lands. That is acceptable and already noted in the description.
• Optional: external links to the old URL will break. A redirect isn't worth it for a personal site, but you could check whether it was shared anywhere.

Performance / security

• Nothing to flag. The bundle gets smaller, and removing the static page adds no new surface.

Tests

• The repo has no test framework, and the manual verification in the test plan (build, tsc, lint, preview) is appropriate for this change.
• The last checkbox, confirming the deployed URL after merge, is the one still open.

Note on the rationale

• Removing a page with a fabricated first-person anecdote is sensible. The links to langsmith-mcp-server and adv-rag went with it. If the MCP architecture content is worth keeping, a rewritten version without the invented story could come back later.

LGTM.

🤖 Generated with Claude Code