fix: skip operationId write-back for alias operations

mromaszewicz · claude · mromaszewicz · commit 44d30accb0b2 · 2026-04-02T21:35:04.000-07:00
Alias paths share the same *openapi3.Operation pointer as the canonical
path. Writing the suffixed alias name (e.g. GetTestAlias0) back to
op.OperationID would corrupt the canonical operation's ID in the
embedded spec.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/pkg/codegen/operations.go b/pkg/codegen/operations.go
@@ -672,8 +672,10 @@ func OperationDefinitions(swagger *openapi3.T) ([]OperationDefinition, error) {
 				operationId = operationId + fmt.Sprintf("Alias%d", n)
 			}
 
-			if !globalState.options.Compatibility.PreserveOriginalOperationIdCasingInEmbeddedSpec {
-				// update the existing, shared, copy of the spec if we're not wanting to preserve it
+			if !globalState.options.Compatibility.PreserveOriginalOperationIdCasingInEmbeddedSpec && !isAlias {
+				// update the existing, shared, copy of the spec if we're not wanting to preserve it.
+				// Skip for aliases: they share the same *Operation as the canonical path,
+				// and writing the suffixed name back would corrupt the original.
 				op.OperationID = operationId
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -672,8 +672,10 @@ func OperationDefinitions(swagger *openapi3.T) ([]OperationDefinition, error) {`
`672`	`672`	`operationId = operationId + fmt.Sprintf("Alias%d", n)`
`673`	`673`	`}`
`674`	`674`
`675`		`- if !globalState.options.Compatibility.PreserveOriginalOperationIdCasingInEmbeddedSpec {`
`676`		`- // update the existing, shared, copy of the spec if we're not wanting to preserve it`
	`675`	`+ if !globalState.options.Compatibility.PreserveOriginalOperationIdCasingInEmbeddedSpec && !isAlias {`
	`676`	`+ // update the existing, shared, copy of the spec if we're not wanting to preserve it.`
	`677`	`+ // Skip for aliases: they share the same *Operation as the canonical path,`
	`678`	`+ // and writing the suffixed name back would corrupt the original.`
`677`	`679`	`op.OperationID = operationId`
`678`	`680`	`}`
`679`	`681`