#7278 N-methylation option is missing even if monomer in selection is eligible for it

ketcher-autotests/tests/specs/File-Management/Open-And-Save-Files/InChi/InChi.spec.ts

@@ -5,7 +5,6 @@ import {
   takeEditorScreenshot,
   openFileAndAddToCanvas,
   pasteFromClipboardAndAddToCanvas,
-  openPasteFromClipboard,
   copyToClipboardByKeyboard,
   openFileAndAddToCanvasAsNewProject,
   readFileContent,

ketcher-autotests/tests/specs/Macromolecule-editor/Import-Saving-Files/import-saving-helm.spec.ts

@@ -7,7 +7,7 @@ import {
   pasteFromClipboardAndAddToMacromoleculesCanvas,
   MacroFileType,
 } from '@utils';
-import { closeOpenStructure, pageReload } from '@utils/common/helpers';
+import { pageReload } from '@utils/common/helpers';
 import { verifyHELMExport } from '@utils/files/receiveFileComparisonData';
 import { ErrorMessageDialog } from '@tests/pages/common/ErrorMessageDialog';
 import { OpenStructureDialog } from '@tests/pages/common/OpenStructureDialog';

ketcher-autotests/tests/specs/Macromolecule-editor/Import-Saving-Files/import-saving-idt.spec.ts

@@ -22,7 +22,7 @@ import {
   waitForPageInit,
 } from '@utils';
 import { selectAllStructuresOnCanvas } from '@utils/canvas/selectSelection';
-import { closeOpenStructure, pageReload } from '@utils/common/helpers';
+import { pageReload } from '@utils/common/helpers';
 import {
   FileType,
   verifyFileExport,

ketcher-autotests/tests/utils/common/helpers.ts

@@ -1,7 +1,6 @@
 import { Page } from '@playwright/test';
 import { waitForKetcherInit } from './loaders/waitForKetcherInit/waitForKetcherInit';
 import { waitForIndigoToLoad } from './loaders/waitForIndigoToLoad';
-import { OpenStructureDialog } from '@tests/pages/common/OpenStructureDialog';
 
 export async function emptyFunction() {
   // Intentionally empty callback used as a default async no-op in wait helpers.

packages/ketcher-macromolecules/src/components/contextMenu/SelectedMonomersContextMenu/helpers.ts

@@ -177,24 +177,26 @@ export const getModifyAminoAcidsMenuItems = (
     }
 
     modificationTypes.forEach((modificationType) => {
-      // If modification does not have R1 or R2 attachment points to persist connection
+      // Check if all monomers in this group already have this modification
       if (
-        monomersWithSameNaturalAnalogCode.some(
-          (monomer: BaseMonomer) =>
-            monomer.label !== monomerLibraryItem.label &&
-            !canModifyAminoAcid(monomer, monomerLibraryItem),
+        monomersWithSameNaturalAnalogCode.every(
+          (monomer) => monomer.label === monomerLibraryItem.label,
         )
       ) {
-        modificationTypesDisabledByAttachmentPoints.add(modificationType);
-
         return;
       }
 
-      if (
-        monomersWithSameNaturalAnalogCode.every(
-          (monomer) => monomer.label === monomerLibraryItem.label,
-        )
-      ) {
+      // Check if at least one monomer in this group can be modified
+      const hasAtLeastOneEligibleMonomer =
+        monomersWithSameNaturalAnalogCode.some(
+          (monomer: BaseMonomer) =>
+            monomer.label !== monomerLibraryItem.label &&
+            canModifyAminoAcid(monomer, monomerLibraryItem),
+        );
+
+      // If NO monomer is eligible, disable this modification
+      if (!hasAtLeastOneEligibleMonomer) {
+        modificationTypesDisabledByAttachmentPoints.add(modificationType);
         return;
       }