fix(tables): preserve the persisted default owner

Author: j15z

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/views-menu/views-menu.test.tsx

@@ -1,3 +1,8 @@
+/**
+ * @vitest-environment jsdom
+ */
+import { act } from 'react'
+import { createRoot } from 'react-dom/client'
 import { renderToStaticMarkup } from 'react-dom/server'
 import { describe, expect, it, vi } from 'vitest'
 import type { TableViewWire } from '@/lib/api/contracts/tables'
@@ -14,6 +19,13 @@ const DEFAULT_VIEW: TableViewWire = {
   updatedAt: new Date('2026-08-15T01:00:00.000Z'),
 }
 
+const SAVED_VIEW: TableViewWire = {
+  ...DEFAULT_VIEW,
+  id: 'view-saved',
+  name: 'Saved',
+  isDefault: false,
+}
+
 function renderMenu(views: TableViewWire[], activeViewId: string | null): string {
   return renderToStaticMarkup(
     <ViewsMenu
@@ -42,4 +54,30 @@ describe('ViewsMenu', () => {
     expect(markup).toContain('All')
     expect(markup).not.toContain('>View<')
   })
+
+  it('only offers deletion for non-default views', () => {
+    const container = document.createElement('div')
+    document.body.appendChild(container)
+    const root = createRoot(container)
+
+    act(() => {
+      root.render(
+        <ViewsMenu
+          views={[DEFAULT_VIEW, SAVED_VIEW]}
+          activeViewId={DEFAULT_VIEW.id}
+          onSelect={vi.fn()}
+          onRename={vi.fn()}
+          onDelete={vi.fn()}
+          onNewView={vi.fn()}
+          canEdit
+        />
+      )
+    })
+    act(() => container.querySelector<HTMLButtonElement>('button[aria-label="Views"]')?.click())
+
+    expect(document.body.querySelectorAll('button[aria-label="Delete"]')).toHaveLength(1)
+
+    act(() => root.unmount())
+    container.remove()
+  })
 })

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/views-menu/views-menu.tsx

@@ -156,11 +156,15 @@ export const ViewsMenu = memo(function ViewsMenu({
                         label: 'Rename',
                         onClick: () => runAndClose(() => onRename(view.id)),
                       },
-                      {
-                        icon: Trash,
-                        label: 'Delete',
-                        onClick: () => runAndClose(() => onDelete(view.id)),
-                      },
+                      ...(!view.isDefault
+                        ? [
+                            {
+                              icon: Trash,
+                              label: 'Delete',
+                              onClick: () => runAndClose(() => onDelete(view.id)),
+                            },
+                          ]
+                        : []),
                     ]
                   : undefined
               }

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

@@ -256,8 +256,8 @@ export function Table({
   const [{ sort: sortColumn, dir: sortDirection, view: activeViewId }, setTableParams] =
     useQueryStates(tableDetailParsers, tableDetailUrlKeys)
 
-  // Read-only mirrors for the resolve effect: it must know whether the user has
-  // already applied a filter / hidden columns without re-running when they change.
+  // Read-only mirrors for the resolve effect and replaceFilter's echo check:
+  // both must read the current values without re-running when they change.
   const filterRef = useRef(filter)
   filterRef.current = filter
   const hiddenColumnsRef = useRef(hiddenColumns)
@@ -418,9 +418,13 @@ export function Table({
    * this an open panel keeps showing the rules of the filter it replaced.
    *
    * The remount discards an unapplied draft, which is the point — the rules on
-   * screen must be the rules in effect.
+   * screen must be the rules in effect. An incoming filter identical to the
+   * current one is skipped entirely: the resolve effect re-applies the config
+   * after this client's own autosave settles, and letting that echo remount an
+   * open panel would wipe keystrokes typed since the flush and steal focus.
    */
   const replaceFilter = useCallback((next: TablePredicate | null) => {
+    if (JSON.stringify(next) === JSON.stringify(filterRef.current)) return
     setFilter(next)
     setFilterSeed((seed) => seed + 1)
   }, [])
@@ -588,11 +592,7 @@ export function Table({
         // `sort` rides the same host URL, so when the view id is inherited the
         // sort beside it is too — not local work, and it must not suppress the
         // default view's own sort.
-        const keep = inheritedParams
-          ? { ...localWork(), sort: false }
-          : legacyAllWithDefault
-            ? undefined
-            : localWork()
+        const keep = inheritedParams ? { ...localWork(), sort: false } : localWork()
         if (defaultView) {
           appliedViewRevisionRef.current = getTableViewRevision(defaultView)
           setTableParams({ view: defaultView.id })
@@ -864,6 +864,10 @@ export function Table({
 
   const handleDeleteView = useCallback(
     (viewId: string) => {
+      if (views.some((view) => view.id === viewId && view.isDefault)) {
+        toast.error('Set another view as default before deleting this view')
+        return
+      }
       deleteViewMutation.mutate(viewId, {
         onSuccess: () => {
           if (viewId !== activeViewId) return