fix(sentry): auto-reload on ChunkLoadError; order form warn not error (REQ-0014/0015)

REQ-0014: ErrorBoundary.componentDidCatch detects ChunkLoadError by name/message
and calls window.location.reload() once (sessionStorage loop guard) instead of
reporting to Sentry. Fixes post-deploy stale-chunk Sentry noise for demo users.

REQ-0015: OrderDialog RHF invalid-submit callback changed from logger.error to
logger.warn. Client-side form validation failures are expected UX, not server
errors — logger.error routes to Sentry, logger.warn does not.

Automated: lint ✓  test 284 ✓  invalidate 200 ✓  build ✓
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: arnobt78

.agile-v/DECISION_LOG.md

@@ -20,3 +20,6 @@ Format: `TIMESTAMP | AGENT | DECISION | RATIONALE | REQ-ID`
 2026-05-19T00:00:00Z | build-agent | Export error HTTP helpers from lib/api | Single import path for hooks and logger consumers | REQ-0012
 2026-05-19T00:00:00Z | build-agent | Track SENTRY audit in docs/ | Historical cases + status header; agile-v pointers | REQ-0009
 2026-05-19T00:00:00Z | build-agent | Remaining API Zod safeParse sweep | Payment/shipping/notifications/auth/AI; logger.warn everywhere | REQ-0013
+2026-06-27T00:00:00Z | build-agent | ChunkLoadError auto-reload in ErrorBoundary | Stale Vercel chunk after deploy; reload once with sessionStorage loop guard; skip Sentry | REQ-0014
+2026-06-27T00:00:00Z | build-agent | OrderDialog logger.error → logger.warn for RHF invalid callback | Client-side form validation is expected UX path, not a server error; logger.error routes to Sentry | REQ-0015
+2026-06-27T00:00:00Z | red-team | Hydration on /admin/dashboard-overall-insights MONITOR only | Single demo user (test@admin.com) Asia/Karachi tz; no date component change warranted yet | REQ-0009

.agile-v/REQUIREMENTS.md

