Repository navigation
Conversation
Contributor
Author
|
Task-claim: orch-teamai-cli badef4ec 2026-09-29T12:52:07Z |
Task-claim: orch-teamai-cli badef4ec 2026-09-29T12:52:07Z
jung0han
force-pushed
the
task/recall-marker-and-dedup
branch
from
September 29, 2026 21:57
923e851 to
1d20fe5
Compare
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.
변경 요약
이 PR은 렌더된
CLAUDE.md와teamai-recall.md규칙 파일에 영어·중국어 self-exemption 문단이 중복되는 문제만 해결합니다. CLI 명령이나 옵션은 추가하지 않습니다.builtin-rules.ts에TEAMAI_RECALL_SELF_EXEMPTION공통 상수를 추출합니다. 문단 내용은 바꾸지 않습니다.compileRecallRulesBlock(recallRulePath?)는 해당 도구에 규칙 파일이 실제 배포돼 있으면 긴 self-exemption 문단 대신 상대경로 링크를 넣습니다. 규칙 파일이 없으면 원래 본문을 inline fallback으로 유지합니다.recall-toggle.ts의 중복 주입 루프를injectRecallBlockIntoTools호출로 통합하고, enable 직후 갱신된 설정을 전달합니다.기준 커밋은 upstream/main의
3a9a24a6f439ec63f4de7ed1de0a6459a6c5315b입니다. 최종 diff에서hook-handlers.ts와 그 단위 테스트는 upstream과 동일합니다.검증 결과
최종 소스 상태에서 실행했습니다. Node
v24.9.0, npm11.6.0, Vitest3.2.7; dependencies는 upstream lockfile로npm ci --no-audit --no-fund설치했습니다.npx tsc --noEmitnpm run lintnpm run buildpull두 번관련 테스트 명령:
whitelist E2E의 self-exemption 관련 assertions는 유지했습니다. 실제 CLI 검증은 self-mode, remote 없는 샘플 HOME을 사용했고, E2E의 Git remote도 로컬 fixture입니다. 외부 provider나 실제 AI 도구 세션을 검증한 것은 아닙니다.
전체 suite와 upstream baseline 비교
전체 suite를 통과했다고 주장하지 않습니다. 실행 간 변동을 포함한 실제 결과는 다음과 같습니다.
npx vitest run1회차init.test.ts17개 +index-import-help.test.ts1개npx vitest run2회차init.test.ts17개 +index-codebase-help.test.ts2개init.test.ts17개commands-reference.test.ts의 일시적인/src/index.js모듈 해석 실패 1개Baseline은
git archive 3a9a24a6f439ec63f4de7ed1de0a6459a6c5315b로 추출한 별도 디렉터리에 같은 dependencies를 연결해 실행했습니다. 변경하지 않은 baseline의init.test.ts도 이번에 별도로 재실행했고, 동일한 17개 실패와 36개 통과를 확인했습니다.추가 help 실패는 테스트 환경 격리 문제와 일치합니다. upstream의
commands-reference.test.ts는TEAMAI_COMMAND_TABLE_ONLY=1을 설정한 뒤 해제하지 않으며,index.ts는 이 값이 있으면 CLI 실행을 생략합니다. help 테스트는 import 시 CLI 실행을 기대합니다. 두 help 파일과 CLI entry는 이 PR에서 변경하지 않았습니다.이 제어된 재현과 baseline 전체 실행의 모듈 해석 실패는 별개 증거입니다. fresh baseline 전체 실행에서 help 실패 자체가 발생했다고 주장하지 않습니다. 집중 테스트는 안정적으로 통과했으며, 범위 밖 테스트 격리 문제는 이 PR에서 수정하지 않았습니다.
Generated with Claude Code