docs(chat): document post-send message ID chain - #897
Open
Anonymity-0 wants to merge 3 commits into
Open
Conversation
wxianfeng
self-requested a review
August 7, 2026 03:42
Collaborator
开源 DWS Codex 自动 CR:不通过,需补充 Agent 测试报告截图docs(chat): document post-send message ID chain
变更总结本次变更补充了个人消息发送后通过 openTaskId 查询 openMessageId/openConversationId,并继续编辑或撤回的帮助文案、路由指南与技能参考,同时新增跨平台帮助输出回归测试。Go 代码仅调整帮助文本,未改变命令执行、参数映射或安全门逻辑;新增示例未包含 --yes 等确认绕过参数,测试命名也符合平台覆盖门约定。受静态审查限制,未执行测试或验证远端 MCP 实际响应结构,剩余风险主要是文档所述返回字段仍依赖服务端契约保持一致。 CI 单测结果
结论说明自动 CR 不通过,原因如下:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
chat message sendreturnsopenTaskId.chat message query-send-statusreturnsopenMessageIdandopenConversationId.send -> query-send-status -> edit/recallexamples to CLI help.Risk tier
packaging, or interface behavior changed
platform, auth/keychain, installer, packaging, release, transport, recovery,
or another fail-closed infrastructure change
Verification
Record the smallest targeted evidence that proves the changed behavior. Do not
repeat the entire CI suite locally only to fill this checklist: CI expands the
selected tier from documentation checks, through affected-package tests, to
the complete high-risk suite.
CHANGELOG.md-only check (otherwiseN/A):N/A — this is not a CHANGELOG-only change.
go test ./internal/helpers -run '^TestCrossPlatformCoverageChatMessageHelpDocumentsPostSendIDChain$' -count=1— passedgo test ./internal/helpers -count=1— passedmake build— passedmake generate-schema— passed./scripts/policy/check-schema-catalog.sh— passed./scripts/policy/check-skill-context-budget.sh— passed./scripts/policy/check-skill-commands.sh— passedgo test ./scripts/policy/multi-im-skill-chain -count=1— passed2bd6b297b0140c60a5274e083bc8101302deffe0):chat message send --help— post-sendopenTaskIdhandoff:chat message query-send-status --help—openMessageIdandopenConversationId:chat message edit --help— complete post-send edit chain:chat message recall --help— complete post-send recall chain:N/A): N/A — Standard tier.tiers; record command/result or
N/A): N/A — Standard tier../scripts/policy/check-generated-drift.sh(when generator inputs or generated artifacts may change) — passed
./scripts/policy/check-command-surface.sh --strict(if command surface changed):N/A — no command paths or flags changed.
./scripts/release/verify-package-managers.sh(after
make package, if packaging or installer surfaces changed):N/A — no packaging or installer changes.
Notes
The repository automatically requests one eligible peer reviewer, including
after a new head push when another review is needed. Once the latest push has
peer approval and all nine required checks are current and green, auto-merge
completes the PR; authors do not need to coordinate a separate routine merge.