Skip to content

NOJIRA-Fix-openapi-manager-vendor-sync - #1146

Merged
pchero merged 3 commits into
mainfrom
NOJIRA-Fix-openapi-manager-vendor-sync
Jul 30, 2026
Merged

NOJIRA-Fix-openapi-manager-vendor-sync#1146
pchero merged 3 commits into
mainfrom
NOJIRA-Fix-openapi-manager-vendor-sync

Conversation

@pchero

@pchero pchero commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Fix a version mismatch between go.mod and the committed generated models in bin-openapi-manager that was causing the CircleCI "Verify generated models match committed" check to fail, and prevent the same class of drift from recurring via automated dependency updates.

  • bin-openapi-manager: Bump oapi-codegen from v2.7.1 to v2.8.0 in go.mod/go.sum. go.mod was pinned to v2.7.1 while the committed gens/models/gen.go was actually generated with v2.8.0 (v2.8.0 keeps the type-name-prefixed enum constants, e.g. PutAisIdJSONBodyTypeInsight, that the rest of the codebase already references, and includes the // Example: doc comments). Running go generate ./... with the old pin regenerated with v2.7.1 instead, stripping the Example comments and renaming enum constants to unprefixed form, which would have broken every consumer service referencing the prefixed names.
  • bin-openapi-manager: Add .github/dependabot.yml ignoring the oapi-codegen dependency for this service, since a prior dependency sweep (NOJIRA-Fix-dependabot-go-security-alerts #1145) silently downgraded oapi-codegen to v2.7.1 as a side effect of an unrelated go mod tidy, without regenerating gens/models/gen.go. This scopes an ignore rule to prevent a future automated bump from reintroducing the same drift.

pchero added 3 commits July 30, 2026 07:59
- bin-openapi-manager: Bump oapi-codegen from v2.7.1 to v2.8.0 in go.mod/go.sum to match the version actually used to generate the committed gens/models/gen.go. go.mod was pinned to v2.7.1 while gen.go was generated with v2.8.0, so `go generate ./...` in CI regenerated with v2.7.1 and produced a diff (stripped Example comments, unprefixed enum constant names) that failed the "Verify generated models match committed" CircleCI check.
- bin-openapi-manager: Add .github/dependabot.yml ignoring the oapi-codegen dependency so future Dependabot version/security updates cannot re-drift go.mod away from the version that generated the committed gens/models/gen.go, which is what caused this PR's underlying CI failure (introduced by #1145's automated dependency sweep).
- bin-openapi-manager: Fix dependabot.yml comment that incorrectly claimed the ignore rule covers Dependabot security-update PRs. ignore and open-pull-requests-limit only govern scheduled version-update PRs; security-update PRs are gated separately by the repo's Dependabot security updates toggle (currently disabled). Comment now states this accurately and flags it as something to revisit if that toggle is ever enabled.
@pchero
pchero merged commit a0438c1 into main Jul 30, 2026
8 checks passed
@pchero
pchero deleted the NOJIRA-Fix-openapi-manager-vendor-sync branch July 30, 2026 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant