Skip to content

docs : README 개선#554

Open
dongjooyun wants to merge 3 commits into
developfrom
docs/readme-onboarding
Open

docs : README 개선#554
dongjooyun wants to merge 3 commits into
developfrom
docs/readme-onboarding

Conversation

@dongjooyun
Copy link
Copy Markdown
Contributor

@dongjooyun dongjooyun commented Apr 8, 2026

🍀 참고사항

서비스 전반적으로 동기화하면서 Claude와 README 수정해보았습니다.
앞으로도 리팩토링하시겠지만 현재 FE 작업하시는 관점에서 어긋난 내용 있는지, 추가 또는 개선하고 싶은 내용 있는지 코멘트 달아주세요!

☘️ 작업 내용

  • 서비스 소개, 브랜치 전략, 도메인 기능, 기술스택, 폴더구조, 인프라 구조도 추가
  • 1:1 멘토링 vs 프리미엄스터디 도메인 혼동 주의 섹션 추가
  • Mermaid 배포 파이프라인 다이어그램 추가

스크린샷 (선택)

image

Summary by CodeRabbit

문서 변경사항

  • Documentation
    • 프로젝트 소개 문구를 IT 커리어 학습 커뮤니티로 업데이트
    • 목차(빠른 시작, 브랜치 전략, 프로덕트 기능, 기술 스택 등) 기반으로 문서 재구성
    • 로컬 실행 요구사항 및 빠른 시작 가이드(환경 변수 포함) 추가
    • 브랜치 전략 및 CI 체크리스트 정리
    • 제품 기능 도메인 재정리(그룹스터디·프리미엄·1:1 등)
    • 기술 스택·폴더 구조·인프라 다이어그램 및 주요 리소스/배포 정보 갱신

- 서비스 소개, 브랜치 전략, 도메인 기능, 기술스택, 폴더구조, 인프라 구조도 추가
- 1:1 멘토링 vs 프리미엄스터디 도메인 혼동 주의 섹션 추가
- Mermaid 배포 파이프라인 다이어그램 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-platform-client-dev Error Error Apr 8, 2026 7:57pm

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 8, 2026

Warning

Rate limit exceeded

@dongjooyun has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 29 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 23 minutes and 29 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cae7294e-cb2c-4fb4-91de-c2563c3617eb

📥 Commits

Reviewing files that changed from the base of the PR and between 506b195 and 107e097.

📒 Files selected for processing (1)
  • README.md
📝 Walkthrough

Walkthrough

README 문서의 프로젝트 소개와 목차 구조를 재편성하고, 로컬 실행 가이드(요구사항·설치·명령어·환경 변수), 브랜치 전략·CI 체크, 제품 도메인 재정리, 기술 스택 및 버전 업데이트, 폴더 구조 재구성, OpenAPI 훅 생성 안내, 인프라 다이어그램 및 배포 환경 표를 추가·갱신했습니다.

Changes

Cohort / File(s) Summary
README 문서 전체 구조 개편
README.md
프로젝트 소개 문구 변경 및 목차 기반 재구성(빠른 시작, 브랜치 전략, 프로덕트 기능, 기술 스택, 폴더 구조, 인프라, 주요 리소스). 로컬 실행 요구사항(Node.js ≥20, Yarn 1.22+), 설치·실행 명령어 및 환경 변수 표 추가. 브랜치 전략 및 CI 체크리스트 정리. 기술 스택 버전 업데이트(React 19, TanStack Query 5, Zustand 5 등) 및 배포 흐름(GitHub Actions → Docker Hub → Cloudflare Tunnel) 명시. 폴더 구조를 기능(feature) 모듈 중심으로 재구성하고 OpenAPI 훅 생성 명령(yarn generate:api <swagger-api-타이틀-이름>) 안내와 인프라 mermaid 다이어그램·배포 환경 표 추가.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

