Skip to content

ci: PR 검증·잠금 설치·브라우저 회귀 테스트 추가 - #3

Open
0disoft wants to merge 1 commit into
mainfrom
review/reproducible-pr-ci
Open

0disoft wants to merge 1 commit into
mainfrom
review/reproducible-pr-ci

Conversation

@0disoft

@0disoft 0disoft commented Sep 22, 2026

Copy link
Copy Markdown
Owner

검토 근거

기준 커밋: 2e3cb0c30fe3ab909256efccc755a3aaedbaee42.

  • 기존 Pages workflowmain push/수동 실행만 처리해 PR의 타입·단위 테스트·빌드 결과를 병합 전에 제공하지 않습니다.
  • package.jsonbun@1.3.14를 선언하지만 배포는 bun-version: latest와 잠금 옵션 없는 bun install을 사용합니다.
  • tests/e2e가 있어도 Actions에서 실행하지 않습니다. Playwright는 trace: on-first-retry이지만 재시도가 없어 실패 추적이 생성되지 않습니다.
  • Pages/OIDC 쓰기 권한이 의존성 설치·테스트 job에도 부여됩니다.

변경

  • 읽기 권한만 가진 PR CI: Svelte/tsgo/Oxlint/Oxfmt/Vitest와 Pages base path 정적 빌드를 검사합니다.
  • 별도 Chromium job에서 기존 E2E를 실행하고 실패 진단을 7일 보관합니다.
  • Bun 버전을 package.json에서 읽고 --frozen-lockfile을 적용합니다. 기존 잠금 파일을 임의로 갱신하지 않습니다.
  • CI 재시도 1회, test.only 차단, worker 2개, job timeout/concurrency를 설정합니다.
  • Pages/OIDC 쓰기 권한을 deploy job에만 제한하고 checkout 인증정보를 남기지 않습니다.

검증 상태

  • 기존 설정·공식 setup-bun/Playwright 문서와 대조한 변경입니다.
  • 현재 검토 환경에서 GitHub clone이 DNS 오류로 실패하여 의존성을 설치한 전체 로컬 빌드/테스트는 실행하지 못했습니다. 이 PR의 Actions 결과로 확인해야 하며, 통과했다고 주장하지 않습니다.
  • 이 PR은 workflow/테스트 실행 설정만 변경하며 앱 동작·의존성 버전·잠금 파일은 변경하지 않습니다.

참고

기본 브랜치 보호/required checks 설정은 저장소 운영 정책이므로 이 PR에서 변경하지 않았습니다.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review Completed 2026-09-22T18:40:48.248261Z e30e0d4 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

0disoft commented Sep 22, 2026

Copy link
Copy Markdown
Owner Author

GitHub Actions 검증 완료

커밋 e30e0d404e32682df7fc70aac5a1a134c5b7c219Pull Request CI #1success로 완료되었습니다.

  • quality: 잠금 설치, Svelte 검사, tsgo 검사, Oxlint, Oxfmt, 단위 테스트, GitHub Pages base path 빌드 모두 성공.
  • browser: Chromium 설치와 기존 Playwright 브라우저 테스트 성공.

이는 GitHub Actions 실제 실행 결과입니다. 로컬 컨테이너에서 전체 의존성을 설치·실행한 결과는 아니며, 실제 DCInside 붙여넣기와 Safari/Firefox 검증까지 의미하지는 않습니다.

0disoft commented Sep 22, 2026

Copy link
Copy Markdown
Owner Author

앱 수정 PR과의 통합 검증

PR #8/#13의 최신 변경까지 합친 검증 전용 브랜치에서 Review Integration Validation #2success로 완료되었습니다.

Bun 1.3.14 잠금 설치, tsgo, Oxlint(오류·경고 0), Oxfmt, 단위 테스트 191개/27파일, Pages base path 빌드, Chromium E2E 5개가 통과했습니다. Svelte check는 오류 0이며, 변경하지 않은 Toolbar.svelte의 기존 CSS 경고 6개는 남아 있습니다.

검증 커밋: 5a39d3726d5774f2b2c1c9b6eb022770e3625400. 이 브랜치는 검증용으로만 만들었으며 main과 배포는 변경하지 않았습니다. 앱 수정 PR의 개별 체크를 상시 실행하려면 이 CI PR을 먼저 병합하는 순서가 적절합니다.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant