fix(contract): OpenAPI 契约漂移修复 + 成功码大小写统一 + 契约门禁防空转 - #2159
Merged
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
- openapi: contacts/search 参数 q→id（对齐实现与前端 SSOT）；messages/sync cursor→after_seq；AgentRunEvent.payload object→string；attachments multipart 补 hash/original_name；/client/sessions/{id}/messages 补 before_seq/limit；audit-events/market/mcp/skills/execution-targets 补全实现已有 query 参数；新增 edge /v1/runs/{runId}/checkpoint(+/file) 两路径
- 成功码大小写：hub health Ready 用 errcode.OK.Code；edge writeSuccess "OK"→"ok"；桌面 edgeClient 与全部 edge 测试 helper 双大小写兼容
- api/events.md 30→31 常量自相矛盾修正
- verify-shared-rest-contract：默认目录改 app/shared/src/hub（此前 CI 扫 0 路径空转），0 路径直接 FAIL
Co-authored-by: Cursor <cursor@vectorcontrol.tech>
DeliciousBuding
force-pushed
the
chore/cleanup-round-16
branch
from
August 31, 2026 21:21
affcff8 to
a0884db
Compare
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
1 participant
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.
Harvey（OpenAPI 契约审计）修复项：P1-2/P1-3/P1-4/P1-5/P1-6 + P2-1（部分）/P2-2/P2-4/P2-6。\n\n- contacts/search q→id、sync cursor→after_seq、payload string、attachments multipart 字段、六个列表端点 query 参数补全、edge checkpoint 两路径\n- 成功 envelope code 大小写统一为小写 ok（hub health + edge writeSuccess），客户端与测试 helper 兼容双大小写\n- verify-shared-rest-contract 默认目录修正 + 0 路径 fail-closed\n- events.md 内部计数矛盾修正\n\n验证：edge/hub 单测全绿；yaml 解析 + openapi-contract + doc-ssot 全过。