fix(objectql): refuse undeclared insert fields at the schema, and keep bound values out of the write-path logs (#8682)

[hotlong]

Fixes #8682

Both halves of the card land. Verified at d99df41.

Half A — validation ordering

An undeclared write key is now refused by the object's field map as the first act inside insert()'s middleware body — after middleware (which may legitimately rewrite data), before applyFieldDefaults, and therefore before the defaults, the summary seeding, the beforeInsert hooks, the secret writes, validation, the autonumber and the statement.

Measured on origin/main @ 3508678 with a real engine and a recording driver, one mistyped key on a single insert:

observable	before	after
autonumber around the refused request	0001 → refused → 0003	0001 → refused → 0002
beforeInsert hook runs	ok-1, bad, ok-2	ok-1, ok-2
driver create() calls	3	2
refusal envelope	SQLITE_ERROR, no status	INVALID_FIELD + 400

The hook did not merely run — its derived value reached the statement, which is the card's period_label = 'Q3 2026' shape: a hook is not a pure function, so "it ran and we threw the result away" is a side effect of a request the server had already decided to refuse.

The wire answer is deliberately unchanged. The refusal carries code: 'INVALID_FIELD', status: 400, field and object, and its message is byte-identical to the one mapDataError's driver-string branch produced. @objectstack/rest re-emits that verbatim, so the caller sees exactly what it saw before — the refusal moved, the answer did not.

Where the door deliberately has no opinion (each measured, not assumed — all three surfaced as test failures in the first full run and are now pinned as controls):

• schema.fields absent or empty — an empty map is indistinguishable from an unpopulated one, so refusing everything on it would be a verdict made from an absence.
• id / created_at / updated_at — tolerated even when a declaration omits them, mirroring the three names find() / findOne() already add to their known set, so a key accepted by a read is not refused by a write.
• Schema drift — a DECLARED field whose physical column is missing is invisible to this door by construction and stays the driver's to refuse.

In all three the driver remains the backstop it has always been; nothing is widened.

insertMany culls such a row per row instead of failing the batch around it — before this, an undeclared key defeated partial-success mode completely, because the row travelled to bulkCreate with the rest and took the whole batch down with it.

Half B — value logging

Logger.error(msg, error, meta) serializes exactly error.message and error.stack, and a knex-shaped driver error opens both with the fully bound statement. Confirmed with the card's canaries on origin/main: message and stack each carried SENSITIVE-CANARY-9f3a2b and the row's other values — so redacting one and not the other would have moved the leak rather than closed it. Both are rebuilt.

What is kept, deliberately: the level (error), the message (Insert operation failed), the object meta, the database's own diagnostic — which names the failing column — and every stack frame. The card's triage is explicit that the failing column and object must still be logged, and this narrows only WHAT is written.

The cut is structural rather than lexical: @objectstack/types' looksLikeInternalErrorLeak already owns "is this a driver dump?" for both HTTP boundaries, so this asks it the verdict and adds only the one thing it does not answer — where the statement ends. Every inlined value sits in the statement and the statement always comes first, so the last - separates the part that may carry values from the part that may not. Last rather than first because a bound value may itself contain -; cutting early would leave a fragment of that value standing in what we then log as "the diagnostic". A dump with no separator carries no statement and is returned untouched, same reference.

The rethrown error is untouched. mapDataError reads the driver's raw message to answer 400 INVALID_FIELD with the failing field, so the redaction applies to the log slot only — one argument at one call site.

One scope call worth a reviewer's eye

The card measured the INSERT logger. The Update operation failed and Delete operation failed loggers three screens away are the same one-argument call, the same knex-shaped error and the same security consequence — an UPDATE inlines the caller's values in its set clause exactly as an INSERT does in its values list. I applied the same redaction to all three and pinned the update and delete phrasings, rather than leave two thirds of a security-labelled hole open for a later round. Happy to trim it to insert-only if you would rather keep the diff on the card's exact letter.

The first sub-claim was already fixed, and I confirmed it

The card reports the false [REST] Unhandled error label as FIXED upstream. Confirmed on current main: packages/rest/src/rest-expected-error-logging.test.ts pins exactly that ("[#4886] Expected 4xx must not be logged as [REST] Unhandled error"). Nothing was needed for it, so this PR implements less than the card describes for that one row.

Fixture triage

The first full run of the objectql suite went from 3591 green to 49 failures, every one an accept-set narrowing. Triaged individually rather than batch-re-spelled:

• 15 × title on task, 1 × customer on sys_license — fixtures whose registry stub is fields: {}. Fixed in the DOOR, not the fixtures: an empty field map now yields no verdict, which is the correct rule and not a test accommodation.
• 1 × id on bf_leave_request — the fixture declares neither id nor the audit columns. Fixed in the door too, mirroring the read path's existing tolerance.
• 24 × package_id on sys_metadata — four minimal stubs omitted a column the REAL object declares (metadata-core/src/objects/sys-metadata.object.ts:65, part of the row's uniqueness key) and that SysMetadataRepository writes. The production writer is correct; the stubs were never spec-valid, and nothing noticed while an undeclared key simply travelled on to the driver. Declarations added.
• 1 × engine-insert-many — the bad row was spelled { slug: 'no-name' } to mean "a row missing the required name", but slug is itself undeclared, so the door refused it before the hooks and the assertion would have measured the wrong refusal. Re-spelled to {}, which says the one thing intended.

No production code outside engine.ts needed a change, and no undeclared-write finding survived triage as a real defect.

Verification — at d99df41

• pnpm --filter @objectstack/objectql test — 205 files, 3617 tests, 0 failures
• pnpm --filter @objectstack/objectql typecheck — clean
• Downstream consumer sweep, pnpm --workspace-concurrency=2 --no-bail --filter '...@objectstack/objectql' test (prefix form = the 43 packages that DEPEND on objectql), after a full pnpm build: 43/43 green, zero Unknown field refusals anywhere — packages/qa/dogfood included, which boots real example apps end to end. That sweep is the evidence that the narrowed accept set costs no live caller anything.
• Reverse verification — engine.ts reverted to origin/main, pins re-run, then restored and proved byte-identical (git hash-object == the committed blob). The direction is deliberately not uniform, and that is the point: 8 of 10 half-A cases go red (expected '0003' to be '0002'; hooks ['ok-1','bad','ok-2']; 1 driver create), while the two half-A controls stay green — they assert the door has NO opinion, so they must pass on origin/main too. On half B exactly one engine-level case flips (the canary case); "the entry survives" and "the failing column is still named" stay green, because origin/main also logged at ERROR with the object and the column — it just also logged the values.
• Gates re-derived from the actual diff with scripts/pm/dispatch-gates.mjs, all green: check:durability-log-level, check:stack-collection-maps, check-engine-split-ratio, plus seven the dispatch list did not name — check:nul-bytes, check:changeset-gate-self-tests, check:objectui-changeset, check:query-options-erasure, check:type-check-coverage, check:type-check-debt (the --re-measure ratchet, 33 entries, none above ceiling), and the three changeset scripts.

The regression pin is the log-independent observable the triage asked for — the autonumber gap across the rejected request. No test in this PR greps log contents for half A, precisely because half B rewrites the logger in the same commit.

---

Generated by Claude Code

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
objectstack	Ignored		Aug 14, 2026 5:01pm

[github-actions]

📓 Docs Drift Check

This PR changes 1 package(s): @objectstack/objectql.

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

• content/docs/concepts/metadata-lifecycle.mdx (via @objectstack/objectql)
• content/docs/data-modeling/formulas.mdx (via packages/objectql)
• content/docs/deployment/migration-from-objectql.mdx (via @objectstack/objectql)
• content/docs/deployment/vercel.mdx (via @objectstack/objectql)
• content/docs/kernel/contracts/data-engine.mdx (via @objectstack/objectql)
• content/docs/kernel/runtime-services/examples.mdx (via packages/objectql)
• content/docs/kernel/services-checklist.mdx (via @objectstack/objectql)
• content/docs/kernel/services.mdx (via @objectstack/objectql)
• content/docs/permissions/authentication.mdx (via @objectstack/objectql)
• content/docs/permissions/system-context.mdx (via packages/objectql)
• content/docs/plugins/index.mdx (via @objectstack/objectql)
• content/docs/plugins/packages.mdx (via @objectstack/objectql)
• content/docs/protocol/kernel/index.mdx (via @objectstack/objectql)
• content/docs/protocol/objectql/query-syntax.mdx (via packages/objectql)
• content/docs/protocol/objectql/state-machine.mdx (via @objectstack/objectql)

⛔ 1 release-owned page(s) also reference the affected code. These are read-only:

• content/docs/releases/implementation-status.mdx (via @objectstack/objectql)

content/docs/releases/ is RELEASE-OWNED (AGENTS.md "Documentation Guardrails"): release
notes are written centrally at release time, and a code PR that edits them is the exact PR
that guardrail exists to stop. They are still audited — read-only. If one of them is actually
wrong, file an issue or open a dedicated docs-only PR; do not edit it here.

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.