UI 복잡 버전 234 RED→GREEN 완료(docs/UI_USECASES.md §A~§J): 라이브 테마 전환(DynamicResource 전환+ThemeResolver+HighContrast 테마, 재시작 없음), 디자인 토큰 체계·WCAG 대비 결함 5건 수정, 설정창 카테고리 내비/검색 재구성+AppConfig 40여 속성, 다운로드 관리자(속도/ETA/세그먼트맵/전체 일시정지-이어받기/CSV 내보내기/예약 게이트/중복 확인), 라이브러리·아카이브·작업공간·재생목록·즐겨찾기 편집 뷰, 접근성(자동화 속성 30여 개·커스텀 피어·docs/ACCESSIBILITY.md), 인터랙션(휠 클릭 새 탭·XButton 탐색·F12·영역 캡처·프로필 전환), 상태/알림(토스트·알림센터·온보딩·CrashReportDialog). 보강 테스트 35건+실창 E2E 4건(StaDispatcher 기반 — 프록시 설정이 기시작 핸들러를 건드리는 크래시 실결함 수정 포함) — 453/453 GREEN. 이전 미커밋 작업(CI 워크플로·Mdns·QueueHub·UpdateChecker·AGENTS/CLAUDE·빌드 스크립트) 일괄 포함

This commit is contained in:
Yun Chan 2026-08-17 15:54:53 +09:00
parent 5a0ce730b2
commit 17f2f91152
88 changed files with 9463 additions and 1127 deletions

View file

@ -0,0 +1,40 @@
---
paths:
- "VideoDownloader.App/**"
- "VideoDownloader.Browser/**"
---
# WPF · WebView2 영역 규칙
## UI 스레드
- WebView2·엔진 이벤트는 백그라운드 스레드에서 온다. UI 를 만지려면
`Dispatcher.BeginInvoke` 로 마샬링한다 (`MainWindow.xaml.cs` 전반의 기존 패턴).
- 백그라운드에서 갱신되는 컬렉션은 `BindingOperations.EnableCollectionSynchronization`
으로 락을 등록한다 (`Core/DownloadManager.cs:58` 참조). 이걸 빼면 랜덤하게 죽는다.
## dispose 이후 이벤트
WebView2 가 dispose 된 뒤에도 이벤트 콜백이 도착한다. 콜백 안에서 `CoreWebView2` 를 만질 때는
로컬 변수로 받아 null 을 확인하고 try 로 감싼다 — `UpdateNavState`(`MainWindow.xaml.cs:453`)가
기준 패턴이다. 탭을 닫는 경로를 건드렸다면 이 방어를 반드시 확인한다.
## 탭
- Environment 는 전 탭 공유, WebView2 인스턴스는 탭마다. 이 구조를 바꾸지 않는다.
- 팝업(`NewWindowRequested`)은 deferral 패턴으로 새 탭에 붙인다. 앱 통제를 벗어나게 두지 않는다.
## 거대 파일
`MainWindow.xaml.cs`(86KB) · `MainWindow.xaml`(55KB)은 통째로 읽지 않는다.
Grep 으로 심볼 위치를 찾고 해당 구간만 Read 한다. 분할 리팩터링은 별도 승인 사항이다.
## 프로세스 실행
외부 프로세스는 문자열 인자 연결로 실행하지 않는다. `ProcessStartInfo` 에 인자를 배열로 넘긴다
(과거 `explorer.exe` 인자 인젝션 취약점이 있었다).
## 테스트 접근
`VideoDownloader.App` 은 `InternalsVisibleTo("VideoDownloader.Tests")` 가 걸려 있다.
테스트를 위해 멤버를 `public` 으로 승격하지 말고 `internal` 로 두면 된다.

View file

@ -0,0 +1,44 @@
---
paths:
- "VideoDownloader.Core/**"
- "VideoDownloader.Server/**"
---
# 엔진 · 서버 영역 규칙
## 플랫폼 중립
`Core` 는 `net8.0` 이다. WPF·Windows 전용 API 를 끌어들이지 않는다.
Avalonia/MAUI 가 같은 엔진을 쓰므로 여기 들어간 Windows 의존은 크로스플랫폼 이관을 막는다.
OS별 경로는 `Platform/CorePaths.cs` 를 통한다.
## 외부 프로세스 (ffmpeg 등)
`Platform/ProcessModels.cs` 의 실행 헬퍼를 쓴다. 직접 `Process.Start` 를 새로 쓰지 않는다.
그 헬퍼가 보장하는 것들을 우회하면 과거 버그가 재발한다:
- stdout/stderr **동시 드레인** — 안 읽으면 파이프 버퍼가 차서 ffmpeg 가 hang 한다
- 취소·실패 시 `Kill(entireProcessTree: true)` — 안 하면 좀비 프로세스가 남는다
- 인자는 배열 전달 — 문자열 연결 금지
- exit code ≠ 0 이면 예외 — 조용히 넘어가면 실패가 `.ts` 잔재로만 드러난다
## 취소
모든 장기 작업은 `CancellationToken` 을 끝까지 전달한다. 중간에 삼키지 않는다.
`Parallel.ForEachAsync` 는 `ParallelOptions.CancellationToken` 에 넣는다
(`Hls/HlsDownloader.cs:77` 패턴).
## 동시성
세그먼트 병렬도는 `AppConfig.Concurrency`, 동시 다운로드 수는 `MaxConcurrentDownloads`(기본 3)로
제어한다. 하드코딩된 병렬도를 새로 만들지 않는다 — 소켓 고갈로 이어진다.
## 데이터 경로
사용자 데이터는 `CorePaths` 를 통해서만 접근한다. `VD_DATA_DIR` 환경변수 오버라이드가
E2E 테스트 격리의 공식 후크이므로, 경로를 직접 조립하면 테스트가 실사용자 폴더를 오염시킨다.
## 서버
`Server` 는 App 안에서 호스팅되는 라이브러리다. 포트 0 이면 빈 포트를 자동 할당한다.
LAN 노출 API 이므로 인증·경로 검증을 우회하는 엔드포인트를 추가하지 않는다.

