fix: resolve format errors and improve download reliability for YouTube Music

- fix info fetch to use stdout JSON even when yt-dlp exits non-zero (handles format warnings gracefully)
- add --no-check-formats and --js-runtimes node to both fetch and download to fix "Requested format is not available" on music.youtube.com
- improve format selector with explicit m4a/webm fallback chain
- add --embed-metadata to write proper title/artist/album tags into downloaded files
- remove video ID from playlist filenames for cleaner names (%(title)s.%(ext)s)
- rename per-playlist archive file from .soundsnatch_archive.txt to .archive.txt
- add global archive path to Config struct for single track deduplication

Author: maskedsyntax

config.go

@@ -10,18 +10,25 @@ import (
 func loadConfig() Config {
 	home, _ := os.UserHomeDir()
 	configPath := filepath.Join(home, ".soundsnatch.yaml")
+	archivePath := filepath.Join(home, ".soundsnatch_archive.txt")
 
 	config := Config{
 		LastSaveDir:   "",
 		DefaultFormat: "mp3",
 		Browser:       "",
+		ArchivePath:   archivePath,
 	}
 
 	data, err := os.ReadFile(configPath)
 	if err == nil {
 		yaml.Unmarshal(data, &config)
 	}
 
+	// Always ensure ArchivePath is set to default if missing from config
+	if config.ArchivePath == "" {
+		config.ArchivePath = archivePath
+	}
+
 	return config
 }
 

downloader.go

@@ -16,7 +16,7 @@ import (
 
 func fetchInfoCmd(url string, browser string) tea.Cmd {
 	return func() tea.Msg {
-		args := []string{"-J", "--flat-playlist", "--no-warnings", "--quiet", "--ignore-errors"}
+		args := []string{"-J", "--flat-playlist", "--no-warnings", "--quiet", "--ignore-errors", "--ignore-config", "--no-check-formats", "--js-runtimes", "node"}
 		if browser != "" && browser != "none" {
 			args = append(args, "--cookies-from-browser", browser)
 		}
@@ -27,37 +27,44 @@ func fetchInfoCmd(url string, browser string) tea.Cmd {
 			cmd = exec.Command("python3", append([]string{"-m", "yt_dlp"}, args...)...)
 		}
 
-		var stderr strings.Builder
+		var stdout, stderr strings.Builder
+		cmd.Stdout = &stdout
 		cmd.Stderr = &stderr
-		out, err := cmd.Output()
-		if err != nil {
-			errMsgStr := stderr.String()
-			if errMsgStr == "" {
-				errMsgStr = err.Error()
+		runErr := cmd.Run()
+
+		outStr := strings.TrimSpace(stdout.String())
+
+		// Prefer valid JSON output even if yt-dlp exited non-zero (e.g. format warnings)
+		if outStr != "" {
+			var info map[string]interface{}
+			if err := json.Unmarshal([]byte(outStr), &info); err == nil {
+				title, _ := info["title"].(string)
+				durFloat, _ := info["duration"].(float64)
+				return infoFetchedMsg{
+					title:    title,
+					duration: durFloat,
+				}
 			}
-			return errMsg{err: fmt.Errorf("could not fetch info: %s", errMsgStr)}
 		}
 
-		var info map[string]interface{}
-		if err := json.Unmarshal(out, &info); err != nil {
-			return errMsg{err: fmt.Errorf("failed to parse info: %v", err)}
+		// No usable JSON — surface the actual error
+		if runErr != nil {
+			errMsgStr := strings.TrimSpace(stderr.String())
+			if errMsgStr == "" {
+				errMsgStr = runErr.Error()
+			}
+			return errMsg{err: fmt.Errorf("could not fetch info: %s", errMsgStr)}
 		}
 
-		title, _ := info["title"].(string)
-		durFloat, _ := info["duration"].(float64)
-
-		return infoFetchedMsg{
-			title:    title,
-			duration: durFloat,
-		}
+		return errMsg{err: fmt.Errorf("could not fetch info: no output from yt-dlp")}
 	}
 }
 
 func searchCmd(query string) tea.Cmd {
 	return func() tea.Msg {
-		cmd := exec.Command("yt-dlp", "ytsearch5:"+query, "-j", "--no-warnings", "--quiet", "--flat-playlist")
+		cmd := exec.Command("yt-dlp", "ytsearch5:"+query, "-j", "--no-warnings", "--quiet", "--flat-playlist", "--ignore-config")
 		if _, err := exec.LookPath("yt-dlp"); err != nil {
-			cmd = exec.Command("python3", "-m", "yt_dlp", "ytsearch5:"+query, "-j", "--no-warnings", "--quiet", "--flat-playlist")
+			cmd = exec.Command("python3", "-m", "yt_dlp", "ytsearch5:"+query, "-j", "--no-warnings", "--quiet", "--flat-playlist", "--ignore-config")
 		}
 
 		out, err := cmd.Output()
@@ -96,36 +103,41 @@ func searchCmd(query string) tea.Cmd {
 	}
 }
 
-func startDownloadTask(c chan tea.Msg, url, saveDir, saveFilename, format, browser string) {
+func startDownloadTask(c chan tea.Msg, url, saveDir, saveFilename, format, browser, archivePath string) {
 	outtmpl := filepath.Join(saveDir, saveFilename+"."+format)
 	isPlaylist := strings.Contains(url, "list=") || strings.Contains(url, "playlist")
-	
+
 	args := []string{
-		"-f", "bestaudio/best",
+		"-f", "bestaudio[ext=m4a]/bestaudio[ext=webm]/bestaudio/best",
 		"--extract-audio",
 		"--audio-format", format,
+		"--audio-quality", "0",
+		"--no-check-formats",
+		"--js-runtimes", "node",
+		"--embed-metadata",
 		"--no-warnings",
 		"--newline",
 		"--progress",
 		"--ignore-errors",
 		"--no-cache-dir",
 		"--lazy-playlist",
 		"--no-overwrites",
+		"--ignore-config",
 	}
 
 	if isPlaylist {
 		playlistDir := filepath.Join(saveDir, saveFilename)
 		os.MkdirAll(playlistDir, 0755)
-		
-		// LOCAL ARCHIVE: Keep track of downloads INSIDE the playlist folder
-		// This prevents duplicates within the folder but allows downloading 
-		// the same song to a DIFFERENT folder.
-		archivePath := filepath.Join(playlistDir, ".soundsnatch_archive.txt")
-		args = append(args, "--download-archive", archivePath)
-		
-		outtmpl = filepath.Join(playlistDir, "%(title)s [%(id)s].%(ext)s")
+		outtmpl = filepath.Join(playlistDir, "%(title)s.%(ext)s")
+		// Use a per-playlist archive so the same song can exist in different playlists
+		// and playlists don't interfere with each other
+		localArchive := filepath.Join(playlistDir, ".archive.txt")
+		args = append(args, "--download-archive", localArchive)
 	} else {
 		args = append(args, "--no-playlist")
+		if archivePath != "" {
+			args = append(args, "--download-archive", archivePath)
+		}
 	}
 
 	args = append(args, "-o", outtmpl)
@@ -184,7 +196,7 @@ func startDownloadTask(c chan tea.Msg, url, saveDir, saveFilename, format, brows
 		}
 	}
 
-	c <- downloadDoneMsg{message: fmt.Sprintf("🎉 Sync Complete!\nFiles are in: %s", saveDir)}
+	c <- downloadDoneMsg{message: fmt.Sprintf("🎉 Sync Complete!\nYour library at: %s is now up to date.", saveDir)}
 }
 
 func waitForMsg(c chan tea.Msg) tea.Cmd {

types.go

@@ -46,6 +46,7 @@ type Config struct {
 	LastSaveDir   string `yaml:"last_save_dir"`
 	DefaultFormat string `yaml:"default_format"`
 	Browser       string `yaml:"browser"`
+	ArchivePath   string `yaml:"archive_path"`
 }
 
 type formatItem struct {

ui.go

@@ -348,7 +348,7 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				if ok {
 					m.selectedFormat = i.ext
 					m.state = stateDownloading
-					go startDownloadTask(m.msgChan, m.url, m.saveDir, m.saveFilename, m.selectedFormat, m.config.Browser)
+					go startDownloadTask(m.msgChan, m.url, m.saveDir, m.saveFilename, m.selectedFormat, m.config.Browser, m.config.ArchivePath)
 					return m, tea.Batch(m.spinner.Tick, waitForMsg(m.msgChan))
 				}
 			}