Skip to content

feat(notify): 관계 질문에 "잘 모르겠다" 선택지 추가 - #61

Merged
RCatMin merged 2 commits into
mainfrom
feat/sil-67-unsure-later
Oct 16, 2026
Merged

RCatMin merged 2 commits into
mainfrom
feat/sil-67-unsure-later

Conversation

@RCatMin

@RCatMin RCatMin commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

관련 이슈

  • Part of SIL-67 ([개발] 담당자에게 알리고 답 기록하기)
  • SIL-71 사내 테스트 의견, 10/12 호윤님 회의 결정
  • 시안: SIL-68 부건님 10/13 시안 (notice.py "잘 모르겠다", prompt_submit.py)

개선 사항

1. 위키와 팀 DB

  • notify.answer.two-questions를 v2로 바꿨습니다. 관계 질문 보기가 관계있다/관계없다/잘 모르겠다 세 개가 됩니다.
  • notify.answer.unsure-later를 새로 더했습니다. 잘 모르겠다를 고르면 답으로 기록하지 않고 대기로 남깁니다.
  • 둘 다 T1 절차로 묻고 팀 DB에 저장했습니다(10/16 10:26). 위키 파일, INDEX 기준 수, CHANGELOG 두 줄을 고쳤습니다.

2. 터미널 흐름

  • 관계 질문에 세 번째 보기 "잘 모르겠다 · 지금은 판단하기 어려워요. 나중에 답할게요"를 더했습니다.
  • 고르면 Claude가 respond를 unsure: true로 부릅니다. DB에는 아무것도 쓰지 않습니다.
  • hooks가 노란 막대 한 줄 "나중에 답할 수 있게 남겨 두었어요. "결정 사항 확인"으로 다시 볼 수 있어요."를 띄우고, 남은 변경이 있으면 다음 상세로 넘어갑니다.
  • 이번 세션에는 다시 묻지 않고, 작업을 마친 뒤 한 줄의 건수에서도 뺍니다.
  • "결정 사항 확인"을 입력하거나 새 세션을 열면 다시 묻습니다.
  • 앞은 잘 모르겠다, 끝은 답한 경우 마무리 줄은 "모두"를 빼고 "확인할 변경 N건을 답했어요."로 씁니다.
  • hooks가 없는 사람을 위해 서버 안내문 8번에도 같은 방법을 적었습니다.

개선 사유

사내 테스트에서 바로 판단하기 어려울 때 고를 보기가 없어 Esc나 Chat about this로 넘어간다는 의견이 있었습니다(SIL-71).
10/12 회의에서 정했지만 시안 브랜치에만 있고 main 위키와 팀 DB에는 없어서, 이번에 함께 맞췄습니다.

검증

  • pnpm check를 통과했습니다 (notify 91개, 전체 vitest 259개).
  • respond 도구: 잘 모르겠다는 DB를 부르지 않음, 다른 사람의 변경은 거절, 관계와 잘 모르겠다가 모두 없으면 빠진 칸.
  • hooks: 한 줄 + 다음 상세, 남은 게 없으면 한 줄만, 이번 세션 다음 입력과 작업 뒤 한 줄에서 빠짐, "결정 사항 확인"과 새 세션에서 다시 물음, 답한 건수만 말하는 마무리 줄.
  • python3 team-wiki/scripts/check_wiki.py team-wiki: 오류 0개, 경고 0개.

향후 확인 필요점

  • 실제 Claude Code에서 보기 세 개 선택 화면을 확인하지 않았습니다. 부건님 시안 문서에는 상세와 합쳐 13줄이 된다는 확인 거리가 있습니다(호윤-14).
  • 시안은 잘 모르겠다를 고른 횟수를 따로 세지만, 팀 DB에는 그런 칸이 없어 세지 않습니다. 필요하면 정민님과 따로 정하겠습니다.
  • 웹 앱의 답하기 화면은 자체 보기를 써서 이 PR의 영향을 받지 않습니다. 앱에도 넣을지는 신지님·호윤님과 따로 정해야 합니다.
  • 서버만 바뀌어서 팀원 재등록은 필요 없습니다.

