[Bug] types of properties in group are inferred as unknown if results parameter is used
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- node.js, typescript
- Área
- cli
Línea de trabajo
Comienza con la reproducción en StackBlitz e inspecciona las definiciones de tipos de TypeScript de la función p.group, especialmente el parámetro results y su tratamiento de los cancel symbols. Confirma los tipos inferidos indicados y verifica después que el ejemplo infiere addNpmScriptsPrefix como boolean sin usar any.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Environment
- OS: webcontainers
- Node Version: 18.20.3
- Package:
@clack/prompts - Package Version: v0.10.0
Describe the bug
const options = await p.group(
{
injectNpmScripts: async () => {
return p.confirm({
message: 'Add VitePress npm scripts to package.json?',
});
},
addNpmScriptsPrefix: async ({ results }) => {
if (!results.injectNpmScripts) return false;
return p.confirm({
message: 'Add a prefix for VitePress npm scripts?',
});
},
},
{
onCancel: () => {
p.cancel('Cancelled.');
process.exit(0);
},
}
);
options here is inferred as:
const options: {
injectNpmScripts: boolean;
addNpmScriptsPrefix: unknown;
}
To Reproduce
https://stackblitz.com/edit/node-1z7qkqdt?file=index.ts
Expected behavior
Return correct types:
const options: {
injectNpmScripts: boolean;
addNpmScriptsPrefix: boolean;
}
Additional Information
It works fine if we make it ({ results }: any). The generic parameter of group function doesn't work because the functions return cancel symbol too.
- Lenguaje dominante
- TypeScript
- Estrellas
- 8.1k
- Forks
- 218
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de bombshell-dev/clack
-
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
bombshell-dev/clack#613 · 4 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
bombshell-dev/clack#597 ·
-
rfc: accessible modeAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
bombshell-dev/clack#585 · 1 comentario · 1 reacción ·
-
[Bug] Spinner does not run `onCancel` function on `Ctrl + C`Posiblemente ocupada @cyphercodes la tomó hace 103 días. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
bombshell-dev/clack#573 · 1 comentario · 2 reacciones ·
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
bombshell-dev/clack#565 · 1 comentario · 1 reacción ·
Todos los issues de bombshell-dev/clack
Issues similares
-
cleanName keeps a state code that's joined to the word before it (SavannahGa → "Savannah Ga")Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
openwatersio/slackwater-database#281 ·
Los mantenedores suelen responder en 1 día
-
accepted bug via-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
pingdotgg/t3code#18690 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Bayern feature:Application ready web
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
digitalfabrik/entitlementcard#3176 ·
Los mantenedores suelen responder en 7 días
-
[BUG] 运行位置的分支列表不自动刷新，外部新建的分支看不到Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
NanmiCoder/cc-haha#1533 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día