kcl-lang / cli / 31017517112

28%

main: 28%

Build Type

Pull #364

github

Committed by claude

Commit Message

feat(version): fall back to debug.ReadBuildInfo when LDFLAGS is missing

Adds a secondary fallback so binaries built without the
`-ldflags "-X .../pkg/version.version=..."` injection still report a
useful version instead of silently displaying the hardcoded
VersionTypeLatest constant.

The lookup order is now:

  1. LDFLAGS-injected `version` variable (used by goreleaser and the
     musl release build).
  2. `debug.ReadBuildInfo().Main.Version` from the Go runtime (works
     for `go install kcl-lang.io/cli@vX.Y.Z` and similar module-aware
     builds). The leading "v" is stripped and "(devel)" / pseudo-versions
     are treated as "no version available" so the fallback can still
     kick in.
  3. Hardcoded `VersionTypeLatest` constant.

A new test asserts that the build info path returns the empty string
under `go test` (where the main module is reported as "(devel)").

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #364: fix(cli): ensure version is correctly embedded in all release binaries

Coverage Stats

10 of 16 new or added lines in 1 file covered. (62.5%)

7 existing lines in 1 file now uncovered.

247 of 892 relevant lines covered (27.69%)

0.3 hits per line