fix: 누적 SR은 세션 종료 후에 반영

Author: 1117mg

src/main/java/com/climbup/climbup/attempt/service/AttemptServiceImpl.java

@@ -170,36 +170,32 @@ public CreateAttemptResponse createAttempt(Long userId, CreateAttemptRequest req
 
         activeSession.setAttemptedCount(activeSession.getAttemptedCount() + 1);
 
-
         Integer srGained = 0;
         Integer currentSr = user.getSr();
 
         if (request.getSuccess()) {
             activeSession.setCompletedCount(activeSession.getCompletedCount() + 1);
             try {
-                LevelSRReward reward = LevelSRReward.fromLevelName(mission.getDifficulty());
-                srGained = reward.getSrReward();
-
-                Integer srBefore = user.getSr();
-                Integer srAfter = srBefore + srGained;
+                int gained = LevelSRReward.fromLevelName(mission.getDifficulty()).getSrReward();
 
-                user.updateSr(srAfter);
-                currentSr = srAfter;
+                int sessionGain = (activeSession.getSrGained() == null ? 0 : activeSession.getSrGained());
+                int before = user.getSr() + sessionGain;
+                int after  = before + gained;
 
-                activeSession.setSrGained(activeSession.getSrGained() + srGained);
+                activeSession.setSrGained(sessionGain + gained);
 
-                SRHistory srHistory = SRHistory.builder()
+                srHistoryRepository.save(SRHistory.builder()
                         .user(user)
                         .session(activeSession)
                         .mission(mission)
-                        .srBefore(srBefore)
-                        .srAfter(srAfter)
-                        .build();
+                        .srBefore(before)
+                        .srAfter(after)
+                        .build());
 
-                srHistoryRepository.save(srHistory);
+                srGained = gained;
 
-                log.info("SR updated for user: {} from {} to {} (+{}) for difficulty: {}",
-                        userId, srBefore, srAfter, srGained, mission.getDifficulty());
+                log.info("SR pending (session accumulate): user={}, +{}, difficulty={}, sessionId={}",
+                        userId, srGained, mission.getDifficulty(), activeSession.getId());
             } catch (IllegalArgumentException e) {
                 log.warn("Invalid difficulty level: {} for mission: {}", mission.getDifficulty(), mission.getId());
                 throw new ValidationException(ErrorCode.INVALID_DIFFICULTY_LEVEL, mission.getDifficulty());

src/main/java/com/climbup/climbup/session/service/UserSessionServiceImpl.java

@@ -75,25 +75,29 @@ public UserSession getSession(Long userId, Long sessionId) {
     @Override
     @Transactional
     public UserSession finishSession(Long userId, Long sessionId) {
-        UserSession session = userSessionRepository.findById(sessionId).orElseThrow(UserSessionNotFoundException::new);
+        UserSession session = userSessionRepository.findById(sessionId)
+                .orElseThrow(UserSessionNotFoundException::new);
 
-        if(!session.getUser().getId().equals(userId)) {
+        if (!session.getUser().getId().equals(userId))
             throw new AccessDeniedException("해당 세션에 접근할 수 없습니다.");
-        }
-
-        if(session.getEndedAt() != null) {
+        if (session.getEndedAt() != null)
             throw new UserSessionAlreadyFinishedException();
-        }
 
         LocalDateTime endTime = LocalDateTime.now();
         session.setEndedAt(endTime);
 
         Duration duration = Duration.between(session.getStartedAt(), endTime);
         session.setTotalDuration((int) duration.getSeconds());
-        
-        userSessionRepository.save(session);
 
-        return session;
+        User user = session.getUser();
+        int srBefore = user.getSr();
+        int gained   = session.getSrGained() == null ? 0 : session.getSrGained();
+        int srAfter  = srBefore + gained;
+
+        user.updateSr(srAfter);
+
+        userRepository.save(user);
+        return userSessionRepository.save(session);
     }
 
     @Override