View file

@ -0,0 +1,40 @@
---
paths:
- "VideoDownloader.Mobile/**"
- "VideoDownloader.Mobile.Core/**"
---
# MAUI 모바일 영역 규칙
## 로직은 Mobile.Core 로
`VideoDownloader.Mobile` 은 `net10.0-android`(macOS 에서는 +`net10.0-ios`)이라
`VideoDownloader.Tests`(net8.0-windows)가 참조할 수 없다. **테스트 가능한 로직은
`VideoDownloader.Mobile.Core`(net8.0)에 둔다.** UI 프로젝트에는 화면 결선만 남긴다.
새 기능을 UI 프로젝트에 통째로 넣으면 검증할 방법이 사라진다.
## 링커 함정
Release/링커 트리밍에서 참조가 정적으로 안 보이는 타입은 통째로 잘려나가고,
**런타임 `FileNotFoundException` 으로만** 드러난다. 빌드는 통과한다.
실제 사례: 모달 `NavigationPage` 가 크래시 → `Xamarin.AndroidX.LocalBroadcastManager` 를
csproj 에 명시 참조해 해결. 리플렉션·XAML 로만 참조되는 타입을 쓸 때 이 함정을 의심한다.
## 플랫폼 분기
TFM 조건은 `$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'android'`
형식으로 쓴다(csproj 기존 패턴). Windows 개발머신에는 iOS 워크로드가 없으므로
iOS TFM 은 macOS 조건부로만 켠다 — 이 조건을 무조건부로 바꾸면 Windows 빌드가 깨진다.
## 빌드 · 검증
```powershell
dotnet build VideoDownloader.Mobile -f net10.0-android
```
- MAUI 워크로드가 없으면 실패한다. 없는 환경이면 `Mobile.Core` 만 고치고
UI 결선은 워크로드가 있는 환경에서 확인하도록 사용자에게 알린다.
- 화면 동작을 바꿨으면 에뮬레이터에 배포해 확인한 내용을 보고에 적는다.
단위 테스트만으로 "됐다"고 하지 않는다.

44
.claude/rules/tests.md Normal file
View file

@ -0,0 +1,44 @@
---
paths:
- "VideoDownloader.Tests/**"
---
# 테스트 영역 규칙
## 배치
`VideoDownloader.Tests/<영역>/<대상>Tests.cs` — 영역은
`Browser` · `E2E` · `Infrastructure` · `Mobile` · `Platform` · `Server`.
새 영역을 만들기 전에 기존 영역에 맞는지 먼저 본다.
## 격리 (필수)
사용자 데이터 경로를 건드리는 테스트는 `VD_DATA_DIR` 을 임시 디렉터리로 지정하고
끝나면 `null` 로 되돌린다. `E2E/MainWindowSmokeE2ETests.cs:17` 이 기준 패턴이다.
이걸 빼면 테스트가 실사용자의 설정·세션·프로필을 덮어쓴다.
## WPF 테스트
테스트 프로세스에는 `App.xaml` 리소스가 없다. WPF 요소를 만드는 테스트는
`Infrastructure/WpfTestApp.Ensure(dispatcher)` 를 먼저 호출해 테마 사전을 로드한다.
직접 `new Application()` 을 만들지 않는다.
## 금지
- 통과시키려고 테스트를 **삭제하거나 `Skip` 처리하지 않는다.** 회귀를 숨기는 것이다.
- 실제 네트워크에 의존하는 단정을 새로 만들지 않는다. 외부 사이트가 바뀌면 무작위로 깨진다.
(`Cli` 의 `--dashparse` 같은 수동 검증 도구는 예외 — 테스트가 아니다.)
- 시간·순서에 의존하는 `Thread.Sleep` 기반 단정을 쓰지 않는다.
## 알려진 경고
이 프로젝트에 남아 있는 경고: CS8602(null 역참조) 몇 건, xUnit1031(블로킹 대기),
xUnit2013(`Assert.Equal` 로 개수 확인). **개수를 늘리지 않는다.**
새 테스트에서는 `Assert.Single`/`Assert.Empty` 와 `async` 대기를 쓴다.
## 실행
```powershell
dotnet test VideoDownloader.Tests # 전체 (~13초)
dotnet test VideoDownloader.Tests --filter "FullyQualifiedName~<이름>" # 단건
```