should fix #61

[mmattbtw]

haven't tested so beware

[mmattbtw]

@fatfingers23 if you wanna test, i don't have spotify T-T

[Copilot]

Pull request overview

This PR introduces cross-service arbitration to prevent conflicting “now playing” updates and duplicate scrobbles when multiple music services are linked, and adds a user-configurable streaming service priority to control which service wins.

Changes:

• Add a new service/arbiter coordinator that resolves the “current” track by per-user service priority and gates publish/clear/scrobble actions.
• Persist a service_priority preference on models.User / users table, expose an API to update it, and add a home-page UI to manage it.
• Update Spotify, Last.fm, and Apple Music services to report “live” / “last stamped” tracks and to consult the arbiter before publishing/clearing/scrobbling.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
service/arbiter/arbiter.go	New coordinator enforcing per-user service priority and dedupe window rules.
service/arbiter/arbiter_test.go	Tests for priority resolution, dedupe behavior, and gating decisions.
models/user.go	Add service constants + ServicePriority field and normalization/parsing helpers.
models/user_test.go	Tests for priority normalization and default fallback.
db/db.go	Add service_priority column, load/save it, and add DB update method.
cmd/handlers.go	Build priority options for UI; add current-track resolver handler + priority update API handler.
cmd/routes.go	Route current-track through arbiter; add service-priority preferences endpoint.
cmd/main.go	Instantiate arbiter and inject into Spotify/Last.fm/Apple Music services.
cmd/service_priority_test.go	Handler/unit tests for updating service priority and current-track resolution handler.
pages/templates/home.gohtml	Add UI + JS to select and save service priority.
service/spotify/spotify.go	Add live/last-stamped tracking + arbiter checks for publish/clear/scrobble.
service/lastfm/lastfm.go	Add live/last-stamped tracking + arbiter checks for publish/clear/scrobble.
service/lastfm/lastfm_test.go	Tests asserting Last.fm publish/clear/scrobble are blocked by higher-priority live state.
service/applemusic/applemusic.go	Add TTL-based live-state tracking + arbiter checks + avoid republishing unchanged tracks.
service/applemusic/applemusic_test.go	Tests for TTL expiry and “unchanged track does not republish”.
config/config.go	Add Apple Music now-playing TTL default.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.