Merge branch 'fix/version-channel-label' into 'main'
fix(version): channel-based label (dev/local), no stale package.json fallback See merge request 2-6/2-6-4/terravault/ordinis!49
This commit is contained in:
@@ -14,6 +14,7 @@ export function VersionBadge() {
|
||||
const {
|
||||
clientVersion,
|
||||
clientTag,
|
||||
clientCI,
|
||||
clientCommit,
|
||||
clientBuiltAt,
|
||||
serverVersion,
|
||||
@@ -22,18 +23,31 @@ export function VersionBadge() {
|
||||
updateAvailable,
|
||||
} = useAppVersion()
|
||||
|
||||
// Priority: serverTag (что прод RUNS) → clientTag (что build'илось на tag
|
||||
// pipeline) → v${clientVersion} (package.json fallback для local / branch).
|
||||
// 'none' — Spring Boot build-info sentinel когда tag отсутствует.
|
||||
// Channel-based label per "Vote A" — НЕ показываем package.json fallback
|
||||
// как версию когда tag'а нет. Stale package.json (e.g. 1.2.0 когда уже
|
||||
// выпущен v1.2.2) создавал ложное впечатление rollback'а.
|
||||
//
|
||||
// Channels:
|
||||
// serverTag real — что прод/staging RUNS (most authoritative)
|
||||
// clientTag — что build'ил CI tag pipeline
|
||||
// CI без tag — 'dev' (staging / branch CI build)
|
||||
// no CI no tag — 'local' (pnpm dev на машине разработчика)
|
||||
//
|
||||
// package.json.version используется ТОЛЬКО в tooltip как build-time floor
|
||||
// — пользователь не видит его если не наведёт.
|
||||
const displayVersion =
|
||||
serverTag && serverTag !== 'none'
|
||||
? serverTag
|
||||
: clientTag
|
||||
? `v${clientTag}`
|
||||
: `v${clientVersion}`
|
||||
: clientCI
|
||||
? 'dev'
|
||||
: 'local'
|
||||
|
||||
const tooltip = [
|
||||
`client: ${clientTag ? `v${clientTag}` : `v${clientVersion}`} (${clientCommit})`,
|
||||
`client: ${
|
||||
clientTag ? `v${clientTag}` : clientCI ? `dev (floor v${clientVersion})` : `local (floor v${clientVersion})`
|
||||
} (${clientCommit})`,
|
||||
`server: ${serverTag || serverVersion} (${serverCommit ?? '...'})`,
|
||||
`built: ${clientBuiltAt}`,
|
||||
].join('\n')
|
||||
|
||||
@@ -21,8 +21,11 @@ export function useAppVersion() {
|
||||
const clientCommit = __APP_COMMIT__
|
||||
const clientVersion = __APP_VERSION__
|
||||
// Empty string when build wasn't на tag pipeline. VersionBadge / UpdateBanner
|
||||
// решают priority: serverTag → clientTag → fallback v${clientVersion}.
|
||||
// решают priority: serverTag → clientTag → channel-based label.
|
||||
const clientTag = __APP_TAG__
|
||||
// true когда GitLab CI собрал bundle. Отличает branch build (dev) от
|
||||
// local pnpm dev (local) — оба не имеют tag'а, но семантически разные.
|
||||
const clientCI = __APP_CI__
|
||||
const clientBuiltAt = __APP_BUILT_AT__
|
||||
|
||||
const knownClient = clientCommit !== 'unknown' && clientCommit !== ''
|
||||
@@ -33,6 +36,7 @@ export function useAppVersion() {
|
||||
return {
|
||||
clientVersion,
|
||||
clientTag,
|
||||
clientCI,
|
||||
clientCommit,
|
||||
clientBuiltAt,
|
||||
serverVersion: server?.version,
|
||||
|
||||
Vendored
+1
@@ -24,5 +24,6 @@ interface ImportMeta {
|
||||
*/
|
||||
declare const __APP_VERSION__: string
|
||||
declare const __APP_TAG__: string
|
||||
declare const __APP_CI__: boolean
|
||||
declare const __APP_COMMIT__: string
|
||||
declare const __APP_BUILT_AT__: string
|
||||
|
||||
@@ -28,6 +28,10 @@ const APP_VERSION: string = JSON.parse(
|
||||
// --build-arg GIT_TAG="${CI_COMMIT_TAG:-none}"). Filter → пустая строка.
|
||||
const RAW_TAG = process.env.CI_COMMIT_TAG ?? ''
|
||||
const APP_TAG: string = RAW_TAG === 'none' ? '' : RAW_TAG.replace(/^v/, '')
|
||||
// GitLab CI ставит CI=true. Используем для discriminate "CI branch build"
|
||||
// (нет tag'а, но есть pipeline) vs "local pnpm dev" (нет ни того ни другого).
|
||||
// VersionBadge показывает 'dev' для первого, 'local' для второго.
|
||||
const APP_CI: boolean = process.env.CI === 'true'
|
||||
const APP_COMMIT: string =
|
||||
process.env.CI_COMMIT_SHORT_SHA ??
|
||||
(() => {
|
||||
@@ -45,6 +49,7 @@ export default defineConfig({
|
||||
define: {
|
||||
__APP_VERSION__: JSON.stringify(APP_VERSION),
|
||||
__APP_TAG__: JSON.stringify(APP_TAG),
|
||||
__APP_CI__: JSON.stringify(APP_CI),
|
||||
__APP_COMMIT__: JSON.stringify(APP_COMMIT),
|
||||
__APP_BUILT_AT__: JSON.stringify(APP_BUILT_AT),
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user