T-ZRO1-hoist-the-duplicated-migrationerror-class-to-one-shared-home
State: closed/done · Impact: low · Complexity: small
From the Phase-4 audit’s recommended task list
(docs/planning/notes/N-OCUK-2026-09-24-sdlc-1-0-audit.md, on origin/sdlc/p4-audit): Hoist the
duplicated MigrationError class to one shared home; extract a shared
reviewFindingSchemas(entityNoun) factory.
| Area | Note |
|---|---|
lib/model/entities/ | MigrationError (capability + task) |
lib/services/report/ | principle/standard review-report schemas |
Acceptance criteria
Section titled “Acceptance criteria”- AC-1: One MigrationError class is imported by both capability and task entities.
- AC-2: A shared reviewFindingSchemas(entityNoun) factory replaces the duplicated principle/standard schema definitions.
Out of scope
Section titled “Out of scope”- none
Dependencies
Section titled “Dependencies”- none