[saphid]

Problem

On mobile, Settings > Scheduled tasks labels each task with its next run. When a run later today is between 59m30s and 60m away, the label reads "Next run in 60 minutes" instead of "Next run in 1 hour". The screen refreshes its clock every minute from Date.now(), so this shows up regularly in the last minute or so before a task is an hour away.

formatNextScheduledTaskRun picks "minute" or "hour" from the unrounded time left, then rounds the count, so 59.5 minutes stays in minutes and rounds to 60. On web, the same row already says "in 1h" at this point.

Change

The function now rounds to minutes first and only then picks the unit, so 59.5 minutes moves up to "1 hour". This is the same approach formatDuration in packages/shared/src/orchestrationTiming.ts uses ("Choose the unit after rounding so 59.5 seconds never renders as '60s'"). Hour rounding and every other branch work as before.

Scope and approval

Very small, focused fix for an obvious bug, so no prior issue: one helper in one mobile file chose its unit before rounding, which a shared sibling already guards against. It changes 4 non-test lines, with no behavior or product change outside that boundary.

Overlap search: the local PR index plus REST searches for "next run" mobile and scheduledTaskPresentation. The closest PRs are #26436, #23911, #23111, #26001 and #25930, which change scheduled-task features and labels but not this rounding. #17627 (closed) rewrote interval formatting, not the next-run label.

Verification

• Red on main: vp test run src/features/settings/scheduledTaskPresentation.test.ts (in apps/mobile) failed with AssertionError: expected 'Next run in 60 minutes' to be 'Next run in 1 hour'.
• Green: the same command, 1 test passed. The test pins a fixed local now and checks 59 min → "59 minutes", 59.5 min → "1 hour" and 150 min → "3 hours". The module had no test file, so this one is new.
• vp run check:changed: passed (format, lint, knip, typecheck, tests, React Compiler).
• Not checked: I did not capture the screen in a running mobile client. The change is limited to the string this pure function returns.

Reviewed with sol-loop: 1 round with GPT-6.1 Sol on high. Sol verdict: approved.

Created with Claude Opus 5.5 in Claude Code (T3 Code).

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at cf0679f

Macroscope's review found this PR approvable — This changes the mobile next-run label to use “1 hour” when rounded minutes reach 60. It affects display text only, not scheduling behavior or product defaults.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The next-run formatter now selects minutes or hours based on the rounded minute count. Tests cover durations of 59, 59.5, and 150 minutes.

Changes: Scheduled task formatting

Layer / File(s)	Summary
Rounded time display apps/mobile/src/features/settings/scheduledTaskPresentation.ts, apps/mobile/src/features/settings/scheduledTaskPresentation.test.ts	The formatter selects its display unit from the rounded minute count. Tests check the output for 59, 59.5, and 150 minutes.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Merge Risk: Minimal · up to cf067

Scheduled tasks due within the hour now show "1 hour" instead of "60 minutes". The change is small and covered by tests, so merge risk is minimal.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title names the main change: scheduled tasks due within the hour no longer show "Next run in 60 minutes". It follows the conventional commit form and matches the raw summary and objectives, which …
Description check		The description covers the problem, the change, scope and approval, and verification. It explains that the unit was chosen before rounding, so 59.5 minutes rendered as 60 minutes. It states the change…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title names the main change: scheduled tasks due within the hour no longer show "Next run in 60 minutes". It follows the conventional commit form and matches the raw summary and objectives, which describe the rounding-before-unit fix in formatNextScheduledTaskRun.

Full details: Description check

Explanation

The description covers the problem, the change, scope and approval, and verification. It explains that the unit was chosen before rounding, so 59.5 minutes rendered as 60 minutes. It states the change and cites a shared helper with the same approach. It gives a red-then-green test run, a check:changed run, and a note on what was not checked. It follows the template's required sections, and the template's guidance on scope and evidence is addressed.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 0487654
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5d892687-c639-496e-8a51-7c2027cc2e6a

Commits

Reviewing files that changed from the base of the PR and between e63f39a and cf0679f.

Files selected for processing: 1

• apps/mobile/src/features/settings/scheduledTaskPresentation.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 0 remain after this review.

[saphid]

Review requested

Date (UTC)	Reviewer	Where
2026-11-13	Julius	Discord DM

_{Logged so this PR shows when a maintainer was asked to review it.}

Dismissing prior approval to re-evaluate cf0679f

[juliusmarminge]

Evidence

Evidence captured in the real Android app on the same Pixel 9, Android 36, 1080×2424, light theme, with separate fresh servers and synthetic data.

On each commit, create 'Hourly check 2' with prompt 'Say hi', Interval = 60 minutes, Enabled on. Immediately return to Settings and reopen Scheduled tasks to refresh the screen clock.

Before	After
0487654fc8	cf0679f918
Before recording	After recording

Project seed script and reproduction steps. Attempted iOS scenario; the verdict above comes from the manual Android captures.

Not shown:

• iOS was not proven: scenario attempts stopped during dev-client installation or video startup before reaching the reproduction. Android demonstrates the changed mobile formatting behavior.