Skip to content

Conversation

@0Chencc
Copy link
Member

@0Chencc 0Chencc commented Feb 11, 2026

Summary

Spotify's new Development Mode rules (effective 2025-02-11) limit each Client ID to 5 authorized users. Shipping a shared public Client ID is no longer viable.

  • Removed hardcoded spotifyClientId from AppConfig
  • effectiveSpotifyClientIdProvider no longer falls back to any default — returns empty string if not configured
  • Login screen: Client ID input is always visible and required before the Connect button is enabled
  • Settings page: Client ID section promoted from hidden "Advanced Options" to a top-level editable field
  • L10n: Updated descriptions in zh/en/ja to reflect this is a requirement, not optional

Test plan

  • Fresh install: login button disabled until Client ID is saved
  • Enter Client ID → save → login button enabled → auth works
  • Settings: can view and change Client ID (prompts re-login)
  • App restart: saved Client ID persists and login works

Spotify's new Development Mode rules (2025-02-11) limit each Client ID
to 5 authorized users, making shared public IDs unviable.

- Remove hardcoded spotifyClientId from AppConfig
- effectiveSpotifyClientIdProvider no longer falls back to a default
- Login screen: Client ID input always visible and required before login
- Settings: Client ID section promoted from hidden advanced option
- Updated l10n descriptions (zh/en/ja) to reflect requirement
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.42%. Comparing base (0a71bfc) to head (7bf07d3).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   71.42%   71.42%           
=======================================
  Files           1        1           
  Lines           7        7           
=======================================
  Hits            5        5           
  Misses          2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0Chencc 0Chencc merged commit 8ff8564 into main Feb 11, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant