Skip to content

feat(detect): T1 새 결정 사항 화면을 프로토타입에 맞춤 (SIL-94) - #50

Merged
k0nghaa merged 3 commits into
mainfrom
feat/sil-94-t1-new-standard-screens
Oct 15, 2026
Merged

k0nghaa merged 3 commits into
mainfrom
feat/sil-94-t1-new-standard-screens

Conversation

@k0nghaa

@k0nghaa k0nghaa commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

관련 이슈

SIL-94 [개발] T1 직접 요청 경로: 사람이 결정 사항을 직접 바꿔 달라고 할 때 (새 결정 사항 — 기존 결정 사항이 없을 때)
https://linear.app/pj-3/issue/SIL-94

무엇을 바꿨나요

  • 다른 결정 사항이 없을 때: Claude가 글로 쓰던 "요청하신 변경과 다른 결정 사항이 없어요."(direct.noConflict) 대신, 다른 결정 사항 화면을 conflicts: []로 띄워 초록 막대 한 줄만 보여 줘요. conflicts.ts는 빈 목록을 직접 요청(trigger: "direct")일 때만 받아요.
  • ③ 새 결정 사항 표시: preview.ts에 "isNew": true를 넘기면 제목 줄 오른쪽 끝에 "새 결정 사항"을 붙여요(자리가 모자라면 다음 줄 오른쪽 끝). 받는 이슈는 0개로 "연결된 이슈가 없어서 알림을 받는 사람은 없어요."를 보여요.
  • 질문: 새 결정 사항은 "이대로 등록할까요?"(preview.confirm)로 물어요. CLAUDE.md에 direct.confirm을 쓰지 않는다고 분명히 적었어요.
  • 설명 글 막기: 두 화면 hook이 에이전트에게만 가는 말(additionalContext)로 "화면을 이미 보여 줬으니 설명하지 말고 다음 단계로 가라"를 전해요. "다른 결정 사항 화면을 띄웠어요." 같은 글이 붙던 문제예요.
  • 글자 색: Claude Code가 hook 글을 흐리게 칠해서 내용(39)이 회색으로, 초기화(0) 뒤 글이 흰색으로 나오던 문제를 고쳤어요. 칠할 때 먼저 모두 풀고(0;) 끝나면 흐림(0;2)으로 돌려, 값·이유·항상 적용 값·선은 흰색, 이름표·"새 결정 사항"·안내 줄은 회색이에요(프로토타입과 같음).

왜 바꿨나요

부건님 프로토타입 "T1 결정 사항 등록"(10/14)의 "새 결정 사항" 흐름과 맞추려고요. 경우 1은 #48, 경우 2는 #47에서 맞췄어요.

어떻게 확인했나요

  • pnpm check 통과 (main 위로 올리면서 커밋마다 실행, 204개 통과·1개 건너뜀: .env에 DB 주소가 있으면 건너뛰는 기존 테스트 apps/mcp-server/src/http.test.ts:591)
  • 새 세션 터미널에서 "앱 알림 본문은 2줄 이하로 쓰기로 결정 사항 만들어 줘"로 초록 막대 → 이유 묻기 → ③("새 결정 사항" 표시)까지 확인했어요. 결정 사항은 지울 수 없어서 저장하지 않았어요.

스크린샷

image

직접 확인하지 못한 부분

  • 글자 색 고친 뒤(흰색/회색)와 새 결정 사항 질문 문구 고친 뒤의 터미널 화면은 마지막 커밋 뒤에 다시 보지 못했어요.
  • 이유를 묻는 문구가 "이 결정 사항을 바꾸는 이유…"라 새 결정 사항에는 조금 어색해요. 프로토타입도 같은 문구라 그대로 뒀어요.
  • 이번 PR에 없는 것: ④ 등록 완료 hook, 예외 6가지, "아니요" 뒤 롤백 지시(SIL-95).

🤖 Generated with Claude Code

k0nghaa and others added 3 commits October 15, 2026 16:13
기존 결정 사항이 없는 직접 요청은 다른 결정 사항 화면을 conflicts: []로 띄워
초록 막대 "요청하신 변경과 다른 결정 사항이 없어요" 한 줄만 보여요(Claude 글 noConflict 대신).
③은 preview.ts에 isNew: true를 넘기면 제목 줄 오른쪽 끝에 "새 결정 사항"을 붙이고,
받는 이슈는 0개로 "연결된 이슈가 없어서 알림을 받는 사람은 없어요."를 보여요.
프로토타입 "T1 결정 사항 등록"(10/14)과 맞춰요.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
에이전트는 hook 화면이 사람에게 떴는지 몰라 "다른 결정 사항 화면을 띄웠어요."를 덧붙였어요.
두 hook이 화면(systemMessage)과 함께 에이전트에게만 가는 말(additionalContext)로
"이미 보여 줬으니 설명하지 말고 다음 단계로 가라"를 전하고, CLAUDE.md 규칙도 분명히 해요.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude Code가 hook 글을 흐리게 칠해서 기본 글자색(39)만으로는 내용이 회색으로 나오고,
초기화(0) 뒤 글은 흰색이 됐어요. 칠할 때 먼저 모두 풀고(0) 끝나면 흐림(2)으로 돌려
값·이유·항상 적용 값·선은 흰색, 이름표·"새 결정 사항"·안내 줄은 회색으로 보여요.
새 결정 사항은 "이대로 등록할까요?"로 묻도록 CLAUDE.md를 분명히 해요.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@k0nghaa
k0nghaa merged commit 9903d85 into main Oct 15, 2026
@github-actions

Copy link
Copy Markdown

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

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

바뀐 결정 사항이 없어요. (살펴본 영역: detect, wiki, copy, work, notify)

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