From ee6577936d2b1b31a117600ba3b5783825cb00cb Mon Sep 17 00:00:00 2001 From: "opencode-agent[bot]" <219766164+opencode-agent[bot]@users.noreply.github.com> Date: Fri, 31 Jul 2026 20:32:46 -0400 Subject: [PATCH] refactor(tui): remove unused duration formatter (#39964) Co-authored-by: Kit Langton --- packages/tui/src/util/format.ts | 20 --------- packages/tui/test/util/format.test.ts | 59 --------------------------- 2 files changed, 79 deletions(-) delete mode 100644 packages/tui/src/util/format.ts delete mode 100644 packages/tui/test/util/format.test.ts diff --git a/packages/tui/src/util/format.ts b/packages/tui/src/util/format.ts deleted file mode 100644 index 4ae62eac450..00000000000 --- a/packages/tui/src/util/format.ts +++ /dev/null @@ -1,20 +0,0 @@ -export function formatDuration(secs: number) { - if (secs <= 0) return "" - if (secs < 60) return `${secs}s` - if (secs < 3600) { - const mins = Math.floor(secs / 60) - const remaining = secs % 60 - return remaining > 0 ? `${mins}m ${remaining}s` : `${mins}m` - } - if (secs < 86400) { - const hours = Math.floor(secs / 3600) - const remaining = Math.floor((secs % 3600) / 60) - return remaining > 0 ? `${hours}h ${remaining}m` : `${hours}h` - } - if (secs < 604800) { - const days = Math.floor(secs / 86400) - return days === 1 ? "~1 day" : `~${days} days` - } - const weeks = Math.floor(secs / 604800) - return weeks === 1 ? "~1 week" : `~${weeks} weeks` -} diff --git a/packages/tui/test/util/format.test.ts b/packages/tui/test/util/format.test.ts deleted file mode 100644 index 5b346e7f6bc..00000000000 --- a/packages/tui/test/util/format.test.ts +++ /dev/null @@ -1,59 +0,0 @@ -import { describe, expect, test } from "bun:test" -import { formatDuration } from "../../src/util/format" - -describe("util.format", () => { - describe("formatDuration", () => { - test("returns empty string for zero or negative values", () => { - expect(formatDuration(0)).toBe("") - expect(formatDuration(-1)).toBe("") - expect(formatDuration(-100)).toBe("") - }) - - test("formats seconds under a minute", () => { - expect(formatDuration(1)).toBe("1s") - expect(formatDuration(30)).toBe("30s") - expect(formatDuration(59)).toBe("59s") - }) - - test("formats minutes under an hour", () => { - expect(formatDuration(60)).toBe("1m") - expect(formatDuration(61)).toBe("1m 1s") - expect(formatDuration(90)).toBe("1m 30s") - expect(formatDuration(120)).toBe("2m") - expect(formatDuration(330)).toBe("5m 30s") - expect(formatDuration(3599)).toBe("59m 59s") - }) - - test("formats hours under a day", () => { - expect(formatDuration(3600)).toBe("1h") - expect(formatDuration(3660)).toBe("1h 1m") - expect(formatDuration(7200)).toBe("2h") - expect(formatDuration(8100)).toBe("2h 15m") - expect(formatDuration(86399)).toBe("23h 59m") - }) - - test("formats days under a week", () => { - expect(formatDuration(86400)).toBe("~1 day") - expect(formatDuration(172800)).toBe("~2 days") - expect(formatDuration(259200)).toBe("~3 days") - expect(formatDuration(604799)).toBe("~6 days") - }) - - test("formats weeks", () => { - expect(formatDuration(604800)).toBe("~1 week") - expect(formatDuration(1209600)).toBe("~2 weeks") - expect(formatDuration(1609200)).toBe("~2 weeks") - }) - - test("handles boundary values correctly", () => { - expect(formatDuration(59)).toBe("59s") - expect(formatDuration(60)).toBe("1m") - expect(formatDuration(3599)).toBe("59m 59s") - expect(formatDuration(3600)).toBe("1h") - expect(formatDuration(86399)).toBe("23h 59m") - expect(formatDuration(86400)).toBe("~1 day") - expect(formatDuration(604799)).toBe("~6 days") - expect(formatDuration(604800)).toBe("~1 week") - }) - }) -})