Skip to content

Conversation

@ChuckNovice
Copy link
Owner

Summary

  • Remove RecurrenceEndTime from RecurrenceCreate input model - users no longer need to pass this redundant property
  • Remove RecurrenceEndTime from RecurrenceDetails output model - end time can be parsed from RRule if needed
  • Automatically extract recurrence end time from RRule UNTIL clause in the engine
  • Replace regex-based RRule validation with Ical.Net's RecurrencePattern parser for more robust validation

Breaking Changes

Change Before After
RecurrenceCreate.RecurrenceEndTime Required property REMOVED - derived from RRule UNTIL
RecurrenceDetails.RecurrenceEndTime Property on output model REMOVED - use RRule
RRule validation Regex-based Ical.Net parser-based

Test plan

  • All 161 unit tests pass
  • dotnet build succeeds
  • dotnet format --verify-no-changes passes
  • Create recurrence with just RRule (no RecurrenceEndTime) - RecurrenceEndTime is extracted from UNTIL

🤖 Generated with Claude Code

- Remove RecurrenceEndTime from RecurrenceCreate input model
- Remove RecurrenceEndTime from RecurrenceDetails output model
- Extract recurrence end time automatically from RRule UNTIL clause
- Replace regex-based RRule validation with Ical.Net RecurrencePattern parser
- Update README files with simplified API examples

BREAKING CHANGE: RecurrenceEndTime property removed from RecurrenceCreate
and RecurrenceDetails. The recurrence end time is now automatically
extracted from the RRule UNTIL clause.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ChuckNovice ChuckNovice merged commit 1783cf7 into main Jan 27, 2026
2 checks passed
@ChuckNovice ChuckNovice deleted the feature/drop-recurrence-end-time branch January 27, 2026 23:37
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.

2 participants