Skip to content

fix: Claude 이슈 모드의 비공개 lease 입력 (1.1.1) - #12

Merged
tourbut merged 1 commit into
mainfrom
tourbut/fix-claude-issue-capability
Oct 8, 2026
Merged

tourbut merged 1 commit into
mainfrom
tourbut/fix-claude-issue-capability

Conversation

@tourbut

@tourbut tourbut commented Oct 8, 2026 •

Copy link
Copy Markdown
Owner

Summary

Refs #10

 enable
-  stdout → 작업 공간 밖 /tmp 파일 → 별도 파일 권한 검사 → 거부
+  stdout → 작업 공간 안의 Git 제외·0600 파일 → 실제 활성화 성공
 wait/checkpoint/question/answers/sync/reconcile/complete/poll/claim
-  --token <비밀값> 또는 인라인 Python 전달
+  --token-file <enable 결과 JSON> → CLI 내부 로드 → 기존 lease/현재 coor 검증

플러그인 1.1.1. 기존 --token은 유지하며 두 입력은 상호 배타적입니다. 파일은 같은 작업 공간의 Git 제외 일반 파일로 제한하고, host 권한 파일·잘못된 JSON/token·POSIX 공개 권한을 차단합니다. 스킬과 운영/릴리스 안내, 생성 native 패키지를 함께 갱신했습니다. 전역 권한/우회 모드/서비스 정책은 변경하지 않습니다.

Evidence

  • Before: i-Docs의 실제 Claude Code 2.1.294 / Sonnet 5.5에서 configure/enable의 정확한 두 세션 허용 규칙을 적용해도 /tmp로 출력하는 enable은 [Auto-Mode Bypass]로 거부됐습니다. native 실행 기록을 읽는 재현 검사도 실패했습니다.
    After: 같은 provider session·Run·terminal incarnation·enable 인자·허용 규칙을 유지하고 출력 경로만 작업 공간 안의 Git 제외·0600 파일로 바꾸자 enable이 실제 성공했습니다. activated, poll_started, owner epoch 1과 PID를 공개 status에서 확인했습니다. 1.1.1로 다시 실행한 enable도 epoch 2로 성공했습니다.
  • Before: 실제 CLI 회귀 검사 wait --token-file ...는 필수 --token 오류로 실패했습니다.
    After: python3 tests/issue-mode.py 통과. 실제 CLI 파일 입력과 기존 coor fencing, Git 제외/비공개 권한/잘못된 token 거부, token 비출력을 확인했습니다.
  • npm ci, npm test, python3 tests/review-check.py, git diff --check 통과. native 파일은 python3 scripts/build.py로 생성했습니다.
  • 원격 CI 4개 검사 통과: core, review, Windows/macOS portable.
  • 남은 실환경 검증: 현재 사용자가 세션 허용을 configure/enable 두 명령으로 한정했으므로 wait --token-file도 자동 심사에서 거부됐습니다. 이 일반 명령까지 허용하는 범위를 확인 중입니다. GitHub 수신/질문·답변/worker/draft PR은 아직 성공으로 처리하지 않으며, QA 이슈도 미리 만들지 않았습니다.

Merge Danger

Door: two-way

CLI 입력 옵션 추가이며 기존 token 입력과 상태 schema를 유지합니다. 기본 OFF와 기존 승인/lease/identity 경계는 그대로입니다. POSIX 파일 권한 검사를 실제 Linux에서 검증했으며 Windows ACL 검증 결과는 주장하지 않습니다.

Blast Radius: 이슈모드

수정은 lease 입력과 운영 안내에 한정합니다. 전체 GitHub 수용 QA 완료 또는 #10 종료를 의미하지 않습니다.

@tourbut tourbut left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검토 범위: base 55706af14a8a856b14801106ba48e0a89ea962f0 → head 2370c271bf21963b518ee7ab74f88739faa45a35.

Standards

통과. AGENTS.md·개발 지침에 따라 portable source와 build 생성물이 일치하고 1.1.1 버전·릴리스 기록이 반영됐습니다. 토큰 파일의 작업 공간·Git 제외·일반 파일·POSIX 소유자/비공개 권한·JSON/token 형식 검사를 확인했습니다. 기존 coor/lease 검증, token 비출력, 입력 옵션의 상호 배타성을 유지합니다. 차단 사항과 선택적 리팩토링 제안은 없습니다.

Spec

통과. 확인된 출력 경로 문제의 조치와 --token-file 입력이 요청 범위를 충족하며 기존 --token 호환을 유지합니다. 전역 권한/우회 모드/서비스 정책을 확장하지 않습니다. 실제 GitHub 왕복 QA가 미완료라는 한계를 문서와 PR에 정확히 기록했습니다. 전체 #10 수용 검증은 이 패치 병합의 완료 범위에 포함하지 않습니다.

두 축의 독립 검토 결과 차단 사항 0건입니다. 기존 전체 로컬 검사와 원격 CI 4개가 통과했고, 사용자 요청에 따라 이 고정 head를 main에 병합합니다.

@tourbut
tourbut marked this pull request as ready for review October 8, 2026 08:03
@tourbut
tourbut merged commit adc100e into main Oct 8, 2026
4 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T08:04:34.298357Z 2370c27 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@tourbut

tourbut commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

사용자 요청에 따라 PR #12의 1.1.1 안정화 패치를 main에 병합하고 완료 처리했습니다.

  • 병합 커밋: adc100e08bb830da98572b9232d381b98ca87d92
  • 고정 head 2370c271bf21963b518ee7ab74f88739faa45a35의 Standards/Spec 독립 검토: 차단 사항 0건.
  • 로컬 전체 npm test, 실제 OCR tests/review-check.py, 원격 PR CI 4개 검사 통과.
  • 병합 후 main의 1.1.1 manifest와 PR head 대비 동일한 소스 tree를 확인했습니다.
  • 병합 후 main CI: 4개 검사 모두 통과 (core, review, Windows/macOS portable).

이 완료 기록은 PR #12 안정화 패치 범위입니다. #10의 실제 GitHub foreground wait/질문·답변/worker/draft PR 수용 검증은 아직 미완료이므로 해당 이슈는 열린 상태로 유지합니다. QA 자동 모드는 OFF이고 큐/기존 사용자 작업을 보존했습니다.

@tourbut
tourbut deleted the tourbut/fix-claude-issue-capability branch October 9, 2026 10:36
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