feat: add --alarm flag for EKAlarm support in add/edit commands#39
Open
judeyang wants to merge 1 commit intosteipete:mainfrom
Open
feat: add --alarm flag for EKAlarm support in add/edit commands#39judeyang wants to merge 1 commit intosteipete:mainfrom
judeyang wants to merge 1 commit intosteipete:mainfrom
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 steipete#40, steipete#39, steipete#17, steipete#35, steipete#13
jakubswierczek
added a commit
to jakubswierczek/remindctl
that referenced
this pull request
Mar 30, 2026
…pete#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>
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
2 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.