chore(api): bump etl to persist dropped track/comment fields (#343)#924
Merged
Conversation
Bumps go-openaudio to c5fcacf, which fixes the track create/update handlers dropping license/isrc/iswc/preview_start_seconds/comments_disabled/no_ai_use/ cover_original_*, and the comment update handler dropping video_url. Code-only (no new migration). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
4 tasks
raymondjacobson
added a commit
that referenced
this pull request
Jun 9, 2026
## Summary Bumps `github.com/OpenAudio/go-openaudio` and `.../pkg/etl` from `c5fcacf` (#924) to **`e8586ac`**. **Primary:** go-openaudio **#348** — fixes `playlists.last_added_to` never being written (perpetually NULL), which silently broke the "recently added to" sort and the playlist-update notification. Now set to the block time of the most recent track add (code-only in #348). **Also carried** (main advanced past the baseline): #345/#346 — the **TrackCollaborator** entity. This adds ETL migration **0033**, which `CREATE TABLE IF NOT EXISTS track_collaborators` (a new, empty table) plus one index on it. `core-indexer` runs this vendored image, so the bump is required to ship the fix. ## Changes - `go.mod` / `go.sum`: both go-openaudio modules → `v1.3.1-0.20260609013415-e8586ac9ce16` ## Deploy note — migration 0033 is low risk 0033 creates a brand-new empty table + index with `IF NOT EXISTS`. There's no existing data to scan and no `ACCESS EXCLUSIVE` lock on a populated table (unlike the 0031 slug index), so it applies instantly — no pre-build/CONCURRENT step needed. The `last_added_to` fix itself is code-only (no migration). ## Test plan - [x] `go mod tidy` — only go.mod/go.sum changed - [x] `go build ./...` passes - [x] resolved module contains the `last_added_to` writes (`LastAddedTo` in playlist_row.go) and migration `0033` - [ ] Post-deploy: migration version 33 applied clean; creating/editing a playlist populates `last_added_to`, and a rename preserves it 🤖 Generated with [Claude Code](https://claude.com/claude-code) 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
1 participant
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
github.com/OpenAudio/go-openaudioand.../pkg/etlfrom4cccb46(#920) toc5fcacfto consume go-openaudio #343.That change fixes entity-manager handlers silently dropping columns that exist on the table and arrive in metadata (same class as the social-links fix, found by auditing all write handlers):
license,isrc,iswc,preview_start_seconds,comments_disabled,no_ai_use,cover_original_song_title,cover_original_artist. Prod scale of the prior loss: 642,291 tracks have alicense, 94,894 anisrc, 31,615 a custompreview_start_seconds.video_url(a text-only edit no longer wipes an attached video).core-indexerruns this vendoredaudius/apiimage, so this bump ships the fix.Changes
go.mod/go.sum: both go-openaudio modules →v1.3.1-0.20260609000702-c5fcacffbb79Deploy note — no migration
This is code-only: every affected column already exists in migration
0002/0032, so there's no new migration and no schema/rollout concern. Latest ETL migration stays0032.Test plan
go mod tidy— only go.mod/go.sum changedgo build ./...passesLicensein track_row.go) and commentvideo_urlCOALESCE; latest migration unchanged (0032)🤖 Generated with Claude Code