Skip to content

[시험] 알림 제목 32자 — PR 경로 확인용, 머지하지 않음 (SIL-66) - #35

Closed
k0nghaa wants to merge 1 commit into
mainfrom
test/sil-66-pr-path-demo
Closed

k0nghaa wants to merge 1 commit into
mainfrom
test/sil-66-pr-path-demo

Conversation

@k0nghaa

@k0nghaa k0nghaa commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

머지하지 않는 시험용 PR이에요. SIL-66 PR 경로(바뀐 결정 사항 찾기)를 실제 PR로 확인하려고 만들었어요. 확인이 끝나면 닫고 브랜치를 지워요.

무엇을 바꿨나요

Windows 알림 창에서 제목이 32자를 넘으면 끝이 잘려 보여요. 그래서 앱 알림 제목을 32자까지만 쓰도록 줄였어요.
결정 사항 app.notify.title-length(앱 알림 제목은 40자 이하다)를 32자로 바꾸는 변경이에요.

🤖 Generated with Claude Code

@k0nghaa

k0nghaa commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator Author

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

시험 실행이라 저장하지 않았어요. 실제로 머지되면 아래처럼 해요.

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

결정 사항 "앱 알림 제목은 40자 이하다"를 32자로 바꾸려는 건가요, 아니면 결정 사항은 그대로 두고 코드만 더 짧게 자르려는 건가요?

app.notify.title-length 앱 알림 제목은 40자 이하다

  • 지금 값: 운영체제 알림 창에 뜨는 제목은 공백 포함 40자 이하로 쓴다
  • 바뀐 것 같은 값: 운영체제 알림 창에 뜨는 제목은 공백 포함 32자 이하로 쓴다
  • 이렇게 본 까닭: 코드가 알림 제목을 32자로 잘라 쓰게 되어 결정 사항의 40자보다 엄격해졌지만, 40자 이하라는 결정 사항을 어기지는 않아서 결정 사항을 바꾼 것인지 확실하지 않아요.
+/** OS 알림 제목 최대 길이 (공백 포함). Windows 알림 창에서 32자를 넘으면 끝이 잘려요 (app.notify.title-length) */
+const MAX_NOTIFICATION_TITLE_LENGTH = 32;
+    : `${title.slice(0, MAX_NOTIFICATION_TITLE_LENGTH - 1)}…`;
+        toNotificationTitle(notification.title),

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