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.
Sign up free Scan another repo
101 of your 129 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 41.11s for a 54.3 MB repo slow.
  • Repobility's analysis ran in 5.87s after the clone landed.

florianfesti/boxes

https://github.com/florianfesti/boxes · scanned 2026-06-05 20:13 UTC (4 days, 13 hours ago) · 10 languages

285 raw signals (125 security + 160 graph) 11/13 scanners ran 63rd percentile · Python · medium (20-100K LoC) System graph score 79 (lower by 8)

File as issue Image
UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

Last scanned 4 days, 13 hours ago · v2 · 88 actionable findings from 2 signal sources. 117 repeated signals grouped for readability. Security checks, system graph analysis, and verified AI-agent feedback are merged into one review queue.

JSON
Combined
80.8
/100
Security checks
82
30 findings
System graph
79
88.9% cov · 58 findings
Critical
0
click to filter
Agents
6
0 votes
Crowd
0
reported
Score breakdown â 2026-05-18-v5
Component Sub-score Weight Contribution
structure_score 75.0 0.15 11.25
security_score 100.0 0.25 25.00
testing_score 25.0 0.20 5.00
documentation_score 86.0 0.15 12.90
practices_score 79.0 0.15 11.85
code_quality 58.0 0.10 5.80
Overall 1.00 71.8
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
Severity: Critical 0 High 11 Medium 15 Low 45 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Security checks 30 System graph 58 Crowd 0 Layer: Software 29 Quality 38 Cicd 11 Security 3 Api 1 Frontend 2 Hardware 4
Scan summary Quality grade B (72/100). Dimensions: security 100, maintainability 75. 125 findings (29 security). 41,978 lines analyzed.

Showing 71 of 88 actionable findings. 205 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.

