From 700c399367a95c2cfd3ada3a220045a89a0c7ef5 Mon Sep 17 00:00:00 2001 From: shankar0123 Date: Thu, 14 May 2026 17:16:40 +0000 Subject: [PATCH] =?UTF-8?q?chore(web):=20remove=20darkMode:=20'class'=20fr?= =?UTF-8?q?om=20tailwind=20config=20=E2=80=94=20Phase=207=20retired?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Operator decision 2026-05-14: "no dark mode and no future dark mode wiring to maintain." The originally-optional Phase 7 (the rebuild path that would have superseded Phase 0's rip-out if customer signal materialized) is formally retired in the frontend-design-audit.html banner stack + Phase 7 H3 header. Phase 0's closure rationale ("leave `darkMode: 'class'` in tailwind config for the eventual Phase 7 rebuild") is now superseded — keeping that line set would resurface as the same half-wired-hook pattern that drove the original FE-H1 finding, just at the config layer instead of the HTML layer. Phase 0 removed `class="dark"` from + the body `bg-slate-900`; this commit closes the loop by also removing the tailwind config option that pointed at a future feature that won't arrive. If the decision ever reverses, this line restores in a one-diff revert + a full re-audit of every primitive and page for `dark:` variants (see the retired Phase 7 executable prompt for the rules: ship complete or not at all; piecemeal dark-mode is exactly the original finding). Verification: • npx tsc --noEmit — exits 0 • npx vite build — ✓ built in 3.20s (Tailwind doesn't need darkMode set to compile; output is identical because there are zero `dark:` classes in src/ to gate behind anything) • Audit HTML (workspace-only, not repo-tracked) updated with: - Phase 7 RETIRED banner at top of banner stack (amber accent) - Phase 7 H3 header flipped to "✗ Retired 2026-05-14" - FE-H1 row note extended with the lock-in decision - Phase 0's "Do NOT delete darkMode: 'class'" guidance struck through + marked SUPERSEDED with a pointer to the new banner --- web/tailwind.config.cjs | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/web/tailwind.config.cjs b/web/tailwind.config.cjs index 7be2141..9819292 100644 --- a/web/tailwind.config.cjs +++ b/web/tailwind.config.cjs @@ -4,7 +4,16 @@ module.exports = { "./index.html", "./src/**/*.{js,ts,jsx,tsx}", ], - darkMode: 'class', + // Dark mode intentionally NOT wired. Phase 0 ripped out the dead + // class="dark" + residual dark: classes; the audit prompt for the + // optional Phase 7 was rejected by the operator on 2026-05-14 with + // the explicit posture: "no dark mode and no future dark mode + // wiring to maintain". Leaving `darkMode: 'class'` set would + // resurface as exactly the kind of half-wired hook that drove the + // original FE-H1 finding, so removing it locks in the decision at + // the config layer. If the decision ever reverses, restore this + // line + re-audit every primitive and page for dark: variants + // (NOT a piecemeal migration — see retired Phase 7 in the audit). theme: { extend: { colors: {