Reduce false-positive “New Music Update” issues by diffing net-new music links

[Copilot]

music-social was still opening “New Music Update” issues on non-content edits in music.html (e.g., rewrites/format churn in the auto-generated YouTube block).
This PR narrows detection to actual new music links introduced in <!-- YOUTUBE-AUTO-START --> ... <!-- YOUTUBE-AUTO-END -->.

• Detection logic hardening

  • Replaced “added-line contains music domain” heuristic with old vs new link-set comparison inside the auto block.
  • Issue creation now triggers only when links exist in new state and are absent from previous state (comm -13).

• Link matching scope

  • Added explicit patterns for:
    • YouTube watch URLs
    • YouTube shorts URLs
    • youtu.be short URLs
    • SoundCloud track URLs
    • Bandcamp URLs (including subdomains)

• Deterministic comparison

  • Enforced locale-stable sorting (LC_ALL=C sort -u + LC_ALL=C comm) to keep set-diff behavior consistent across runners.

extract_music_links() {
  echo "$1" | grep -oiE "$MUSIC_LINK_PATTERN" | LC_ALL=C sort -u
}

if extract_music_links "$NEW_MUSIC_BLOCK" | grep -q .; then
  NEW_LINKS_ONLY=$(LC_ALL=C comm -13 \
    <(extract_music_links "$OLD_MUSIC_BLOCK") \
    <(extract_music_links "$NEW_MUSIC_BLOCK") || true)
fi

[bolt-new-by-stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[Copilot]

Pull request overview

This PR tightens the music-social workflow’s “New Music Update” trigger by comparing the set of music links found inside the <!-- YOUTUBE-AUTO-START --> … <!-- YOUTUBE-AUTO-END --> block between the previous and current versions of music.html, and only opening an issue when net-new links are detected.

Changes:

• Replaced the prior “added lines contain a music domain” heuristic with an old-vs-new link set diff (comm -13) scoped to the auto-generated YouTube block.
• Expanded link matching to cover YouTube watch/shorts/youtu.be URLs, SoundCloud URLs, and Bandcamp (including subdomains).
• Enforced locale-stable sorting for deterministic set comparisons (LC_ALL=C sort -u, LC_ALL=C comm).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[DavidKRK]

Le correctif est clair et améliore bien la détection des vrais nouveaux liens musicaux en comparant l’état précédent et l’état actuel du bloc auto-généré.
La logique est plus robuste et devrait réduire les faux positifs. Rien à signaler de bloquant.