Skip to content

[Task] GRB-04 backend verification contract reset#78

Merged
alexization merged 2 commits intodevelopfrom
feat/grb-04-backend-verification-contract-reset
Apr 9, 2026
Merged

[Task] GRB-04 backend verification contract reset#78
alexization merged 2 commits intodevelopfrom
feat/grb-04-backend-verification-contract-reset

Conversation

@alexization
Copy link
Copy Markdown
Owner

@alexization alexization commented Apr 9, 2026

1) 요약

  • backend repo에 남아 있던 legacy verification surface를 제거했습니다.
  • harness 기준 verification contract를 다시 구성하기 전에 stale command, stale CI, stale 문서가 canonical source처럼 남지 않도록 clean baseline을 먼저 만들 필요가 있었습니다.

2) 연관 이슈

3) 문제와 목표

  • 문제: integrationTest, coverage gate, OpenAPI regeneration, backend CI, verification 전용 문서/테스트가 pre-harness 방식으로 흩어져 있어 다음 contract 정규화 단계에서 old/new verification semantics가 섞일 위험이 있었습니다.
  • 사용자/운영자 관점의 결과: backend repo가 legacy verification surface 없이 clean baseline 상태가 되어, 후속 작업에서 harness-aligned command/CI/doc ownership을 다시 설계할 수 있습니다.
  • 비목표: 새 verification contract 구현, production behavior 변경, frontend 변경

4) 영향 범위

  • 변경된 패키지/모듈:
    • build.gradle
    • .github/workflows/
    • docs/openapi/
    • verification-only test sources
    • README.md
  • API/DTO/Schema 영향: 없음. public API behavior는 변경하지 않았고 tracked OpenAPI snapshot surface만 제거했습니다.
  • DB/Cache/Batch/Scheduler 영향: 없음
  • 보안/권한 영향: 없음

5) 검증 증거

유형 명령어 / 증거 결과
Build ./gradlew build 성공
Unit ./gradlew test 성공
Integration 미실행(legacy integration lane cleanup으로 command 제거) 미실행
Coverage 미실행(legacy coverage gate cleanup으로 command 제거) 미실행
API/Manual Smoke 미실행(public API behavior 비변경, OpenAPI tracked surface cleanup only) 미실행

6) 관측성 확인

  • 확인한 로그: 불필요
  • 확인한 메트릭: 불필요
  • 확인한 trace/dashboard/query: 불필요

7) AI 리뷰 메모 (선택)

  • Codex: session-isolated reviewer pool(Ohm, Godel, coordinator Nietzsche) refreshed pass approved
  • CodeRabbitAI:

8) 리스크 및 롤백

  • 리스크: 후속 rebuild 전까지 backend repo의 automated verification guardrail이 줄어든 상태입니다.
  • 롤백 계획: 이 PR을 되돌리면 legacy verification surface와 tracked artifacts를 즉시 복원할 수 있습니다.

9) 체크리스트

  • 연관 이슈가 연결되어 있음
  • Build / Unit / Integration 결과가 기입되어 있음
  • API/스키마/배치 영향이 반영되었거나 없음을 명시함
  • 로그/메트릭/trace 확인 내용을 적었거나 불필요 사유를 적음
  • 문서 또는 후속 이슈가 업데이트되었거나 불필요 사유를 적음

Summary by CodeRabbit

릴리스 노트

  • Chores
    • CI/CD 자동화 워크플로우 제거
    • 테스트 인프라 및 코드 검증 도구 제거
    • API 명세 문서 및 관련 생성 도구 제거
    • 개발 검증 가이드 문서 제거

- legacy verification Gradle task와 CI workflow 제거
- OpenAPI tracked 문서와 verification 전용 테스트 제거
- harness 재구성을 위한 clean baseline 정리
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 9, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 84465e56-0ea8-4bc7-883d-90675e1ecba2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

개요

레거시 검증 인프라 제거: GitHub Actions 워크플로우(CI/품질 게이트), Gradle 태스크(JaCoCo, 통합 테스트, Docker 프리플라이트, OpenAPI 생성), 관련 테스트 및 문서를 삭제하여 검증 계약 정규화를 위한 클린 베이스라인 구성.

