fix(web): API key settings follow-ups (error handling, CSRF origin check, empty scopes label)
Los mantenedores suelen responder en 1 día
@mrgoonie ya está trabajando en esto.
Desde el 14/10/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Follow-up from the independent review of #77 (scoped publishing keys in Settings). All low severity; none block.
apps/web/app/routes/api.user-api-keys.tsDELETE with a malformed JSON body returns 500 instead of 400 (POST got the guard).apps/web/app/routes/settings.tsxthe create dialog closes before the server responds; on 400/403 the user loses the name and selected publishers.apps/web/app/components/settings/api-keys-table.tsxa key whose stored scopes parse to[]shows an empty Access cell instead of "No access".api.user-api-keys.tsscopes: nullreturns 400 while docs only describe omitting the field (fails safe; document or accept null as omitted).- The route has no Origin/CSRF check and relies on the session cookie's default
SameSite=Lax; a sibling subdomain could send cookie-bearing requests. Pre-existing; consider an Origin check on mutating auth routes. settings.tsxafter Done the plaintext key stays in the in-memory action result until the next navigation (not re-shown).- Separate: the
api.skill-favoriteclient chunk contains thekey_hashcolumn name (no data); check that no server-only module is imported client-side there.
- Lenguaje dominante
- TypeScript
- Estrellas
- 189
- Forks
- 40
- Merge medio
- 7 h 38 min
- PR fusionados (30 d)
- 30
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 nextlevelbuilder/skillx
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
nextlevelbuilder/skillx#49 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
nextlevelbuilder/skillx#28 ·
Los mantenedores suelen responder en 1 día
-
refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
nextlevelbuilder/skillx#27 ·
Los mantenedores suelen responder en 1 día
-
Personal-kit API gaps for AgentKit kit publish/syncPosiblemente ocupada @mrgoonie la tomó hace 1 día. Abierto
nextlevelbuilder/skillx#88 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 12/100
nextlevelbuilder/skillx#81 ·
Los mantenedores suelen responder en 1 día
Todos los issues de nextlevelbuilder/skillx
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
callstackincubator/simlock#639 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
aws/agentcore-cli#2606 ·
Los mantenedores suelen responder en 1 día
-
ai-driven-qa bug Cosmetic
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
ai-driven-qa bug claude Minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
linagora/twake-mail-admin#301 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día