fix(ui-core): ActionDef stops mirroring the aged-out shortcut and bulkEnabled action keys

CI's Build Core went red on the merged head in ui/packages/core: `ActionDef`
declared `shortcut?: SpecActionInput['shortcut']` and `bulkEnabled?:
SpecActionInput['bulkEnabled']`, two mirrors of spec-17 `retiredKey()`
tombstones that resolved to `undefined` for as long as the tombstones
stood. The ≤16.x kits left the schema one major on, so the indexed access
has nothing to name and tsc refuses it. The two members leave `ActionDef`
with their rows in `ACTION_DEF_KEYS` and `SPEC_ACTION_KEYS` (the key
inventories the pin tests hold against the spec's `ActionSchema` shape),
and `SPEC_ACTION_KEYS` drops its `execute` row for the same reason: the
pin that walks the spec shape reported it stale once that tombstone aged
out. `RETIRED_ACTION_KEYS` keeps the console's own `execute` rename
prescription (a stored row can reach the runner unparsed), and
`ActionEngine.registerAction(action, { shortcut, bulkEnabled })` keeps
both as host-passed options, as before; the closed-surface pin's comment
and the tombstone test's title say what refuses the key now.

Claude-Session: https://claude.ai/code/session_01H37vtTJRVBf1LkyYNW5yfo
Co-authored-by: Claude <noreply@anthropic.com>

Author: claude

ui/packages/core/src/actions/ActionRunner.ts

@@ -502,22 +502,14 @@ export interface ActionDef {
    * active language before dispatch, because the runner has no language.
    */
   outcomeMessages?: SpecActionInput['outcomeMessages'];
-  /**
-   * @deprecated Retired in `@objectstack/spec` 17 as a `retiredKey()` tombstone —
-   * authoring it is a hard parse rejection, so this resolves to `undefined` and
-   * assigning a value is a compile error. A HOST may still pass a shortcut
-   * explicitly via `ActionEngine.registerAction(action, { shortcut })`; it is
-   * only no longer sourced from authored metadata. No replacement key.
-   */
-  shortcut?: SpecActionInput['shortcut'];
-  /**
-   * @deprecated Retired in `@objectstack/spec` 17 as a `retiredKey()` tombstone —
-   * authoring it is a hard parse rejection, so this resolves to `undefined` and
-   * assigning a value is a compile error. Use the list view's `bulkActions` /
-   * `bulkActionDefs` instead; `ActionEngine.registerAction(action, { bulkEnabled })`
-   * remains available to a HOST passing it explicitly.
-   */
-  bulkEnabled?: SpecActionInput['bulkEnabled'];
+  // No `shortcut` and no `bulkEnabled`: spec 17 retired both as `retiredKey()`
+  // tombstones and this interface mirrored each as `SpecActionInput['…']`
+  // (always `undefined`) until objectstack#25767 aged the ≤16.x kits out of
+  // the schema one major on, which left nothing to mirror. A HOST still passes
+  // either one explicitly through `ActionEngine.registerAction(action,
+  // { shortcut, bulkEnabled })`; neither is sourced from authored metadata,
+  // and a bulk action is declared in the list view's `bulkActions` /
+  // `bulkActionDefs`.
 }
 
 /**

ui/packages/core/src/actions/__tests__/actionDef-closed-surface.test.ts

@@ -67,7 +67,7 @@ const _typoTarget: ActionDef = { type: 'script', targt: 'saveRecord' };
 
 // A retired SPEC key that is not a tombstone, to show the closure is general
 // rather than a list of two special cases.
-// @ts-expect-error - `bulkEnabled` is a retiredKey() tombstone in spec 17
+// @ts-expect-error - `bulkEnabled` is not an `ActionDef` key: its spec-17 tombstone aged out at objectstack#25767 and the mirror left with it
 const _retiredBulk: ActionDef = { type: 'api', bulkEnabled: true };
 
 // ── The unified `visible` / `disabled` shape: three arms each ────────────────

ui/packages/core/src/actions/__tests__/actionKeys.pin.test.ts

@@ -140,7 +140,7 @@ describe('action key inventory (objectstack#4075 step 1)', () => {
     expect({ missing, stale }).toEqual({ missing: [], stale: [] });
   });
 
-  it('`execute` is still a live spec tombstone, so it must not count as known', () => {
+  it('`execute` is still refused by the spec (the 17 tombstone aged out at objectstack#25767; the strict unknown-key refusal took over), so it must not count as known', () => {
     const parsed = SpecActionSchema.safeParse({
       name: 'mark_done',
       label: 'Mark Done',

ui/packages/core/src/actions/actionKeys.ts

@@ -199,8 +199,8 @@ export const ACTION_DEF_KEYS = [
   // Spec 17.6.0's success copy per handler outcome (objectui#11344), declared
   // when the runner learned to read it: the success toast's first rung.
   'outcomeMessages',
-  'shortcut',
-  'bulkEnabled',
+  // `shortcut` and `bulkEnabled` left with their spec tombstones
+  // (objectstack#25767): `ActionDef` no longer declares either.
 ] as const;
 
 /**
@@ -234,7 +234,6 @@ export const SPEC_ACTION_KEYS = [
   'body',
   'bodyExtra',
   'bodyShape',
-  'bulkEnabled',
   'component',
   'confirmText',
   // Added to `ActionSchema` in @objectstack/spec 17.0.0-rc.6. Listed here purely
@@ -248,7 +247,12 @@ export const SPEC_ACTION_KEYS = [
   'description',
   'disabled',
   'errorMessage',
-  'execute',
+  // No `execute`: its spec-17 tombstone (`retiredKey()`, objectstack#3855) aged
+  // out of `ActionSchema` at objectstack#25767 — the spec now refuses it as an
+  // unknown key naming the protocol upgrade guide — so the spec no longer
+  // declares it and this inventory, which restates the spec's declared keys,
+  // drops the row. `RETIRED_ACTION_KEYS` below still carries the console's own
+  // rename prescription: a stored row can reach the runner unparsed.
   // Added to `ActionSchema` in @objectstack/spec 17.5.0 (objectui#11073): the
   // bulk dispatch contract the action's body is written for, in
   // `bulkActionDefs`' own `'perRecord' | 'aggregate'` vocabulary. Listed for the
@@ -310,7 +314,6 @@ export const SPEC_ACTION_KEYS = [
   // question.
   'requiresMembershipReach',
   'resultDialog',
-  'shortcut',
   'successMessage',
   'target',
   'type',