Repository navigation
feat(notify): 관계있다로 답한 변경의 적용 범위를 AI에게 전달 - #63
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…(SIL-98) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
바뀐 결정 사항 찾기 · PR #63
@RCatMin 이 PR에서 결정 사항이 여러 개 바뀐 것 같아요. 한 번에 하나만 저장해서 직접 확인을 부탁드려요.
+ '모든 질문이 끝난 뒤 원래 요청을 처리하기 전에, scope.code 경로부터 훑어서 지금 결과물이 value와 다른 곳을 찾는다. 고치지는 않는다. 사람이 고치라고 할 때만 고친다. ' +
+ 'scopeCheck가 있으면 그 guide대로 적용 범위부터 훑어서 새 값과 다른 곳을 알려 주기만 해요. ' +
+ if (!checked.affected || !check) return result;
+ guide: SCOPE_GUIDE,
+ : '모든 변경에 답했거나(잘 모르겠다로 남긴 것 포함) 사용자가 질문을 닫았으면, respond 결과에 scopeCheck 가 있던 변경마다 그 guide 대로 적용 범위를 훑어 새 값과 다른 곳을 먼저 알려 준다(고치지는 않는다). 그다음 사용자의 원래 요청을 처리한다. 그 밖에는 확인한 변경에 대해 따로 말하지 않는다.';
+ 'scope가 null이거나 비어 있으면 "바뀐 결정 사항에 적용 범위가 적혀 있지 않아 다른 곳을 찾지 못했어요. 필요하신 경우 제목과 값으로 코드를 찾아볼게요."라고만 말한다. ' +
+ '적힌 경로를 하나도 찾지 못하면 "적힌 경로 <경로>를 찾지 못했어요. 필요하신 경우 제목과 값으로 코드를 찾아볼게요."라고만 말한다. ' +
+ '어느 경우든 사람이 찾아 달라고 할 때만 title과 value로 코드를 찾아본다. 조용히 넘어가지 않는다.';
+ scope: hasScope(scope) ? scope : null,결정 사항은 그대로 두었어요. 바뀐 게 맞으면 Claude Code에서 "결정 사항을 바꿔 줘"라고 말해 주세요. 바뀐 게 아니면 이 댓글은 넘어가셔도 돼요. |
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.
관련 이슈
변경점
StandardScope를 그대로 가져다 씁니다. DB는 바뀌지 않습니다list_pending항목의standard.scope에 그 버전의 적용 범위를 담았습니다. 웹 앱도 같은 목록 함수를 써서 같은 칸을 받습니다.respond가 관계있다로 저장하면 결과에scopeCheck(결정 사항 ID, 버전, 적용 범위, 안내)를 더합니다. 관계없다면 넣지 않습니다.scopeCheck를 따르라는 문장을 더했습니다.notify.answer.scope-hint,notify.answer.scope-missing을 더했습니다. T1 절차로 묻고 팀 DB에 저장했습니다(10/16 11:17, 11:18).scopeCheck에 결정 사항 제목과 값을 더했습니다.notify.answer.scope-missing을 v2로 바꿨습니다. T1 절차로 묻고 팀 DB에 저장했습니다(10/16 12:14).packages/notify,apps/mcp-serverREADME를 고쳤습니다.변경 사유
알림에는 이전 값, 지금 값, 이유, 이슈만 있어서 관계있다로 답해도 AI가 어디를 봐야 할지 단서가 없었습니다.
10/14 호윤님 결정대로 서비스는 단서(적용 범위)까지만 주고, 코드를 찾는 것은 에이전트, 고칠지는 사람이 정합니다.
적힌 코드 경로 38개 중 10개만 실제로 있어서, 경로를 못 찾을 때 조용히 넘어가지 않게 했습니다.
검증
pnpm check를 통과했습니다 (notify 91개, 전체 vitest 261개).respond테스트: 적용 범위가 없을 때scope: null과 "적혀 있지 않아요" 안내, 적용 범위가 있을 때 그 버전의 값, 관계없다면scopeCheck없음.list_pending에 같은 적용 범위가 들어 있는 것, 웹 방식과 같은 목록인 것을 확인했습니다.python3 team-wiki/scripts/check_wiki.py team-wiki: 오류 0개, 경고 0개.향후 확인이 필요한 점
StandardScope를 쓴 것은 정민님이 확인했습니다 (10/16).pnpm sil scope-check).