Skip to content

Phase 1.2 — Core services: AuditRecorder + FieldDiffService + HandlerRegistry#91

Draft
sjelfull wants to merge 1 commit into
foundation-enum-jsonfrom
core-services-recorder
Draft

Phase 1.2 — Core services: AuditRecorder + FieldDiffService + HandlerRegistry#91
sjelfull wants to merge 1 commit into
foundation-enum-jsonfrom
core-services-recorder

Conversation

@sjelfull
Copy link
Copy Markdown
Owner

@sjelfull sjelfull commented Apr 26, 2026

Part of the v3 refactor stack for FRE-56. Closes FRE-125.

Diff: 12 files changed, 896 insertions(+), 2 deletions(-)
Tests: 69 → 69 (+0 (refactor))
Verified on fresh DB: DROP DATABASE; pest green

Stack navigation

↑ Builds on #90 foundation-enum-json (P1.1)
↓ Followed by #92 field-handlers-native-third-party (P1.3)

Review notes

  • Single-commit branch — review the commit, not just the diff
  • BC preserved — old AuditService API still works via @deprecated delegators
  • This is a draft — not a request to merge yet. Marked ready when the stack is approved end-to-end.

@sjelfull
Copy link
Copy Markdown
Owner Author

📚 Stack navigation

Part of the FRE-56 v3 refactor — 12-branch stack via git-spice. Merge in order: bottom-up, each PR rebases the next.

# Stage Title
#90 P1.1 Foundation — enum + JSON migration
#91 P1.2 Core services — recorder, diff, registry
#92 P1.3 15 field handlers
#93 P1.4 Project config tracking
#94 P1.5 UI — diff renderer + 20 templates
#95 P1.6 Tests — +19 coverage
#96 P2.1 Decompose — Element/User/UserGroup handlers
#97 P2.2 Decompose — 5 more handlers
#98 P2.3 Repoint event wiring
#99 P2.4 Tests — migrate to enum
#111 P2.5 CI — PHP 8.2/8.3/8.4 matrix
#110 P3.0 Docs — escape hatch (filter-records, getting-started, events)

Quick jump

#90 P1.1#92 P1.3


📌 Reviewing: focus on the single commit. BC preserved via @deprecated delegators. Tests: 49 → 131 across the full stack, fresh-DB-verified per branch. CI: ECS + PHPStan green per branch.

@sjelfull
Copy link
Copy Markdown
Owner Author

sjelfull commented Apr 28, 2026

This change is part of the following stack:

Change managed by git-spice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant