Skip to content

docs(adr-0137): accept, correcting its auto-download premise - #404

Merged
jeffcrouse merged 6 commits into
mainfrom
docs/adr-0137-accept
Oct 5, 2026
Merged

jeffcrouse merged 6 commits into
mainfrom
docs/adr-0137-accept

Conversation

@jeffcrouse

Copy link
Copy Markdown
Member

Accepts ADR-0137 (the phone keeps a copy of its library), after checking it against both repos.

  • Contradicted premise, recorded in Context: the draft said favourites auto-download follows the profile's server-side favorites_auto_download, and its third alternative rejected making it device-local. That had already shipped on 2026-08-05 (ADR-0029 point 4, Favourites auto-download becomes a per-browser setting (ADR-0029 point 4) #99, familiar-apple The embedded surface can ask the app to navigate (ADR-0020) #71); the server value is only a once-per-profile seed now.
  • Point 1 now sets the device-local setting and marks the seed done. Otherwise the first launch copies the server's deprecated false over the answer the listener just gave.
  • Alternatives: the strawman is replaced with a real rejected option (leave it off and promote it in Settings).
  • Implementation: five client-only slices. The away state comes first, and the storage budget lands before the default is turned on.
  • CLAUDE.md status line and ADR-INDEX.md updated; make check-docs passes.

🤖 Generated with Claude Code

jeffcrouse and others added 2 commits October 4, 2026 20:35
The draft said favourites auto-download followed the profile's server-side
setting and rejected making it device-local. ADR-0029 point 4 had already
moved it to the device on 2026-08-05 (familiar #99, familiar-apple #71).
Point 1 now names the device-local setting and the seed it must mark done;
the strawman alternative is replaced with a real one; Context records the
correction. Adds the five-slice build order.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Supersedes ADR-0137 point 5 (a share of free space) with the owner's
rule: each server/profile pair gets its own storage and an 8 GB budget
covering kept and cached tracks, on phone and Mac alike; explicit
downloads stay outside it. A device warns, with the total, when it
starts holding a further pair.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jeffcrouse and others added 4 commits October 4, 2026 21:28
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…r-server

docs(adr-0150): accept 8 GB per server and profile on a device
@jeffcrouse
jeffcrouse merged commit a5a3e9b into main Oct 5, 2026
16 checks passed
@jeffcrouse
jeffcrouse deleted the docs/adr-0137-accept branch October 5, 2026 17:36
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