@runwisp/ui 1.4.0 → 1.5.1

package/src/lib/components/dashboard/run-helpers.ts

@@ -1,25 +1,17 @@
 // SPDX-FileCopyrightText: PoppyCake, s.r.o.
 // SPDX-License-Identifier: Apache-2.0
 
-import type { Run, RunStatus } from "@runwisp/common";
-import { formatDuration } from "../../utils/format.js";
+import type { APIPaths, Run, RunStatus, Trigger } from "@runwisp/common";
+import { formatBytes, formatDuration } from "../../utils/format.js";
 
 export interface RunVerdict {
     /** Verb phrase for the outcome, ending in its preposition when `timed`. */
     verb: string;
-    /**
-     * Whether the phrase expects a duration after it. False for statuses that
-     * never produced one (nothing ran, or nothing has yet), so the caller
-     * renders the verb alone rather than "skipped after —".
-     */
+    /** Whether a duration follows the verb; false when nothing ran yet. */
     timed: boolean;
 }
 
-/**
- * The run's outcome as a verb phrase, so a detail view can state it as one
- * sentence ("succeeded in 933ms") instead of a status badge competing with a
- * separate duration readout for the same glance.
- */
+// The run's outcome as a verb phrase, read as one sentence: "succeeded in 933ms".
 const RUN_VERDICTS: Record<RunStatus, RunVerdict> = {
     succeeded: { verb: "succeeded in", timed: true },
     failed: { verb: "failed after", timed: true },
@@ -54,12 +46,8 @@ export function runDuration(
 }
 
 /**
- * Gap between when a run was scheduled (`createdAt`, the cron tick) and when
- * it actually started (`startedAt`), formatted — or undefined when the two are
- * within a second of each other. This is the visible face of `jitter`: a
- * jittered run is created at its tick but starts later inside the window, and
- * this is by how much. It also surfaces queue-wait, since a queued run is
- * created when it joins the line and started when the line clears.
+ * How long a run waited between `createdAt` and `startedAt` (jitter or queue
+ * wait), or undefined when it started within a second.
  */
 export function runStartDelay(run: Pick<Run, "createdAt" | "startedAt">): string | undefined {
     if (!run.startedAt) return undefined;
@@ -68,14 +56,7 @@ export function runStartDelay(run: Pick<Run, "createdAt" | "startedAt">): string
     return formatDuration(delay);
 }
 
-/**
- * Display suffix for a run's instance slot. A service configured with more than
- * one instance gets a 1-based suffix (`#1`, `#2`, …) on every one of its runs;
- * a single-instance task (or any non-service, where `instanceCount` is 1)
- * returns an empty string so the bare task name is shown. `instanceIndex` is
- * the stored 0-based slot; `instanceCount` is the task's currently configured
- * instance count.
- */
+/** 1-based `#N` suffix for a multi-instance service run, else "". */
 export function instanceSuffix(instanceIndex: number, instanceCount: number): string {
     if (instanceCount > 1) {
         return `#${String(instanceIndex + 1)}`;
@@ -83,23 +64,32 @@ export function instanceSuffix(instanceIndex: number, instanceCount: number): st
     return "";
 }
 
+// Typed against the generated paths so a route rename fails the build.
+const RAW_LOG_PATH: keyof APIPaths = "/api/runs/{runId}/log/raw";
+
+/** The full log of a run as one plain-text download. */
+export function runLogDownloadUrl(runId: string): string {
+    return RAW_LOG_PATH.replace("{runId}", encodeURIComponent(runId));
+}
+
+/** Labels per trigger source: `short` for the row badge, `long` for descriptions. */
+export const TRIGGER_LABELS: Record<Trigger, { short: string; long: string }> = {
+    cron: { short: "Cron", long: "Scheduled (cron)" },
+    api: { short: "API", long: "REST API" },
+    ui: { short: "UI", long: "UI" },
+    cli: { short: "CLI", long: "CLI" },
+    station: { short: "Station", long: "Control plane" },
+    service: { short: "Service", long: "Service auto-start" },
+    startup: { short: "Startup", long: "On daemon start" },
+    hook: { short: "Hook", long: "Hook" },
+};
+
 /** Human label for why a run fired (the `triggeredBy` source). */
 export function formatTriggeredByLabel(triggeredBy: Run["triggeredBy"]): string {
-    if (triggeredBy === "api") return "API";
-    if (triggeredBy === "ui") return "UI";
-    if (triggeredBy === "cli") return "CLI";
-    if (triggeredBy === "cron") return "Cron";
-    if (triggeredBy === "service") return "Service";
-    if (triggeredBy === "startup") return "Startup";
-    if (triggeredBy === "hook") return "Hook";
-    return "Station";
+    return TRIGGER_LABELS[triggeredBy].short;
 }
 
-/**
- * "retry #N" label when a run is a retry of an earlier one, else undefined.
- * A run is a retry if it carries a positive attempt number or points back at
- * the run it re-attempts.
- */
+/** "retry #N" when the run re-attempts an earlier one, else undefined. */
 export function runRetryLabel(run: Pick<Run, "retryAttempt" | "retryOfRunId">): string | undefined {
     if (run.retryAttempt > 0 || run.retryOfRunId) {
         return `retry #${String(run.retryAttempt)}`;
@@ -107,6 +97,17 @@ export function runRetryLabel(run: Pick<Run, "retryAttempt" | "retryOfRunId">):
     return undefined;
 }
 
+/**
+ * "peak 48 MB · CPU 1s" for a run whose resources were measured (shell runs
+ * only), else undefined.
+ */
+export function runUsageLabel(run: Pick<Run, "peakMemoryBytes" | "cpuTimeMs">): string | undefined {
+    const parts: string[] = [];
+    if (run.peakMemoryBytes !== undefined) parts.push("peak " + formatBytes(run.peakMemoryBytes));
+    if (run.cpuTimeMs !== undefined) parts.push("CPU " + formatDuration(run.cpuTimeMs));
+    return parts.length > 0 ? parts.join(" · ") : undefined;
+}
+
 /**
  * Monotonic position of a run's status in its lifecycle. Used to reject stale
  * updates (e.g. a `pending` HTTP response arriving after an SSE already

package/src/lib/components/dashboard/status-config.ts

@@ -16,25 +16,44 @@ import {
     HeartCrack,
 } from "@lucide/svelte";
 import type { Component } from "svelte";
-import { displayStatus, type RunStatus, type Run } from "@runwisp/common";
+import type { RunStatus } from "@runwisp/common";
 
-export interface RunStatusConfig {
+interface RunStatusConfig {
     icon: Component;
     color: string;
     bg: string;
-    border: string;
     dot: string;
     badge: string;
     /** One-sentence explanation of what this status means, for tooltips. */
     description: string;
 }
 
+// Full literal class strings (not built from a tone name) so Tailwind's
+// scanner picks them up.
+const DANGER = {
+    color: "text-danger-surface",
+    bg: "bg-danger-soft",
+    dot: "bg-danger-surface",
+    badge: "bg-danger-soft text-danger-soft-text",
+};
+const WARNING = {
+    color: "text-warning-surface",
+    bg: "bg-warning-soft",
+    dot: "bg-warning-surface",
+    badge: "bg-warning-soft text-warning-soft-text",
+};
+const NEUTRAL = {
+    color: "text-on-surface-muted",
+    bg: "bg-surface-sunken",
+    dot: "bg-on-surface-faint",
+    badge: "bg-surface-sunken text-on-surface",
+};
+
 export const RUN_STATUS_CONFIG: Record<RunStatus, RunStatusConfig> = {
     running: {
         icon: LoaderCircle,
         color: "text-info-surface",
         bg: "bg-info-soft",
-        border: "border-info-soft",
         dot: "bg-info-surface animate-pulse",
         badge: "bg-info-soft text-info-soft-text",
         description: "This run is executing right now.",
@@ -43,151 +62,87 @@ export const RUN_STATUS_CONFIG: Record<RunStatus, RunStatusConfig> = {
         icon: CircleCheck,
         color: "text-success-surface",
         bg: "bg-success-soft",
-        border: "border-success-soft",
         dot: "bg-success-surface",
         badge: "bg-success-soft text-success-soft-text",
         description: "The run finished with exit code 0 - everything OK.",
     },
     failed: {
+        ...DANGER,
         icon: CircleX,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description: "The run exited with a non-zero code.",
     },
     crashed: {
+        ...DANGER,
         icon: CircleAlert,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description:
             "The process was killed, or the daemon found it still 'running' after a hard crash and marked it crashed. It was not resumed.",
     },
     stopped: {
+        ...WARNING,
         icon: CircleStop,
-        color: "text-warning-surface",
-        bg: "bg-warning-soft",
-        border: "border-warning-soft",
-        dot: "bg-warning-surface",
-        badge: "bg-warning-soft text-warning-soft-text",
         description: "A human or an external script manually stopped this run before it finished.",
     },
     timeout: {
+        ...WARNING,
         icon: TimerOff,
-        color: "text-warning-surface",
-        bg: "bg-warning-soft",
-        border: "border-warning-soft",
-        dot: "bg-warning-surface",
-        badge: "bg-warning-soft text-warning-soft-text",
         description:
             "The run exceeded its configured timeout and was terminated. Timeout duration can be changed.",
     },
     skipped: {
+        ...NEUTRAL,
         icon: SkipForward,
-        color: "text-on-surface-muted",
-        bg: "bg-surface-sunken",
-        border: "border-outline-faint",
-        dot: "bg-on-surface-faint",
-        badge: "bg-surface-sunken text-on-surface",
         description:
             'Skipped by the concurrency policy (on_overlap = "skip") because a previous run was still going.',
     },
     log_overflow: {
+        ...DANGER,
         icon: FileExclamationPoint,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description: "The run hit log_max_size and was handled because of log_on_full.",
     },
     queue_full: {
+        ...WARNING,
         icon: SkipForward,
-        color: "text-warning-surface",
-        bg: "bg-warning-soft",
-        border: "border-warning-soft",
-        dot: "bg-warning-surface",
-        badge: "bg-warning-soft text-warning-soft-text",
         description: "Skipped because the task's queue was already at max_queued.",
     },
     dst_skipped: {
+        ...NEUTRAL,
         icon: SkipForward,
-        color: "text-on-surface-muted",
-        bg: "bg-surface-sunken",
-        border: "border-outline-faint",
-        dot: "bg-on-surface-faint",
-        badge: "bg-surface-sunken text-on-surface",
         description:
             "Skipped: this cron tick was the duplicate half of a DST fall-back, so it was recorded but not run.",
     },
     daemon_stopped: {
+        ...WARNING,
         icon: CircleStop,
-        color: "text-warning-surface",
-        bg: "bg-warning-soft",
-        border: "border-warning-soft",
-        dot: "bg-warning-surface",
-        badge: "bg-warning-soft text-warning-soft-text",
         description:
             "The daemon shut down while this run was in flight and it exceeded shutdown_timeout; it was not resumed.",
     },
     missed: {
+        ...DANGER,
         icon: CalendarX,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description:
             "A scheduled run never happened because the daemon was down. Detected and recorded on restart.",
     },
     start_failed: {
+        ...DANGER,
         icon: OctagonX,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description:
             "The run (or service instance) kept failing; after restart_attempts consecutive failures, RunWisp gave up restarting it automatically.",
     },
     unhealthy: {
+        ...DANGER,
         icon: HeartCrack,
-        color: "text-danger-surface",
-        bg: "bg-danger-soft",
-        border: "border-danger-soft",
-        dot: "bg-danger-surface",
-        badge: "bg-danger-soft text-danger-soft-text",
         description:
             "The service instance kept failing its health_check, so RunWisp stopped it. Its restart policy decides what happens next.",
     },
     pending: {
+        ...NEUTRAL,
         icon: Clock,
-        color: "text-on-surface-muted",
-        bg: "bg-surface-sunken",
-        border: "border-outline",
-        dot: "bg-on-surface-faint",
-        badge: "bg-surface-sunken text-on-surface",
         description: "Queued and waiting to start.",
     },
     ended: {
-        icon: CircleDashed,
+        ...NEUTRAL,
         color: "text-on-surface-faint",
-        bg: "bg-surface-sunken",
-        border: "border-outline-faint",
-        dot: "bg-on-surface-faint",
-        badge: "bg-surface-sunken text-on-surface",
+        icon: CircleDashed,
         description: "The run finished. No specific end reason was recorded.",
     },
 };
-
-export function getRunStatusConfig(status: RunStatus): RunStatusConfig {
-    return RUN_STATUS_CONFIG[status];
-}
-
-export function runDisplayStatus(run: Pick<Run, "status" | "endReason">): RunStatus {
-    return displayStatus(run.status, run.endReason);
-}

package/src/lib/components/dashboard/types.ts

@@ -1,32 +1,6 @@
 // SPDX-FileCopyrightText: PoppyCake, s.r.o.
 // SPDX-License-Identifier: Apache-2.0
 
-import type { Task as CommonTask, Run as CommonRun } from "@runwisp/common";
-
-/**
- * Web UI component types.
- * Source of truth for status values: {@link RunStatus} from `@runwisp/common`.
- */
-
-export type Task = CommonTask;
-
-export type Run = CommonRun;
-
-export interface DaemonState {
-    name: string;
-    version: string;
-    uptime: string;
-    status: "connected" | "disconnected";
-    host: string;
-    backendUrl: string;
-    cpus: number;
-    memory: string;
-    os: string;
-    arch: string;
-    workDir: string;
-    fingerprint: string;
-}
-
 export interface DaemonStats {
     cpuUsage: number;
     memUsage: number;

package/src/lib/components/data-grid-selection.ts

@@ -1,14 +1,6 @@
 // SPDX-FileCopyrightText: PoppyCake, s.r.o.
 // SPDX-License-Identifier: Apache-2.0
 
-// DataGrid's header checkbox (checked / indeterminate / unchecked) must
-// reflect which rows on the current page are ACTUALLY selected, not merely
-// how many rows are selected. A bare count comparison is wrong the moment
-// `selectedRows` holds a different set of rows than `pagedRows` but happens
-// to be the same size — e.g. rows selected on a page the grid has since
-// paged/filtered away from. Extracted so the membership check is unit
-// testable without a component-render harness.
-
 export interface SelectionState {
     allSelected: boolean;
     someSelected: boolean;
@@ -17,7 +9,8 @@ export interface SelectionState {
 /**
  * Header checkbox state for `pagedRows`: fully checked only when every row
  * currently on the page is selected, indeterminate when some (but not all)
- * of them are. Matches rows by `rowKey`, not by array position or count.
+ * of them are. Matches rows by `rowKey`, not by count: the selection can
+ * hold rows from a page the grid has since moved away from.
  */
 export function selectionState<T>(
     pagedRows: readonly T[],

package/src/lib/components/filter-spec.ts

@@ -3,7 +3,7 @@
 
 // Declarative filter spec. A `FilterField[]` + one bound `FilterValues` object
 // drives <FilterBar>, the client engine (`applyFilters`), and the URL sync
-// helper — the same way DataGrid's `Column[]` drives the table. Add a field
+// helper, the same way DataGrid's `Column[]` drives the table. Add a field
 // type here, and every table that adopts it renders identically.
 
 import type { SelectOption } from "./Select.svelte";
@@ -44,7 +44,7 @@ export type FilterField =
           primary?: boolean;
       };
 
-// All values are strings — DOM inputs are string-valued and URL params are
+// All values are strings, DOM inputs are string-valued and URL params are
 // strings; the client engine coerces (e.g. Number()) where a field needs it.
 export type FilterValues = Record<string, string>;
 
@@ -65,7 +65,7 @@ export function isBlank(v: string | undefined): boolean {
     return !v || v === "all";
 }
 
-// Which value keys a field owns — daterange owns two, everything else one.
+// Which value keys a field owns, daterange owns two, everything else one.
 // Used by chips, active-state detection, clear, and URL (de)serialization so
 // none of them has to special-case the range type.
 export function fieldKeys(field: FilterField): string[] {

package/src/lib/components/radio-group.ts

@@ -0,0 +1,11 @@
+// SPDX-FileCopyrightText: PoppyCake, s.r.o.
+// SPDX-License-Identifier: Apache-2.0
+
+export const RADIO_GROUP_KEY = Symbol("radio-group");
+
+// What a RadioGroup hands its Radio children: the shared input name and the
+// selected value, read and written through the group's bindable `value`.
+export interface RadioGroupContext {
+    readonly name: string;
+    value: string;
+}

package/src/lib/index.ts

@@ -1,8 +1,6 @@
 // SPDX-FileCopyrightText: PoppyCake, s.r.o.
 // SPDX-License-Identifier: Apache-2.0
 
-export { default as theme } from "./theme.css?inline";
-
 export { default as Logo } from "./components/Logo.svelte";
 
 export { default as PageContainer } from "./components/PageContainer.svelte";
@@ -19,13 +17,6 @@ export { default as Spinner } from "./components/Spinner.svelte";
 
 export { default as Button } from "./components/Button.svelte";
 export { default as LinkButton } from "./components/LinkButton.svelte";
-export {
-    BUTTON_BASE,
-    BUTTON_SIZES,
-    BUTTON_VARIANTS,
-    type ButtonSize,
-    type ButtonVariant,
-} from "./components/button-styles.js";
 export { default as Checkbox } from "./components/Checkbox.svelte";
 export { default as CronInput } from "./components/CronInput.svelte";
 export { default as DurationInput } from "./components/DurationInput.svelte";
@@ -33,7 +24,7 @@ export { default as FormField } from "./components/FormField.svelte";
 export { default as Input } from "./components/Input.svelte";
 export { default as Radio } from "./components/Radio.svelte";
 export { default as RadioGroup } from "./components/RadioGroup.svelte";
-export { default as Select, type SelectOption } from "./components/Select.svelte";
+export { default as Select } from "./components/Select.svelte";
 export { default as Textarea } from "./components/Textarea.svelte";
 export { default as TimezoneSelect } from "./components/TimezoneSelect.svelte";
 export { default as Toggle } from "./components/Toggle.svelte";
@@ -88,71 +79,45 @@ export { default as ThemeToggle } from "./components/ThemeToggle.svelte";
 
 export { default as CodeBlock } from "./components/CodeBlock.svelte";
 export { default as TaskCard } from "./components/TaskCard.svelte";
-export type { TaskCardAccent } from "./components/TaskCard.svelte";
 
 export { default as LogConsole } from "./components/LogConsole.svelte";
 export type { FetchLogsFn, LogEvent, LogSlice } from "./log-console/types.js";
-export { isLogEvent } from "./log-console/types.js";
-export { LogCache } from "./log-console/LogCache.svelte.js";
-export { LogFetcher } from "./log-console/LogFetcher.svelte.js";
 
 export { default as RunDetailPanel } from "./components/dashboard/RunDetailPanel.svelte";
 export { default as RunsList } from "./components/dashboard/RunsList.svelte";
 export { default as RunFilterPopover } from "./components/dashboard/RunFilterPopover.svelte";
 export type { RunOutputMatch } from "./components/dashboard/RunsList.svelte";
 export {
+    activeFilterCount,
     emptyRunFilters,
     FAILURE_STATUS_TOKEN,
-    NEEDS_ATTENTION_STATUSES,
-    isNeedsAttention,
-    activeFilterCount,
-    activeDimensions,
-    dimensionActive,
-    clearDimension,
-    clearPopoverFilters,
     humanizeStatus,
-    statusChipLabel,
     exitCodeRange,
     type ExitCodeRange,
     type RunsListFilters,
-    type RunsListSortDirection,
-    type FilterDimension,
 } from "./components/dashboard/run-filters.js";
-export {
-    getRunStatusConfig,
-    runDisplayStatus,
-    RUN_STATUS_CONFIG,
-    type RunStatusConfig,
-} from "./components/dashboard/status-config.js";
+export { RUN_STATUS_CONFIG } from "./components/dashboard/status-config.js";
 export {
     runDuration,
-    runStartDelay,
     runPhaseOrder,
     formatTriggeredByLabel,
-    runRetryLabel,
     instanceSuffix,
 } from "./components/dashboard/run-helpers.js";
-export { humanizeCron, type HumanizedCron } from "./components/dashboard/cron-format.js";
-export type { DaemonState, DaemonStats } from "./components/dashboard/types.js";
+export { humanizeCron } from "./components/dashboard/cron-format.js";
+export type { DaemonStats } from "./components/dashboard/types.js";
 
-export { toast, type Toast, type ToastType } from "./utils/toast.svelte.js";
-export {
-    themeStore,
-    THEME_STORAGE_KEY,
-    type ThemePreference,
-    type ResolvedTheme,
-} from "./utils/theme.svelte.js";
+export { toast } from "./utils/toast.svelte.js";
 export { extractErrorMessage } from "./utils/error.js";
 export {
     formatBytes,
     formatRelativeTime,
     formatRelativeTimeWithAbsolute,
-    formatDateTime,
     formatDuration,
     formatFullDateTime,
 } from "./utils/format.js";
 export { formatShortId } from "./utils/id.js";
 export { debounce } from "./utils/debounce.js";
 export { TickingNow } from "./utils/ticking-now.svelte.js";
+export { CopyFeedback, copyText } from "./utils/clipboard.svelte.js";
 export { RunMotion } from "./utils/run-motion.js";
-export { arrival, leave, shift, prefersReducedMotion } from "./actions/row-motion.js";
+export { arrival, leave, shift } from "./actions/row-motion.js";

package/src/lib/log-console/LogCache.svelte.ts

@@ -14,7 +14,7 @@ import { visibleColumns } from "./ansi.js";
  */
 // Streams pinned to the front of the overlay, in render order. The companion
 // Set backs the membership test in `overlayRows`. Both are module-level,
-// immutable, and non-reactive — a fixed lookup table, not component state.
+// immutable, and non-reactive, a fixed lookup table, not component state.
 const PREFERRED_STREAM_ORDER = ["stdout", "stderr"] as const;
 const PREFERRED_STREAMS: ReadonlySet<string> = new Set(PREFERRED_STREAM_ORDER);
 

package/src/lib/log-console/LogFetcher.svelte.ts

@@ -1,13 +1,12 @@
 // SPDX-FileCopyrightText: PoppyCake, s.r.o.
 // SPDX-License-Identifier: Apache-2.0
 
-import { createLoggerFactory } from "@runwisp/common";
+import { createLogger } from "@runwisp/common";
 import { SvelteSet } from "svelte/reactivity";
 import type { LogCache } from "./LogCache.svelte.js";
 import type { FetchLogsFn } from "./types.js";
 import { isLogEvent } from "./types.js";
 
-const createLogger = createLoggerFactory();
 const logger = createLogger("LogFetcher");
 
 /**
@@ -20,8 +19,8 @@ const logger = createLogger("LogFetcher");
  */
 export class LogFetcher {
     private readonly cache: LogCache;
-    private fetchLogsFn: FetchLogsFn | undefined;
-    private chunkSize: number;
+    private readonly fetchLogsFn: FetchLogsFn | undefined;
+    private readonly chunkSize = 4096;
 
     readonly MAX_INFLIGHT = 2;
     readonly MIN_REQUEST_INTERVAL_MS = 140;
@@ -38,23 +37,13 @@ export class LogFetcher {
     constructor(
         cache: LogCache,
         fetchLogsFn: FetchLogsFn | undefined,
-        chunkSize: number = 4096,
         onDataLoaded?: (min: number, max: number) => void,
     ) {
         this.cache = cache;
         this.fetchLogsFn = fetchLogsFn;
-        this.chunkSize = Math.max(1, Math.floor(chunkSize));
         this.onDataLoaded = onDataLoaded;
     }
 
-    setFetchLogsFn(fn: FetchLogsFn | undefined) {
-        this.fetchLogsFn = fn;
-    }
-
-    setChunkSize(size: number) {
-        this.chunkSize = Math.max(1, Math.floor(size));
-    }
-
     enqueue(from: number, to: number) {
         if (!this.fetchLogsFn) return;
 

package/src/lib/log-console/ansi.ts

@@ -45,7 +45,7 @@ const ANSI_ESCAPE = /\x1b\[[0-9;?]*[A-Za-z]/g;
 const TAB_WIDTH = 8;
 
 // Count the visible columns a log line occupies once ANSI escape sequences
-// are stripped. Tabs are charged the full tab width — an over-estimate that
+// are stripped. Tabs are charged the full tab width, an over-estimate that
 // only adds trailing slack to the horizontal scroll surface, never clips.
 export function visibleColumns(text: string): number {
     let columns = 0;