Skip to content

feat: surface isFlagged status in remindctl output#44

Closed
psiman-wandera wants to merge 3 commits into
openclaw:mainfrom
psiman-wandera:main
Closed

feat: surface isFlagged status in remindctl output#44
psiman-wandera wants to merge 3 commits into
openclaw:mainfrom
psiman-wandera:main

Conversation

@psiman-wandera
Copy link
Copy Markdown

@psiman-wandera psiman-wandera commented Apr 13, 2026

No description provided.

Copilot AI and others added 3 commits April 13, 2026 11:14
Initial plan
723c518
@psiman-wandera
feat: surface isFlagged status in remindctl output
29ff1a9 
Agent-Logs-Url: https://github.com/psiman-wandera/remindctl/sessions/cd135466-c8dd-44d6-8c52-b63daa994178

Co-authored-by: psiman-wandera <65292131+psiman-wandera@users.noreply.github.com>
@psiman-wandera
Merge pull request #1 from psiman-wandera/copilot/add-flagged-status-…
407ec20 
…to-reminders

[WIP] Add support for flagged status in Apple Reminders CLI output
@psiman-wandera
Copy link
Copy Markdown
Author

psiman-wandera commented Apr 13, 2026

Hi @steipete, I figured it would be helpful to also surface isFlagged in the remindctl output. Here is my take on this feature. Can you please review?

@psiman-wandera psiman-wandera mentioned this pull request Apr 13, 2026
Closed
hezhongtang added a commit to hezhongtang/remindctl that referenced this pull request Apr 17, 2026
@hezhongtang
Merge upstream PRs: tags, recurrence, alarms, sections, flagged, etc
fdcbbdc 
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
@steipete
Copy link
Copy Markdown
Collaborator

steipete commented May 4, 2026

Thanks for the PR. I tried to fold this into current main, but the repo builds against an EventKit SDK where EKReminder does not expose isFlagged. The attempted implementation fails to compile with value of type EKReminder has no member isFlagged, and the local EventKit headers for EKReminder.h do not list a flagged/isFlagged property. Runtime selector checks for isFlagged/flagged also return false here.

Because this would require a non-EventKit/private or AppleScript-backed path rather than a small read-only model/output addition, I am closing this PR as not viable in the current implementation shape.

@steipete steipete closed this May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@psiman-wandera @steipete