Skip to content
This repository was archived by the owner on Aug 11, 2026. It is now read-only.

feat(plugin-protocol): export ghost summary length limit - #44

Merged
fmfsaisai merged 1 commit into
mainfrom
feat/ghost-summary-limit
Aug 7, 2026
Merged

fmfsaisai merged 1 commit into
mainfrom
feat/ghost-summary-limit

Conversation

@fmfsaisai

Copy link
Copy Markdown
Collaborator

这次改了什么

摘要

为插件身份卡的 description 与 whenToUse 导出统一的 300 字符上限,保持原有校验语义和错误文案不变。客户端花名册、manifest 校验和插件作者手册可以引用同一个协议常量,避免各处漂移。

变更类型

  • feat 新功能
  • fix 缺陷修复
  • refactor / perf 重构或性能优化
  • docs / test / chore 文档、测试或工程维护

范围

  • 本 PR 包含:packages/plugin-protocol/src/manifest.ts 新增 GHOST_MANIFEST_SUMMARY_MAX_CHARS 并用于 description / whenToUse 校验。
  • 明确不包含:客户端接线、花名册投影、ghost_info 或插件运行时改动。
  • 用户可见变化:无直接变化;为客户端和插件作者手册提供统一契约来源。
  • 是否存在 breaking change:无。

怎么验证的

自动验证

pnpm --filter @cindy/plugin-protocol typecheck
pnpm --filter @cindy/plugin-protocol test
结果:通过;父仓客户端分支另行运行根 pnpm test:unit。

手工验证

不涉及。

未执行的验证

无。

风险

风险分类

  • 协议兼容
  • 其他

影响与回滚

  • 影响范围:新增导出常量,不改变现有字段限制、错误码或错误文案;旧客户端无需使用该导出即可继续工作。
  • 回滚 / 降级方式:回退本 PR,并将客户端 gitlink 回退到合并前的协议提交。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名(git commit -s)
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档
  • 已确认测试结果或说明未执行原因。

🤖 Generated with Claude Code

@greptile-apps

greptile-apps Bot commented Aug 7, 2026 •

Copy link
Copy Markdown

Greptile Summary

新增并导出插件身份卡摘要字段的统一 300 字符上限,同时保持现有校验语义不变。

  • 使用 GHOST_MANIFEST_SUMMARY_MAX_CHARS 统一校验 description 与 whenToUse
  • 增加两个字段在上限及超出上限时的边界测试

Confidence Score: 5/5

该 PR 看起来可以安全合并。

未发现仍然存在的阻塞性故障。

Important Files Changed

Filename Overview
packages/plugin-protocol/src/manifest.ts 导出摘要长度常量并替换两处等值的硬编码限制及错误文案插值,未改变校验行为。
packages/plugin-protocol/src/tests/manifest.test.ts 验证导出常量值以及 description、whenToUse 两个字段的 300/301 字符边界行为。

Reviews (4): Last reviewed commit: "feat(plugin-protocol): export ghost summ..." | Re-trigger Greptile

@fmfsaisai
fmfsaisai force-pushed the feat/ghost-summary-limit branch 2 times, most recently from 32c5c83 to 71adc70 Compare August 7, 2026 08:00
Use one exported 300-character limit for description and whenToUse while preserving the existing validation semantics and error messages.

Signed-off-by: fmfsaisai <fmfsaisai@gmail.com>
@fmfsaisai
fmfsaisai force-pushed the feat/ghost-summary-limit branch from 71adc70 to d99e3d0 Compare August 7, 2026 10:27
@fmfsaisai
fmfsaisai merged commit 9fcfe5f into main Aug 7, 2026
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant