Security Review Checklist

Also available: Markdown ยท Plain text

Security Review Checklist โ€” Gnomad Desktop Assistant

Version: 0.2.0-beta.1
Type: Structured internal review (substitute for external pen test until v1.0 GA)
Last updated: June 2026

Use this checklist before beta/GA releases. External penetration testing is still recommended for v1.0 enterprise deployments.


1. Shell and privilege

# Check Pass
1.1 No IPC path executes shell without shell_session_run + validation โ˜
1.2 Legacy execute_shell_command removed from invoke handler โ˜
1.3 Risky commands require HITL token (not boolean bypass) โ˜
1.4 HITL tokens are single-use and command-bound โ˜
1.5 YOLO sandbox active when enabled (platform-appropriate level) โ˜
1.6 Injection patterns blocked in privilege.rs heuristics โ˜

2. Filesystem

# Check Pass
2.1 Out-of-workspace access requires Path Gate token โ˜
2.2 Path tokens reject boolean bypass โ˜
2.3 fs_write preferred over shell redirects (UI hint) โ˜
2.4 Audit log records shell sandboxed and success โ˜

3. Secrets and network

# Check Pass
3.1 API keys in OS keychain / credential store only โ˜
3.2 No secrets in repo, logs, or error-log.jsonl by default โ˜
3.3 Cloud traffic uses HTTPS; base URL configurable โ˜
3.4 Updater verifies artifact signatures (real pubkey, not placeholder) โ˜

4. Supply chain

# Check Pass
4.1 npm ci / lockfile in CI โ˜
4.2 Release artifacts built on GitHub Actions with pinned actions โ˜
4.3 macOS builds signed; notarized before enterprise distribution โ˜
4.4 CHANGELOG documents security-relevant changes โ˜

5. Privacy

# Check Pass
5.1 Error log opt-in only โ˜
5.2 Voice input opt-in; user informed of browser STT โ˜
5.3 No telemetry to Gnomad servers by default โ˜

6. Regression tests

npm run test
cd src-tauri && cargo test
npm run verify:updater   # before release with real keys

Sign-off

Role Name Date
Maintainer
Reviewer


Built with โค๏ธ by Gnomad Studio ๐Ÿฆ™