From 0386632c176fc997dc0a05961bcd8827064c8ca1 Mon Sep 17 00:00:00 2001 From: Alex Southwell <4596216+saphid@users.noreply.github.com> Date: Tue, 3 Nov 2026 13:12:26 +1100 Subject: [PATCH] fix(web): editing a scheduled task shows its saved time when the hour has one digit Co-Authored-By: Claude Opus 5.5 --- .../settings/scheduledTasksSettings.logic.test.ts | 10 ++++++++++ .../settings/scheduledTasksSettings.logic.ts | 3 ++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/apps/web/src/components/settings/scheduledTasksSettings.logic.test.ts b/apps/web/src/components/settings/scheduledTasksSettings.logic.test.ts index c4c66c1cfbae..5d3240d41c9e 100644 --- a/apps/web/src/components/settings/scheduledTasksSettings.logic.test.ts +++ b/apps/web/src/components/settings/scheduledTasksSettings.logic.test.ts @@ -170,6 +170,16 @@ describe("editing scheduled task branch settings", () => { }); }); +describe("fixed-time scheduled tasks", () => { + it("pads a stored single-digit hour so the time input shows it", () => { + const draft = taskToDraft({ + ...legacyTask, + schedule: { type: "fixed_time", timeOfDay: "9:05" }, + }); + expect(draft.timeOfDay).toBe("09:05"); + }); +}); + describe("one-time scheduled tasks", () => { afterEach(() => vi.unstubAllEnvs()); diff --git a/apps/web/src/components/settings/scheduledTasksSettings.logic.ts b/apps/web/src/components/settings/scheduledTasksSettings.logic.ts index 9e0d0dad27cb..94c7d68e1627 100644 --- a/apps/web/src/components/settings/scheduledTasksSettings.logic.ts +++ b/apps/web/src/components/settings/scheduledTasksSettings.logic.ts @@ -248,7 +248,8 @@ export function taskToDraft(task: ScheduledTask): DraftState { scheduleMode: schedule.type === "fixed_time" ? "fixed" : schedule.type, intervalMinutes: schedule.type === "interval" ? String(Math.max(1, schedule.everyMs / 60_000)) : "15", - timeOfDay: schedule.type === "fixed_time" ? schedule.timeOfDay : "09:00", + // A time input shows nothing for an unpadded hour such as 9:05, which agents may save. + timeOfDay: schedule.type === "fixed_time" ? schedule.timeOfDay.padStart(5, "0") : "09:00", runAt: schedule.type === "once" ? toDateTimeLocalValue(new Date(schedule.runAt)) : "", savedRunAt: schedule.type === "once" ? schedule.runAt : "", weekdays,