# COMPLETED.md — DeepDrftHome Archive of items that have moved out of `PLAN.md` and `CMS-PLAN.md`. Per `CONTEXT.md §6`, completed items are moved here rather than deleted. Each entry preserves the original "What / Why / Shape" body so this file reads as a decision record, not just an outcome list. Newest entries at the top. Group by phase/wave header (mirroring `PLAN.md` / `CMS-PLAN.md` themes) when there are enough entries to warrant it. --- ## WaveformSeeker Wave 2 — DOM seekbar + Interop module **Status:** W2 (WaveformSeeker component) landed on 2026-06-05 (branch `waveform-w2-seeker`, pending merge to dev). ### W2 — WaveformSeeker component (seekbar replacement) **Landed 2026-06-05.** Implemented the interactive WaveformSeeker component: a bar-chart-styled seekbar replacing `MudSlider` in `PlayerSeekZone`, with DOM-rendered progress split via CSS and lazy-loaded pointer-capture drag interop. **Component changes (`DeepDrftPublic.Client/Controls/AudioPlayerBar`):** - `WaveformSeeker.razor` (+ `.cs`, `.css`) — new component consuming `WaveformProfile double[]?` and `Duration`, rendering bars as DOM elements with clip-overlay progress. Single CSS variable (`--seek-position`) changes per seek gesture; no per-bar re-render. - Pointer-capture drag wired via `waveformSeeker.js` (ES module, lazy-loaded). Calculates seek target from click/drag position and invokes `OnSeekRequested` callback (delegates to `IPlayerService.SeekAsync`). - Flat floor-height fallback when profile is unavailable — seek gesture always works, with or without loudness data. - `PlayerSeekZone.razor` — now hosts `WaveformSeeker` in place of the removed `MudSlider` placeholder. **Interop changes (`DeepDrftPublic/Interop/audio/`):** - New `waveformSeeker.ts` module (separate from the TS audio bundle) — `PointerCaptureHandler` class managing `pointerdown` / `pointermove` / `pointerup` lifecycle. Compiled to `waveformSeeker.js` in `wwwroot/js/audio/`. - Module loaded on first use (not bundled with audio stack) to defer its parse cost until the player is expanded and the seekbar is visible. **`.gitignore` scoping:** - Added scoped negation to track hand-authored `waveformSeeker.js` alongside existing TS-output ignore rule — allows the compiled JS to be committed for fast startup without committing intermediate TS compiler outputs. **Service changes (`IPlayerService` / `AudioPlayerService` / `StreamingAudioPlayerService`):** - New `WaveformProfile double[]?` property added to service interface and implementations. - Fetched fire-and-forget on track load via `GetWaveformProfileAsync(trackId, cancellationToken)` — existing HTTP call from W1-T2. - Cancellable via the track-reset flow (same cancellation token that stops spectrum animation). - Cleared on reset with all other track state. **Testing:** - Manual verification: seekbar renders flat when profile unavailable; dragable when profile present; CSS clip-overlay tracks seek position correctly. **Architecture notes:** - WaveformSeeker does not re-fetch the profile — it consumes the same `IPlayerService.WaveformProfile` fetched during track load. No additional HTTP round-trip per seek gesture. - Interop module (`waveformSeeker.js`) is independent of the audio playback stack — can be updated or replaced without touching audio scheduling logic. - Pointer-capture semantics ensure seek is responsive even when the browser's event queue is saturated by animation frames. - Flat fallback ensures seek gestures always work, even on tracks with no profile data (uploaded before W1, or on profile-generation failure). --- ## WaveformSeeker Wave 1 — Loudness profile + layout refactor **Status:** W1-T1 (backend loudness computation), W1-T2 (HTTP transport), and W1-T3 (player layout refactor) landed on 2026-06-05. ### W1-T1 — Backend waveform loudness profiling **Landed 2026-06-05.** Implemented Phase 1 of the WaveformSeeker feature (`product-notes/spectrum-seeker.md`): loudness-profile computation and storage for preprocessed waveform data. **Backend changes (`DeepDrftContent`):** - Added `ILoudnessAlgorithm` strategy interface for swappable loudness computation. - Implemented `RmsLoudnessAlgorithm` — first loudness algorithm using root-mean-square; future LUFS implementation swaps in via the same interface without touching service, wire format, or storage. - `WaveformProfileService` — computes peak-normalized loudness profile from PCM WAV (one linear buffer pass), buckets by time slice, normalizes to `[0,1]`, stores as byte-quantized sidecar in new `profiles` vault (FileDatabase `MediaFileVault`). - `WaveformProfileOptions` — config-bound options object carrying `BucketCount` (default 512) and future algorithm-selection knobs. **Integration changes (`DeepDrftAPI`):** - Wired `WaveformProfileService` into `UnifiedTrackService.UploadAsync` — profile computed on upload, stored immediately, failure silently swallowed (consistent with FileDatabase philosophy in `CLAUDE.md`). **Models (`DeepDrftModels`):** - `WaveformProfileDto` — carries quantized profile data; format independent of algorithm or bucket count. **Testing (`DeepDrftTests`):** - 4 new unit tests: RMS algorithm correctness against known-good PCM samples, swappable-algorithm contract (two strategies swap cleanly), and integration with `WaveformProfileService`. **Architecture notes:** - Profile is derived binary content; stored in FileDatabase vault sidecar per `CLAUDE.md` principle ("binary content lives in the vault"). - Loudness measure is an abstraction (not hardwired RMS) — RMS→LUFS future change requires only a new `ILoudnessAlgorithm` implementation, no refactoring of service, component, or wire format. - No external audio-processing dependency pulled in for RMS — reuses existing PCM parser from `AudioProcessor`. - Cost: one linear pass over PCM buffer at upload (few hundred ms for typical WAV); never on playback path. ### W1-T2 — Waveform profile HTTP transport **Landed 2026-06-05.** Implemented Phase 2 of the WaveformSeeker feature: HTTP transport layer for waveform profile data from backend to client, enabling client-side display of loudness profiles in future seeking UI. **API endpoint (`DeepDrftAPI`):** - New `GET api/track/{trackId}/waveform` endpoint — unauthenticated, returns `WaveformProfileDto` (base64-encoded quantized bytes + `BucketCount`) on success, 404 if track or profile not found. - Leverages existing `WaveformProfileService` to load profile from vault on demand. - No authentication required — mirrors `GET api/track/{id}` streaming policy (public audio access). **Proxy forward (`DeepDrftPublic`):** - Thin buffered forward in `TrackProxyController` — proxies request from client to `DeepDrftAPI` waveform endpoint with same path parameters. - Preserves error semantics: 404 from API passes through to client; network errors surface as HTTP errors. **HTTP client (`DeepDrftPublic.Client`):** - New `TrackMediaClient.GetWaveformProfileAsync(trackId, cancellationToken)` method on the content HTTP client. - 404 response maps to `Result.Failure` (fail-result signal for WaveformSeeker to render flat fallback). - Network/timeout errors map to separate `Result.Failure` with distinct code. - Callsite can discriminate via result error code whether to retry (transient) or render fallback (not found). **Architecture notes:** - Transport layer is independent of loudness algorithm (W1-T1) — client receives opaque quantized bytes; future algorithm changes on backend do not affect wire format, as long as `BucketCount` is included. - HTTP caching via ETag/Last-Modified is deferred to Phase 2 optimization work. - Profile loading from vault is on-demand (not pre-cached in memory) — load cost amortizes across all requests to the same track. - 404 handling unambiguous: client renders flat fallback, distinguishing "track has no profile" from "track not found" via error code. ### W1-T3 — Player layout refactor (SpectrumVisualizer relocation + VolumeZone rename) **Landed 2026-06-05.** Implemented Phase 3 of the WaveformSeeker feature: architectural layout move separating live-spectrum visualization from loudness-over-time seeking. **Conceptual split:** - Live-spectrum (FFT frequency bars, `SpectrumVisualizer`) moved from `PlayerSeekZone` → stacked above the volume slider in new `VolumeZone`. Conceptually with the output level. - Static loudness-over-time (future `WaveformSeeker`) takes over the seek zone. Conceptually with transport position. **Component changes (`DeepDrftPublic.Client/Controls/AudioPlayerBar`):** - `VolumeControls.razor` → renamed **`VolumeZone.razor`** for symmetry with transport and seek zones; now a vertical stack hosting `SpectrumVisualizer` above the volume slider. - `SpectrumVisualizer` — `BucketCount` parameter defaulted to 24 buckets (down from 32) to fit the narrow volume cluster; set `flex-shrink: 0` to pin the spectrum to a fixed footprint above the volume control. - `PlayerSeekZone.razor` — `SpectrumVisualizer` block removed; placeholder for future `WaveformSeeker` component. **CSS changes (`AudioPlayerBar.razor.css`):** - Adjusted volume cluster width constraints to accommodate the 24-bucket spectrum stacked above. - Responsive layout unchanged at 600px breakpoint (single-row transport/volume with full-width seek below on narrow; same 3-zone layout on wide). **Scope:** - Pure layout move; zero change to spectrum animation lifecycle, player logic, or seek gesture handling. - Both `AudioPlayerBar` and `SpectrumVisualizer` components affected. - Build clean: 0 errors, 0 new warnings. **Notes for future work:** - `PlayerSeekZone` is now ready for the `WaveformSeeker` component (W1-T4/Phase 4 onwards). - Volume cluster can comfortably accommodate 24 FFT bars; 32 would cause visual cramping (why the override exists). - Spectrum visualization lifecycle (subscription to `StateChanged`, animation via `AudioInteropService.StartSpectrumAnimationAsync`) unchanged — only position in the DOM tree changed. --- ## Phase 2 — Product surface: player and theming **Status:** Track card CSS scoping landed on 2026-06-05. Track card glass theming landed on 2026-06-05. AudioPlayerBar responsive unification and SpectrumVisualizer fix landed on 2026-06-05. Track view CSS consolidation landed on 2026-06-05. ### Track Card CSS Scoping **Landed 2026-06-05.** Moved track card rules from the global stylesheet into an isolated scoped stylesheet, eliminating style leakage and enabling independent maintenance of the component's appearance. **CSS changes:** - `DeepDrftPublic/wwwroot/styles/deepdrft-styles.css` §8 — removed all track card rules (`.deepdrft-track-card-*`, `.deepdrft-track-title`, `.deepdrft-track-artist`, `.deepdrft-track-meta`); replaced with a pointer comment directing readers to `TrackCard.razor.css`. - `DeepDrftShared.Client/Components/TrackCard.razor.css` — created new scoped stylesheet with all card rules: container styling, text-colour hierarchy (title, artist, meta), theme-variant selectors (`.deepdrft-theme-dark` / `.deepdrft-theme-light`), and glass background + border styling. - Applied `::deep` pseudo-selector to the three MudText text-color rules (`deepdrft-track-title`, `deepdrft-track-artist`, `deepdrft-track-meta`) so CSS isolation doesn't suppress colour overrides on MudBlazor elements. - Eliminated all theme-variant selectors in favour of a single-vocabulary colour scheme: navy-glass fallback, `--deepdrft-white` title, `--deepdrft-green-accent` artist, `rgba(250,250,248,0.45)` meta. Matches the `NowPlayingCard` aesthetic. - `DeepDrftShared.Client/Components/TracksGallery.razor.css` — moved `.deepdrft-track-gallery-item-center` layout rule from global stylesheet into scoped CSS alongside the existing gallery container rules. **Scope:** - Affected components: `TrackCard.razor` (shared, consumed by public site and CMS) and `TracksGallery.razor` (shared). - CSS in `DeepDrftPublic/wwwroot/styles/deepdrft-styles.css` (global) and two scoped stylesheets. - Build clean: 0 errors, 0 new warnings. **Architecture notes:** - CSS isolation now protects track card rules from accidental mutation by unrelated global changes. - Light-mode visual is now consistent: single vocabulary eliminates the three-green collision and establishes a stable text hierarchy (off-white title → muted artist → fainter meta). - Scoped stylesheet pattern mirrors existing usage in other components (`AudioPlayerBar.razor.css`, `NowPlayingCard.razor.css`), establishing a consistent maintenance model. --- ### Track View CSS Consolidation **Landed 2026-06-05.** Implemented CSS consolidation and hierarchy fixes across three components: removed dead layout rules, unified horizontal inset ownership, and resolved the three-green collision in dark mode by demoting artist text and changing the genre chip variant. **Component changes:** - `DeepDrftPublic.Client/Pages/TracksView.razor` — removed dead `tracks-page-wrapper` class and associated inert flex/height/padding rules; `MudContainer` now owns horizontal inset via `MaxWidth.Large`. - `DeepDrftShared.Client/Components/TracksGallery.razor.css` — reduced to `box-sizing: border-box`; removed redundant padding and inert height constraint. - `DeepDrftShared.Client/Components/TrackCard.razor` — changed genre chip from `Variant.Filled` to `Variant.Outlined` to distinguish it from the play FAB. **CSS changes (`DeepDrftPublic/wwwroot/styles/deepdrft-styles.css` §8):** - Text color rules restructured: base `color: inherit`, both dark and light treatments guarded under `.deepdrft-theme-dark` / `.deepdrft-theme-light` ancestors at `0,2,0` specificity. - Artist text demoted from `green-accent` to `rgba(250,250,248,0.65)` in dark mode (leaving green as a purely accent/interactive signal — FAB and chip border). - Meta text (album/year) at `rgba(250,250,248,0.45)` in dark mode. - Genre chip treatment now supports outlined styling (borders + text only, no filled ground). **Scope:** - CSS in `deepdrft-styles.css` and scoped stylesheets for `TracksView.razor` and `TracksGallery.razor`. - Both `DeepDrftPublic.Client` and `DeepDrftShared.Client` components affected. - Build clean: 0 errors, 0 new warnings. **Architecture notes:** - Resolved the three-green visual hierarchy collapse (artist + genre chip + play FAB all rendered the same saturated green). Now: title off-white, artist muted, genre = outlined green tag, FAB = solid green action — a clear three-tier hierarchy matching `NowPlayingCard` vocabulary. - Consolidated horizontal inset ownership to `MudContainer` (removes duplicate paddings that stacked across three layers). - Removed inert flex-grow and height rules that encoded a sticky-footer intent that was not actually achieved; page layout via normal block flow is cleaner. **Status:** ### Track Card Glass Theming **Landed 2026-06-05.** Aligned `TrackCard` component visual language with the `NowPlayingCard` aesthetic via glass background + text hierarchy. Two coordinated changes: **Razor changes (`DeepDrftShared.Client/Components/TrackCard.razor`):** - Removed `mud-theme-secondary` class and `Color="Color.Surface"` attributes from all four `MudText` elements, handing color control to CSS. - Added semantic class hooks: `deepdrft-track-title` (track name), `deepdrft-track-artist` (artist), `deepdrft-track-meta` (album and release year). - Changed MudCard `Elevation="4"` → `Elevation="0"` to align with glass-panel vocabulary (no drop shadow). **CSS changes (`DeepDrftPublic/wwwroot/styles/deepdrft-styles.css` §8):** - Dark theme: navy-glass fallback panel (`color-mix(in srgb, var(--deepdrft-navy) 55%, transparent)` + `backdrop-filter: blur(8px)` + translucent border), matching `NowPlayingCard` glass vocabulary. - Text hierarchy (dark): title in off-white, artist in moss-green accent, meta in muted off-white — mirrors the `NowPlayingCard` hierarchy. - Content scrim behind text (dark): dark navy gradient to guarantee legibility over both glass fallback and album art. - Light theme: subtle navy-tint fallback on off-white, light text inherits body colour for legibility. - Glass border on card container (dark): `1px solid rgba(250, 250, 248, 0.12)` for aesthetic consistency. **Scope:** - `TrackCard` component in shared `DeepDrftShared.Client` consumed by both public site and CMS. - CSS in `DeepDrftPublic/wwwroot/styles/deepdrft-styles.css` (public site only, not loaded by CMS). - Build clean: 0 errors, 0 new warnings. **Notes for future work:** - Genre chip text still uses `Color.Primary` (moss-green); it now sits alongside moss-green artist text. Consider a distinct genre-chip treatment (3a) in future polish work. --- **Status:** AudioPlayerBar responsive unification and SpectrumVisualizer fix landed on 2026-06-05. ### AudioPlayerBar Responsive Unification **Landed 2026-06-05.** Collapsed the two divergent Razor trees in `AudioPlayerBar.razor` (`@if (_isDesktop)` / `@else`) into a single markup tree where CSS — not a runtime breakpoint flag — drives the responsive layout. Removed `IBrowserViewportService`, the `_isDesktop` field, `OnAfterRenderAsync`, and the viewport subscription/unsubscription from the code-behind. **Structural changes:** - Single `.player-layout` flex container (in `AudioPlayerBar.razor.css`) replaces the dual-branch conditional. Three children (`PlayerTransportZone`, `VolumeControls`, `PlayerSeekZone`) in source order; media query at 600px (`Sm` breakpoint) reorders via CSS `order` property and forces `SeekZone` to full-width below the transport/volume row on narrow viewports. - `PlayerTransportZone` flips its internal axis (vertical ↔ horizontal) via scoped CSS override of `MudStack` `flex-direction` at the 600px boundary — no parameter added to the component. - `::deep` prefix removed from `MudBlazor` component-class selectors in `PlayerTransportZone.razor.css` now that axis is purely CSS-driven and no runtime flag determines structure. - **SpectrumVisualizer bars now appear on first expand** — fixed by subscribing to the multicast `StateChanged` event (same pattern used by `AudioPlayerBar`), ensuring animation is initialized after mount. **Scope:** - Unified responsive layout (desktop/mobile branches merged into single tree). - Both `AudioPlayerBar` and `SpectrumVisualizer` components affected. - Build clean: 0 errors, 0 new warnings. **Notes for future work:** - First-render layout flash eliminated by construction (CSS media query evaluates at paint, not async subscription). ### Track Card Plain-Shell Refactor **Landed 2026-06-05.** Eliminated `!important` declarations from track card CSS by replacing MudBlazor surface components with plain HTML. Implemented per `product-notes/track-card-css-architecture.md` Option A. **Razor changes (`DeepDrftShared.Client/Components/TrackCard.razor`):** - `MudCard` → `