Per Bert's request (2026-06-03). Replaces the 'developer pushes manually'
rule with a Push & Merge Authority policy: Claude may push feature branches,
open Gitea PRs, and merge them without a separate approval click, provided
the merge gate is green (reviewer PASS, tests/lint/typecheck clean, backend
guards where applicable). Never push directly to main, never force-push,
always --no-ff via a reviewed branch, delete the branch post-merge. CLAUDE.md
supersedes the build-module skill's HUMAN GATE 2.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Review follow-up (crewli-reviewer). With header:'!hidden' the default close
button was still mounted (display:none); PrimeVue Drawer.focus() falls back to
focusing it (no [autofocus] in our slots), a no-op that strands keyboard focus
behind the overlay. Set :show-close-icon="false" so the button is never
created — focus() no-ops cleanly and v-focustrap focuses the visible brand-row
X. Also: flip the previously tautological showCloseIcon test to assert the
decision (toBe(false)); align the mobile close aria-label to English 'Close
menu' (matching 'Collapse sidebar') for locale consistency.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Defects 1 (logo) and 2 (close-X overlap) fixed and tested. Defect 3
(WorkspaceSwitcher) audited as already-correct full-height flex column with a
min-h-0 guard; not device-verified, dynamic-viewport-height flagged as the
remaining suspect. Records the resolution per the GRADIENT-BRAND-ALIGNMENT
convention.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
MOBILE-SHELL-PARITY subtask 5. Automated visual evidence via Playwright-CT is
not feasible — the CT harness loads only Vuetify styles (no Tailwind / PrimeVue
theme / PrimeVue plugin), so an AppSidebar screenshot renders unstyled. Add a
MobileDrawer Storybook story instead (Storybook DOES register PrimeVue + load
Tailwind via preview.ts) as the manual visual-verification target for all three
defects: single non-overlapping close X, expanded brand row/logo, and the
bottom-anchored WorkspaceSwitcher. Narrow the window <1024px to mount the drawer
(no viewport addon in this repo's Storybook).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
MOBILE-SHELL-PARITY. Adds 7 Vitest assertions for the three defects:
- AppSidebar: header pt is '!hidden' (default close-X suppressed), content pt
is a full-height flex column (flex-col/h-full/min-h-0), showCloseIcon is not
forced false, and WorkspaceSwitcher renders inside the drawer.
- SidebarHeader: the Icon stub now exposes data-icon; mobile brand-row control
is an explicit close (aria-label 'Sluit menu', tabler-x), desktop stays the
collapse chevron, and the header renders expanded on mobile even when
sidebarCollapsed is true (logo parity).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
MOBILE-SHELL-PARITY defect 1. SidebarHeader read shell.sidebarCollapsed
directly, so a desktop-collapsed state made the full-width (256px) mobile
drawer render a collapsed brand row (logo-only, no wordmark, expand-chevron
row) while AppSidebar forces SidebarNav + WorkspaceSwitcher to expanded —
the mismatch read as incorrect logo placement. Gate all collapse-dependent
branches on a new effectiveCollapsed computed (!isMobile && sidebarCollapsed):
desktop honours the collapse state, mobile is never collapsed so the drawer
header matches the expanded nav/switcher. Also keeps the subtask-1 close X
(on the expanded-row control) reliably visible on mobile.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
MOBILE-SHELL-PARITY defect 3. Static audit (code + @primeuix base CSS) shows
the mobile drawer content is ALREADY a correct full-height flex column:
.p-drawer-left .p-drawer-content gets height:100% + flex-grow:1 from base
styles, our pt makes it flex flex-col, SidebarNav (flex-1 min-h-0) claims the
slack and WorkspaceSwitcher (flex-shrink-0) anchors at the bottom — matching
the desktop aside. Add min-h-0 as flex hygiene (children shrink, nav scrolls
internally) and document the chain in-code, citing the @primeuix rules.
The 'broken height chain' premise did not match the code, so no redundant
height fix was fabricated. If WorkspaceSwitcher is still clipped on a real
mobile device, the remaining suspect is dynamic-viewport-height (a shrinking
address bar making 100% exceed the visual viewport) — flagged for a
real-device visual check at the merge gate.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
MOBILE-SHELL-PARITY defect 2. PrimeVue's default drawer header close-X was
leaking through 'header: hidden' (non-important 'hidden' lost to PrimeVue's
base .p-drawer-header display in stylesheet order) and overlapping the brand
row. Force-hide it with the important variant '!hidden' (matching the file's
existing !w-64), and provide the mobile close affordance in SidebarHeader's
brand row: on mobile the top-right control renders an X ('Sluit menu') and
closes the drawer (handleCollapseClick already calls setMobileOpen(false) on
mobile), mirroring the slot the desktop sidebar uses for its collapse chevron.
Single, non-overlapping close control; showCloseIcon is left at its default.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Isolated subagents (frontend/backend-implementer) run in a git worktree
created under .claude/worktrees/. The protect-files PreToolUse hook blocked
all edits to any .claude/ path, making every worktree-isolated agent unable
to edit its own checkout. Exempt .claude/worktrees/ (ephemeral agent scratch
space) while still protecting the real tooling config under .claude/.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
