feat(skills): finding-unknowns를 Codex에도 노출 — 동등 지휘자 + 행동 계약 중립화 - #1087
Merged
Conversation
grilling 세션 합의 (PR #1082 후속): - 노출 전환: intentionallyNotExposed → exposedCodexSkills (verify-ai-compat EXPECTED_EXPOSED 이동, SKILL_NEUTRAL_LINT_EXCLUDE 제거) - 근거 실측: Codex user 스코프는 ~/.agents/skills — grilling/prototype이 맥북·miniPC 양쪽에서 이미 발견됨 (codex exec 스킬 목록 probe로 확인). 기존 ~/.codex/skills 노출 경로도 인식 확인 - 중립화: 도구명 리터럴(AskUserQuestion/request_user_input)과 runtime-mapping 링크를 제거하고 행동 계약만 유지 — 'blocking 질문 도구로 한 문항씩 답을 기다리고, plain-text로 묻고 지나가거나 가정하고 진행 금지' - Codex 성향 보정: 게이트 A·인터뷰에 명시 지시 (default mode의 make-assumptions-and-only-stop-if-blocked 정책 대응, run-da 선례) - 이식성: grilling/prototype 미설치 환경은 문서 요지로 직접 수행 폴백 Claude-Session: https://claude.ai/code/session_01PbbrsHKxkzkaKn9iJdgYDH
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthrough
Changesfinding-unknowns 실행 및 호환성 정비
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Contributor
There was a problem hiding this comment.
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 `@modules/shared/programs/claude/files/skills/finding-unknowns/SKILL.md`:
- Line 44: finding-unknowns 스킬의 저위험 미지 처리 규칙을 Gate A 조건과 일치시키세요. 사용자가 “라벨된 가정과
함께 진행”을 명시적으로 허용하기 전에는 기본값을 적용해 구현을 진행하지 말고, 해당 기본값은 제안하고 기록만 하도록 수정하세요.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 477203aa-2b3e-44a3-9b98-72dc4423f657
📒 Files selected for processing (5)
modules/shared/programs/claude/files/skills/finding-unknowns/SKILL.mdmodules/shared/programs/claude/files/skills/finding-unknowns/references/tactics.mdmodules/shared/programs/claude/files/skills/finish-pr/SKILL.mdmodules/shared/programs/codex/default.nixscripts/ai/verify-ai-compat.sh
CodeRabbit 지적 반영: '기본값을 선택하고'가 강화된 게이트 A(가정 직진 금지)와 표면 충돌해 오독 여지 — 기본값은 원장·계획에 잠정 기록되고 게이트 A에서 계획과 함께 확인받는다는 우선순위를 문구로 명시. Claude-Session: https://claude.ai/code/session_01PbbrsHKxkzkaKn9iJdgYDH
Owner
Author
머지 후 검증 결과
|
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.
Summary
finding-unknowns(미지 방법론 오케스트레이터)를 Codex에도 노출해 동등 지휘자로 만든다 — Codex 단독 세션도 구현 전(정찰·blindspot·인터뷰·계획)부터 구현 후(퀴즈)까지 풀사이클을 지휘할 수 있다.기존 문제/배경
PR #1082는 finding-unknowns를 Claude 전용으로 신설하며 Codex를
intentionallyNotExposed에 두었다. 당시 근거는 "지휘 대상인 grilling·prototype이 Codex에 없다"였는데, 이 전제가 실측으로 반증됐다: Codex의 user 스코프 스킬 경로는~/.codex/skills가 아니라~/.agents/skills이며(upstream 문서 확인), npx로 설치된 Matt Pocock 스킬들이 정확히 그 경로에 있어 맥북·miniPC 양쪽 Codex 세션에서 이미 발견된다 (codex exec 스킬 목록 probe + ssh 실측). 남은 장애물은 노출 목록과 Claude 전용 도구 리터럴뿐이었다.CIR (Change Intent Record)
trade-off: 도구명 힌트가 사라져 각 런타임이 자기 질문 도구를 스스로 찾아야 하지만, 사용자 실측대로 현행 모델은 "질문해"라는 행동 지시만으로 올바른 도구를 찾는다. 반면 "멈춰서 물어라"는 성향 보정은 도구명과 무관하게 필요해 행동 계약으로 남겼다.
ADR
구현 상세
modules/shared/programs/codex/default.nixintentionallyNotExposed→exposedCodexSkills이동 (실측 근거 주석)scripts/ai/verify-ai-compat.shEXPECTED_EXPOSED이동,SHARED_EXPOSURE_EXCLUDE·SKILL_NEUTRAL_LINT_EXCLUDE에서 제거 (도구-중립 lint 편입)skills/finding-unknowns/SKILL.mdskills/finding-unknowns/references/tactics.mdskills/finish-pr/SKILL.md참고 레퍼런스
~/.agents/skills확인.claude/skills/configuring-codex/SKILL.md— "default mode 모델은 질문 도구를 자동 호출하지 않으므로 본문 명시 지시 필요" 실측 기록 (행동 계약 유지 근거)Human Test Plan
정상 동작 검증
nrs후ls -la ~/.codex/skills/finding-unknowns를 실행한다../scripts/ai/verify-ai-compat.sh를 실행한다.$finding-unknowns를 입력한다.codex exec로 스킬 목록 probe 후 ~/.codex/skills 인식 여부 확인.행동 계약 검증
request_user_input(blocking) 호출로 이뤄지고, plain-text 질문 후 가정 직진하지 않는다.Negative·Regression
grep -rn "AskUserQuestion\|runtime-mapping" modules/shared/programs/claude/files/skills/finding-unknowns/를 실행한다.https://claude.ai/code/session_01PbbrsHKxkzkaKn9iJdgYDH
Summary by CodeRabbit
finding-unknowns의 단계별 지침(생략 사유 기록, 무단 생략 금지)과 예외 처리 기준을 구체화했습니다.인터뷰/게이트 A에서 질문은 답을 기다리는 방식으로 진행하고, 침묵을 동의로 보거나 “막히지 않음”으로 가정하지 않도록 규칙을 강화했습니다.finish-pr의 퀴즈 게이트 질문 진행 문구를 정리해 일관성을 높였습니다.finding-unknowns의 환경별 노출/검증 정책을 정비해 shared 스킬로 처리되도록 조정했습니다.