refactor: use ??= operator for cleaner globalThis singleton init

Drickon · steipete · commit d0a3743abdd1 · 2026-03-03T00:27:10.000Z
Addresses greptile review: collapses the if-guard + assignment into
a single ??= expression so TypeScript can narrow the type without
a non-null assertion.
diff --git a/src/hooks/internal-hooks.ts b/src/hooks/internal-hooks.ts
@@ -213,10 +213,10 @@ export type InternalHookHandler = (event: InternalHookEvent) => Promise<void> |
 const _g = globalThis as typeof globalThis & {
   __openclaw_internal_hook_handlers__?: Map<string, InternalHookHandler[]>;
 };
-if (!_g.__openclaw_internal_hook_handlers__) {
-  _g.__openclaw_internal_hook_handlers__ = new Map<string, InternalHookHandler[]>();
-}
-const handlers = _g.__openclaw_internal_hook_handlers__;
+const handlers = (_g.__openclaw_internal_hook_handlers__ ??= new Map<
+  string,
+  InternalHookHandler[]
+>());
 const log = createSubsystemLogger("internal-hooks");
 
 /**
