chore(utils): remove unused utilities (asserts, safeJsonParse, isNonNull)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

.claude/rules/global.md

@@ -36,13 +36,6 @@ Use shared helpers from `@/lib/core/utils/helpers` instead of writing inline imp
 - `sleep(ms)` — async delay. Never write `new Promise(resolve => setTimeout(resolve, ms))`
 - `toError(value)` — normalize unknown caught values to `Error`. Never write `e instanceof Error ? e : new Error(String(e))`
 - `toError(value).message` — get error message safely. Never write `e instanceof Error ? e.message : String(e)`
-- `safeJsonParse(str, fallback?)` — parse JSON without throwing. Never write `try { JSON.parse(str) } catch { return default }`
-- `isNonNull(value)` — type-narrowing filter predicate for null/undefined
-
-Use assertion utilities from `@/lib/core/utils/asserts`:
-
-- `invariant(condition, message)` — assert a condition is truthy, throws if not
-- `assertNever(value)` — exhaustive switch/if-else check, TypeScript errors at compile time if a case is unhandled
 
 ```typescript
 // ✗ Bad

.cursor/rules/global.mdc

@@ -43,13 +43,6 @@ Use shared helpers from `@/lib/core/utils/helpers` instead of writing inline imp
 - `sleep(ms)` — async delay. Never write `new Promise(resolve => setTimeout(resolve, ms))`
 - `toError(value)` — normalize unknown caught values to `Error`. Never write `e instanceof Error ? e : new Error(String(e))`
 - `toError(value).message` — get error message safely. Never write `e instanceof Error ? e.message : String(e)`
-- `safeJsonParse(str, fallback?)` — parse JSON without throwing. Never write `try { JSON.parse(str) } catch { return default }`
-- `isNonNull(value)` — type-narrowing filter predicate for null/undefined
-
-Use assertion utilities from `@/lib/core/utils/asserts`:
-
-- `invariant(condition, message)` — assert a condition is truthy, throws if not
-- `assertNever(value)` — exhaustive switch/if-else check, TypeScript errors at compile time if a case is unhandled
 
 ```typescript
 // ✗ Bad

CLAUDE.md

@@ -8,7 +8,7 @@ You are a professional software engineer. All code must follow best practices: a
 - **Comments**: Use TSDoc for documentation. No `====` separators. No non-TSDoc comments
 - **Styling**: Never update global styles. Keep all styling local to components
 - **ID Generation**: Never use `crypto.randomUUID()`, `nanoid`, or `uuid` package. Use `generateId()` (UUID v4) or `generateShortId()` (compact) from `@/lib/core/utils/uuid`
-- **Common Utilities**: Use shared helpers from `@/lib/core/utils/helpers` instead of inline implementations. `sleep(ms)` for delays, `toError(e)` to normalize caught values, `safeJsonParse(str, fallback?)` for safe JSON parsing, `isNonNull(v)` for type-narrowing null filters. Use `invariant(cond, msg)` and `assertNever(val)` from `@/lib/core/utils/asserts` for runtime assertions and exhaustive checks.
+- **Common Utilities**: Use shared helpers from `@/lib/core/utils/helpers` instead of inline implementations. `sleep(ms)` for delays, `toError(e)` to normalize caught values.
 - **Package Manager**: Use `bun` and `bunx`, not `npm` and `npx`
 
 ## Architecture

apps/sim/lib/core/utils/asserts.ts

@@ -6,28 +6,6 @@ export function sleep(ms: number): Promise<void> {
   return new Promise((resolve) => setTimeout(resolve, ms))
 }
 
-/**
- * Parses a JSON string, returning a fallback value on failure instead of throwing.
- * Replaces the common `try { JSON.parse(str) } catch { return default }` pattern.
- */
-export function safeJsonParse<T>(value: string): T | undefined
-export function safeJsonParse<T>(value: string, fallback: T): T
-export function safeJsonParse<T>(value: string, fallback?: T): T | undefined {
-  try {
-    return JSON.parse(value) as T
-  } catch {
-    return fallback
-  }
-}
-
-/**
- * Type-safe filter predicate that removes null and undefined values.
- * Fixes the common `.filter(Boolean)` pattern which doesn't narrow types in TypeScript.
- */
-export function isNonNull<T>(value: T | null | undefined): value is T {
-  return value != null
-}
-
 /**
  * Normalizes an unknown caught value into an Error instance.
  * Replaces the common `e instanceof Error ? e : new Error(String(e))` pattern in catch clauses.