[RELEASE] v2.2.6 PR 생성 시 테스트 자동 실행 CI 워크플로우 추가

Author: pooreumjung

.claude/commands/review.md .claude/commands/backend-review.md.claude/commands/review.md renamed to .claude/commands/backend-review.md

@@ -38,3 +38,12 @@ Rules:
 - Do not invent implementation details that are not implied by the request.
 - Prefer concrete acceptance criteria over vague descriptions.
 - If there is no attachment/reference, write `- 없음`.
+
+When the user explicitly asks to create the issue (not just draft it), run:
+
+```bash
+gh issue create --title "feature: <summary>" --body "<final markdown body>" --assignee @me --label feat
+```
+
+- Always include `--assignee @me` so the issue is assigned to the current user.
+- Always include `--label feat` to match this template's convention.

.claude/commands/feature-issue.md

@@ -38,3 +38,12 @@ Rules:
 - If the issue is operational, include the relevant environment such as prod, dev, EC2, Docker, or CloudWatch.
 - Do not claim a root cause unless it is directly supported by the given context.
 - If there is no attachment/reference, write `- 없음`.
+
+When the user explicitly asks to create the issue (not just draft it), run:
+
+```bash
+gh issue create --title "fix: <summary>" --body "<final markdown body>" --assignee @me --label fix
+```
+
+- Always include `--assignee @me` so the issue is assigned to the current user.
+- Always include `--label fix` to match this template's convention.

.claude/commands/fix-issue.md

@@ -56,3 +56,25 @@ Rules:
 - For the checklist, mark an item checked only if the evidence is clear from the context or command results.
 - If tests were not run, leave the test checkbox unchecked and mention that in reviewer notes.
 - Do not include unrelated refactoring in the summary.
+
+When the user explicitly asks to create the PR (not just draft it), run:
+
+```bash
+gh pr create --title "<type>: <summary>" --body "<final markdown body>" --assignee @me --reviewer eunseo9311,sjinssun --label <label>
+```
+
+- Always include `--assignee @me` so the PR is assigned to the current user.
+- Always include `--reviewer eunseo9311,sjinssun`.
+- Determine `<label>` from the current branch's type prefix (`feat/`, `fix/`, `refactor/`, `docs/`, `test/`, `chore/`, `style/`) using this mapping:
+
+  | Branch type | Label |
+  | --- | --- |
+  | feat | feat |
+  | fix | fix |
+  | refactor | refactoring |
+  | docs | docs |
+  | test | test |
+  | chore | chore |
+  | style | (no label — omit `--label`) |
+
+  If the branch type has no matching label in the table, omit `--label` entirely rather than guessing.

.claude/commands/pr.md

@@ -0,0 +1,13 @@
+---
+paths:
+  - "src/main/java/**/*.java"
+  - "src/test/java/**/*.java"
+---
+
+# Commenting Convention
+
+- Comments should be written in a single line whenever possible.
+- Each comment must include the author's name and the date.
+- Use IntelliJ's default `FIX ME` comment format.
+  - Use this for parts of the code where there may be a potential issue but immediate exception handling is unnecessary.
+  - Also use this for parts that are not yet finalized due to unclear business requirements.

.claude/rules/commenting.md

@@ -0,0 +1,13 @@
+---
+paths:
+  - "**/persistence/**/*.java"
+  - "**/repository/**/*.java"
+  - "**/entity/**/*.java"
+  - "src/main/resources/db/migration/**"
+---
+
+# Persistence & Migration Convention
+
+- Complex queries use the `*RepositoryCustom` interface + `*RepositoryImpl` (QueryDSL) pattern.
+- Schema is managed with Flyway migration files at `src/main/resources/db/migration/V{n}__{설명}.sql`.
+- `ddl-auto` is set to `none`, so any schema change must be accompanied by a new migration file.

.claude/rules/persistence.md

@@ -0,0 +1,10 @@
+---
+paths:
+  - "src/test/**/*.java"
+---
+
+# Test Code Convention
+
+- Test method names must be written in **English**.
+- If the code is related to test, write comments for "given, when, then".
+- Tests run with `@SpringBootTest` connected to a real DB. Do not mock the DB.

.claude/rules/testing.md

@@ -0,0 +1,67 @@
+name: EAT-SSU Server PR 테스트 게이트
+
+on:
+  pull_request:
+    branches: [ "develop", "main" ]
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    services:
+      mysql:
+        image: mysql:8.0
+        env:
+          MYSQL_DATABASE: eatssu_test
+          MYSQL_ROOT_PASSWORD: test
+        ports:
+          - 3306:3306
+        options: >-
+          --health-cmd="mysqladmin ping -h localhost"
+          --health-interval=10s
+          --health-timeout=5s
+          --health-retries=10
+    env:
+      EATSSU_DB_URL_TEST: jdbc:mysql://localhost:3306/eatssu_test
+      EATSSU_DB_USERNAME: root
+      EATSSU_DB_PASSWORD: test
+      EATSSU_AWS_ACCESS_KEY_DEV: dummy-access-key
+      EATSSU_AWS_SECRET_KEY_DEV: dummy-secret-key
+      EATSSU_SLACK_TOKEN: dummy-slack-token
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: JDK 17 설치
+        uses: actions/setup-java@v3
+        with:
+          java-version: '17'
+          distribution: 'temurin'
+
+      - name: Gradle 캐싱
+        uses: actions/cache@v3
+        with:
+          path: |
+            ~/.gradle/caches
+            ~/.gradle/wrapper
+          key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }}
+          restore-keys: |
+            ${{ runner.os }}-gradle-
+
+      - name: gradlew 실행 권한 부여
+        run: chmod +x gradlew
+
+      - name: 테스트용 JWT 시크릿 생성
+        run: echo "EATSSU_JWT_SECRET_TEST=$(openssl rand -base64 64 | tr -d '\n')" >> "$GITHUB_ENV"
+
+      - name: 테스트 실행
+        run: ./gradlew test --no-daemon
+
+      - name: 테스트 리포트 업로드
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: test-results
+          path: build/reports/tests/test

