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

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,682 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
75/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend, testing

Hướng nghiên cứu

Đọc src/lib/evidence-reach.ts và các trường hợp trong src/lib/evidence-reach.test.ts. Trước tiên, chạy npx vitest run src/lib/evidence-reach.test.ts để xem hành vi hiện tại, sau đó bổ sung kiểm thử cho các dạng wrapper được liệt kê và flyctl. Hoàn tất khi các trường hợp mới báo needs-human với lý do production-cli, trường hợp env FOO=1 npx vitest run vẫn là worktree và toàn bộ tệp kiểm thử chạy thành công.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
TypeScript
Star
1
Fork
1
Merge trung bình
10 giờ 12 phút
Pull request đã merge (30 ngày)
78

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của sequant-io/sequant

Tất cả issue của sequant-io/sequant

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.