feat: add --alarm flag for EKAlarm support in add/edit commands#39
Closed
judeyang wants to merge 1 commit into
Closed
feat: add --alarm flag for EKAlarm support in add/edit commands#39judeyang wants to merge 1 commit into
judeyang wants to merge 1 commit into
Conversation
- Add alarmDate field to ReminderInput, ReminderDraft, and ReminderUpdate - CreateReminder: auto-add EKAlarm (uses alarmDate if set, falls back to dueDate) - UpdateReminder: support updating/clearing alarms - AddCommand: add --alarm/-a option for setting alarm date - EditCommand: add --alarm/-a and --clear-alarm options This enables reminders to trigger native iOS/macOS notifications. Without this, reminders created via CLI have due dates but no alarm triggers, so they never pop up as notifications on devices.
Author
|
就是说吧，我特么的也不知道 ai 给我改了些啥。 |
|
I was running into this issue and I can confirm this PR solves it for me. I pulled and built this branch and reminders do trigger notifications now ❤️ |
bobsahur-robot
pushed a commit
to raymondlee-personal-org/remindctl
that referenced
this pull request
Mar 29, 2026
…nds, and output - AddCommand: --repeat, --alarm, --url, --parent, --tags flags - EditCommand: same + --clear-alarm, --clear-repeat, --clear-url - EventKitStore: full read/write for all new fields via EKReminder - OutputFormatting: display recurrence, alarm, url, tags in text + JSON - Cherry-picked model extensions from upstream PRs openclaw#40, openclaw#39, openclaw#17, openclaw#35, openclaw#13
jakubswierczek
added a commit
to jakubswierczek/remindctl
that referenced
this pull request
Mar 30, 2026
…claw#39) add: --alarm sets alarm, defaults to due date if not specified edit: --alarm sets, --clear-alarm removes alarm alarms use EKAlarm absoluteDate for native iOS/macOS notifications Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
hezhongtang
added a commit
to hezhongtang/remindctl
that referenced
this pull request
Apr 17, 2026
Merged 9 PRs from steipete/remindctl: - openclaw#29 lint fix, openclaw#26 open filter, openclaw#35 tags support - openclaw#30 all-day/date-only reminders, openclaw#40 recurrence rules - openclaw#39 alarm support, openclaw#27 reminder sections - openclaw#44 isFlagged support, openclaw#13 creationDate in output
Collaborator
|
Landed the EventKit alarm support on main in b7f1ac9, with the implementation adjusted to the current ParsedUserDate/all-day behavior and tests/docs/changelog added. What shipped:
Verified with |
Closed
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
--alarm/-aflag toaddandeditcommands, enabling native iOS/macOS notification triggers viaEKAlarm.Problem
Reminders created via
remindctl add --due <date>set a due date but no alarm trigger. This means they show up in the Reminders app but never pop up as notifications on iPhone/iPad/Mac.Solution
alarmDatefield toReminderInput,ReminderDraft, andReminderUpdatecreateReminder: auto-addEKAlarm(usesalarmDateif set, falls back todueDate)updateReminder: support updating/clearing alarmsAddCommand: add--alarm/-aoptionEditCommand: add--alarm/-aand--clear-alarmoptionsUsage
Changes
Sources/RemindCore/Models.swift— addedalarmDatetoReminderInputandReminderUpdateSources/RemindCore/EventKitStore.swift—EKAlarmcreation in create/update flowsSources/remindctl/Commands/AddCommand.swift—--alarm/-aoptionSources/remindctl/Commands/EditCommand.swift—--alarm/-aand--clear-alarmoptionsTested on macOS 26.3.1 (Tahoe), reminders sync correctly to iPhone/iPad with alarm triggers.