Read 2026-11-03
openshot/openshot-qt: technical due diligence, read from its repository
What Borehole read in openshot/openshot-qt's code and history at 55123d7: its licence, security advisories, releases, review practice, tests, dependencies and operations. Every finding cites the file and line it rests on. The full report also covers the team and asks the questions the code cannot answer.
| Dimension | Rating |
|---|---|
| Architecture & Codebase — including AI provenance | strong |
| Engineering Process & SDLC Maturity | strong |
| Product & Engineering Maturity | not assessable |
| Security & Reliability Posture | strong |
| Observability & Operations | not assessable |
| Fit with the Acquirer | not assessable |
Architecture & Codebase — including AI provenance
strongsrc/windows/main_window.py is 5,975 lines, 37× this codebase's median file
adequateThe median source file here is 159 lines. 17 files exceed 1,590 lines, the largest being 5,975. Size is not a defect on its own — some problems genuinely live in one place — but these are where merge conflicts, review fatigue and single-owner knowledge concentrate, and they are the first thing that slows an inheriting engineer down.
src/windows/main_window.py:1 — 5,975 lines (37× median) |
src/windows/views/timeline.py:1 — 5,450 lines (34× median) |
src/windows/views/timeline_backend/qwidget/base.py:1 — 4,190 lines (26× median) |
src/windows/views/timeline_backend/qwidget/keyframe_panel.py:1 — 3,561 lines (22× median) |
src/windows/video_widget.py:1 — 3,186 lines (20× median) |
147 errors caught and discarded in product code
adequate147 places catch an error and do nothing with it, across 49 files (0.9 per thousand lines). Each is a failure the system will not report. They surface later as problems nobody can explain, and they are cheap to fix now.
freeze.py:113 — error caught and discarded |
freeze.py:779 — error caught and discarded |
installer/fix_qt5_rpath.py:36 — error caught and discarded |
Engineering Process & SDLC Maturity
strongNo finding.
Product & Engineering Maturity
not assessableNot rated: only 9 of 19 checks could reach a conclusion here; the rest do not apply to this repository or could not read it.
Security & Reliability Posture
strong3 shell commands built from strings
adequate3 places run a command through a shell (`shell=True`, `os.system`, `exec` with a template) rather than passing arguments. Where any part comes from outside, it is command injection. Ask where each one's input comes from.
installer/build_server.py:145 — command run through a shell |
installer/build_server.py:323 — command run through a shell |
src/language/generate_translations.py:448 — command run through a shell |
2 deserialisers that can run code
adequate2 places read data with a deserialiser that executes what it reads (pickle). That is safe on data the system wrote itself and remote code execution on anything else. Ask where each one's input comes from.
src/classes/clipboard.py:81 — code-executing deserialiser |
src/classes/project_data.py:678 — code-executing deserialiser |
Observability & Operations
not assessableNot rated: only 5 of 16 checks could reach a conclusion here; the rest do not apply to this repository or could not read it.
Fit with the Acquirer
not assessableNot rated: structurally not assessable from a repository.
Read more, or read your own
The full report on openshot/openshot-qt: every dimension, the questions for the room, and the audit log.
Survey a repository: free on public repositories. Private code is read on your own machine by a collector whose source you can read first.