Repository navigation
feat: rework /ai for hiring teams with proof, mobile layout and nav link - #119
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
8 of 12 tasks
8 of 12 tasks
This branch was successfully deployed
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.
What and why
Turns
/aiinto a page for hiring teams. It opens with Roman's approved intro and proof points and ends with a hiring call to action. It links a public project (Magic Kick) as proof and no longer names two private projects. The page is readable on phones: project cards instead of a wide table, and long lists collapsed./aiis now in the main menu and has a teaser on the homepage.No linked issue.
Changes
Copy:
/ai(src/data/aiKnowledgeBase.js,KnowledgeBasePage.jsx)/roman-mazuryk-cv.pdf, the site's existing CV) and "Email me" (mailto:romazuryk@gmail.com). The old "Start with the five pillars" and "Jump to glossary" buttons are gone. The eyebrow "AI Field Guide · Last reviewed 30 Sep 2026" staysIn usePrivacy:
/aiand everything it rendersagent-os)learning-product). The note no longer mentions German B1 learnersMobile layout:
/aibelow 768px (knowledge-base.css)statusfield inkbProjectsaria-expanded,aria-controls). They are closed by default. While a glossary search has text, every match is shownNavigation (
Navbar.jsx,Footer.jsx,site.js)/aiin the header list (desktop and the mobile menu) and in the footer nav. It shows as active witharia-current="page"on/ai/aionly, the Services link is hidden in the header and footer, so the page shows no services or pricesSection navigation on
/ai(common/SectionNav.jsx,aiKnowledgeBase.js,knowledge-base.css)/services:site/ServicesSectionNav.jsxtocommon/SectionNav.jsxand is renamed toSectionIndex/SectionRail.ServicesPage.jsxonly changes its import/aisets--sv2-accent-textto the same values as/services..kb-sectiongetsscroll-margin-top, so anchor jumps clear the fixed headerHomepage (
HomePage.jsx,site.js)#ai), using the sameSectionHeaderand link style as the rest of the page. It links to/aiVerification
npm run lintexits 0npm run buildexits 0 (this runsscripts/check-consistency.mjsfirst)Local check on
vite previewwith headless Edge, light theme:grep -ri "ai-business-os|business-os|deutschon|agents.md"on the/aicomponent, its data,ai/index.htmlanduseOgMeta.js: no hits/servicesat 1500px: the rail still works (03 highlighted on Proof).How to test
/aion desktop:/aion a phone or a 375px viewport:/on desktop and mobile:/ai/aiat 1360px or wider:/servicesto confirm it still works./aifor "DeutschOn", "Business-OS" and "AGENTS": no resultsCore loop: home → Work → a proof link → Contact (email copy, CV download) still works.
Not tested:
/aishows the table.Quality gates
aria-labelon icon-only controls, focus rings,prefers-reduced-motionICONSmap (FileDown and Mail are named imports used directly, not data-driven)Redirects in(no route changes)vercel.jsonchecked if a route was added, moved or removedDocs
CHANGELOG.mdupdated under[Unreleased](no architecture change)docs//CLAUDE.mdupdated if architecture, routing or scope changedScreenshots
Taken locally, not attached. To be added from the Vercel preview.
Notes for review
/ai, left unchanged as asked. A grep ofsrc/for the old names finds two hits, both on the homepage:src/content/site.js:102, AI Field Guide card outcome: "…including my M.Sc. capstone (DeutschOn-AI) and this site."src/content/site.js:156, Experience entry: "DeutschOn-AI (capstone, preparing for pilot), AlphaRhythm (pre-beta)."mdconvention). The header switches to the burger menu at 800px, as before./ai. I read "Don't show any services or prices on /ai" as covering the header and footer links too. It is a singleisAiPagecheck in each component if you'd rather keep the link.Navbar.jsx./aititle and description used by search engines and link previews are unchanged ("AI Field Guide: Harness, Skills, Memory, Tools, Cost"). They could be aligned with the new headline in a follow-up.In usebecause it is live.🤖 Generated with Claude Code