🛠️ FIX - 프리뷰 이름의 콜론을 제거해 스크린샷 골든 업로드를 살린다

[DongChyeon]

🛠 Related issue

closed #157

어떤 변경사항이 있었나요?

• 🐞 BugFix Something isn't working
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (Junit, etc.)

✅ CheckPoint

PR이 다음 요구 사항을 충족하는지 확인하세요.

• PR 컨벤션에 맞게 작성했습니다. (필수)
• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
• BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

✏️ Work Description

• FeedCard.kt 의 종횡비 프리뷰 3개 이름에서 콜론을 제거했습니다.
  • FeedCard - Square (1:1) Interactive → (1x1)
  • FeedCard - Landscape (5:4) Interactive → (5x4)
  • FeedCard - Portrait (4:5) Interactive → (4x5)

😅 Uncompleted Tasks

• 이 PR 자체는 여전히 비교 코멘트를 받지 못합니다. develop 에 머지되어 골든 기록 잡이
  한 번 성공해야 다음 PR부터 비교가 동작합니다. 머지 후 후속 PR 로 실제 diff 코멘트가
  달리는지 확인할 예정입니다.

📢 To Reviewers

원인

Roborazzi 는 @Preview 의 name 을 그대로 골든 파일명에 씁니다. 종횡비 표기에 쓰인
콜론이 파일명으로 그대로 넘어가는데, actions/upload-artifact 는 NTFS 등 파일시스템
호환을 위해 : 가 든 경로를 거부합니다.

##[error]The path for one of the files in artifact is not valid:
/core/designsystem/build/outputs/roborazzi/...FeedCard_-_Landscape_(5:4)_Interactive_....png.
Contains the following character:  Colon :

영향

Record preview screenshots 는 성공하고 Upload goldens 에서만 실패해서, #153 도입
이후 screenshot-goldens 아티팩트가 한 번도 만들어진 적이 없습니다. 그래서 모든 PR 의
비교 워크플로가 기준선을 찾지 못하고 "base 브랜치에 골든이 없어 이번에는 비교하지
못했습니다" 코멘트만 남기는 상태였습니다. 같은 경로 패턴을 쓰는
screenshot-diff-reports 업로드도 동일하게 막힙니다. 덤으로 develop push 의 CI 가
상시 빨간불이었습니다.

참고 run: https://github.com/Nexters/BuyOrNot-Android/actions/runs/35454558084
(머지 직후 run 과 재실행 run 모두 같은 지점에서 실패)

검증

로컬에서 출력 디렉터리를 비우고 :core:designsystem:recordRoborazziDebug --rerun-tasks
를 돌려 골든 47장을 재생성했고, 파일명에 " : < > | * ? \r \n 중 어떤 것도 남지 않는
것을 확인했습니다. 소스 전체를 훑어 콜론이 든 프리뷰 이름은 이 3개가 전부였습니다.

남는 이야깃거리 (A)

앞으로 같은 실수를 막으려면 preview-network-urls 처럼 프리뷰 이름의 금지 문자를
검사하는 잡을 하나 두는 방법이 있습니다. 이번 PR 범위 밖이라 넣지 않았습니다.

📃 RCA 룰

• R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
• C: 웬만하면 반영해 주세요. (Comment)
• A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Summary by CodeRabbit

• 변경 사항
  • 미리보기 이미지 비율 표기를 콜론 형식에서 x 형식으로 변경했습니다.
  • 정사각형은 1x1, 가로형은 5x4, 세로형은 4x5로 표시됩니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: Nexters/BuyOrNot-Android/.coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: 50bd05ba-eb89-4895-89a9-6f6346ed195d

📥 Commits

Reviewing files that changed from the base of the PR and between fbcbcf7 and ea40e9a.

📒 Files selected for processing (1)

• core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/FeedCard.kt

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

---

Walkthrough

피드 카드 미리보기 3개의 비율 표기를 1:1, 5:4, 4:5에서 각각 1x1, 5x4, 4x5로 변경했습니다.

Changes

피드 카드 미리보기 비율 표기

Layer / File(s)	Summary
미리보기 이름 비율 표기 변경 core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/FeedCard.kt	정사각형, 가로형, 세로형 미리보기 이름의 비율 표기를 각각 1x1, 5x4, 4x5로 변경했습니다.

Priority: ➖ Normal

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

Change: Bug fix · Severity of issue fixed: Medium

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 프리뷰 이름의 콜론 제거와 스크린샷 골든 업로드 문제 해결을 명확하게 설명합니다. 변경 내용의 주요 목적과 일치합니다.
Linked Issues check	✅ Passed	직접 연결된 이슈 #157의 코딩 요구사항을 충족합니다. FeedCard.kt의 프리뷰 3개 이름을 (1:1), (5:4), (4:5)에서 각각 (1x1), (5x4), (4x5)로 변경했습니다. 따라서 Roborazzi가 생성하는 파일 경로에 콜론이 포함되지 않습니다. 아티팩트 생성과 후속 diff 코멘트 확인은 운영 검증 작업이…
Out of Scope Changes check	✅ Passed	변경 사항은 FeedCard.kt의 프리뷰 이름 3개 수정으로 제한됩니다. 이 변경은 이슈 #157의 아티팩트 업로드 실패 원인 제거와 직접 연결됩니다. 요약에 unrelated 변경 사항은 없습니다.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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.}

[github-actions]

스크린샷 비교 리포트

base 브랜치에 골든이 없어 이번에는 비교하지 못했습니다.

스크린샷 비교 CI를 막 도입해 아직 기준선이 없는 상태입니다.
develop 에 머지되어 골든 기록 잡이 한 번 돌면 다음 PR부터 동작합니다.