feat: move interview count to per-application field

Adds interview_count column to job_applications so each application
tracks its own count. Dashboard interviewCount now sums across all
applications. Removes the per-user UserInterviewMetrics path and the
manual update endpoint/MCP tool (editing is now done via the
application form).

https://claude.ai/code/session_015kfgrc2oHA881RT3bEYHZ6

Author: claude

src/main/java/com/jobtracker/controller/DashboardController.java

@@ -1,20 +1,14 @@
 package com.jobtracker.controller;
 
 import com.jobtracker.dto.dashboard.DashboardSummaryResponse;
-import com.jobtracker.dto.dashboard.UpdateInterviewCountRequest;
 import com.jobtracker.service.DashboardService;
-import com.jobtracker.service.InterviewMetricsService;
-import com.jobtracker.util.SecurityUtils;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.media.Content;
 import io.swagger.v3.oas.annotations.media.Schema;
 import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import io.swagger.v3.oas.annotations.tags.Tag;
-import jakarta.validation.Valid;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.GetMapping;
-import org.springframework.web.bind.annotation.PatchMapping;
-import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
@@ -24,15 +18,9 @@
 public class DashboardController {
 
     private final DashboardService dashboardService;
-    private final InterviewMetricsService interviewMetricsService;
-    private final SecurityUtils securityUtils;
 
-    public DashboardController(DashboardService dashboardService,
-                               InterviewMetricsService interviewMetricsService,
-                               SecurityUtils securityUtils) {
+    public DashboardController(DashboardService dashboardService) {
         this.dashboardService = dashboardService;
-        this.interviewMetricsService = interviewMetricsService;
-        this.securityUtils = securityUtils;
     }
 
     @Operation(
@@ -48,19 +36,4 @@ public DashboardController(DashboardService dashboardService,
     public ResponseEntity<DashboardSummaryResponse> getSummary() {
         return ResponseEntity.ok(dashboardService.getSummary());
     }
-
-    @Operation(
-        summary = "Update interview count",
-        description = "Manually sets the cumulative interview count for the authenticated user",
-        responses = {
-            @ApiResponse(responseCode = "204", description = "Count updated"),
-            @ApiResponse(responseCode = "400", description = "Invalid count value"),
-            @ApiResponse(responseCode = "401", description = "Not authenticated")
-        }
-    )
-    @PatchMapping("/interview-count")
-    public ResponseEntity<Void> updateInterviewCount(@Valid @RequestBody UpdateInterviewCountRequest request) {
-        interviewMetricsService.setInterviewCount(securityUtils.getCurrentUserId(), request.count());
-        return ResponseEntity.noContent().build();
-    }
 }

src/main/java/com/jobtracker/dto/application/ApplicationRequest.java

@@ -2,6 +2,7 @@
 
 import com.fasterxml.jackson.annotation.JsonFormat;
 import io.swagger.v3.oas.annotations.media.Schema;
+import jakarta.validation.constraints.Min;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.PastOrPresent;
 import jakarta.validation.constraints.Pattern;
@@ -53,5 +54,9 @@ public record ApplicationRequest(
         String note,
 
         @Schema(description = "Platform or job board where the vacancy was found", example = "LinkedIn")
-        String platform
+        String platform,
+
+        @Schema(description = "Number of interviews held for this application", example = "2")
+        @Min(0)
+        Integer interviewCount
 ) {}

src/main/java/com/jobtracker/dto/application/ApplicationResponse.java

@@ -60,6 +60,8 @@ public record ApplicationResponse(
         @Schema(description = "Timestamp when the latest Google Docs resume was generated", example = "2024-06-11T14:45:00")
         @JsonFormat(pattern = "yyyy-MM-dd'T'HH:mm:ss")
         LocalDateTime driveResumeGeneratedAt,
+        @Schema(description = "Number of interviews held for this application", example = "2")
+        int interviewCount,
         @Schema(description = "Record creation timestamp", example = "2024-06-01T10:00:00")
         @JsonFormat(pattern = "yyyy-MM-dd'T'HH:mm:ss")
         LocalDateTime createdAt,

src/main/java/com/jobtracker/dto/dashboard/UpdateInterviewCountRequest.java

@@ -43,6 +43,9 @@ public class JobApplication {
     @Column(name = "interview_scheduled", nullable = false)
     private boolean interviewScheduled;
 
+    @Column(name = "interview_count", nullable = false)
+    private int interviewCount = 0;
+
     @Column(name = "next_step_date_time")
     private LocalDateTime nextStepDateTime;
 
@@ -147,6 +150,9 @@ protected void onUpdate() {
     public boolean isInterviewScheduled() { return interviewScheduled; }
     public void setInterviewScheduled(boolean interviewScheduled) { this.interviewScheduled = interviewScheduled; }
 
+    public int getInterviewCount() { return interviewCount; }
+    public void setInterviewCount(int interviewCount) { this.interviewCount = interviewCount; }
+
     public LocalDateTime getNextStepDateTime() { return nextStepDateTime; }
     public void setNextStepDateTime(LocalDateTime nextStepDateTime) { this.nextStepDateTime = nextStepDateTime; }
 

src/main/java/com/jobtracker/entity/JobApplication.java

@@ -31,6 +31,7 @@ public ApplicationResponse toResponse(JobApplication app) {
             app.getDriveResumeFileName(),
             app.getDriveResumeDocumentUrl(),
             app.getDriveResumeGeneratedAt(),
+            app.getInterviewCount(),
             app.getCreatedAt(),
             app.getUpdatedAt()
         );

src/main/java/com/jobtracker/mapper/ApplicationMapper.java

@@ -9,7 +9,6 @@
 import com.jobtracker.mapper.ApplicationMapper;
 import com.jobtracker.repository.ApplicationRepository;
 import com.jobtracker.service.ApplicationService;
-import com.jobtracker.service.InterviewMetricsService;
 import com.jobtracker.util.SecurityUtils;
 import org.springaicommunity.mcp.annotation.McpTool;
 import org.springaicommunity.mcp.annotation.McpTool.McpAnnotations;
@@ -34,39 +33,18 @@ public class McpAnalyticsTools {
     private final ApplicationRepository applicationRepository;
     private final ApplicationMapper applicationMapper;
     private final ApplicationService applicationService;
-    private final InterviewMetricsService interviewMetricsService;
     private final SecurityUtils securityUtils;
 
     public McpAnalyticsTools(ApplicationRepository applicationRepository,
                               ApplicationMapper applicationMapper,
                               ApplicationService applicationService,
-                              InterviewMetricsService interviewMetricsService,
                               SecurityUtils securityUtils) {
         this.applicationRepository = applicationRepository;
         this.applicationMapper = applicationMapper;
         this.applicationService = applicationService;
-        this.interviewMetricsService = interviewMetricsService;
         this.securityUtils = securityUtils;
     }
 
-    @McpTool(
-            name = "Update-Interview-Count",
-            title = "Update Interview Count",
-            description = "Manually sets the cumulative interview count for the current user. Use this when the automatic counter missed interviews or needs correction.",
-            annotations = @McpAnnotations(
-                    title = "Update Interview Count",
-                    readOnlyHint = false,
-                    destructiveHint = false,
-                    idempotentHint = true,
-                    openWorldHint = false))
-    @Transactional
-    public String updateInterviewCount(
-            @McpToolParam(required = true, description = "The new total interview count (must be >= 0)") long count) {
-        UUID userId = securityUtils.getCurrentUserId();
-        interviewMetricsService.setInterviewCount(userId, count);
-        return "Interview count updated to " + count;
-    }
-
     @McpTool(
             name = "Get-Analytics",
             title = "Get Analytics",

src/main/java/com/jobtracker/mcp/tools/McpAnalyticsTools.java

@@ -23,6 +23,9 @@ public interface ApplicationRepository extends JpaRepository<JobApplication, UUI
 
     long countByUserIdAndArchivedFalse(UUID userId);
 
+    @Query("SELECT COALESCE(SUM(a.interviewCount), 0) FROM JobApplication a WHERE a.user.id = :userId AND a.archived = false")
+    long sumInterviewCountByUserId(@Param("userId") UUID userId);
+
     long countByUserIdAndInterviewScheduledTrueAndArchivedFalse(UUID userId);
 
     @Query("SELECT COUNT(a) FROM JobApplication a WHERE a.user.id = :userId AND a.status = :status AND a.archived = false")

src/main/java/com/jobtracker/repository/ApplicationRepository.java

@@ -222,6 +222,9 @@ private void mapRequestToEntity(ApplicationRequest request, JobApplication app)
         app.setRecruiterDmReminderEnabled(Boolean.TRUE.equals(request.recruiterDmReminderEnabled()));
         app.setNote(normalizeOptionalText(request.note()));
         app.setPlatform(request.platform());
+        if (request.interviewCount() != null) {
+            app.setInterviewCount(request.interviewCount());
+        }
     }
 
     private void applyStatusChange(JobApplication app, ApplicationStatus newStatus) {

src/main/java/com/jobtracker/service/ApplicationService.java

@@ -2,12 +2,9 @@
 
 import com.jobtracker.entity.InterviewEvent;
 import com.jobtracker.entity.JobApplication;
-import com.jobtracker.entity.User;
-import com.jobtracker.entity.UserInterviewMetrics;
 import com.jobtracker.entity.enums.ApplicationStatus;
+import com.jobtracker.repository.ApplicationRepository;
 import com.jobtracker.repository.InterviewEventRepository;
-import com.jobtracker.repository.UserInterviewMetricsRepository;
-import com.jobtracker.repository.UserRepository;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -28,16 +25,13 @@ public class InterviewMetricsService {
             ApplicationStatus.RH_NEGOCIACAO
     );
 
-    private final UserInterviewMetricsRepository metricsRepository;
+    private final ApplicationRepository applicationRepository;
     private final InterviewEventRepository eventRepository;
-    private final UserRepository userRepository;
 
-    public InterviewMetricsService(UserInterviewMetricsRepository metricsRepository,
-                                   InterviewEventRepository eventRepository,
-                                   UserRepository userRepository) {
-        this.metricsRepository = metricsRepository;
+    public InterviewMetricsService(ApplicationRepository applicationRepository,
+                                   InterviewEventRepository eventRepository) {
+        this.applicationRepository = applicationRepository;
         this.eventRepository = eventRepository;
-        this.userRepository = userRepository;
     }
 
     public boolean isInterviewStatus(String status) {
@@ -67,49 +61,17 @@ public void recordStatusTransition(JobApplication application,
             return;
         }
 
-        User user = application.getUser();
-        UserInterviewMetrics metrics = findOrCreateMetrics(user);
-        metrics.setInterviewCount(metrics.getInterviewCount() + 1);
-        metricsRepository.save(metrics);
-
         InterviewEvent event = new InterviewEvent();
-        event.setUser(user);
+        event.setUser(application.getUser());
         event.setApplication(application);
         event.setOldStatus(oldStatus);
         event.setNewStatus(newStatus);
         event.setOccurredAt(LocalDateTime.now());
         eventRepository.save(event);
     }
 
-    @Transactional
-    public void setInterviewCount(UUID userId, long count) {
-        if (count < 0) throw new IllegalArgumentException("Interview count cannot be negative");
-        UserInterviewMetrics metrics = metricsRepository.findByUser_Id(userId)
-                .orElseGet(() -> {
-                    User user = userRepository.getReferenceById(userId);
-                    UserInterviewMetrics created = new UserInterviewMetrics();
-                    created.setUser(user);
-                    created.setInterviewCount(0);
-                    return created;
-                });
-        metrics.setInterviewCount(count);
-        metricsRepository.save(metrics);
-    }
-
     @Transactional(readOnly = true)
     public long getInterviewCount(UUID userId) {
-        return metricsRepository.findById(userId)
-                .map(UserInterviewMetrics::getInterviewCount)
-                .orElseGet(() -> eventRepository.countByUser_Id(userId));
-    }
-
-    private UserInterviewMetrics findOrCreateMetrics(User user) {
-        return metricsRepository.findByUser_Id(user.getId())
-                .orElseGet(() -> {
-                    UserInterviewMetrics created = new UserInterviewMetrics();
-                    created.setUser(user);
-                    created.setInterviewCount(0);
-                    return created;
-                });
+        return applicationRepository.sumInterviewCountByUserId(userId);
     }
 }