Skip to content

apple mail search fails #30

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
cduguet opened this issue Apr 7, 2025 · 3 comments · May be fixed by #32
Open

apple mail search fails #30

cduguet opened this issue Apr 7, 2025 · 3 comments · May be fixed by #32

Comments

@cduguet
Copy link

cduguet commented Apr 7, 2025

I have been trying to make the mail mcp work, but it fails at searching, and listing all messages every time.
Simple things like listing all latest 5 messages of a certain account, don't work.
For example, a general search tries to be performed with "*", when wildcards are not accepted by AppleScript. I would like to contribute to make this work, but don't have any experience with building an MCP, and I don't know if this is something that can be fixed from the code, of from the model side.

Does somebody have any idea how to start solving this?

@Dhravya
Copy link
Collaborator

Dhravya commented Apr 7, 2025

This is mostly solvable by adding more specific tools for different mail tasks. We use applescripts to do this scripting, in mail.ts file.
The more tools the better claude can execute them (and the code gets more reliable too)

Dhravya added a commit that referenced this issue Apr 8, 2025
@Dhravya Dhravya linked a pull request Apr 8, 2025 that will close this issue
@aadian
Copy link

aadian commented Apr 11, 2025

it looks like there's some issue on code line 1245 on the branch

 if (cc && typeof cc !== "string") return false;
 if (bcc and typeof bcc !== "string") return false;

@Dhravya
Copy link
Collaborator

Dhravya commented Apr 12, 2025

Yeah. agreed

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 a pull request may close this issue.

3 participants