Modernize AgentMail plugins for current clients#2
Merged
Conversation
- Remove root CLAUDE.md (gitignored local symlink documented in AGENTS.md) and strict-validate the plugin manifest in CI, which the directory form of claude plugin validate does not cover. - Reword check-email and send-email skills to fetch bodies with get_thread: the hosted MCP server has no message-level fetch tool and list/search results contain only previews. - Run the exact-pin upstream compatibility check only on schedule and workflow_dispatch so registry releases no longer fail unrelated PRs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…mples - Move upstream-drift detection (check_compatibility.py) to a weekly scheduled drift.yml so registry releases and outages no longer block unrelated pushes and PRs; validate.yml now tests pinned versions only. - Restore LangChain examples (TS and Python) and document the MCP adapter in the toolkit skill, verified against agentmail-toolkit 0.4.2 / 0.2.7 shipped interfaces; the frontmatter already advertised both. The TS examples are type-checked in CI via tests/sdk_typescript.ts. - Add executable Svix webhook verification fixtures (Python unittest + tsx script) proving the documented verify path accepts valid signatures and rejects tampered bodies and stale timestamps. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LWVTg3BUb2cPLLqTfgSLKr
- Use OAuth for Cursor like Claude and Codex: drop mcp.cursor.json and the API-key setup docs; keep the x-api-key header pattern as a generic fallback for clients without OAuth. Needs a manual Cursor sign-in check before the marketplace release. - Trim validate_repo.py: drop the hard-coded skill set, migration-era stale patterns, and the double manifest parse. - Shrink check_compatibility.py fetch retry and flatten the signature tests to plain asserts. - Run push CI only on main so PRs run once, and drop the Claude manifest skills/mcpServers fields the platform defaults already cover. - Delete the README repository-structure tree that restated ls. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
No live email was sent or deleted during validation.