chore(registry): SX-3 review follow-ups (scanner encodings, scan memory, deploy guide env flags)
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ó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- devops, documentation, security
Hướng nghiên cứu
The payload names no scanner source file, so first locate the scan and budget code in the repo. The deployment item is the most concrete: check wrangler.jsonc for declared environments, then remove the --env production and --environment production flags from the deployment guide and README, keeping skillx-web as the worker name. The scanner and memory items need design choices (streaming versus a size cap), so confirm scope with the maintainers before editing.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Follow-ups from the PR #59 round-2 review (non-blocking while SKILLX_RELEASES_API is off):
- Scanner false positives (r2-m1): UTF-16
.ps1scripts (with a BOM), legacy-codepage.bat/.cmd/.csvfiles, binaries underscripts/, andLICENSE.txtmay be quarantined. Decode UTF-16 when it has a BOM, and tune the injection scan for license and plain-text files. - Scan memory (r2-m2): each file is built as a full string, plus a join and a decode. A 32 MiB tree can exceed the 128 MB isolate. Stream the scan per file, or cap the text size scanned.
- Budget guard: move it before the scan and the signing.
- Settings UI: label scoped keys.
- Deployment guide: it uses
--env production/--environment productionthroughout, butwrangler.jsoncdeclares no environments. The production worker isskillx-web, so drop those flags across the guide and the README.
Refs #54, #59
- Ngôn ngữ chính
- TypeScript
- Star
- 189
- Fork
- 40
- Merge trung bình
- 6 giờ 54 phút
- Pull request đã merge (30 ngày)
- 24
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nextlevelbuilder/skillx
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
nextlevelbuilder/skillx#49 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
nextlevelbuilder/skillx#28 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
nextlevelbuilder/skillx#27 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Harden personal kit proposal scanning before enabling the API flagCó thể đã có người làm @mrgoonie đã nhận hôm nay. Đang mở
nextlevelbuilder/skillx#86 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 18/100
nextlevelbuilder/skillx#84 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của nextlevelbuilder/skillx
Issue tương tự
-
fix(frontend): correct VITE_IS_DEVELPOMENT typo in shared/app.ts and align dev environment flagsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
vicharanashala/ajrasakha#1439 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add: TV5Monde AsieĐang mởapproved check:passed streams:add
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
backnotprop/plannotator#1932 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 62/100
ibm-granite/granite.build#504 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
reproduced
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
anomalyco/opencode#55676 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày