fix: address reviewer feedback — dedupe double refresh, use generic MutationEvent<T>

1. types/data.ts: onMutation now uses MutationEvent<T> to preserve record typing
2. ObjectView: skip onMutation subscription when renderListView is provided (avoids double refresh)
3. ListView: skip onMutation subscription when refreshTrigger is provided (avoids double refresh)

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/b095f623-c45f-4bc7-8ffd-a5cac05aac07

Author: Copilot

packages/plugin-list/src/ListView.tsx

@@ -416,15 +416,17 @@ export const ListView = React.forwardRef<ListViewHandle, ListViewProps>(({
   }), []);
 
   // --- P2: Auto-subscribe to DataSource mutation events ---
+  // When an external refreshTrigger is provided, rely on that instead of
+  // subscribing to dataSource mutations to avoid double refreshes.
   React.useEffect(() => {
-    if (!dataSource?.onMutation || !schema.objectName) return;
+    if (!dataSource?.onMutation || !schema.objectName || schema.refreshTrigger) return;
     const unsub = dataSource.onMutation((event) => {
       if (event.resource === schema.objectName) {
         setRefreshKey(k => k + 1);
       }
     });
     return unsub;
-  }, [dataSource, schema.objectName]);
+  }, [dataSource, schema.objectName, schema.refreshTrigger]);
 
   // Dynamic page size state (wired from pageSizeOptions selector)
   const [dynamicPageSize, setDynamicPageSize] = React.useState<number | undefined>(undefined);

packages/plugin-view/src/ObjectView.tsx

@@ -228,15 +228,20 @@ export const ObjectView: React.FC<ObjectViewProps> = ({
   // When a DataSource implements onMutation(), ObjectView auto-refreshes
   // its own data fetch (for non-grid view types like kanban, calendar, etc.)
   // whenever a create/update/delete occurs on the same objectName.
+  //
+  // ListView-driven configurations already manage refreshKey via
+  // form success / delete handlers. To avoid double refreshes and
+  // duplicate find() calls, skip auto-subscription when renderListView is provided.
   useEffect(() => {
     if (!dataSource?.onMutation || !schema.objectName) return;
+    if (renderListView) return;
     const unsub = dataSource.onMutation((event: any) => {
       if (event.resource === schema.objectName) {
         setRefreshKey(prev => prev + 1);
       }
     });
     return unsub;
-  }, [dataSource, schema.objectName]);
+  }, [dataSource, schema.objectName, renderListView]);
 
   // Data fetching state for non-grid views
   const [data, setData] = useState<any[]>([]);

packages/types/src/data.ts

@@ -342,7 +342,7 @@ export interface DataSource<T = any> {
    * unsub?.();
    * ```
    */
-  onMutation?(callback: (event: MutationEvent) => void): () => void;
+  onMutation?(callback: (event: MutationEvent<T>) => void): () => void;
 }
 
 /**