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

Harden personal kit proposal scanning before enabling the API flag

Cerrado
  • ¿Algo no está bien?
#86 0 comentarios 0 reacciones 1 asignado Ver en GitHub

Los mantenedores suelen responder en 1 día

@mrgoonie ya está trabajando en esto.

Desde el 14/10/2026.

  • #87 de @mrgoonie — fusionado

Evaluación

Este issue todavía no se ha evaluado.

Descripción

Follow-up from the independent review of #85. This must land before the personal kits API flag is turned on in production. Today the flag is off in wrangler.jsonc and every request is authenticated.

Medium

  • Quadratic regex cost in proposal path detectors (apps/web/app/lib/personal-kits/proposal-detectors.ts).
    • On a long run of / or \, every position is a possible match start, and each one backtracks across the rest of the run.
    • The link-scheme check has the same shape on text like a-a-a-….
    • About 800 KB of slashes in one added file can exceed the Worker CPU limit.
    • Fix: collapse long separator runs, or cap the scanned line length, before matching. Add a test with a time bound.

Low

  • Local paths still slip through:
    • compiler flags such as -I/opt/x and -I/home/<user>;
    • vscode://file/... links;
    • a first segment with non-ASCII letters (/Usuários/...);
    • /usr/home and /usr/local/<org> under the allowlist.
  • YAML password block misses some forms: list items (- password: with the value on the next line) and quoted keys ("password":).
  • Activation without a web session: rebasing a reviewed session-derived proposal produces a draft that an API key can activate without a web session. Decide whether drafts derived from a session-derived revision inherit the web-session activation gate.
  • Shared aliases need ops config: skills that ship in more than one official AgentKit kit without an ak- prefix still need SKILLX_SHARED_SKILL_ALIASES, and nothing sets it yet. Decide who owns the canonical list. AgentKit kit composition is the source.

Refs #55, #56, #82, #83, #85

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

  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 nextlevelbuilder/skillx

Todos los issues de nextlevelbuilder/skillx

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.