Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

fix(knowledge): video-digest cookie-retry pattern 'extracting cookies' is tested only against stderr

Đã đóng
  • Có gì không ổn?
#6,981 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
55/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript
Lĩnh vực
backend

Hướng nghiên cứu

Start in acquire-yt-dlp-auth.js, where COOKIE_PROFILE_RETRY_HINT_PATTERNS is defined, then read spawnFailureDetail in acquire-with-retry.js, which returns only stderr. First confirm with a live yt-dlp run which stream the 'Extracting cookies' line uses. If it is stderr, remove the pattern; if stdout, include it in the cookie-profile detail. Done when a test using captured output proves each pattern can match its input.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

needs-triage priority: low status: needs-info

Summary

COOKIE_PROFILE_RETRY_HINT_PATTERNS includes /extracting cookies/i, but the only text it is ever tested against is the failed spawn's stderr. If yt-dlp reports its "Extracting cookies from " progress line on stdout, the pattern can never match, so the entry is dead and gives a false impression that the case is handled.

Failure scenario

Expected: every pattern in the retry-hint list can match text the caller actually passes in.

Actual: spawnFailureDetail returns result.stderr (trimmed), falling back to result.error, then "spawn failed". stdout is never read. The caller passes that detail to isCookieProfileRetryableError.

Unverified: which stream yt-dlp writes that message to. My recollection is that it is an informational line printed on stdout, but I did not confirm it against yt-dlp source or a live run in this session, so the first step of the fix is to check. If it does go to stderr, this issue reduces to the pattern being redundant with the more specific failure messages in the list.

Evidence

Suggested fix

Confirm the stream with a live yt-dlp run against a browser whose cookie store cannot be read. Then either drop the pattern, or, if the message is on stdout and the retry is wanted, include stdout in the detail for the cookie-profile path. Add a test using the real captured output.

Related

  • #6816 (browser-cookie fallback)

🤖 Generated with Claude Code

Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
6 giờ 11 phút
Pull request đã merge (30 ngày)
955

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của melodic-software/claude-code-plugins

Tất cả issue của melodic-software/claude-code-plugins

Issue tương tự

Thêm issue về Shell/Bash

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.