feat(files.service): update 'isDangerous' rules

Author: zaldih

__tests__/files.service.test.ts

@@ -147,24 +147,38 @@ describe('File Service', () => {
   });
 
   describe('#isDangerous', () => {
-    it('should get false if is considered dangerous', () => {
+    it('should return false for paths that are not considered dangerous', () => {
       expect(
         fileService.isDangerous('/home/apps/myapp/node_modules'),
       ).toBeFalsy();
       expect(fileService.isDangerous('node_modules')).toBeFalsy();
       expect(
         fileService.isDangerous('/home/user/projects/a/node_modules'),
       ).toBeFalsy();
+      expect(
+        fileService.isDangerous('/Applications/NotAnApp/node_modules'),
+      ).toBeFalsy();
+      expect(
+        fileService.isDangerous('C:\\Users\\User\\Documents\\node_modules'),
+      ).toBeFalsy();
     });
 
-    it('should get true if is not considered dangerous ', () => {
+    it('should return true for paths that are considered dangerous', () => {
       expect(
         fileService.isDangerous('/home/.config/myapp/node_modules'),
       ).toBeTruthy();
       expect(fileService.isDangerous('.apps/node_modules')).toBeTruthy();
       expect(
         fileService.isDangerous('.apps/.sample/node_modules'),
       ).toBeTruthy();
+      expect(
+        fileService.isDangerous('/Applications/MyApp.app/node_modules'),
+      ).toBeTruthy();
+      expect(
+        fileService.isDangerous(
+          'C:\\Users\\User\\AppData\\Local\\node_modules',
+        ),
+      ).toBeTruthy();
     });
   });
 

src/services/files/files.service.ts

@@ -64,16 +64,26 @@ export abstract class FileService implements IFileService {
     return path.includes(targetFolder);
   }
 
-  /** We consider a directory to be dangerous if it is hidden.
-   *
+  /**
    * > Why dangerous?
    * It is probable that if the node_module is included in some hidden directory, it is
    * required by some application like "spotify", "vscode" or "Discord" and deleting it
    * would imply breaking the application (until the dependencies are reinstalled).
+   *
+   * In the case of macOS applications and Windows AppData directory, these locations often contain
+   * application-specific data or configurations that should not be tampered with. Deleting node_modules
+   * from these locations could potentially disrupt the normal operation of these applications.
    */
   isDangerous(path: string): boolean {
     const hiddenFilePattern = /(^|\/)\.[^/.]/g;
-    return hiddenFilePattern.test(path);
+    const macAppsPattern = /(^|\/)Applications\/[^/]+\.app\//g;
+    const windowsAppDataPattern = /(^|\\)AppData\\/g;
+
+    return (
+      hiddenFilePattern.test(path) ||
+      macAppsPattern.test(path) ||
+      windowsAppDataPattern.test(path)
+    );
   }
 
   async getRecentModificationInDir(path: string): Promise<number> {