feat: register import auto-create-or-fail-clearly (#1487)

[rubenvdlinde]

Summary

• ImportService::importFromJson() threw a bare "requires a specific schema" error whenever the target register did not exist (Register import workflow: auto-create register from JSON or fail clearly #1487) — a register bundle (register + schemas + objects) could never be moved into an instance that lacked the register.
• When the caller cannot resolve the target register by slug, the payload is now checked for a register-bundle envelope (components.registers + components.schemas, the same shape ConfigurationService::importFromApp() understands): a bundle auto-creates the register + schemas by reusing ImportHandler::importRegister()/importSchema() (no duplicated creation logic, ADR-011), then imports the bundle's objects; re-import is idempotent (both methods look the register/schema up by slug first).
• A payload that is not a creatable bundle now fails with a RegisterNotFoundException naming the missing slug and the two remedies (create the register first, or supply a full bundle) — the controller's existing catch-all maps this to an actionable 400, never a 500.
• RegistersController::import() no longer 404s immediately on a missing register; a JSON upload from an admin is let through (register: null) so the bundle can auto-create — every other case (non-admin, or a non-JSON/non-bundle upload) keeps the existing 404.

Implements openspec/changes/register-import-auto-create (REQ-IMP-AC-01/02/03), archived in this PR.

Test plan

• ImportServiceRegisterAutoCreateTest: bundle for a non-existent register → register + schema created, object imported; re-import → no duplicate register (createFromArray asserted once()); plain object list for a missing register → clear error naming the slug, 4xx-range code.
• RegistersControllerTest: admin + JSON register-bundle for a missing register passes through to the importer; non-admin same request still 404s; non-JSON (Excel) upload for a missing register still 404s unchanged.
• Full existing ImportService/ImportHandler/RegistersController unit suites pass locally (nextcloud:34.0.0-apache container, PHPUnit) with no regressions (verified against an unmodified origin/development baseline — identical pre-existing failure count in both).
• phpcs/phpmd clean on all three changed lib/ files.
• openspec validate --all --strict — spec/data-import-export passes; no new failures introduced.

Note: this repo's CI has known pre-existing infra failures (npm lockfile, eslint/stylelint/license, Newman) unrelated to backend PHP changes.

[github-actions]

Quality Report — ConductionNL/openregister @ 402ab62

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	❌
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		❌
stylelint		❌
composer			❌	✅ 172/172
npm			❌	❌
PHPUnit					⏭️
Newman					⏭️
Playwright					⏭️

---

Quality workflow — 2026-07-23 22:08 UTC

Download the full PDF report from the workflow artifacts.