변경 사항

코호트 / 파일 요약
GitHub Actions 워크플로우
.github/workflows/ci.yml, .github/workflows/quality-gate.yml
Backend CI 파이프라인(Java 21, Gradle 캐싱, 유닛/통합 테스트 실행 및 아티팩트 업로드)과 품질 게이트 워크플로우(JaCoCo 커버리지 검증 및 리포트 업로드)를 완전히 제거.
Gradle 빌드 구성
build.gradle
JaCoCo 플러그인 및 전체 커버리지 검증 로직, integrationTest/verifyDockerAvailable/generateOpenApiSpec 태스크, ArchUnit 테스트 의존성, Docker/Testcontainers 통합 제거.
OpenAPI 문서
docs/openapi/README.md, docs/openapi/openapi.json
OpenAPI 재생성 지침, 런타임 엔드포인트 경로, 보안 스킴 정의를 포함한 전체 OpenAPI 3.1 스펙 문서 및 생성된 명세 파일 제거.
검증 관련 테스트
src/test/java/com/gitranker/api/architecture/ArchitectureGuardrailTest.java, src/test/java/com/gitranker/api/docs/OpenApiDocsTest.java
아키텍처 제약 검증(ArchUnit 규칙)과 OpenAPI 엔드포인트(/v3/api-docs, /swagger-ui/index.html) 노출 검증 테스트 클래스 삭제.
Docker 프리플라이트 지원 코드
src/test/java/com/gitranker/api/testsupport/DockerPreflightCheck.java, src/test/java/com/gitranker/api/testsupport/DockerPreflightCheckTest.java, src/test/java/com/gitranker/api/testsupport/DockerPreflightMain.java
Docker 환경 확인 구현(CommandRunner, CommandResult, ProcessCommandRunner 포함) 및 관련 테스트 스위트, 메인 진입점 제거.
문서 업데이트
README.md
개발 검증 섹션(## 🧪 Development Verification) 및 목차 링크 제거.

예상 코드 리뷰 시간

🎯 3 (Moderate) | ⏱️ ~22 minutes

관련 가능성 있는 PR

🐰 레거시 검증, 이제 안녕 ~
워크플로우, 태스크, 테스트들 정리하고,
클린한 베이스라인 위에
새로운 계약이 피어날 때를 기다리며
정규화의 길을 나아가네! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목 '[Task] GRB-04 backend verification contract reset'는 변경사항의 주요 목적을 명확하게 요약합니다. 레거시 검증 자산(workflow, Gradle 작업, OpenAPI 문서, 관련 테스트) 제거라는 핵심 변경사항을 정확히 반영합니다.
Linked Issues check ✅ Passed PR은 연결된 이슈 #77의 모든 주요 요구사항을 충족합니다. build.gradle에서 레거시 검증 작업/플러그인/의존성 제거, .github/workflows/의 레거시 CI 제거, 검증 관련 문서 및 테스트 자산 정리가 완료되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항은 이슈 #77의 범위 내입니다. 새로운 검증 계약 구현, 백엔드 기능 변경, 공개 API 동작 변경 또는 프론트엔드 변경이 없으며, 오직 레거시 검증 자산의 제거만 수행되었습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/grb-04-backend-verification-contract-reset

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

https://github.com/alexization/git-ranker/blob/5c971664a91390827bb47e61e0b90acfbc0240ce/build.gradle#L89-L91
P1 Badge Reinstate an executable lane for integration tests

This change removes the integrationTest task that was the only task including **/*IT.class, while the test task still excludes that pattern, so repository integration tests such as UserRepositoryIT and ActivityLogRepositoryIT no longer run in any standard Gradle verification path. In practice, DB/Testcontainers regressions can now pass CI/build and surface only after deployment; keep a dedicated integration-test task/workflow or stop excluding *IT from all verification commands.

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@alexization alexization merged commit 467b3aa into develop Apr 9, 2026
2 checks passed
@alexization alexization deleted the feat/grb-04-backend-verification-contract-reset branch April 9, 2026 07:39
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