Repository navigation
feat: show build version and git commit in UI and startup log - #1177
Merged
Merged
Conversation
Closes HelpCode-ai#1000 - packages/backend/src/common/app-version.ts: single source of truth exporting APP_VERSION, APP_COMMIT (APP_COMMIT || SENTRY_RELEASE || dev) and APP_BUILD_DATE (APP_BUILD_DATE || null), moving readAppVersion() out of mcp-endpoint.controller.ts - packages/backend/src/tracing.ts + instrument.ts: use APP_VERSION as the service.version / Sentry release fallback instead of npm_package_version - packages/backend/src/main.ts: log version, commit, build date and mode at startup - packages/backend/src/health/version.controller.ts: new GET /api/version behind AuthGuard('jwt') returning { version, commit, buildDate, deploymentMode }; registered in health.module.ts - packages/frontend/src/components/footer.tsx: show v{version} for signed-in users, with full version+commit in the tooltip - Dockerfile: APP_COMMIT and APP_BUILD_DATE build args to ENV, plus org.opencontainers.image.revision and .version OCI labels - app-version.spec.ts: unit tests for env precedence and dev fallback - docs: deployment.md documents the build args; observability.md notes the service.version source change
- admin/page.tsx: import versionApi (TS2304 at line 68) - admin/page.tsx: use copyText from @/lib/clipboard for the Copy button (navigator.clipboard is undefined on plain-HTTP origins); show "Version info unavailable" when the fetch fails - docker-publish.yml: APP_BUILD_DATE from steps.meta labels instead of github.event.head_commit.timestamp (empty on workflow_dispatch) - observability.md: OTEL_SERVICE_VERSION default is APP_VERSION from package.json; sentence stays below the table - deployment.md: Build provenance is a ### subsection under Production Deployment (after "With Docker (Manual)"); use git rev-parse HEAD (40-char SHA) instead of --short HEAD so the footer link works - footer.tsx: reset versionPromise on failed fetch so a mid-session token expiry doesn't hide the version until page reload
5 tasks done
Contributor
|
Merged, thanks @Furox-Art! It goes out with the next release, and after that every instance shows its version and commit in the footer and on the admin page. Thanks for seeing #1000 through. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Re-opens #1023, which was closed when its head branch was lost during a fork cleanup. This branch carries the exact same commits (head
324f185), with your approval and green CI from the final review round intact.What this does
APP_VERSION,APP_COMMITandAPP_BUILD_DATEthrough the Docker build/publish workflowdeployment.md, service identity row inobservability.mdLocal verification on
324f185npm ci— passapp-version.spec.ts— 6 tests passThanks @helpcodeai — sorry for the churn on the branch.