simplify(resources): decide list emptiness in one place

"This list holds nothing" was derived in four pages, each with the same seven
clauses under the same nine-line comment. Adding the `error` gate one commit
ago took four identical edits, and the skills empty state that was reverted
off this branch would have made it five copies.

`isResourceListEmpty` now owns the rule and the reasoning behind each gate.
Logs omits the folder argument because it has no folder navigation; the other
three pass theirs.

`Resource.Table` also wraps the slot in its own growth box, so the empty state
centres because the table says so rather than because the node handed to it
happened to carry `flex-1`.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/components/resource/is-resource-list-empty.ts

@@ -0,0 +1,55 @@
+interface ResourceListEmptyInput {
+  rowCount: number
+  /** The list query's first load. No rows yet means "not arrived", not "none exist". */
+  isLoading: boolean
+  /** The query is serving the previous key's rows while the new key refetches. */
+  isPlaceholderData: boolean
+  /** A failed load also leaves the list empty. */
+  error: unknown
+  /**
+   * The search term the rows are actually filtered by — the debounced value, never the
+   * instant URL one.
+   */
+  search: string
+  filterCount: number
+  /** The open folder, or `null` at the root. Omit for lists without folder navigation. */
+  folderId?: string | null
+}
+
+/**
+ * Whether a resource list holds nothing, as opposed to merely showing nothing.
+ *
+ * A zero-data graphic invites someone to create their first item, so it may only
+ * appear when that is the true answer. Every other way `rows` empties out is a
+ * different message and gets gated here:
+ *
+ * - A search or filter that matched nothing, or an empty subfolder — the copy would
+ *   be wrong. The search must be the debounced value the rows are filtered by;
+ *   reading the instant URL value flashes the graphic for one debounce window after
+ *   a search is cleared.
+ * - The list still arriving. Server prefetches are allowed to seed nothing, and the
+ *   files list deliberately seeds nothing above 300 rows — so without this the
+ *   emptiest-looking screen is shown to the fullest workspaces.
+ * - The query serving the previous key's rows. Filters are part of the query key and
+ *   every list keeps previous data, so `isLoading` is false across a filter change.
+ * - A failed load, which is not an invitation to create anything.
+ */
+export function isResourceListEmpty({
+  rowCount,
+  isLoading,
+  isPlaceholderData,
+  error,
+  search,
+  filterCount,
+  folderId = null,
+}: ResourceListEmptyInput): boolean {
+  return (
+    rowCount === 0 &&
+    !isLoading &&
+    !isPlaceholderData &&
+    !error &&
+    folderId === null &&
+    !search.trim() &&
+    filterCount === 0
+  )
+}

apps/sim/app/workspace/[workspaceId]/components/resource/resource.tsx

@@ -230,6 +230,9 @@ interface ResourceTableProps {
    * empty, filling the otherwise blank scroll area. It never replaces the table
    * region or the headers — the chrome guarantee holds — so a consumer can show
    * a zero-data graphic without the list losing its structure.
+   *
+   * The table owns the growth box the slot is centred in, so any node centres —
+   * the slot does not have to carry its own `flex-1` to sit in the middle.
    */
   emptyState?: ReactNode
 }
@@ -453,7 +456,7 @@ const ResourceTable = memo(function ResourceTable({
                 ))}
           </div>
         </div>
