← Back to scan
File as GitHub Issue repo: CleverRaven/Cataclysm-DDA

Push this scan report to CleverRaven/Cataclysm-DDA

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

Missing import: `json` used but not imported

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 MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/npc_class…:7
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/item_cate…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/recipe.py:15
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/map_extra…:8
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/ammunitio…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/body_part…:11
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/palette.py:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/recipe_ca…:12
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/sub_body_…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/fault_fix…:11
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/city.py:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/morale_ty…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/mutation_…:20
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/speed_des…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/weakpoint…:6
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/martial_a…:12
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/examine_a…:20
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/proficien…:8
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/keybindin…:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/faction.py:8
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/gate.py:5
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/item_acti…:6
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/parsers/npc.py:85
CRIT MINED107 [MINED107] Missing import: `string` used but not imported: The file uses `string.somethin… lang/string_extractor/pot_export.py:107
CRIT MINED107 [MINED107] Missing import: `json` used but not imported: The file uses `json.something(..… lang/string_extractor/write_text.py:52
CRIT MINED022 [MINED022] C Strcpy: strcpy/strcat dont bounds-check; use strncpy or snprintf. tools/format/getpost.h:39
CRIT MINED022 [MINED022] C Strcpy: strcpy/strcat dont bounds-check; use strncpy or snprintf. tools/clang-tidy-plugin/Utils.h:224
HIGH MINED108 [MINED108] `self.summ_desc` used but never assigned in __init__: Method `__str__` of clas… tools/generate_changelog.py:220
HIGH MINED108 [MINED108] `self.summ_type` used but never assigned in __init__: Method `__str__` of clas… tools/generate_changelog.py:219
HIGH MINED108 [MINED108] `self.merge_dttm` used but never assigned in __init__: Method `__str__` of cla… tools/generate_changelog.py:219
HIGH MINED108 [MINED108] `self.id` used but never assigned in __init__: Method `__str__` of class `CDDA… tools/generate_changelog.py:219
HIGH MINED108 [MINED108] `self.author` used but never assigned in __init__: Method `__str__` of class `… tools/generate_changelog.py:216
HIGH MINED108 [MINED108] `self.title` used but never assigned in __init__: Method `__str__` of class `C… tools/generate_changelog.py:216
HIGH MINED108 [MINED108] `self.summ_type` used but never assigned in __init__: Method `__str__` of clas… tools/generate_changelog.py:215
HIGH MINED108 [MINED108] `self.merge_dttm` used but never assigned in __init__: Method `__str__` of cla… tools/generate_changelog.py:215
HIGH MINED108 [MINED108] `self.id` used but never assigned in __init__: Method `__str__` of class `CDDA… tools/generate_changelog.py:215
HIGH MINED108 [MINED108] `self.summ_type` used but never assigned in __init__: Method `__str__` of clas… tools/generate_changelog.py:213
HIGH MINED108 [MINED108] `self.has_valid_summary` used but never assigned in __init__: Method `__str__`… tools/generate_changelog.py:217
HIGH MINED108 [MINED108] `self.has_valid_summary` used but never assigned in __init__: Method `__str__`… tools/generate_changelog.py:213
HIGH MINED108 [MINED108] `self.id` used but never assigned in __init__: Method `_get_summary` of class … tools/generate_changelog.py:203
HIGH MINED108 [MINED108] `self.summ_desc` used but never assigned in __init__: Method `has_valid_summar… tools/generate_changelog.py:189
HIGH MINED108 [MINED108] `self.summ_type` used but never assigned in __init__: Method `has_valid_summar… tools/generate_changelog.py:189
HIGH MINED108 [MINED108] `self.summ_type` used but never assigned in __init__: Method `has_valid_summar… tools/generate_changelog.py:188
HIGH MINED108 [MINED108] `self._summ_type` used but never assigned in __init__: Method `summ_type` of c… tools/generate_changelog.py:184
HIGH MINED108 [MINED108] `self._summ_type` used but never assigned in __init__: Method `summ_type` of c… tools/generate_changelog.py:180
HIGH MINED108 [MINED108] `self._resume_main_thread` used but never assigned in __init__: Method `wait_f… tools/windows_limit_memory.py:650
HIGH MINED108 [MINED108] `self.has_io_port` used but never assigned in __init__: Method `wait_for_job` … tools/windows_limit_memory.py:651
HIGH MINED108 [MINED108] `self.is_started_process` used but never assigned in __init__: Method `wait_fo… tools/windows_limit_memory.py:648
HIGH MINED108 [MINED108] `self._set_information_job_object` used but never assigned in __init__: Method… tools/windows_limit_memory.py:633
HIGH MINED108 [MINED108] `self._query_information_job_object` used but never assigned in __init__: Meth… tools/windows_limit_memory.py:624
HIGH MINED108 [MINED108] `self._set_information_job_object` used but never assigned in __init__: Method… tools/windows_limit_memory.py:533
HIGH MINED001 [MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e… utilities/make_iso.py:198
HIGH MINED001 [MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e… tools/json_tools/cddatags.py:96
HIGH MINED036 [MINED036] Python Os System Call: os.system() invokes shell with no escaping. tools/json_tools/convert_item_damage.py:119
HIGH MINED036 [MINED036] Python Os System Call: os.system() invokes shell with no escaping. tools/json_tools/convert_armor.py:298
HIGH MINED036 [MINED036] Python Os System Call: os.system() invokes shell with no escaping. tools/json_tools/adjust_values.py:32
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … lang/string_extractor/pot_export.py:95
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … build-scripts/post-diff-as-comments.py:291
HIGH SEC128 [SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call … android/app/src/main/java/com/cleverrav…:211
HIGH DKR014 Dockerfile copies the entire context without .dockerignore build-scripts/Dockerfile.debian-build:64
HIGH MINED134 [MINED134] Binary file `android/gradle/wrapper/gradle-wrapper.jar` committed in source re… android/gradle/wrapper/gradle-wrapper.j…:1
HIGH MINED115 [MINED115] Action `actions/download-artifact` pinned to mutable ref `@v4`: `uses: actions… .github/workflows/sdl3-matrix.yml:156
HIGH MINED115 [MINED115] Action `actions/upload-artifact` pinned to mutable ref `@v4`: `uses: actions/u… .github/workflows/sdl3-matrix.yml:119
HIGH MINED117 [MINED117] Workflow declares `permissions: write-all`: The job's GITHUB_TOKEN gets EVERY … .github/workflows/release-android-bundl…:8
HIGH MINED117 [MINED117] Workflow declares `permissions: write-all`: The job's GITHUB_TOKEN gets EVERY … .github/workflows/release.yml:271
HIGH MINED117 [MINED117] Workflow declares `permissions: write-all`: The job's GITHUB_TOKEN gets EVERY … .github/workflows/release.yml:31
HIGH MINED115 [MINED115] Action `actions/upload-artifact` pinned to mutable ref `@v4`: `uses: actions/u… .github/workflows/release.yml:142
HIGH MINED118 [MINED118] Dockerfile FROM `ubuntu:focal` not pinned by digest: `FROM ubuntu:focal` resol… tools/llama/gcc-focal/Dockerfile:2
HIGH MINED118 [MINED118] Dockerfile FROM `ghcr.io/nelhage/llama (no tag)` not pinned by digest: `FROM g… tools/llama/gcc-focal/Dockerfile:1
HIGH MINED118 [MINED118] Dockerfile FROM `mcr.microsoft.com/devcontainers/cpp:0-ubuntu-22.04` not pinne… .devcontainer/cross-compile/Dockerfile:1
HIGH MINED118 [MINED118] Dockerfile FROM `mcr.microsoft.com/devcontainers/cpp:0-ubuntu-22.04` not pinne… .devcontainer/graphical/Dockerfile:1
HIGH MINED118 [MINED118] Dockerfile FROM `debian:testing` not pinned by digest: `FROM debian:testing` r… build-scripts/Dockerfile.debian-build:8
HIGH MINED118 [MINED118] Dockerfile FROM `mcr.microsoft.com/devcontainers/cpp:0-ubuntu-22.04` not pinne… .devcontainer/Dockerfile:1
HIGH SEC013 [SEC013] Path Traversal — User Input in File Path: User-controlled input used in file pat… tools/json_tools/update-translate-dialo…:75
HIGH SEC004 [SEC004] SQL Injection Risk: String interpolation in SQL execution. Allows SQL injection. lang/string_extractor/parsers/mission_d…:26
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… tools/json_tools/cddatags.py:71
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… tools/json_tools/update-translate-dialo…:82
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… tools/json_tools/lister.py:48
MED MINED111 [MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r… tools/json_tools/util.py:38
MED MINED109 [MINED109] Mutable default argument in `parse_dynamic_line` (list): `def parse_dynamic_li… lang/string_extractor/parsers/talk_topi…:50
MED MINED109 [MINED109] Mutable default argument in `write_to_pot` (list): `def write_to_pot(... = []/… lang/string_extractor/pot_export.py:121
MED SEC134 [SEC134] AI scaffold leftover — Lorem ipsum / example.com / John Doe in code: Lorem ipsum… tools/spell_checker/__init__.py:100
MED SEC123 [SEC123] Production stack trace / debug output exposed: Debug mode left on in production … src/third-party/snmalloc/ds_core/define…:123
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … src/math_parser_jmath.cpp:77
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … src/math_parser_impl.h:69
MED SEC045 [SEC045] eval()/exec() on stored or user-supplied data: eval() and exec() on data — even … src/math_parser.h:27
MED COMP001 [COMP001] High cognitive complexity: Function `run_iwyu_on` has cognitive complexity 17 (… build-scripts/ci-iwyu-run.py:171
MED DKR002 Dockerfile base image has no explicit tag tools/llama/gcc-focal/Dockerfile:1
MED DKR007 Docker build context has no .dockerignore .dockerignore
MED DKR009 Dockerfile separates apt update from install tools/llama/gcc-focal/Dockerfile:3
MED DKR009 Dockerfile separates apt update from install .devcontainer/graphical/Dockerfile:3
MED DKR009 Dockerfile separates apt update from install .devcontainer/cross-compile/Dockerfile:25
MED DKR009 Dockerfile separates apt update from install .devcontainer/Dockerfile:3
MED DKR001 Docker final stage has no non-root USER tools/llama/gcc-focal/Dockerfile:2
MED DKR001 Docker final stage has no non-root USER .devcontainer/graphical/Dockerfile:1
MED DKR001 Docker final stage has no non-root USER .devcontainer/cross-compile/Dockerfile:1
MED DKR001 Docker final stage has no non-root USER .devcontainer/Dockerfile:1
MED AIC001 Parallel implementation file sits beside a canonical file src/overmap_special_fixed.cpp:1
MED AIC001 Parallel implementation file sits beside a canonical file lang/string_extractor/parsers/fault_fix…:1
MED AIC004 Suspicious implementation file appears unreferenced tools/gfx_tools/png_update.py:1
MED SEC005 [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input. tools/json_tools/convert_item_damage.py:119
MED SEC005 [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input. tools/json_tools/convert_armor.py:298
MED SEC005 [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input. tools/json_tools/adjust_values.py:32
MED CORE_LARGE_FILES Average file size is 550 lines (recommend <300) —
LOW SEC132 [SEC132] String concat where the language has interpolation (AI style drift): String buil… utilities/make_iso.py:49
LOW COMP001 [COMP001] High cognitive complexity: Function `get_affected_files` has cognitive complexi… build-scripts/ci-iwyu-run.py:87
LOW COMP001 [COMP001] High cognitive complexity: Function `filter_analyzable_files` has cognitive com… build-scripts/ci-iwyu-run.py:151
LOW AIC005 Duplicate top-level symbol appears in a patch-style file tools/gfx_tools/png_update.py:1
LOW AIC002 Source file name looks like an AI patch artifact tools/gfx_tools/png_update.py:1
INFO MINED047 [MINED047] Emoji In Source: Emoji ✅ ❌ 🚀 in code/comments — common AI output unless explic… tools/spell_checker/__init__.py:86
INFO MINED064 [MINED064] Python Input Call: input() blocks for stdin. Inappropriate in services. tools/json_tools/name_strings_to_object…:75
INFO MINED050 [MINED050] Stub Only Function: Function declared but body is just pass, return None, rais… utilities/make_iso.py:199
INFO MINED050 [MINED050] Stub Only Function: Function declared but body is just pass, return None, rais… tools/json_tools/cddatags.py:97
INFO MINED069 [MINED069] Debug True Prod: Django/Flask DEBUG=True or app.debug=True in non-test files. src/third-party/snmalloc/ds_core/define…:123
INFO MINED075 [MINED075] C Malloc No Check: malloc/calloc/realloc return value used without checking fo… src/mmap_file.cpp:48
INFO MINED042 [MINED042] Cpp New Without Delete: C++ raw new without RAII / unique_ptr — memory leak ri… src/pimpl.h:44
INFO MINED042 [MINED042] Cpp New Without Delete: C++ raw new without RAII / unique_ptr — memory leak ri… src/mmap_file.cpp:337
INFO MINED042 [MINED042] Cpp New Without Delete: C++ raw new without RAII / unique_ptr — memory leak ri… src/mingw.thread.h:64
INFO MINED043 [MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr… src/dependency_tree.h:83
INFO MINED077 [MINED077] Python Open No Context: fp = open(path) outside with-block leaks file handles. lang/strip_line_numbers.py:18
INFO MINED049 [MINED049] Print Pii: Logging password/token/email/ssn directly to stdout. build-scripts/post-diff-as-comments.py:60
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … tools/json_tools/cddaUpdateJsonVolume.js:10
INFO MINED044 [MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger … build-scripts/generate-release-notes.js:103
Reset to top 5 123 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `CleverRaven/Cataclysm-DDA`

**Score: 88/100 (A-)**  ·  124 findings  ·  scanned 2026-06-05 20:11 UTC  ·  963,058 LOC

| Severity | Count |
|---|---|
| CRITICAL | 27 |
| HIGH | 48 |
| MEDIUM | 29 |
| LOW | 5 |

📊 [Full filterable report](https://repobility.com/scan/74ca0b50-7228-42e5-8a73-a70ebe684044/)  ·  ![scorecard](https://repobility.com/scan/74ca0b50-7228-42e5-8a73-a70ebe684044/report.png?v=1780690298-s2)

### Top findings

1. **CRITICAL** `MINED107` — Missing import: `json` used but not imported
   `lang/string_extractor/parsers/npc_class.py:7` · ✓ Repobility
2. **CRITICAL** `MINED107` — Missing import: `json` used but not imported
   `lang/string_extractor/parsers/item_category.py:5` · ✓ Repobility
3. **CRITICAL** `MINED107` — Missing import: `json` used but not imported
   `lang/string_extractor/parsers/recipe.py:15` · ✓ Repobility
4. **CRITICAL** `MINED107` — Missing import: `json` used but not imported
   `lang/string_extractor/parsers/map_extra.py:8` · ✓ Repobility
5. **CRITICAL** `MINED107` — Missing import: `json` used but not imported
   `lang/string_extractor/parsers/ammunition_type.py:5` · ✓ Repobility

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/74ca0b50-7228-42e5-8a73-a70ebe684044/_
Megaproject â high spam risk
Could not determine 'CleverRaven/Cataclysm-DDA' 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.
Open in GitHub to file this issue Cancel

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.