Plan Review: Add a docs landing hub

The plan is sound with revisions. All seven reviewers agree the scope is appropriate and the architecture is correct for a single-file static HTML change. The team identified several actionable gaps: the smoke test path and naming violated project conventions (tests/pages/test-*), no step created the test files described in the Tests section, the E2E spec assumed Playwright infrastructure that does not exist, the existing test-root-redirect.sh would break silently, the hub page lacked a required skip-link, and card descriptions were too vague. All issues have been addressed in inline edits applied to the source plan.

Fit: Architecturally sound for scope — single static HTML file, no build pipeline, no external dependencies.

Concerns:

• CSS duplication across hub and galleries will drift low
• No back-navigation from galleries to hub low
• Guides directory is a dead branch at launch low
• Smoke test git-diff assertion is brittle low

Assessment: Well-scoped and largely executable, but had meaningful gaps.

Gaps addressed:

• No step created test files — added Step 6 high
• Step 1 verify lacked HTML boilerplate checks medium
• Card descriptions unspecified medium
• E2E spec listed as deliverable with no Playwright infra high
• No commit step referencing test-root-redirect.sh medium

Coverage: Adequate for the narrow scope after fixes.

Gaps addressed:

• Smoke test had no CSS-token assertion medium
• Git-diff assertion replaced with test -f check medium
• E2E spec marked deferred with explicit status note high
• No responsive layout test for below 640px low

Risk level: Low

• Relative links break if galleries move medium
• CDN propagation delay after deploy low
• test-root-redirect.sh would have broken silently low
• No CSP or subresource integrity low

Fit: Good overall after corrections.

• Smoke test moved to tests/pages/test-docs-hub.sh high
• E2E spec deferred since no Playwright config exists medium
• test-root-redirect.sh breakage added to Files and AC9 low

User fit: Clear happy path with direct routing.

• Skip-link added to Step 4 requirements and AC8 low
• Card descriptions required to be meaningful medium
• Tablet-range viewport check added low
• Back-to-home link deferred to Next Steps high
• Headline/tagline copy unresolved medium

A11y compliance: Largely WCAG 2.1 AA compliant after fixes.

• Skip-link requirement added to built hub page high
• E2E test key case updated to check for skip-link medium
• Copy button touch targets below 44px — accepted as non-primary actions low
• Completion checklist disabled checkboxes lack aria-disabled hint medium

No material conflicts. UX reviewer rated "back-to-home link" as high severity and recommended promoting it to a required step; Architecture and Risk reviewers rated it low. Resolution: kept as Next Steps since it modifies files outside this plan's scope (docs/plans/index.html and docs/media/social/index.html), but the prompt is ready to execute immediately after this plan ships.