Skip to content

fix: repair CodeGuard v1.8.0 regressions - #168

Merged
alxxjohn merged 8 commits into
mainfrom
fix/v1.8.0-repairs
Aug 29, 2026
Merged

fix: repair CodeGuard v1.8.0 regressions#168
alxxjohn merged 8 commits into
mainfrom
fix/v1.8.0-repairs

Conversation

@alxxjohn

Copy link
Copy Markdown
Contributor

Summary

  • resolve Go imports through owning modules, nested modules, and go.work use/replace metadata
  • make baseline matching deterministic and one-to-one while separating exact duplicates from context/content collisions
  • keep audit rule counts aligned with confidence and language distributions
  • report linker- or Go-module-injected versions before the development fallback
  • exempt regexp.MustCompile globals unless reassignment is demonstrated
  • allow grammatical imperative boolean parameter names in Go and C++

Testing

  • added regression tests first for workspace imports, collision matching/accounting, version precedence/build injection, mutable globals, and boolean naming
  • go test ./... -count=1
  • go vet ./...
  • gofmt check
  • go build -trimpath -o /private/tmp/codeguard-v180-repairs ./cmd/codeguard

Notes

  • public baseline/report schemas, CLI flags, and rule IDs remain unchanged
  • golangci-lint was not available in the local environment; CI remains authoritative for that gate

@alxxjohn
alxxjohn merged commit 7443747 into main Aug 29, 2026
16 checks passed
@alxxjohn
alxxjohn deleted the fix/v1.8.0-repairs branch August 29, 2026 21:11
alxxjohn added a commit that referenced this pull request Aug 29, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.8.1](v1.8.0...v1.8.1)
(2026-08-29)


### Bug Fixes

* consume baseline matches deterministically
([bf865ce](bf865ce))
* improve Go global and boolean naming precision
([dcb4ad6](dcb4ad6))
* repair CodeGuard v1.8.0 regressions
([#168](#168))
([7443747](7443747))
* report installed module version
([f828a65](f828a65))
* resolve Go imports by owning workspace module
([8b952e3](8b952e3))
* satisfy strict lint checks
([b4a1c18](b4a1c18))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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