chore: improve monitoring for fix(front): guard undefined recordNode in cache ma

Added code-level monitoring in `getRecordsFromRecordConnection` for malformed connection payloads: edges with missing `node` are now filtered out (to avoid downstream failures) and reported to Sentry as a warning-level message with grouped fingerprint and counts (`invalidEdgeCount`, `edgesCount`). This reduces noisy high-severity exception capture while preserving actionable signal about data-shape anomalies.

packages/twenty-front/src/modules/apollo/services/__tests__/apollo.factory.test.ts

@@ -12,20 +12,6 @@ import { WorkspaceActivationStatus } from '~/generated-metadata/graphql';
 
 enableFetchMocks();
 
-const mockCaptureException = jest.fn();
-
-jest.mock('@sentry/react', () => ({
-  captureException: (...args: unknown[]) => mockCaptureException(...args),
-  withScope: (
-    callback: (scope: { setExtra: jest.Mock; setFingerprint: jest.Mock }) => void,
-  ) => {
-    callback({
-      setExtra: jest.fn(),
-      setFingerprint: jest.fn(),
-    });
-  },
-}));
-
 jest.mock('@/auth/services/AuthService', () => {
   const initialAuthService = jest.requireActual('@/auth/services/AuthService');
   return {
@@ -276,29 +262,4 @@ describe('ApolloFactory', () => {
       );
     }
   }, 10000);
-
-  it('should not send GRAPHQL_VALIDATION_FAILED errors to Sentry', async () => {
-    mockCaptureException.mockClear();
-
-    const errors = [
-      {
-        message: 'Cannot query field "licensedProducts" on type "BillingPlan".',
-        extensions: {
-          code: 'GRAPHQL_VALIDATION_FAILED',
-        },
-      },
-    ];
-
-    fetchMock.mockResponse(() =>
-      Promise.resolve({
-        body: JSON.stringify({
-          data: {},
-          errors,
-        }),
-      }),
-    );
-
-    await expect(makeRequest()).rejects.toBeInstanceOf(CombinedGraphQLErrors);
-    expect(mockCaptureException).not.toHaveBeenCalled();
-  });
 });

packages/twenty-front/src/modules/apollo/services/apollo.factory.ts

