Skip to content

New structure#13

Merged
LinusWestling merged 1 commit intomainfrom
feature/restructuring
Mar 26, 2026
Merged

New structure#13
LinusWestling merged 1 commit intomainfrom
feature/restructuring

Conversation

@LinusWestling
Copy link
Copy Markdown
Collaborator

@LinusWestling LinusWestling commented Mar 26, 2026

Summary by CodeRabbit

Revert

  • Application case creation and case management features have been completely removed
  • Audit logging system that tracked all case-related activities and events has been disabled
  • Case event publishing functionality for real-time notifications and updates has been removed
  • Database storage and persistence infrastructure for cases has been removed
  • User permission and authorization system for case operations has been removed

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d2a31816-e53e-4d73-8cc0-e0e35a739661

📥 Commits

Reviewing files that changed from the base of the PR and between 244e585 and 9444017.

📒 Files selected for processing (17)
  • src/main/java/org/example/projektarendehantering/application/ports/AuditLogPort.java
  • src/main/java/org/example/projektarendehantering/application/ports/CaseEventPublisherPort.java
  • src/main/java/org/example/projektarendehantering/application/ports/CaseRepositoryPort.java
  • src/main/java/org/example/projektarendehantering/application/ports/CurrentUserPort.java
  • src/main/java/org/example/projektarendehantering/domain/AuditEvent.java
  • src/main/java/org/example/projektarendehantering/domain/AuditEventType.java
  • src/main/java/org/example/projektarendehantering/domain/Case.java
  • src/main/java/org/example/projektarendehantering/domain/CaseEvent.java
  • src/main/java/org/example/projektarendehantering/domain/CaseId.java
  • src/main/java/org/example/projektarendehantering/domain/CasePermissions.java
  • src/main/java/org/example/projektarendehantering/domain/CaseStatus.java
  • src/main/java/org/example/projektarendehantering/infrastructure/audit/ConsoleAuditLogAdapter.java
  • src/main/java/org/example/projektarendehantering/infrastructure/persistence/InMemoryCaseRepositoryAdapter.java
  • src/main/java/org/example/projektarendehantering/infrastructure/realtime/NoopCaseEventPublisherAdapter.java
  • src/main/java/org/example/projektarendehantering/infrastructure/security/HeaderCurrentUserAdapter.java
  • src/main/java/org/example/projektarendehantering/presentation/dto/CreateCaseRequest.java
  • src/main/java/org/example/projektarendehantering/presentation/dto/CreateCaseResponse.java
💤 Files with no reviewable changes (16)
  • src/main/java/org/example/projektarendehantering/domain/CaseStatus.java
  • src/main/java/org/example/projektarendehantering/presentation/dto/CreateCaseResponse.java
  • src/main/java/org/example/projektarendehantering/application/ports/CurrentUserPort.java
  • src/main/java/org/example/projektarendehantering/infrastructure/audit/ConsoleAuditLogAdapter.java
  • src/main/java/org/example/projektarendehantering/infrastructure/realtime/NoopCaseEventPublisherAdapter.java
  • src/main/java/org/example/projektarendehantering/application/ports/AuditLogPort.java
  • src/main/java/org/example/projektarendehantering/domain/CaseId.java
  • src/main/java/org/example/projektarendehantering/domain/CasePermissions.java
  • src/main/java/org/example/projektarendehantering/application/ports/CaseEventPublisherPort.java
  • src/main/java/org/example/projektarendehantering/application/ports/CaseRepositoryPort.java
  • src/main/java/org/example/projektarendehantering/domain/AuditEvent.java
  • src/main/java/org/example/projektarendehantering/presentation/dto/CreateCaseRequest.java
  • src/main/java/org/example/projektarendehantering/domain/CaseEvent.java
  • src/main/java/org/example/projektarendehantering/domain/Case.java
  • src/main/java/org/example/projektarendehantering/infrastructure/persistence/InMemoryCaseRepositoryAdapter.java
  • src/main/java/org/example/projektarendehantering/domain/AuditEventType.java

📝 Walkthrough

Walkthrough

This PR deletes the entire case management domain layer, removing all port interfaces, domain aggregates, value objects, and infrastructure adapters related to case creation, audit logging, and event publishing.

Changes

Cohort / File(s) Summary
Port Interfaces
AuditLogPort.java, CaseEventPublisherPort.java, CaseRepositoryPort.java, CurrentUserPort.java
Removed four core application ports: audit logging, case event publishing, case persistence, and current user resolution contracts.
Domain Model & Enums
AuditEvent.java, AuditEventType.java, Case.java, CaseEvent.java, CaseId.java, CasePermissions.java, CaseStatus.java
Removed domain aggregates and value objects: case entity with factory methods, event types and records, case identifiers, permissions enforcement, and status tracking.
Infrastructure Adapters
ConsoleAuditLogAdapter.java, InMemoryCaseRepositoryAdapter.java, NoopCaseEventPublisherAdapter.java
Removed three port implementations: console logging adapter, in-memory repository adapter, and no-op event publisher adapter.
HTTP DTOs
CreateCaseRequest.java, CreateCaseResponse.java
Removed request/response records for case creation API endpoints.
Security Adapter
HeaderCurrentUserAdapter.java
Modified class to remove CurrentUserPort interface implementation; retained currentUser() method without @Override annotation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 Hop away, cases old and dear,
Ports and adapters disappear,
Domain models fade to dust,
Fresh foundations—rebuild we must!
~Rabbit Engineer 🔧✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'New structure' is vague and generic, providing no meaningful information about what structural changes were made or why they are important. Replace with a more descriptive title that specifies the key architectural change, such as 'Remove case management ports and adapters' or 'Refactor to remove domain event publishing infrastructure'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/restructuring

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@LinusWestling LinusWestling merged commit ecaa3a6 into main Mar 26, 2026
2 checks passed
@LinusWestling LinusWestling deleted the feature/restructuring branch March 26, 2026 13:31
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