fix(internal/librariangen): run goimports on the output root (#13248)

jskeet · web-flow · commit 050f4005ab3e · 2025-10-28T08:49:06.000-06:00
diff --git a/internal/librariangen/postprocessor/postprocessor.go b/internal/librariangen/postprocessor/postprocessor.go
@@ -37,6 +37,7 @@ var (
 //
 //  1. Modify the generated snippets to specify the current version
 //  2. Run `goimports` to format the code.
+//  3. For new modules only, run "go mod init" and "go mod tidy"
 func PostProcess(ctx context.Context, req *request.Library, outputDir, moduleDir string, moduleConfig *config.ModuleConfig) error {
 	slog.Debug("librariangen: starting post-processing", "directory", moduleDir)
 
@@ -53,7 +54,7 @@ func PostProcess(ctx context.Context, req *request.Library, outputDir, moduleDir
 		return fmt.Errorf("librariangen: failed to update snippets metadata: %w", err)
 	}
 
-	if err := goimports(ctx, moduleDir); err != nil {
+	if err := goimports(ctx, outputDir); err != nil {
 		return fmt.Errorf("librariangen: failed to run 'goimports': %w", err)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ var (`
`37`	`37`	`//`
`38`	`38`	`// 1. Modify the generated snippets to specify the current version`
`39`	`39`	// 2. Run `goimports` to format the code.
	`40`	`+// 3. For new modules only, run "go mod init" and "go mod tidy"`
`40`	`41`	`func PostProcess(ctx context.Context, req request.Library, outputDir, moduleDir string, moduleConfig config.ModuleConfig) error {`
`41`	`42`	`slog.Debug("librariangen: starting post-processing", "directory", moduleDir)`
`42`	`43`
`@@ -53,7 +54,7 @@ func PostProcess(ctx context.Context, req *request.Library, outputDir, moduleDir`
`53`	`54`	`return fmt.Errorf("librariangen: failed to update snippets metadata: %w", err)`
`54`	`55`	`}`
`55`	`56`
`56`		`- if err := goimports(ctx, moduleDir); err != nil {`
	`57`	`+ if err := goimports(ctx, outputDir); err != nil {`
`57`	`58`	`return fmt.Errorf("librariangen: failed to run 'goimports': %w", err)`
`58`	`59`	`}`
`59`	`60`