Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

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

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#7,994 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Ya se ha fusionado un pull request relacionado.

  • #8151 de @melodic-automation-lanes — fusionado

Evaluación

Dificultad
2/5
Tiempo estimado
Medio día
Aptitud para principiantes
84/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript, shell
Área
backend, testing-qa

Línea de trabajo

Comienza con COOKIE_PROFILE_RETRY_HINT_PATTERNS y isCookieProfileRetryableError y luego lee plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js para conocer los casos existentes de reintento de cookies. Ejecuta scripts/run-test-file.sh plugins/knowledge/skills/video-digest/extraction/acquisition/acquire-yt-dlp-auth.test.js para verificar el comportamiento específico. El cambio está listo cuando la prueba indicada cubre los casos de stderr y stdout y la comprobación del fragmento del changelog pasa.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Shell
Estrellas
22
Forks
2
Merge medio
7 h 8 min
PR fusionados (30 d)
874

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de melodic-software/claude-code-plugins

Todos los issues de melodic-software/claude-code-plugins

Issues similares

Más issues de Shell/Bash

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.