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): drop the dead 'extracting cookies' cookie-retry pattern

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#7,994 0 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

Một pull request liên quan đã được merge.

  • #8151 của @melodic-automation-lanes — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
Nửa ngày
Mức phù hợp với người mới
84/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript, shell
Lĩnh vực
backend, testing-qa

Hướng nghiên cứu

Bắt đầu với COOKIE_PROFILE_RETRY_HINT_PATTERNS và isCookieProfileRetryableError, sau đó đọc plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js để xem các trường hợp thử lại cookie hiện có. Chạy scripts/run-test-file.sh plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js để xác minh hành vi được nhắm mục tiêu. Thay đổi sẵn sàng khi bài kiểm thử được chỉ định bao phủ các trường hợp stderr và stdout và bước kiểm tra mảnh changelog hoàn tất thành công.

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

Mô tả

needs-human work-class: mechanical

Refs: #6981

Agent Brief

Type: Bug
Summary: Remove /extracting cookies/i from the cookie-profile retry hints and pin the real yt-dlp failure text in a test.

Current behavior:
COOKIE_PROFILE_RETRY_HINT_PATTERNS includes /extracting cookies/i, but the detail it is matched against is the failed spawn's stderr only. yt-dlp 2026.08.19 prints Extracting cookies from <browser> on stdout, so the pattern never matches. The failure for an unreadable profile arrives on stderr as ERROR: could not find chrome cookies database in "<path>", which /could not find .* cookies/i already matches.

Desired behavior:

  • The retry-hint list holds only patterns that can match stderr text; /extracting cookies/i is gone.
  • A missing-profile failure still advances to the next browser profile.
  • The detail stays stderr-first; stdout is not added to it.

Key interfaces:

  • COOKIE_PROFILE_RETRY_HINT_PATTERNS (exported list).
  • isCookieProfileRetryableError(detail, errorPatterns): same signature and results for every remaining pattern.

Acceptance criteria:

  • The diff removes /extracting cookies/i and no other pattern.
  • A test asserts isCookieProfileRetryableError returns true for the captured stderr ERROR: could not find chrome cookies database in "/tmp/noprofile".
  • A test asserts it returns false for the stdout line Extracting cookies from chrome alone.
  • scripts/run-test-file.sh plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js passes.
  • A knowledge changelog fragment is added and scripts/check-changelog-fragments.sh passes.

Out of scope:

  • spawnFailureDetail() and the spawn wrapper.
  • The browser profile order and the bot-challenge patterns.
  • Re-verifying against other yt-dlp versions.

This was generated by AI during backlog shaping. Split from #6981; the owner approved filing it in an attended backlog-shaping session on 2026-10-18.

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

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.