feat(runtime): /automation 域 handler 体抽出 — ADR-0076 D11 步骤③ PR-6 (#2462) - #3542
Merged
Conversation
…D11 step ③ PR-6 (#2462) The automation service bridge (~260 lines) moves to domains/automation.ts. Cleanest extraction of the series: zero new DomainHandlerDeps growth (getService/resolveService/success/error cover it). Route-order subtlety preserved verbatim — /actions, /connectors and /_status keep their guard positions before the /:name→getFlow catch-all (a flow literally named "actions" must not shadow them; a new seam test locks the guard position). NOTE: /actions (handleActions) is NOT in this cut — it is deep-coupled to env-resolution state (this.kernel / kernelResolver / resolveDefaultProject + 4 private helpers) and goes with the /data /meta deep-coupling batch. Verified: seam suite 37 tests, runtime 642 green, http-conformance 41 green, dependent closure builds with DTS (--force). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
documentation
Contributor
📓 Docs Drift CheckThis PR changes 1 package(s): 17 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:
|
6 tasks
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.
动机（#2462 D11 步骤③，系列第六刀）
抽出
/automation（flow CRUD、trigger/execute、runs 历史、pause/resume——ADR-0018/0019/0022 面，~260 行）。系列中最干净的一刀：deps 契约零增长（getService/resolveService/success/error 全覆盖）。改动
domains/automation.ts：体逐字搬迁。路由顺序精妙处原样保留——/actions、/connectors、/_status三个守卫分支必须先于/:name → getFlowcatch-all（字面叫 "actions" 的 flow 不得遮蔽它们），新测试锁定守卫位置。/actions（handleActions）有意不在本刀：调查确认它深耦合 env 解析状态（this.kernel/kernelResolver/resolveDefaultProject+ 4 个私有 helper），归入 /data /meta 深耦合批单独设计。验证
接缝套件 37 测试（新增 3：列表 / 守卫位置+过滤 / 无服务 fall-through）；runtime 全量 642 绿；http-conformance 41 断言绿；下游闭包
--force含 DTS 全绿。关联 #2462（D11 步骤③ PR-6）。剩余：/auth /mcp /ai（中）+ /actions /meta /data（深耦合批，需单独设计）。
🤖 Generated with Claude Code