Skip to content

chore: 조망 지도 제외 반영 (rutter 갱신) - #38

Merged
jeongph merged 1 commit into
mainfrom
feature/apply-maps-removal
Jul 26, 2026
Merged

chore: 조망 지도 제외 반영 (rutter 갱신)#38
jeongph merged 1 commit into
mainfrom
feature/apply-maps-removal

Conversation

@jeongph

@jeongph jeongph commented Jul 26, 2026

Copy link
Copy Markdown
Member

rutter가 docs/maps(조직 저장소 조망)를 합성 대상에서 제외(followingseas/rutter#14)한 것을 pilot apply로 반영한다.

변경

  • AGENTS.md·.github/copilot-instructions.md에서 "조직 저장소 조망" 섹션 제거 — 개별 저장소 작업에 필요한 규약이 아니라 참조 지식이라, 이제 rutter 저장소에서 직접 읽는다
  • 남은 규약: 문서 작성 규약, 저장소 위생(shipshape) + PolicySet rule 5개
  • .pilot/rutter.lock·release.yaml 갱신 (revision 3)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 문서

    • 조직 구조 변경 시 관련 문서와 지도를 먼저 갱신해야 하는 이유를 안내하도록 지침을 보강했습니다.
    • 문서의 출처 정보와 생성 이력을 추가·정비했습니다.
  • 릴리스 관리

    • 릴리스 메타데이터와 변경 이력이 최신 상태로 갱신되었습니다.
    • 배포 상태와 패키지 버전은 기존과 동일하게 유지됩니다.

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

조직 구조 변경 시 지도 갱신 규칙에 Why 설명과 출처 메타데이터가 추가됐다. 관련 릴리스의 revision, 생성 시각, 이전 revision 및 artifact checksum도 갱신됐다.

Changes

문서 및 릴리스 갱신

Layer / File(s) Summary
문서 규칙 및 provenance 갱신
.github/copilot-instructions.md, AGENTS.md
조직 구조 변경 시 docs/maps 갱신 규칙에 Why 설명을 추가하고, 문서의 package 및 git digest 출처 정보를 갱신했다.
릴리스 revision 및 checksum 갱신
.pilot/release.yaml
릴리스 revision을 3으로 변경하고 생성 시각, 이전 revision, 추적 artifact의 SHA-256 checksum을 갱신했다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 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 제목이 rutter 갱신과 docs/maps 제외 반영이라는 변경의 핵심을 잘 요약합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/apply-maps-removal

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

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 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 @.pilot/release.yaml:
- Around line 19-25: .pilot/release.yaml의 파일 manifest에서 현재 트리에 없는 context.md 및
CLAUDE.md 항목을 제거하고, AGENTS.md와 copilot-instructions.md 항목의 sha256 값을 현재 파일 내용으로
다시 계산한 값으로 갱신하세요.
🪄 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 Plus

Run ID: 0d7f9e0c-4021-4b91-9fe6-46b24da29bd5

📥 Commits

Reviewing files that changed from the base of the PR and between efa4f82 and 3e8b155.

⛔ Files ignored due to path filters (1)
  • .pilot/rutter.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • .github/copilot-instructions.md
  • .pilot/release.yaml
  • AGENTS.md

Comment thread .pilot/release.yaml
@jeongph
jeongph merged commit 8601114 into main Jul 26, 2026
2 checks passed
@jeongph
jeongph deleted the feature/apply-maps-removal branch July 26, 2026 03:02
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