@@ -214,6 +214,48 @@ Canonical REQ source. All artifacts link via `REQ-XXXX`. Status: `done` | `verif
 
 ---
 
+## REQ-0014 — ChunkLoadError auto-reload in ErrorBoundary
+
+| Field | Value |
+|-------|-------|
+| **Priority** | P1 |
+| **Risk** | R2 |
+| **Status** | done |
+
+**Intent:** After a Vercel deploy, users with stale tab/page receive `ChunkLoadError` when a lazy import tries to fetch a now-invalidated chunk hash. Currently `ErrorBoundary.componentDidCatch` logs and reports to Sentry and shows a crash UI. The correct fix is to silently auto-reload on `ChunkLoadError`, restoring the user to the fresh deploy without a Sentry event.
+
+**Acceptance criteria**
+
+- AC1: `ErrorBoundary.componentDidCatch` detects `ChunkLoadError` by name and triggers `window.location.reload()` — no Sentry capture, no crash UI shown
+- AC2: Non-`ChunkLoadError` errors continue to report to Sentry and show fallback UI unchanged
+- AC3: A `sessionStorage` guard prevents an infinite reload loop (reload once, then fall through to crash UI)
+- AC4: `ErrorBoundary.tsx` updated; `app/layout.tsx` unchanged
+
+**Artifacts:** `components/shared/ErrorBoundary.tsx`
+
+---
+
+## REQ-0015 — OrderDialog RHF validation logger level
+
+| Field | Value |
+|-------|-------|
+| **Priority** | P1 |
+| **Risk** | R1 |
+| **Status** | done |
+
+**Intent:** `OrderDialog.tsx:949` calls `logger.error("Order form validation errors:", errors)` in the RHF `handleSubmit` invalid callback. This fires for pure client-side form validation failures (missing fields, wrong type) — it never reaches any API. `logger.error` routes through Sentry; `logger.warn` does not. Client-side form validation is expected UX feedback, not a production error.
+
+**Acceptance criteria**
+
+- AC1: `logger.error` at line ~949 changed to `logger.warn`
+- AC2: `console.error` at line ~945 changed to `console.warn` (debug noise reduction)
+- AC3: API-level errors at lines ~487 and ~628 (`logger.error("Order creation error:", ...)`, `logger.error("Order update error:", ...)`) remain as `logger.error` (those are genuine server failures)
+- AC4: No other logic changed in `OrderDialog.tsx`
+
+**Artifacts:** `components/orders/OrderDialog.tsx`
+
+---
+
 ## REQ-0013 — Remaining API Zod consistency
 
 | Field | Value |

.agile-v/STATE.md

@@ -3,20 +3,21 @@
 | Field | Value |
 |-------|-------|
 | **Cycle** | C1 |
-| **Phase** | `phases/01-sentry-groq-select` |
-| **Infinity Loop stage** | Verify → Evolve (Human Gate 2 pending) |
-| **Last updated** | 2026-05-19 |
-| **Active REQ range** | REQ-0001 … REQ-0013 |
-| **Prod deploy target** | `9a2e37c` (REQ-0013); docs HEAD `c0e15b3` |
+| **Phase** | `phases/02-sentry-chunk-order-fix` |
+| **Infinity Loop stage** | Prove → Verify (REQ-0014/0015 code done, lint+test PASS, awaiting deploy) |
+| **Last updated** | 2026-06-27 |
+| **Active REQ range** | REQ-0001 … REQ-0015 |
+| **Prod deploy target** | pending push (REQ-0014/0015); baseline `9a2e37c` |
 | **Human Gate 1** | APPROVED (retroactive bootstrap) |
-| **Human Gate 2** | PENDING — Sentry 24h + manual nav smoke |
+| **Human Gate 2** | PENDING — Sentry 24h + manual nav smoke; new REQ-0014/0015 pending automated pass |
 | **Resume token** | — |
 
 ## Current focus
 
-1. **REQ-0009** — 24h Sentry regression after prod deploy (`docs/SENTRY_ERRORS.md`)
-2. **REQ-0001/0006** — manual removeChild nav smoke (optional)
-3. **C2** — new work only via new REQ-XXXX + Infinity Loop
+1. **REQ-0014** — ChunkLoadError auto-reload in `ErrorBoundary` (stale deploy chunk)
+2. **REQ-0015** — `OrderDialog` `logger.error` → `logger.warn` for RHF client-side validation callback
+3. **REQ-0009** — Sentry regression watch: hydration on `/admin/dashboard-overall-insights` MONITOR only
+4. **REQ-0001/0006** — manual removeChild nav smoke (optional)
 
 ## Session resume (every chat)
 

.agile-v/VALIDATION_SUMMARY.md

@@ -1,8 +1,8 @@
 # Validation Summary — Cycle C1
 
-**Generated:** 2026-05-28  
+**Generated:** 2026-06-27  
 **eval_gate_status:** PENDING (Human Gate 2)  
-**Red Team:** automated pass; manual prod QA partial
+**Red Team:** automated pass; manual prod QA partial; REQ-0014/0015 lint+test PASS
 
 ---
 
@@ -38,6 +38,9 @@
 | VS-004 | INFO | Products POST/PUT Zod + 4xx logger guard | PASS (automated) |
 | VS-005 | INFO | Catalog CRUD Zod + API barrel exports | PASS (automated) |
 | VS-006 | INFO | Remaining API Zod sweep (payment/shipping/auth/AI) | PASS (automated) |
+| VS-007 | INFO | ChunkLoadError auto-reload in ErrorBoundary | PASS (lint + 284 tests) |
+| VS-008 | INFO | OrderDialog RHF validation logger level | PASS (lint + 284 tests) |
+| VS-009 | INFO | Hydration /admin/dashboard-overall-insights | MONITOR (single demo user) |
 
 ---
 
@@ -48,6 +51,8 @@
 - [ ] Bell dropdown QA (post-deploy)
 - [ ] Dialog nav smoke (OrderDialog)
 - [ ] Sentry 24h review (`.agile-v/REVALIDATION_LOG.md`)
+- [ ] lint + test + build pass for REQ-0014/0015
+- [ ] Deploy REQ-0014/0015 and confirm no new ChunkLoadError / Order validation Sentry events
 
 **Approver:** _pending_  
 **Date:** _pending_

LICENSE

@@ -1,5 +1,7 @@
 MIT License
 
+SPDX-License-Identifier: MIT
+
 Copyright (c) 2026 Arnob Mahmud
 
 Permission is hereby granted, free of charge, to any person obtaining a copy

components/orders/OrderDialog.tsx

@@ -941,12 +941,12 @@ export default function OrderDialog({
               onSubmit={createFormMethods.handleSubmit(
                 handleCreateOrder,
                 (errors) => {
-                  // Log validation errors to console for debugging
-                  console.error(
+                  // Client-side RHF validation failure — expected UX path, not a server error
+                  console.warn(
                     "Form validation errors:",
                     JSON.stringify(errors, null, 2),
                   );
-                  logger.error("Order form validation errors:", errors);
+                  logger.warn("Order form validation errors:", errors);
 
                   // Helper function to extract error messages from nested objects
                   const extractErrorMessages = (

components/shared/ErrorBoundary.tsx

@@ -39,6 +39,23 @@ export class ErrorBoundary extends Component<Props, State> {
   }
 
   public componentDidCatch(error: Error, errorInfo: ErrorInfo) {
+    // ChunkLoadError fires when a Vercel deploy invalidates a cached JS chunk that the
+    // user's stale tab still references. Silently reload once to pick up the new build.
+    // A sessionStorage guard prevents an infinite reload loop if the chunk is genuinely missing.
+    if (
+      error.name === "ChunkLoadError" ||
+      error.message?.includes("Failed to load chunk")
+    ) {
+      const key = "chunk_reload_attempted";
+      if (!sessionStorage.getItem(key)) {
+        sessionStorage.setItem(key, "1");
+        window.location.reload();
+        return;
+      }
+      // Already tried once — fall through to normal error handling below
+      sessionStorage.removeItem(key);
+    }
+
     logger.error("ErrorBoundary caught an error:", {
       error: error.message,
       stack: error.stack,