restrict idField

Author: paula-stacho

packages/compass-data-modeling/src/components/drawer/diagram-editor-side-panel.tsx

@@ -20,6 +20,7 @@ import { getDefaultRelationshipName } from '../../utils';
 import FieldDrawerContent from './field-drawer-content';
 import type { FieldPath } from '../../services/data-model-storage';
 import { getFieldFromSchema } from '../../utils/schema-traversal';
+import { isIdField } from '../../utils/utils';
 
 export const DATA_MODELING_DRAWER_ID = 'data-modeling-drawer';
 
@@ -110,11 +111,15 @@ function DiagramEditorSidePanel({
           ></FieldDrawerContent>
         ),
         actions: [
-          {
-            action: 'delete',
-            label: 'Delete Field',
-            icon: 'Trash' as const,
-          },
+          ...(!isIdField(selectedItems.fieldPath)
+            ? [
+                {
+                  action: 'delete',
+                  label: 'Delete Field',
+                  icon: 'Trash' as const,
+                },
+              ]
+            : []),
         ],
         handleAction: (actionName: string) => {
           if (actionName === 'delete') {

packages/compass-data-modeling/src/components/drawer/field-drawer-content.tsx

@@ -26,7 +26,11 @@ import {
 import { useChangeOnBlur } from './use-change-on-blur';
 import { RelationshipsSection } from './relationships-section';
 import { getFieldFromSchema } from '../../utils/schema-traversal';
-import { areFieldPathsEqual, isRelationshipOfAField } from '../../utils/utils';
+import {
+  areFieldPathsEqual,
+  isIdField,
+  isRelationshipOfAField,
+} from '../../utils/utils';
 
 type FieldDrawerContentProps = {
   namespace: string;
@@ -138,6 +142,7 @@ const FieldDrawerContent: React.FunctionComponent<FieldDrawerContentProps> = ({
         <DMFormFieldContainer>
           <TextInput
             label="Field name"
+            disabled={isIdField(fieldPath)}
             data-testid="data-model-collection-drawer-name-input"
             sizeVariant="small"
             value={fieldName}

packages/compass-data-modeling/src/utils/utils.ts

@@ -1,5 +1,8 @@
 import type { FieldPath, Relationship } from '../services/data-model-storage';
 
+export const isIdField = (fieldPath: FieldPath): boolean =>
+  fieldPath.length === 1 && fieldPath[0] === '_id';
+
 export function areFieldPathsEqual(
   fieldA: FieldPath,
   fieldB: FieldPath