From 3bb4d4bf820ad4a5607e832217e5a8180e958095 Mon Sep 17 00:00:00 2001 From: Alex Fallah Date: Mon, 26 Jan 2026 23:45:23 -0500 Subject: [PATCH] fix: make opts parameter optional in parseActiveHoursTime The function signature had an optional raw parameter followed by a required opts parameter, which violates TypeScript rules. Since opts is always provided together with raw in actual usage, make it optional. Co-Authored-By: Claude Haiku 4.5 --- src/infra/heartbeat-runner.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/infra/heartbeat-runner.ts b/src/infra/heartbeat-runner.ts index 012fff229..d8fcad6f3 100644 --- a/src/infra/heartbeat-runner.ts +++ b/src/infra/heartbeat-runner.ts @@ -89,14 +89,14 @@ function resolveActiveHoursTimezone(cfg: ClawdbotConfig, raw?: string): string { } } -function parseActiveHoursTime(raw?: string, opts: { allow24: boolean }): number | null { +function parseActiveHoursTime(raw?: string, opts?: { allow24: boolean }): number | null { if (!raw || !ACTIVE_HOURS_TIME_PATTERN.test(raw)) return null; const [hourStr, minuteStr] = raw.split(":"); const hour = Number(hourStr); const minute = Number(minuteStr); if (!Number.isFinite(hour) || !Number.isFinite(minute)) return null; if (hour === 24) { - if (!opts.allow24 || minute !== 0) return null; + if (!(opts?.allow24 ?? false) || minute !== 0) return null; return 24 * 60; } return hour * 60 + minute;