You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-ups from the PR #59 round-2 review (non-blocking while SKILLX_RELEASES_API is off):
Scanner false positives (r2-m1): UTF-16 .ps1 scripts (with a BOM), legacy-codepage .bat/.cmd/.csv files, binaries under scripts/, and LICENSE.txt may 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 production throughout, but wrangler.jsonc declares no environments. The production worker is skillx-web, so drop those flags across the guide and the README.
Shipped: fixed by #96 (squash fbd0333, merged 2026-10-20 after 6 isolated review rounds). Deployed to production skillx-web (worker version 27d34bce at fbd0333; current 7d29ff0a at 79f98fb). Smoke 2026-10-20: /, /api/search and registry keys 200.
Activity
fix(releases): tune release scan encodings and memory, check budget b…
mrgoonie commentedon Oct 20, 2026
Shipped: fixed by #96 (squash fbd0333, merged 2026-10-20 after 6 isolated review rounds). Deployed to production skillx-web (worker version 27d34bce at fbd0333; current 7d29ff0a at 79f98fb). Smoke 2026-10-20:
/,/api/searchand registry keys 200.fix: alias slugs, .pi untracking, release scan tuning, API key settin…