Skip to content

Fix about smoke marker after title rename #133

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
a6b8 merged 1 commit into from
Jun 22, 2026
Merged

Fix about smoke marker after title rename #133

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion tests/playwright/phase-9-routes.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,9 @@
// .trash). Those routes now live in FORBIDDEN_ROUTES (Block 2: 404/redirect = PASS).
const EN_ROUTES = [
// Get-Started-Gruppe (Memo 144 T9: About FlowMCP merged into Get Started; standalone About group removed)
{ path: '/about/', expectedMarker: 'About FlowMCP', group: 'Get Started', locale: 'en' },
// Marker is a body phrase, not the title: the title was shortened to "About",
// which now also appears in the global sidebar nav and would false-pass anywhere.
{ path: '/about/', expectedMarker: 'FlowMCP started as', group: 'Get Started', locale: 'en' },

// Memo 144 T8: sidebar label "CLI Setup" (Memo 060) now matches the page H1 "CLI Setup".
{ path: '/quickstart/quickstart/', expectedMarker: 'CLI Setup', group: 'Get Started', locale: 'en' },
Expand Down
Loading