Test

[sweetmantech]

Summary by cubic

Fixes catalog visibility for valuation-captured songs. We now carry Spotify artists through capture and run artist linking and notes queuing so these tracks render like manually added songs.

• Bug Fixes
  • mapUnmappedAlbumTracks keeps track.artists as spotifyArtists during ISRC resolution and dedupe.
  • After upserts, calls linkSongsToArtists(songs) and queueRedisSongs(songs) to link artists and queue note generation.
  • Added a test that asserts captured songs are linked to their Spotify artists and queued for notes.

^{Written for commit 6fc10cc. Summary will update on new commits.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
api	Building	Preview	Jun 18, 2026 11:53pm

[coderabbitai]

Warning

Review limit reached

@sweetmantech, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 12 minutes and 52 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: da0bb9df-3354-4505-9f77-9982e22190ff

📥 Commits

Reviewing files that changed from the base of the PR and between 34b2c84 and 6fc10cc.

⛔ Files ignored due to path filters (1)

• lib/research/playcounts/__tests__/mapUnmappedAlbumTracks.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**

📒 Files selected for processing (1)

• lib/research/playcounts/mapUnmappedAlbumTracks.ts

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cubic-dev-ai]

2 issues found across 2 files

Confidence score: 3/5

• In lib/research/playcounts/mapUnmappedAlbumTracks.ts, failures in artist-link enrichment now abort the whole mapping path, so successful identifier upserts can be misclassified as bootstrap failures and retried unnecessarily; this can create noisy reprocessing and unstable playcount bootstrapping—make artist-link errors non-fatal (or isolate them) before merging.
• lib/research/playcounts/mapUnmappedAlbumTracks.ts is over the project’s 100-line guideline, with mapping, artist linking, and Redis queuing concerns combined, which increases the chance of future regressions when touching this flow—split secondary responsibilities into helpers/modules to de-risk follow-up changes.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="lib/research/playcounts/mapUnmappedAlbumTracks.ts">

<violation number="1" location="lib/research/playcounts/mapUnmappedAlbumTracks.ts:98">
P1: Artist-link enrichment failure now aborts mapping output. This causes successful identifier upserts to be treated as failed bootstrap and retried unnecessarily.</violation>

<violation number="2" location="lib/research/playcounts/mapUnmappedAlbumTracks.ts:98">
P2: Custom agent: **Enforce Clear Code Style and Maintainability Practices**

File exceeds the 100-line limit (107 lines). Split secondary concerns (artist linking and Redis queuing) into a separate module or helper to bring this file back under 100 lines and preserve single responsibility.</violation>
</file>

Architecture diagram

sequenceDiagram
    participant AP as Apify Actor
    participant API as API Route
    participant MUAT as mapUnmappedAlbumTracks
    participant SPOT as Spotify API
    participant DB as Supabase DB
    participant SL as Song Service
    participant AL as Artist Link Service
    participant RQ as Redis Queue

    Note over AP,RQ: Catalog enrichment flow for valuation-captured tracks

    AP->>API: Trigger album playcount sync
    API->>MUAT: mapUnmappedAlbumTracks(albums, mappedTracks)

    MUAT->>SPOT: getTracks(trackIds)
    SPOT-->>MUAT: Track objects with artists

    MUAT->>MUAT: Extract spotifyArtists from track.artists

    Note over MUAT: Dedupe by ISRC, carry spotifyArtists through deduplication

    alt New tracks found
        MUAT->>DB: upsertSongs(songs without spotifyArtists)
        DB-->>MUAT: Songs upserted

        MUAT->>DB: upsertSongIdentifiers(track IDs, ISRCs)
        DB-->>MUAT: Identifiers upserted

        MUAT->>AL: linkSongsToArtists(songs with spotifyArtists)
        AL->>DB: Link songs to Spotify artists (auto-create artists)
        DB-->>AL: Artist links created
        AL-->>MUAT: Complete

        MUAT->>RQ: queueRedisSongs(songs)
        RQ-->>MUAT: Queued for note enrichment
    end

    MUAT-->>API: Map of trackId -> ISRC
    API-->>AP: Response

Loading

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

P1: Artist-link enrichment failure now aborts mapping output. This causes successful identifier upserts to be treated as failed bootstrap and retried unnecessarily.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At lib/research/playcounts/mapUnmappedAlbumTracks.ts, line 98:

<comment>Artist-link enrichment failure now aborts mapping output. This causes successful identifier upserts to be treated as failed bootstrap and retried unnecessarily.</comment>

<file context>
@@ -72,6 +91,13 @@ export async function mapUnmappedAlbumTracks(
+    // manual/CSV flow — link artists (auto-creating the artist account) and
+    // queue note generation — so valuation tracks aren't "missing info" and
+    // render in the catalog view rather than being filtered out.
+    await linkSongsToArtists(songs);
+    await queueRedisSongs(songs);
+
</file context>

Suggested change

	await linkSongsToArtists(songs);
	await linkSongsToArtists(songs).catch(error => {
	console.error("[playcounts] artist linking failed:", error);
	});

[cubic-dev-ai]

P2: Custom agent: Enforce Clear Code Style and Maintainability Practices

File exceeds the 100-line limit (107 lines). Split secondary concerns (artist linking and Redis queuing) into a separate module or helper to bring this file back under 100 lines and preserve single responsibility.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At lib/research/playcounts/mapUnmappedAlbumTracks.ts, line 98:

<comment>File exceeds the 100-line limit (107 lines). Split secondary concerns (artist linking and Redis queuing) into a separate module or helper to bring this file back under 100 lines and preserve single responsibility.</comment>

<file context>
@@ -72,6 +91,13 @@ export async function mapUnmappedAlbumTracks(
+    // manual/CSV flow — link artists (auto-creating the artist account) and
+    // queue note generation — so valuation tracks aren't "missing info" and
+    // render in the catalog view rather than being filtered out.
+    await linkSongsToArtists(songs);
+    await queueRedisSongs(songs);
+
</file context>