AI Slop Code Risk Monitor

yairm210/Unciv

SlopCodeMonitor badge for yairm210/Unciv Adjusted legacy signal; badge bands: 25 / 50 / 75 (different from dashboard bands).

Open-source Android/Desktop remake of Civ V

1.0 Low pattern signal View on GitHub

Open-Source Code Quality, Security and Technical Debt Review

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.

1.0
Adjusted Signal
1.0
Raw Pattern Score
100%
Time Factor
2026-10-29
Last Push
11.5K
Stars
Kotlin
Language
221.3K
Lines of Code
1.5K
Files
59
Pattern Hits
2026-10-31
Scan Date
0.01
HC Hit Rate

Risk Review · Payload v1

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.

Cognitive Ownership Risk 1.7/100 · LOW

Can maintainers explain intent, assumptions, and trade-offs? 42 finding(s).

Hallucinated Dependency & Supply Chain Risk 3.3/100 · LOW

Are imports declared and package/API names verified? 15 finding(s).

Agent Over-Engineering Ratio 0.1/100 · LOW

Are wrappers, guards, and repeated structures necessary? 2 finding(s).

PR Review Velocity Impact 4.7/100 · LOW

Which changes need focused human review? 59 finding(s).

What These Metrics Mean

Adjusted Signal
Legacy pattern score normalized per 1,000 scanned lines and multiplied by the repository-level time factor. It is not an estimate of AI authorship.
Raw Pattern Score
Sum of severity- and context-weighted pattern findings before applying the time factor.
Time Factor
A coarse multiplier based on repository last-push metadata. It does not identify when individual code was written.
Pattern Hits
Count of configured pattern findings. Review each location and surrounding code before acting.
HC Hit Rate
High- and critical-severity findings per scanned file. Severity is a configured detector weight, not proof of a vulnerability or authorship.
Lines / Files
Scan coverage denominators used by the legacy normalized score.

Pattern-Signal History

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.

Severity Breakdown

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.

CRITICAL 15HIGH 1MEDIUM 6LOW 37

Directory Score Breakdown

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.

Pattern Findings

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.

