Skip to content

feat: align datasource plugin with generic workflow - #215

Merged
wbxl2000 merged 1 commit into
mainfrom
qer/datasource-two-tool-workflow
May 29, 2026
Merged

feat: align datasource plugin with generic workflow#215
wbxl2000 merged 1 commit into
mainfrom
qer/datasource-two-tool-workflow

Conversation

@wbxl2000

Copy link
Copy Markdown
Collaborator

Related Issue

No linked issue.

Problem

The TypeScript Kimi Datasource plugin exposed a stock-specific query_stock MCP tool, while the Python plugin exposes only the generic discovery and call tools. That split kept a hard-coded shortcut and a separate skill workflow in the TS plugin.

What changed

  • Removed query_stock from the MCP tool list and handler.
  • Updated datasource skill instructions to use get_data_source_desc before call_data_source_tool, matching the Python plugin workflow.
  • Bumped the official datasource plugin and marketplace version to 3.1.0 and added a plugin changelog entry.
  • Added a regression test for the two-tool MCP surface.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

@changeset-bot

changeset-bot Bot commented May 29, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a1f5d63

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented May 29, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@moonshot-ai/kimi-code@a1f5d63
npx https://pkg.pr.new/@moonshot-ai/kimi-code@a1f5d63

commit: a1f5d63

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a1f5d63d2a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

{
"name": "kimi-datasource",
"version": "3.0.0",
"version": "3.1.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Publish the removed MCP tool as a major change

This release removes the public query_stock MCP tool, so any existing prompt, automation, or saved workflow that calls mcp__plugin-kimi-datasource-data__query_stock will now get Unknown tool, but the plugin is advertised as a 3.0.03.1.0 update. Since removing a command/tool is a breaking change, either keep a compatibility shim for query_stock or ship this as a major version so marketplace users are not offered a non-breaking upgrade that breaks existing calls.

Useful? React with 👍 / 👎.

@wbxl2000
wbxl2000 merged commit b9860e9 into main May 29, 2026
7 checks passed
@wbxl2000
wbxl2000 deleted the qer/datasource-two-tool-workflow branch May 29, 2026 11:51
@github-actions github-actions Bot mentioned this pull request May 29, 2026
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