RCatMin and others added 2 commits October 16, 2026 10:27
…r 추가 ("잘 모르겠다" 보기)

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@RCatMin RCatMin changed the title feat(notify): 관계 질문에 "잘 모르겠다" 보기 추가 feat(notify): 관계 질문에 "잘 모르겠다" 선택지 추가 Oct 16, 2026
@RCatMin
RCatMin merged commit 9240597 into main Oct 16, 2026
@github-actions

Copy link
Copy Markdown

바뀐 결정 사항 찾기 · PR #61

지금은 결정 사항을 저장하지 않고 결과만 알려 주는 기간이에요. 저장했다면 아래처럼 했을 거예요.

@RCatMin 이 PR에서 결정 사항이 여러 개 바뀐 것 같아요. 한 번에 하나만 저장해서 직접 확인을 부탁드려요.

notify.answer.two-questions 담당자는 관계 보기 3개 중 하나를, 관계있으면 알았나를 고른다

  • 지금 값: 첫 질문은 이 변경이 내 작업에 영향을 주는지(관계있다/관계없다/잘 모르겠다)이고, 관계있다면 미리 알았는지(알았다/몰랐다)를 묻는다
  • 바뀐 것 같은 값: 첫 질문은 이 변경이 내 작업에 영향을 주는지(관계있다/관계없다/잘 모르겠다)이고, 관계있다면 미리 알았는지(알았다/몰랐다)를 묻는다
  • 코드에서 바뀐 것: 관계 질문의 보기가 관계있다와 관계없다 두 개에서 잘 모르겠다를 더한 세 개로 바뀌었고, 결정 사항은 이미 이 값으로 고쳐져 있어요.
  • PR에 적힌 이유: 사내 테스트에서 바로 판단하기 어려울 때 고를 보기가 없어 Esc나 Chat about this로 넘어간다는 의견이 있었습니다(SIL-71).
-export const RELATION_LABELS = { affected: '관계있다', unaffected: '관계없다' };
+  unsure: '잘 모르겠다',
+          label: RELATION_LABELS.unsure,
+          description: '지금은 판단하기 어려워요. 나중에 답할게요',
+    expect(labels('relation')).toEqual(['관계있다', '관계없다', '잘 모르겠다']);

notify.answer.unsure-later "잘 모르겠다"를 고르면 답하지 않은 변경으로 남겨 나중에 답한다

  • 지금 값: 담당자가 잘 모르겠다를 고르면 답으로 기록하지 않고 그 확인할 변경을 대기로 남긴 채 다음 변경으로 넘어간다
  • 바뀐 것 같은 값: 담당자가 잘 모르겠다를 고르면 답으로 기록하지 않고 그 확인할 변경을 대기로 남긴 채 다음 변경으로 넘어간다
  • 코드에서 바뀐 것: 잘 모르겠다를 고르면 답을 저장하지 않고 대기로 남긴 채 다음 변경으로 넘어가는 동작이 새로 생겼고, 결정 사항은 이미 이 값으로 들어가 있어요.
  • PR에 적힌 이유: 사내 테스트에서 바로 판단하기 어려울 때 고를 보기가 없어 Esc나 Chat about this로 넘어간다는 의견이 있었습니다(SIL-71).
+    if (input.unsure) {
+      // 잘 모르겠다: DB에 아무것도 쓰지 않아요. 이번 세션에 다시 묻지 않는 것은 hooks가 기억해요
+      flow.deferred.add(pendingCheckId);
+      lines.push(unsureLine());
+    const remaining = items.filter((i) => !flow.deferred.has(i.pendingCheckId));

결정 사항은 그대로 두었어요. 바뀐 게 맞으면 Claude Code에서 "결정 사항을 바꿔 줘"라고 말해 주세요. 바뀐 게 아니면 이 댓글은 넘어가셔도 돼요.

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