Hallucination Indicators15 hits · 150 pts
SeverityFileLineSnippetContext
CRITICAL…com/unciv/ui/screens/devconsole/ConsoleUnitCommands.kt44 val options = console.gameInfo.ruleset.unitPromotions.keys.asSequence()CODE
CRITICAL…com/unciv/ui/screens/devconsole/ConsoleTileCommands.kt176 console.gameInfo.ruleset.terrains.values.asSequence()CODE
CRITICALcore/src/com/unciv/logic/battle/CityCombatant.kt58 val techsPercentKnown: Float = if (techCount > 0) city.civ.tech.techsResearched.size.toFloat() / techCount else CODE
CRITICALcore/src/com/unciv/logic/city/managers/CityFounder.kt198 foundingCiv.gameInfo.ruleset.nations.values.asSequence().filter { nation ->CODE
CRITICAL…com/unciv/logic/civilization/managers/PolicyManager.kt94 get() = civInfo.gameInfo.ruleset.policyBranches.values.toSet()CODE
CRITICAL…com/unciv/logic/civilization/managers/PolicyManager.kt172 val worldSizeModifier = civInfo.gameInfo.tileMap.mapParameters.mapSize.getPredefinedOrNextSmaller().policyCostPeCODE
CRITICAL…nciv/logic/civilization/managers/GreatPersonManager.kt111 fun getGreatPeople() = civInfo.gameInfo.ruleset.units.values.asSequence()CODE
CRITICAL…c/com/unciv/logic/civilization/managers/TurnManager.kt213 val unitToSpawn = civInfo.gameInfo.ruleset.units.values.asSequence().filter {CODE
CRITICAL…c/com/unciv/logic/civilization/managers/TechManager.kt114 val mapSizePredef = civInfo.gameInfo.tileMap.mapParameters.mapSize.getPredefinedOrNextSmaller()CODE
CRITICALcore/src/com/unciv/logic/map/mapunit/UnitPromotions.kt188 return unit.civ.gameInfo.ruleset.unitPromotions.values.asSequence().filter { isAvailable(it) }CODE
CRITICALcore/src/com/unciv/logic/automation/Automation.kt192 val chosenUnit = chooseMilitaryUnit(city, city.civ.gameInfo.ruleset.units.values.asSequence())CODE
CRITICAL…/unciv/logic/automation/unit/SpecificUnitAutomation.kt118 && !unit.civ.gameInfo.ruleset.modOptions.hasUnique(UniqueType.CityStatesSearchForFirstCitySite)CODE
CRITICAL…/unciv/logic/automation/unit/SpecificUnitAutomation.kt239 val averageTerrainStatsValue = unit.civ.gameInfo.ruleset.terrains.values.asSequence()CODE
CRITICALcore/src/com/unciv/models/Spy.kt367 val techSize = civInfo.gameInfo.ruleset.technologies.values.size.toFloat()CODE
CRITICAL…com/unciv/models/ruleset/unique/UniqueParameterType.kt200 com.unciv.models.stats.Stats.isStats(parameterText)CODE
Over-Commented Block19 hits · 19 pts
SeverityFileLineSnippetContext
LOW…src/com/unciv/ui/screens/worldscreen/unit/UnitTable.kt221 if (curUnit != null && curUnit.isPreparingAirSweep()) returnCOMMENT
LOW…src/com/unciv/ui/components/tilegroups/TileGroupMap.kt221 return super.hit(x, y, touchable)COMMENT
LOWcore/src/com/unciv/ui/popups/options/AutomationTab.kt61// }COMMENT
LOWcore/src/com/unciv/logic/city/City.kt621 "in City-State cities" -> civ.isCityStateCOMMENT
LOWcore/src/com/unciv/logic/trade/TradeEvaluation.kt541 val totalAdvantage = (absoluteAdvantage * percentageAdvantage).toInt()COMMENT
LOW…c/com/unciv/logic/map/mapunit/movement/MovementCost.kt161 return 0f // should not reach this pointCOMMENT
LOW…nciv/logic/map/mapgenerator/MapGenerationRandomness.kt81 // OPTIMIZATION NOTE: Since `seed` and `scale` are constant for all tiles during a single COMMENT
LOWcore/src/com/unciv/logic/multiplayer/storage/DropBox.kt141// fun fileExists(fileName: String): Boolean = try {COMMENT
LOWcore/src/com/unciv/models/ModConstants.kt21 var maxXPfromBarbarians = 30COMMENT
LOW…/ui/components/tilegroups/EditorMapHolderMemoryTest.kt81COMMENT
LOW…iv/logic/civilization/CapitalConnectionsFinderTests.kt81 }COMMENT
LOW…iv/logic/civilization/CapitalConnectionsFinderTests.kt121 createMedium(0,2, RoadStatus.Road)COMMENT
LOW…iv/logic/civilization/CapitalConnectionsFinderTests.kt141// val notConnectedCity2 = testGame.addCity(ourCiv, testGame.tileMap[2, 0])COMMENT
LOWtests/src/com/unciv/uniques/DeprecatedUniquesTest.kt141 )COMMENT
LOWtests/src/com/unciv/dev/OjhBenchmark.kt261 seconds: Double, lateTurns: Int) {COMMENT
LOW…droid/assets/jsons/Civ V - Gods & Kings/Buildings.json721// {COMMENT
LOW…oid/assets/jsons/translations/Persian instructions.txt1 -> General Notes for Pinglish:COMMENT
LOW.github/workflows/detekt_config/detekt-warnings.yml21 # - 'KtFileCountProcessor'COMMENT
LOW.github/workflows/detekt_config/detekt-errors.yml21 # - 'KtFileCountProcessor'COMMENT
Verbosity Indicators8 hits · 17 pts
SeverityFileLineSnippetContext
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt662 // Step 1: Release the destination tileCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt666 // Step 2: Perform the movementCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt668 // Step 3: Release the newly taken tileCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt672 // Step 4: Restore the initial position after step 1COMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt678 // Step 5: Perform the another movementCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt680 // Step 6: Restore the position in the new tile after step 3COMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt686 // Step 6: Update statesCOMMENT
LOWandroid/src/com/unciv/app/AndroidGame.kt32 * changes, but also on other things. So we need to check if that was actuallyCOMMENT
Structural Annotation Overuse7 hits · 16 pts
SeverityFileLineSnippetContext
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt662 // Step 1: Release the destination tileCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt666 // Step 2: Perform the movementCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt668 // Step 3: Release the newly taken tileCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt672 // Step 4: Restore the initial position after step 1COMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt678 // Step 5: Perform the another movementCOMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt680 // Step 6: Restore the position in the new tile after step 3COMMENT
LOW⚡…c/com/unciv/logic/map/mapunit/movement/UnitMovement.kt686 // Step 6: Update statesCOMMENT
AI Slop Vocabulary4 hits · 12 pts
SeverityFileLineSnippetContext
MEDIUM…nciv/logic/map/mapgenerator/MapGenerationRandomness.kt108 // Map the 2D coordinates onto a 3D cylinder to achieve seamless X-axis world wrapping.COMMENT
MEDIUM…/unciv/logic/automation/unit/SpecificUnitAutomation.kt63 // a more robust scoring may be necessary to avoid leaving generals en-prise,COMMENT
MEDIUM…ts/src/com/unciv/logic/civilization/CityMovingTests.kt26 // Required for enemy to utilize roadsCOMMENT
MEDIUM…iv/logic/civilization/CapitalConnectionsFinderTests.kt36 //Add techs to utilize roadsCOMMENT
AI Response Leakage1 hit · 8 pts
SeverityFileLineSnippetContext
HIGH…iv/logic/automation/civilization/ReligionAutomation.kt232 // All of these Random.nextFloat() don't exist in the original, but I've added them to make things a bit more raCOMMENT
Decorative Section Separators2 hits · 6 pts
SeverityFileLineSnippetContext
MEDIUM…com/unciv/ui/components/tilegroups/layers/TileLayer.kt34 // ── scene-graph helpers ──────────────────────────────────────────────────COMMENT
MEDIUM…com/unciv/ui/components/tilegroups/layers/TileLayer.kt60 // ── positioning helpers ──────────────────────────────────────────────────COMMENT
Example Usage Blocks2 hits · 3 pts
SeverityFileLineSnippetContext
LOW…unciv/ui/screens/civilopediascreen/ICivilopediaText.kt14 * ### Usage:COMMENT
LOWtests/src/com/unciv/logic/TranslationTests.kt831 * ### Usage:COMMENT
Slop Phrases1 hit · 2 pts
SeverityFileLineSnippetContext
LOWcore/src/com/unciv/logic/GameInfo.kt61 * And don't forget to add backwards compatibility for the previous format.COMMENT