fix(release): use --output path directly for container mount (#2015)

jinseopkim0 · web-flow · commit 45aa04a8e7d2 · 2025-09-09T17:35:28.000-04:00
The 'release init' command was incorrectly creating a nested 'output' subdirectory within the path provided by the --output flag (aliased to WorkRoot). This caused the container's /output volume to be mounted to <workroot>/output instead of <workroot>. This commit changes the behavior to use the r.workRoot value directly as the source for the /output volume mount, aligning with user expectations for the --output flag. Fixes #2014
diff --git a/internal/librarian/release_init.go b/internal/librarian/release_init.go
@@ -117,7 +117,7 @@ func newInitRunner(cfg *config.Config) (*initRunner, error) {
 }
 
 func (r *initRunner) run(ctx context.Context) error {
-	outputDir := filepath.Join(r.workRoot, "output")
+	outputDir := r.workRoot
 	if err := os.MkdirAll(outputDir, 0755); err != nil {
 		return fmt.Errorf("failed to create output dir: %s", outputDir)
 	}

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ func newInitRunner(cfg config.Config) (initRunner, error) {`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`func (r *initRunner) run(ctx context.Context) error {`
`120`		`- outputDir := filepath.Join(r.workRoot, "output")`
	`120`	`+ outputDir := r.workRoot`
`121`	`121`	`if err := os.MkdirAll(outputDir, 0755); err != nil {`
`122`	`122`	`return fmt.Errorf("failed to create output dir: %s", outputDir)`
`123`	`123`	`}`