Skip to content

Scrub internal route examples#677

Merged
sungjunlee merged 1 commit into
mainfrom
codex/scrub-internal-route-examples
Jun 1, 2026
Merged

Scrub internal route examples#677
sungjunlee merged 1 commit into
mainfrom
codex/scrub-internal-route-examples

Conversation

@sungjunlee
Copy link
Copy Markdown
Owner

@sungjunlee sungjunlee commented Jun 1, 2026

Summary

  • replace company-specific route examples with neutral example/... provider/model routes
  • remove internal-route wording from route policy docs
  • update config and policy tests to use the neutral fixtures

Tests

  • node --test tests/relay-config/scripts/*.test.js
  • node --test tests/relay-dispatch/scripts/relay-config.test.js tests/relay-dispatch/scripts/relay-policy.test.js tests/relay-dispatch/scripts/docs-defaults.test.js
  • node --test tests/skills-lint/scripts/*.test.js
  • git diff --check
  • rg -n "kakao|opencode-glm" . (no matches)

Summary by CodeRabbit

릴리스 노트

  • Documentation

    • 모델 라우트 설정 예시가 일관되게 업데이트되었습니다.
    • README, 정책 설명서, 운영자 가이드의 라우팅 구성 예시가 표준화되었습니다.
    • 설정 검증 명령어 및 라우팅 정책 예시가 최신 패턴으로 반영되었습니다.
  • Tests

    • 설정 및 정책 검증 테스트의 모델 라우트 예시가 동기화되었습니다.
    • 테스트 시나리오의 예상 결과가 업데이트되었습니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f77f141a-4478-4b48-b3b3-57e235ce96e9

📥 Commits

Reviewing files that changed from the base of the PR and between d3a596f and 882660f.

📒 Files selected for processing (7)
  • README.md
  • docs/model-route-policy.md
  • docs/relay-operator-guide.md
  • skills/relay-config/SKILL.md
  • tests/relay-config/scripts/relay-config.test.js
  • tests/relay-dispatch/scripts/relay-config.test.js
  • tests/relay-dispatch/scripts/relay-policy.test.js

Walkthrough

이 PR은 전체 문서, 스킬 가이드, 테스트를 통해 예시 라우트 네임스페이스를 kakao/opencode-glm-*에서 example/opencode-model-*로 일관되게 업데이트합니다. 또한 사이드카 라우팅 규칙명을 docs-approved-sidecar로 정규화합니다.

Changes

예시 라우트 참조 업데이트

Layer / File(s) Summary
문서 및 가이드 업데이트
README.md, docs/model-route-policy.md, docs/relay-operator-guide.md
README의 "Configure Routes" 섹션과 라우팅 정책 표, 모델 라우트 정책 가이드, 운영 가이드에서 OpenCode 허용 라우트 예시가 kakao/opencode-glm-*에서 example/opencode-model-*로 변경됩니다. 또한 company policy의 routing_rules에서 사이드카 규칙명이 docs-internal-sidecar에서 docs-approved-sidecar로 업데이트되고, advisory_review sidecar의 모델이 example/opencode-model-fast로 변경됩니다.
스킬 가이드 문서 업데이트
skills/relay-config/SKILL.md
대화형 relay-config 스킬 가이드의 Use-case 설명, 사용자 입력 예시, init company, check 명령어 예시 모두에서 라우트 패턴이 kakao/opencode-glm-*에서 example/opencode-model-*로, 특정 모델 참조는 example/opencode-model-fast로 변경됩니다.
Relay config 테스트 픽스처
tests/relay-config/scripts/relay-config.test.js
check 관련 테스트의 allow-route 입력 및 dispatch/advisory_review 검증 명령에서 모델 라우트 패턴이 example/opencode-model-*example/opencode-model-fast로 업데이트됩니다.
Relay policy 테스트 데이터 및 어설션
tests/relay-dispatch/scripts/relay-config.test.js, tests/relay-dispatch/scripts/relay-policy.test.js
정책 검증, 라우트 매칭, allow-route/deny-route 테스트의 입력 데이터와 기대 결과(matchedRoute, reason 값 포함)가 모두 example/* 네임스페이스로 일관되게 업데이트됩니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • sungjunlee/dev-relay#565: 본 PR이 이전 PR #565에서 도입된 relay-config 스킬 및 정책 문서의 예시 라우트 문자열을 일관되게 업데이트하고 있습니다.

Poem

토끼가 웃으며 예시를 바꾸네 🐰
kakao는 안녕, example이 왔네
테스트는 모두 똑같이 달리고
문서는 명확하게 반짝이네 ✨
네임스페이스 깔끔, 모두가 행복!

🚥 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 PR 제목이 '내부 라우트 예제 제거(Scrub internal route examples)'이며, 변경 사항의 핵심인 회사별 라우트 예제를 중립적인 example/... 형식으로 교체한 내용을 정확하게 요약합니다.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 codex/scrub-internal-route-examples

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.

@sungjunlee sungjunlee merged commit 20eaf62 into main Jun 1, 2026
2 checks passed
@sungjunlee sungjunlee deleted the codex/scrub-internal-route-examples branch June 1, 2026 00:10
sungjunlee added a commit that referenced this pull request Jun 1, 2026
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