Skip to content

fix: patch TIDAL search API change - #15

Merged
treyturner merged 2 commits into
mainfrom
fix/tidal-search-api-patches
Sep 4, 2026
Merged

treyturner merged 2 commits into
mainfrom
fix/tidal-search-api-patches

Conversation

@treyturner

@treyturner treyturner commented Sep 4, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • sync 6781.diff with the current source changes in the upstream PR
  • add 6990.diff for the updated TIDAL v2 search endpoint and response shape
  • apply both fixes independently through the patch manifest

Validation

  • verified 6781.diff, 6990.diff, and 6912.diff apply in series to beets v2.13.1
  • verified the patched TIDAL modules compile
  • verified the patched production sources exactly match the combined upstream changes
  • upstream beets PR 6990 passes formatting, Ruff, MyPy, docs, and the full test matrix

Related upstream work:

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-04T20:59:17.951597Z 9dbb5c9 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@treyturner
treyturner merged commit 5607964 into main Sep 4, 2026
1 check passed
@treyturner
treyturner deleted the fix/tidal-search-api-patches branch September 4, 2026 22:43
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