Repository navigation
dusty-jukebox: アルバム一覧をCDラックの分け方にする(R14) - #501
Merged
Merged
Conversation
- CD1枚=1件:アルバム名+まとめる単位のフォルダで1枚にする(曲ごとのアーティストが違っても分けない) - CD1/Disc 2等のフォルダに分かれた何枚組は、上のフォルダで1枚に(foldersタブのフォルダ名を使う) - 棚:アルバムアーティスト → 半分を超える曲のアーティスト → オムニバス/不明。見出しに枚数 - 並べ方:アーティストの棚(発売順、既定)/発売の古い順/新しい順/アルバム名順 - 純粋ロジックはsrc/albumRack.ts(ユニット24件)、E2Eはfoldersタブ付きのモックで確認 - npm run deploy(871 unit + 122 E2E、SW v0.1.190) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Awe6vCLv7CpaEWVTkrHvpU
kumacha666
marked this pull request as ready for review
November 1, 2026 15:23
Owner
Author
|
レビュー対象HEAD: 今回の確認範囲では、修正が必要な指摘はありません。 変更9ファイルと、関連するカタログ読み込み・フォルダ解決・絞り込み・再生リストへの設定を確認しました。Android版のcore/AlbumRack・AlbumFoldersとも照合しています。
検証結果:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
Android版で先に入れた「部屋のCDラック」のアルバム一覧を、Web版にもそろえる(R14)。
旧方式は「アルバム名+アルバムアーティスト(空なら曲のアーティスト)+フォルダ」で1枚にまとめていた。そのため次の2つの問題があった。
変更
src/albumRack.ts(純粋ロジック。Android版のcore/AlbumRack・core/AlbumFoldersと同じ規則)CD1・Disc 2・DISK04・ディスク1等のフォルダに入っている曲は、その上のフォルダで1枚にまとめる。収録曲はディスク番号の順に並べ、タグが無ければフォルダの番号を使う。その後トラック番号の順に並べる。main.ts:loadCatalog()が読むfoldersタブのフォルダ名・親を使って何枚組をまとめる。今回Driveへ問い合わせた分も使う。一覧の描画はalbumRackRows()に置き換えた。index.html:見出しの枚数の見た目。dusty-jukebox/CLAUDE.mdに記録。テスト(HEAD
4d8f959)npm run deploy:unit 871件・E2E 122件すべて成功。app.js315.84 kB、実クライアントIDを含むことを確認済み。SW v0.1.189→v0.1.190。albumRack.test.ts(24件)rackCatalogモック(foldersタブ付き)で、2枚組が1件になり、オムニバスの棚に入り、再生リストがディスク順になることを確認。.shelf-name)を見るように直した。foldersタブの情報を渡さないようにすると、新規E2Eが失敗する(2枚組が2件に分かれる)。元に戻して成功を確認済み。/code-review:未実施。Generated by Claude Code