chore: 운영 문서 docs 하위 이동#620
Merged
Merged
Conversation
Co-authored-by: OmX <omx@oh-my-codex.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthrough버전 관리 운영 문서를 ChangesVersion Management Documentation Reorganization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
skills_context/SHARED/zeroone-version-management.md (1)
19-21: ⚡ Quick win읽기 순서 목록에서 문서 항목 중복을 제거해 주세요.
Line 9-11에서 이미
docs/ops/*3개를 필수 선행 읽기로 정의했는데, Line 19-21에 다시 반복되어 “docs 먼저, 그 다음 scripts/workflows” 규칙이 흐려집니다. 하위 목록에서는 스크립트/워크플로우만 남기는 쪽이 명확합니다.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@skills_context/SHARED/zeroone-version-management.md` around lines 19 - 21, 문서의 읽기 순서 목록에서 중복된 항목을 제거하세요: 현재 하위 목록에 다시 나열된 `docs/ops/onboarding.md`, `docs/ops/version-management.md`, `docs/ops/release-record-shared-contract.md` 항목을 삭제하고 하위 목록에는 스크립트/워크플로우 항목만 남기도록 수정해 `docs` 항목이 상위 목록(필수 선행 읽기)에서만 유지되게 하세요.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/asdfasdf.md`:
- Line 1: 파일에 의미 없는 플레이스홀더 텍스트 'asdfasdfsa'가 들어가 있습니다; 해당 문자열을 삭제하거나 운영 문서로서 유효한
실제 내용(목적, 개요, 사용법 등)으로 교체하고, 문서가 프로젝트 문서 인덱스나 TOC에 링크되어 있으면 관련 참조(예: README/TOC
항목)를 함께 갱신하여 깨진 참조가 없도록 해 주세요.
In `@docs/ops/release-record-shared-contract.md`:
- Around line 290-298: 문서에서 사용 중인 승인 토큰 표기 'bootstrap:approved'가 다른 SSOT 문서의
'bootstrap: approved'와 불일치하므로 하나로 통일해야 합니다; 문서 내 모든 언급(예: 문장 "백엔드는 ...
`bootstrap:approved` 승인을 받은 경우"와 프론트엔드 기록 예시 `metadata.bootstrap_mode: true`)을
찾아 SSOT에서 정한 표기('bootstrap: approved' 또는 SSOT에 명시된 정확한 공백 형식)로 일괄 교정하고, 문서 머리나
변경 이력에 어떤 표기를 표준으로 사용하는지 한 줄로 명시해 주세요.
In `@docs/ops/version-management.md`:
- Around line 61-65: The `release_id` is defined as a unique identifier but the
current format `prod-YYYYMMDD-HHmm` can collide for multiple releases inside the
same minute; update the `release_id` rule to include higher resolution (e.g.,
change `prod-YYYYMMDD-HHmm` to `prod-YYYYMMDD-HHmmss`) or append a deterministic
suffix/run number to guarantee uniqueness; update the docs text and examples to
show the new format and mention that `release_id` must be unique per deployment.
---
Nitpick comments:
In `@skills_context/SHARED/zeroone-version-management.md`:
- Around line 19-21: 문서의 읽기 순서 목록에서 중복된 항목을 제거하세요: 현재 하위 목록에 다시 나열된
`docs/ops/onboarding.md`, `docs/ops/version-management.md`,
`docs/ops/release-record-shared-contract.md` 항목을 삭제하고 하위 목록에는 스크립트/워크플로우 항목만
남기도록 수정해 `docs` 항목이 상위 목록(필수 선행 읽기)에서만 유지되게 하세요.
🪄 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: da88761a-ce84-4b1f-98b9-100459227ed9
📒 Files selected for processing (32)
.claude/rules/version-management-frontend.md.claude/skills/zeroone-version-management/SKILL.md.codex/skills/zeroone-version-management/SKILL.md.github/workflows/deploy-prod.yml.github/workflows/release-record-check.ymlAGENTS.mddocs/2026-03-15-login-fail-fix/AUTH_STAGED_CHANGE_BREAKDOWN.mddocs/2026-03-15-login-fail-fix/FRONTEND_OAUTH_REDIRECTION_CONTRACT.mddocs/2026-03-15-login-fail-fix/MIDDLEWARE_AUTH_REFACTORING_NEXT_STEPS.mddocs/2026-03-15-login-fail-fix/MIDDLEWARE_ROUTE_POLICY_GUIDE.mddocs/2026-03-15-login-fail-fix/PR_AUTH_REFACTORING_OVERVIEW.mddocs/2026-03-15-login-fail-fix/SOCIAL_LOGIN_REFACTORING_PLAN.mddocs/2026-03-26-markdown-editor/COMMON_MARKDOWN_EDITOR_USAGE.mddocs/asdfasdf.mddocs/auth-proposition/AUTH_LOGICAL_PROPOSITIONS_200.mddocs/auth-proposition/AUTH_PROPOSITION_121_150_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_151_180_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_181_200_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_1_30_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_31_60_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_61_90_AUDIT.mddocs/auth-proposition/AUTH_PROPOSITION_91_120_AUDIT.mddocs/ops/onboarding.mddocs/ops/release-record-shared-contract.mddocs/ops/version-management.mdops/backend-release-dispatch.mdops/deploy-checklist.mdops/release-intent.mdops/release-record-shared-contract.mdops/rollback.mdops/version-management.mdskills_context/SHARED/zeroone-version-management.md
💤 Files with no reviewable changes (13)
- ops/deploy-checklist.md
- docs/2026-03-15-login-fail-fix/MIDDLEWARE_ROUTE_POLICY_GUIDE.md
- docs/2026-03-26-markdown-editor/COMMON_MARKDOWN_EDITOR_USAGE.md
- ops/rollback.md
- ops/version-management.md
- docs/2026-03-15-login-fail-fix/PR_AUTH_REFACTORING_OVERVIEW.md
- ops/release-intent.md
- docs/2026-03-15-login-fail-fix/AUTH_STAGED_CHANGE_BREAKDOWN.md
- docs/2026-03-15-login-fail-fix/MIDDLEWARE_AUTH_REFACTORING_NEXT_STEPS.md
- ops/release-record-shared-contract.md
- docs/2026-03-15-login-fail-fix/FRONTEND_OAUTH_REDIRECTION_CONTRACT.md
- docs/2026-03-15-login-fail-fix/SOCIAL_LOGIN_REFACTORING_PLAN.md
- ops/backend-release-dispatch.md
| @@ -0,0 +1 @@ | |||
| asdfasdfsa No newline at end of file | |||
There was a problem hiding this comment.
의미 없는 플레이스홀더 텍스트가 문서로 추가되었습니다.
Line 1의 asdfasdfsa는 운영 문서로서 유효한 내용이 아니어서, 이 파일은 PR에서 제거하거나 실제 문서 내용으로 교체해 주세요.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/asdfasdf.md` at line 1, 파일에 의미 없는 플레이스홀더 텍스트 'asdfasdfsa'가 들어가 있습니다; 해당
문자열을 삭제하거나 운영 문서로서 유효한 실제 내용(목적, 개요, 사용법 등)으로 교체하고, 문서가 프로젝트 문서 인덱스나 TOC에 링크되어
있으면 관련 참조(예: README/TOC 항목)를 함께 갱신하여 깨진 참조가 없도록 해 주세요.
Comment on lines
+290
to
+298
| 현재 운영이 아직 canonical image tag 위에 있지 않다면, 백엔드는 백엔드 쪽에서 명시적인 `bootstrap:approved` 승인을 받은 경우에만 bootstrap mode를 사용할 수 있습니다. | ||
|
|
||
| Bootstrap 버전 계산은 다음과 같습니다. | ||
|
|
||
| - `release:patch` -> `v0.0.1` | ||
| - `release:minor` -> `v0.1.0` | ||
| - `release:major` -> `v1.0.0` | ||
|
|
||
| 프론트엔드는 백엔드 bootstrap 사실을 `metadata.bootstrap_mode: true`로 기록합니다. |
There was a problem hiding this comment.
Bootstrap 승인 토큰 표기를 다른 규칙 문서와 통일해 주세요.
이 문서는 bootstrap:approved를 사용하지만, 공유 SSOT 문서에서는 bootstrap: approved를 요구하고 있습니다. 운영자가 어느 형식을 써야 하는지 혼선이 생기면 PR 본문 파싱 실패로 배포가 막힐 수 있어, 한 가지 표기로 고정해야 합니다.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/ops/release-record-shared-contract.md` around lines 290 - 298, 문서에서 사용
중인 승인 토큰 표기 'bootstrap:approved'가 다른 SSOT 문서의 'bootstrap: approved'와 불일치하므로 하나로
통일해야 합니다; 문서 내 모든 언급(예: 문장 "백엔드는 ... `bootstrap:approved` 승인을 받은 경우"와 프론트엔드 기록
예시 `metadata.bootstrap_mode: true`)을 찾아 SSOT에서 정한 표기('bootstrap: approved' 또는
SSOT에 명시된 정확한 공백 형식)로 일괄 교정하고, 문서 머리나 변경 이력에 어떤 표기를 표준으로 사용하는지 한 줄로 명시해 주세요.
Comment on lines
+61
to
+65
| `release_id`는 운영 배포 1건을 식별하는 고유 ID입니다. 사람이 언제 배포됐는지 바로 알 수 있도록 배포 시각을 포함합니다. | ||
|
|
||
| ```txt | ||
| prod-YYYYMMDD-HHmm | ||
| ``` |
There was a problem hiding this comment.
release_id 고유성 정의와 포맷 분해능이 충돌합니다.
Line 61에서 고유 ID라고 정의했는데, Line 64 포맷(prod-YYYYMMDD-HHmm)은 같은 분에 2회 배포 시 충돌할 수 있습니다. 초 단위(HHmmss) 또는 suffix(예: run number)를 포함하도록 규칙을 보강해 주세요.
제안 수정안
-`release_id`는 운영 배포 1건을 식별하는 고유 ID입니다. 사람이 언제 배포됐는지 바로 알 수 있도록 배포 시각을 포함합니다.
+`release_id`는 운영 배포 1건을 식별하는 고유 ID입니다. 사람이 언제 배포됐는지 바로 알 수 있도록 배포 시각을 포함하고, 동일 시각 충돌을 피할 수 있어야 합니다.
```txt
-prod-YYYYMMDD-HHmm
+prod-YYYYMMDD-HHmmss</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @docs/ops/version-management.md around lines 61 - 65, The release_id is
defined as a unique identifier but the current format prod-YYYYMMDD-HHmm can
collide for multiple releases inside the same minute; update the release_id
rule to include higher resolution (e.g., change prod-YYYYMMDD-HHmm to
prod-YYYYMMDD-HHmmss) or append a deterministic suffix/run number to guarantee
uniqueness; update the docs text and examples to show the new format and mention
that release_id must be unique per deployment.
</details>
<!-- fingerprinting:phantom:triton:hawk -->
<!-- This is an auto-generated comment by CodeRabbit -->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요약
docs/ops로 정리하고 온보딩/상세 가이드/공유 계약 구조로 단순화했습니다.skills_context/SHARED/zeroone-version-management.md를 보도록 경로를 정리했습니다.검증
ruby -e "require 'yaml'; YAML.load_file('.github/workflows/deploy-prod.yml'); YAML.load_file('.github/workflows/release-record-check.yml'); puts 'workflow yaml ok'"\n-node scripts/release/validate-release-record.mjs releases\n-git diff --check HEAD^..HEAD\n- push hooknext build통과Summary by CodeRabbit
릴리스 노트
문서화
구성 개선