Skip to content

Fix duplicate upload notifications for collection track publishes#14031

Merged
dylanjeffers merged 1 commit intomainfrom
cursor/bug-report-resolution-b211
Apr 10, 2026
Merged

Fix duplicate upload notifications for collection track publishes#14031
dylanjeffers merged 1 commit intomainfrom
cursor/bug-report-resolution-b211

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor bot commented Mar 31, 2026

Summary

  • Ensure collection (playlist/album) child tracks are explicitly marked as playlist uploads before publish.
  • Preserve is_playlist_upload through web upload metadata adaptation and SDK upload schema parsing.
  • Prevent backend create track notifications from fanning out for each child track in collection upload retries.

Changes

  • packages/common/src/api/tan-query/upload/usePublishCollection.ts: set metadata.is_playlist_upload = true when composing child track metadata for collection uploads.
  • packages/common/src/adapters/track.ts: include is_playlist_upload in trackMetadataForUploadToSdk() output.
  • packages/sdk/src/sdk/api/tracks/types.ts: add isPlaylistUpload to UploadTrackMetadataSchema so strict parsing keeps the field.

Notes

  • Root cause traced to missing propagation of is_playlist_upload; discovery-provider trigger logic only suppresses per-track create notifications when this field is true.
  • Targeted tests were not runnable in this environment because vitest is not available in the package runtime path.
Open in Web View Automation 

Co-authored-by: Ray Jacobson <raymondjacobson@users.noreply.github.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 31, 2026

⚠️ No Changeset found

Latest commit: edd6abb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14031.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

@dylanjeffers dylanjeffers marked this pull request as ready for review April 10, 2026 17:54
@dylanjeffers dylanjeffers merged commit ad63e39 into main Apr 10, 2026
15 checks passed
@dylanjeffers dylanjeffers deleted the cursor/bug-report-resolution-b211 branch April 10, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants