Skip to content

AB#3268: add journal drift verb - #508

Merged
PolyphonyRequiem merged 1 commit into
architecture/rationalizationfrom
arch-rat/journal-phase-4
May 22, 2026
Merged

AB#3268: add journal drift verb#508
PolyphonyRequiem merged 1 commit into
architecture/rationalizationfrom
arch-rat/journal-phase-4

Conversation

@PolyphonyRequiem

Copy link
Copy Markdown
Owner

Summary

  • add Phase 4 polyphony journal drift command wiring and result model
  • project expected journal resource state and diff it with read-only observers
  • cover the new verb with projection, command, DI, and JSON contract tests

Testing

  • dotnet build src\Polyphony\Polyphony.csproj -c Release
  • dotnet build tests\Polyphony.Tests\Polyphony.Tests.csproj -c Release
  • dotnet test tests\Polyphony.Tests -c Release --filter "FullyQualifiedNameJournalDrift|FullyQualifiedNameJsonOutputContractTests|FullyQualifiedNamePolyphonyServiceRegistrationTests|FullyQualifiedNameVerbCoverageTests"
  • dotnet test tests\Polyphony.Tests --no-build -c Release --filter "FullyQualifiedNameJournal|FullyQualifiedNamePolyphonyServiceRegistrationTests|FullyQualifiedName~VerbCoverageTests"

Notes

  • full-suite validation is still blocked by two unrelated issues in this worktree: missing artifacts\verb-output-schemas.json and LifecycleRouterScriptTests failing to launch pwsh

AB#3268

Implement Phase 4 journal drift analysis and CLI wiring.

- project expected state from journal resource effects
- add read-only observers with explicit deferred kinds
- wire the drift verb, DI, JSON context, and tests

AB#3268

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@PolyphonyRequiem
PolyphonyRequiem merged commit e080468 into architecture/rationalization May 22, 2026
@PolyphonyRequiem
PolyphonyRequiem deleted the arch-rat/journal-phase-4 branch May 22, 2026 06:57
PolyphonyRequiem pushed a commit that referenced this pull request May 22, 2026
Drift verb landed on architecture/rationalization. Phase 5 (rewrite
reset root against OwnedResources projection) is next. Notably AB#3245
and AB#3246 already shipped to main as targeted fixes via PR #507, so
the Phase 5 closure criteria for those two items is partially met
ahead of schedule.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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