-        {showEmptyState ? emptyState : null}
+        {showEmptyState ? <div className='flex min-h-0 flex-1 flex-col'>{emptyState}</div> : null}
         {hasMore && (
           <div ref={loadMoreRef} className='flex items-center justify-center py-3'>
             {isLoadingMore && (

apps/sim/app/workspace/[workspaceId]/files/files.tsx

@@ -86,6 +86,7 @@ import {
 } from '@/app/workspace/[workspaceId]/components/folders'
 import { ResourceActionBar } from '@/app/workspace/[workspaceId]/components/resource/components/action-bar'
 import { FilesEmptyState } from '@/app/workspace/[workspaceId]/components/resource/components/resource-empty-state'
+import { isResourceListEmpty } from '@/app/workspace/[workspaceId]/components/resource/is-resource-list-empty'
 import { DeleteConfirmModal } from '@/app/workspace/[workspaceId]/files/components/delete-confirm-modal'
 import { FileRowContextMenu } from '@/app/workspace/[workspaceId]/files/components/file-row-context-menu'
 import type { PreviewMode } from '@/app/workspace/[workspaceId]/files/components/file-viewer'
@@ -1810,26 +1811,15 @@ export function Files() {
     return tags
   }, [typeFilter, sizeFilter, uploadedByFilter, membersById])
 
-  /**
-   * Only for a workspace that genuinely holds nothing — a search or filter that
-   * matched nothing, or an empty subfolder, would make the copy wrong. Reads the
-   * debounced query because `rows` is filtered by it: gating on the instant URL
-   * value flashes the graphic for one debounce window after a search is cleared.
-   * The loading and placeholder gates cover the paints where `rows` is empty only
-   * because the list has not arrived — a cold cache when the server prefetch
-   * declined to seed, and the retained previous result during a workspace switch.
-   * `error` is gated too: a failed load also leaves `rows` empty, and inviting
-   * someone to create their first item is the wrong answer to a request that
-   * did not complete.
-   */
-  const showEmptyState =
-    rows.length === 0 &&
-    !isLoading &&
-    !isPlaceholderData &&
-    !error &&
-    currentFolderId === null &&
-    !debouncedSearchTerm.trim() &&
-    filterTags.length === 0
+  const showEmptyState = isResourceListEmpty({
+    rowCount: rows.length,
+    isLoading,
+    isPlaceholderData,
+    error,
+    search: debouncedSearchTerm,
+    filterCount: filterTags.length,
+    folderId: currentFolderId,
+  })
 
   if (fileIdFromRoute && !selectedFile && isLoading) {
     return (

apps/sim/app/workspace/[workspaceId]/knowledge/knowledge.tsx

@@ -55,6 +55,7 @@ import {
 } from '@/app/workspace/[workspaceId]/components/folders'
 import { ResourceActionBar } from '@/app/workspace/[workspaceId]/components/resource/components/action-bar'
 import { KnowledgeEmptyState } from '@/app/workspace/[workspaceId]/components/resource/components/resource-empty-state'
+import { isResourceListEmpty } from '@/app/workspace/[workspaceId]/components/resource/is-resource-list-empty'
 import { BaseTagsModal } from '@/app/workspace/[workspaceId]/knowledge/[id]/components'
 import {
   CreateBaseModal,
@@ -1339,26 +1340,15 @@ export function Knowledge() {
     return tags
   }, [connectorFilter, contentFilter, ownerFilter, members])
 
-  /**
-   * Only for a workspace that genuinely holds nothing — a search or filter that
-   * matched nothing, or an empty subfolder, would make the copy wrong. Reads the
-   * debounced query because `rows` is filtered by it: gating on the instant URL
-   * value flashes the graphic for one debounce window after a search is cleared.
-   * The loading and placeholder gates cover the paints where `rows` is empty only
-   * because the list has not arrived — a cold cache when the server prefetch
-   * declined to seed, and the retained previous result during a workspace switch.
-   * `error` is gated too: a failed load also leaves `rows` empty, and inviting
-   * someone to create their first item is the wrong answer to a request that
-   * did not complete.
-   */
-  const showEmptyState =
-    rows.length === 0 &&
-    !isLoading &&
-    !isPlaceholderData &&
-    !error &&
-    currentFolderId === null &&
-    !debouncedSearchQuery.trim() &&
-    filterTags.length === 0
+  const showEmptyState = isResourceListEmpty({
+    rowCount: rows.length,
+    isLoading,
+    isPlaceholderData,
+    error,
+    search: debouncedSearchQuery,
+    filterCount: filterTags.length,
+    folderId: currentFolderId,
+  })
 
   return (
     <>

apps/sim/app/workspace/[workspaceId]/logs/logs.tsx

@@ -59,6 +59,7 @@ import type {
 } from '@/app/workspace/[workspaceId]/components'
 import { Resource, type ResourceTableHandle } from '@/app/workspace/[workspaceId]/components'
 import { LogsEmptyState } from '@/app/workspace/[workspaceId]/components/resource/components/resource-empty-state'
+import { isResourceListEmpty } from '@/app/workspace/[workspaceId]/components/resource/is-resource-list-empty'
 import { useLogFilters } from '@/app/workspace/[workspaceId]/logs/hooks/use-log-filters'
 import { useSearchState } from '@/app/workspace/[workspaceId]/logs/hooks/use-search-state'
 import {
@@ -901,24 +902,15 @@ export default function Logs() {
     setTimeRange,
   ])
 
-  /**
-   * The zero-data graphic means "this workspace has never run anything" — a
-   * search or filter that matched nothing is a different message, so neither gets
-   * the graphic. Held back while the first page is in flight so it cannot flash
-   * before the runs arrive, and while the query is serving the previous key's
-   * result: the filters are part of the query key, so clearing a search that
-   * matched nothing keeps `isLoading` false while `rows` is still the stale empty
-   * set, and only the placeholder gate suppresses the graphic across that refetch.
-   * `error` is gated too: a failed load also leaves `rows` empty, and telling
-   * someone they have never run anything is the wrong answer to a failed request.
-   */
-  const showEmptyState =
-    rows.length === 0 &&
-    !logsQuery.isLoading &&
-    !logsQuery.isPlaceholderData &&
-    !logsQuery.error &&
-    !debouncedSearchQuery &&
-    filterTags.length === 0
+  /** Logs has no folder navigation, so the graphic means "nothing has ever run here". */
+  const showEmptyState = isResourceListEmpty({
+    rowCount: rows.length,
+    isLoading: logsQuery.isLoading,
+    isPlaceholderData: logsQuery.isPlaceholderData,
+    error: logsQuery.error,
+    search: debouncedSearchQuery,
+    filterCount: filterTags.length,
+  })
 
   const workflowsData = useMemo<WorkflowData[]>(
     () =>

apps/sim/app/workspace/[workspaceId]/tables/tables.tsx

@@ -54,6 +54,7 @@ import {
 } from '@/app/workspace/[workspaceId]/components/folders'
 import { ResourceActionBar } from '@/app/workspace/[workspaceId]/components/resource/components/action-bar'
 import { TablesEmptyState } from '@/app/workspace/[workspaceId]/components/resource/components/resource-empty-state'
+import { isResourceListEmpty } from '@/app/workspace/[workspaceId]/components/resource/is-resource-list-empty'
 import { useRegisterGlobalCommands } from '@/app/workspace/[workspaceId]/providers/global-commands-provider'
 import { useUserPermissionsContext } from '@/app/workspace/[workspaceId]/providers/workspace-permissions-provider'
 import {
@@ -737,26 +738,15 @@ export function Tables() {
     return tags
   }, [rowCountFilter, ownerFilter, membersById, setRowCountFilter, setOwnerFilter])
 
-  /**
-   * Only for a workspace that genuinely holds nothing — a search or filter that
-   * matched nothing, or an empty subfolder, would make the copy wrong. Reads the
-   * debounced query because `rows` is filtered by it: gating on the instant URL
-   * value flashes the graphic for one debounce window after a search is cleared.
-   * The loading and placeholder gates cover the paints where `rows` is empty only
-   * because the list has not arrived — a cold cache when the server prefetch
-   * declined to seed, and the retained previous result during a workspace switch.
-   * `error` is gated too: a failed load also leaves `rows` empty, and inviting
-   * someone to create their first item is the wrong answer to a request that
-   * did not complete.
-   */
-  const showEmptyState =
-    rows.length === 0 &&
-    !isLoading &&
-    !isPlaceholderData &&
-    !error &&
-    currentFolderId === null &&
-    !debouncedSearchTerm.trim() &&
-    filterTags.length === 0
+  const showEmptyState = isResourceListEmpty({
+    rowCount: rows.length,
+    isLoading,
+    isPlaceholderData,
+    error,
+    search: debouncedSearchTerm,
+    filterCount: filterTags.length,
+    folderId: currentFolderId,
+  })
 
   const handleContentContextMenu = useCallback(
     (e: React.MouseEvent) => {