high Security checks quality Quality conf 1.00 ✓ Repobility 25 occurrences [MINED107] Missing import: `warnings` used but not imported: The file uses `warnings.something(...)` but never imports `warnings`. This raises NameError at runtime the first time the line executes.
Add `import warnings` at the top of the file.
12 files, 14 locations
boxes/generators/bintray.py:23, 129 (2 hits)
boxes/generators/regularstarbox.py:26, 76 (2 hits)
boxes/gears.py:638
boxes/generators/angledcutjig.py:48
boxes/generators/breadbox.py:39
boxes/generators/carbonfilter.py:97
boxes/generators/cardholder.py:41
boxes/generators/coindisplay.py:24
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.
setup.py:28
high Security checks quality Quality conf 1.00 ✓ Repobility [MINED036] Python Os System Call: os.system() invokes shell with no escaping.
Review and fix per the pattern semantics. See CWE-78 / for context.
setup.py:42
high Security checks quality Quality conf 1.00 ✓ Repobility 25 occurrences [MINED108] `self.bedBoltSettings` used but never assigned in __init__: Method `open` of class `Boxes` reads `self.bedBoltSettings`, 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.bedBoltSettings = <default>` in __init__, or add a class-level default.
2 files, 25 locations
setup.py:43, 48, 49, 50, 51, 52, 58, 59, +4 more (15 hits)
boxes/__init__.py:460, 461, 463, 465, 470, 471, 472, 473, +1 more (10 hits)
high Security checks software dependencies conf 0.90 ✓ Repobility 2 occurrences [MINED118] Dockerfile FROM `python:3.12-slim` not pinned by digest: `FROM python:3.12-slim` 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 python:3.12-slim@sha256:<digest>`. Get the digest from `docker manifest inspect`. Re-pin via a scheduled bot (Renovate, Dependabot).
lines 1, 13
scripts/Dockerfile:1, 13 (2 hits)
high Security checks software dependencies conf 0.90 ✓ Repobility [MINED119] Dockerfile `ADD https://github.com/florianfesti/boxes.git`: Dockerfile `ADD <url>` downloads a remote artifact into the image with no integrity check. If the host or DNS is compromised between layers — or if the URL serves a different file later — malicious content gets baked into the image.
Download the file in CI with a known checksum, vendor it into the repo, and COPY it during the build. Or use `RUN curl -sSL URL | sha256sum -c <(echo '<expected> -')` to verify.
scripts/Dockerfile:7
high Security checks software dependencies conf 0.90 ✓ Repobility 8 occurrences [MINED131] pre-commit hook `https://github.com/pre-commit/pre-commit-hooks` pinned to mutable rev `v6.0.0`: `.pre-commit-config.yaml` references `https://github.com/pre-commit/pre-commit-hooks` at `rev: v6.0.0`. If `{rev}` is a branch or version tag, the repo owner can push new code there and `pre-commit install --install-hooks` will fetch it on every developer's machine.
Pin to a commit SHA: `rev: <40-char-sha>` and bump it through `pre-commit autoupdate` (which writes to PRs that are reviewed).
lines 8, 22, 33, 39, 45, 65, 71, 76
.pre-commit-config.yaml:8, 22, 33, 39, 45, 65, 71, 76 (8 hits)
low Security checks cicd CI/CD security conf 0.90 ✓ Repobility 12 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…
3 files, 12 locations
.github/workflows/pages.yml:20, 22, 42, 66 (7 hits)
.github/workflows/precommit.yml:16, 17, 24 (3 hits)
.github/workflows/docker-publish.yml:38 (2 hits)
CI/CD securitySupply chainGitHub Actions
medium Security checks cicd CI/CD security conf 0.90 ✓ Repobility GitHub Action is tag-pinned rather than SHA-pinned
[MINED115] Action `pre-commit/action` pinned to mutable ref `@v3.0.1`: `uses: pre-commit/[email protected]` 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 commi…
.github/workflows/precommit.yml:23 CI/CD securitySupply chainGitHub Actions
medium Security checks quality Quality conf 1.00 ✓ Repobility 4 occurrences [MINED109] Mutable default argument in `addTopEdgeSettings` (dict): `def addTopEdgeSettings(... = []/{}/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 addTopEdgeSettings(x=None): x = x or []`
2 files, 4 locations
boxes/scripts/boxesserver.py:189, 219, 228 (3 hits)
boxes/lids.py:258
medium Security checks quality Quality conf 1.00 ✓ Repobility 4 occurrences [MINED109] Mutable default argument in `text` (list): `def text(... = []/{}/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 text(x=None): x = x or []`
3 files, 4 locations
boxes/__init__.py:1529, 2475 (2 hits)
boxes/lids.py:314
boxes/scripts/boxesserver.py:46
medium Security checks software dependencies conf 0.90 ✓ Repobility 8 occurrences [MINED124] requirements.txt: `markdown` has no version pin: Unpinned pip requirement means every fresh install may resolve a different version. Newer releases can introduce malicious code (typosquats, account compromises). Reproducible installs need exact pins.
Replace `markdown` with `markdown==<version>` and manage upgrades through PRs / Dependabot.
lines 2, 3, 4, 5, 7, 8, 10, 11
requirements.txt:2, 3, 4, 5, 7, 8, 10, 11 (8 hits)
low Security checks security Injection conf 0.50 [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input.
Use subprocess with shell=False and a list of args. Never eval user input.
setup.py:28
low Security checks quality Error handling conf 0.55 ✓ Repobility 3 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.
3 files, 3 locations
boxes/scripts/boxes_generator.py:330
boxes/scripts/boxesserver.py:693
boxes/svgmerge.py:129
Error handlingquality
medium Security checks cicd CI/CD security conf 0.90 Docker build context has no .dockerignore
Add .dockerignore with at least .git, .env, private keys, dependency folders, build outputs, and local databases.
.dockerignore CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.82 Docker final stage has no non-root USER
Add a non-root USER in the final runtime stage after files and permissions are prepared.
scripts/Dockerfile:14 CI/CD securitycontainers
medium Security checks cicd CI/CD security conf 0.84 Dockerfile ADD downloads remote content
Use curl/wget with a pinned URL, verify checksum or signature, and prefer COPY for local files.
scripts/Dockerfile:8 CI/CD securitycontainers
medium System graph hardware Supply chain conf 1.00 Dockerfile ADD downloads remote content without checksum
Remote build inputs can change or be replaced upstream. Use Dockerfile ADD --checksum or download with an explicit digest/signature verification step.
scripts/Dockerfile:7 containersChecksum
medium System graph hardware Security conf 1.00 Dockerfile runs as root: scripts/Dockerfile
No non-root USER set. Containers running as root expand the blast radius of any vulnerability inside the image.
Container
medium System graph cicd CI/CD security conf 1.00 GitHub Actions workflow grants broad write permissions
CI tokens with write permissions increase blast radius when an action, dependency, or PR workflow is compromised. Prefer job-level least-privilege permissions.
.github/workflows/docker-publish.yml CI/CD securitySupply chainGithub actions
medium System graph cicd CI/CD security conf 1.00 GitHub Actions workflow grants broad write permissions
CI tokens with write permissions increase blast radius when an action, dependency, or PR workflow is compromised. Prefer job-level least-privilege permissions.
.github/workflows/pages.yml CI/CD securitySupply chainGithub actions
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in setup.py:28
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
setup.py:28 Subprocess shell true
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — boxes/formats.py:96
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — boxes/scripts/boxes_proxy.py:54
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph security Coverage conf 1.00 No auth library detected
The scanner did not find any standard auth library (JWT, OAuth, NextAuth, Auth0, etc.). Either auth lives in custom code, in a separate service, or is missing.
auth
medium System graph quality Tests conf 1.00 Very low test-to-source ratio
5 test file(s) for 209 source file(s) (ratio 0.02). Consider adding integration or unit tests for critical paths.
Coverage
high Security checks cicd CI/CD security conf 0.56 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.
docker-compose.yml:1 CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.62 Compose service lacks no-new-privileges hardening
Add `security_opt: ["no-new-privileges:true"]` unless the service has a documented need for privilege escalation.
docker-compose.yml:1 CI/CD securitycontainers
high Security checks cicd CI/CD security conf 0.72 2 occurrences Dockerfile keeps pip download cache
Use `pip install --no-cache-dir ...` in container builds.
lines 6, 9
scripts/Dockerfile:6, 9 (2 hits)
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/*`.
scripts/Dockerfile:16 CI/CD securitycontainers
low Security checks quality Quality conf 0.60 11 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.
11 files, 11 locations
boxes/generators/flexbox4.py:4
boxes/generators/shutterbox.py:87
boxes/generators/smallpartstray.py:109
boxes/generators/triangularwall.py:3
boxes/generators/typetray.py:155
boxes/generators/wallrollholder.py:9
boxes/generators/wallslottedholder.py:38
boxes/generators/wallstackablebin.py:60
duplicationquality
low System graph quality Maintenance conf 1.00 45 TODO/FIXME markers
High count of TODO/FIXME/HACK markers — track them as issues so they're not forgotten.
low System graph hardware Coverage conf 1.00 Containers defined but no K8s/orchestration manifest found
Repo has Dockerfiles/compose but no Kubernetes/Nomad manifests. If the target deployment is K8s, the manifests may live in a separate ops repo.
Deployment
low System graph hardware Supply chain conf 1.00 2 occurrences Docker base image is tag-pinned but not digest-pinned: python:3.12-slim
Container tags can be retagged upstream. Pin production base images to a reviewed digest (`image@sha256:...`) when reproducibility and supply-chain integrity matter.
lines 1, 13
scripts/Dockerfile:1, 13 (2 hits)
containersPinned dependencies
low System graph software Dead code candidate conf 1.00 File has no detected symbols: documentation/src/conf.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph quality Integrity conf 1.00 17 occurrences Near-duplicate function bodies in 2 places
Functions with the same first-5-line body hash: boxes/__init__.py:f, boxes/__init__.py:f This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidate or document why they're separate.
17 occurrences
repo-level (17 hits)
duplicatesduplication
low System graph quality Integrity conf 1.00 2 occurrences Near-duplicate function bodies in 3 places
Functions with the same first-5-line body hash: boxes/__init__.py:restore, boxes/__init__.py:r, boxes/__init__.py:r This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidate or document why they're separate.
2 occurrences
repo-level (2 hits)
duplicatesduplication
low System graph quality Integrity conf 1.00 Near-duplicate function bodies in 5 places
Functions with the same first-5-line body hash: boxes/walledges.py:lengths, boxes/walledges.py:lengths, boxes/walledges.py:lengths, boxes/walledges.py:lengths This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidate or document …
duplicatesduplication
low System graph software Dead code conf 1.00 Possibly dead Python function: argparseColumnsDefinition
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/generators/keyboard.py:106
low System graph software Dead code conf 1.00 Possibly dead Python function: argparseInts
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:210
low System graph software Dead code conf 1.00 Possibly dead Python function: bottomCB
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/generators/angledcutjig.py:36
low System graph software Dead code conf 1.00 Possibly dead Python function: buildInkscapeExt
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
setup.py:18
low System graph software Dead code conf 1.00 Possibly dead Python function: edges
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/servos.py:75
low System graph software Dead code conf 1.00 Possibly dead Python function: endAngle
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/edges.py:353
low System graph software Dead code conf 1.00 Possibly dead Python function: example_output_fname_formatter
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/scripts/boxes_main.py:254
low System graph software Dead code conf 1.00 Possibly dead Python function: flushlen
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/edges.py:1550
low System graph software Dead code conf 1.00 Possibly dead Python function: generate_mo_files
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
setup.py:33
low System graph software Dead code conf 1.00 Possibly dead Python function: hexHolesHex
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:2128
low System graph software Dead code conf 1.00 Possibly dead Python function: inx
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:262
low System graph software Dead code conf 1.00 Possibly dead Python function: mirrorY
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:3136
low System graph software Dead code conf 1.00 Possibly dead Python function: outsetlen
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/edges.py:1511
low System graph software Dead code conf 1.00 Possibly dead Python function: pdiff
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/drawing.py:36
low System graph software Dead code conf 1.00 Possibly dead Python function: r
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:3142
low System graph software Dead code conf 1.00 Possibly dead Python function: regulatorCB
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/generators/makitapowersupply.py:112
low System graph software Dead code conf 1.00 Possibly dead Python function: ringSegment
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/parts.py:144
low System graph software Dead code conf 1.00 Possibly dead Python function: serve
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/scripts/boxesserver.py:624
low System graph software Dead code conf 1.00 Possibly dead Python function: set_font_size
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/drawing.py:368
low System graph software Dead code conf 1.00 Possibly dead Python function: spacing_type
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/__init__.py:411
low System graph software Dead code conf 1.00 Possibly dead Python function: startAngle
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/edges.py:349
low System graph software Dead code conf 1.00 Possibly dead Python function: text_extents
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/drawing.py:378
low System graph software Dead code conf 1.00 Possibly dead Python function: updatePOT
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
setup.py:25
low System graph software Dead code conf 1.00 Possibly dead Python function: vclip
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
boxes/vectors.py:30
low System graph frontend Frontend quality conf 1.00 Stray `console.log` in TS/JS — static/self.js:161
Replace with the toast helper, an error boundary, or remove. `console.warn` / `console.error` are acceptable. Why: Hygiene — easy to leak debug output. Rule id: fq.console-leak
Fq console leak
low System graph quality Integrity conf 1.00 Stub function `checkValues` (body is just `pass`/`return`) — boxes/generators/stevensonscreenbox.py:52
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `cutout` (body is just `pass`/`return`) — boxes/generators/beequeentransportbox.py:31
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `flush` (body is just `pass`/`return`) — boxes/drawing.py:55
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `side` (body is just `pass`/`return`) — boxes/generators/lamp.py:73
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Complexity conf 1.00 Very large file: boxes/__init__.py (3147 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
low System graph quality Complexity conf 1.00 Very large file: boxes/edges.py (2713 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
low System graph quality Complexity conf 1.00 Very large file: boxes/generators/beequeentransportbox.py (985 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
low System graph quality Complexity conf 1.00 Very large file: boxes/generators/kamishibai.py (803 lines)
Files with >800 lines often hide complexity hotspots and discourage tests.
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/57162b1e-634b-42c9-b6ff-6f3bc90ba2fd/

To check status programmatically (no auth required): 

curl -s https://repobility.com/api/v1/public/scan/57162b1e-634b-42c9-b6ff-6f3bc90ba2fd/

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.