Skip to content

chore: 서버에서 폐기된 토너먼트 에러 코드 카탈로그 정리 - #742

Merged
ychany merged 1 commit into
devfrom
chore/741-remove-retired-error-codes
Oct 19, 2026
Merged

ychany merged 1 commit into
devfrom
chore/741-remove-retired-error-codes

Conversation

@ychany

@ychany ychany commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

작업 요약

  • 서버에서 폐기된 토너먼트 에러 코드 3개(024·029·031)를 에러 코드 카탈로그에서 제거합니다

작업 세부 내용

배경

서버 TeamPiKi/core#1191(CLONE 잔재 정리 4-1단계)에서 아래 세 코드를 결번 처리했습니다. CLONE 제거(TeamPiKi/core#1027) 이후 서버에서 던지는 곳이 0곳이 된 코드들입니다. 카탈로그의 원본이 서버 API 문서라 같이 맞춥니다.

코드 상수 원래 의미
TOURNAMENT-024 TOURNAMENT_CLONED_CANNOT_SHARE_PLAY_LINK 클론은 플레이 링크 공유 불가
TOURNAMENT-029 TOURNAMENT_ALREADY_CLONED 이미 이 링크로 만든 클론 있음
TOURNAMENT-031 TOURNAMENT_CLONED_CANNOT_VIEW_GROUP_RESULT 클론은 그룹 결과 조회 불가

변경

  • packages/core/src/consts/errorCode.ts
    • ERROR_MESSAGE_MAP 에서 문구 3개 제거
    • ERROR_CODE 에서 상수 3개 제거

영향

  • web·app 모두 세 상수를 참조하는 곳이 0곳이라 동작 변화는 없습니다
  • 서버가 더 이상 보내지 않는 코드라, 구버전 앱이 이 코드를 받을 일도 없습니다

확인

  • @piki/core · piki-web · piki-app 타입 검사·lint 통과

후속

스크린샷

없음 (UI 변화 없음)

연관 이슈

closes #741

Summary by CodeRabbit

  • 변경 사항
    • 토너먼트 복제와 관련된 오류 메시지 및 오류 코드 3개가 제거되었습니다.

@vercel

vercel Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
piki Ready Ready Preview Oct 19, 2026 1:45pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
piki-notify Skipped Skipped Oct 19, 2026 1:45pm UTC

@ychany ychany added the chore label Oct 19, 2026
@ychany ychany added APP Good for newcomers WEB labels Oct 19, 2026
@vercel
vercel Bot temporarily deployed to Preview – piki-notify October 19, 2026 13:43 Inactive
@ychany ychany self-assigned this Oct 19, 2026
@github-actions
github-actions Bot requested a review from iOdiO89 October 19, 2026 13:43
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 560f9ab4-4ee4-4af2-aa64-92b8df8f9ae8
📥 Commits

Reviewing files that changed from the base of the PR and between 7d1eb79 and e068b14.

📒 Files selected for processing: 1
  • packages/core/src/consts/errorCode.ts
💤 Files with no reviewable changes: 1
  • packages/core/src/consts/errorCode.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

오류 코드 카탈로그에서 폐기된 토너먼트 오류 코드 3개와 해당 메시지 및 상수를 제거했습니다. 다른 토너먼트 오류 코드 항목은 유지했습니다.

Changes: 토너먼트 오류 코드 카탈로그

Layer / File(s) Summary
폐기된 코드와 메시지 제거
packages/core/src/consts/errorCode.ts
ERROR_MESSAGE_MAP에서 TOURNAMENT-024, TOURNAMENT-029, TOURNAMENT-031 메시지를 삭제했습니다. ERROR_CODE에서 대응하는 상수 3개를 삭제했습니다. TOURNAMENT-025~028 및 TOURNAMENT-030 항목은 유지했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to e068b

The catalog is aligned with the reported server code removal, with no identified issue requiring a fix before merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed 제목은 서버에서 폐기된 토너먼트 에러 코드를 카탈로그에서 제거하는 주요 변경 사항을 정확하게 요약합니다.
Linked Issues check Passed 활성 직접 연결 이슈 #741은 TOURNAMENT-024, TOURNAMENT-029, TOURNAMENT-031의 메시지 3개와 상수 3개를 packages/core/src/consts/errorCode.ts에서 제거하도록 요구합니다. 변경 diff는 ERROR_MESSAGE_MAP에서 세 키를 제거하고 ERROR_CODE에서 대응…
Out of Scope Changes check Passed 변경은 packages/core/src/consts/errorCode.ts의 이슈 대상 에러 코드 제거로만 구성됩니다. diff에는 메시지 3개와 상수 3개의 삭제만 있으며, #741과 연결되지 않은 구현 변경은 확인되지 않습니다.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check

Explanation

활성 직접 연결 이슈 #741은 TOURNAMENT-024, TOURNAMENT-029, TOURNAMENT-031의 메시지 3개와 상수 3개를 packages/core/src/consts/errorCode.ts에서 제거하도록 요구합니다. 변경 diff는 ERROR_MESSAGE_MAP에서 세 키를 제거하고 ERROR_CODE에서 대응하는 세 상수를 제거합니다. TOURNAMENT-032는 유지되어 이슈의 별도 처리 범위도 보존됩니다.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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

@ychany
ychany merged commit 5a47459 into dev Oct 19, 2026
11 checks passed
@ychany
ychany deleted the chore/741-remove-retired-error-codes branch October 19, 2026 13:53

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – piki — e068b144 Deployed Oct 19, 2026 by vercel[bot]
Preview – piki-notify — e068b144 Deployed Oct 19, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

APP Good for newcomers chore WEB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: 서버에서 폐기된 토너먼트 에러 코드 카탈로그 정리

2 participants