Skip to content

chore(tools): 移除 MPE 相关联动功能#34

Merged
KhazixW2 merged 1 commit into
MAA-AI:mainfrom
kqcoxn:remove-mpe
Jun 12, 2026
Merged

chore(tools): 移除 MPE 相关联动功能#34
KhazixW2 merged 1 commit into
MAA-AI:mainfrom
kqcoxn:remove-mpe

Conversation

@kqcoxn

@kqcoxn kqcoxn commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 移除 open_pipeline_in_browser tool 及其辅助函数 generate_share_link
  • 移除 MPE 相关常量(MPE_BASE_URLMPE_SHARE_PARAMMPE_MAX_URL_SIZEMPE_SHARE_VERSION
  • 移除 lzstring 依赖
  • 更新 README 中英文文档

Why

该工具存在 path traversal 安全风险(CWE-22),且 MPE 联动功能使用场景有限。直接移除比沙盒化更干净。

Resolve #32

Summary by Sourcery

移除基于 MPE 的流水线可视化工具及其相关配置和依赖。

增强内容:

  • 从流水线工具模块中移除 MPE 浏览器可视化工具及其相关常量。
  • 由于不再生成 MPE 分享链接,移除运行时依赖 lzstring

文档:

  • 更新中文版和英文版 README 文件,移除对 MPE 浏览器可视化工具的相关文档说明。
Original summary in English

Summary by Sourcery

Remove the MPE-based pipeline visualization tool and its associated configuration and dependency.

Enhancements:

  • Remove the MPE browser visualization tool and related constants from the pipeline tools module.
  • Drop the lzstring runtime dependency now that MPE sharing links are no longer generated.

Documentation:

  • Update Chinese and English README files to remove documentation of the MPE browser visualization tool.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

你好——我已经审查了你的改动,一切看起来都非常棒!


Sourcery 对开源项目是免费的——如果你喜欢我们的评审,请考虑分享给更多人 ✨
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈改进后续评审。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@KhazixW2 KhazixW2 merged commit c93ef45 into MAA-AI:main Jun 12, 2026
10 checks passed
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.

[security] open_pipeline_in_browser lacks path validation (adjacent to #31)

2 participants