fix: rutter-v2 픽스처 추적 누락 수정 및 PR 테스트 워크플로 추가

[jeongph]

무엇을

1. 미추적 테스트 픽스처 커밋 — tests/fixtures/rutter-v2/docs/conventions/style.md, docs/maps/projects.md

지금까지 이 두 파일은 로컬에만 있었다. 로컬 .git/info/exclude 의 앵커 없는 docs/ 패턴이 모든 깊이의 docs 디렉터리를 가려, 형제 파일(rutter.yaml·defaults.yaml·policies/)만 커밋되고 이 둘이 빠진 것을 아무도 알아채지 못했다. 새로 클론하면 테스트 4건이 실패한다 (docs.conventions 디렉터리 'docs/conventions'이 없습니다). npm test 는 릴리스 워크플로에서만 돌기 때문에 첫 배포 시점에야 터졌을 문제다.

2. PR 테스트·타입검사 워크플로 — .github/workflows/test.yml

위 문제가 어떤 자동 검증에도 걸리지 않은 원인은 PR 검증이 commitlint 뿐이라는 공백이다. PR 마다 테스트를 돌려 메운다.

• typecheck 포함 — tsc --noEmit 은 vitest(esbuild)·tsup 어느 쪽에서도 돌지 않아, 타입 오류는 릴리스에서조차 잡히지 않고 배포되는 상태였다
• node 매트릭스 [20, 24] — engines 하한(>=20)과 publish.yml 이 배포에 쓰는 24 양쪽 검증
• main push 트리거(PR 간 의미 충돌 탐지), concurrency 취소, npm 캐시

3. .gitignore 에 .DS_Store·.claude/ 추가 — 워킹 트리에 로컬 Claude Code 산출물이 노출돼 있어 git add -A 한 번이면 공개 저장소에 올라가는 상태였다. docs 차단(#15)과 같은 계열의 유출 방지.

검증

• 커밋된 트리(git archive)를 격리 실행: 141/141 통과, 픽스처 2개만 제거 시 4건 실패 재현
• npm run typecheck 통과
• 워크플로: pull_request(≠ pull_request_target) + contents: read — 포크 PR 에서 시크릿 없음, 인젝션 보간 지점 없음
• 로컬 리뷰(code-reviewer·comment-analyzer, opus) 완료 — critical 0, important 1(typecheck 누락 → 반영), 제안 4건 반영

🤖 Generated with Claude Code

Summary by CodeRabbit

• Tests
  • Improved automated CI validation for pull requests and main branch runs, including type checks and test execution across supported Node.js versions.
  • Updated the test runner to execute the CLI via the locally installed tsx binary for consistent e2e/CLI/init test behavior.
• Documentation
  • Added coding convention guidance (including commit message standards).
  • Added a project repository overview section to the documentation fixtures.
• Chores
  • Expanded .gitignore to exclude macOS and local session artifacts.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bfd3fee2-56ed-4236-92ae-123bb12009a9

📥 Commits

Reviewing files that changed from the base of the PR and between 103a079 and d161004.

📒 Files selected for processing (5)

• .github/workflows/test.yml
• tests/apply-cli.test.ts
• tests/cli.test.ts
• tests/e2e.test.ts
• tests/init.test.ts

🚧 Files skipped from review as they are similar to previous changes (1)

• .github/workflows/test.yml

---

📝 Walkthrough

Walkthrough

Adds CI coverage for Node.js 20 and 24, updates an OIDC-related workflow comment, runs CLI tests through the local tsx binary, ignores local artifacts, and extends Korean fixture documentation.

Changes

Continuous integration validation

Layer / File(s)	Summary
Node.js test workflow .github/workflows/test.yml, .github/workflows/publish.yml	Adds pull request and main push validation with dependency installation, type checking, and tests across Node.js 20 and 24; updates the publish workflow’s Node.js/OIDC comment.
Local CLI test execution tests/apply-cli.test.ts, tests/cli.test.ts, tests/e2e.test.ts, tests/init.test.ts	Replaces npx tsx invocations with direct execution of the locally installed tsx binary while preserving arguments and environment handling.

Repository hygiene and fixture documentation

Layer / File(s)	Summary
Repository hygiene and fixture content .gitignore, tests/fixtures/rutter-v2/docs/conventions/style.md, tests/fixtures/rutter-v2/docs/maps/projects.md	Ignores .DS_Store and .claude/, and adds Korean coding convention and repository map documentation.

Estimated code review effort: 2 (Simple) | ~15 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	The title accurately summarizes the main changes: fixing missing rutter-v2 fixture tracking and adding a PR test workflow.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/track-docs-fixtures

---

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

[coderabbitai]

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 @.github/workflows/test.yml:
- Line 25: Update the actions/checkout@v4 step in the workflow to set
persist-credentials to false, preventing the GitHub token from remaining in
local git configuration for subsequent steps.

🪄 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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5bda8b97-d9cd-416a-8e8c-8fe04d08d373

📥 Commits

Reviewing files that changed from the base of the PR and between d5e35d1 and 103a079.

📒 Files selected for processing (5)

• .github/workflows/publish.yml
• .github/workflows/test.yml
• .gitignore
• tests/fixtures/rutter-v2/docs/conventions/style.md
• tests/fixtures/rutter-v2/docs/maps/projects.md

[jeongph]

persist-credentials: false 반영했습니다 (d161004). publish.yml 이 이미 쓰는 패턴과 일치시켰습니다.