Skip to content

fix: restore Apple Calendar permission flow#3888

Merged
louis030195 merged 1 commit into
screenpipe:mainfrom
divanshu-go:fix/apple-calendar-connection
Jun 6, 2026
Merged

fix: restore Apple Calendar permission flow#3888
louis030195 merged 1 commit into
screenpipe:mainfrom
divanshu-go:fix/apple-calendar-connection

Conversation

@divanshu-go

@divanshu-go divanshu-go commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Description

  • route Calendar through the generic permission commands (checkPermission, requestPermission, resetPermission) so Connect requests access and Revoke resets TCC

  • update EventKit handling to require full read access, reset event stores after grant/revoke.

  • fix Calendar status detection, TCC reset service, and automatic UI refresh after approval

Before

Screenshot 2026-06-06 at 1 59 16 PM

After

Grant permission

image
Screen.Recording.2026-06-07.at.1.59.16.AM.mov

Revoke permission

image
Screen.Recording.2026-06-07.at.1.58.58.AM.mov

Closes #3434

@divanshu-go divanshu-go changed the title fix(app): restore Apple Calendar permission flow fix: restore Apple Calendar permission flow Jun 6, 2026
@divanshu-go divanshu-go force-pushed the fix/apple-calendar-connection branch from edf78b9 to 38a96e8 Compare June 6, 2026 20:44
@louis030195 louis030195 merged commit bdb7417 into screenpipe:main Jun 6, 2026
10 of 12 checks passed
louis030195 pushed a commit that referenced this pull request Jun 6, 2026
Restores the Apple Calendar permission flow (#3888) and makes
current_sync_streams available in enterprise builds.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
louis030195 pushed a commit that referenced this pull request Jun 6, 2026
The eventkit-rs crate is imported as `eventkit` in calendar.rs and
permissions.rs (macOS Apple Calendar flow, added in #3888), but
cargo-machete matches by the normalized package name `eventkit_rs`
so it can't see the use and flags it as unused, reding the
"Dependency & Performance Checks" job on main.

Add it to the cargo-machete ignore list, same as the existing
screenpipe-redact / thiserror precedent.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] apple calendar works in dev mode but not on prod build

2 participants