Skip to content

fix(cli): validate spotdl sync manifests - #3

Merged
labithiotis merged 1 commit into
mainfrom
fix/validate-spotdl-manifests
Aug 1, 2026
Merged

fix(cli): validate spotdl sync manifests#3
labithiotis merged 1 commit into
mainfrom
fix/validate-spotdl-manifests

Conversation

@labithiotis

Copy link
Copy Markdown
Owner

Summary

  • decode spotDL sync files before reading query or song fields
  • prefer validated collection URLs from song metadata and reject raw searches, keywords, and unsupported collection kinds
  • normalize Spotify collection URIs to provider-compatible HTTPS URLs
  • process multiple spotDL files in stable filename order and skip invalid files
  • remove fabricated album/song ID fallbacks

Test plan

  • bun test src (80 pass)
  • bun run typecheck
  • biome check
  • repository pre-commit hooks

@labithiotis
labithiotis merged commit 2c925a2 into main Aug 1, 2026
4 checks passed
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