{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"}]}},"results":[{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: AndroidNavigator: TooManyMethods \u2014 36 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"alligator/src/main/java/me/aartikov/alligator/AndroidNavigator.kt"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"116a52f450a95213a9de534b270c9142af4eeba67e71eac9411aeb3c310d225f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): alligator/src/main/java/me/aartikov/alligator/helpers/ScreenResultHelper.kt:78-89 | alligator/src/main/java/me/aartikov/alligator/helpers/ScreenResultHelper.kt:108-119 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060alligator/src/main/java/me/aartikov/alligator/helpers/ScreenResultHelper.kt:78\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"alligator/src/main/java/me/aartikov/alligator/helpers/ScreenResultHelper.kt"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"bf168bb612c2a7411ea584e43788811f8bb53e79e625ab2520e72756ccb0cc25"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): flowsample/src/main/java/me/aartikov/flowsample/ui/TestSmallFragment.kt:52-61 | navigationmethodssample/src/main/java/me/aartikov/navigationmethodssample/ui/TestFragment.kt:51-60 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060flowsample/src/main/java/me/aartikov/flowsample/ui/TestSmallFragment.kt:52\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"flowsample/src/main/java/me/aartikov/flowsample/ui/TestSmallFragment.kt"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"870b903bd3ecdad5960b2b82f43fbd337ed813a40e2d1aa5f7a1ce56ea6c1a56"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 3): flowsample/src/main/java/me/aartikov/flowsample/ui/TestFlowFragment.kt:37-43 | flowsample/src/main/java/me/aartikov/flowsample/ui/TestSmallFragment.kt:38-43 | navigationmethodssample/src/main/java/me/aartikov/navigationmethodssample/ui/TestFragment.kt:36-42 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made 3 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"flowsample/src/main/java/me/aartikov/flowsample/ui/TestFlowFragment.kt"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"238c68b7d340c29f0a133ed2a1e395bbf717af244c29ccc535a3807756e6edb2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): flowsample/src/main/java/me/aartikov/flowsample/SampleTransitionAnimationProvider.kt:21-25 | navigationmethodssample/src/main/java/me/aartikov/navigationmethodssample/SampleTransitionAnimationProvider.kt:21-25 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060flowsample/src/main/java/me/aartikov/flowsample/SampleTransitionAnimationProvider.kt:21\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"flowsample/src/main/java/me/aartikov/flowsample/SampleTransitionAnimationProvider.kt"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"e09459aa6dd213cccada028b4e95a57c848a80b2245f2a2bccb61e3b78924c21"}},{"ruleId":"D16","level":"warning","message":{"text":"dormant codebase \u2014 no living knowledge left to concentrate: All 32 significant source file(s) were last meaningfully changed so long ago that no living knowledge remains \u2014 nothing since has been substantial enough to re-establish ownership (a broad, mechanical sweep that touches many files shallowly does not count, and neither does no activity at all). There is no concentration to measure, so the bus factor is not scored. This is not a clean bill: nobody currently holds working knowledge of this code (see D34 Knowledge Freshness)."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"569d55ea5cb0330f13c125b289d07e5abdfcceac947a2c2645277937942e7d5f"}},{"ruleId":"D34","level":"note","message":{"text":"Dormant codebase: 34 of 34 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 34 separate risks. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: MainActivity.kt \u2194 MessageInputActivity.kt: \u0060screenresultsample/src/main/java/me/aartikov/screenresultsample/ui/MainActivity.kt\u0060 and \u0060screenresultsample/src/main/java/me/aartikov/screenresultsample/ui/MessageInputActivity.kt\u0060 change together 67% of the time (12 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"screenresultsample/src/main/java/me/aartikov/screenresultsample/ui/MainActivity.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a84204949e95647e89a9c38fd24814a54015e6f62e0e9f88391ba1e85308a27"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling clique: ActivityA.kt, ActivityB.kt, FragmentC.kt, FragmentD.kt: 4 files \u2014 \u0060simplenavigationsample/src/main/java/me/aartikov/simplenavigationsample/ui/ActivityA.kt\u0060, \u0060simplenavigationsample/src/main/java/me/aartikov/simplenavigationsample/ui/ActivityB.kt\u0060, \u0060simplenavigationsample/src/main/java/me/aartikov/simplenavigationsample/ui/FragmentC.kt\u0060, \u0060simplenavigationsample/src/main/java/me/aartikov/simplenavigationsample/ui/FragmentD.kt\u0060 \u2014 all change together with no explicit dependency: a fully-connected co-change clique, not 6 separate couplings. They share one concern (thin parallel siblings over a common abstraction), so extract the shared part into ONE unit and the whole clique\u0027s coupling clears at once \u2014 you do not need to break each pair individually."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"simplenavigationsample/src/main/java/me/aartikov/simplenavigationsample/ui/ActivityA.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f3683eae349c4374ded132affe86e3629e0d8442c8ab5b12ec037cc944bf872"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: BackCommand.kt \u2194 ReplaceCommand.kt: \u0060alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt\u0060 and \u0060alligator/src/main/java/me/aartikov/alligator/commands/ReplaceCommand.kt\u0060 change together 60% of the time (24 of the 40 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c3042a4d768533eceb9a4359ff4f223fa440009c798f03bab5f2abd81c4971b1"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: BackCommand.kt \u2194 BackToClassCommand.kt: \u0060alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt\u0060 and \u0060alligator/src/main/java/me/aartikov/alligator/commands/BackToClassCommand.kt\u0060 change together 60% of the time (24 of the 40 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ab3fd68d2516b079ea533c342a3f9a3e62fcbcd1be2cf34c780c348c5b03a85"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: BackCommand.kt \u2194 FinishCommand.kt: \u0060alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt\u0060 and \u0060alligator/src/main/java/me/aartikov/alligator/commands/FinishCommand.kt\u0060 change together 58% of the time (22 of the 38 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"alligator/src/main/java/me/aartikov/alligator/commands/BackCommand.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"844b0033a67eb6f8f6189ea9ca7f69c8038178ada3b90060c30aa69129ca24a5"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":0,"secretScannerRunsExcluded":0}}}]}