Skip to content

fix(detect): T1 hooks 화면을 프로토타입에 맞춤 — 긴 줄 색, 설명 줄 짧게 (SIL-107) - #67

Merged
space-angel merged 2 commits into
mainfrom
fix/sil-107-t1-hooks-match-prototype
Oct 22, 2026
Merged

space-angel merged 2 commits into
mainfrom
fix/sil-107-t1-hooks-match-prototype

Conversation

@k0nghaa

@k0nghaa k0nghaa commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

관련 이슈

SIL-107 [개발] T1 hooks 화면을 프로토타입에 맞추기 (줄바꿈, 간결하고 읽기 쉬운 내용)
https://linear.app/pj-3/issue/SIL-107

무엇을 바꿨나요

  • 긴 줄이 넘어가면 회색으로 보이던 문제 (apps/detect/src/t1/screens.ts)
    • 예외 안내(완전히 같음·뜻이 같음·되돌릴 값이 지금과 같음)의 기존 결정 사항 줄을 다른 T1 화면처럼 76칸에서 직접 나눠요.
    • 넘어간 줄도 같은 들여쓰기와 기본 글자색으로 칠해요. [영역] 제목은 그대로 굵게 보여요.
  • ② 다른 결정 사항 화면의 설명 줄 (CLAUDE.md T1 2번)
    • why를 프로토타입처럼 바뀌는 값만 짧게 쓰도록 지시를 바꿨어요.
    • 변경: "40자 이하로 정한 길이를 32자 이하로 다시 정해요"
    • 어긋남: "결정 사항 제목(최대 40자)을 그대로 쓰면 32자에서 잘려요"

왜 바꿨나요

  • 터미널이 알아서 넘긴 줄은 꾸밈이 풀려 hook 기본색(회색)이 돼요. 그래서 일부러 나눈 줄이 아닌데 다른 문장처럼 보였어요.
  • ② 설명 줄이 "지금 결정 사항은 …인데, 부탁하신 작업은 …"처럼 길게 나왔어요. 프로토타입보다 한눈에 보기 어려웠어요.
  • 프로토타입: https://claude.ai/artifact/GrDAiKvv5A9PTUZxxJR5gf

어떻게 확인했나요

  • pnpm check 통과: 테스트 266개 통과, 1개 건너뜀(.env에 DB 주소가 있으면 건너뛰는 기존 테스트)
  • 긴 값을 넣으면 76칸을 넘지 않고, 넘어간 줄도 기본 글자색으로 시작하는지 테스트를 더했어요.
  • 실제 터미널에서 notice.ts로 "완전히 같음" 화면을 다시 띄웠어요.

직접 확인하지 못한 부분

  • ② 설명 줄은 Claude가 지시를 따르는지에 달려 있어요. 실제 T1 흐름으로 다시 띄워 보지는 못했어요.
  • 터미널 폭이 78칸보다 좁으면, 터미널이 다시 알아서 넘겨서 회색이 나올 수 있어요.
  • ③ 등록될 모습의 "값" 줄은 부건님과 정해서 지금처럼 저장되는 문장 전체를 그대로 보여 주기로 했어요(SIL-107 3번). 코드는 이미 그렇게 동작해서 이 PR에서 바꾼 것이 없어요. Claude가 값을 짧게 넘기지 않도록 지시하는 건 BDT-108에서 함께 해요.

🤖 Generated with Claude Code

@space-angel
space-angel merged commit b300e7a into main Oct 22, 2026
@github-actions

Copy link
Copy Markdown

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

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

@k0nghaa 이 PR에서 결정 사항이 바뀐 것 같은데 확실하지 않아요.

② 다른 결정 사항 화면의 설명 줄을 '바뀌는 값만 짧게 한 문장'으로 쓰기로 한 것을, 팀 결정 사항으로 새로 등록할까요, 아니면 이 화면 지시에만 해당하는 규칙인가요?

결정 사항은 그대로 두었어요. 바뀐 게 맞으면 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.

2 participants