Skip to content

fix(cleanup): 审计收口第七波——skills 面精简 / wsIPRL shutdown 接线 / ACP 测试构造面 - #2156

Merged
DeliciousBuding merged 1 commit into
masterfrom
chore/cleanup-round-13
Aug 31, 2026
Merged

fix(cleanup): 审计收口第七波——skills 面精简 / wsIPRL shutdown 接线 / ACP 测试构造面#2156
DeliciousBuding merged 1 commit into
masterfrom
chore/cleanup-round-13

Conversation

@DeliciousBuding

Copy link
Copy Markdown
Collaborator

#2154 剩余三项:\n\n- edge skills 包 1814→约 500 行:删 hot-reload watcher / trigger 匹配 / lazy body / Get/List 等零生产接线面,生产仅 Discover+Count+SystemPromptContext;fsnotify 出依赖\n- hub StopWSIPRateLimiter 重新引入并接线 App.Shutdown(第 7.5 步),含幂等测试\n- edge ACP NewAcpAdapter/NewAcpAdapterWithID 删除,4 个测试改用 NewAcpAdapterConfig\n- test-sleep ratchet 同步(移除 skills hot-reload sleep 条目)\n\n验证:edge/hub go vet + 全量单测绿;hub lint 0 issues、edge lint ratchet PASS;doc-ssot/ci-gates ok。

- edge skills: 删除零接线 hot-reload watcher(skill_watch.go 302 行)、trigger 匹配簇(MatchTrigger/WordBoundary/compileTriggers)、LoadBody/Get/List/ParseBody/ParseFull;仅保留生产面 Discover+Count+SystemPromptContext+ParseFrontmatter(#2154);fsnotify 依赖移除
- hub: StopWSIPRateLimiter 接线 App.Shutdown(7.5 步),全局 cleanup goroutine 不再跨过优雅关停窗口(#2154)
- edge ACP: 删除仅测试使用的 NewAcpAdapter/NewAcpAdapterWithID/newAcpAdapter(4 个测试改走 NewAcpAdapterConfig)
- test-sleep 基线同步(skills hot-reload sleep 条目移除)

Co-authored-by: Cursor <cursor@vectorcontrol.tech>
@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: b4d50529-1d74-4a00-bcb5-38bf9b677f12

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@DeliciousBuding
DeliciousBuding merged commit c6a39f1 into master Aug 31, 2026
39 checks passed
@DeliciousBuding
DeliciousBuding deleted the chore/cleanup-round-13 branch August 31, 2026 20:40
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.

1 participant