Conversation
[REFACTOR] URL 분석 공통 콘텐츠 탐지 정확도 개선
[REFACTOR] URL 분석 정확도 개선
[CHORE] test-sites 디렉터리 제거
- Dockerfile 작성: app 소스 선복사로 hatchling 빌드 정상화, non-root(appuser) 실행, exec 기반 graceful shutdown, HEALTHCHECK 추가 - entrypoint.sh: 기동 시 alembic 마이그레이션 자동 적용 후 uvicorn 실행 - .dockerignore로 secret/테스트/로컬 DB 등 빌드 컨텍스트 제외 - CI 워크플로우: ruff lint/format, mypy, pytest 실행 - CD 워크플로우: Docker Hub 빌드·푸시 후 SSM으로 EC2 docker compose 배포 IssueNum #30
- ai_openai.py: AI 응답 스키마 description 길이 100자 이내로 축소 (E501) - analyze.py: 업스트림 시그널 검사 결과를 bool로 직접 반환 (SIM103) - db_independent_pipeline.py, pipeline.py: 리다이렉트 시그널 보강 중첩 if를 and로 결합 (SIM102) - test_db_independent_pipeline.py: 테스트 함수명 100자 이내로 축소 (E501) IssueNum #30
- 기존 코드 12개 파일이 ruff format 미적용 상태로 커밋돼 있어 format --check 가 실패함 (ruff 0.8.4/0.8.6 동일, 버전 무관한 기존 포맷 빚). - CI/CD 도입 PR 범위를 벗어나는 대량 포맷 변경을 피하기 위해 일단 제거. 포맷 일괄 정리는 별도 PR/이슈로 분리 권장. - lint(ruff check) / mypy / pytest 게이트는 그대로 유지. IssueNum #30
- INTERNAL_API_KEY 는 Settings 필수 필드라 미설정 시 config import 단계에서 ValidationError 로 pytest 수집 전체가 실패함. CI env 에 더미값 주입해 해결. - mypy 는 기존 코드 7건(남은 파일 타입 불일치, playwright 미설치 stub 등)으로 실패 — 이 PR 범위를 벗어나는 기존 빚이라 단계 제거. 타입 정리는 별도 PR 권장. - 최종 CI 게이트: ruff check(lint) + pytest. IssueNum #30
- CI: docker build 스텝 추가. Dockerfile/패키징/.dockerignore 오류를 dev PR 단계에서 잡도록 보강 (기존엔 main 머지 후 CD에서야 빌드돼 노출됨). - CD: SSM 폴링 루프가 타임아웃(300s)으로 끝났을 때 status != Success 면 exit 1 하도록 검증 추가. Pending/InProgress 지속 시 배포 성공 오인 방지. IssueNum #30
[BUILD] 배포 자동화 세팅
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
2 participants
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.
No description provided.