Open-source Android/Desktop remake of Civ V
This report summarizes scanner evidence for yairm210/Unciv, a Kotlin project with 11,514 GitHub stars. SynthScan examined 221,314 lines of code across 1507 source files, recording 59 pattern matches distributed across 9 syntactic categories. The legacy adjusted pattern signal is 1.0 (Low pattern signal); it is not an authorship estimate.
Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.
Use each vector to guide manual verification of the underlying findings.
Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.
Can maintainers explain intent, assumptions, and trade-offs? 42 finding(s).
Are imports declared and package/API names verified? 15 finding(s).
Are wrappers, guards, and repeated structures necessary? 2 finding(s).
Which changes need focused human review? 59 finding(s).
This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.
Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.
This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.
The scanner identified 59 distinct pattern matches across 9 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.
Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| CRITICAL | …com/unciv/ui/screens/devconsole/ConsoleUnitCommands.kt | 44 | val options = console.gameInfo.ruleset.unitPromotions.keys.asSequence() | CODE |
| CRITICAL | …com/unciv/ui/screens/devconsole/ConsoleTileCommands.kt | 176 | console.gameInfo.ruleset.terrains.values.asSequence() | CODE |
| CRITICAL | core/src/com/unciv/logic/battle/CityCombatant.kt | 58 | val techsPercentKnown: Float = if (techCount > 0) city.civ.tech.techsResearched.size.toFloat() / techCount else | CODE |
| CRITICAL | core/src/com/unciv/logic/city/managers/CityFounder.kt | 198 | foundingCiv.gameInfo.ruleset.nations.values.asSequence().filter { nation -> | CODE |
| CRITICAL | …com/unciv/logic/civilization/managers/PolicyManager.kt | 94 | get() = civInfo.gameInfo.ruleset.policyBranches.values.toSet() | CODE |
| CRITICAL | …com/unciv/logic/civilization/managers/PolicyManager.kt | 172 | val worldSizeModifier = civInfo.gameInfo.tileMap.mapParameters.mapSize.getPredefinedOrNextSmaller().policyCostPe | CODE |
| CRITICAL | …nciv/logic/civilization/managers/GreatPersonManager.kt | 111 | fun getGreatPeople() = civInfo.gameInfo.ruleset.units.values.asSequence() | CODE |
| CRITICAL | …c/com/unciv/logic/civilization/managers/TurnManager.kt | 213 | val unitToSpawn = civInfo.gameInfo.ruleset.units.values.asSequence().filter { | CODE |
| CRITICAL | …c/com/unciv/logic/civilization/managers/TechManager.kt | 114 | val mapSizePredef = civInfo.gameInfo.tileMap.mapParameters.mapSize.getPredefinedOrNextSmaller() | CODE |
| CRITICAL | core/src/com/unciv/logic/map/mapunit/UnitPromotions.kt | 188 | return unit.civ.gameInfo.ruleset.unitPromotions.values.asSequence().filter { isAvailable(it) } | CODE |
| CRITICAL | core/src/com/unciv/logic/automation/Automation.kt | 192 | val chosenUnit = chooseMilitaryUnit(city, city.civ.gameInfo.ruleset.units.values.asSequence()) | CODE |
| CRITICAL | …/unciv/logic/automation/unit/SpecificUnitAutomation.kt | 118 | && !unit.civ.gameInfo.ruleset.modOptions.hasUnique(UniqueType.CityStatesSearchForFirstCitySite) | CODE |
| CRITICAL | …/unciv/logic/automation/unit/SpecificUnitAutomation.kt | 239 | val averageTerrainStatsValue = unit.civ.gameInfo.ruleset.terrains.values.asSequence() | CODE |
| CRITICAL | core/src/com/unciv/models/Spy.kt | 367 | val techSize = civInfo.gameInfo.ruleset.technologies.values.size.toFloat() | CODE |
| CRITICAL | …com/unciv/models/ruleset/unique/UniqueParameterType.kt | 200 | com.unciv.models.stats.Stats.isStats(parameterText) | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | …src/com/unciv/ui/screens/worldscreen/unit/UnitTable.kt | 221 | if (curUnit != null && curUnit.isPreparingAirSweep()) return | COMMENT |
| LOW | …src/com/unciv/ui/components/tilegroups/TileGroupMap.kt | 221 | return super.hit(x, y, touchable) | COMMENT |
| LOW | core/src/com/unciv/ui/popups/options/AutomationTab.kt | 61 | // } | COMMENT |
| LOW | core/src/com/unciv/logic/city/City.kt | 621 | "in City-State cities" -> civ.isCityState | COMMENT |
| LOW | core/src/com/unciv/logic/trade/TradeEvaluation.kt | 541 | val totalAdvantage = (absoluteAdvantage * percentageAdvantage).toInt() | COMMENT |
| LOW | …c/com/unciv/logic/map/mapunit/movement/MovementCost.kt | 161 | return 0f // should not reach this point | COMMENT |
| LOW | …nciv/logic/map/mapgenerator/MapGenerationRandomness.kt | 81 | // OPTIMIZATION NOTE: Since `seed` and `scale` are constant for all tiles during a single | COMMENT |
| LOW | core/src/com/unciv/logic/multiplayer/storage/DropBox.kt | 141 | // fun fileExists(fileName: String): Boolean = try { | COMMENT |
| LOW | core/src/com/unciv/models/ModConstants.kt | 21 | var maxXPfromBarbarians = 30 | COMMENT |
| LOW | …/ui/components/tilegroups/EditorMapHolderMemoryTest.kt | 81 | COMMENT | |
| LOW | …iv/logic/civilization/CapitalConnectionsFinderTests.kt | 81 | } | COMMENT |
| LOW | …iv/logic/civilization/CapitalConnectionsFinderTests.kt | 121 | createMedium(0,2, RoadStatus.Road) | COMMENT |
| LOW | …iv/logic/civilization/CapitalConnectionsFinderTests.kt | 141 | // val notConnectedCity2 = testGame.addCity(ourCiv, testGame.tileMap[2, 0]) | COMMENT |
| LOW | tests/src/com/unciv/uniques/DeprecatedUniquesTest.kt | 141 | ) | COMMENT |
| LOW | tests/src/com/unciv/dev/OjhBenchmark.kt | 261 | seconds: Double, lateTurns: Int) { | COMMENT |
| LOW | …droid/assets/jsons/Civ V - Gods & Kings/Buildings.json | 721 | // { | COMMENT |
| LOW | …oid/assets/jsons/translations/Persian instructions.txt | 1 | -> General Notes for Pinglish: | COMMENT |
| LOW | .github/workflows/detekt_config/detekt-warnings.yml | 21 | # - 'KtFileCountProcessor' | COMMENT |
| LOW | .github/workflows/detekt_config/detekt-errors.yml | 21 | # - 'KtFileCountProcessor' | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 662 | // Step 1: Release the destination tile | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 666 | // Step 2: Perform the movement | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 668 | // Step 3: Release the newly taken tile | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 672 | // Step 4: Restore the initial position after step 1 | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 678 | // Step 5: Perform the another movement | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 680 | // Step 6: Restore the position in the new tile after step 3 | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 686 | // Step 6: Update states | COMMENT |
| LOW | android/src/com/unciv/app/AndroidGame.kt | 32 | * changes, but also on other things. So we need to check if that was actually | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 662 | // Step 1: Release the destination tile | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 666 | // Step 2: Perform the movement | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 668 | // Step 3: Release the newly taken tile | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 672 | // Step 4: Restore the initial position after step 1 | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 678 | // Step 5: Perform the another movement | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 680 | // Step 6: Restore the position in the new tile after step 3 | COMMENT |
| LOW⚡ | …c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt | 686 | // Step 6: Update states | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | …nciv/logic/map/mapgenerator/MapGenerationRandomness.kt | 108 | // Map the 2D coordinates onto a 3D cylinder to achieve seamless X-axis world wrapping. | COMMENT |
| MEDIUM | …/unciv/logic/automation/unit/SpecificUnitAutomation.kt | 63 | // a more robust scoring may be necessary to avoid leaving generals en-prise, | COMMENT |
| MEDIUM | …ts/src/com/unciv/logic/civilization/CityMovingTests.kt | 26 | // Required for enemy to utilize roads | COMMENT |
| MEDIUM | …iv/logic/civilization/CapitalConnectionsFinderTests.kt | 36 | //Add techs to utilize roads | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| HIGH | …iv/logic/automation/civilization/ReligionAutomation.kt | 232 | // All of these Random.nextFloat() don't exist in the original, but I've added them to make things a bit more ra | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | …com/unciv/ui/components/tilegroups/layers/TileLayer.kt | 34 | // ── scene-graph helpers ────────────────────────────────────────────────── | COMMENT |
| MEDIUM | …com/unciv/ui/components/tilegroups/layers/TileLayer.kt | 60 | // ── positioning helpers ────────────────────────────────────────────────── | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | …unciv/ui/screens/civilopediascreen/ICivilopediaText.kt | 14 | * ### Usage: | COMMENT |
| LOW | tests/src/com/unciv/logic/TranslationTests.kt | 831 | * ### Usage: | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | core/src/com/unciv/logic/GameInfo.kt | 61 | * And don't forget to add backwards compatibility for the previous format. | COMMENT |