토끼가 토실토실 README 정리했네 🐰
섹션들 쪼개고 빠른 시작 적고,
브랜치 길잡이와 스택도 새로 쓰고,
인프라 그림에 배포 주소까지,
모두 함께 달려가자, 코드를 향해! 🚀

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive 제목이 변경사항의 핵심(README 구조 재편성 및 콘텐츠 개선)을 포함하지만, '개선'이라는 표현이 구체성이 부족하고 광범위한 변경의 구체적 내용을 명확히 전달하지 못합니다. 더 구체적인 제목으로 개선하길 권장합니다(예: 'docs: README 구조 개편 및 온보딩 정보 강화' 또는 'docs: README 목차 기반 재구성 및 기술스택 업데이트').
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 docs/readme-onboarding

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.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Around line 66-68: The README contains fenced code blocks without language
tags (triggering markdownlint MD040) around the branch diagram "main ◄───
develop ◄─── feat/*, fix/*, refactor/*, docs/*, chore/* ...", the feature/fix
examples starting with "feat : 새 기능      →  feat/기능명" and the project tree
beginning with "src/"; update each triple-backtick fence to include a language
identifier (e.g., ```text) so the blocks are explicitly tagged and MD040
warnings are resolved, keeping the block contents unchanged.
- Line 67: Branch naming rules are inconsistent: the header branch list (main
◄─── develop ◄─── feat/*, fix/*, refactor/*, docs/*, chore/* ...) is missing
style/* and test/* which are allowed later; update the README.md branch strategy
to be consistent by either adding style/* and test/* to the primary list
(alongside feat/*, fix/*, refactor/*, docs/*, chore/* under the heading that
includes main and develop) or remove style/* and test/* from the allowed rules
further down (the block that currently mentions allowed patterns) so both places
list the same set of branch prefixes (referencing the symbols main, develop,
feat/*, fix/*, refactor/*, docs/*, chore/*, style/*, test/*).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a3213690-2117-4f69-88e7-41954d0fa765

📥 Commits

Reviewing files that changed from the base of the PR and between a314fb5 and 8afb89f.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md Outdated
Comment thread README.md Outdated
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

♻️ Duplicate comments (2)
README.md (2)

66-68: ⚠️ Potential issue | 🟡 Minor

fenced code block에 언어 태그를 명시해주세요.

Line 66, Line 82, Line 154의 코드블록이 언어 미지정이라 MD040 경고가 발생합니다. ```text처럼 언어를 지정해 주세요.

수정 예시
-```
+```text
main ◄─── develop ◄─── feat/*, fix/*, refactor/*, docs/*, chore/* ...
-```
+```text
feat : 새 기능      →  feat/기능명
...
-```
+```text
src/
├── app/
...

Also applies to: 82-90, 154-186

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 66 - 68, Several fenced code blocks in README.md (the
blocks containing the lines "main ◄─── develop ◄─── feat/*, fix/*, refactor/*,
docs/*, chore/* ...", the block starting with "feat : 새 기능      →  feat/기능명",
and the directory tree block beginning with "src/ ├── app/") are missing
language tags causing MD040 warnings; update each triple-backtick fence to
include a language tag (use ```text) for those specific blocks and the other
similar blocks referenced (lines 82-90 and 154-186) so the linter stops flagging
them.

67-67: ⚠️ Potential issue | 🟡 Minor

브랜치 prefix 목록을 한 기준으로 통일해주세요.

Line 67에는 style/*, test/*가 없고, Line 86-89에는 포함되어 있어 규칙 해석이 갈립니다. 둘 중 하나로 통일해 주세요.

Also applies to: 83-90

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 67, The branch prefix lists in the README are
inconsistent: the sequence "feat/*, fix/*, refactor/*, docs/*, chore/* ..." (as
shown in the main ◄─── develop diagram) omits "style/*" and "test/*" while
another section includes them; pick one canonical set and make both occurrences
identical. Update the README entries that list branch prefixes so that the
diagram string "main ◄─── develop ◄─── feat/*, fix/*, refactor/*, docs/*,
chore/*, style/*, test/*" (or the shorter set without style/test if you prefer)
is used consistently wherever branch prefixes are documented.
🧹 Nitpick comments (1)
README.md (1)

10-13: 배포 환경 표가 중복되어 유지보수 시 드리프트 위험이 있습니다.

상단(프론트 2행)과 하단(프론트/백엔드 통합)에서 배포 정보를 이중 관리하고 있어, 추후 한쪽만 수정될 가능성이 큽니다. 한 곳을 source of truth로 두고 다른 위치는 링크/참조로 정리하는 것을 권장합니다.

Also applies to: 287-292

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 10 - 13, The README contains duplicated deployment
environment tables (the front-end 2-row table and the integrated front/backend
table referenced at lines ~287-292); pick one canonical table (e.g., the
integrated front/backend section) and remove the other duplicate, then replace
the removed table with a short reference or link to the canonical table (add an
anchor or heading like "배포 환경" and link to it) so there is a single source of
truth for the staging/production URLs and triggers; update any surrounding text
that assumed the removed table so references point to the canonical "배포 환경"
table.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@README.md`:
- Around line 66-68: Several fenced code blocks in README.md (the blocks
containing the lines "main ◄─── develop ◄─── feat/*, fix/*, refactor/*, docs/*,
chore/* ...", the block starting with "feat : 새 기능      →  feat/기능명", and the
directory tree block beginning with "src/ ├── app/") are missing language tags
causing MD040 warnings; update each triple-backtick fence to include a language
tag (use ```text) for those specific blocks and the other similar blocks
referenced (lines 82-90 and 154-186) so the linter stops flagging them.
- Line 67: The branch prefix lists in the README are inconsistent: the sequence
"feat/*, fix/*, refactor/*, docs/*, chore/* ..." (as shown in the main ◄───
develop diagram) omits "style/*" and "test/*" while another section includes
them; pick one canonical set and make both occurrences identical. Update the
README entries that list branch prefixes so that the diagram string "main ◄───
develop ◄─── feat/*, fix/*, refactor/*, docs/*, chore/*, style/*, test/*" (or
the shorter set without style/test if you prefer) is used consistently wherever
branch prefixes are documented.

---

Nitpick comments:
In `@README.md`:
- Around line 10-13: The README contains duplicated deployment environment
tables (the front-end 2-row table and the integrated front/backend table
referenced at lines ~287-292); pick one canonical table (e.g., the integrated
front/backend section) and remove the other duplicate, then replace the removed
table with a short reference or link to the canonical table (add an anchor or
heading like "배포 환경" and link to it) so there is a single source of truth for
the staging/production URLs and triggers; update any surrounding text that
assumed the removed table so references point to the canonical "배포 환경" table.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 479c429b-5c3b-4418-b337-87423aa69ca8

📥 Commits

Reviewing files that changed from the base of the PR and between 8afb89f and 506b195.

📒 Files selected for processing (1)
  • README.md

- 코드블록 3곳에 ```text 언어 태그 추가 (MD040)
- 브랜치 다이어그램에 style/*, test/* 누락 항목 추가하여 명명 규칙과 일치

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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