fix: BROS-777: Disallow adding more points for locked Vectors (#9320)

Co-authored-by: nick-skriabin <nick-skriabin@users.noreply.github.com>

Author: nick-skriabin

web/libs/editor/src/components/KonvaVector/KonvaVector.tsx

@@ -1609,9 +1609,19 @@ export const KonvaVector = forwardRef<KonvaVectorRef, KonvaVectorProps>((props,
       };
     },
     // Programmatic point creation methods
-    startPoint: (x: number, y: number) => pointCreationManager.startPoint(x, y),
-    updatePoint: (x: number, y: number) => pointCreationManager.updatePoint(x, y),
-    commitPoint: (x: number, y: number) => pointCreationManager.commitPoint(x, y),
+    // These check the disabled prop to prevent point creation when the region is locked
+    startPoint: (x: number, y: number) => {
+      if (disabled) return false;
+      return pointCreationManager.startPoint(x, y);
+    },
+    updatePoint: (x: number, y: number) => {
+      if (disabled) return;
+      pointCreationManager.updatePoint(x, y);
+    },
+    commitPoint: (x: number, y: number) => {
+      if (disabled) return;
+      pointCreationManager.commitPoint(x, y);
+    },
     // Programmatic point transformation methods
     translatePoints: (dx: number, dy: number, pointIds?: string[]) => {
       const pointsToTransform = pointIds ? initialPoints.filter((p) => pointIds.includes(p.id)) : initialPoints;

web/libs/editor/src/regions/VectorRegion.jsx

@@ -456,6 +456,11 @@ const Model = types
       },
 
       addPoint(x, y) {
+        // Don't allow adding points when region is locked or readonly
+        if (self.locked || self.isReadOnly()) {
+          return null;
+        }
+
         const image = self.parent.currentImageEntity;
         const width = image.naturalWidth;
         const height = image.naturalHeight;
@@ -483,6 +488,10 @@ const Model = types
       // Uses KonvaVector startPoint to start drawing
       // This will only initiate point drawing, but won't create actual point
       startPoint(x, y) {
+        // Don't allow adding points when region is locked or readonly
+        if (self.locked || self.isReadOnly()) {
+          return;
+        }
         self.vectorRef.startPoint(x, y);
       },
 
@@ -492,13 +501,21 @@ const Model = types
       //
       // This method is designed to create Bezier curve
       updatePoint(x, y) {
+        // Don't allow modifying points when region is locked or readonly
+        if (self.locked || self.isReadOnly()) {
+          return;
+        }
         self.vectorRef.updatePoint(x, y);
       },
 
       // Commits previously created point and resets the state
       //
       // Will create a new point if it was started but never updated (regular click)
       commitPoint(x, y) {
+        // Don't allow adding points when region is locked or readonly
+        if (self.locked || self.isReadOnly()) {
+          return;
+        }
         self.vectorRef?.commitPoint(x, y);
       },