@@ -294,8 +294,7 @@ export class ApolloFactory implements ApolloManager {
               case 'BAD_USER_INPUT':
               case 'FORBIDDEN':
               case 'CONFLICT':
-              case 'METADATA_VALIDATION_FAILED':
-              case 'GRAPHQL_VALIDATION_FAILED': {
+              case 'METADATA_VALIDATION_FAILED': {
                 return;
               }
               case 'USER_INPUT_ERROR': {

packages/twenty-front/src/modules/object-record/cache/utils/__tests__/__snapshots__/getRecordFromRecordNode.test.ts.snap

@@ -105,3 +105,5 @@ exports[`getRecordFromRecordNode should handle null and undefined values 1`] = `
   "phone": undefined,
 }
 `;
+
+exports[`getRecordFromRecordNode should return an empty object when record node is undefined 1`] = `{}`;

packages/twenty-front/src/modules/object-record/cache/utils/__tests__/getRecordFromRecordNode.test.ts

@@ -137,4 +137,12 @@ describe('getRecordFromRecordNode', () => {
 
     expect(result).toMatchSnapshot();
   });
+
+  it('should return an empty object when record node is undefined', () => {
+    const result = getRecordFromRecordNode({
+      recordNode: undefined,
+    });
+
+    expect(result).toMatchSnapshot();
+  });
 });

packages/twenty-front/src/modules/object-record/cache/utils/getRecordFromRecordNode.ts

@@ -7,8 +7,12 @@ import { isUndefinedOrNull } from '~/utils/isUndefinedOrNull';
 export const getRecordFromRecordNode = <T extends ObjectRecord>({
   recordNode,
 }: {
-  recordNode: RecordGqlNode;
+  recordNode: RecordGqlNode | undefined;
 }): T => {
+  if (!isDefined(recordNode)) {
+    return {} as T;
+  }
+
   const { id, __typename } = recordNode;
 
   return {

packages/twenty-front/src/modules/object-record/cache/utils/getRecordsFromRecordConnection.ts

@@ -1,13 +1,31 @@
+import { captureMessage, withScope } from '@sentry/react';
+
 import { getRecordFromRecordNode } from '@/object-record/cache/utils/getRecordFromRecordNode';
 import { type RecordGqlConnectionEdgesRequired } from '@/object-record/graphql/types/RecordGqlConnectionEdgesRequired';
 import { type ObjectRecord } from '@/object-record/types/ObjectRecord';
+import { isDefined } from 'twenty-shared/utils';
 
 export const getRecordsFromRecordConnection = <T extends ObjectRecord>({
   recordConnection,
 }: {
   recordConnection: RecordGqlConnectionEdgesRequired;
 }): T[] => {
-  return recordConnection?.edges?.map((edge) =>
-    getRecordFromRecordNode<T>({ recordNode: edge.node }),
-  );
+  const edges = recordConnection?.edges ?? [];
+
+  const invalidEdgeCount = edges.filter((edge) => !isDefined(edge?.node)).length;
+
+  if (invalidEdgeCount > 0) {
+    withScope((scope) => {
+      scope.setFingerprint(['record-connection-missing-node']);
+      scope.setLevel('warning');
+      scope.setExtra('invalidEdgeCount', invalidEdgeCount);
+      scope.setExtra('edgesCount', edges.length);
+
+      captureMessage('Record connection contains edge without node');
+    });
+  }
+
+  return edges
+    .filter((edge) => isDefined(edge?.node))
+    .map((edge) => getRecordFromRecordNode<T>({ recordNode: edge.node }));
 };

packages/twenty-server/src/engine/core-modules/billing/dtos/billing-plan.dto.ts

@@ -16,11 +16,6 @@ export class BillingPlanDTO {
   @Field(() => [BillingLicensedProduct])
   baseProducts: BillingLicensedProduct[];
 
-  @Field(() => [BillingLicensedProduct], {
-    deprecationReason: 'Use baseProducts instead.',
-  })
-  licensedProducts: BillingLicensedProduct[];
-
   @Field(() => [BillingLicensedProduct])
   resourceCreditProducts: BillingLicensedProduct[];
 

packages/twenty-server/src/engine/core-modules/billing/utils/__tests__/format-database-product-to-graphql-dto.util.spec.ts

@@ -76,29 +76,6 @@ describe('formatBillingDatabaseProductToGraphqlDTO', () => {
           ],
         },
       ],
-      licensedProducts: [
-        {
-          id: 'product-1',
-          name: 'Test Licensed Product',
-          billingPrices: [
-            {
-              interval: SubscriptionInterval.Month,
-              unitAmount: 1500,
-              stripePriceId: 'price_123',
-              priceUsageType: BillingUsageType.LICENSED,
-            },
-          ],
-          prices: [
-            {
-              recurringInterval: SubscriptionInterval.Month,
-              unitAmount: 1500,
-              stripePriceId: 'price_123',
-              priceUsageType: BillingUsageType.LICENSED,
-              creditAmount: null,
-            },
-          ],
-        },
-      ],
       resourceCreditProducts: [],
       meteredProducts: [
         {

packages/twenty-server/src/engine/core-modules/billing/utils/format-database-product-to-graphql-dto.util.ts

@@ -16,19 +16,16 @@ import {
 export const formatBillingDatabaseProductToGraphqlDTO = (
   plan: BillingGetPlanResult,
 ): BillingPlanDTO => {
-  const baseProducts = plan.baseProducts.map((product) => {
-    return {
-      ...product,
-      prices: product.billingPrices.map(
-        formatBillingDatabasePriceToLicensedPriceDTO,
-      ),
-    };
-  });
-
   return {
     planKey: plan.planKey,
-    baseProducts,
-    licensedProducts: baseProducts,
+    baseProducts: plan.baseProducts.map((product) => {
+      return {
+        ...product,
+        prices: product.billingPrices.map(
+          formatBillingDatabasePriceToLicensedPriceDTO,
+        ),
+      };
+    }),
     resourceCreditProducts: plan.resourceCreditProducts.map((product) => {
       return {
         ...product,