Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

Commit 277ace9

Browse files
mromaszewiczclaude
andauthored
Fix runtime params/types/helpers package name collisions (#18) (#23)
Use explicit import aliases (oapiCodegenParamsPkg, oapiCodegenTypesPkg, oapiCodegenHelpersPkg) for runtime sub-packages so generated code no longer collides with local variables named `params` or `types`. Fixes #18 Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent bb4e2af commit 277ace9

2 files changed

Lines changed: 11 additions & 11 deletions

File tree

experimental/codegen/internal/codegen.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ func Generate(doc libopenapi.Document, specData []byte, cfg Configuration) (stri
2424
var runtimePrefixes RuntimePrefixes
2525
if cfg.Generation.RuntimePackage != nil {
2626
runtimePrefixes = RuntimePrefixes{
27-
Params: "params.",
28-
Types: "types.",
29-
Helpers: "helpers.",
27+
Params: "oapiCodegenParamsPkg.",
28+
Types: "oapiCodegenTypesPkg.",
29+
Helpers: "oapiCodegenHelpersPkg.",
3030
}
3131
ctx.SetRuntimePrefixes(runtimePrefixes.Params, runtimePrefixes.Types, runtimePrefixes.Helpers)
3232
}
@@ -343,12 +343,12 @@ func Generate(doc libopenapi.Document, specData []byte, cfg Configuration) (stri
343343
if cfg.Generation.RuntimePackage != nil {
344344
// Runtime package is configured — don't embed helpers, import them.
345345
// Add imports for whichever sub-packages are actually needed.
346-
ctx.AddImport(cfg.Generation.RuntimePackage.TypesImport())
346+
ctx.AddImportAlias(cfg.Generation.RuntimePackage.TypesImport(), "oapiCodegenTypesPkg")
347347
if ctx.HasAnyParams() {
348-
ctx.AddImport(cfg.Generation.RuntimePackage.ParamsImport())
348+
ctx.AddImportAlias(cfg.Generation.RuntimePackage.ParamsImport(), "oapiCodegenParamsPkg")
349349
}
350350
if len(ctx.RequiredHelpers()) > 0 {
351-
ctx.AddImport(cfg.Generation.RuntimePackage.HelpersImport())
351+
ctx.AddImportAlias(cfg.Generation.RuntimePackage.HelpersImport(), "oapiCodegenHelpersPkg")
352352
}
353353
} else {
354354
// Resolve param template dependencies first — this may register

experimental/examples/webhook/doorbadge.gen.go

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)