Public scan — anyone with this URL can view this analysis. Sign up to track your own repos privately, run scheduled re-scans, and get AI fix prompts via your dashboard.
136 of your 258 findings came from Repobility's proprietary detections. ✓ Repobility tags below mark them.
Upstream (GitHub) caused delay on this scan — not Repobility.
  • GitHub API rate-limited (HTTP 403) — preflight skipped, fell back to direct git clone.
  • Clone from GitHub took 320.0s for a 370.9 MB repo slow.
  • Repobility's analysis ran in 20.85s after the clone landed.

NovaSector/NovaSector

https://github.com/NovaSector/NovaSector · scanned 2026-06-06 00:30 UTC (3 days, 20 hours ago) · 10 languages

235 findings 11/13 scanners ran 27th percentile · Typescript · large (100-500K LoC)

UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

94 actionable findings from 1 signal source. 141 repeated signals grouped for readability. Security checks, system graph analysis, and verified AI-agent feedback are merged into one review queue.

Score breakdown â 2026-05-18-v5
Component Sub-score Weight Contribution
structure_score 60.0 0.15 9.00
security_score 100.0 0.25 25.00
testing_score 20.0 0.20 4.00
documentation_score 63.0 0.15 9.45
practices_score 100.0 0.15 15.00
code_quality 62.0 0.10 6.20
Overall 1.00 68.7
security_score may be inflated — optional security scanners were skipped on this fast scan
Severity distribution — click a segment to filter
Active filters: excluding tests × Reset all

Showing 51 of 94 actionable findings. 235 raw detector signals were grouped into reader-sized issues. Click TP / FP to vote on a finding's accuracy — votes adjust the confidence weighting and improve detection across the platform.

