chore: Separate file validation steps in integration workflow

Author: HC-kang

.github/workflows/integration.yaml

@@ -25,25 +25,16 @@ jobs:
         env:
           GH_TOKEN: ${{ github.token }}
 
-      - name: Check for 1. missing end line breaks and 2. control characters in filenames and 3. filename rules
+      # 줄바꿈 체크
+      - name: Check for missing end line breaks
         run: |
-          # 필요한 값들 미리 설정
-          pr_author="${{ github.event.pull_request.user.login }}"
-          pr_number="${{ github.event.pull_request.number }}"
-          labels_json=$(gh pr view $pr_number --json labels -q '.labels[].name')
-          has_maintenance=false
-          if echo "$labels_json" | grep -q "maintenance"; then
-            has_maintenance=true
-          fi
-
           # 따옴표를 제거하고 파일 목록 가져오기
           files=$(git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | tr -d '"')
+          success=true
+
           echo "변경된 파일 목록:"
           echo "$files"
 
-          success=true
-
-          # 줄바꿈 체크
           echo "## 줄바꿈 누락 파일" >> $GITHUB_STEP_SUMMARY
           for file in $files; do
             if [ -s "$file" ] && [ "$(tail -c 1 $file | wc -l)" -eq 0 ]; then
@@ -53,63 +44,69 @@ jobs:
             fi
           done
 
-          # 제어문자 체크
-          echo -e "\n## 제어문자가 포함된 파일명" >> $GITHUB_STEP_SUMMARY
+          if [ "$success" = false ]; then
+            echo -e "\n:warning: 파일 끝의 누락된 줄바꿈을 추가해 주세요." >> $GITHUB_STEP_SUMMARY
+            exit 1
+          fi
+
+      # 제어문자 체크
+      - name: Check for control characters in filenames
+        run: |
+          files=$(git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | tr -d '"')
+          success=true
+
+          echo "## 제어문자가 포함된 파일명" >> $GITHUB_STEP_SUMMARY
           for file in $files; do
             # basename으로 파일명만 추출하고 따옴표 제거
             filename=$(basename "$file" | tr -d '"')
 
             # 백슬래시로 시작하는 제어문자들 체크 (\b, \n, \r, \t 등)
-            if printf '%q' "$filename" | grep -q '\\[bnrtfv]'; then
-              echo "- $file (제어문자 포함)" >> $GITHUB_STEP_SUMMARY
-              success=false
-            fi
+            if printf '%q' "$filename" | grep -q '\\[bnrtfv]' || \
 
-            # 일반적인 제어문자들 체크 (0x00-0x1F, 0x7F)
-            if echo -n "$filename" | LC_ALL=C grep -q '[[:cntrl:]]'; then
-              echo "- $file (제어문자 포함)" >> $GITHUB_STEP_SUMMARY
-              success=false
-            fi
+               # 일반적인 제어문자들 체크 (0x00-0x1F, 0x7F)
+               echo -n "$filename" | LC_ALL=C grep -q '[[:cntrl:]]' || \
 
-            # 특수 제어문자들 체크
-            if echo -n "$filename" | grep -q $'[\x00-\x1F\x7F]'; then
-              echo "- $file (제어문자 포함)" >> $GITHUB_STEP_SUMMARY
-              success=false
-            fi
+               # 특수 제어문자들 체크
+               echo -n "$filename" | grep -q $'[\x00-\x1F\x7F]' || \
 
-            # 이스케이프 시퀀스 체크
-            if [[ "$filename" =~ (\\[0-7]{1,3}|\\x[0-9a-fA-F]{1,2}) ]]; then
+               # 이스케이프 시퀀스 체크
+               [[ "$filename" =~ (\\[0-7]{1,3}|\\x[0-9a-fA-F]{1,2}) ]]; then
               echo "- $file (제어문자 포함)" >> $GITHUB_STEP_SUMMARY
               success=false
             fi
           done
 
-          # maintenance 라벨이 없는 경우에만 파일명 규칙 체크
-          if [ "$has_maintenance" != "true" ]; then
-            echo -e "\n## 파일명 규칙 위반" >> $GITHUB_STEP_SUMMARY
-            for file in $files; do
-              if [ -f "$file" ]; then
-
-                # 파일명만 추출 (경로 제외)
-                filename=$(basename "$file")
-
-                # 파일명이 GitHub계정명인지 확인
-                shopt -s nocasematch
-                if [[ ! "$filename" = "$pr_author"* ]]; then
-                  echo "- $file (파일명은 '$pr_author'형식으로 해주셔야 합니다)" >> $GITHUB_STEP_SUMMARY
-                  success=false
-                fi
-              fi
-            done
+          if [ "$success" = false ]; then
+            echo -e "\n:warning: 파일명에서 제어문자를 제거해 주세요." >> $GITHUB_STEP_SUMMARY
+            exit 1
           fi
 
-          if [ "$success" = false ]; then
-            echo -e "\n:warning: 위 문제들을 해결해 주세요:" >> $GITHUB_STEP_SUMMARY
-            echo "1. 파일 끝의 누락된 줄바꿈을 추가해 주세요." >> $GITHUB_STEP_SUMMARY
-            echo "2. 파일명에서 제어문자를 제거해 주세요." >> $GITHUB_STEP_SUMMARY
-            if [[ ! "$pr_labels" =~ "maintenance" ]]; then
-              echo "3. 파일명은 반드시 'GitHub계정명' 또는 'GitHub계정명-xxx' 형식으로 해주셔야 합니다. (예: ${pr_author}.ts, ${pr_author}-1.ts, ${pr_author}-2.ts)" >> $GITHUB_STEP_SUMMARY
+      # 파일명 규칙 체크 - maintenance 라벨이 없는 경우에만 실행
+      - name: Check filename rules
+        if: ${{ steps.pr-labels.outputs.has_maintenance != 'true' }}
+        run: |
+          files=$(git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | tr -d '"')
+          pr_author="${{ github.event.pull_request.user.login }}"
+          success=true
+
+          echo "## 파일명 규칙 위반" >> $GITHUB_STEP_SUMMARY
+          for file in $files; do
+            if [ -f "$file" ]; then
+
+              # 파일명만 추출 (경로 제외)
+              filename=$(basename "$file")
+
+              # 파일명이 GitHub계정명인지 확인
+              shopt -s nocasematch
+              if [[ ! "$filename" = "$pr_author"* ]]; then
+                echo "- $file (파일명은 '$pr_author'형식으로 해주셔야 합니다)" >> $GITHUB_STEP_SUMMARY
+                success=false
+              fi
             fi
+          done
+
+          if [ "$success" = false ]; then
+            echo -e "\n:warning: 파일명은 반드시 'GitHub계정명' 또는 'GitHub계정명-xxx' 형식으로 해주셔야 합니다. (예: ${pr_author}.ts, ${pr_author}-1.ts, ${pr_author}-2.ts)" >> $GITHUB_STEP_SUMMARY
             exit 1
           fi
         env: