feat(markitdown): add markitdown plugin to marketplace - #83
Conversation
Add Microsoft's MarkItDown MCP server as a plugin for converting documents (PDF, DOCX, PPTX, images, etc.) to Markdown via `uvx markitdown-mcp`.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request integrates the MarkItDown plugin, a powerful tool for converting diverse document formats into Markdown, into the existing plugin ecosystem. This enhancement expands the platform's capabilities by providing users with a new utility for document processing and content transformation, making it easier to manage and utilize various file types within a Markdown-centric workflow. Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds the markitdown plugin to the marketplace. The changes are well-structured and correctly add the plugin manifest, marketplace entry, release configuration, and documentation. I have a couple of suggestions to improve the plugin's discoverability by making the keywords in plugin.json and marketplace.json more comprehensive and consistent with the plugin's description.
Note: Security Review has been skipped due to the limited scope of the PR.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 4 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="plugins/markitdown/.claude-plugin/plugin.json">
<violation number="1" location="plugins/markitdown/.claude-plugin/plugin.json:10">
P2: Remove the unsupported `repository` field from this plugin manifest.
(Based on your team's feedback about disallowing `repository` in plugin.json manifests.) [FEEDBACK_USED]</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
…etection - Add SKILL.md with trigger phrases for document conversion use cases - Add PreToolUse hook that warns when Read is used on .docx/.pptx/.xlsx files - Add check-binary-doc.sh shell script for fast extension-based detection
- Add 'images' keyword to marketplace.json entry for better discoverability - Add 'pptx' and 'images' keywords to plugin.json to match description
There was a problem hiding this comment.
2 issues found across 3 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="plugins/markitdown/skills/markitdown/SKILL.md">
<violation number="1" location="plugins/markitdown/skills/markitdown/SKILL.md:53">
P1: The example uses a non-standard `path` argument for `convert_to_markdown`; this tool expects a `uri` argument (typically a `file://` URI).</violation>
</file>
<file name="plugins/markitdown/hooks/check-binary-doc.sh">
<violation number="1" location="plugins/markitdown/hooks/check-binary-doc.sh:11">
P2: Build the JSON response with proper escaping. Interpolating the user-derived extension into a raw here-doc can emit invalid JSON if the file path contains quotes/backslashes/newlines.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
…arkdown - Fix SKILL.md examples to use uri="file:///path" instead of path= argument - The markitdown-mcp tool accepts uri (http/https/file/data), not path - Use jq -n for safe JSON output in check-binary-doc.sh - Update hook additionalContext to show correct file:// URI syntax
1 participant
Summary
uvx markitdown-mcpto convert documents (PDF, DOCX, PPTX, images, etc.) to MarkdownChanges
plugins/markitdown/.claude-plugin/plugin.json— new plugin manifest.claude-plugin/marketplace.json— new marketplace entry (category: tooling)release-please-config.json— release tracking for the pluginREADME.md— plugin documentation and install commandInstall
Summary by cubic
Adds Microsoft's MarkItDown as a new
markitdownMCP plugin to convert documents (PDF, DOCX, PPTX, images) to Markdown viauvx markitdown-mcp. Includes a SKILL and a PreToolUse hook that warns whenReadtargets Office files, plus marketplace entry, release tracking, README install command/plugin install markitdown@pleaseai, CLAUDE.md plugin validation commands, and updated usage to theuriargument withfile://.Bug Fixes
pathtouri="file:///..."(supports http/https/file/data).file://syntax and emit JSON usingjq -nfor safety.Dependencies
bun.lockwith minor bumps (e.g.,@nuxt/devtools3.2.3,@vue/devtools-*8.0.7,fast-npm-meta1.4.2,simple-git3.33.0).Written for commit 8a2cfd4. Summary will update on new commits.