Skip to content

fix(workflow): align MCP tool template behavior#7287

Draft
FinleyGe wants to merge 2 commits into
labring:mainfrom
FinleyGe:codex/fix-tool-description-on-add
Draft

fix(workflow): align MCP tool template behavior#7287
FinleyGe wants to merge 2 commits into
labring:mainfrom
FinleyGe:codex/fix-tool-description-on-add

Conversation

@FinleyGe

@FinleyGe FinleyGe commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • preserve MCP child tool description as the preview node intro when adding an MCP tool
  • prevent MCP toolsets from being dragged directly from the node template list, matching system toolset behavior
  • avoid duplicate getSystemToolTemplates requests when opening system toolsets by letting the system-tools effect own those loads

Tests

  • git diff --check
  • git diff --check HEAD~1..HEAD
  • not run: local Vitest/tsc are unavailable because this worktree does not have completed node_modules; corepack pnpm exec reports Command not found for vitest and tsc

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 18.07% 1598 / 8839
🔵 Statements 18.09% 1680 / 9285
🔵 Functions 15.99% 337 / 2107
🔵 Branches 15.08% 762 / 5050
File CoverageNo changed files found.
Generated in workflow #1354 for commit 309e29f by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Admin Preview Image Ready!

ghcr.io/labring/fastgpt-pr:admin_309e29fed8f115f434e319fcfa758cee54e85a56

🕒 Time: 2026-07-09 17:02:10 (UTC+8)

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Build Successful - Preview fastgpt Image for this PR:

ghcr.io/labring/fastgpt-pr:fastgpt_309e29fed8f115f434e319fcfa758cee54e85a56

🕒 Time: 2026-07-09 17:01:26 (UTC+8)

@FinleyGe
FinleyGe force-pushed the codex/fix-tool-description-on-add branch from deebcc4 to c76b934 Compare July 9, 2026 08:35
@FinleyGe FinleyGe changed the title fix(workflow): preserve tool descriptions on add fix(workflow): preserve MCP tool intro on add Jul 9, 2026
@FinleyGe FinleyGe changed the title fix(workflow): preserve MCP tool intro on add fix(workflow): align MCP tool template behavior Jul 9, 2026
@FinleyGe
FinleyGe marked this pull request as draft July 10, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant