feat: Recurrence state and edit scopes - #160
Open
benjaminVadon wants to merge 3 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Public occurrence-level scopes are exposed without scope-aware update or deletion operations capable of honoring them.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds recurrence-state metadata and user-facing recurrence edit scopes to calendar events.
Changes:
- Derives standalone, master, and occurrence states.
- Adds recurrence edit/delete scope options.
- Centralizes recurrence-set detection and adds tests.
File summaries
| File | Description |
|---|---|
RecurrenceEditScopesTest.kt |
Tests scope selection rules. |
EventRecurrenceStateTest.kt |
Tests recurrence-state derivation. |
RecurringEventExpansion.kt |
Uses shared recurrence detection. |
RecurrenceSetPresence.kt |
Centralizes recurrence-set detection. |
RecurrenceEditScopes.kt |
Computes available deletion scopes. |
RecurrenceEditScope.kt |
Defines recurrence operation scopes. |
EventRecurrenceState.kt |
Defines event recurrence states. |
Event.kt |
Exposes derived recurrence metadata. |
RemoteDavEventToEntity.kt |
Uses shared recurrence detection. |
RecurrenceRuleToBoundsEntity.kt |
Uses shared recurrence detection for bounds. |
Review details
- Files reviewed: 10/10 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
benjaminVadon
force-pushed
the
recurrence-state-and-edit-scopes
branch
from
September 11, 2026 11:18
9131a92 to
cc605c6
Compare
benjaminVadon
changed the base branch from
main
to
p14-upsert-override-vevent
September 11, 2026 12:41
benjaminVadon
force-pushed
the
recurrence-state-and-edit-scopes
branch
from
September 11, 2026 12:47
cc605c6 to
034628a
Compare
|
This PR/issue depends on:
|
benjaminVadon
force-pushed
the
p14-upsert-override-vevent
branch
from
September 11, 2026 14:15
13f7fc8 to
9f87d80
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
benjaminVadon
force-pushed
the
p14-upsert-override-vevent
branch
from
September 11, 2026 14:17
9f87d80 to
497e412
Compare
benjaminVadon
force-pushed
the
recurrence-state-and-edit-scopes
branch
from
September 11, 2026 14:20
034628a to
faf9346
Compare
|
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
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.
depends on #159