Skip to content

feat: 마지막으로 고른 Unified/Split 레이아웃을 기억한다 - #105

Merged
say8425 merged 2 commits into
mainfrom
feat/remember-diff-style
Oct 27, 2026
Merged

say8425 merged 2 commits into
mainfrom
feat/remember-diff-style

Conversation

@say8425

@say8425 say8425 commented Oct 27, 2026

Copy link
Copy Markdown
Owner

요약

툴바에서 고른 Unified/Split 레이아웃을 기억합니다. 다음에 뷰어를 열면 마지막에 고른 레이아웃으로 시작합니다.

  • 결정 순서는 URL(?style=) → localStorage(diffdeck:diff-style) → 기본값(unified)입니다. flatten·트리 위치와 같은 방식입니다.
  • 툴바 토글을 누를 때만 저장합니다.
  • --split은 그 실행에만 적용되고 저장값을 바꾸지 않습니다. 기존 "뷰 플래그는 이번 실행에만" 계약을 따릅니다.
  • --split 설명을 "unified is the default"에서 "else the last layout you picked"로 바꿨습니다. cli.ts HELP, README.md와 번역 4종, apps/viewer/README.md, SKILL.md를 함께 고쳤습니다.
  • .claude/rules/viewer.md의 프리퍼런스 규칙에 Unified/Split을 저장 대상으로 옮겼습니다.

화면 모양은 바뀌지 않아 스크린샷은 붙이지 않았습니다.

테스트 계획

  • bun run typecheck, bun run lint, bun run format:check
  • bun run test:coverage: resolveDiffStyle의 URL·저장값·기본값 순서 유닛 테스트
  • diffstyle-persist.e2e.ts: Split을 고르고 새로고침하면 Split, 다시 Unified로 바꾸고 새로고침하면 Unified인지 확인
  • diffstyle-persist.e2e.ts: --split으로 띄워도 저장값이 생기지 않는지 확인
  • 판별력: main.ts의 저장 줄을 지우면 새로고침 스펙이 실패하는 것을 확인
  • 기존 diffstyle-scroll·tree-fold-sync-diffstyle·flags-sync e2e 통과

🤖 Generated with Claude Code

https://claude.ai/code/session_01A1spPGo8kYzkrEzDdJ7XuM

툴바에서 고른 레이아웃을 localStorage(`diffdeck:diff-style`)에 저장하고,
다음 실행에서 URL → 저장값 → 기본값(unified) 순으로 정한다. `--split`은
그 실행에만 적용되고 저장값을 바꾸지 않는다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A1spPGo8kYzkrEzDdJ7XuM
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

PR이 닫혀 프리뷰를 지웠습니다.

리뷰 반영: 저장값 "unified"를 먼저 두고 --split으로 띄워 값이 남는지
본다. resolveDiffStyle의 알 수 없는 값 폴백을 유닛 테스트에 더하고,
--split 설명에 첫 실행은 unified라는 것을 되살린다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A1spPGo8kYzkrEzDdJ7XuM
@say8425
say8425 enabled auto-merge (squash) October 27, 2026 09:05
@say8425
say8425 merged commit 14108db into main Oct 27, 2026
14 of 15 checks passed
@say8425
say8425 deleted the feat/remember-diff-style branch October 27, 2026 09:28
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