feat(projects): add Dolphin MCP Pilot project and blog post - #81
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dongjiang1989
changed the title
- Add new project Dolphin MCP Pilot (iflytek/dolphin-mcp-pilot) — production-grade MCP server for Apache DolphinScheduler with 53+ tools covering workflow creation, scheduling, instance control, resource management, and raw API passthrough. Category: agent-skills, Language: Python, License: Apache-2.0 - Add release blog post (dolphin-mcp-pilot-release.md) covering motivation, technical architecture, tool matrix, quick start guide, and real-world usage scenarios - Update .cache/project-stats.json (10 repos) and .cache/contributors.json (140 contributors) Signed-off-by: dongjiang <dongjiang1989@126.com>
1fed2c9 to
74ee506
Compare
|
Thanks for putting this together. Two issues before merge: 1. Please split this PRThe description lists 4 files, but the diff contains 10. The additional 6 files are unrelated to Dolphin MCP Pilot:
These changes (especially dropping three partner logos and rolling back HER Hack-Astron to single-repo scope) deserve separate review. Please keep only the 4 Dolphin MCP Pilot files in this PR. 2. Tool names don't match implementationVerified against the actual 59 registered tools in
These appear in the tool matrix table and usage examples. Readers will copy them into agent prompts and hit "tool not found" errors. Also: the post says 53+ tools in three places, while the repo currently registers 59. Consider updating both to 58 (excluding internal helper The technical content itself (guided troubleshooting, serial complement ordering) is accurate. Happy to review again once split and tool names are fixed. |
Per PR review feedback: - Fix 6 incorrect tool names to match actual implementation: ds_update_task → ds_update_task_param ds_create_schedule → ds_set_schedule ds_rerun_process → ds_rerun_process_instance ds_view_task_log → ds_get_task_log ds_raw_api → ds_raw_get/post/put/delete (4 tools) ds_rollback_workflow → ds_rollback_workflow_version - Update tool count from "53+" to "58" (actual registered tools) Signed-off-by: dongjiang <dongjiang1989@126.com>
@Hui-of-limin This AI commit bug.
Update tools name done. |
- Added: Dolphin MCP Pilot project + release blog post - Fixed: 5 npm audit vulnerabilities (4 high, 1 moderate) Signed-off-by: dongjiang <dongjiang1989@126.com>
|
LGTM! Tool names are all fixed (verified against the actual 59 registered tools). Ready to merge. @dongjiang1989 Could you merge this when you get a chance? I don't have write permission on this repo. |
Hui-of-limin
left a comment
Hui-of-limin
left a comment
There was a problem hiding this comment.
Tool names verified against the 59 registered tools in iflytek/dolphin-mcp-pilot - all 20 cited names are correct now, and the count is updated to 58. Technical content (guided troubleshooting, serial complement ordering) matches the implementation. All 10 CI checks pass.
Approving. The unrelated her-partner file changes are acknowledged as an AI commit artifact per the discussion above.
2 participants
Summary
iflytek/dolphin-mcp-pilot) — production-grade MCP Server for Apache DolphinScheduler with 58 tools covering workflow creation, scheduling, instance control, resource management, and raw API passthrough. Category:agent-skills, Language: Python, License: Apache-2.0dolphin-mcp-pilot-release.md— release announcement covering project motivation, technical architecture (MCP protocol, dual auth modes, guided troubleshooting), tool matrix, quick start guide, and real-world usage scenarios.cache/project-stats.json(10 repos) and.cache/contributors.json(140 contributors)Changed Files
src/data/projects/dolphin-mcp-pilot.yamlsrc/data/post/dolphin-mcp-pilot-release.md.cache/project-stats.json.cache/contributors.jsonCHANGELOG.mdWhich issue(s) this PR fixes:
Fixes #
Special notes for reviewers:
iflytek/dolphin-mcp-pilotdomux-smart-home-command-model.md)