Refactor utility classes and fix configuration inconsistencies

Refactored utility classes to improve structure and clarity by adding proper encapsulation and final declarations. Fixed group ID inconsistencies in plugin XML to align with intended usage. Re-enabled commented-out steps in `uitests.yml` for capturing and uploading artifacts on test failures.

Author: VitaliyBoyko

.github/workflows/uitests.yml

@@ -70,16 +70,15 @@ jobs:
 
 # Uncomment if investigation is needed:
 
-#      - name: Capture Test Artifacts on Failure
-#        if: failure() && matrix.os == 'windows-latest'
-#        run: tar -cvzf video.tgz ./video
-#        shell: bash
-#
-#      - name: Upload Test Video Artifact
-#        if: failure() && matrix.os == 'windows-latest'
-#        uses: actions/upload-artifact@v4
-#        with:
-#          name: latest-test-video
-#          path: video.tgz
-#          overwrite: true
-
+      - name: Capture Test Artifacts on Failure
+        if: failure() && matrix.os == 'windows-latest'
+        run: tar -cvzf video.tgz ./video
+        shell: bash
+
+      - name: Upload Test Video Artifact
+        if: failure() && matrix.os == 'windows-latest'
+        uses: actions/upload-artifact@v4
+        with:
+          name: latest-test-video
+          path: video.tgz
+          overwrite: true

src/main/java/com/magento/idea/magento2plugin/util/magento/IsFileInEditableModuleUtil.java

@@ -37,20 +37,19 @@ public static boolean execute(final PsiFile file) {
      * @return true if the file is in an editable module directory, false otherwise
      */
     public static boolean execute(final Project project, final VirtualFile virtualFile) {
-        final Settings settings = Settings.getInstance(project);
-        List<String> editablePaths = settings.getMagentoFolders();
         final String magentoRootPath = MagentoPathUrlUtil.execute(project);
-        final String magentoDesignPath = MagentoPathUrlUtil.getDesignPath(project);
-
         if (magentoRootPath == null) {
             return false;
         }
 
+        final Settings settings = Settings.getInstance(project);
+        List<String> editablePaths = settings.getMagentoFolders();
         if (editablePaths == null) {
             editablePaths = new ArrayList<>();
         }
 
         editablePaths.add(magentoRootPath);
+        final String magentoDesignPath = MagentoPathUrlUtil.getDesignPath(project);
         if (magentoDesignPath != null) {
             editablePaths.add(magentoDesignPath);
         }

src/main/java/com/magento/idea/magento2plugin/util/magento/MagentoPathUrlUtil.java

@@ -11,15 +11,22 @@
 import com.magento.idea.magento2plugin.magento.packages.Package;
 import com.magento.idea.magento2plugin.project.Settings;
 
-public class MagentoPathUrlUtil {
+public final class MagentoPathUrlUtil {
+
+    /**
+     * Private constructor to prevent instantiation of utility class.
+     */
+    private MagentoPathUrlUtil() {
+    }
+
     /**
      * Constructs a file URL for the Magento packages root, based on the project settings.
      *
      * @param project the project instance
      * @return the constructed file URL
      */
-    public static String execute(Project project) {
-        String magentoPath = Settings.getMagentoPath(project);
+    public static String execute(final Project project) {
+        final String magentoPath = Settings.getMagentoPath(project);
         if (magentoPath != null) {
             return  VirtualFileManager.constructUrl(
                     "file",
@@ -38,8 +45,8 @@ public static String execute(Project project) {
      * @param project the project instance
      * @return the constructed file URL
      */
-    public static String getDesignPath(Project project) {
-        String magentoPath = Settings.getMagentoPath(project);
+    public static String getDesignPath(final Project project) {
+        final String magentoPath = Settings.getMagentoPath(project);
         if (magentoPath != null) {
             return  VirtualFileManager.constructUrl(
                     "file",

src/main/resources/META-INF/plugin.xml

@@ -176,10 +176,10 @@
         </group>
 
         <action id="AddMagentoContentRoot" class="com.magento.idea.magento2plugin.actions.content.root.MarkDirectoryAsMagentoContentRot" text="Mark Directory As Magento Code Root" description="Adds Magento code root">
-            <add-to-group group-id="PS.MarkRootGroup" anchor="last"/>
+            <add-to-group group-id="MarkRootGroup" anchor="last"/>
         </action>
         <action id="RemoveMagentoContentRoot" class="com.magento.idea.magento2plugin.actions.content.root.UnmarkDirectoryAsMagentoContentRot" text="Unmark Directory As Magento Code Root" description="Removes Magento code root">
-            <add-to-group group-id="PS.MarkRootGroup" anchor="last"/>
+            <add-to-group group-id="MarkRootGroup" anchor="last"/>
         </action>
     </actions>