chore: invalid empty bundle types filtered from result

src/resolve/adapters/bundleSourceAdapter.ts

@@ -5,6 +5,8 @@
  * For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 import { MixedContentSourceAdapter } from './mixedContentSourceAdapter';
+import { SourcePath } from '../../common';
+import { SourceComponent } from '../sourceComponent';
 
 /**
  * Handles _bundle_ types. A bundle component has all its source files, including the
@@ -26,4 +28,30 @@ import { MixedContentSourceAdapter } from './mixedContentSourceAdapter';
  */
 export class BundleSourceAdapter extends MixedContentSourceAdapter {
   protected ownFolder = true;
+
+  /**
+   * Excludes empty bundle directories.
+   *
+   * e.g.
+   * lwc/
+   * ├── myFoo/
+   * |   ├── myFoo.js
+   * |   ├── myFooStyle.css
+   * |   ├── myFoo.html
+   * |   ├── myFoo.js-meta.xml
+   * ├── emptyLWC/
+   *
+   * so we shouldn't populate with the `emptyLWC` directory
+   *
+   * @param trigger Path that `getComponent` was called with
+   * @param component Component to populate properties on
+   * @protected
+   */
+  protected populate(trigger: SourcePath, component?: SourceComponent): SourceComponent {
+    if (this.tree.isDirectory(trigger) && !this.tree.readDirectory(trigger)?.length) {
+      // if it's an empty directory, don't include it (e.g., lwc/emptyLWC)
+      return;
+    }
+    return super.populate(trigger, component);
+  }
 }

test/mock/registry/type-constants/bundleConstants.ts

@@ -39,3 +39,22 @@ export const COMPONENT = SourceComponent.createVirtualComponent(
     },
   ]
 );
+
+export const EMPTY_BUNDLE = SourceComponent.createVirtualComponent(
+  {
+    name: 'a',
+    type,
+    xml: XML_PATH,
+    content: CONTENT_PATH,
+  },
+  [
+    {
+      dirPath: TYPE_DIRECTORY,
+      children: [COMPONENT_NAME],
+    },
+    {
+      dirPath: CONTENT_PATH,
+      children: [],
+    },
+  ]
+);

test/resolve/adapters/bundleSourceAdapter.ts

@@ -7,7 +7,8 @@
 
 import { mockRegistry, bundle } from '../../mock/registry';
 import { expect } from 'chai';
-import { BundleSourceAdapter } from '../../../src/resolve/adapters/bundleSourceAdapter';
+import { BundleSourceAdapter } from '../../../src/resolve/adapters';
+import { CONTENT_PATH } from '../../mock/registry/type-constants/bundleConstants';
 
 describe('BundleSourceAdapter', () => {
   const adapter = new BundleSourceAdapter(
@@ -21,6 +22,20 @@ describe('BundleSourceAdapter', () => {
     expect(adapter.getComponent(bundle.XML_PATH)).to.deep.equal(bundle.COMPONENT);
   });
 
+  it('Should return expected SourceComponent when given a bundle directory', () => {
+    expect(adapter.getComponent(bundle.CONTENT_PATH)).to.deep.equal(bundle.COMPONENT);
+  });
+
+  it('Should exclude empty bundle directories', () => {
+    const emptyBundleAdapter = new BundleSourceAdapter(
+      bundle.EMPTY_BUNDLE.type,
+      mockRegistry,
+      undefined,
+      bundle.EMPTY_BUNDLE.tree
+    );
+    expect(emptyBundleAdapter.getComponent(CONTENT_PATH)).to.be.undefined;
+  });
+
   it('Should return expected SourceComponent when given a source path', () => {
     const randomSource = bundle.SOURCE_PATHS[1];
     expect(adapter.getComponent(randomSource)).to.deep.equal(bundle.COMPONENT);