chore(helm/v2-alpha): add --input-dir flag for flexible project path resolution

Add --input-dir flag to the helm v2-alpha plugin to align with the
alpha generate command pattern. This provides a consistent user
experience across commands and makes the plugin more flexible.

Changes:
- Add --input-dir flag to specify the directory containing PROJECT file and manifests
- Resolve manifests path relative to input-dir when not absolute
- Add documentation and examples for the new flag

Co-Authored-By: porridge@redhat.com

Generated-by: Claude

Author: camilamacedo86

docs/book/src/plugins/available/helm-v2-alpha.md

@@ -72,6 +72,14 @@ kubebuilder edit --plugins=helm/v2-alpha --output-dir=charts
 kubebuilder edit --plugins=helm/v2-alpha \
   --manifests=manifests/install.yaml \
   --output-dir=helm-charts
+
+# Generate chart from a different project directory
+kubebuilder edit --plugins=helm/v2-alpha --input-dir=./path/to/project
+
+# Use custom manifests file (resolved relative to input-dir)
+kubebuilder edit --plugins=helm/v2-alpha \
+  --input-dir=./path/to/project \
+  --manifests=custom/install.yaml
 ```
 
 ## Chart Structure
@@ -353,9 +361,10 @@ The Makefile targets use sensible defaults extracted from your project configura
 
 | Flag                | Description                                                                 |
 |---------------------|-----------------------------------------------------------------------------|
-| **--manifests**     | Path to YAML file containing Kubernetes manifests (default: `dist/install.yaml`) |
-| **--output-dir** string | Output directory for chart (default: `dist`)                                |
-| **--force**         | Regenerates preserved files except `Chart.yaml` (`values.yaml`, `NOTES.txt`, `_helpers.tpl`, `.helmignore`, `test-chart.yml`) |
+| **--input-dir**     | Path to the directory containing the PROJECT file and manifests. Defaults to current directory. |
+| **--manifests**     | Path to YAML file containing Kubernetes manifests from kustomize output. Default: `dist/install.yaml`. Resolved relative to input-dir if not absolute. |
+| **--output-dir**    | Output directory for the generated Helm chart. Default: `dist`. |
+| **--force**         | If true, regenerates all preserved files except `Chart.yaml` (includes `values.yaml`, `NOTES.txt`, `_helpers.tpl`, `.helmignore`, `test-chart.yml`). |
 
 <aside class="note" role="note">
 <p class="note-title"> Examples </p>

pkg/plugins/optional/helm/v2alpha/edit.go

@@ -50,6 +50,7 @@ var _ plugin.EditSubcommand = &editSubcommand{}
 type editSubcommand struct {
 	config        config.Config
 	force         bool
+	inputDir      string
 	manifestsFile string
 	outputDir     string
 }
@@ -76,6 +77,12 @@ distribution of your project. When enabled, adds Helm helpers targets to Makefil
 # Generate from custom manifests to custom output directory
   %[1]s edit --plugins=%[2]s --manifests=manifests/install.yaml --output-dir=helm-charts
 
+# Generate chart from a different project directory
+  %[1]s edit --plugins=%[2]s --input-dir=./path/to/project
+
+# Use custom manifests file (resolved relative to input-dir)
+  %[1]s edit --plugins=%[2]s --input-dir=./path/to/project --manifests=custom/install.yaml
+
 # Typical workflow:
   make build-installer  # Generate dist/install.yaml with latest changes
   %[1]s edit --plugins=%[2]s  # Generate/update Helm chart in dist/chart/
@@ -103,8 +110,11 @@ The generated chart structure mirrors your config/ directory:
 
 func (p *editSubcommand) BindFlags(fs *pflag.FlagSet) {
 	fs.BoolVar(&p.force, "force", false, "if true, regenerates all the files")
+	fs.StringVar(&p.inputDir, "input-dir", "",
+		"path to the directory containing the PROJECT file and manifests (defaults to current directory)")
 	fs.StringVar(&p.manifestsFile, "manifests", DefaultManifestsFile,
-		"path to the YAML file containing Kubernetes manifests from kustomize output")
+		"path to the YAML file containing Kubernetes manifests from kustomize output "+
+			"(resolved relative to input-dir if not absolute)")
 	fs.StringVar(&p.outputDir, "output-dir", DefaultOutputDir, "output directory for the generated Helm chart")
 }
 
@@ -114,14 +124,38 @@ func (p *editSubcommand) InjectConfig(c config.Config) error {
 }
 
 func (p *editSubcommand) Scaffold(fs machinery.Filesystem) error {
+	// Resolve input directory (defaults to current working directory)
+	if p.inputDir == "" {
+		cwd, err := os.Getwd()
+		if err != nil {
+			return fmt.Errorf("failed to get current working directory: %w", err)
+		}
+		p.inputDir = cwd
+	}
+
+	// Convert input directory to absolute path if needed
+	if !filepath.IsAbs(p.inputDir) {
+		absInputDir, err := filepath.Abs(p.inputDir)
+		if err != nil {
+			return fmt.Errorf("failed to resolve input directory %q: %w", p.inputDir, err)
+		}
+		p.inputDir = absInputDir
+	}
+
+	// Resolve manifests path relative to input directory if not absolute
+	manifestsPath := p.manifestsFile
+	if !filepath.IsAbs(manifestsPath) {
+		manifestsPath = filepath.Join(p.inputDir, manifestsPath)
+	}
+
 	// If using default manifests file, ensure it exists by running make build-installer
 	if p.manifestsFile == DefaultManifestsFile {
-		if err := p.ensureManifestsExist(); err != nil {
-			slog.Warn("Failed to generate default manifests file", "error", err, "file", p.manifestsFile)
+		if err := p.ensureManifestsExist(manifestsPath); err != nil {
+			slog.Warn("Failed to generate default manifests file", "error", err, "file", manifestsPath)
 		}
 	}
 
-	scaffolder := scaffolds.NewKustomizeHelmScaffolder(p.config, p.force, p.manifestsFile, p.outputDir)
+	scaffolder := scaffolds.NewKustomizeHelmScaffolder(p.config, p.force, manifestsPath, p.outputDir)
 	scaffolder.InjectFS(fs)
 	err := scaffolder.Scaffold()
 	if err != nil {
@@ -175,7 +209,7 @@ func (p *editSubcommand) Scaffold(fs machinery.Filesystem) error {
 	if isFirstRun {
 		slog.Info("adding Helm deployment targets to Makefile...")
 		// Extract namespace from manifests for accurate Makefile generation
-		namespace := p.extractNamespaceFromManifests()
+		namespace := p.extractNamespaceFromManifests(manifestsPath)
 		if err := p.addHelmMakefileTargets(namespace); err != nil {
 			slog.Warn("failed to add Helm targets to Makefile", "error", err)
 		}
@@ -185,8 +219,20 @@ func (p *editSubcommand) Scaffold(fs machinery.Filesystem) error {
 }
 
 // ensureManifestsExist runs make build-installer to generate the default manifests file
-func (p *editSubcommand) ensureManifestsExist() error {
-	slog.Info("Generating default manifests file", "file", p.manifestsFile)
+func (p *editSubcommand) ensureManifestsExist(manifestsPath string) error {
+	slog.Info("Generating default manifests file", "file", manifestsPath)
+
+	// Change to input directory to run make commands
+	originalDir, err := os.Getwd()
+	if err != nil {
+		return fmt.Errorf("failed to get current directory: %w", err)
+	}
+	if err := os.Chdir(p.inputDir); err != nil {
+		return fmt.Errorf("failed to change to input directory %q: %w", p.inputDir, err)
+	}
+	defer func() {
+		_ = os.Chdir(originalDir)
+	}()
 
 	// Run the required make targets to generate the manifests file
 	targets := []string{"manifests", "generate", "build-installer"}
@@ -197,11 +243,11 @@ func (p *editSubcommand) ensureManifestsExist() error {
 	}
 
 	// Verify the file was created
-	if _, err := os.Stat(p.manifestsFile); err != nil {
-		return fmt.Errorf("manifests file %s was not created: %w", p.manifestsFile, err)
+	if _, err := os.Stat(manifestsPath); err != nil {
+		return fmt.Errorf("manifests file %s was not created: %w", manifestsPath, err)
 	}
 
-	slog.Info("Successfully generated manifests file", "file", p.manifestsFile)
+	slog.Info("Successfully generated manifests file", "file", manifestsPath)
 	return nil
 }
 
@@ -265,17 +311,17 @@ func (p *editSubcommand) addHelmMakefileTargets(namespace string) error {
 
 // extractNamespaceFromManifests parses the manifests file to extract the manager namespace.
 // Returns projectName-system if manifests don't exist or namespace not found.
-func (p *editSubcommand) extractNamespaceFromManifests() string {
+func (p *editSubcommand) extractNamespaceFromManifests(manifestsPath string) string {
 	// Default to project-name-system pattern
 	defaultNamespace := p.config.GetProjectName() + "-system"
 
 	// If manifests file doesn't exist, use default
-	if _, err := os.Stat(p.manifestsFile); os.IsNotExist(err) {
+	if _, err := os.Stat(manifestsPath); os.IsNotExist(err) {
 		return defaultNamespace
 	}
 
 	// Parse the manifests to get the namespace
-	file, err := os.Open(p.manifestsFile)
+	file, err := os.Open(manifestsPath)
 	if err != nil {
 		return defaultNamespace
 	}

pkg/plugins/optional/helm/v2alpha/edit_test.go

@@ -84,6 +84,10 @@ version: "3"
 			editCmd.BindFlags(flagSet)
 
 			// Check that flags were added
+			inputDirFlag := flagSet.Lookup("input-dir")
+			Expect(inputDirFlag).NotTo(BeNil())
+			Expect(inputDirFlag.DefValue).To(Equal(""))
+
 			manifestsFlag := flagSet.Lookup("manifests")
 			Expect(manifestsFlag).NotTo(BeNil())
 			Expect(manifestsFlag.DefValue).To(Equal(DefaultManifestsFile))