refactor(service-automation): deprecate non-canonical CRUD node config-key aliases

[os-zhuang]

Audit: "lenient input" config-key aliases in automation node executors

FlowNodeSchema.config is z.record(z.string(), z.unknown()) (flow.zod.ts:102), so the spec blesses no particular config key — the executor is the only runtime gate on key names. That makes every cfg.canonical ?? cfg.alias in a node executor a Postel's-law tolerance that fossilizes a wrong shape into a de-facto second contract and hides metadata-generation bugs. This PR is Step 1 of converging them: it follows the fields/fieldValues precedent (reject the wrong shape at author/publish time; don't absorb it at runtime).

What this PR changes (clear-cut, zero-risk)

• readAliasedConfig() helper — returns the canonical value; on the alias path, emits a one-time logger.warn naming the canonical key, then keeps working. (graph-lint runs only at publish, so it can't reach flows already stored in prod that are never re-published — the runtime warn covers that population.)
• Wired into get/create/update/delete_record for objectName and filter.
• Fixed the one authoring source that taught the alias: skills/objectstack-automation/SKILL.md update_record example object: → objectName: (every example .flow.ts already uses objectName/filter).

No alias is removed — all are plausibly present in stored prod flows. Removal is a tracked follow-up gated on (a) the warn window and (b) a cloud graph-lint rule.

Findings table (full audit)

Canonical is established by docs/skill/examples + cloud graph-lint (the spec is z.record, vacuously permissive). "Debt" = one canonical key + tolerated alias; "Union" = spec genuinely blesses both.

#	Alias (file:line)	Spec says	Class	Action
1	crud-nodes.ts objectName ?? object (get/create/update/delete)	z.record (any key)	Debt — objectName canonical (all 8 example flows); object only in SKILL.md	✅ this PR: warn + fix skill
2	crud-nodes.ts filter ?? filters (get/update/delete)	z.record	Debt — filter canonical; filters zero usage	✅ this PR: warn
3	notify-node.ts:75-77 recipients ?? to, title ?? subject, message ?? body	z.record	Debt — email-flavored aliases; notify unused in examples	Follow-up: same warn treatment
4	screen-nodes.ts:198 inputs ?? input (script)	z.record	Debt — singular slip; zero usage	Follow-up: warn
5	screen-nodes.ts:135 function ?? functionName (script)	z.record	Debt — deliberate (in-code: "AI/templates commonly emit it", #1870)	Fix AI/skill emission first, then warn (platform self-emits it today)
6	wait-node.ts:61,108 … ?? loose.duration, … ?? loose.signal	waitEventConfig.timerDuration/signalName (typed)	Debt — off-spec short aliases; zero usage	Follow-up: remove/warn (low traffic)
7	wait-node.ts waitEventConfig.* ?? config.*	spec defines waitEventConfig; examples author under config	Structural dual-home (borderline)	Pick one home + document; lower priority
8	logic-nodes.ts:71 variable ?? name ?? key (assignment)	z.record (+ documented 3-surface normalizer)	Debt slice in a legit multi-shape normalizer	Keep normalizer; warn on name/key array-item aliases
9	action.zod.ts:529 p.name ?? p.field (+ refine :70)	.refine(name || field) blesses both	Legitimate union	✅ Leave as-is
10	http-dispatcher.ts:2836 ctxBody.objectName ?? ctxBody.object (API POST /:name/trigger)	n/a (HTTP body)	Debt — same as #1, on the trigger surface	Converge with #1 (separate PR; runtime pkg)
11	http-dispatcher.ts:2892 inputs ?? variables (resume body)	n/a	Debt (lean) / low traffic	Follow-up

Out of scope (separate convergence tracks — flagged, not folded in)

• Query-DSL multi-spelling — metadata-protocol/protocol.ts:2060,2077 (orderBy ?? sort, filter ?? filters ?? $filter ?? where) and http-dispatcher.ts:1792 (where ?? filter ?? filters): mostly a legit OData/ObjectQL union; filters plural is the debt slice. Owner: ObjectQL query options.
• REST camelCase↔snake_case — rest-server.ts (~40 sites, e.g. body.object ?? body.object_name): a deliberate, documented REST body convention. Leave.
• DB row column reconciliation — plugin-approvals/approval-service.ts (raw.flow_node_id ?? raw.current_step), auth/security *_id ?? *Id: reading persisted rows, not author config. Separate (legacy column) concern.
• Spec-ambiguous — service-datasource/default-datasource-driver-factory.ts:92 user ?? username: datasource.zod prose says user, the postgres/mongo driver schemas say username. Fix the spec first (pick one), then converge — not clean runtime debt.

Why deprecation window, not straight removal

The spec is z.record, so the runtime has been the only gate; any tolerated alias may sit in a stored prod flow and there's no telemetry to prove otherwise. Several are documented/intuitive (object is literally in the skill; to/subject read as email; filters/input/duration are natural slips). Removal's failure mode is loud-and-safe (create_record: objectName required) rather than corruption — but it's still an availability regression for running flows, which is exactly what the window avoids. So: warn now, remove after the window + a graph-lint rule, and stop teaching/emitting the alias at the source.

Tests

• @objectstack/service-automation: 225 pass (incl. new crud-config-aliases.test.ts — alias still resolves, warns once per alias, canonical path silent).
• @objectstack/spec: 6600 pass.
• tsup ESM/CJS/DTS build clean.

Suggested follow-ups

1. Apply the same readAliasedConfig warn to notify/screen/wait/logic aliases (Implement ObjectStack protocol specification with Zod schemas and TypeScript interfaces #3,4,6,8) and the http-dispatcher trigger object (chore: version packages #10).
2. Add the cloud graph-lint rule rejecting non-canonical CRUD config keys at publish.
3. File the removal-tracking issue; remove alias paths after one release with the warn + graph-lint enforcing.
4. Resolve the datasource user/username spec ambiguity before touching that runtime tolerance.

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
spec	Ready	Preview, Comment	Jun 28, 2026 11:00am

[github-actions]

📓 Docs Drift Check

This PR changes 1 package(s): packages/services.

5 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:

• content/docs/guides/packages.mdx (via packages/services)
• content/docs/guides/runtime-services/audit-service.mdx (via packages/services)
• content/docs/guides/runtime-services/index.mdx (via packages/services)
• content/docs/guides/runtime-services/settings-service.mdx (via packages/services)
• content/docs/protocol/objectos/i18n-standard.mdx (via packages/services)

Advisory only. To re-verify, run the docs-accuracy-audit workflow scoped to these files:
node scripts/docs-audit/affected-docs.mjs origin/main → pass the list as args.docs.