chore(audit-logs): scope the refresh refetch, drop a test that could not fail

`refetch` ignores `enabled`, so refreshing the unscoped feed fired a workspace
lookup it has no use for and could fail a refresh that otherwise succeeded. It
now runs only when a scope asked for it.

The hook test claiming to cover an unresolved workspace scope passed `enabled:
false` with no workspace at all, so it asserted TanStack's disabled handling and
would have passed with the scope protection removed. The rule it named is derived
in the component and is covered there by `presentableAuditEntries`; a test that
cannot fail for its stated reason is worse than none.

Author: waleedlatif1

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

@@ -407,11 +407,14 @@ export function AuditLogs({ organizationId }: AuditLogsProps) {
     }, REFRESH_SPINNER_DURATION_MS)
     refreshTimers.add(timerId)
     /*
-      Both, because a failed workspace lookup closes the feed — refreshing only the
-      feed would leave the one control on screen unable to clear the state it is
-      showing.
+      The workspace lookup too, but only when a scope asked for it: a failed lookup
+      closes the feed, so refreshing only the feed would leave the one control on
+      screen unable to clear the state it is showing. `refetch` ignores `enabled`, so
+      an unconditional call would fire a request the unscoped feed has no use for and
+      could fail a refresh that otherwise succeeded.
     */
-    Promise.all([refetch(), orgWorkspaces.refetch()]).catch((error: unknown) => {
+    const pending = workspaceScope ? [refetch(), orgWorkspaces.refetch()] : [refetch()]
+    Promise.all(pending).catch((error: unknown) => {
       logger.error('Failed to refresh audit logs', { error })
     })
   }

apps/sim/ee/audit-logs/hooks/audit-logs.test.tsx

@@ -54,14 +54,12 @@ function AuditProbe({
   organizationId,
   workspaceId,
   search,
-  enabled = true,
 }: {
   organizationId: string
   workspaceId?: string
   search?: string
-  enabled?: boolean
 }) {
-  const auditLogs = useAuditLogs(organizationId, { workspaceId, search }, enabled)
+  const auditLogs = useAuditLogs(organizationId, { workspaceId, search })
   const entries = auditLogs.data?.pages.flatMap((page) => page.data) ?? []
 
   return (
@@ -75,7 +73,6 @@ function AuditProbe({
 interface RenderOptions {
   workspaceId?: string
   search?: string
-  enabled?: boolean
 }
 
 function renderAuditLogs(organizationId: string, options: RenderOptions = {}) {
@@ -86,7 +83,6 @@ function renderAuditLogs(organizationId: string, options: RenderOptions = {}) {
           organizationId={organizationId}
           workspaceId={options.workspaceId}
           search={options.search}
-          enabled={options.enabled ?? true}
         />
       </QueryClientProvider>
     )
@@ -197,19 +193,4 @@ describe('useAuditLogs identity transitions', () => {
     expect(container).not.toHaveTextContent('Updated Organization A')
     expect(container.querySelector('button')).toBeNull()
   })
-
-  /**
-   * The scope a link asks for is a ceiling, not a hint. A workspace id that no longer
-   * resolves must leave the feed closed rather than answering with the whole
-   * organization's history under a URL that still claims to be scoped.
-   */
-  it('never queries unscoped while a workspace scope is unresolved', async () => {
-    mockRequestJson.mockResolvedValue(AUDIT_PAGE_A)
-
-    renderAuditLogs('org-a', { enabled: false })
-    await flushQueries()
-
-    expect(mockRequestJson).not.toHaveBeenCalled()
-    expect(container).not.toHaveTextContent('Updated Organization A')
-  })
 })