.github/workflows/ci.yml

@@ -11,22 +11,11 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 - Suggest based on logical reasons.
 - For every request, follow this order before writing any code:
   1. Analyze the process (현재 구조/흐름 파악)
-  2. Suggest a direction (어떤 방향으로 접근할지 제시)
-  3. Present trade-offs (각 방향의 장단점 제시)
-  4. Do NOT write code unless the user explicitly asks with a direct command such as "개발해줘", "작성해줘", "구현해줘".
-
-## Commenting Convention
-
-- Comments should be written in a single line whenever possible.
-- Each comment must include the author's name and the date.
-- Use IntelliJ's default `FIX ME` comment format.
-  - Use this for parts of the code where there may be a potential issue but immediate exception handling is unnecessary.
-  - Also use this for parts that are not yet finalized due to unclear business requirements.
-
-## Test Code Convention
-
-- Test method names must be written in **English**.
-- But if the codes is related to test, write the comment for "given, when, then".
+  2. If anything is unclear or requires a decision only the user can make, ask clarifying questions before proposing a direction
+  3. Suggest a direction based on the analysis (and any answers received)
+  4. Present trade-offs for each direction
+  5. Do NOT write code unless the user explicitly asks with a direct command such as "개발해줘", "작성해줘", "구현해줘".
+- When implementing from a GitHub issue with multiple feature items, process one item at a time: complete the full analyze → clarify → direction → trade-off → explicit approval cycle for a single item before moving to the next.
 
 ## Build & Run Commands
 
@@ -84,18 +73,10 @@ src/main/java/ssu/eatssu/
 
 **다국어(i18n)**: `Localizable` 인터페이스의 `getLocalizedValue(language, ko, en, ja, vi)`를 구현하여 언어별 필드를 반환한다.
 
-**복잡한 쿼리**: `*RepositoryCustom` 인터페이스 + `*RepositoryImpl`(QueryDSL) 패턴을 사용한다.
-
-**DB 마이그레이션**: Flyway를 사용하며, `src/main/resources/db/migration/V{n}__{설명}.sql` 형식으로 관리한다. `ddl-auto: none`이므로 스키마 변경 시 반드시 마이그레이션 파일을 추가해야 한다.
-
 ### API 버전 관리
 
 일부 Controller/Service는 V1/V2로 버전이 분리되어 있다(예: `ReviewController`/`ReviewControllerV2`, `ReviewService`/`ReviewServiceV2`). 새 기능은 V2 이상에 추가하거나, 필요 시 새 버전을 생성한다.
 
-### 테스트
-
-테스트는 `@SpringBootTest`로 실제 DB에 연결하여 실행한다. 목(Mock) DB를 사용하지 않는다.
-
 ### 배포
 
 GitHub Actions(`deploy.yml`)로 Docker 이미지를 빌드하여 EC2에 SSH 배포한다. `prod`/`dev` 브랜치에 따라 배포 환경이 분기된다.

CLAUDE.md

@@ -21,14 +21,23 @@ public class Ratings {
     private Integer tasteRating;
 
     private Ratings(Integer mainRating, Integer amountRating, Integer tasteRating) {
-        Assert.isTrue(mainRating >= 0 && mainRating <= 5, "mainRating must be between 0 and 5");
-        Assert.isTrue(amountRating >= 0 && amountRating <= 5, "amountRating must be between 0 and 5");
-        Assert.isTrue(tasteRating >= 0 && tasteRating <= 5, "tasteRating must be between 0 and 5");
+        Assert.notNull(mainRating, "mainRating must not be null");
+        validateRange(mainRating, "mainRating");
+        // FIXME(pooreumjung, 2026-07-04): amountRating/tasteRating을 선택값으로 둘지는 비즈니스 요구사항 확정 필요
+        validateRange(amountRating, "amountRating");
+        validateRange(tasteRating, "tasteRating");
         this.mainRating = mainRating;
         this.amountRating = amountRating;
         this.tasteRating = tasteRating;
     }
 
+    private static void validateRange(Integer rating, String fieldName) {
+        if (rating == null) {
+            return;
+        }
+        Assert.isTrue(rating >= 1 && rating <= 5, fieldName + " must be between 1 and 5");
+    }
+
     public static Ratings of(Integer mainRating, Integer amountRating, Integer tasteRating) {
         return new Ratings(mainRating, amountRating, tasteRating);
     }