refactor(server): name the area-detail package after what it holds

The package was called reviewhistory but held both halves of the area detail
page: the review history AND the trend query service behind
/practice-areas/{areaSlug}/trend. Its own controller already told the truth —
PracticeAreaDetailController — so the package name was the odd one out, and a
reader opening reviewhistory had no reason to expect a trend service there.

Renamed to areadetail, with a package-info that records where the boundary
runs: the trend QUERY lives here because it answers for a screen, while the
trend MATHEMATICS stays in practices.observation.trend. Nothing outside the
package references any of these types, so the move is closed under the compiler
and no module boundary shifts.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017XBuU2tfPFntHoE34Cmw1Z

Author: Valentin Grüner

…istory/PracticeAreaDetailController.java …detail/PracticeAreaDetailController.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaDetailController.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaDetailController.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaDetailController.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaDetailController.java

@@ -1,7 +1,7 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory;
+package de.tum.cit.aet.hephaestus.practices.areadetail;
 
+import de.tum.cit.aet.hephaestus.practices.areadetail.dto.PracticeAreaReviewMomentDTO;
 import de.tum.cit.aet.hephaestus.practices.observation.trend.dto.PracticeAreaTrendDTO;
-import de.tum.cit.aet.hephaestus.practices.reviewhistory.dto.PracticeAreaReviewMomentDTO;
 import de.tum.cit.aet.hephaestus.workspace.context.WorkspaceContext;
 import de.tum.cit.aet.hephaestus.workspace.context.WorkspaceScopedController;
 import io.swagger.v3.oas.annotations.Operation;

…acticeAreaReviewHistoryFilterParams.java …acticeAreaReviewHistoryFilterParams.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaReviewHistoryFilterParams.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaReviewHistoryFilterParams.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaReviewHistoryFilterParams.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaReviewHistoryFilterParams.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory;
+package de.tum.cit.aet.hephaestus.practices.areadetail;
 
 import de.tum.cit.aet.hephaestus.integration.core.signal.ArtifactKind;
 import de.tum.cit.aet.hephaestus.practices.model.Severity;

…ry/PracticeAreaReviewHistoryService.java …il/PracticeAreaReviewHistoryService.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaReviewHistoryService.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaReviewHistoryService.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaReviewHistoryService.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaReviewHistoryService.java

@@ -1,7 +1,10 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory;
+package de.tum.cit.aet.hephaestus.practices.areadetail;
 
 import de.tum.cit.aet.hephaestus.integration.core.signal.ArtifactKind;
 import de.tum.cit.aet.hephaestus.practices.PracticeAreaService;
+import de.tum.cit.aet.hephaestus.practices.areadetail.dto.PracticeAreaReviewArtifactDTO;
+import de.tum.cit.aet.hephaestus.practices.areadetail.dto.PracticeAreaReviewFindingDTO;
+import de.tum.cit.aet.hephaestus.practices.areadetail.dto.PracticeAreaReviewMomentDTO;
 import de.tum.cit.aet.hephaestus.practices.feedback.FeedbackObservationRepository;
 import de.tum.cit.aet.hephaestus.practices.feedback.FeedbackObservationRepository.DeliveredFeedbackBinding;
 import de.tum.cit.aet.hephaestus.practices.feedback.FeedbackRatingState;
@@ -10,9 +13,6 @@
 import de.tum.cit.aet.hephaestus.practices.model.Severity;
 import de.tum.cit.aet.hephaestus.practices.observation.ObservationRepository;
 import de.tum.cit.aet.hephaestus.practices.observation.ObservationRepository.ReviewHistoryRunRow;
-import de.tum.cit.aet.hephaestus.practices.reviewhistory.dto.PracticeAreaReviewArtifactDTO;
-import de.tum.cit.aet.hephaestus.practices.reviewhistory.dto.PracticeAreaReviewFindingDTO;
-import de.tum.cit.aet.hephaestus.practices.reviewhistory.dto.PracticeAreaReviewMomentDTO;
 import de.tum.cit.aet.hephaestus.practices.spi.CurrentDeveloperLookup;
 import de.tum.cit.aet.hephaestus.practices.spi.ReviewRunTargetLookup;
 import de.tum.cit.aet.hephaestus.workspace.context.WorkspaceContext;

