Skip to content

fix(app): compact MCP and HTTP workflow tool references - #7634

Open
FinleyGe wants to merge 9 commits into
labring:mainfrom
FinleyGe:fix/app-storage-jsonschema-string
Open

fix(app): compact MCP and HTTP workflow tool references#7634
FinleyGe wants to merge 9 commits into
labring:mainfrom
FinleyGe:fix/app-storage-jsonschema-string

Conversation

@FinleyGe

@FinleyGe FinleyGe commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Persist MCP and HTTP workflow references as { toolId } and remove obsolete version, tool snapshot, and schema data from workflow nodes.
  • Resolve the latest tool metadata and JSON Schema from the referenced toolset App at runtime.
  • Keep root toolList schema persistence for standalone MCP/HTTP toolset Apps, with object/string read compatibility for historical data.
  • Compact MCP/HTTP tool references in a single traversal; no new getChildren API is introduced.

Fixes #7477

Verification

  • Focused Global Vitest: 2 files, 256 tests passed.
  • Focused Service Vitest: 8 files, 234 tests passed.
  • pnpm --dir projects/app typecheck passed.
  • Prettier check and git diff --check passed.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Admin Preview Image Ready!

ghcr.io/labring/fastgpt-pr:admin_3b95ae59185bd424f06f81c6e34b9bd3beb1d450

🕒 Time: 2026-09-04 06:54:05 (UTC+8)

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 33.36% 3340 / 10010
🔵 Statements 33.26% 3526 / 10601
🔵 Functions 27.88% 667 / 2392
🔵 Branches 27.93% 1662 / 5949
File CoverageNo changed files found.
Generated in workflow #2941 for commit 3b95ae5 by the Vitest Coverage Report Action

@FinleyGe
FinleyGe force-pushed the fix/app-storage-jsonschema-string branch from 29351e3 to e1ffdcb Compare September 2, 2026 07:47
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Build and publish successful - Preview fastgpt Image:

ghcr.io/labring/fastgpt-pr:fastgpt_3b95ae59185bd424f06f81c6e34b9bd3beb1d450

🕒 Time: 2026-09-04 06:53:55 (UTC+8)

nodes: normalizedWorkflow.nodes,
teamId
});
const storageNodes = encodeToolSetNodesForStorage(normalizedWorkflow.nodes);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

追加到 normalizedWorkflow 里面是不是更好点? 不用链式处理,直接复用循环

@FinleyGe FinleyGe changed the title fix(app): store workflow MCP and HTTP schemas as strings fix(app): compact MCP and HTTP workflow tool references Sep 3, 2026
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.

mcp工具以$开头的字段无法存储

2 participants