Skip to content

docs: refine project docs and formatting conventions#336

Merged
SolfE merged 7 commits into
developfrom
codex/readme-update
Jun 23, 2026
Merged

docs: refine project docs and formatting conventions#336
SolfE merged 7 commits into
developfrom
codex/readme-update

Conversation

@SolfE

@SolfE SolfE commented Jun 23, 2026

Copy link
Copy Markdown
Member

Summary

README를 외부 방문자용 프로젝트 소개로 정리하고, AI 작업용 컨텍스트와 저장소 컨벤션 문서를 재구성했습니다. Java 포맷 기준으로 Spotless + google-java-format AOSP 설정도 추가했습니다.

Problem

기존 문서에는 미션 기반 학습 워크플로우가 전면에 남아 있어 현재 프로젝트 소개 목적과 맞지 않았습니다. 또한 브랜치/커밋/PR 작성 형식과 Java 포맷 기준이 명확히 정의되어 있지 않아 AI 작업이나 PR 작성 시 일관성을 유지하기 어려웠습니다.

Solution

미션 워크플로우 문서를 제거하고 README, AGENTS, docs 문서 구조를 역할별로 분리했습니다. docs/templates/repository-conventions.md에 저장소 컨벤션을 정의하고, Spotless를 도입해 Java 포맷 검사를 자동화했습니다.

Changes

  • 미션 기반 AGENTS/MISSIONS/skill 문서 제거
  • README를 외부 방문자용 소개, 실행, 테스트, 모니터링 중심으로 재작성
  • 루트 AGENTS.md를 AI 작업용 최소 컨텍스트 문서로 재구성
  • repository conventions 문서 추가
  • Spotless + google-java-format AOSP 설정 추가
  • .editorconfig 추가
  • PR CI에서 spotlessCheck test 실행하도록 변경

Example

포맷 검사:

./gradlew spotlessCheck

포맷 자동 적용:

./gradlew spotlessApply

전체 검증:

./gradlew check

Related Issues

없음

Validation

  • ./gradlew spotlessCheck
  • ./gradlew check

@SolfE SolfE marked this pull request as ready for review June 23, 2026 08:40

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7dcffe9157

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

Comment thread build.gradle
@SolfE

SolfE commented Jun 23, 2026

Copy link
Copy Markdown
Member Author

@codex

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 003944fc80

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

Comment thread build.gradle
}

spotless {
ratchetFrom 'origin/develop'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use the PR base for Spotless ratcheting

The PR workflow still runs for both develop and main targets (.github/workflows/pr-run-test.yml:5), but this hard-codes the Spotless ratchet to origin/develop. Spotless ratcheting checks files changed since the configured ref (docs), so PRs targeting main will be checked against the develop diff instead of their base branch once those branches diverge, causing unrelated formatting failures or the wrong baseline. Make the ratchet reference match the PR base branch, or restrict this workflow/config to develop only.

Useful? React with 👍 / 👎.

@SolfE SolfE merged commit d053d93 into develop Jun 23, 2026
1 check passed
@SolfE SolfE deleted the codex/readme-update branch June 23, 2026 09:29
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