slopnode/bot
Watch
2
0
Fork
You've already forked bot
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Point !time users at !tz when they pass a zone #3

Merged
drawblank merged 1 commit from fix/time-nudge into main 2026-08-20 09:42:26 +00:00
Conversation 0 Commits 1 Files changed 2 +40 -6
drawblank commented 2026-08-20 09:42:17 +00:00
Owner
Copy link

Follow-up to #2.

!time became a fixed UTC alias, so !time Europe/Amsterdam silently returned UTC with no hint that the argument was ignored. That was the one change in the refactor a regular user could trip over.

<alice> !time Europe/Amsterdam
  #main    UTC 2026-08-20 09:41:54 [+00:00]
  ->alice  !time is always UTC; try: !tz Europe/Amsterdam

The room still sees only the UTC line; the nudge goes privately to the caller, matching how usage errors are already handled. !time with no argument is unchanged, and !time -h is still intercepted before the handler.

Verification

309 tests (up from 304). One of the new tests takes the suggested command out of the nudge text and dispatches it, so the suggestion is proven to actually work rather than just proven to be a string.

Mutation-checked:

Mutation Tests failed
Nudge removed entirely 4
Nudge sent to the channel instead of privately 3
Follow-up to #2. `!time` became a fixed UTC alias, so `!time Europe/Amsterdam` silently returned UTC with no hint that the argument was ignored. That was the one change in the refactor a regular user could trip over. ``` <alice> !time Europe/Amsterdam #main UTC 2026-08-20 09:41:54 [+00:00] ->alice !time is always UTC; try: !tz Europe/Amsterdam ``` The room still sees only the UTC line; the nudge goes privately to the caller, matching how usage errors are already handled. `!time` with no argument is unchanged, and `!time -h` is still intercepted before the handler. ## Verification 309 tests (up from 304). One of the new tests takes the suggested command out of the nudge text and dispatches it, so the suggestion is proven to actually work rather than just proven to be a string. Mutation-checked: | Mutation | Tests failed | |---|---| | Nudge removed entirely | 4 | | Nudge sent to the channel instead of privately | 3 |
!time became a fixed UTC alias, so `!time Europe/Amsterdam` silently
returned UTC with no hint that the argument was ignored -- the one change
in that refactor a regular user could trip over.

The channel still sees only the UTC line; the nudge goes privately to the
caller, matching how usage errors are already handled. A test dispatches
the suggested !tz command to prove the suggestion actually works.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
slopnode/bot!3
No description provided.