Skip to content

fix(lyrics-plus): use spclient endpoint to fetch tempo#3734

Merged
rxri merged 2 commits intospicetify:mainfrom
cvius:lyrcs
Mar 14, 2026
Merged

fix(lyrics-plus): use spclient endpoint to fetch tempo#3734
rxri merged 2 commits intospicetify:mainfrom
cvius:lyrcs

Conversation

@cvius
Copy link
Contributor

@cvius cvius commented Mar 11, 2026

Summary by CodeRabbit

  • Chores
    • Switched the backend source for audio feature retrieval to improve reliability and tempo extraction accuracy.
  • Bug Fixes
    • Reduced occurrences of incorrect or missing tempo information when analyzing tracks.

@coderabbitai
Copy link

coderabbitai bot commented Mar 11, 2026

📝 Walkthrough

Walkthrough

A single endpoint in the lyrics-plus app was changed: the audio-features fetch now uses an alternative Spotify URL (spclient.wg.spotify.com) with ?format=json. Tempo extraction logic and response semantics were left unchanged.

Changes

Cohort / File(s) Summary
Spotify API Endpoint Update
CustomApps/lyrics-plus/index.js
Replaced audio-features endpoint from https://api.spotify.com/v1/audio-features/{id} to https://spclient.wg.spotify.com/audio-attributes/v1/audio-features/{id}?format=json. Tempo is still read from the audio features response; handling unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop through code, a tiny URL swap,
The beat remains steady, it didn't stop,
Tempo found where it always lay,
New path, same rhythm, ready to play! 🎵

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately describes the main change: updating the tempo fetching implementation to use the spclient endpoint instead of the standard Spotify API endpoint.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can make CodeRabbit's review stricter and more nitpicky using the `assertive` profile, if that's what you prefer.

Change the reviews.profile setting to assertive to make CodeRabbit's nitpick more issues in your PRs.

@rxri rxri changed the title fix(lyrics-plus): use internal endpoint to fetch tempo fix(lyrics-plus): use spclient endpoint to fetch tempo Mar 14, 2026
@rxri rxri merged commit dd308d7 into spicetify:main Mar 14, 2026
8 checks passed
@cvius cvius deleted the lyrcs branch March 15, 2026 03:47
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.

2 participants