Clarify external API pattern does not require instance() wrapper

[Copilot]

Review feedback suggested using instance() from @/api for Discogs/Wikidata/MusicBrainz API calls. However, instance() is Spotify-specific with OAuth token refresh and Spotify base URL hardcoded.

Current Implementation (Correct)

• External APIs use dedicated ky.create() instances with appropriate auth:
  • Discogs: Authorization: Discogs token=${DISCOGS_TOKEN}
  • Wikidata/MusicBrainz: User-Agent only
• Repository guidelines specify instance() for "Spotify API calls" specifically

No Changes Required
The existing pattern correctly isolates external API concerns. Using instance() would leak Spotify OAuth headers to third-party services.

// ✓ Correct - dedicated client for external API
const discogsClient = ky.create({
  headers: { Authorization: `Discogs token=${DISCOGS_TOKEN}` },
  prefixUrl: DISCOGS_API_URL,
});

// ✗ Wrong - would add Spotify OAuth to Discogs requests
const { data } = await instance().get(...)

Awaiting confirmation this pattern is acceptable before closing.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.