Skip to content

chore: remove /v1/users/{id}/feed/for-you endpoint#807

Merged
dylanjeffers merged 1 commit into
mainfrom
chore/remove-for-you-feed
May 13, 2026
Merged

chore: remove /v1/users/{id}/feed/for-you endpoint#807
dylanjeffers merged 1 commit into
mainfrom
chore/remove-for-you-feed

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

Retiring the dedicated For You feed endpoint. The clients are being switched to use `/v1/users/{id}/recommended-tracks` instead — the same endpoint that already powers the Explore page's For You section and works fine in production. See companion PR: AudiusProject/apps#14301.

Why

The custom `/feed/for-you` endpoint had repeated issues since it shipped:

Consolidating on the working endpoint is simpler than continuing to optimize the custom one.

Removed

File What
`api/v1_users_feed_for_you.go` Handler + the 200-row candidate-pool SQL (4 candidate sources, similar_artists CF, diversity pass)
`api/v1_users_feed_for_you_test.go` 9 unit tests
`api/server.go` (1 line) Route registration
`api/auth_middleware.go` (~10 lines) The `/feed/for-you` exemption from #804 — no longer needed
`api/swagger/swagger-v1.yaml` (~70 lines) The endpoint's swagger entry

Test plan

🤖 Generated with Claude Code

The custom For You feed endpoint is being retired in favor of
consolidating on /v1/users/{id}/recommended-tracks, which already
powers the Explore page's For You section and is in active use. The
client is being switched to that endpoint in a companion PR on the
apps repo.

Removed:
* api/v1_users_feed_for_you.go (handler + 200-row candidate-pool SQL
  with similar_artists self-join that scaled to ~300M rows for
  power users)
* api/v1_users_feed_for_you_test.go (9 unit tests)
* Route registration in api/server.go
* The /feed/for-you exemption in authMiddleware (added in #804 as a
  workaround for the auth gate; no longer needed since the route is
  gone)
* Swagger spec entry for the endpoint

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@dylanjeffers dylanjeffers merged commit f3f211a into main May 13, 2026
4 checks passed
@dylanjeffers dylanjeffers deleted the chore/remove-for-you-feed branch May 13, 2026 16: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