Skip to content

fix/critical-issues-and-cleanup#8

Open
Hun425 wants to merge 7 commits intomainfrom
fix/critical-issues-and-cleanup
Open

fix/critical-issues-and-cleanup#8
Hun425 wants to merge 7 commits intomainfrom
fix/critical-issues-and-cleanup

Conversation

@Hun425
Copy link
Copy Markdown
Owner

@Hun425 Hun425 commented Feb 26, 2026

Summary

Kotlin + Spring 서버 코드의 공통 스타일 기준을 문서/도구로 고정하고,
설정 하드코딩 제거 및 핵심 서비스 가독성 리팩토링을 적용했습니다.

이번 PR은 아래 4가지를 목표로 합니다.

  1. 팀 코드 스타일 기준 확립 (docs + lint/format)
  2. 설정값 외부화 및 입력 검증 강화
  3. 핵심 오케스트레이션 코드의 가독성 개선 (상위 의도 / 하위 구현)
  4. 프론트 개발 환경/린트 기준 명확화

변경 사항

docs

  • Kotlin + Spring 코드 스타일 가이드 v1 추가
    (docs/code-style-guide.md, docs/code-style-guide.ko.md)
  • 가독성 리팩토링 학습 문서 추가
    (docs/readability-refactoring-case-study.ko.md)
  • PR 초안 문서 추가
    (docs/pr-draft-style-readability-v1.ko.md)

chore (format/lint baseline)

  • 루트 포맷/린트 기준 정비
    • .editorconfig 추가
    • ktlint 최신 호환 설정 반영 (build.gradle.kts 및 모듈 build 파일)

fix

  • 설정 하드코딩 제거 및 설정 주입 일관화
    • ES index 이름을 application.yml/property로 외부화
    • scheduler timezone 설정값 주입 방식으로 전환
  • 시뮬레이터 안정성 보완
    • AnomalyDetector 스케줄 표현식 정리
    • SimulatorController/TrafficSimulator 입력 검증 강화

refactor

  • EventDetector 가독성 리팩토링
    • 조건 해석, 관측, 배치 발송, 알림 생성 로직 분리
    • 중복 배치 루프 공통화 (sendNotificationsInBatches)
  • LoadTestService 가독성 리팩토링
    • 시작/중지/메트릭 수집/시나리오 실행을 단계 함수로 분리
    • 오케스트레이션 흐름을 메서드명으로 명확화

style

  • Kotlin 전역 스타일 일관화
    • 와일드카드 import 제거
    • KDoc @see 표기 통일
    • Qualifier/로거 선언 스타일 통일
    • 다수 Kotlin 파일 포맷 정렬

chore (frontend)

  • 프론트 런타임/린트 기준 정리
    • .nvmrc 추가
    • package.json engines 갱신
    • ESLint 규칙 강화 및 console.log 제거
    • README 런타임 요구사항 반영

NOT bugs (검증 완료, 오탐/비대상)

  • ./gradlew ktlintCheck test 일괄 실행 실패는 이번 변경 이슈가 아니라
    common-avro 태스크 implicit dependency 검증 문제(기존 Gradle 구성 이슈)로 확인
  • 프론트 빌드 실패 가능성은 코드 오류가 아니라 로컬 Node 버전 미충족 환경 영향(프로젝트 요구 버전 명시로 대응)

Test Plan

  • ./gradlew ktlintFormat -q
  • ./gradlew ktlintCheck -q
  • ./gradlew test -q
  • ./gradlew :notification-service:compileKotlin :simulator:compileKotlin -q
  • npm run build (Node 20.19+/22 환경에서 확인 필요)
  • 프론트 주요 화면 smoke check (/tracing 포함)

커밋 내역

  1. de5dd73 docs: Kotlin+Spring 코드 스타일 가이드 v1 및 학습 문서 추가
  2. f0f330d chore: 루트 포맷/린트 기준 정비 및 ktlint 설정 반영
  3. 70fc86e fix: 설정 하드코딩 제거 및 시뮬레이터 입력 검증 강화
  4. cc7f894 refactor: EventDetector/LoadTestService 가독성 중심 구조 개선
  5. 3a09e53 chore(frontend): Node 버전 명시 및 ESLint 규칙 정리
  6. b1640cb style: Kotlin import/KDoc/Qualifier/로거 스타일 일관화

@Hun425 Hun425 self-assigned this Feb 26, 2026
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.

2 participants