kcl-lang / cli / 36157376644

35%

main: 35%

Pull #389

github

fix(version): always emit `-{goos}-{goarch}` suffix in GetVersionString

The previous implementation only formatted the platform suffix on the
`VersionTypeLatest.String()` fallback, while the two primary branches
(`-ldflags`-injected `version` and `debug.ReadBuildInfo().Main.Version`)
returned the bare version string. As a result, `kcl version` printed
inconsistent output depending on which branch fired for a given binary
— for example "0.12.10" vs "0.12.10-darwin-arm64" — which is what
kcl-lang/cli#386 reports after a release that uses ldflags injection.

Funnel every source through `getVersion(v)` so the output is always
`{version}-{goos}-{goarch}`, and add a regression test that exercises
each of the three resolution paths.

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

6 of 6 new or added lines in 1 file covered. (100.0%)

394 of 1117 relevant lines covered (35.27%)

0.41 hits per line