feat(qa): evidence reach sees production CLIs behind env, npm exec and sh -c, and flyctl

Abierto Apto para principiantes
#1,682 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
75/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
backend, testing

Línea de trabajo

Lee src/lib/evidence-reach.ts y sus casos en src/lib/evidence-reach.test.ts. Ejecuta primero npx vitest run src/lib/evidence-reach.test.ts para ver el comportamiento actual y luego añade cobertura para las formas de wrapper indicadas y flyctl. El trabajo estará terminado cuando los casos nuevos indiquen needs-human con el motivo production-cli, el caso env FOO=1 npx vitest run siga siendo worktree y pase el archivo de pruebas completo.

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

Descripción

Epic: graph-2026-11-simplify (follow-up of #1652) Tier: mechanical Execution: sequant_run

Problem

#1652's classifier (src/lib/evidence-reach.ts) applies P-9 as planned: a segment needs a person when its first word, after VAR=x assignments and npx/bunx/pnpm dlx|exec/yarn dlx, is in the fixed list vercel, supabase, netlify, fly, heroku, wrangler, railway, aws, gcloud, az. QA on #1652 (at 8ad978a6c) found that these slip through as worktree:

  • env vercel env ls production (an env prefix)
  • npm exec vercel … and npm x vercel …
  • sh -c "vercel …" and bash -c '…'
  • flyctl … (the Fly CLI's other binary name)

P-9 rejected a configurable list ("add one when someone asks"). This issue is that ask, for wrapper prefixes and one alias. It doesn't make the list configurable.

Acceptance Criteria

  • AC-1: Each of the five forms above is needs-human with reason production-cli, and env FOO=1 npx vitest run stays worktree. Evidence: npx vitest run src/lib/evidence-reach.test.ts -t "wrapper" → ≥ 6 passed, 0 failed
  • AC-2: #1652's existing cases are unchanged. Evidence: npx vitest run src/lib/evidence-reach.test.ts → 0 failed (constraint)

May touch

src/lib/evidence-reach.ts, src/lib/evidence-reach.test.ts, CHANGELOG.

Non-Goals

  • A configurable list or a settings key (P-9).
  • Blocking commands in a hook (D-5).

Done when

The classifier sees a listed production CLI behind env, npm exec/npm x and sh -c/bash -c, and under the name flyctl.

Lenguaje dominante
TypeScript
Estrellas
1
Forks
1
Merge medio
10 h 12 min
PR fusionados (30 d)
78

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 sequant-io/sequant

Todos los issues de sequant-io/sequant

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.