WIP: adding a note

src/pkg/bundler/fetcher/common.go

@@ -68,13 +68,14 @@ func filterPkgPaths(pkgPaths *layout.PackagePaths, includeLayers []string, optio
 		// include all paths that aren't in the blobs dir
 		if !strings.Contains(path, config.BlobsDir) {
 			// only grab req'd + specified optional components
-			if strings.HasPrefix(path, "components/") {
+			if strings.Contains(path, "/components/") {
 				if shouldInclude := utils.IncludeComponent(path, optionalComponents); shouldInclude {
 					filteredPaths = append(filteredPaths, path)
 					continue
 				}
+			} else {
+				filteredPaths = append(filteredPaths, path)
 			}
-			filteredPaths = append(filteredPaths, path)
 		}
 		// include paths that are in the blobs dir and are in includeLayers
 		for _, layer := range includeLayers {

src/test/e2e/bundle_test.go

@@ -543,6 +543,10 @@ func TestBundleOptionalComponents(t *testing.T) {
 	createLocal(t, bundleDir, e2e.Arch)
 	publishInsecure(t, bundlePath, "localhost:888")
 
+	// todo: look through bundle contents to make sure only the selected components are present
+	// local pkgs will a correct pkg manifest (missing non-selected optional component tarballs)
+	// remote pkgs will not, because they already have a pkg manifest and we don't want to rewrite it
+
 	t.Run("test local deploy", func(t *testing.T) {
 		deploy(t, bundlePath)
 		remove(t, bundlePath)