Repository navigation
ADR-0155: the Mac app may add albums if the listener turns it on - #414
Merged
Merged
Conversation
…n (proposed) Read-only stays the default everywhere and the server never writes the library. An opt-in, add-only importer in the Mac player files Bandcamp downloads the listener already has; the scanner and watcher learn to skip hidden folders first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Discovery and the watcher no longer descend into or scan anything whose name begins with ".", so an import staged in a hidden folder is never seen half-copied, and .Trashes, .Spotlight-V100 and ._ files stop being walked. A library root that is itself hidden is still watched. ADR-0155 accepted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jeffcrouse
changed the title
Member
Author
|
ADR-0155 accepted. Point 5 built in this PR: the scanner and watcher skip hidden folders and files. New tests fail against the old code; full backend suite 2413 passed. Point 7 (check a real Bandcamp download) is next, then the importer in familiar-apple. |
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.
Proposes ADR-0155. Status
proposed. No code.validate_library_path, the:romount and ADR-0140's Seatbelt profile are unchanged.Visit https://….bandcamp.comcomment tag, checked against the NAS library.scanner.py:174,library_watch.py:63).docs/ZERO-TOUCH.mdremoved zip import on purpose; this keeps that as the default and adds one listener-enabled exception. The zip layout is from memory and point 7 makes checking a real download the first build step.Execution order: point 5 (here) → point 7 (a real download) → points 2–4 and 6 (
familiar-apple) → docs.🤖 Generated with Claude Code