Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

fix(knowledge): drop the dead 'extracting cookies' cookie-retry pattern

Aperta Adatta ai principianti
#7,994 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
Mezza giornata
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript, shell

Direzione di ricerca

Inizia con COOKIE_PROFILE_RETRY_HINT_PATTERNS e isCookieProfileRetryableError, poi leggi plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js per i casi esistenti di retry dei cookie. Esegui scripts/run-test-file.sh plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js per verificare il comportamento specifico. La modifica è pronta quando il test indicato copre i casi di stderr e stdout e il controllo del frammento del changelog passa.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Shell
Stelle
22
Fork
2
Merge medio
8h 41m
PR unite (30g)
892

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di melodic-software/claude-code-plugins

Tutte le issue di melodic-software/claude-code-plugins

Issue simili

Altre issue su Shell/Bash

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.