feat(github): GitHub 접근을 호스트 gh로 통일#34
Merged
Merged
Conversation
lbox-limit
marked this pull request as ready for review
July 15, 2026 02:02
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
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.
Type of Change
.claude/skills/<name>/, no source changes)Description
AimClaw의 GitHub 접근을 OneCLI에서 Mac 호스트의 인증된
gh로 통일합니다.ncl github ...로 제공합니다.gh api, merge/close/review/workflow/release/secret 작업은 노출하지 않습니다.GH_TOKEN계열과 OneCLI 프록시 환경을 제거하여 호스트 키체인 로그인을 사용합니다.api.github.com·github.com차단 규칙을 멱등 적용해 기존 OAuth 연결을 통한 우회를 막습니다.geek-refactor적용 후 command metadata를 단일 표로 통합하고 테스트 전용 export와 path 주입을 제거했으며, GitHub 셸 테스트를 한 파일로 합쳤습니다. 핵심 범위는 919→825 LOC, 9→8 files, 8→3 exports로 줄었고 외부 명령과 테스트 수는 유지했습니다.검증:
pnpm run buildpnpm test— 90 files, 836 testsbash -n,git diff --checkGH_TOKEN과 OneCLI proxy 환경을 주입한 상태에서 호스트gh실제 PR 읽기 smoke testFor Skills