Skip to content

Skip non-calendar accounts in calendar reads (#62)#64

Merged
rockfordlhotka merged 2 commits into
mainfrom
rockfordlhotka/62-skip-non-calendar-accounts
Jun 2, 2026
Merged

Skip non-calendar accounts in calendar reads (#62)#64
rockfordlhotka merged 2 commits into
mainfrom
rockfordlhotka/62-skip-non-calendar-accounts

Conversation

@rockfordlhotka

Copy link
Copy Markdown
Member

rockfordlhotka and others added 2 commits June 2, 2026 19:45
Email-only accounts (e.g. IMAP) have no calendar capability, so
GetCalendarEventsAsync/ListCalendarsAsync throw NotSupportedException.
During the all-accounts fan-out, get_calendar_events swallowed that into
a generic "Failed to retrieve events from this account." warning, which
appeared for every fan-out once #59 made all accounts the default.

Centralize calendar-capability detection in a new AccountCapabilities
helper (the logic previously inlined in ListAccountsTool) and use it to:
- skip non-calendar accounts in the get_calendar_events and list_calendars
  fan-outs, and in the calendarId->account resolution lookup
- return an actionable warning ("no calendar capability") instead of the
  generic error when an email-only accountId is targeted explicitly

ListAccountsTool now delegates to the shared helper; its JSON output is
unchanged. Bump version to 1.3.6.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@rockfordlhotka
rockfordlhotka merged commit f887820 into main Jun 2, 2026
1 check passed
@rockfordlhotka
rockfordlhotka deleted the rockfordlhotka/62-skip-non-calendar-accounts branch June 2, 2026 18:04
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.

1 participant