kylecorry31/Trail-Sense

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.

53 of your 95 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 35.37s for a 52.4 MB repo slow.
Repobility's analysis ran in 14.71s after the clone landed.

https://github.com/kylecorry31/Trail-Sense · scanned 2026-06-05 17:11 UTC (4 days, 22 hours ago) · 10 languages

185 raw signals (83 security + 102 graph) 11/13 scanners ran 0th percentile · Kotlin · large (100-500K LoC) System graph score 74 (lower by 4)

File as issue Image

UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

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

JSON

88.9% cov · 26 findings

Score breakdown â 2026-05-18-v5

Component	Sub-score	Weight	Contribution
`structure_score`	65.0	0.15	9.75
`security_score`	100.0	0.25	25.00
`testing_score`	17.0	0.20	3.40
`documentation_score`	77.0	0.15	11.55
`practices_score`	81.0	0.15	12.15
`code_quality`	79.0	0.10	7.90
Overall		1.00	69.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 1 High 10 Medium 13 Low 15 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Security checks 40 System graph 26 Crowd 0 Layer: Quality 39 Software 13 Cicd 4 Security 8 Api 1 Frontend 1

Exclude dismissed / FP Exclude test files

Scan summary Quality grade B- (70/100). Dimensions: security 100, maintainability 65. 83 findings (32 security). 124,618 lines analyzed.

Showing 38 of 66 actionable findings. 134 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 [MINED107] Missing import: `html` used but not imported: The file uses `html.something(...)` but never imports `html`. This raises NameError at runtime the first time the line executes.

Add `import html` at the top of the file.

site/build.py:116

high Security checks quality Quality conf 1.00 ✓ Repobility [MINED107] Missing import: `locale` used but not imported: The file uses `locale.something(...)` but never imports `locale`. This raises NameError at runtime the first time the line executes.

Add `import locale` at the top of the file.

scripts/update-guide.py:10

critical System graph security Secrets conf 1.00 2 occurrences Possible secret in scripts/release-builds.sh

Detected pattern matching password_literal. Rotate the credential and move to a secret manager.

lines 13, 15

scripts/release-builds.sh:13, 15 (2 hits)

high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED029] Kotlin Null Bang: x!! throws NullPointerException if x is null. Bypasses Kotlins null safety.

Review and fix per the pattern semantics. See CWE-476 / for context.

3 files, 3 locations

app/src/androidTest/java/com/kylecorry/trail_sense/tools/ToolsTest.kt:48

app/src/main/java/com/kylecorry/trail_sense/shared/data/GeographicImageSource.kt:281

app/src/main/java/com/kylecorry/trail_sense/shared/extensions/GeoJsonExtensions.kt:41

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.

survival-guide-book/generate-book.py:39

