feat: [AH-1598]: fix bugbash issues

Author: shivanand-harness

cmd/ar/command/push_go.go

@@ -136,7 +136,7 @@ func NewPushGoCmd(c *client.ClientWithResponses) *cobra.Command {
 			// Check response
 			if resp.StatusCode() != http.StatusOK && resp.StatusCode() != http.StatusCreated {
 				progress.Error("Upload failed")
-				return fmt.Errorf("failed to push package: %s", resp.Status())
+				return fmt.Errorf("failed to push package: %s \n response: %s", resp.Status(), resp.Body)
 			}
 
 			progress.Success(fmt.Sprintf("Successfully uploaded package %s", packageName))

module/ar/packages/gopkg/impl.go

@@ -17,6 +17,10 @@ import (
 	"github.com/harness/harness-cli/util/common/vcs"
 )
 
+const (
+	ModulePathRegex = `^[a-z0-9](?:[a-z0-9-]*[a-z0-9])?(?:\.[a-z0-9](?:[a-z0-9-]*[a-z0-9])?)+(?:/[a-z0-9][a-z0-9._-]*)*$`
+)
+
 // DefaultGenerator provides the default implementation of package generation interfaces.
 // It combines the default implementations of ModuleValidator, FileGenerator, and
 // VCSMetadataProvider to provide a complete package generation solution.
@@ -66,6 +70,10 @@ func (v *defaultModuleValidator) ValidateModulePath(modulePath, version string)
 		return errors.NewValidationError("version", err.Error())
 	}
 
+	if !regexp.MustCompile(ModulePathRegex).MatchString(modulePath) {
+		return errors.NewValidationError("module_path", "module path must match regex "+ModulePathRegex)
+	}
+
 	if major <= 1 {
 		if strings.HasSuffix(modulePath, "/v2") || strings.HasSuffix(modulePath, fmt.Sprintf("/v%d", major+1)) {
 			return errors.NewValidationError("module_path", "module path must not have /vN suffix for v1 or below")