Skip to content

Commit 2e99d22

Browse files
refactor: remove redundant review comments
1 parent 828f89f commit 2e99d22

6 files changed

Lines changed: 1 addition & 8 deletions

File tree

server/src/main/java/de/tum/cit/aet/hephaestus/agent/handler/DiffNotePoster.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,10 @@
2020
import org.slf4j.Logger;
2121
import org.slf4j.LoggerFactory;
2222

23-
/** Reconciles sanitized inline observations through the provider-specific channel. */
2423
class DiffNotePoster {
2524

2625
private static final Logger log = LoggerFactory.getLogger(DiffNotePoster.class);
2726

28-
/** Invisible marker appended to diff note bodies to identify hephaestus-posted notes. */
2927
static final String HEPHAESTUS_MARKER = "<!-- hephaestus-diff-note -->";
3028

3129
private final PullRequestCommentPoster commentPoster;
@@ -85,7 +83,6 @@ private DiffNoteResult reconcileInlineNotes(AgentJob job, List<DiffNote> diffNot
8583
packageId
8684
);
8785

88-
// An empty reconcile clears stale notes; non-empty channels reconcile by recurrence key.
8986
if (observations.isEmpty()) {
9087
try {
9188
channel.clearStaleFeedback(target, HEPHAESTUS_MARKER);

server/src/main/java/de/tum/cit/aet/hephaestus/agent/handler/FeedbackLedgerRecorder.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@
4848
import org.springframework.transaction.annotation.Propagation;
4949
import org.springframework.transaction.annotation.Transactional;
5050

51-
/** Persists feedback units and their provider placement handles. */
5251
@Component
5352
public class FeedbackLedgerRecorder {
5453

server/src/main/java/de/tum/cit/aet/hephaestus/agent/job/DeliveryStatus.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package de.tum.cit.aet.hephaestus.agent.job;
22

3-
/** Aggregate delivery outcome for a job, including lanes without dispatch rows. */
43
public enum DeliveryStatus {
54
PENDING,
65
DELIVERED,

server/src/main/java/de/tum/cit/aet/hephaestus/practices/feedback/Feedback.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@ public class Feedback {
8888
@Column(name = "delivery_state", nullable = false, length = 32)
8989
private FeedbackDeliveryState deliveryState;
9090

91-
/** Why delivery stopped. Set on withheld or terminally partial feedback. */
9291
@Enumerated(EnumType.STRING)
9392
@Column(name = "suppression_reason", length = 32)
9493
private @Nullable FeedbackSuppressionReason suppressionReason;

server/src/main/java/de/tum/cit/aet/hephaestus/practices/feedback/FeedbackRepository.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,6 @@ Optional<UUID> findLatestOnThread(
262262
)
263263
int markSuperseded(@Param("workspaceId") Long workspaceId, @Param("id") UUID id);
264264

265-
/** Retires undecided predecessors; decided proposals are immutable. */
266265
@Modifying(flushAutomatically = true)
267266
@Transactional
268267
@Query(

webapp/src/components/admin/practices/PracticeReviewSettings.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -433,7 +433,7 @@ export const CoveragePreviewLoading: Story = {
433433
preview: fn(
434434
() =>
435435
new Promise<PracticeReviewCoveragePreview>(() => {
436-
// Deliberately pending to keep the loading state visible.
436+
// Keep the preview pending so the intermediate interaction remains observable.
437437
}),
438438
),
439439
},

0 commit comments

Comments
 (0)