{"$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":"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":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D12","name":"Dependency Hygiene","shortDescription":{"text":"Dependency Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/D12"},{"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"]}]},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D18","name":"Solution Shape","shortDescription":{"text":"Solution Shape"},"helpUri":"https://codehealth.canine.dev/dimensions/D18"},{"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":"D24","name":"Comment Value","shortDescription":{"text":"Comment Value"},"helpUri":"https://codehealth.canine.dev/dimensions/D24"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D27","name":"Navigability","shortDescription":{"text":"Navigability"},"helpUri":"https://codehealth.canine.dev/dimensions/D27"},{"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"]}]},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29"},{"id":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}]},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"}]}},"results":[{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured: The test suite couldn\u0027t be built/run in-image and no coverage report is committed, so line coverage was not measured \u2014 and it is EXCLUDED from the score rather than scored on a LoC-ratio proxy. Commit the Cobertura/OpenCover/lcov report your CI already produces (anywhere in the repo), or make the suite runnable in-image, and real coverage will be measured."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test projects found in the repository."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c9bf64cbb5a4ae13d6bcd01fa3bc8d2879d860c73bc67f176ee3c2cecb8adce3"}},{"ruleId":"D16","level":"warning","message":{"text":"single-maintainer \u2014 knowledge-concentration (bus factor) risk: single-maintainer \u2014 knowledge-concentration (bus factor) risk (1 author(s) across 1 commit(s) sampled)."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b4b49d961df742f0e507f4cc5c8094fb077ba0391a27fd015d18320865187719"}},{"ruleId":"D21","level":"note","message":{"text":"The \u0060move\u0060 method is implemented in multiple distinct types (\u0060Direction\u0060, \u0060RightUp\u0060, \u0060Up\u0060, \u0060LeftDown\u0060, \u0060Down\u0060, \u0060RightDown\u0060, \u0060LeftUp\u0060) with identical signatures, suggesting these are likely related classes (e.g., direction vectors or enums) that should share a common interface or base class, or at least use a more specific name like \u0060Update\u0060 or \u0060Apply\u0060 to avoid confusion with the \u0060Snake.move\u0060 method.: Consider renaming \u0060move\u0060 to something more specific like \u0060Update\u0060 or \u0060Apply\u0060 to distinguish it from the \u0060Snake.move\u0060 method, or implement a common interface. (symbols: Method: Direction.move(ref int), Method: RightUp.move(ref int), Method: Up.move(ref int), Method: LeftDown.move(ref int), Method: Down.move(ref int), Method: RightDown.move(ref int), Method: LeftUp.move(ref int))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"70423578937c89fb749430625e7bec183881b3343c828e4ae28600afb35d7ad9"}},{"ruleId":"D21","level":"note","message":{"text":"Both \u0060Snake\u0060 and \u0060Body\u0060 classes have a public method named \u0060move\u0060. While they are different classes, the naming is ambiguous because \u0060Body\u0060 likely represents a part of the \u0060Snake\u0060. It is unclear if \u0060Body.move\u0060 is the same logical operation as \u0060Snake.move\u0060.: Rename \u0060Body.move\u0060 to something more specific, such as \u0060Body.UpdatePosition\u0060 or \u0060Body.ApplyMovement\u0060, to distinguish it from \u0060Snake.move\u0060. (symbols: Method: Snake.move(), Method: Body.move())"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e9eff1f90f33ad96c1c035f80e7e0db67d829f0f13bdb657abea4bad2d7e37d9"}},{"ruleId":"D21","level":"note","message":{"text":"Both \u0060Body\u0060 and \u0060Snake\u0060 have methods to set/change direction. \u0060Body\u0060 uses \u0060SetDirection\u0060 while \u0060Snake\u0060 uses \u0060ChangeDirection\u0060. This is an inconsistency in verb choice for the same concept.: Standardize on either \u0060SetDirection\u0060 or \u0060ChangeDirection\u0060 across both classes. (symbols: Method: Body.SetDirection(Direction), Method: Snake.ChangeDirection(Direction))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e6c26155f2a635d9aafcb221eb094858f91ddb7a5b38c927a4f0270e2fa060eb"}},{"ruleId":"D21","level":"note","message":{"text":"The \u0060Body\u0060 class uses \u0060getPosition\u0060 and \u0060setPosition\u0060 for a property named \u0060pos\u0060 or \u0060position\u0060. However, the parameter name in \u0060setPosition\u0060 is \u0060int\u0060, while \u0060getPosition\u0060 returns an int. This is consistent internally, but the method names \u0060getPosition\u0060/\u0060setPosition\u0060 are generic. More importantly, \u0060Snake\u0060 has \u0060GetPositions\u0060 (plural) while \u0060Body\u0060 has \u0060getPosition\u0060 (singular).: Ensure consistent naming for position access. If \u0060Body\u0060 represents a single point, \u0060Get/SetPosition\u0060 is fine. If \u0060Snake\u0060 represents a list of points, \u0060GetPositions\u0060 is fine. However, \u0060Body\u0060 also has \u0060move\u0060 and \u0060setPosition(int)\u0060, which suggests it might be handling position data. Consider if \u0060Body\u0060 should use \u0060Position\u0060 property with getter/setter or \u0060GetPosition\u0060/\u0060SetPosition\u0060 consistently. (symbols: Method: Body.getPosition(), Method: Body.setPosition(int))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"95e76a79f21c72f9d17a82c3e1351da9cbf28de0222bd9e8a1432d8a23fed53c"}},{"ruleId":"D24","level":"note","message":{"text":"redundant comment: \u0022UP=-1 DOWN=\u002B1 RIGHT=\u002B14 LEFT=-14\u0022 \u2014 trim - the constants restated; consider linking them to a enums/keys file"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Assets/Scripts/Tail.cs"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"dc4a7f1d815f1caf274c5ea3d33216926540742ace1d3d57a33c7cc063f30429"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1395","name":"Dependency on Vulnerable Third-Party Component"},{"id":"CWE-798","name":"Use of Hard-coded Credentials"}]}],"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}}}]}