Skip to content

docs(contracts): requiresTools 描述改为短 ID 冲突时不归属(#1763 后续) - #1764

Merged
xiaoyumuxi merged 1 commit into
previewfrom
fix/1763-requires-tools-schema-description
Nov 5, 2026
Merged

xiaoyumuxi merged 1 commit into
previewfrom
fix/1763-requires-tools-schema-description

Conversation

@xiaoyumuxi

@xiaoyumuxi xiaoyumuxi commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

关联 Issue / Linked issue

无。#1763 的后续,Refs #1763

PR 类型 / Type

  • docs

改动说明 / What changes

plugin-manifest.schema.json 里 requiresTools 的 description 还写着 #1763 已取消的旧规则,这里只改这一句:

  • 旧:An ID of another plugin is a qualified ID and needs a verified official origin when short.
  • 新:An ID is short or <pluginID>:<id>; a short ID defined by more than one enabled plugin belongs to none of them.

校验规则、能力版本、fixture 和 Core 代码都不变。

范围检查 / Scope checks

  • 功能矩阵:没有更新,没有用户可观察的变化。“Verify Platform Feature Matrix” 把 shared/contracts 算作实现路径所以失败,请 reviewer 决定是否加 matrix-exempt。作者没有添加该标签。
  • 契约同步:合入后在 lithe-plugin-docs 运行 contract.mjs sync。没有新协议、新能力版本或新宿主方法。
  • 日志、i18n、逻辑归属、性能、设置、发行包、重构:N/A,没有代码改动。

验证 / Verification

  • ./scripts/architecture/verify-shared-contracts.sh:通过
  • 没有启动 Lithe 应用

plugin-manifest.schema.json 中 requiresTools 的描述仍写着短 ID 需要经过验证的官方来源,#1763 已取消这条规则:任何插件都可以定义短 ID,被多个已启用插件定义的短 ID 不归属任何插件。只改描述文字,不改校验规则。
@ghfind-review ghfind-review Bot added the review: high ghfind author score; see https://ghfind.com label Nov 5, 2026
@xiaoyumuxi
xiaoyumuxi marked this pull request as ready for review November 5, 2026 08:11
@lithe-code-review

lithe-code-review Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Lithe Review

结论: ✅ 通过
范围: preview ← fix/1763-requires-tools-schema-description · head: e4749ee

未发现需要阻塞合并的问题。

Scope Check

  • 已完整审查唯一文件的改动:shared/contracts/plugins/v1/plugin-manifest.schema.json:114 仅更新描述,未改变校验约束或运行时行为。
  • 新描述与 base 已有的工具链 README、Core 的 tool_id 校验及 registry 冲突处理一致。
  • 未找到可验证的关联 Issue;按 PR 声明的描述修正目标核对,已完成。
  • 不涉及新增上游能力或架构边界变化;公开契约副本仍需在合并后同步。

Summary

本次修正消除了 schema 与现有契约规则的不一致,未发现本次引入或加剧的问题。

Verification

  • 静态检查了指定 base…head 差异、相关契约 README,以及 Core 的 validate_required_tools、tool_id、registry。
  • 作者报告 verify-shared-contracts.sh 通过;提供的检查记录显示 CI tooling 成功,功能矩阵检查失败,仍需维护者核实并处理。
  • 本次未运行代码、测试、脚本或应用。Swift、Windows 和 Rust Core 测试在提供的检查记录中均为 skipped;未独立验证公开契约副本同步。

@xiaoyumuxi
xiaoyumuxi merged commit 5cc06d5 into preview Nov 5, 2026
22 of 30 checks passed
@xiaoyumuxi
xiaoyumuxi deleted the fix/1763-requires-tools-schema-description branch November 9, 2026 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: high ghfind author score; see https://ghfind.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant