fix: W-18991079 - filter out empty dirs when resolving components from path (#1589)

* fix: filter out empty dirs when resolving components from path

* chore: fix for windows tests

Author: CristiCanizales

src/resolve/metadataResolver.ts

@@ -88,6 +88,11 @@ export class MetadataResolver {
 
       if (this.tree.isDirectory(fsPath)) {
         if (resolveDirectoryAsComponent(this.registry)(this.tree)(fsPath)) {
+          // Filter out empty directories to prevent deployment issues
+          if (this.tree.readDirectory(fsPath).length === 0) {
+            continue;
+          }
+
           const component = this.resolveComponent(fsPath, true);
           if (component && (!inclusiveFilter || inclusiveFilter.has(component))) {
             components.push(component);

test/collections/componentSet.test.ts

@@ -220,7 +220,7 @@ describe('ComponentSet', () => {
 
         let preAndPostRetrieveEventCount = 0;
         lifecycleEmitStub.args.forEach((event) => {
-          if (event[0] === ('scopedPreRetrieve' || 'scopedPostRetrieve')) {
+          if (event[0] === 'scopedPreRetrieve' || event[0] === 'scopedPostRetrieve') {
             preAndPostRetrieveEventCount = preAndPostRetrieveEventCount + 1;
           }
         });

test/resolve/metadataResolver.test.ts

@@ -979,5 +979,40 @@ describe('MetadataResolver', () => {
         expect(result).to.deep.equal([]);
       });
     });
+
+    it('should filter out empty directories when resolving components', () => {
+      const resolver = testUtil.createMetadataResolver([
+        {
+          dirPath: bundle.TYPE_DIRECTORY,
+          children: ['myComponent', 'emptyComponent'],
+        },
+        {
+          dirPath: bundle.CONTENT_PATH,
+          children: [bundle.XML_NAME, ...bundle.COMPONENTS],
+        },
+        {
+          dirPath: join(bundle.TYPE_DIRECTORY, 'emptyComponent'),
+          children: [], // Empty directory
+        },
+      ]);
+
+      testUtil.stubAdapters([
+        {
+          type: registry.types.auradefinitionbundle,
+          componentMappings: [
+            {
+              path: bundle.CONTENT_PATH,
+              component: bundle.COMPONENT,
+            },
+          ],
+        },
+      ]);
+
+      const components = resolver.getComponentsFromPath(bundle.TYPE_DIRECTORY);
+
+      // Should only return the non-empty component
+      expect(components).to.have.length(1);
+      expect(components[0].name).to.equal('myComponent');
+    });
   });
 });