Skip to content

Update README.md to reflect current functionality #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jxnl
Copy link
Owner

@jxnl jxnl commented Apr 2, 2025

Summary

  • Add Calendar tool features that have been implemented
  • Add Web Search tool that was missing from documentation
  • Update Messages tool to include scheduling and unread features
  • Remove Calendar from TODO list

Changes

This PR updates the README to accurately reflect the currently implemented features in the apple-mcp tool. Before this change, the Calendar tool was listed as a TODO item even though it has been implemented, and the Web Search feature wasn't mentioned at all. Additionally, the Messages tool had more functionality than was documented.

🤖 Generated with Claude Code


Important

Update README.md to include current features for Calendar, Web Search, and Messages tools, and remove Calendar from TODO list.

  • README.md Updates:
    • Adds Calendar tool features: search events, list upcoming events, create events, open events.
    • Adds Web Search tool: search using DuckDuckGo, process search results.
    • Updates Messages tool: schedule messages, check unread messages.
    • Removes Calendar from TODO list.

This description was created by Ellipsis for 63119a5. It will automatically update as commits are pushed.

- Add Calendar tool features that have been implemented
- Add Web Search tool that was missing from documentation
- Update Messages tool to include scheduling and unread features
- Remove Calendar from TODO list as it's been implemented

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 63119a5 in 46 seconds

More details
  • Looked at 32 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 8 drafted comments based on config settings.
1. README.md:50
  • Draft comment:
    Updated 'Messages' section: Ensure terminology consistency across features (e.g. 'read messages from conversations' is clearer than 'read out messages').
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
2. README.md:70
  • Draft comment:
    Added detailed 'Calendar' features. Confirm feature names match implementation, particularly 'Open calendar events in the Calendar app'.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
3. README.md:75
  • Draft comment:
    Included new 'Web Search' block. Consider a usage example or note its implementation details for clarity.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
4. README.md:49
  • Draft comment:
    Updated Messages section looks good. Replacing 'Read out messages' with 'Read messages from conversations' and adding scheduling & unread features improves clarity.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
5. README.md:67
  • Draft comment:
    New Calendar and Web Search sections clearly outline their features. Ensure these descriptions remain in sync with the implemented functionality.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
6. README.md:80
  • Draft comment:
    Removed the obsolete Calendar TODO item; this update aligns the documentation with current features.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
7. README.md:84
  • Draft comment:
    Typographical error: In the sample command, consider replacing "people i met" with "people I met" for correct capitalization.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.
8. README.md:100
  • Draft comment:
    Inconsistent casing: The quick install instructions use '@Dhravya/apple-mcp' while the manual installation JSON uses '@dhravya/apple-mcp@latest'. Consider using consistent casing for the username to avoid confusion.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_Acg2IBrDzbyYtCow


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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