[DEPLOY] 배포

[i-meant-to-be]

🚩 연관 이슈

없음

📝 작업 내용

배포

🏞️ 스크린샷 (선택)

없음

🗣️ 리뷰 요구사항 (선택)

없음

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bfd9fb09-5948-45c2-84cc-6e71fed38754

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch develop

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[gemini-code-assist]

Code Review

This pull request introduces a live sharing feature for debate timers, including audience and chairman socket hooks, an audience share page, and live share UI components. It also adds an answer time timer with setting controls and hotkey support, improves Sentry error tracking, and refactors modal wrapper rendering. The reviewer feedback highlights several critical issues: a potential remounting bug in useModal due to unstable onClose references, a state synchronization issue in handleTimerEvent where stale state is captured during transition events, an over-broad regex in normalizeEndpoint that incorrectly replaces API version paths, and a contradiction in useTimerHotkey where KeyA and KeyL hotkeys remain active despite being documented as disabled.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

useModal 훅에서 반환하는 ModalWrapper 컴포넌트가 useCallback으로 감싸져 있지만, 의존성 배열에 closeModal이 포함되어 있습니다. closeModal은 onClose 함수에 의존하는데, 만약 사용처에서 useModal({ onClose: () => { ... } })와 같이 인라인 화살표 함수를 전달하면 매 렌더링마다 onClose 참조가 변경됩니다.

이로 인해 closeModal -> handleOverlayClick -> ModalWrapper 순으로 참조가 매번 새로 생성되어, 부모 컴포넌트가 리렌더링될 때마다 모달 내부의 전체 DOM과 상태가 파괴되고 새로 마운트(Remount)되는 심각한 버그가 발생할 수 있습니다.

이를 방지하기 위해 onClose를 useRef에 저장하여 closeModal과 ModalWrapper 컴포넌트의 참조를 완전히 안정화하는 것을 권장합니다. 예를 들어 다음과 같이 리팩토링할 수 있습니다:

const onCloseRef = useRef(onClose);
useEffect(() => {
  onCloseRef.current = onClose;
}, [onClose]);

const closeModal = useCallback(() => {
  setIsOpen(false);
  onCloseRef.current();
}, []);

(파일 상단에 useRef, useEffect 임포트가 필요합니다.)

[gemini-code-assist]

handleTimerEvent 함수는 현재 렌더링 시점의 index, timerType, remainingTime 등을 클로저로 캡처합니다. 하지만 invoke()가 실행되어 라운드가 변경(index 변경)되거나 팀이 전환될 때, 상태 업데이트는 비동기적으로 예약되므로 issueEvent가 호출되는 시점에는 여전히 이전 라운드의 stale한 데이터가 전송됩니다.

이로 인해 NEXT 또는 BEFORE 이벤트가 발생했을 때 관전자(Audience) 화면은 새로운 라운드의 타이머 타입이나 초기 시간으로 동기화되지 못하고 이전 라운드의 상태에 멈춰있게 되는 심각한 동기화 버그가 발생합니다.

이벤트 타입(NEXT, BEFORE, TEAM_SWITCH)에 따라 다음 상태의 값들을 미리 계산하여 페이로드로 전송하도록 수정해야 합니다.

  // 타이머 이벤트를 핸들링하는 래퍼 함수 선언
  const handleTimerEvent = (invoke: () => void, eventType: SocketEventType) => {
    // 이벤트 실행
    invoke();

    // 만약 소켓 열려 있으면, 발송
    if (isSocketConnected && data) {
      let targetIndex = index;
      if (eventType === 'NEXT') {
        targetIndex = Math.min(data.table.length - 1, index + 1);
      } else if (eventType === 'BEFORE') {
        targetIndex = Math.max(0, index - 1);
      }

      const targetItem = data.table[targetIndex];
      const targetTimerType = targetItem.boxType;

      let targetRemainingTime = remainingTime;
      if (eventType === 'NEXT' || eventType === 'BEFORE') {
        targetRemainingTime = targetTimerType === 'NORMAL' ? (targetItem.time ?? 0) : (targetItem.timePerTeam ?? 0);
      }

      // 타입에 따른 페이로드 준비
      let innerPayload: TimerDataPayload;

      if (targetTimerType === 'NORMAL') {
        innerPayload = {
          timerType: targetTimerType,
          remainingTime: targetRemainingTime,
          sequence: targetIndex,
        } as TimerDataPayload;
      } else if (targetTimerType === 'TIME_BASED') {
        innerPayload = {
          currentTeam: prosConsSelected,
          timerType: targetTimerType,
          remainingTime: targetRemainingTime,
          sequence: targetIndex,
        } as TimerDataPayload;
      } else {
        return;
      }

      const payload = eventType === 'FINISHED' ? null : innerPayload;

      // 이벤트 발행
      issueEvent(eventType, payload);
    }
  };

[gemini-code-assist]

normalizeEndpoint 함수에서 모든 숫자 시퀀스를 :id로 치환하는 replace(/[0-9]+/g, ':id') 로직은 /api/v1/polls와 같은 API 버전 경로(v1, v2 등)까지 /api/v:id/polls로 잘못 치환하는 문제를 발생시킵니다.

이로 인해 Sentry에서 서로 다른 API 버전의 에러가 동일한 엔드포인트로 잘못 그룹화될 수 있습니다. 경로 세그먼트 전체가 숫자인 경우만 /:id로 치환하도록 정규식을 수정하는 것이 안전합니다.

Suggested change

	export function normalizeEndpoint(url?: string) {
	if (!url) {
	return 'unknown';
	}
	return url
	.replace(
	/\b[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}\b/gi,
	':uuid',
	)
	.replace(/[0-9]+/g, ':id');
	}
	export function normalizeEndpoint(url?: string) {
	if (!url) {
	return 'unknown';
	}
	return url
	.replace(
	/\b[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}\b/gi,
	':uuid',
	)
	.replace(/\/\d+\b/g, '/:id');
	}

[gemini-code-assist]

주석 설명('26. 6. 3. 부로 A키와 L키를 활용한 팀 전환은 비활성화) 및 keysToDisable에서의 제외 처리와 달리, switch 문 내부의 case 'KeyA'와 case 'KeyL' 분기 로직이 여전히 활성화되어 있어 실제로 키를 누르면 팀 전환 이벤트가 발생합니다.

의도치 않은 팀 전환을 방지하기 위해 case 'KeyA'와 case 'KeyL' 블록을 완전히 제거하거나 비활성화해 주세요.

        case 'KeyA':
          // '26. 6. 3. 부로 A키를 활용한 팀 전환은 비활성화
          break;
        case 'KeyL':
          // '26. 6. 3. 부로 L키를 활용한 팀 전환은 비활성화
          break;