ci: ruff baseline 정리 전면 실패 분리 및 단계적 적용 #355

Closed
opened 2026-03-01 20:06:36 +09:00 by agentson · 1 comment
Collaborator

Summary

  • CI의 handover strict 이슈(#353) 완화 후, 다음 단계에서 ruff check src/ tests/가 대량 위반으로 즉시 실패합니다.
  • 현재 브랜치 변경과 무관한 기존 코드베이스 전반의 lint debt가 원인입니다.

Reproduction

python3 scripts/session_handover_check.py --strict --ci
python3 scripts/validate_governance_assets.py
python3 scripts/validate_ouroboros_docs.py
ruff check src/ tests/

Observed

  • ruff check에서 110 errors
  • 대표 유형:
    • I001 import 정렬
    • E501 line too long
    • F401/F841 unused import/variable
    • UP*, N* 규칙 위반

Impact

  • 문서/CI 개선 PR에서도 코드 변경과 무관하게 CI가 FAIL
  • PR 품질 신호가 노이즈에 묻혀 회귀 탐지가 어려움

Proposed Options

  1. 단기: CI lint를 changed-files 대상으로 제한 (PR 단위 신호 확보)
  2. 중기: lint baseline 정리 이슈를 배치로 분할 처리
  3. 전환기: baseline 파일(또는 ignore 정책) 도입 후 점진 강화

Scope

  • .gitea/workflows/ci.yml
  • .github/workflows/ci.yml
  • pyproject.toml (필요 시 ruff 정책 조정)
  • src/, tests/ (점진 정리)
## Summary - CI의 handover strict 이슈(#353) 완화 후, 다음 단계에서 `ruff check src/ tests/`가 대량 위반으로 즉시 실패합니다. - 현재 브랜치 변경과 무관한 기존 코드베이스 전반의 lint debt가 원인입니다. ## Reproduction ```bash python3 scripts/session_handover_check.py --strict --ci python3 scripts/validate_governance_assets.py python3 scripts/validate_ouroboros_docs.py ruff check src/ tests/ ``` ## Observed - `ruff check`에서 110 errors - 대표 유형: - `I001` import 정렬 - `E501` line too long - `F401/F841` unused import/variable - `UP*`, `N*` 규칙 위반 ## Impact - 문서/CI 개선 PR에서도 코드 변경과 무관하게 CI가 FAIL - PR 품질 신호가 노이즈에 묻혀 회귀 탐지가 어려움 ## Proposed Options 1. 단기: CI lint를 changed-files 대상으로 제한 (PR 단위 신호 확보) 2. 중기: lint baseline 정리 이슈를 배치로 분할 처리 3. 전환기: baseline 파일(또는 ignore 정책) 도입 후 점진 강화 ## Scope - `.gitea/workflows/ci.yml` - `.github/workflows/ci.yml` - `pyproject.toml` (필요 시 ruff 정책 조정) - `src/`, `tests/` (점진 정리) ## Related - #353 - PR #354
Author
Collaborator

재검증 결과 #355의 본 이슈(ruff baseline 전면 실패)는 현재 해소되었습니다.

검증 (2026-03-01):

  • ruff check src/ tests/ -> All checks passed
  • pytest -q --maxfail=1 -> 1011 passed, 0 failed

현 시점 기준으로 CI 린트 전면 실패를 재현하지 못했으므로, baseline 분리 이슈는 종료합니다.

재검증 결과 #355의 본 이슈(ruff baseline 전면 실패)는 현재 해소되었습니다. 검증 (2026-03-01): - ruff check src/ tests/ -> All checks passed - pytest -q --maxfail=1 -> 1011 passed, 0 failed 현 시점 기준으로 CI 린트 전면 실패를 재현하지 못했으므로, baseline 분리 이슈는 종료합니다.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#355