From 7cb1df6d953713916ede6ba77bcd135ddd9bf699 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sat, 24 Jan 2026 20:42:32 +0000
Subject: [PATCH] fix(unified-session): resolve circular dependency with TimeMs

The import of TimeMs from Defaults/index.ts caused a circular
dependency error in ESM:
"ReferenceError: Cannot access 'TimeMs' before initialization"

Solution: Define TimeMs constants locally in unified-session.ts
instead of importing from Defaults. This avoids the ESM module
initialization order issue.
---
 src/Utils/unified-session.ts                | 21 ++++++++++++++++-----
 src/__tests__/Utils/unified-session.test.ts | 10 +++++++++-
 2 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/src/Utils/unified-session.ts b/src/Utils/unified-session.ts
index 06948242..55d82abb 100644
--- a/src/Utils/unified-session.ts
+++ b/src/Utils/unified-session.ts
@@ -27,11 +27,22 @@ import {
 // Time Constants (Enterprise-grade)
 // ============================================
 
-// Import TimeMs from Defaults (single source of truth)
-import { TimeMs } from '../Defaults/index.js'
-
-// Re-export for convenience (but not as main export to avoid conflicts)
-export { TimeMs }
+/**
+ * Time constants in milliseconds for unified session calculations.
+ * Defined locally to avoid circular dependency issues with Defaults/index.ts
+ */
+const TimeMs = {
+	/** One second in milliseconds */
+	Second: 1_000,
+	/** One minute in milliseconds */
+	Minute: 60_000,
+	/** One hour in milliseconds */
+	Hour: 3_600_000,
+	/** One day in milliseconds */
+	Day: 86_400_000,
+	/** One week in milliseconds */
+	Week: 604_800_000,
+} as const
 
 /**
  * Offset for session ID calculation (3 days in ms).
diff --git a/src/__tests__/Utils/unified-session.test.ts b/src/__tests__/Utils/unified-session.test.ts
index 68e859d8..d0568616 100644
--- a/src/__tests__/Utils/unified-session.test.ts
+++ b/src/__tests__/Utils/unified-session.test.ts
@@ -14,7 +14,15 @@ import {
 	shouldEnableUnifiedSession,
 	type UnifiedSessionOptions
 } from '../../Utils/unified-session.js'
-import { TimeMs } from '../../Defaults/index.js'
+
+// TimeMs constants for testing (matching unified-session.ts)
+const TimeMs = {
+	Second: 1_000,
+	Minute: 60_000,
+	Hour: 3_600_000,
+	Day: 86_400_000,
+	Week: 604_800_000,
+} as const
 
 // Mock the prometheus metrics to avoid side effects
 jest.mock('../../Utils/prometheus-metrics.js', () => ({