critical Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED007] Sql String Concat: cursor.execute(f"... {user_input} ...") — SQL injection.
Review and fix per the pattern semantics. See CWE-89 / A03:2021 for context.
3 files, 3 locations
SQL/ban_conversion_2018-10-28.py:125
tools/ezdb/steps/install_initial_schema.py:21
tools/ezdb/steps/update_schema.py:23
critical Security checks quality Quality conf 1.00 ✓ Repobility [MINED030] Python Pickle Loads: pickle.loads() can execute arbitrary code via __reduce__.
Review and fix per the pattern semantics. See CWE-502 / for context.
tools/minibot/minibot.py:98
critical Security checks quality Quality conf 1.00 ✓ Repobility [MINED102] Shell Injection Via F-string: Shell command built via f-string or .format with non-constant input — command injection. An attacker controlling any interpolated value can execute arbitrary shell commands.
Use the list form of subprocess (e.g. subprocess.run(["cmd", arg1, arg2])) with shell=False. Never combine shell=True with string interpolation.
tools/icon_cutter/check.py:105
critical Security checks quality Quality conf 1.00 [SEC081] Python: pickle.loads / marshal.loads on untrusted data: pickle.load(s) and marshal.load(s) execute arbitrary code on untrusted input. Ported from dlint DUO103 / DUO120 (BSD-3).
Use json, msgpack, or protobuf for untrusted data. If pickle is required, sign the payload with HMAC.
tools/minibot/minibot.py:98
critical Security checks cicd CI/CD security conf 0.96 3 occurrences Compose service contains a literal secret environment value
Rotate the value if real. Move it to Docker Compose secrets, a platform secret manager, or an uncommitted environment file.
2 files, 3 locations
tools/LinuxOneShot/docker-compose.yml:2, 14 (2 hits)
tools/DockerTestServer/docker-compose.yml:25
CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.35 ✓ Repobility 2 occurrences Workflow references repository secrets in a pull_request workflow
Fork pull_request runs do not receive normal repository secrets on GitHub Actions. Review this as a reliability/intent signal, not as direct fork-secret exfiltration. Raise severity only for pull_request_target or another trusted-context path that runs untrusted PR code with secrets.
lines 64, 74
.github/workflows/stale.yml:64, 74 (2 hits)
CI/CD securityworkflow secretsGitHub Actions
low Security checks quality Quality conf 1.00 ✓ Repobility [MINED006] Overcatch Baseexception: except BaseException: ... — prevents Ctrl+C and SystemExit from working.
Review and fix per the pattern semantics. See CWE-705 / for context.
tools/mapmerge2/frontend.py:108
low Security checks quality Quality conf 1.00 ✓ Repobility [MINED006] Overcatch Baseexception: except BaseException: ... — prevents Ctrl+C and SystemExit from working.
Review and fix per the pattern semantics. See CWE-705 / for context.
tools/maplint/source/__main__.py:30
low Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED012] Curl Pipe Bash: curl ... | sh / bash — runs unverified network code.
Review and fix per the pattern semantics. See CWE-494 / A08:2021 for context.
3 files, 3 locations
tools/bootstrap/javascript.sh:22
tools/tgs_scripts/InstallDeps.sh:23
tools/tgs_scripts/PreCompile.sh:32
high Security checks quality Quality conf 1.00 ✓ Repobility [MINED034] Python Subprocess Shell True: subprocess(..., shell=True) enables command injection.
Review and fix per the pattern semantics. See CWE-78 / for context.
tools/icon_cutter/check.py:105
high Security checks quality Quality conf 1.00 ✓ Repobility 25 occurrences [MINED108] `self._assemble_comment` used but never assigned in __init__: Method `to_file` of class `Dmi` reads `self._assemble_comment`, but no assignment to it exists in __init__ (and no class-level fallback). This raises AttributeError the first time the method runs against an instance.
Initialize `self._assemble_comment = <default>` in __init__, or add a class-level default.
4 files, 25 locations
tools/maplint/source/dmm.py:48, 49, 52, 55, 58, 71, 77, 82, +1 more (9 hits)
tools/mapmerge2/dmm.py:33, 38, 49, 58, 61, 96, 103, 113, +1 more (9 hits)
tools/dmi/__init__.py:157, 207, 208, 218 (4 hits)
tools/localhost-asset-webroot-server.py:7, 8, 9 (3 hits)
high Security checks software dependencies conf 0.90 ✓ Repobility 3 occurrences [MINED118] Dockerfile FROM `ubuntu:xenial` not pinned by digest: `FROM ubuntu:xenial` resolves the tag at build time. The registry CAN re-push a different image for the same tag, so every build is potentially different. Production images should pin to `image@sha256:...` for reproducibility + supply-chain integrity.
Replace with: `FROM ubuntu:xenial@sha256:<digest>`. Get the digest from `docker manifest inspect`. Re-pin via a scheduled bot (Renovate, Dependabot).
3 files, 3 locations
Dockerfile:2
tools/tts/tts-api/Dockerfile:2
tools/tts/tts/Dockerfile:2
high Security checks software dependencies conf 0.90 ✓ Repobility [MINED134] Binary file `dreamluau.dll` committed in source repo: `dreamluau.dll` is a .dll binary (1,331,712 bytes) committed to a repo that otherwise has 1247 source files. Trojan binaries inside otherwise-normal source repos are a known supply-chain attack: a compromised dependency or PR slips in a binary that gets executed by build scripts.
Audit the binary's provenance. If it's vendored library code, document it in a VENDORED.md. If it's a build artifact, add the extension to .gitignore and rebuild from source.
dreamluau.dll:1
high Security checks software dependencies conf 0.90 ✓ Repobility [MINED134] Binary file `libdreamluau.so` committed in source repo: `libdreamluau.so` is a .so binary (2,673,016 bytes) committed to a repo that otherwise has 1247 source files. Trojan binaries inside otherwise-normal source repos are a known supply-chain attack: a compromised dependency or PR slips in a binary that gets executed by build scripts.
Audit the binary's provenance. If it's vendored library code, document it in a VENDORED.md. If it's a build artifact, add the extension to .gitignore and rebuild from source.
libdreamluau.so:1
high Security checks software dependencies conf 0.90 ✓ Repobility [MINED134] Binary file `rust_g.dll` committed in source repo: `rust_g.dll` is a .dll binary (9,733,120 bytes) committed to a repo that otherwise has 1247 source files. Trojan binaries inside otherwise-normal source repos are a known supply-chain attack: a compromised dependency or PR slips in a binary that gets executed by build scripts.
Audit the binary's provenance. If it's vendored library code, document it in a VENDORED.md. If it's a build artifact, add the extension to .gitignore and rebuild from source.
rust_g.dll:1
high Security checks software dependencies conf 0.90 ✓ Repobility [MINED134] Binary file `tools/dmifonts/DmiFonts.exe` committed in source repo: `tools/dmifonts/DmiFonts.exe` is a .exe binary (27,648 bytes) committed to a repo that otherwise has 1247 source files. Trojan binaries inside otherwise-normal source repos are a known supply-chain attack: a compromised dependency or PR slips in a binary that gets executed by build scripts.
Audit the binary's provenance. If it's vendored library code, document it in a VENDORED.md. If it's a build artifact, add the extension to .gitignore and rebuild from source.
tools/dmifonts/DmiFonts.exe:1
high Security checks security Injection conf 0.50 3 occurrences [SEC004] SQL Injection Risk: String interpolation in SQL execution. Allows SQL injection.
Use parameterized queries: cursor.execute('SELECT * FROM t WHERE id = %s', [id]). For dynamic table or column names, choose identifiers from a hard-coded allowlist and keep values in parameters.
3 files, 3 locations
SQL/admin_import_2018-02-03.py:85
SQL/ban_conversion_2018-10-28.py:172
SQL/populate_key_2018-07-09.py:53
low Security checks security Injection conf 1.00 [SEC103] LDAP injection — non-constant search filter: User input concatenated into an LDAP search filter. Attackers inject `*)(uid=*` style payloads to bypass auth or enumerate accounts.
Escape with javax.naming.ldap.Rdn.escapeValue or equivalent. For python-ldap, use ldap.filter.escape_filter_chars. Better: use parameterized search APIs (Spring LdapTemplate filter encoders).
tools/tts/tts-api/tts-api.py:62
low Security checks security Injection conf 1.00 [SEC103] LDAP injection — non-constant search filter: User input concatenated into an LDAP search filter. Attackers inject `*)(uid=*` style payloads to bypass auth or enumerate accounts.
Escape with javax.naming.ldap.Rdn.escapeValue or equivalent. For python-ldap, use ldap.filter.escape_filter_chars. Better: use parameterized search APIs (Spring LdapTemplate filter encoders).
SQL/populate_key_2018-07-09.py:66
high Security checks cicd CI/CD security conf 0.84 Database service publishes a host port
Use `expose` for service-to-service access, bind to 127.0.0.1 for local-only access, or protect the port with firewall rules.
tools/LinuxOneShot/docker-compose.yml:2 CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.92 Dockerfile pipes a remote script into a shell
Download the artifact, verify its checksum or signature, pin the version, and then execute it.
Dockerfile:49 CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.90 ✓ Repobility 20 occurrences GitHub Action is tag-pinned rather than SHA-pinned
[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout@v6` resolves at workflow-run time. Tags and branches can be re-pushed by the action owner; that made the tj-actions/changed-files compromise (2025) instantly affect ~23K repos. Pin to a 40-char commit SHA + lo…
9 files, 20 locations
.github/workflows/auto_changelog.yml:17, 21, 31 (3 hits)
.github/workflows/compile_changelogs.yml:26, 38, 59 (3 hits)
.github/workflows/discord_discussions.yml:31, 37, 44 (3 hits)
.github/workflows/remove_guide_comments.yml:12, 14 (2 hits)
.github/workflows/rerun_flaky_tests.yml:13, 15 (2 hits)
.github/workflows/setup_build_artifact.yml:17, 38 (2 hits)
.github/workflows/test_merge_bot.yml:26, 35 (2 hits)
.github/workflows/update_tgs_dmapi.yml:17, 43 (2 hits)
CI/CD securitySupply chainGitHub Actions
medium Security checks cicd CI/CD security conf 0.90 ✓ Repobility 4 occurrences GitHub Action is tag-pinned rather than SHA-pinned
[MINED115] Action `re-actors/alls-green` pinned to mutable ref `@release/v1`: `uses: re-actors/alls-green@release/v1` resolves at workflow-run time. Tags and branches can be re-pushed by the action owner; that made the tj-actions/changed-files compromise (2025) instantly affect ~23K repos. Pin to a…
3 files, 4 locations
.github/workflows/update_tgs_dmapi.yml:26, 53 (2 hits)
.github/workflows/ci_suite.yml:86
.github/workflows/generate_client_storage.yml:29
CI/CD securitySupply chainGitHub Actions
high Security checks cicd CI/CD security conf 0.90 ✓ Repobility GitHub Action is tag-pinned rather than SHA-pinned
[MINED115] Action `ad-m/github-push-action` pinned to mutable ref `@master`: `uses: ad-m/github-push-action@master` resolves at workflow-run time. Tags and branches can be re-pushed by the action owner; that made the tj-actions/changed-files compromise (2025) instantly affect ~23K repos. Pin to a 4…
.github/workflows/compile_changelogs.yml:70 CI/CD securitySupply chainGitHub Actions
medium Security checks quality Error handling conf 1.00 [ERR002] Empty Catch Block: Empty catch blocks hide errors.
Log the error or rethrow it. Use console.error() at minimum.
tgui/packages/tgui-dev-server/link/client.ts:66
medium Security checks quality Error handling conf 1.00 [ERR002] Empty Catch Block: Empty catch blocks hide errors.
Log the error or rethrow it. Use console.error() at minimum.
html/admin/search.js:32
medium Security checks quality Quality conf 1.00 ✓ Repobility 2 occurrences [MINED109] Mutable default argument in `__init__` (dict): `def __init__(... = []/{}/set())` — Python's default value is constructed ONCE at function definition time and shared across all calls. Mutating it in one call mutates it for every future call too.
Use None as the default and create the collection inside the function: `def __init__(x=None): x = x or []`
lines 50, 133
tools/maplint/source/lint.py:50, 133 (2 hits)
medium Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED109] Mutable default argument in `opus2midi` (list): `def opus2midi(... = []/{}/set())` — Python's default value is constructed ONCE at function definition time and shared across all calls. Mutating it in one call mutates it for every future call too.
Use None as the default and create the collection inside the function: `def opus2midi(x=None): x = x or []`
2 files, 3 locations
tools/midi2piano/MidiDependencies/midi.py:150, 306 (2 hits)
tools/tts/tts-api/tts-api.py:27
medium Security checks quality Quality conf 1.00 ✓ Repobility 2 occurrences [MINED109] Mutable default argument in `timeshift` (set): `def timeshift(... = []/{}/set())` — Python's default value is constructed ONCE at function definition time and shared across all calls. Mutating it in one call mutates it for every future call too.
Use None as the default and create the collection inside the function: `def timeshift(x=None): x = x or []`
lines 453, 542
tools/midi2piano/MidiDependencies/midi.py:453, 542 (2 hits)
low Security checks security Deserialization conf 1.00 [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code.
Use yaml.safe_load() instead of yaml.load(). Avoid pickle for untrusted data.
tools/minibot/minibot.py:98
low Security checks security Deserialization conf 1.00 [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code.
Use yaml.safe_load() instead of yaml.load(). Avoid pickle for untrusted data.
tgui/packages/tgui/interfaces/Changelog.jsx:93
medium Security checks quality Quality conf 1.00 3 occurrences [SEC087] JS: weak Math.random for crypto: Math.random() is not cryptographically secure; using it for tokens/keys/nonces is predictable. Ported from gosec G404 / eslint detect-pseudoRandomBytes concept (Apache-2.0).
Use `crypto.randomBytes(32).toString('hex')` (Node) or `crypto.getRandomValues()` (browser).
3 files, 3 locations
tgui/packages/tgui/interfaces/DetectiveBoard/Evidence.tsx:37
tgui/packages/tgui/interfaces/SyndContractor.jsx:53
tgui/packages/tgui/interfaces/SyndicateContractor.tsx:107
low Security checks quality Quality conf 1.00 [SEC136] AI-typical over-broad exception handler swallowing all errors: Catch-all exception block that silently returns success or no-ops. AI agents reach for this pattern when a flaky test or an unfamiliar API throws — wrap, swallow, return success. Real bugs are masked, observability is destroyed, and callers think the operation worked. CWE-396 (improperly-generalized exception). Distinct from intentional fallback because there's no log line and the success value is fabricated.
Catch the specific exception type, log at error level with full exception info, and return a failure-shaped result. If the operation is genuinely best-effort, log at warning and document why in a comment so the next reader (or scanner) knows.
tgui/packages/tgui-dev-server/util.ts:25
low Security checks quality Error handling conf 0.55 ✓ Repobility 9 occurrences Broad exception handler needs review
This handler catches Exception/BaseException. It is actionable when it swallows errors without logging, re-raising, or returning a structured error. Handlers that intentionally convert exceptions into typed error results should not be treated as high risk.
7 files, 9 locations
tools/maplint/source/__main__.py:32, 73, 91 (3 hits)
tools/CatchUnescapedBrackets/__main__.py:144
tools/HitboxExpander/__main__.py:68
tools/mapmerge2/frontend.py:122
tools/midi2piano/midi2piano.py:129
tools/minibot/nudge.py:11
tools/silo_grapher/silo_graph_script.py:83
Error handlingquality
medium Security checks cicd CI/CD security conf 0.94 Compose service `tgs` image uses the latest tag
Pin to a maintained version tag or digest and update it deliberately through dependency automation.
tools/LinuxOneShot/docker-compose.yml:14 CI/CD securitycontainers
medium Security checks cicd CI/CD security conf 0.84 Database data bind mount is inside the Docker build context
Prefer a named volume or a host path outside the build context. If a repo-local path is required, add it to .dockerignore and .gitignore and verify backups separately.
tools/LinuxOneShot/docker-compose.yml:2 CI/CD securitycontainers
medium Security checks cicd CI/CD security conf 0.88 Database service has no healthcheck
Add a database-native healthcheck such as pg_isready, mysqladmin ping, redis-cli ping, or the vendor's readiness command.
tools/LinuxOneShot/docker-compose.yml:2 CI/CD securitycontainers
medium Security checks cicd CI/CD security conf 0.84 Docker build context is very large
Shrink the build context with .dockerignore, move generated/runtime data outside the build context, and copy only the manifest files needed for cached dependency layers.
.dockerignore CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.82 3 occurrences Docker final stage has no non-root USER
Add a non-root USER in the final runtime stage after files and permissions are prepared.
3 files, 3 locations
Dockerfile:74
tools/tts/tts-api/Dockerfile:2
tools/tts/tts/Dockerfile:2
CI/CD securitycontainers
medium Security checks cicd CI/CD security conf 0.76 3 occurrences Dockerfile copies broad context with incomplete .dockerignore
Tighten .dockerignore or replace COPY . with explicit COPY statements.
3 files, 3 locations
Dockerfile:42
tools/tts/tts-api/Dockerfile:31
tools/tts/tts/Dockerfile:33
CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.72 .dockerignore misses sensitive defaults
Add missing patterns such as .env, .git, private keys, certificates, dependency folders, and local databases.
.dockerignore CI/CD securitycontainers
low Security checks software Race condition conf 1.00 [SEC124] TOCTOU file access (os.access then open): Check-then-use file pattern (access/exists then open) lets an attacker swap the file between check and use (symlink attack). `mktemp` is deprecated for the same reason.
Use `os.open(path, os.O_CREAT | os.O_EXCL | os.O_WRONLY)` for atomic create-only. Use `tempfile.NamedTemporaryFile()` (not `mktemp`). For locking, use `fcntl.flock`.
tools/ss13_genchangelog.py:91
low Security checks cicd CI/CD security conf 0.68 4 occurrences App service does not wait for database health
Give the database a healthcheck and change the dependency to `depends_on: { db: { condition: service_healthy } }`.
2 files, 4 locations
tools/DockerTestServer/docker-compose.yml:4, 38 (2 hits)
tools/LinuxOneShot/docker-compose.yml:14, 38 (2 hits)
CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.56 6 occurrences Compose service does not declare a runtime user
Set a non-root `user:` in Compose or ensure the final image stage has a non-root USER directive.
3 files, 6 locations
tools/DockerTestServer/docker-compose.yml:4, 38 (2 hits)
tools/LinuxOneShot/docker-compose.yml:14, 38 (2 hits)
tools/tts/docker-compose.yml:1, 16 (2 hits)
CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.62 6 occurrences Compose service lacks no-new-privileges hardening
Add `security_opt: ["no-new-privileges:true"]` unless the service has a documented need for privilege escalation.
3 files, 6 locations
tools/DockerTestServer/docker-compose.yml:4, 38 (2 hits)
tools/LinuxOneShot/docker-compose.yml:14, 38 (2 hits)
tools/tts/docker-compose.yml:1, 16 (2 hits)
CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.72 Dockerfile installs recommended OS packages
Add `--no-install-recommends` and explicitly list only packages the image needs.
tools/tts/tts-api/Dockerfile:5 CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.72 Dockerfile installs recommended OS packages
Add `--no-install-recommends` and explicitly list only packages the image needs.
tools/tts/tts/Dockerfile:5 CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.72 Dockerfile keeps pip download cache
Use `pip install --no-cache-dir ...` in container builds.
tools/tts/tts-api/Dockerfile:25 CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.72 Dockerfile keeps pip download cache
Use `pip install --no-cache-dir ...` in container builds.
tools/tts/tts/Dockerfile:25 CI/CD securitycontainers
low Security checks cicd CI/CD security conf 0.74 Dockerfile leaves apt package indexes in the image layer
End the apt install layer with `rm -rf /var/lib/apt/lists/*`.
Dockerfile:4 CI/CD securitycontainers
low Security checks quality Quality conf 0.60 20 occurrences Duplicated implementation block across source files
Duplicate implementation blocks are maintenance debt. Keep them visible, but they are not a high-severity defect unless the duplicated logic is security-sensitive or drifting.
12 files, 17 locations
tgui/packages/tgui/interfaces/Fabricator.tsx:142, 147, 155 (3 hits)
tgui/packages/tgui/interfaces/AtmosPump.tsx:39, 66 (2 hits)
tgui/packages/tgui/interfaces/ChemReactionChamber.tsx:38, 47 (2 hits)
tgui/packages/tgui/interfaces/ExosuitFabricator.tsx:91, 106 (2 hits)
tgui/packages/tgui/interfaces/AiStatusDisplayPicker.tsx:54
tgui/packages/tgui/interfaces/AntagInfoTraitor.tsx:160
tgui/packages/tgui/interfaces/AtmosTempGate.tsx:19
tgui/packages/tgui/interfaces/AtmosTempPump.tsx:18
duplicationquality
For AI agents: Voting guide (TP/FP) MCP manifest Stdio wrapper SARIF Integrate Findings queue Vote TP/FP on findings to calibrate the engine.
For AI agents + API integrations
Email me when this repo regresses
Free. We re-scan periodically; new criticals → your inbox. No signup required for the scan itself.
API access

This page is publicly accessible at: https://repobility.com/scan/f8952fa4-ed0a-4e8c-b907-53439ca008de/

To check status programmatically (no auth required):

curl -s https://repobility.com/api/v1/public/scan/f8952fa4-ed0a-4e8c-b907-53439ca008de/

Important — please don't re-submit the same URL repeatedly. The submission endpoint is idempotent: re-submitting the same git URL returns this same scan_token, not a new one. To re-scan this repo, sign up free and use the dashboard.