← Back to scan
File as GitHub Issue repo: charlesvestal/schwung

Push this scan report to charlesvestal/schwung

Click the green button below to open GitHub’s new-issue form, pre-filled with the report title, summary table, top findings, and an embedded score-card image. No authentication needed — you review on GitHub before submitting. Repobility is credited as the scanner.

Embedded score card image

This image will render at the top of the issue body. Hosted on Repobility, refreshes automatically after re-scans.

Repobility score card

Issue title

JS: child_process.exec with non-literal

Curate findings to include

Pick exactly which findings appear in the issue body. By default the top 5 are included. Uncheck noise, check what matters.

Top 5 (default)
Severity Rule Title File:line
CRIT MINED022 [MINED022] C Strcpy: strcpy/strcat dont bounds-check; use strncpy or snprintf. src/modules/audio_fx/freeverb/freeverb.c:310
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … src/shared/input_filter.mjs:151
HIGH MINED027 [MINED027] React State Array Mutation: state.X.push/splice/sort followed by setState — Re… src/shared/filepath_browser.mjs:175
HIGH MINED012 [MINED012] Curl Pipe Bash: curl ... | sh / bash — runs unverified network code. scripts/uninstall.sh:124
HIGH SEC085 [SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in… src/shared/parse_move_manual.mjs:79
HIGH SEC085 [SEC085] JS: child_process.exec with non-literal: child_process.exec with user-derived in… scripts/fetch_move_manual.sh:86
HIGH SEC093 [SEC093] Go: exec.Command with non-literal: exec.Command(<var>) — variable command name a… schwung-manager/self_heal.go:61
HIGH COMP001 [COMP001] High cognitive complexity: Function `migrate_module` has cognitive complexity 7… scripts/migrate-module-params.py:58
HIGH MINED115 [MINED115] Action `softprops/action-gh-release` pinned to mutable ref `@v1`: `uses: softp… .github/workflows/release.yml:66
HIGH MINED115 [MINED115] Action `actions/setup-go` pinned to mutable ref `@v5`: `uses: actions/setup-go… .github/workflows/release.yml:26
HIGH MINED115 [MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout… .github/workflows/release.yml:16
HIGH MINED118 [MINED118] Dockerfile FROM `debian:bookworm` not pinned by digest: `FROM debian:bookworm`… Dockerfile:3
HIGH SEC020 [SEC020] Secret Printed to Logs: Debug or diagnostic code appears to print a credential-b… src/modules/tools/config-test/ui.js:108
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… scripts/analyze_wavs.py:197
MED SEC136 [SEC136] AI-typical over-broad exception handler swallowing all errors: Catch-all excepti… src/shared/parse_move_manual.mjs:211
MED ERR002 [ERR002] Empty Catch Block: Empty catch blocks hide errors. src/shared/parse_move_manual.mjs:303
MED ERR002 [ERR002] Empty Catch Block: Empty catch blocks hide errors. src/modules/overtake/rnbo-runner/ui.js:128
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … src/shared/parse_move_manual.mjs:79
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … scripts/fetch_move_manual.sh:86
MED COMP001 [COMP001] High cognitive complexity: Function `main` has cognitive complexity 18 (SonarSo… scripts/analyze_wavs.py:186
MED DKR007 Docker build context has no .dockerignore .dockerignore
MED DKR001 Docker final stage has no non-root USER Dockerfile:4
MED AGT006 React interval is created without an explicit cleanup schwung-manager/static/htmx.min.js:1
MED AGT015 Remote install command pipes network code directly to a shell scripts/uninstall.sh:124
MED AGT015 Remote install command pipes network code directly to a shell README.md:45
MED CORE_LARGE_FILES Average file size is 686 lines (recommend <300)
LOW SEC132 [SEC132] String concat where the language has interpolation (AI style drift): String buil… src/shadow/shadow_ui_tools.mjs:308
LOW SEC132 [SEC132] String concat where the language has interpolation (AI style drift): String buil… src/modules/overtake/rnbo-runner/ui.js:41
LOW SEC132 [SEC132] String concat where the language has interpolation (AI style drift): String buil… scripts/fetch_move_manual.sh:121
LOW COMP001 [COMP001] High cognitive complexity: Function `resolve_inputs` has cognitive complexity 1… scripts/analyze_wavs.py:164
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackEngineControl.h:5
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackEngineControl.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackEngineControl.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackEngine.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackEngine.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDummyDriver.h:3
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDummyDriver.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDriverLoader.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDriverInfo.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDriver.h:3
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDriver.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackDebugClient.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackConstants.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackConnectionMana…:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackClientInterfac…:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackClientInterfac…:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackClientControl.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackClient.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackChannel.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAudioDriver.h:3
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAudioAdapterIn…:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAudioAdapterIn…:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAudioAdapter.h:2
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAtomicState.h:4
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAtomicState.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAtomicArraySta…:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackAtomic.h:1
LOW AIC003 Duplicated implementation block across source files src/lib/jack2/common/JackArgParser.h:2
LOW AIC003 Duplicated implementation block across source files src/host/tts_engine_flite.c:91
LOW AIC003 Duplicated implementation block across source files libs/quickjs/quickjs-2025-04-26/qjsc.c:357
LOW DKR011 Dockerfile installs recommended OS packages Dockerfile:9
INFO MINED049 [MINED049] Print Pii: Logging password/token/email/ssn directly to stdout. src/modules/tools/config-test/ui.js:108
INFO MINED075 [MINED075] C Malloc No Check: malloc/calloc/realloc return value used without checking fo… src/modules/tools/wav-player/wav_player…:198
INFO MINED075 [MINED075] C Malloc No Check: malloc/calloc/realloc return value used without checking fo… src/modules/tools/seq-test/dsp/seq-test…:97
INFO MINED075 [MINED075] C Malloc No Check: malloc/calloc/realloc return value used without checking fo… src/modules/midi_fx/velocity_scale/dsp/…:81
INFO MINED042 [MINED042] Cpp New Without Delete: C++ raw new without RAII / unique_ptr — memory leak ri… src/host/lfo_common.h:123
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … src/modules/overtake/rnbo-runner/ui.js:41
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … src/modules/controller/ui.js:326
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … schwung-manager/static/schwung-remote-a…:10
Reset to top 5 69 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `charlesvestal/schwung`

**Score: 80/100 (B-)**  ·  69 findings  ·  scanned 2026-06-05 20:40 UTC  ·  205,738 LOC

| Severity | Count |
|---|---|
| CRITICAL | 1 |
| HIGH | 12 |
| MEDIUM | 13 |
| LOW | 35 |

📊 [Full filterable report](https://repobility.com/scan/3e1c4285-0bca-4ba4-9c2b-65cfa56d6ae4/)  ·  ![scorecard](https://repobility.com/scan/3e1c4285-0bca-4ba4-9c2b-65cfa56d6ae4/report.png?v=1780692003-s2)

### Top findings

1. **CRITICAL** `MINED022` — C Strcpy
   `src/modules/audio_fx/freeverb/freeverb.c:310` · CWE-120 · ✓ Repobility
2. **HIGH** `SEC128` — Async function without await — fire-and-forget Promise (AI mistake)
   `src/shared/input_filter.mjs:151`
3. **HIGH** `MINED027` — React State Array Mutation
   `src/shared/filepath_browser.mjs:175` · CWE-682 · ✓ Repobility
4. **HIGH** `MINED012` — Curl Pipe Bash
   `scripts/uninstall.sh:124` · CWE-494 · ✓ Repobility
5. **HIGH** `SEC085` — JS: child_process.exec with non-literal
   `src/shared/parse_move_manual.mjs:79`

---

**Security note**: this issue is public. If any flagged finding is a real, exploitable vulnerability, please redirect to your `SECURITY.md` policy or open a [private security advisory](https://docs.github.com/en/code-security/security-advisories/guidance-on-reporting-and-writing-information-about-vulnerabilities/privately-reporting-a-security-vulnerability) instead. We're happy to close this and re-submit privately.

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/3e1c4285-0bca-4ba4-9c2b-65cfa56d6ae4/_
Megaproject â high spam risk
Could not determine 'charlesvestal/schwung' star count (GitHub API rate-limited or unreachable). When in doubt about repo size, prefer opening a focused PR or a discussion rather than an issue.

The button opens GitHubâs new-issue page in a new tab. You will see the title + body pre-filled â review, edit if you want, then click GitHubâs "Submit new issue" button. Repobility never posts anything on your behalf.

For real security findings on big repos: use the project's SECURITY.md or private advisory flow instead of a public issue.