From 24b6a67bee9d7f1941e15efaf3c9e28918cb0486 Mon Sep 17 00:00:00 2001 From: Jeffrey Alan Scudder Date: Tue, 24 Mar 2026 13:07:35 -0700 Subject: [PATCH] fix: show correct build variant (C/CL/both) in os.mjs telemetry Add variant field to build snapshots. os.mjs now reads variant instead of only checking stage prefix for cl- detection. Co-Authored-By: Claude Opus 4.6 (1M context) --- oven/native-builder.mjs | 1 + system/public/aesthetic.computer/disks/os.mjs | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/oven/native-builder.mjs b/oven/native-builder.mjs index 26b97855f8..a576d37d00 100644 --- a/oven/native-builder.mjs +++ b/oven/native-builder.mjs @@ -188,6 +188,7 @@ function makeSnapshot(job, opts = {}) { exitCode: job.exitCode, error: job.error, buildName: job.buildName || null, + variant: job.variant || "c", commitMsg: job.commitMsg || null, logCount: job.logs.length, elapsedMs: job.startedAt diff --git a/system/public/aesthetic.computer/disks/os.mjs b/system/public/aesthetic.computer/disks/os.mjs index b578fa17f7..03a1c9acf1 100644 --- a/system/public/aesthetic.computer/disks/os.mjs +++ b/system/public/aesthetic.computer/disks/os.mjs @@ -468,8 +468,8 @@ function paint($) { // --- ACTIVE BUILD --- if (activeBuild) { const rawStage = activeBuild.stage || "starting"; - const isCLBuild = rawStage.startsWith("cl-"); - const buildVariant = isCLBuild ? "Common Lisp" : "C"; + const isCLBuild = rawStage.startsWith("cl-") || activeBuild.variant === "cl"; + const buildVariant = activeBuild.variant === "both" ? "C + CL" : isCLBuild ? "Common Lisp" : "C"; const logH = buildLogLines.length * (matrixH + 1) + 8; sectionHeader("Building (" + buildVariant + ")", dark ? [20, 28, 20] : [215, 235, 215], dark ? [14, 20, 14] : [228, 240, 228], 120 + logH); -- 2.51.2