…story/PracticeAreaTrendQueryService.java …etail/PracticeAreaTrendQueryService.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaTrendQueryService.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaTrendQueryService.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaTrendQueryService.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaTrendQueryService.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory;
+package de.tum.cit.aet.hephaestus.practices.areadetail;
 
 import de.tum.cit.aet.hephaestus.practices.PracticeAreaService;
 import de.tum.cit.aet.hephaestus.practices.model.Observation;

…y/dto/PracticeAreaReviewArtifactDTO.java …l/dto/PracticeAreaReviewArtifactDTO.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewArtifactDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewArtifactDTO.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewArtifactDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewArtifactDTO.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory.dto;
+package de.tum.cit.aet.hephaestus.practices.areadetail.dto;
 
 import de.tum.cit.aet.hephaestus.integration.core.signal.ArtifactKind;
 import de.tum.cit.aet.hephaestus.integration.core.spi.IntegrationKind;

…ry/dto/PracticeAreaReviewFindingDTO.java …il/dto/PracticeAreaReviewFindingDTO.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewFindingDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewFindingDTO.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewFindingDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewFindingDTO.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory.dto;
+package de.tum.cit.aet.hephaestus.practices.areadetail.dto;
 
 import de.tum.cit.aet.hephaestus.practices.feedback.FeedbackRatingState;
 import de.tum.cit.aet.hephaestus.practices.model.Assessment;

…ory/dto/PracticeAreaReviewMomentDTO.java …ail/dto/PracticeAreaReviewMomentDTO.javaserver/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewMomentDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewMomentDTO.java server/src/main/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/dto/PracticeAreaReviewMomentDTO.java renamed to server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/dto/PracticeAreaReviewMomentDTO.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory.dto;
+package de.tum.cit.aet.hephaestus.practices.areadetail.dto;
 
 import io.swagger.v3.oas.annotations.media.Schema;
 import java.time.Instant;

server/src/main/java/de/tum/cit/aet/hephaestus/practices/areadetail/package-info.java

@@ -0,0 +1,16 @@
+/**
+ * Read models behind one practice area's detail page: the direction it is heading and the review moments
+ * that produced its feedback. Both halves are served by a single controller under
+ * {@code /practice-areas/{areaSlug}}, which is why they share a package rather than sitting with the
+ * machinery they read from.
+ *
+ * <p>The trend QUERY service lives here; the trend MATHEMATICS does not. Posterior estimation, bundling and
+ * classification stay in {@code practices.observation.trend} — this package only asks that module a
+ * question on behalf of a screen, scoped to one area. Naming the package after the review history alone
+ * hid that second half.
+ *
+ * <p>Deliberately not a named interface: nothing outside reads these types. Everything the rest of the
+ * application needs about an area's standing comes through {@code practices.observation}, and keeping this
+ * package closed is what lets a screen's shape change without a module boundary moving.
+ */
+package de.tum.cit.aet.hephaestus.practices.areadetail;

…eAreaTrendControllerIntegrationTest.java …eAreaTrendControllerIntegrationTest.javaserver/src/test/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaTrendControllerIntegrationTest.java renamed to server/src/test/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaTrendControllerIntegrationTest.java server/src/test/java/de/tum/cit/aet/hephaestus/practices/reviewhistory/PracticeAreaTrendControllerIntegrationTest.java renamed to server/src/test/java/de/tum/cit/aet/hephaestus/practices/areadetail/PracticeAreaTrendControllerIntegrationTest.java

@@ -1,4 +1,4 @@
-package de.tum.cit.aet.hephaestus.practices.reviewhistory;
+package de.tum.cit.aet.hephaestus.practices.areadetail;
 
 import de.tum.cit.aet.hephaestus.integration.scm.domain.user.User;
 import de.tum.cit.aet.hephaestus.practices.PracticeAreaRepository;