Skip to content

feat: support event checklists#15

Merged
ehs208 merged 2 commits into
ehs208:mainfrom
seisyo58:main
May 10, 2026
Merged

feat: support event checklists#15
ehs208 merged 2 commits into
ehs208:mainfrom
seisyo58:main

Conversation

@seisyo58
Copy link
Copy Markdown
Contributor

@seisyo58 seisyo58 commented May 8, 2026

Summary

  • Add checklist support to event create/update operations
  • Include checklist in event list/update response payloads
  • Add TimeTree attachment/checklist types

Test plan

  • npm run build
  • hermes mcp test timetree
  • Manual test against a real TimeTree calendar through Hermes MCP:
    • Create an event without checklist
    • Add checklist to an existing event
    • Update checklist items to checked: true
    • Update checklist items back to checked: false
    • Remove checklist by setting checklist: []
    • Delete the event
    • Create an event with checklist

The create/update responses returned checklist as expected.

Copy link
Copy Markdown
Owner

@ehs208 ehs208 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! This is a great addition — checklist support is something users will definitely find useful, and the implementation is clean.

One small suggestion: could you add a mention of checklist support in the description string of create_event and update_event tools? AI clients rely on the tool description to decide which tool to invoke, so without it, a user saying "add a checklist to my event" might not trigger the right tool.

Something like appending "Supports attaching checklist items." to the existing description would be enough.

@seisyo58
Copy link
Copy Markdown
Contributor Author

seisyo58 commented May 9, 2026

Thanks! Updated the tool descriptions to mention support for attaching checklist items.

Copy link
Copy Markdown
Owner

@ehs208 ehs208 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for the first contribution! Merging now.

@ehs208 ehs208 merged commit f908e0f into ehs208:main May 10, 2026
3 checks passed
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