fix(url-state): review fixes — resolve custom-block deep links before opening detail, per-surface time-range fallback

- custom-blocks: gate the detail view on the resolved block (matching mcp/
  access-control), so a dead or still-loading ?custom-block-id no longer flashes
  a bogus create screen
- parseAsTimeRange: unknown tokens now parse to null so each surface's
  .withDefault applies (logs keeps 'All time'; audit-logs keeps 'Past 30 days'
  instead of silently widening to all time on a malformed link)
- audit-logs: date-picker cancel target can never be 'Custom range' itself on a
  dateless custom deep link
- refresh shared ?search= consumer lists in TSDoc

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/logs/search-params.ts

@@ -41,12 +41,13 @@ const TOKEN_TO_TIME_RANGE: Record<string, TimeRange> = Object.fromEntries(
 ) as Record<string, TimeRange>
 
 /**
- * Parser for the `timeRange` param. Serializes labels to kebab tokens and
- * tolerantly maps unknown tokens back to the default ("All time").
+ * Parser for the `timeRange` param. Serializes labels to kebab tokens. Unknown
+ * tokens parse to `null` so each consuming surface's `.withDefault(...)` decides
+ * the fallback (logs: "All time"; audit-logs: "Past 30 days").
  */
 export const parseAsTimeRange = createParser<TimeRange>({
   parse(value) {
-    return TOKEN_TO_TIME_RANGE[value] ?? DEFAULT_TIME_RANGE
+    return TOKEN_TO_TIME_RANGE[value] ?? null
   },
   serialize(value) {
     return TIME_RANGE_TO_TOKEN[value] ?? 'all-time'

apps/sim/app/workspace/[workspaceId]/settings/components/search-params.ts

@@ -3,8 +3,9 @@ import { parseAsString } from 'nuqs/server'
 /**
  * Shared URL query-param definition for the settings list search boxes
  * (teammates, api-keys, copilot, custom-tools, mcp, secrets,
- * workflow-mcp-servers). Settings sections never co-render, so they all share
- * the `search` key without collisions.
+ * workflow-mcp-servers, and the ee sections: audit-logs, access-control,
+ * custom-blocks, data-drains, forks). Settings sections never co-render, so
+ * they all share the `search` key without collisions.
  *
  * Consume via `useSettingsSearch` (`settings/components/use-settings-search`),
  * which owns the debounced-write wiring — the input is controlled directly by

apps/sim/app/workspace/[workspaceId]/settings/components/use-settings-search.ts

@@ -9,7 +9,8 @@ import { useDebouncedSearchSetter } from '@/hooks/use-debounced-search-setter'
 
 /**
  * The shared `?search=` binding for settings list search boxes (teammates,
- * api-keys, copilot, custom-tools, mcp, secrets, workflow-mcp-servers).
+ * api-keys, copilot, custom-tools, mcp, secrets, workflow-mcp-servers, and the
+ * ee sections: audit-logs, access-control, custom-blocks, data-drains, forks).
  * Composes `useDebouncedSearchSetter`, so it carries the canonical semantics:
  * the value updates instantly (drives the controlled input and the in-memory
  * filter), non-empty URL writes are debounced, and clearing (or a

apps/sim/ee/audit-logs/components/audit-logs.tsx

@@ -238,7 +238,10 @@ export function AuditLogs({ organizationId }: AuditLogsProps) {
   const customStartDate = urlFilters.startDate ?? ''
   const customEndDate = urlFilters.endDate ?? ''
   const [datePickerOpen, setDatePickerOpen] = useState(false)
-  const previousTimeRangeRef = useRef<TimeRange>(timeRange)
+  /** Cancel target for the date picker — never 'Custom range' itself, so a deep link straight to an empty custom range still cancels back to a preset. */
+  const previousTimeRangeRef = useRef<TimeRange>(
+    timeRange === 'Custom range' ? 'Past 30 days' : timeRange
+  )
   const dateRangeAppliedRef = useRef(false)
   const [searchTerm, setSearchTerm] = useSettingsSearch()
   const debouncedSearch = useDebounce(searchTerm, SEARCH_DEBOUNCE_MS).trim()

apps/sim/ee/custom-blocks/components/custom-blocks.tsx

@@ -60,6 +60,9 @@ export function CustomBlocks() {
     return blocks.filter((b) => b.name.toLowerCase().includes(q))
   }, [blocks, searchTerm])
 
+  /** Open the detail only once the deep-linked id resolves to a loaded block. */
+  const selectedBlock = selectedBlockId ? blocks.find((b) => b.id === selectedBlockId) : undefined
+
   if (isLoading) return null
 
   if (!canManage) {
@@ -70,11 +73,11 @@ export function CustomBlocks() {
     )
   }
 
-  if ((isCreating || (selectedBlockId && canAdmin)) && workspaceId) {
+  if ((isCreating || (selectedBlock && canAdmin)) && workspaceId) {
     return (
       <CustomBlockDetail
-        key={isCreating ? 'new' : selectedBlockId}
-        blockId={isCreating ? null : selectedBlockId}
+        key={isCreating ? 'new' : selectedBlock?.id}
+        blockId={isCreating ? null : (selectedBlock?.id ?? null)}
         workspaceId={workspaceId}
         onBack={() => {
           setIsCreating(false)