chore(deps): bump go-openaudio ETL to playlist_contents key fix#878
Merged
Conversation
Picks up go-openaudio #321, which canonicalizes playlist_contents entry keys on write (track_id/timestamp/metadata_timestamp -> track/time/ metadata_time). Pre-fix, the vendored ETL indexer persisted the SDK's alias keys verbatim, so playlists whose contents used those keys rendered empty — api's v1_playlist_tracks query, the handle_playlist notification trigger, and the get_playlists play-count rollup all read only the `track` key. New writes now land in the canonical shape. Already-broken is_current rows are repaired by a separate one-shot SQL backfill run out-of-band. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps
go-openaudioandgo-openaudio/pkg/etlfrom6ea0f10→35ad422, picking up go-openaudio#321.That PR fixes the ETL indexer's
playlist_contentsJSONB write path: pre-fix it persisted the SDK'strack_id/timestamp/metadata_timestampalias keys verbatim, but every reader on this side only understands the canonicaltrackkey —api/v1_playlist_tracksquery —(e.value->>'track')::inthandle_playlistnotification trigger —(track_item->>'track')::intdbv1get_playlistsplay-count rollup —(e.item->>'track')::intso affected playlists rendered empty even though the
playlist_tracksjunction table was correct. After this bump, new writes land in the canonical{track, time, metadata_time}shape.Backfill
Already-broken
is_currentrows (65 and climbing at time of writing) are not repaired by this PR — they're fixed by a one-shot SQL backfill run out-of-band. The backfill maps the same alias keys onto the canonical names and was dry-run-verified against prod (2194 entries in → 2194 out, zero drops).Test plan
go build ./...clean against the bumped depsstern -n api core-indexer)🤖 Generated with Claude Code