Preserve critical defaults and improve comments

Co-authored-by: krau <71133316+krau@users.noreply.github.com>

client/bot/handlers/ytdlp.go

@@ -43,6 +43,8 @@ func handleYtdlpCmd(ctx *ext.Context, update *ext.Update) error {
 				nextArg := strings.TrimSpace(args[i+1])
 				if nextArg != "" && !strings.HasPrefix(nextArg, "-") {
 					// Check if it's clearly a URL (has ://)
+					// This handles common video URLs (http://, https://)
+					// For other yt-dlp inputs, users should ensure proper formatting
 					if strings.Contains(nextArg, "://") {
 						// It's a URL, don't consume it as a flag value
 						continue

core/tasks/ytdlp/execute.go

@@ -80,7 +80,8 @@ func (t *Task) Execute(ctx context.Context) error {
 func (t *Task) downloadFiles(ctx context.Context, tempDir string) ([]string, error) {
 	logger := log.FromContext(ctx)
 
-	// Configure yt-dlp command with default settings
+	// Configure yt-dlp command with essential settings
+	// Always set output path to ensure files go to temp directory
 	cmd := ytdlp.New().
 		Output(filepath.Join(tempDir, "%(title)s.%(ext)s"))
 
@@ -91,6 +92,8 @@ func (t *Task) downloadFiles(ctx context.Context, tempDir string) ([]string, err
 			RecodeVideo("mp4").
 			RestrictFilenames()
 	}
+	// Note: If custom flags are provided, users have full control over format/quality
+	// The output path is always set above to ensure downloads go to the correct directory
 
 	if t.Progress != nil {
 		t.Progress.OnProgress(ctx, t, "Downloading...")