Skip to content

fix(Pipeline): 完善 focus 协议 Schema - #1473

Merged
MistEO merged 1 commit into
MaaXYZ:mainfrom
Craun718:chore/schema
Sep 7, 2026
Merged

fix(Pipeline): 完善 focus 协议 Schema#1473
MistEO merged 1 commit into
MaaXYZ:mainfrom
Craun718:chore/schema

Conversation

@Craun718

@Craun718 Craun718 commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 为 Pipeline focus 字段补充结构化 Schema 定义
  • 支持全部 21 个 Node.* 回调消息类型
  • 支持 string / object 模板,以及 contentdisplaytrace 字段
  • 补充正确与错误 schema fixtures

Validation

  • rtk uvx check-jsonschema --schemafile tools/pipeline.schema.json test/schema/pipeline.schema.correct.json
  • rtk uvx check-jsonschema --schemafile tools/pipeline.schema.json test/schema/pipeline.schema.error.json(预期失败,并命中新增约束)
  • git diff --cached --check

Sourcery 摘要

完善 Pipeline focus 字段的结构化 Schema 校验能力。

新功能:

  • 为 Pipeline focus 字段定义结构化 Schema,涵盖全部 21 种 Node.* 回调消息类型及支持的模板字段。

问题修复:

  • 通过强制执行新定义的结构,改进 Pipeline focus 配置的校验。

测试:

  • 添加有效和无效的 Schema fixture,覆盖扩展后的 focus 约束。
Original summary in English

Summary by Sourcery

完善 Pipeline focus 字段的结构化 Schema 校验能力。

New Features:

  • Define a structured schema for the Pipeline focus field, covering all 21 Node.* callback message types and supported template fields.

Bug Fixes:

  • Improve validation of Pipeline focus configurations by enforcing the newly defined structure.

Tests:

  • Add valid and invalid schema fixtures covering the expanded focus constraints.

@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 ✨

@MistEO
MistEO merged commit e6290c9 into MaaXYZ:main Sep 7, 2026
5 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.

2 participants