Conversation
chore(ci): AI PR pytest·Ruff 자동 검증 추가
22 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #34
변경 이유
기존 AI 저장소는
main배포 Workflow만 있어 PR 단계에서 앱 회귀, 선택 의존성 누락, 운영체제 차이를 구분하기 어려웠습니다. 프로젝트 마무리 단계에서 핵심 앱 품질과 환경 전용 검증을 자동으로 확인할 수 있도록 PR CI를 완성합니다.변경 내용
핵심 앱 검증
develop·main대상 PR에서 Python 3.12와 고정 uv 버전 사용uv sync --frozen --extra dev기반 재현 가능한 설치app + testsRuff 검사환경별 계약 분리
CI 환경 격리
FOWOCO_ENV_FILE=/dev/null로 개발자의 로컬.env가 CI 결과에 섞이지 않도록 처리검증 결과
uv run ruff check app tests: 통과Core pytest · App Ruff: 성공Qdrant contract: 성공Windows OCR script contract: 성공배포 및 호환성
FOWOCO_ENV_FILE을 설정하지 않으면 기존처럼.env를 읽습니다.ai-envSecret의FOWOCO_*환경변수를 그대로 사용합니다./dev/null설정은 PR CI Workflow에만 적용됩니다.main병합 후 기존 배포 Workflow가 동일한 방식으로 이미지 빌드와 Rollout을 수행합니다.이번 PR에서 분리한 범위
hwp-editor와 운영 스크립트의 기존 Ruff 부채는 앱 PR 판정에서 제외합니다.개인정보·보안
리뷰 포인트
.env격리가 실제 배포 환경변수 주입에 영향을 주지 않는지develop → main변경 범위가 CI 관련 9개 파일로 한정되는지