fix: address code review — reduce E2E timeout, remove error filter, fix parse fn

Agent-Logs-Url: https://github.com/nitrocode/token-deathclock/sessions/e7e66a7b-8a0e-44c0-91ca-473ea4a3dcd3

Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>

Author: Copilot

chart-date-adapter.js

@@ -45,8 +45,8 @@
       if (value === null || value === undefined || value === '') return null;
       if (value instanceof Date) return isNaN(value.getTime()) ? null : value.getTime();
       if (typeof value === 'number') return value;
-      var d = new Date(value);
-      return isNaN(d.getTime()) ? null : d.getTime();
+      var ms = new Date(value).getTime();
+      return isNaN(ms) ? null : ms;
     },
 
     // Format a Unix timestamp (ms) using a simple format string.

script.js

@@ -397,9 +397,9 @@
       ${state !== 'dead' ? 'tabindex="0" role="button"' : 'aria-disabled="true"'}>${content}</div>`;
   }
 
-  // Maximum days to render in life-blocks view.
+  // Maximum days to render in life-blocks view (~10 years).
   // Beyond this, a summary block is shown to avoid creating millions of DOM elements.
-  const MAX_LB_DAYS = 3650; // 10 years
+  const MAX_LB_DAYS = 3650;
 
   // ---- View renderers ----
 

tests/e2e/death-clock.spec.js

@@ -16,7 +16,7 @@ const { test, expect } = require('@playwright/test');
 /**
  * Wait until an element's textContent is non-empty and not "Loading…".
  */
-async function waitForCounter(page, selector, timeout = 8000) {
+async function waitForCounter(page, selector, timeout = 3000) {
   await expect(async () => {
     const text = await page.locator(selector).textContent();
     expect(text).toBeTruthy();
@@ -196,9 +196,7 @@ test.describe('AI Death Clock — end-to-end', () => {
     page.on('pageerror', (err) => errors.push(err.message));
     await page.reload();
     await page.waitForLoadState('networkidle');
-    // Filter out expected/benign Chart errors if adapter is missing in test env
-    const fatal = errors.filter((e) => !e.includes('date adapter'));
-    expect(fatal).toHaveLength(0);
+    expect(errors).toHaveLength(0);
   });
 
   // ── Security: no XSS via milestone content ────────────────────────────────