From 0ca7ec65b6da3f34cd36a19a50a157cc9879e450 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 11:47:14 -0400
Subject: [PATCH 1/9] chore(plans): add site-check-153 plan

Issue #153 walks the live site on desktop and mobile and collects five
findings. Four are shippable together because they all land in the app
shell or in one screen's outbound links; the fifth (replace the Home hero
CTA with a mailing-list invite) has no mechanism to build against, so the
plan records the block up front rather than inventing one.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 plans/site-check-153.md | 151 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 151 insertions(+)
 create mode 100644 plans/site-check-153.md

diff --git a/plans/site-check-153.md b/plans/site-check-153.md
new file mode 100644
index 0000000..d288c9a
--- /dev/null
+++ b/plans/site-check-153.md
@@ -0,0 +1,151 @@
+---
+status: in-progress
+depends: []
+specs:
+  - specs/behaviors/app-shell.md
+  - specs/screens/volunteer.md
+issues: [153]
+---
+
+# Plan: site check — header order, mobile sheet padding, dead outbound links
+
+## Scope
+
+Issue [#153](https://github.com/CodeForPhilly/codeforphilly-ng/issues/153)
+("Site check for desktop & mobile") collects a walkthrough of the live site on
+both breakpoints. Four of its items are shippable now; one is blocked (see
+Follow-ups).
+
+What ships:
+
+- **Desktop header reorder** (spec-governed). The Volunteer CTA leaves the
+  content nav and becomes the rightmost element of the header, after the auth
+  control; About joins the left cluster's text links; a GitHub icon link is
+  added to the right cluster.
+- **Mobile sheet padding + accessible name.** The sheet's nav and search sat
+  flush against the panel edge. Fixed with the intended shadcn structure
+  (`SheetHeader` + `SheetTitle`) plus explicit horizontal padding — which also
+  gives the underlying Radix dialog the accessible name it was missing.
+- **Header ARIA cleanups.** Three defects surfaced by an accessibility pass over
+  the header, done here because this plan rewrites the same file.
+- **Dead outbound links** (spec-governed). The whole `codeforphilly.gitbook.io`
+  space returns 404 "Content owner not found"; `Volunteer.tsx`'s two remaining
+  GitBook links are repointed at live equivalents. Same class of defect as
+  [`home-start-project-cta`](home-start-project-cta.md) (PR #128), which fixed
+  the Home screen's copy of the same dead URL.
+- **Footer repo URL.** The "view this site on GitHub" link still pointed at
+  `codeforphilly-rewrite`; the repo is `codeforphilly-ng` and the old URL only
+  resolves through GitHub's rename redirect.
+
+Explicitly out of scope:
+
+- **Replacing the Home hero's Volunteer CTA with a mailing-list invite** (also
+  recommended by #153) — blocked, see Follow-ups. `Home.tsx` is untouched.
+- Any other visual restyle of the header. The Volunteer button keeps its
+  existing green treatment; only its position changes.
+
+## Implements
+
+- [app-shell.md](../specs/behaviors/app-shell.md) — "Center / right at ≥ md"
+  split into a left content cluster and a right utility cluster, with the new
+  item order; "Auth controls" repositioned second-from-right; GitHub link added
+  to the right cluster and to the mobile sheet; the sheet's accessible name
+  added under Accessibility.
+- [volunteer.md](../specs/screens/volunteer.md) — "Show up to meetups" card
+  links to the Meetup group; "Start your own project" band links to the
+  `CodeForPhilly/partnerships` first-steps guide. Both replace dead GitBook
+  URLs.
+
+## Approach
+
+### 1. Spec changes first (specops — source of truth leads)
+
+`specs/behaviors/app-shell.md` and `specs/screens/volunteer.md` both prescribed
+the current (wrong) state, so they lead. Header spacing is deliberately *not*
+specced — [specs/README.md:49](../specs/README.md) puts spacing outside spec
+scope — so the mobile-sheet padding fix carries no spec change.
+
+### 2. `apps/web/src/components/AppHeader.tsx`
+
+- Left `<nav>`: Projects, Help Wanted, Members, About ▾. `gap-2` replaces
+  `gap-1` + per-child `ml-1`, so the parent gap is the single source of spacing
+  at the same effective density (4px + 4px → 8px).
+- Right cluster: GitHub icon link → SearchBox → AuthControls → Volunteer button.
+  New hand-rolled `GitHubIcon` SVG follows the file's existing icon convention
+  (`ChevronDownIcon` / `MenuIcon`); path data copied from `LoginPlaceholder.tsx`.
+- Mobile sheet: `SheetHeader` + `SheetTitle` ("Menu") replace the `pt-8` hack;
+  nav and search get `px-4`. Mobile item order mirrors the new desktop order,
+  with a GitHub row added and Volunteer last.
+- ARIA: `aria-hidden` replaces `aria-label` on the roleless loading-skeleton
+  div; the hand-written `aria-expanded` comes off the `SheetTrigger` (Radix
+  `Dialog.Trigger` supplies it); `aria-label="About menu"` comes off the About
+  trigger so its visible text is the accessible name. The account-menu
+  `aria-label` **stays** — below `sm` the person's name span is `display:none`,
+  so that label is the only accessible name there.
+
+### 3. `apps/web/src/screens/Volunteer.tsx`
+
+`HACK_NIGHT_URL` → `MEETUP_URL` = `https://www.meetup.com/Code-for-Philly/`
+(the same target the footer's Meetup social icon already uses).
+`START_PROJECT_URL` → the `CodeForPhilly/partnerships` first-steps markdown,
+which is the surviving source of the retired GitBook page. Both stay external.
+
+### 4. `apps/web/src/components/AppFooter.tsx`
+
+One-line repo URL swap to `codeforphilly-ng`.
+
+### 5. Tests
+
+- `AppHeader.test.tsx` — new nav shape, About queried by its visible text, the
+  GitHub link's label + href, Radix still supplying `aria-expanded`, and the
+  sheet dialog's accessible name.
+- `Volunteer.test.tsx` (new) — both CTA hrefs plus a dead-link regression
+  assertion that no `codeforphilly.gitbook.io` URL survives anywhere in the
+  rendered document, mirroring the idiom from `Home.test.tsx`.
+- `AppFooter.test.tsx` — updated repo URL.
+
+## Validation
+
+- [ ] Specs updated before code: app-shell header clusters + volunteer link targets.
+- [ ] Desktop header order is Projects · Help Wanted · Members · About ▾ … GitHub · Search · Sign in · Volunteer, with Volunteer rightmost and still green.
+- [ ] No `ml-1` spacing hacks remain among the header nav's children.
+- [ ] GitHub link is icon-only, labelled "Code for Philly on GitHub", and opens `https://github.com/CodeForPhilly` in a new tab with `rel="noopener noreferrer"`.
+- [ ] Mobile sheet has a "Menu" title, horizontal padding on nav + search, and no `pt-8`; the title does not collide with the close button.
+- [ ] The sheet dialog exposes an accessible name; Radix still supplies `aria-expanded` on the trigger.
+- [ ] Loading skeleton uses `aria-hidden`; About trigger's accessible name is its visible text; account-menu label retained.
+- [ ] Every mobile sheet item closes the sheet on click, including Contact.
+- [ ] No `codeforphilly.gitbook.io` URL remains in `apps/web/src`.
+- [ ] Footer "view this site on GitHub" points at `codeforphilly-ng`.
+- [ ] `npm run -w packages/shared build && npm run type-check && npm run lint && npm test` clean.
+- [ ] Browser test: desktop header order + mobile sheet padding at < md, both breakpoints.
+
+## Risks
+
+- **Low, but layout-shaped.** Moving Volunteer out of the flex-1 nav and into
+  the `ml-auto` cluster changes how much room the SearchBox has to expand at
+  narrow desktop widths. Watched by the browser-test criterion above rather
+  than by a unit test — jsdom has no layout.
+- **`aria-expanded` regression risk.** Removing the hand-written attribute is
+  only safe because Radix supplies its own; asserted in the header test so a
+  future primitive swap can't silently drop it.
+
+## Notes
+
+(To be populated at closeout.)
+
+## Follow-ups
+
+(To be populated at closeout. Known now:)
+
+- **Tracked as: blocked — hero "mailing list invite" CTA.** Issue #153
+  recommends replacing the Home hero's Volunteer CTA with a mailing-list
+  invite. There is no anonymous mailing-list mechanism anywhere in the repo:
+  newsletter subscription exists only as an auth-gated checkbox on `/account`
+  (writing `PrivateProfile.newsletter`), and a public signup surface is
+  explicitly deferred — [app-shell.md:140](../specs/behaviors/app-shell.md)
+  lists "Newsletter signup (defer)" in the footer's Connect column, and
+  [deferred.md:100-104](../specs/deferred.md) defers the whole newsletter
+  sending pipeline with a promotion path (`/api/newsletter/send`, Resend-backed
+  worker, unsubscribe tokens). Building an anonymous-capture CTA ahead of that
+  spec would invent unspecified behavior. `Home.tsx` is deliberately untouched
+  here; the CTA swap should follow the newsletter spec work, not precede it.

From d1372261e4e20f483ea4ac85fe47c6fc39ec6b10 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 11:47:24 -0400
Subject: [PATCH 2/9] docs(specs): reorder header clusters, repoint dead
 GitBook links

Specs lead, so both files change before the code does.

app-shell: the header's single "primary nav" table conflated content
navigation with utilities, which is why the Volunteer CTA ended up buried
between Members and About. Splitting it into a content cluster and a
right-pinned utility cluster makes the CTA's position a stated rule
rather than an accident, and gives the GitHub link and the auth control
a declared home. Also states the sheet's accessible name and the
icon-only-controls labelling rule, both of which the header violated.

volunteer: the whole codeforphilly.gitbook.io space now returns 404
"Content owner not found", so the spec was prescribing two dead targets.
The Meetup group and the CodeForPhilly/partnerships first-steps doc are
the live equivalents; both verified reachable.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 specs/behaviors/app-shell.md | 31 +++++++++++++++++++++++++++----
 specs/screens/volunteer.md   |  4 ++--
 2 files changed, 29 insertions(+), 6 deletions(-)

diff --git a/specs/behaviors/app-shell.md b/specs/behaviors/app-shell.md
index 67659f3..2262f09 100644
--- a/specs/behaviors/app-shell.md
+++ b/specs/behaviors/app-shell.md
@@ -35,16 +35,31 @@ Sticky at the top of the viewport. Background opaque, slight shadow on scroll.
 
 ### Center / right at ≥ md
 
-Primary nav, items in this order:
+Two clusters. The **content cluster** sits next to the logo; the **utility
+cluster** is pinned to the right edge and carries the outbound link, search,
+auth, and the call to action.
+
+Content cluster, items in this order:
 
 | Item | Target | Style |
 | ---- | ------ | ----- |
 | Projects | `/projects` | text link |
 | Help Wanted | `/help-wanted` | text link |
 | Members | `/members` | text link |
-| Volunteer | `/volunteer` | button (success, filled) — emphasized because it's the call to action |
 | About ▾ | dropdown | text link with caret |
+
+Utility cluster, items in this order (left to right):
+
+| Item | Target | Style |
+| ---- | ------ | ----- |
+| GitHub | `https://github.com/CodeForPhilly` | icon-only external link, accessible name "Code for Philly on GitHub", opens in a new tab |
 | Search 🔍 | inline expand | icon button |
+| Sign in / account menu | see [Auth controls](#auth-controls) | button / avatar dropdown |
+| Volunteer | `/volunteer` | button (success, filled) — emphasized because it's the call to action |
+
+**Volunteer is the rightmost element in the header.** It sits after the auth
+control rather than among the content links so the call to action reads as the
+header's terminal step, not as one more section.
 
 ### About dropdown
 
@@ -57,7 +72,11 @@ Primary nav, items in this order:
 
 The `/pages/*` URLs serve **static content pages** authored as MDX/Markdown in the code repo (`apps/web/src/content/pages/`). They have no per-page screen spec — the content is the spec. Source copy ports from `codeforphilly.org/site-root/pages/` in the legacy repo.
 
-### Auth controls (rightmost)
+### Auth controls
+
+Second from the right in the utility cluster — between Search and the Volunteer
+button. On mobile the auth control sits in the header bar itself, outside the
+sheet.
 
 - **Anonymous:** "Sign in" (primary button) → `/login`. There is no separate "Sign up" button — sign-in and sign-up are the same flow once GitHub OAuth is specified (first sign-in creates the account).
 - **User:** Avatar + name dropdown:
@@ -75,7 +94,9 @@ The `/pages/*` URLs serve **static content pages** authored as MDX/Markdown in t
 
 ### Mobile (< md)
 
-Header collapses to: logo + hamburger menu + auth control. Hamburger opens a sheet (right-side overlay) with all nav items stacked vertically. Search is inside the sheet, not inline.
+Header collapses to: logo + hamburger menu + auth control. Hamburger opens a sheet (right-side overlay) with all nav items stacked vertically — the content-cluster links, the About items under an "About" label, the GitHub link, and Volunteer last, mirroring the desktop order. Search is inside the sheet, not inline. The auth control stays in the header bar, outside the sheet.
+
+Every item in the sheet closes the sheet when activated.
 
 ## Search
 
@@ -168,6 +189,8 @@ It does not block initial paint waiting on `me`. Auth controls render skeletons
 - Skip link at the very top: "Skip to main content" → focuses the `<main>` element
 - All dropdowns are keyboard-navigable
 - The mobile sheet traps focus while open and returns it to the trigger on close
+- The mobile sheet is a dialog with the accessible name "Menu"
+- Every icon-only control carries an accessible name; controls with visible text use that text as their accessible name rather than duplicating it in a label
 
 ## Print
 
diff --git a/specs/screens/volunteer.md b/specs/screens/volunteer.md
index 661925d..2bf03e1 100644
--- a/specs/screens/volunteer.md
+++ b/specs/screens/volunteer.md
@@ -27,7 +27,7 @@ Three cards in a row at ≥ md, stacked below:
 
 1. **Join Slack** — "We coordinate everything in our Slack workspace." Button: "Open Slack →" → `/chat`
 2. **Pick a project** — "Browse 268 active projects and find one that matches your interests." Button: "Browse projects →" → `/projects`
-3. **Show up to meetups** — "We meet weekly. Bring your laptop, or just yourself." Button: "When we meet →" → external link (currently the GitBook hack-night-program-details URL)
+3. **Show up to meetups** — "We meet weekly. Bring your laptop, or just yourself." Button: "When we meet →" → external link to the Meetup group (`https://www.meetup.com/Code-for-Philly/`), where upcoming hack nights are listed. Same target as the footer's Meetup social link.
 
 The "268" is read from a cheap call to `GET /api/projects?perPage=1` and rendered live; falls back to "hundreds of" if the call fails.
 
@@ -48,7 +48,7 @@ Static content emphasizing the non-developer roles (designers, project managers,
 
 Footer-style band at the bottom: "Have an idea? Start your own project."
 
-- Link to the external GitBook "creating-new-partnerships/first-steps" page (matches current codeforphilly.org)
+- Link to the external "first steps" partnership guide (`https://github.com/CodeForPhilly/partnerships/blob/master/creating-new-partnerships/first-steps.md`) — the surviving canonical source of the retired GitBook page of the same name
 - Secondary link "or create one on the site →" to `/projects/create` (signed-in) or `/login?return=/projects/create` (anonymous)
 
 ## Actions

From 0fac65036b011893244c410385bebdacfd493d33 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:02:24 -0400
Subject: [PATCH 3/9] fix(web): move Volunteer CTA rightmost, pad the mobile
 sheet
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Site check (#153) found the Volunteer button buried mid-nav between
Members and About, where it read as one more section link rather than the
call to action. It now closes the utility cluster, after the auth control,
per the reordered app-shell spec. About joins the content links; a GitHub
icon link is added to both breakpoints.

The mobile sheet had no horizontal padding at all: SheetContent's base
classes carry none and the only override here was a pt-8 hack, so nav
items and the search box sat flush against the panel edge. Replaced with
the structure shadcn intends — SheetHeader + SheetTitle, which bring
their own p-4 — plus explicit px-4 on the nav and search. SheetTitle also
gives the underlying Radix dialog the accessible name it never had.

Three ARIA defects fixed while the file was open: aria-label on a roleless
skeleton div (prohibited; now aria-hidden), a hand-written aria-expanded
duplicating what Dialog.Trigger already supplies, and an aria-label
overriding the About trigger's own visible text. The account-menu label
stays — below sm the person's name is display:none, so it is the only
accessible name there.

Per-child ml-1 margins are gone; the parent gap-2 is now the single source
of spacing at the same effective density.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 apps/web/src/components/AppHeader.tsx | 123 ++++++++++++++++++--------
 apps/web/tests/AppHeader.test.tsx     |  52 ++++++++++-
 2 files changed, 136 insertions(+), 39 deletions(-)

diff --git a/apps/web/src/components/AppHeader.tsx b/apps/web/src/components/AppHeader.tsx
index c0d6516..dd414de 100644
--- a/apps/web/src/components/AppHeader.tsx
+++ b/apps/web/src/components/AppHeader.tsx
@@ -8,11 +8,19 @@ import {
   DropdownMenuSeparator,
   DropdownMenuTrigger,
 } from '@/components/ui/dropdown-menu';
-import { Sheet, SheetContent, SheetTrigger } from '@/components/ui/sheet';
+import {
+  Sheet,
+  SheetContent,
+  SheetHeader,
+  SheetTitle,
+  SheetTrigger,
+} from '@/components/ui/sheet';
 import { Separator } from '@/components/ui/separator';
 import { SearchBox } from '@/components/SearchBox';
 import { useAuth } from '@/hooks/useAuth';
 
+const GITHUB_URL = 'https://github.com/CodeForPhilly';
+
 function ChevronDownIcon() {
   return (
     <svg
@@ -53,6 +61,21 @@ function MenuIcon() {
   );
 }
 
+function GitHubIcon() {
+  return (
+    <svg
+      xmlns="http://www.w3.org/2000/svg"
+      width="18"
+      height="18"
+      viewBox="0 0 24 24"
+      fill="currentColor"
+      aria-hidden="true"
+    >
+      <path d="M12 0c-6.626 0-12 5.373-12 12 0 5.302 3.438 9.8 8.207 11.387.599.111.793-.261.793-.577v-2.234c-3.338.726-4.033-1.416-4.033-1.416-.546-1.387-1.333-1.756-1.333-1.756-1.089-.745.083-.729.083-.729 1.205.084 1.839 1.237 1.839 1.237 1.07 1.834 2.807 1.304 3.492.997.107-.775.418-1.305.762-1.604-2.665-.305-5.467-1.334-5.467-5.931 0-1.311.469-2.381 1.236-3.221-.124-.303-.535-1.524.117-3.176 0 0 1.008-.322 3.301 1.23.957-.266 1.983-.399 3.003-.404 1.02.005 2.047.138 3.006.404 2.291-1.552 3.297-1.23 3.297-1.23.653 1.653.242 2.874.118 3.176.77.84 1.235 1.911 1.235 3.221 0 4.609-2.807 5.624-5.479 5.921.43.372.823 1.102.823 2.222v3.293c0 .319.192.694.801.576 4.765-1.589 8.199-6.086 8.199-11.386 0-6.627-5.373-12-12-12z" />
+    </svg>
+  );
+}
+
 function AuthControls({ mobile = false }: { mobile?: boolean }) {
   const { person, loading, signOut } = useAuth();
 
@@ -60,7 +83,7 @@ function AuthControls({ mobile = false }: { mobile?: boolean }) {
     return (
       <div
         className={`h-8 ${mobile ? 'w-full' : 'w-20'} bg-muted animate-pulse rounded`}
-        aria-label="Loading auth state"
+        aria-hidden="true"
       />
     );
   }
@@ -143,12 +166,8 @@ function AboutDropdown() {
   return (
     <DropdownMenu>
       <DropdownMenuTrigger asChild>
-        <Button
-          variant="ghost"
-          size="sm"
-          className="flex items-center gap-1"
-          aria-label="About menu"
-        >
+        {/* No aria-label: the visible "About" text is the accessible name. */}
+        <Button variant="ghost" size="sm" className="flex items-center gap-1">
           About <ChevronDownIcon />
         </Button>
       </DropdownMenuTrigger>
@@ -181,6 +200,21 @@ const navLinkClass = ({ isActive }: { isActive: boolean }) =>
     isActive ? 'text-primary' : 'text-muted-foreground'
   }`;
 
+function GitHubLink() {
+  return (
+    <Button asChild variant="ghost" size="icon-sm">
+      <a
+        href={GITHUB_URL}
+        target="_blank"
+        rel="noopener noreferrer"
+        aria-label="Code for Philly on GitHub"
+      >
+        <GitHubIcon />
+      </a>
+    </Button>
+  );
+}
+
 export function AppHeader() {
   const [mobileOpen, setMobileOpen] = useState(false);
 
@@ -201,38 +235,37 @@ export function AppHeader() {
           />
         </Link>
 
-        {/* Desktop nav */}
+        {/* Desktop content cluster. The parent gap is the only source of
+            spacing between children — no per-child margins. */}
         <nav
           aria-label="Primary navigation"
-          className="hidden md:flex items-center gap-1 ml-4 flex-1"
+          className="hidden md:flex items-center gap-2 ml-4 flex-1"
         >
           <NavLink to="/projects" className={navLinkClass}>
             Projects
           </NavLink>
-          <NavLink
-            to="/help-wanted"
-            className={({ isActive }) => navLinkClass({ isActive }) + ' ml-1'}
-          >
+          <NavLink to="/help-wanted" className={navLinkClass}>
             Help Wanted
           </NavLink>
-          <NavLink
-            to="/members"
-            className={({ isActive }) => navLinkClass({ isActive }) + ' ml-1'}
-          >
+          <NavLink to="/members" className={navLinkClass}>
             Members
           </NavLink>
-          <Button asChild size="sm" className="ml-1 bg-green-600 hover:bg-green-700 text-white">
-            <NavLink to="/volunteer">Volunteer</NavLink>
-          </Button>
-          <div className="ml-1">
-            <AboutDropdown />
-          </div>
+          <AboutDropdown />
         </nav>
 
-        {/* Desktop: search + auth */}
+        {/* Desktop utility cluster: GitHub, search, auth, then the Volunteer
+            CTA pinned rightmost (specs/behaviors/app-shell.md). */}
         <div className="hidden md:flex items-center gap-2 ml-auto">
+          <GitHubLink />
           <SearchBox />
           <AuthControls />
+          <Button
+            asChild
+            size="sm"
+            className="bg-green-600 hover:bg-green-700 text-white"
+          >
+            <NavLink to="/volunteer">Volunteer</NavLink>
+          </Button>
         </div>
 
         {/* Mobile: auth + hamburger */}
@@ -240,19 +273,24 @@ export function AppHeader() {
           <AuthControls />
           <Sheet open={mobileOpen} onOpenChange={setMobileOpen}>
             <SheetTrigger asChild>
+              {/* No aria-expanded here — Radix's Dialog.Trigger supplies it. */}
               <Button
                 variant="ghost"
                 size="sm"
                 aria-label="Open navigation menu"
-                aria-expanded={mobileOpen}
               >
                 <MenuIcon />
               </Button>
             </SheetTrigger>
-            <SheetContent side="right" className="w-72 flex flex-col gap-4 pt-8">
+            <SheetContent side="right" className="w-72">
+              {/* SheetHeader/SheetTitle carry the panel's own padding and give
+                  the underlying Radix dialog its accessible name. */}
+              <SheetHeader className="pb-0">
+                <SheetTitle>Menu</SheetTitle>
+              </SheetHeader>
               <nav
                 aria-label="Mobile navigation"
-                className="flex flex-col gap-2"
+                className="flex flex-col gap-2 px-4"
               >
                 <NavLink
                   to="/projects"
@@ -275,13 +313,6 @@ export function AppHeader() {
                 >
                   Members
                 </NavLink>
-                <NavLink
-                  to="/volunteer"
-                  className={navLinkClass}
-                  onClick={() => setMobileOpen(false)}
-                >
-                  Volunteer
-                </NavLink>
                 <Separator />
                 <p className="text-xs text-muted-foreground font-medium uppercase tracking-wide">
                   About
@@ -324,12 +355,32 @@ export function AppHeader() {
                 <a
                   href="mailto:hello@codeforphilly.org"
                   className="text-sm font-medium text-muted-foreground hover:text-primary"
+                  onClick={() => setMobileOpen(false)}
                 >
                   Contact
                 </a>
+                <Separator />
+                <a
+                  href={GITHUB_URL}
+                  target="_blank"
+                  rel="noopener noreferrer"
+                  className="text-sm font-medium text-muted-foreground hover:text-primary"
+                  onClick={() => setMobileOpen(false)}
+                >
+                  GitHub
+                </a>
+                <NavLink
+                  to="/volunteer"
+                  className={navLinkClass}
+                  onClick={() => setMobileOpen(false)}
+                >
+                  Volunteer
+                </NavLink>
               </nav>
               <Separator />
-              <SearchBox inline />
+              <div className="px-4 pb-4">
+                <SearchBox inline />
+              </div>
             </SheetContent>
           </Sheet>
         </div>
diff --git a/apps/web/tests/AppHeader.test.tsx b/apps/web/tests/AppHeader.test.tsx
index 5b58ff4..116d260 100644
--- a/apps/web/tests/AppHeader.test.tsx
+++ b/apps/web/tests/AppHeader.test.tsx
@@ -1,5 +1,5 @@
 import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
-import { screen, waitFor } from '@testing-library/react';
+import { screen, waitFor, within } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { renderWithRouter } from './test-utils.js';
 import { AppHeader } from '../src/components/AppHeader.js';
@@ -38,10 +38,25 @@ describe('AppHeader', () => {
 
   it('renders primary nav links', async () => {
     renderWithRouter(<Wrapped />);
+    const nav = screen.getByRole('navigation', { name: /primary navigation/i });
     expect(screen.getByRole('link', { name: 'Projects' })).toBeInTheDocument();
     expect(screen.getByRole('link', { name: 'Help Wanted' })).toBeInTheDocument();
     expect(screen.getByRole('link', { name: 'Members' })).toBeInTheDocument();
-    expect(screen.getByRole('link', { name: 'Volunteer' })).toBeInTheDocument();
+    expect(within(nav).getByRole('button', { name: 'About' })).toBeInTheDocument();
+
+    // The Volunteer CTA lives in the utility cluster, not the content nav —
+    // it is the rightmost header element (specs/behaviors/app-shell.md).
+    const volunteer = screen.getByRole('link', { name: 'Volunteer' });
+    expect(volunteer).toHaveAttribute('href', '/volunteer');
+    expect(nav).not.toContainElement(volunteer);
+  });
+
+  it('renders the GitHub link in the utility cluster', async () => {
+    renderWithRouter(<Wrapped />);
+    const gh = screen.getByRole('link', { name: 'Code for Philly on GitHub' });
+    expect(gh).toHaveAttribute('href', 'https://github.com/CodeForPhilly');
+    expect(gh).toHaveAttribute('target', '_blank');
+    expect(gh).toHaveAttribute('rel', 'noopener noreferrer');
   });
 
   it('shows Sign in button(s) when anonymous', async () => {
@@ -59,7 +74,8 @@ describe('AppHeader', () => {
     const user = userEvent.setup();
     renderWithRouter(<Wrapped />);
 
-    const aboutBtn = screen.getByRole('button', { name: /about menu/i });
+    // The trigger's visible text is its accessible name — no aria-label.
+    const aboutBtn = screen.getByRole('button', { name: 'About' });
     await user.click(aboutBtn);
 
     await waitFor(() => {
@@ -75,6 +91,8 @@ describe('AppHeader', () => {
 
     const hamburger = screen.getByRole('button', { name: /open navigation menu/i });
     expect(hamburger).toBeInTheDocument();
+    // aria-expanded is supplied by Radix's Dialog.Trigger, not hand-written.
+    expect(hamburger).toHaveAttribute('aria-expanded', 'false');
 
     // Open
     await user.click(hamburger);
@@ -83,6 +101,7 @@ describe('AppHeader', () => {
       // Sheet content includes "Mobile navigation" aria-label
       expect(screen.getByRole('navigation', { name: /mobile navigation/i })).toBeInTheDocument();
     });
+    expect(hamburger).toHaveAttribute('aria-expanded', 'true');
 
     // Close via Escape key
     await user.keyboard('{Escape}');
@@ -91,4 +110,31 @@ describe('AppHeader', () => {
       expect(screen.queryByRole('navigation', { name: /mobile navigation/i })).not.toBeInTheDocument();
     });
   });
+
+  it('gives the mobile sheet dialog an accessible name', async () => {
+    const user = userEvent.setup();
+    renderWithRouter(<Wrapped />);
+
+    await user.click(screen.getByRole('button', { name: /open navigation menu/i }));
+
+    const dialog = await screen.findByRole('dialog', { name: 'Menu' });
+    expect(dialog).toBeInTheDocument();
+  });
+
+  it('lists GitHub and Volunteer in the mobile sheet', async () => {
+    const user = userEvent.setup();
+    renderWithRouter(<Wrapped />);
+
+    await user.click(screen.getByRole('button', { name: /open navigation menu/i }));
+
+    const nav = await screen.findByRole('navigation', { name: /mobile navigation/i });
+    expect(within(nav).getByRole('link', { name: 'GitHub' })).toHaveAttribute(
+      'href',
+      'https://github.com/CodeForPhilly',
+    );
+    expect(within(nav).getByRole('link', { name: 'Volunteer' })).toHaveAttribute(
+      'href',
+      '/volunteer',
+    );
+  });
 });

From 4b7aa7e5464db2a58b8ca08b6a3a712594fba14d Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:02:33 -0400
Subject: [PATCH 4/9] fix(web): repoint dead GitBook links and the stale footer
 repo URL
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The whole codeforphilly.gitbook.io space now returns 404 "Content owner
not found", so both of the Volunteer screen's outbound CTAs were dead.
PR #128 fixed the Home screen's copy of the same URL; these are the last
two in the SPA. "When we meet" goes to the Meetup group, which is where
hack nights are actually announced and which the footer already links to;
"Read the guide" goes to the partnerships repo's first-steps doc, the
surviving source of the GitBook page it replaces.

The footer's "view this site on GitHub" link still named the repo
codeforphilly-rewrite. That only resolves through GitHub's rename
redirect, which is not something to depend on indefinitely.

New Volunteer test asserts both hrefs and, following the Home dead-link
idiom, that no gitbook.io URL survives anywhere in the rendered screen —
so a copy-paste of the old constant cannot come back unnoticed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 apps/web/src/components/AppFooter.tsx |  2 +-
 apps/web/src/screens/Volunteer.tsx    |  9 +--
 apps/web/tests/AppFooter.test.tsx     |  2 +-
 apps/web/tests/Volunteer.test.tsx     | 86 +++++++++++++++++++++++++++
 4 files changed, 93 insertions(+), 6 deletions(-)
 create mode 100644 apps/web/tests/Volunteer.test.tsx

diff --git a/apps/web/src/components/AppFooter.tsx b/apps/web/src/components/AppFooter.tsx
index 620b03f..11fbe62 100644
--- a/apps/web/src/components/AppFooter.tsx
+++ b/apps/web/src/components/AppFooter.tsx
@@ -200,7 +200,7 @@ export function AppFooter() {
             Copyright &copy; Code for Philly {FOUNDED_YEAR}&ndash;{CURRENT_YEAR}
           </p>
           <a
-            href="https://github.com/CodeForPhilly/codeforphilly-rewrite"
+            href="https://github.com/CodeForPhilly/codeforphilly-ng"
             target="_blank"
             rel="noopener noreferrer"
             className="hover:text-foreground transition-colors"
diff --git a/apps/web/src/screens/Volunteer.tsx b/apps/web/src/screens/Volunteer.tsx
index 8b58cf9..458652f 100644
--- a/apps/web/src/screens/Volunteer.tsx
+++ b/apps/web/src/screens/Volunteer.tsx
@@ -5,10 +5,11 @@ import { HelpWantedCard } from '@/components/HelpWantedCard';
 import { useAuth } from '@/hooks/useAuth';
 import { api } from '@/lib/api';
 
-const HACK_NIGHT_URL =
-  'https://codeforphilly.gitbook.io/projects/contributing-to-projects/hack-night-program-details';
+// The codeforphilly.gitbook.io space is gone (404 "Content owner not found"),
+// so both of these point at the surviving live equivalents.
+const MEETUP_URL = 'https://www.meetup.com/Code-for-Philly/';
 const START_PROJECT_URL =
-  'https://codeforphilly.gitbook.io/projects/creating-new-partnerships/first-steps';
+  'https://github.com/CodeForPhilly/partnerships/blob/master/creating-new-partnerships/first-steps.md';
 
 export function Volunteer() {
   const { person } = useAuth();
@@ -69,7 +70,7 @@ export function Volunteer() {
               We meet weekly. Bring your laptop, or just yourself.
             </p>
             <Button asChild variant="outline" size="sm">
-              <a href={HACK_NIGHT_URL} target="_blank" rel="noopener noreferrer">
+              <a href={MEETUP_URL} target="_blank" rel="noopener noreferrer">
                 When we meet →
               </a>
             </Button>
diff --git a/apps/web/tests/AppFooter.test.tsx b/apps/web/tests/AppFooter.test.tsx
index 6b0eb49..8924baf 100644
--- a/apps/web/tests/AppFooter.test.tsx
+++ b/apps/web/tests/AppFooter.test.tsx
@@ -17,7 +17,7 @@ describe('AppFooter', () => {
     expect(link).toBeInTheDocument();
     expect(link).toHaveAttribute(
       'href',
-      'https://github.com/CodeForPhilly/codeforphilly-rewrite',
+      'https://github.com/CodeForPhilly/codeforphilly-ng',
     );
   });
 
diff --git a/apps/web/tests/Volunteer.test.tsx b/apps/web/tests/Volunteer.test.tsx
new file mode 100644
index 0000000..3a45b0c
--- /dev/null
+++ b/apps/web/tests/Volunteer.test.tsx
@@ -0,0 +1,86 @@
+import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
+import { screen, waitFor } from '@testing-library/react';
+import { renderScreen, mockPaginated } from './test-utils.js';
+import { Volunteer } from '../src/screens/Volunteer.js';
+import { AuthProvider } from '../src/hooks/useAuth.js';
+
+const MEETUP_URL = 'https://www.meetup.com/Code-for-Philly/';
+const START_PROJECT_URL =
+  'https://github.com/CodeForPhilly/partnerships/blob/master/creating-new-partnerships/first-steps.md';
+
+describe('Volunteer', () => {
+  beforeEach(() => {
+    vi.spyOn(globalThis, 'fetch').mockImplementation(((input: string) => {
+      if (input.startsWith('/api/auth/me')) {
+        return Promise.resolve(new Response(null, { status: 404 }));
+      }
+      if (input.startsWith('/api/projects')) {
+        return Promise.resolve(
+          new Response(JSON.stringify(mockPaginated([], { totalItems: 268 })), {
+            status: 200,
+            headers: { 'content-type': 'application/json' },
+          }),
+        );
+      }
+      return Promise.resolve(
+        new Response(JSON.stringify(mockPaginated([])), {
+          status: 200,
+          headers: { 'content-type': 'application/json' },
+        }),
+      );
+    }) as typeof fetch);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  function renderVolunteer() {
+    return renderScreen(
+      <AuthProvider>
+        <Volunteer />
+      </AuthProvider>,
+    );
+  }
+
+  it('renders the hero headline', () => {
+    renderVolunteer();
+    expect(
+      screen.getByRole('heading', {
+        name: /volunteer with code for philly/i,
+        level: 1,
+      }),
+    ).toBeInTheDocument();
+  });
+
+  it('points "When we meet →" at the live Meetup group, not the dead GitBook page', () => {
+    renderVolunteer();
+    const link = screen.getByRole('link', { name: /when we meet/i });
+    expect(link).toHaveAttribute('href', MEETUP_URL);
+    expect(link).toHaveAttribute('target', '_blank');
+    expect(link).toHaveAttribute('rel', 'noopener noreferrer');
+  });
+
+  it('points "Read the guide →" at the partnerships repo, not the dead GitBook page', () => {
+    renderVolunteer();
+    const link = screen.getByRole('link', { name: /read the guide/i });
+    expect(link).toHaveAttribute('href', START_PROJECT_URL);
+    expect(link).toHaveAttribute('target', '_blank');
+    expect(link).toHaveAttribute('rel', 'noopener noreferrer');
+  });
+
+  it('has no codeforphilly.gitbook.io links anywhere on the screen', async () => {
+    const { container } = renderVolunteer();
+
+    // Wait for the live project count so the fully-settled DOM is asserted on.
+    await waitFor(() => {
+      expect(screen.getByText(/browse 268 active projects/i)).toBeInTheDocument();
+    });
+
+    const hrefs = Array.from(container.querySelectorAll('a')).map(
+      (a) => a.getAttribute('href') ?? '',
+    );
+    expect(hrefs.filter((h) => h.includes('gitbook.io'))).toHaveLength(0);
+    expect(container.innerHTML).not.toContain('codeforphilly.gitbook.io');
+  });
+});

From 4c2e392bff3473dc84224af85c9fa23ad57481c4 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:04:30 -0400
Subject: [PATCH 5/9] fix(web): let the mobile sheet nav scroll when it
 overflows

The sheet is a full-height flex column with no scroll container, so a nav
list taller than the viewport was simply unreachable below the fold. That
was already latent; adding the GitHub row makes it one row likelier on
short phones. min-h-0 lets the flex child shrink at all, and overflow-y-auto
gives it somewhere to put the excess.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 apps/web/src/components/AppHeader.tsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/apps/web/src/components/AppHeader.tsx b/apps/web/src/components/AppHeader.tsx
index dd414de..7991907 100644
--- a/apps/web/src/components/AppHeader.tsx
+++ b/apps/web/src/components/AppHeader.tsx
@@ -288,9 +288,11 @@ export function AppHeader() {
               <SheetHeader className="pb-0">
                 <SheetTitle>Menu</SheetTitle>
               </SheetHeader>
+              {/* min-h-0 + overflow-y-auto so the list stays reachable on
+                  short viewports instead of overflowing the panel. */}
               <nav
                 aria-label="Mobile navigation"
-                className="flex flex-col gap-2 px-4"
+                className="flex flex-col gap-2 px-4 min-h-0 overflow-y-auto"
               >
                 <NavLink
                   to="/projects"

From 2d7b986fec4c7aaf6527b60054c48bcb17b888f1 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:24:58 -0400
Subject: [PATCH 6/9] chore(plans): record site-check-153 validation results
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Ticks the criteria verified during implementation and, deliberately,
leaves two unticked rather than rewriting them to match what was
achievable: the browser pass belongs to whoever does UI QA, and the
all-workspaces test gate cannot close on this Windows dev box.

Notes record why. apps/api fails ten tests here on a tree whose API code
is byte-identical to develop, because the fixtures assume POSIX —
store.test.ts injects a write failure via /dev/null/impossible-path,
which Windows will happily create, so the expected rejection never comes.
Reproduces with the files run alone, so it is not runner contention.
Filed as a follow-up rather than fixed: cross-platform fixtures are their
own scope, and silently ticking a gate that did not run is worse than
leaving the box open.

Also flags HomeStub.tsx, which carries the same stale repo URL the footer
had but is imported by nothing.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 plans/site-check-153.md | 53 +++++++++++++++++++++++++++++++----------
 1 file changed, 41 insertions(+), 12 deletions(-)

diff --git a/plans/site-check-153.md b/plans/site-check-153.md
index d288c9a..5a18097 100644
--- a/plans/site-check-153.md
+++ b/plans/site-check-153.md
@@ -106,17 +106,20 @@ One-line repo URL swap to `codeforphilly-ng`.
 
 ## Validation
 
-- [ ] Specs updated before code: app-shell header clusters + volunteer link targets.
-- [ ] Desktop header order is Projects · Help Wanted · Members · About ▾ … GitHub · Search · Sign in · Volunteer, with Volunteer rightmost and still green.
-- [ ] No `ml-1` spacing hacks remain among the header nav's children.
-- [ ] GitHub link is icon-only, labelled "Code for Philly on GitHub", and opens `https://github.com/CodeForPhilly` in a new tab with `rel="noopener noreferrer"`.
-- [ ] Mobile sheet has a "Menu" title, horizontal padding on nav + search, and no `pt-8`; the title does not collide with the close button.
-- [ ] The sheet dialog exposes an accessible name; Radix still supplies `aria-expanded` on the trigger.
-- [ ] Loading skeleton uses `aria-hidden`; About trigger's accessible name is its visible text; account-menu label retained.
-- [ ] Every mobile sheet item closes the sheet on click, including Contact.
-- [ ] No `codeforphilly.gitbook.io` URL remains in `apps/web/src`.
-- [ ] Footer "view this site on GitHub" points at `codeforphilly-ng`.
-- [ ] `npm run -w packages/shared build && npm run type-check && npm run lint && npm test` clean.
+- [x] Specs updated before code: app-shell header clusters + volunteer link targets.
+- [x] Desktop header order is Projects · Help Wanted · Members · About ▾ … GitHub · Search · Sign in · Volunteer, with Volunteer rightmost and still green.
+- [x] No `ml-1` spacing hacks remain among the header nav's children.
+- [x] GitHub link is icon-only, labelled "Code for Philly on GitHub", and opens `https://github.com/CodeForPhilly` in a new tab with `rel="noopener noreferrer"`.
+- [x] Mobile sheet has a "Menu" title, horizontal padding on nav + search, and no `pt-8`; the title does not collide with the close button.
+- [x] The sheet dialog exposes an accessible name; Radix still supplies `aria-expanded` on the trigger.
+- [x] Loading skeleton uses `aria-hidden`; About trigger's accessible name is its visible text; account-menu label retained.
+- [x] Every mobile sheet item closes the sheet on click, including Contact.
+- [x] No `codeforphilly.gitbook.io` URL remains in `apps/web/src`.
+- [x] Footer "view this site on GitHub" points at `codeforphilly-ng`.
+- [x] Both replacement URLs return 200 and carry the expected content.
+- [x] `npm run -w packages/shared build`, `npm run type-check`, and `npm run lint` clean.
+- [x] `npm test` clean for the workspaces this plan touches: web 96/96, shared 75/75.
+- [ ] `npm test` clean for **all** workspaces — `apps/api` cannot pass on the Windows dev box used here (see Notes); needs a Linux run or CI to close.
 - [ ] Browser test: desktop header order + mobile sheet padding at < md, both breakpoints.
 
 ## Risks
@@ -131,7 +134,20 @@ One-line repo URL swap to `codeforphilly-ng`.
 
 ## Notes
 
-(To be populated at closeout.)
+(To be populated at closeout. Recorded during implementation:)
+
+- **`apps/api` tests do not pass on Windows, independent of this plan.** Ten
+  failures across `scrub-data.test.ts` (4), `internal-reload.test.ts` (4), and
+  `store.test.ts` (2), on a tree where `git diff develop..HEAD -- apps/api
+  packages/` is empty — this branch touches no API code. The mechanism is
+  POSIX-isms in the test fixtures: `store.test.ts` injects a write failure by
+  pointing the private store at `/dev/null/impossible-path` and asserting the
+  transaction rejects, but on Windows that is an ordinary creatable directory,
+  so the write succeeds and the expected throw never happens. They reproduce
+  with the files run alone, so it is not test-runner contention. CI runs the
+  same gate on Linux, where the fixture behaves as intended. Worth a
+  cross-platform fixture cleanup if Windows dev boxes are to be supported;
+  filed under Follow-ups.
 
 ## Follow-ups
 
@@ -149,3 +165,16 @@ One-line repo URL swap to `codeforphilly-ng`.
   worker, unsubscribe tokens). Building an anonymous-capture CTA ahead of that
   spec would invent unspecified behavior. `Home.tsx` is deliberately untouched
   here; the CTA swap should follow the newsletter spec work, not precede it.
+
+- **Tracked as: dead file, not fixed here — `apps/web/src/pages/HomeStub.tsx`.**
+  It carries the same stale `codeforphilly-rewrite` URL the footer had, but
+  nothing imports or routes it (`App.tsx` imports only `LoginPlaceholder` from
+  `src/pages/`; every live screen lives in `src/screens/`). Left alone because
+  the right fix is deleting the file, not patching a URL nobody renders — and
+  that deletion wants its own scope. Flagging so a future grep for the old repo
+  name doesn't read as an unfixed live link.
+
+- **Issue — make the `apps/api` test fixtures cross-platform.** The `/dev/null`
+  failure-injection idiom (and whatever the other seven failures share) makes
+  the API suite unrunnable on a Windows dev box, so the documented validation
+  gate can only be completed on Linux or in CI. See Notes for the mechanism.

From 848bbf0c4b207678afb0f508f2cd7692de725d3f Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:48:12 -0400
Subject: [PATCH 7/9] chore(plans): record exact api test counts in
 site-check-153 notes

Pins the numbers behind the unticked all-workspaces gate so a future
reader can tell a known Windows baseline from a real regression, and
notes the develop re-run that confirms it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 plans/site-check-153.md | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/plans/site-check-153.md b/plans/site-check-153.md
index 5a18097..1f0fe6d 100644
--- a/plans/site-check-153.md
+++ b/plans/site-check-153.md
@@ -136,10 +136,13 @@ One-line repo URL swap to `codeforphilly-ng`.
 
 (To be populated at closeout. Recorded during implementation:)
 
-- **`apps/api` tests do not pass on Windows, independent of this plan.** Ten
-  failures across `scrub-data.test.ts` (4), `internal-reload.test.ts` (4), and
-  `store.test.ts` (2), on a tree where `git diff develop..HEAD -- apps/api
-  packages/` is empty — this branch touches no API code. The mechanism is
+- **`apps/api` tests do not pass on Windows, independent of this plan.**
+  `apps/api` finishes 3 failed | 30 passed (33 files), 10 failed | 413 passed
+  (423 tests) — the ten spread across `scrub-data.test.ts` (4),
+  `internal-reload.test.ts` (4), and `store.test.ts` (2), on a tree where
+  `git diff develop..HEAD -- apps/api packages/` is empty — this branch touches
+  no API code. Checking out `develop` and re-running `store.test.ts` there
+  reproduces its 2 failed | 11 passed exactly. The mechanism is
   POSIX-isms in the test fixtures: `store.test.ts` injects a write failure by
   pointing the private store at `/dev/null/impossible-path` and asserting the
   transaction rejects, but on Windows that is an ordinary creatable directory,

From b91aaef24696a1a4c736a13e0553b135a2926bf1 Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:52:56 -0400
Subject: [PATCH 8/9] chore(plans): record browser validation for
 site-check-153
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Desktop header verified in headed Chrome at 1400px; the sheet verified via
its portal at desktop width because the automation harness could not shrink
a maximized window below md — same limitation web-shell.md's plan recorded.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 plans/site-check-153.md | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/plans/site-check-153.md b/plans/site-check-153.md
index 1f0fe6d..9178493 100644
--- a/plans/site-check-153.md
+++ b/plans/site-check-153.md
@@ -120,7 +120,15 @@ One-line repo URL swap to `codeforphilly-ng`.
 - [x] `npm run -w packages/shared build`, `npm run type-check`, and `npm run lint` clean.
 - [x] `npm test` clean for the workspaces this plan touches: web 96/96, shared 75/75.
 - [ ] `npm test` clean for **all** workspaces — `apps/api` cannot pass on the Windows dev box used here (see Notes); needs a Linux run or CI to close.
-- [ ] Browser test: desktop header order + mobile sheet padding at < md, both breakpoints.
+- [x] Browser test (headed Chrome, Vite dev server): desktop header order,
+      spacing, GitHub icon, and Volunteer-rightmost verified at 1400px; the
+      sheet verified open — "Menu" title, padded nav/search, no title/close
+      collision — and Escape closes it. Caveat: the harness could not shrink
+      the (maximized) window below md, so the sheet was opened via its
+      CSS-hidden trigger at desktop width. The sheet is a fixed `w-72`
+      portal, so its rendering is identical at < md; the < md *header bar*
+      (logo + auth + hamburger row) still rides on the jsdom tests, same
+      limitation `web-shell.md:109` recorded.
 
 ## Risks
 

From 8c94dc83e5127b70a0fe067254e134507f48f2bf Mon Sep 17 00:00:00 2001
From: Heyoub <hello@forgestack.app>
Date: Mon, 24 Aug 2026 12:54:45 -0400
Subject: [PATCH 9/9] chore(plans): mark site-check-153 done (PR #154)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 plans/site-check-153.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/plans/site-check-153.md b/plans/site-check-153.md
index 9178493..021ed1a 100644
--- a/plans/site-check-153.md
+++ b/plans/site-check-153.md
@@ -1,10 +1,11 @@
 ---
-status: in-progress
+status: done
 depends: []
 specs:
   - specs/behaviors/app-shell.md
   - specs/screens/volunteer.md
 issues: [153]
+pr: 154
 ---
 
 # Plan: site check — header order, mobile sheet padding, dead outbound links