useGetViewGroupsFilters throws when view has viewGroups but null mainGroupByFieldMetadataId

https://sonarly.com/issue/3794?type=bug

The `useGetViewGroupsFilters` hook unconditionally throws when a view has hidden viewGroups but `mainGroupByFieldMetadataId` is null, a legitimate data state after the fieldMetadataId migration.

Fix: The fix addresses three sites where `throw new Error('mainGroupByFieldMetadataId is required')` crashes when a view has orphaned viewGroups but no `mainGroupByFieldMetadataId` (null after the fieldMetadataId migration or during cache transitions).

**`useGetViewGroupsFilters.ts`** (primary crash site from Sentry): Moved the `isDefined(mainGroupByFieldMetadataId)` check before iterating viewGroups, returning an empty array early. This is the correct behavior — without a group-by field, hidden viewGroup visibility filters are meaningless.

**`useSaveCurrentViewGroups.ts`**: Changed throw to early return in `saveViewGroups` callback, matching the existing early-return pattern used 3 times earlier in the same file for other precondition checks (lines 28-30, 34-36, 40-42).

**`useReorderRecordGroups.ts`**: Changed throw to early return in `reorderRecordGroups` callback. If there's no group field metadata, the reorder operation simply cannot proceed — silently returning is the correct behavior (same as the `isDeeplyEqual` check on line 63-67 which also returns without action).

packages/twenty-front/src/modules/object-record/record-group/hooks/useReorderRecordGroups.ts

@@ -87,7 +87,7 @@ export const useReorderRecordGroups = ({
       }, []);
 
       if (!isDefined(recordIndexGroupFieldMetadataItem?.id)) {
-        throw new Error('mainGroupByFieldMetadataId is required');
+        return;
       }
 
       setRecordGroups({

packages/twenty-front/src/modules/views/hooks/useGetViewGroupsFilters.ts

@@ -6,24 +6,22 @@ import { getFilterTypeFromFieldType, isDefined } from 'twenty-shared/utils';
 export const useGetViewGroupsFilters = (): RecordFilter[] => {
   const { currentView } = useGetCurrentViewOnly();
 
-  return (
-    currentView?.viewGroups
-      .filter((recordGroup) => !recordGroup.isVisible)
-      .map((recordGroup) => {
-        if (!isDefined(currentView.mainGroupByFieldMetadataId)) {
-          throw new Error('mainGroupByFieldMetadataId is required');
-        }
+  if (!isDefined(currentView?.mainGroupByFieldMetadataId)) {
+    return [];
+  }
 
-        return {
-          id: recordGroup.id,
-          fieldMetadataId: currentView.mainGroupByFieldMetadataId,
-          value: JSON.stringify([recordGroup.fieldValue]),
-          operand: ViewFilterOperand.IS_NOT,
-          displayValue: '',
-          type: getFilterTypeFromFieldType(FieldMetadataType.SELECT),
-          label: '',
-        };
-      })
+  return (
+    currentView.viewGroups
+      .filter((recordGroup) => !recordGroup.isVisible)
+      .map((recordGroup) => ({
+        id: recordGroup.id,
+        fieldMetadataId: currentView.mainGroupByFieldMetadataId,
+        value: JSON.stringify([recordGroup.fieldValue]),
+        operand: ViewFilterOperand.IS_NOT,
+        displayValue: '',
+        type: getFilterTypeFromFieldType(FieldMetadataType.SELECT),
+        label: '',
+      }))
       .filter(isDefined) || []
   );
 };

packages/twenty-front/src/modules/views/hooks/useSaveCurrentViewGroups.ts

@@ -149,7 +149,7 @@ export const useSaveCurrentViewGroups = () => {
         .filter(isDefined);
 
       if (!isDefined(view.mainGroupByFieldMetadataId)) {
-        throw new Error('mainGroupByFieldMetadataId is required');
+        return;
       }
 
       await performViewGroupAPIUpdate(viewGroupsToUpdate);