high Security checks quality Quality conf 1.00 ✓ Repobility 4 occurrences [MINED108] `self.__get_urls` used but never assigned in __init__: Method `check` of class `URLMismatch` reads `self.__get_urls`, 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.__get_urls = <default>` in __init__, or add a class-level default.

lines 92, 93, 143, 146

scripts/string-fixer.py:92, 93, 143, 146 (4 hits)

high Security checks software dependencies conf 0.90 ✓ Repobility [MINED134] Binary file `gradle/wrapper/gradle-wrapper.jar` committed in source repo: `gradle/wrapper/gradle-wrapper.jar` is a .jar binary (43,504 bytes) committed to a repo that otherwise has 1753 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.

gradle/wrapper/gradle-wrapper.jar:1

high Security checks security path traversal conf 0.80 [SEC013] Path Traversal — User Input in File Path: User-controlled input used in file path without sanitization. Allows reading arbitrary files.

Use os.path.realpath() and verify the path starts with your expected base directory. Use secure_filename() for uploads.

site/generation/markdown_converter.py:6

Use os.path.realpath() and verify the path starts with your expected base directory. Use secure_filename() for uploads.

site/build.py:41

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).

release.py:16

medium Security checks cicd CI/CD security conf 0.90 ✓ Repobility 12 occurrences GitHub Action is tag-pinned rather than SHA-pinned

[MINED115] Action `gradle/actions/setup-gradle` pinned to mutable ref `@v3`: `uses: gradle/actions/setup-gradle@v3` 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…

4 files, 12 locations

.github/workflows/codeql.yml:71, 95 (4 hits)

.github/workflows/android-test.yml:36, 41 (3 hits)

.github/workflows/staging-smoke-test.yml:37, 43 (3 hits)

.github/workflows/release.yml:39 (2 hits)

CI/CD securitySupply chainGitHub Actions

low Security checks cicd CI/CD security conf 0.90 ✓ Repobility 35 occurrences GitHub Action is tag-pinned rather than SHA-pinned

[MINED115] Action `actions/checkout` pinned to mutable ref `@v4`: `uses: actions/checkout@v4` 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…

8 files, 35 locations

.github/workflows/nightly.yml:15, 21, 30, 59, 66 (9 hits)

.github/workflows/debug-apk.yml:13, 19, 31 (5 hits)

.github/workflows/staging-smoke-test.yml:18, 24, 52 (5 hits)

.github/workflows/android-test.yml:17, 23, 57 (4 hits)

.github/workflows/site.yml:36, 49, 52, 57 (4 hits)

.github/workflows/android.yml:18, 24 (3 hits)

.github/workflows/release.yml:14, 20 (3 hits)

.github/workflows/codeql.yml:61 (2 hits)

CI/CD securitySupply chainGitHub Actions

medium Security checks quality Quality conf 1.00 ✓ Repobility [MINED109] Mutable default argument in `convert_to_book` (list): `def convert_to_book(... = []/{}/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 convert_to_book(x=None): x = x or []`

survival-guide-book/generate-book.py:15

medium Security checks software Resource exhaustion conf 1.00 [SEC037] Uncontrolled Recursion — stack/depth exhaustion: Parsing arbitrary-depth user input (XML, JSON, YAML) without a depth limit, or recursive function over user-controlled structure. Attacker sends `{"a":{"a":{"a":...10000 levels...}}}` to blow the stack. Real CVEs: CVE-2019-16935 (Python xmlrpc), CVE-2020-25659 (PyYAML before 5.4). CWE-674/1325.

Use `defusedxml.ElementTree` instead of `xml.etree.ElementTree` — it rejects deeply-nested + billion-laughs payloads. For JSON: set a depth limit explicitly: import json data = json.loads(s) # then validate structure depth manually For YAML: always use `yaml.safe_load`. For recursive code over…

site/build.py:16

low Security checks quality Error handling conf 0.55 ✓ Repobility 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.

site/watch.py:25 Error handlingquality

medium Security checks quality Quality conf 0.78 Suspicious implementation file appears unreferenced

Confirm whether this file is reachable. If not, delete it; if yes, wire it through explicit imports, routes, or entry points and add a test that proves the path executes.

scripts/files-to-refactor.py:1

medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — site/src/devices.js:12

Bare `fetch(...)` will throw an unhandled rejection on network failure. Wrap in try/catch, attach a `.catch(...)`, or pass an AbortSignal with a timeout.

runtime safetyRobustness

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/release.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/site.yml CI/CD securitySupply chainGithub actions

medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — release.py:27

`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 — scripts/create-weather-test-data.py:23

`urllib.request.urlopen(...)` 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 — scripts/update_contributors.py:12

`requests.get(...)` 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 — site/watch.py:48

`subprocess.Popen(...)` 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

2 test file(s) for 25 source file(s) (ratio 0.08). Consider adding integration or unit tests for critical paths.

Coverage

low Security checks quality Quality conf 0.60 8 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.

8 files, 8 locations

app/src/androidTest/java/com/kylecorry/trail_sense/test_utils/ToolTestBase.kt:40

app/src/main/java/com/kylecorry/trail_sense/plugins/map_layers/PluginTileSource.kt:36

app/src/main/java/com/kylecorry/trail_sense/shared/dem/map_layers/ElevationMapTileSource.kt:22

app/src/main/java/com/kylecorry/trail_sense/shared/extensions/TrailSenseReactiveFragment.kt:37

app/src/main/java/com/kylecorry/trail_sense/shared/map_layers/ui/layers/geojson/features/GeoJsonPolygonRenderer.kt:114

app/src/main/java/com/kylecorry/trail_sense/shared/sensors/overrides/CachedGPS.kt:34

app/src/main/java/com/kylecorry/trail_sense/shared/sensors/overrides/OverrideGPS.kt:22

app/src/main/java/com/kylecorry/trail_sense/shared/text/search/MultilingualFuzzySearchStrategy.kt:40

duplicationquality

high Security checks quality Quality conf 0.62 Source file name looks like an AI patch artifact

Rename it to the domain concept it implements or merge it into the existing module it was meant to change.

scripts/files-to-refactor.py:1

low System graph software Dead code candidate conf 1.00 File has no detected symbols: release.py