Skip to content

feat(ui): display running version in footer (fixes #629) #1409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
opspawn wants to merge 4 commits into
base: main
Choose a base branch
from
+41 −2
Open

feat(ui): display running version in footer (fixes #629) #1409

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
78899bd
feat(ui): display running version in footer
jmhbh Feb 21, 2026
efa3798
fix: strip leading 'v' prefix from version to prevent duplication
jmhbh Feb 21, 2026
829ae10
Trigger CI re-run (build (ui) failed due to transient Alpine mirror e…
jmhbh Feb 27, 2026
e188fb7
fix: handle transient bun segfault during UI build
Feb 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion ui/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,15 +43,22 @@ RUN --mount=type=cache,target=/cache/node_modules,rw \
### STAGE 2: Build
FROM --platform=$BUILDPLATFORM deps AS builder

ARG VERSION
ENV NEXT_PUBLIC_KAGENT_VERSION=$VERSION

# Copy source files
COPY . .

# Build the application (native compilation for speed)
# Note: bun may segfault during process cleanup after a successful build
# (known bun bug, exit code 132/SIGILL). We capture the exit code and
# verify the build output exists to distinguish real failures from crashes.
RUN --mount=type=cache,target=/cache/node_modules,rw \
--mount=type=cache,target=/app/ui/.next/cache,rw \
export NEXT_TELEMETRY_DEBUG=1 \
&& bun install --frozen-lockfile \
&& bun run build \
&& { bun run build; BUN_EXIT=$?; } \
&& if [ $BUN_EXIT -ne 0 ] && [ ! -d /app/ui/.next/standalone ]; then exit $BUN_EXIT; fi \
Copy link

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workaround for the known bun segfault bug only checks for directory existence ([ ! -d /app/ui/.next/standalone ]), not whether the build output is complete or valid. A scenario where bun run build exits with a non-zero code (e.g. due to a TypeScript error or OOM) but a partial /app/ui/.next/standalone directory already exists from a previous cached build layer would silently pass, producing a Docker image built from stale artifacts.

Consider also checking for a key file inside the standalone directory, such as server.js, to more reliably distinguish a successful build from a partial/stale one: [ ! -f /app/ui/.next/standalone/server.js ].

Suggested change
&& if [ $BUN_EXIT -ne 0 ] && [ ! -d /app/ui/.next/standalone ]; then exit $BUN_EXIT; fi \
&& if [ "$BUN_EXIT" -ne 0 ] && { [ ! -d /app/ui/.next/standalone ] || [ ! -f /app/ui/.next/standalone/server.js ]; }; then exit "$BUN_EXIT"; fi \

Copilot uses AI. Check for mistakes.
&& mkdir -p /app/ui/public

### STAGE 3: Runtime
Expand Down
10 changes: 10 additions & 0 deletions ui/conf/nginx.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,16 @@ http {
return 200 'OK';
}

# Version endpoint
location /version {
proxy_pass http://kagent_backend/version;
proxy_http_version 1.1;
proxy_set_header Host $host;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $scheme;
}
Comment on lines +82 to +89
Copy link

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using location /version (prefix match) means nginx will also proxy /versions, /version/anything, or any other URI starting with /version to the backend. Since the backend likely has no route for such paths and would return 404, this is low risk, but using location = /version (exact match) — consistent with how the /health check could be written — would be more precise and prevent unexpected requests from reaching the backend. Note that /health has the same issue, so this could be addressed holistically if desired.

Copilot uses AI. Check for mistakes.

# Backend routes
location /api/ {
proxy_pass http://kagent_backend/api/;
Expand Down
24 changes: 23 additions & 1 deletion ui/src/components/Footer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,33 @@
"use client";

import { useEffect, useState } from "react";
import KagentLogo from "./kagent-logo";

export function Footer() {
const [version, setVersion] = useState(process.env.NEXT_PUBLIC_KAGENT_VERSION || "");

useEffect(() => {
if (version) return;

fetch("/version")
.then((res) => (res.ok ? res.json() : null))
.then((data) => {
if (data?.kagent_version) {
setVersion(data.kagent_version);
}
})
.catch(() => {
// Version endpoint not available — leave blank
});
}, [version]);

return (
<footer className="mt-auto py-5">
<div className="text-center text-sm text-muted-foreground flex items-center justify-center gap-2">
<KagentLogo animate={true} className="h-6 w-6 text-[#942DE7]" />
<p>is an open source project</p>
<p>
{version ? `v${version.replace(/^v/, "")} · ` : ""}is an open source project
</p>
</div>
</footer>
);
Expand Down
Loading