fix: skips submodule metdata autogen if readme isn't present

cli/Makefile

@@ -1,7 +1,7 @@
 SHELL := /bin/bash
 
 # Changing this value will trigger a new release
-VERSION=v0.5.4
+VERSION=v0.5.5
 BINARY=bin/cft
 GITHUB_REPO=github.com/GoogleCloudPlatform/cloud-foundation-toolkit
 PLATFORMS := linux windows darwin

cli/bpmetadata/cmd.go

@@ -58,8 +58,8 @@ func generate(cmd *cobra.Command, args []string) error {
 	var allBpPaths []string
 	currBpPath := path.Join(wdPath, mdFlags.path)
 	allBpPaths = append(allBpPaths, currBpPath)
-
 	var errors []string
+
 	// if nested, check if modules/ exists and create paths
 	// for submodules
 	if mdFlags.nested {
@@ -77,8 +77,22 @@ func generate(cmd *cobra.Command, args []string) error {
 		}
 	}
 
-	for _, path := range allBpPaths {
-		err = generateMetadataForBpPath(path)
+	for _, modPath := range allBpPaths {
+		// check if module path has readme.md
+		_, err := os.Stat(path.Join(modPath, readmeFileName))
+
+		// throw an error and exit if root module doesn't have a readme.md
+		if !strings.Contains(modPath, modulesPath+"/") && err != nil {
+			return fmt.Errorf("Top-level module does not have a readme. Details: %w\n", err)
+		}
+
+		// log info if a sub-module doesn't have a readme.md and continue
+		if err != nil {
+			Log.Info("Skiping metadata for sub-module", "Path:", modPath)
+			continue
+		}
+
+		err = generateMetadataForBpPath(modPath)
 		if err != nil {
 			errors = append(errors, err.Error())
 		}