fix(core): the console runs a stored action that leaves type to its default as a script action - #26964

Requested by Steedos · project thread

Fixes #26879

Clause-②: no
Metadata-protocol: none

维护者速读

• 改了什么:控制台动作运行器(@object-ui/core 的 ActionRunner)在选派发路由的唯一一处,把「没写 type」的已存动作读成 spec 的默认值 script,于是它走已注册的 script 派发,打平台动作路由 POST /api/v1/actions/{object}/{action}。
• 为什么改:spec 的 ActionSchema.type 默认是 script,但控制台跑的是存储里的原样行、不是解析后的行。Studio「新建」骨架和旧版 Studio 存下的无 type 动作,路由被解析成动作自己的名字,匹配不到执行器,被报「not executable」,点了没反应。
• 风险与回滚:低。只影响「无 type、无 actionType、有名字、名下没注册处理器、也没写运行器自有键(navigate/api/endpoint/onClick/redirect/显式 reload、close)」的动作:原来必然被拒,现在走 script。未知 type、空动作仍拒绝;按名注册的处理器仍优先。回滚即还原 resolveDispatchType 一个函数。
• 先看:ui/packages/core/src/actions/ActionRunner.ts 的 resolveDispatchType 与 declaresRunnerBehavior(新增)以及 execute 里改用它的那一行;ActionRunner.specDefaultType-26879.test.ts(8 个用例)。
• 两分钟验证:pnpm --dir ui exec vitest run packages/core/src/actions/__tests__/ActionRunner.specDefaultType-26879.test.ts,预期 8 个测试全绿。
• 需要你做的:无,席位落地。

What changes

• One place picks the route. execute used to resolve the dispatch key as type || actionType || name. It now calls resolveDispatchType, which keeps the first three legs (an authored type, the action:* block's actionType, the name when a handler is registered under it or the action carries a runner-own key) and, for a named action that has none of those, answers ActionSchema.type's default, script. Nothing unnamed takes the default, so {} still lands in executeActionSchema and is refused.
• The same key is asked twice, as before: by refusalBeforeInput (the dispatch's early refusal, so a typeless script action meets the script dispatch's row guard) and by dispatchCollected.
• declaresRunnerBehavior names the runner-own keys once; executeActionSchema's guard now reads it instead of its own copy (same keys: navigate never reaches that guard because dispatchCollected runs it first).
• Fork taken: the runner, not the serve path. The card offered either the console reading a typeless action as script at route selection, or the serve path returning the parsed row. The runner is in this lane's file surface and covers every unparsed source at once (stored rows from any Studio build or by hand), with no server change. No writer adds type, and no stored row is rewritten.
• @object-ui/core's exports are unchanged: both helpers and the constant are module-private.
• One root changeset, @object-ui/core patch.

Tests

New ui/packages/core/src/actions/__tests__/ActionRunner.specDefaultType-26879.test.ts, 8 cases:

• A typeless { name, body } and a typeless { name, operation: 'update', patch } both POST /api/v1/actions/tasks/NAME through the real createServerActionHandler.
• A typeless action is asked the script dispatch's refuseBeforeInput.
• The default is the spec's: the stored row is parsed with @objectstack/spec's ActionSchema, and with a spy handler under every runnable type the typeless action reaches exactly the handler named by the parsed type. If the spec's default ever moves, this case goes red.
• Not swallowed by the default: an unknown type is refused, {} is refused as "not executable", a handler registered under the action's name runs ahead of script, and a typeless named redirect action still redirects.

The typeless update shape already reached the route through the operation check that runs ahead of type (pinned in ActionRunner.updateOperation.test.ts); it is pinned again here at the new resolution.

Acceptance notes

• The card's two shapes run on the script route; an unknown type and an empty schema are still refused (cases above).
• The New skeleton in ObjectActionsPanel.tsx still leaves type off, as its comment intends; with this change that row runs. Its pinned Type control reading "—" until the first write is untouched (outside this claim's file surface).

Verification

All at head e7e7e357, a worktree off origin/main 8074feb6 (main has moved 27 commits since, none in ui/packages/core/; git merge-tree against it is clean).

No dogfood battery: no ! in the title and no (narrowing) arm.

Generated by Claude Code