feat(flow): object-write can pass an item through unwritten (skipWhen)

[rubenvdlinde]

Why filter cannot do this job

The decomposed synchronization now decides skip for an unchanged object and commits zero contracts on a re-run. It still rewrote every target object, because a skipped item has to keep flowing into object-write.

openregister.filter would DROP it — and openconnector.contract-sweep decides what to delete from the target ids its items carry. So dropping an unchanged item is exactly what makes the next sweep delete the object that was fine. That hazard is why skips were still being written.

And reaching the write at all moves @self.updated, because SaveObject::updateObject() stamps it unconditionally.

What this adds

skipWhen — a dot-path on the item. When it resolves to true or the string skip, that item is emitted unchanged and unwritten: no write, no cap consumption, position and identity preserved. Honoured by both the per-item and the bulk path; an all-skipped bulk page never calls saveObjects() at all.

Absent, behaviour is byte-identical to today. This node is heavily guarded and widely used, so the guard is strictly opt-in.

Truthiness is deliberately narrow: true, or the string skip — the vocabulary the contract step already stamps. "false" and "0" are truthy in PHP and must not silence a write; there is a test for exactly that.

Verification

Six new cases, all positive-controlled: neutering isSkipped() back to the pre-fix behaviour fails 5 of the 8 skip tests.

581 flow tests green. phpcs, phpmd and phpstan clean — phpstan positive-controlled with a deliberate strlen(array) to prove the file is genuinely analysed rather than silently excluded (this repo has excludePaths that have produced exactly that false [OK]).

writeBulk() crossed phpmd's length and complexity thresholds once the skip branch landed, so the row planning and the pass-through were extracted into their own methods rather than baselined.

Follow-up (openconnector)

SynchronizationFlowGenerator should then emit skipWhen: "contract.outcome" on its object-write node, which completes task 2.3's zero-write re-run. openconnector #1297's partial-idempotency caveat and #1296's test.fail() can both be revisited once that lands.

🤖 Generated with Claude Code

[github-actions]

Quality Report — ConductionNL/openregister @ a71142f

Check	PHP	Vue	Security	License	Tests
lint	⏭️
phpcs	⏭️
phpmd	⏭️
psalm	⏭️
phpstan	⏭️
phpmetrics	⏭️
eslint		⏭️
stylelint		⏭️
build		⏭️
composer			⏭️	⏭️
npm			⏭️	⏭️
app:check-code	⏭️
info.xml	⏭️
REUSE	⏭️
PHPUnit					❌
Newman					❌
Playwright					❌
Hydra gates					❌

---

Quality workflow — 2026-08-19 22:12 UTC

Download the full PDF report from the workflow artifacts.

[github-actions]

Quality Report — ConductionNL/openregister @ c25f7b3

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-specs		✅
test-l10n		✅
format		✅
composer			✅	✅ 175/175
npm			✅	✅ 528/528
app:check-code	⏭️
info.xml	✅
REUSE	❌
PHPUnit					✅
Newman					✅
Playwright					✅
Hydra gates					✅

---

Quality workflow — 2026-08-19 23:05 UTC

Download the full PDF report from the workflow artifacts.