[Apps] Small improvements (#17129)

- Fix undefined in breadcrumbs
<img width="783" height="220" alt="breadcrumbs_bad"
src="https://github.com/user-attachments/assets/557648da-f825-4bf6-b66f-2ac3992eeab1"
/>
- Make role required in app definition

packages/twenty-front/src/pages/settings/serverless-functions/SettingsServerlessFunctionDetail.tsx

@@ -37,7 +37,7 @@ export const SettingsServerlessFunctionDetail = () => {
   const navigate = useNavigate();
   const currentWorkspace = useRecoilValue(currentWorkspaceState);
 
-  const { data } = useFindOneApplicationQuery({
+  const { data, loading: applicationLoading } = useFindOneApplicationQuery({
     variables: { id: applicationId },
     skip: !applicationId,
   });
@@ -187,7 +187,8 @@ export const SettingsServerlessFunctionDetail = () => {
         ];
 
   return (
-    !loading && (
+    !loading &&
+    !applicationLoading && (
       <SubMenuTopBarContainer
         title={
           <SettingsServerlessFunctionLabelContainer

packages/twenty-sdk/src/application/__tests__/define-app.spec.ts

@@ -7,6 +7,7 @@ describe('defineApp', () => {
       displayName: 'My App',
       description: 'My app description',
       icon: 'IconWorld',
+      functionRoleUniversalIdentifier: '68bb56f3-8300-4cb5-8cc3-8da9ee66f1b2',
     };
 
     const result = defineApp(config);

packages/twenty-sdk/src/application/objects/define-object.ts

@@ -1,5 +1,5 @@
-import { type ObjectManifest } from 'twenty-shared/application';
 import { isNonEmptyString } from '@sniptt/guards';
+import { type ObjectManifest } from 'twenty-shared/application';
 import { validateFieldsOrThrow } from './validate-fields';
 
 /**

packages/twenty-sdk/src/cli/utils/__tests__/validate-manifest.spec.ts

@@ -9,6 +9,7 @@ describe('validateManifest - objectExtensions', () => {
   const validApplication: Application = {
     universalIdentifier: '4ec0391d-18d5-411c-b2f3-266ddc1c3ef7',
     displayName: 'Test App',
+    functionRoleUniversalIdentifier: '68bb56f3-8300-4cb5-8cc3-8da9ee66f1b2',
   };
 
   const validObjectExtension: ObjectExtensionManifest = {

packages/twenty-shared/src/application/applicationType.ts

@@ -2,9 +2,9 @@ import { type ApplicationVariables } from '@/application';
 import { type SyncableEntityOptions } from '@/application/syncableEntityOptionsType';
 
 export type Application = SyncableEntityOptions & {
+  functionRoleUniversalIdentifier: string;
   displayName?: string;
   description?: string;
   icon?: string;
   applicationVariables?: ApplicationVariables;
-  functionRoleUniversalIdentifier?: string;
 };

packages/twenty-shared/src/application/relationFieldManifestType.ts

@@ -1,17 +1,13 @@
-import { type SyncableEntityOptions } from '@/application/syncableEntityOptionsType';
+import { type FieldManifest } from '@/application/fieldManifestType';
 import { type FieldMetadataType } from '@/types';
 import { type RelationOnDeleteAction } from '@/types/RelationOnDeleteAction.type';
 import { type RelationType } from '@/types/RelationType';
 
-export type RelationFieldManifest = SyncableEntityOptions & {
-  name: string;
-  label: string;
-  description?: string;
-  icon?: string;
-  relationType: RelationType;
-  targetObjectUniversalIdentifier: string;
-  targetFieldLabel: string;
-  targetFieldIcon?: string;
-  onDelete?: RelationOnDeleteAction;
-  type: FieldMetadataType.RELATION;
-};
+export type RelationFieldManifest =
+  FieldManifest<FieldMetadataType.RELATION> & {
+    relationType: RelationType;
+    targetObjectUniversalIdentifier: string;
+    targetFieldLabel: string;
+    targetFieldIcon?: string;
+    onDelete?: RelationOnDeleteAction;
+  };