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

[Bug] `text` validate function for text when value is empty with default value

Abierto
#397 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
node.js, typescript
Área
cli

Línea de trabajo

Comienza con core/prompts/text y reproduce el caso de entrada vacía usando el ejemplo enlazado de StackBlitz. Revisa el borrador de PR #396 junto con las dos direcciones de solución propuestas. Se considera terminado cuando pulsar Enter con un valor predeterminado configurado hace que la validación use ese valor predeterminado, preservando al mismo tiempo el comportamiento documentado del prompt de texto.

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

Descripción

bug

Environment

  • OS: macOS
  • Node Version: 22.5.1
  • Package: @clack/promts
  • Package Version: 0.11.0

Describe the bug
When calling the text's validate function without user input but with a default value defined, it should use the default value instead of relying solely on user input.

To Reproduce

Steps to reproduce the behavior:

  1. Open repro link
  2. Run node index.js
  3. Don't type anything, just press [Enter]
  4. See behavior described on terminal

Expected behavior
Clear use case example:
Image

CLI dev creates a text step for user to enter a path as input. There's some predefined value that CLI dev established to be the default use case for most users, but still allowing for that value to be changed.
Currently, the user would have to type/copy that entire path, causing a terrible user experience.

Fix Recommendation (starting a convo)
I see 2 possible ways to fix this:

  1. Keep validate as is and add a check to the value on core/prompts/text - link to draft PR
  2. Add a second optional argument, which would be the default value to validate. That way, Devs would have another quick option when using the validate function, without adding extra checks to the core library.

My goal with this section is to start a conversation and introduce my ideas about this issue, not a way for me to impose how I think things should be.
This would be my first contribution, so I'm pretty new here. Let me know what you think and hopefully let's get this fixed!

Lenguaje dominante
TypeScript
Estrellas
8.1k
Forks
217
Merge medio
1 d 11 h
PR fusionados (30 d)
5

Guía de contribución

Abrir la guía de contribución

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 bombshell-dev/clack

Todos los issues de bombshell-dev/clack

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.