{"$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":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"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":"D10","name":"Test Quality","shortDescription":{"text":"Test Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D10"},{"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"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"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":"D23","name":"Boundary Type-Coupling","shortDescription":{"text":"Boundary Type-Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D23"},{"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"]},{"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":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"id":"D39","name":"IL Efficiency","shortDescription":{"text":"IL Efficiency"},"helpUri":"https://codehealth.canine.dev/dimensions/D39"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"MethodCreatorTools.EmitCallParameter (cyclomatic 75): MethodCreatorTools.EmitCallParameter has cyclomatic complexity 75 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreatorTools.cs"},"region":{"startLine":157}}}],"partialFingerprints":{"codehealthFindingId/v1":"56ed40bf0a224ef1bec77eac6795271bc13f9de48853a1de69880bd2b0d0202a"}},{"ruleId":"D1","level":"warning","message":{"text":"InlineSignatureParser.ImportCallSite (cyclomatic 47): InlineSignatureParser.ImportCallSite has cyclomatic complexity 47 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/InlineSignatureParser.cs"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"aff497d0240365b699cadb31e1b70b6f38846abb47daf8875a6e849945c488dc"}},{"ruleId":"D1","level":"warning","message":{"text":"PatchTools.GetOriginalMethod (cyclomatic 44): PatchTools.GetOriginalMethod has cyclomatic complexity 44 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/PatchTools.cs"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"6fc79b0384f41915b921202e5eae74da34369e0ceec1274e563a49be4cf9b75f"}},{"ruleId":"D1","level":"warning","message":{"text":"MethodCreator.CreateReplacement (cyclomatic 35): MethodCreator.CreateReplacement has cyclomatic complexity 35 (threshold 15). Of this number, 30 points are the body\u0027s own statements and 5 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreator.cs"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"980626e7ad6babca6f8bf4481518815a270baa114a3bebe1eac1d8c3f0115b9c"}},{"ruleId":"D1","level":"warning","message":{"text":"AccessTools.MethodDelegate (cyclomatic 30): AccessTools.MethodDelegate has cyclomatic complexity 30 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1789}}}],"partialFingerprints":{"codehealthFindingId/v1":"30372a289cc0190b1a7719854237d37c93a3e4f24d0d5f9090350291f6513c10"}},{"ruleId":"D1","level":"warning","message":{"text":"MethodBodyReader.ReadOperand (cyclomatic 25): MethodBodyReader.ReadOperand has cyclomatic complexity 25 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCopier.cs"},"region":{"startLine":416}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a857c15cd0431a5f10fd5810d2c7455ac7a8d59c13b86ab0de5eccc2163ce61"}},{"ruleId":"D1","level":"warning","message":{"text":"CodeInstructionExtensions.LoadsConstant (cyclomatic 24): CodeInstructionExtensions.LoadsConstant has cyclomatic complexity 24 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":523}}}],"partialFingerprints":{"codehealthFindingId/v1":"14e80090d9e0a01b7cd0b813b730eec7e5bf40a3494a868f8304c5860e4c6f43"}},{"ruleId":"D1","level":"warning","message":{"text":"AccessTools.MakeDeepCopy (cyclomatic 23): AccessTools.MakeDeepCopy has cyclomatic complexity 23 (threshold 15). Of this number, 19 points are the body\u0027s own statements and 4 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":2154}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5eb6f05afc2d1a150851b312eb9e32b2fe0707cf578cbca053dcdaca0b8c652"}},{"ruleId":"D1","level":"warning","message":{"text":"CodeTranspiler.ShouldAddExceptionInfo (cyclomatic 21): CodeTranspiler.ShouldAddExceptionInfo has cyclomatic complexity 21 (threshold 15). Of this number, 13 points are the body\u0027s own statements and 8 belong to 4 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/CodeTranspiler.cs"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"fce9377ac47eeaf72941e7daf16a9b951fa4481e363782269e99b15beffeef07"}},{"ruleId":"D1","level":"warning","message":{"text":"CodeInstructionExtensions.IsLdarg (cyclomatic 19): CodeInstructionExtensions.IsLdarg has cyclomatic complexity 19 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":416}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b433452814f6d44cfeaa1ed03bbc7e11270b319e625d864823b4c4bfc09db1e"}},{"ruleId":"D1","level":"warning","message":{"text":"CodeInstructionExtensions.LocalIndex (cyclomatic 18): CodeInstructionExtensions.LocalIndex has cyclomatic complexity 18 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":621}}}],"partialFingerprints":{"codehealthFindingId/v1":"f33954323da0542bdfb6f9a2f394a9e115a81075e489dbc02d8d666844dfb7c6"}},{"ruleId":"D1","level":"warning","message":{"text":"MethodInvoker.GetHandler (cyclomatic 17): MethodInvoker.GetHandler has cyclomatic complexity 17 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Extras/MethodInvoker.cs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"6cc5df9e04b1fa138918519563a9f5bd6401dffa5f811daa858b74de9ee1ddc5"}},{"ruleId":"D1","level":"warning","message":{"text":"CodeMatch.Matches (cyclomatic 16): CodeMatch.Matches has cyclomatic complexity 16 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/CodeMatch.cs"},"region":{"startLine":133}}}],"partialFingerprints":{"codehealthFindingId/v1":"72a61efec848e6ac9cc89198368c3f87bcc73c86256f7cd65c1becf4c6c96bda"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodCreatorTools.EmitCallParameter (cognitive 216): MethodCreatorTools.EmitCallParameter has cognitive complexity 216 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreatorTools.cs"},"region":{"startLine":157}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b12bde55fb277fa3ec6bba08142ea990ed82322fb43ad919a9baaecb553de5e"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodCreator.CreateReplacement (cognitive 43): MethodCreator.CreateReplacement has cognitive complexity 43 (threshold 15). Of this number, 33 points are the body\u0027s own statements and 10 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreator.cs"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f08980e33c3e6993de369bbb5d19a2d9ee7a47abb718a003f6682b079844a9a"}},{"ruleId":"D2","level":"warning","message":{"text":"AccessTools.MethodDelegate (cognitive 40): AccessTools.MethodDelegate has cognitive complexity 40 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1789}}}],"partialFingerprints":{"codehealthFindingId/v1":"febc00306c18b8b2cfec0b4aebff8036decbd4f4dec0c1c1bd11d702487a0534"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeTranspiler.ShouldAddExceptionInfo (cognitive 38): CodeTranspiler.ShouldAddExceptionInfo has cognitive complexity 38 (threshold 15). Of this number, 18 points are the body\u0027s own statements and 20 belong to 4 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/CodeTranspiler.cs"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"36026d41cb7e18be9714af2435ddc4b709ef7ba6c18564a4e675cbc191672a71"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodInvoker.GetHandler (cognitive 36): MethodInvoker.GetHandler has cognitive complexity 36 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Extras/MethodInvoker.cs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"14a389f497588a221ffed022e1bd71c814767a3716ccc955f35ce90248d73ff8"}},{"ruleId":"D2","level":"warning","message":{"text":"AccessTools.MakeDeepCopy (cognitive 34): AccessTools.MakeDeepCopy has cognitive complexity 34 (threshold 15). Of this number, 30 points are the body\u0027s own statements and 4 belong to 2 function literals inside it that branch. To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":2154}}}],"partialFingerprints":{"codehealthFindingId/v1":"258ee82683835a63dbf59bdf2d94643c81f564521d75f143fd9aac831a524231"}},{"ruleId":"D2","level":"warning","message":{"text":"Dialog_FormCaravan_CheckForErrors_Patch.Transpiler (cognitive 23): Dialog_FormCaravan_CheckForErrors_Patch.Transpiler has cognitive complexity 23 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/patching-transpiler.cs"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e5e0ec87273d33c6c6fbb34d276c10ac20c747d4de72d7142b7ca4993dd002c"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodBodyReader.ReadOperand (cognitive 23): MethodBodyReader.ReadOperand has cognitive complexity 23 (threshold 15). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCopier.cs"},"region":{"startLine":416}}}],"partialFingerprints":{"codehealthFindingId/v1":"bac1fab2e34abc3896e627102df2af5f542c4e1731daf32effa9494c850165fd"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeInstructionExtensions.LoadsConstant (cognitive 22): CodeInstructionExtensions.LoadsConstant has cognitive complexity 22 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":523}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e058f3240e35f6ecd748749ea81641ebd2ca20f6693091e8be9ff09c4dd9787"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodCreator.AddPrefixes (cognitive 21): MethodCreator.AddPrefixes has cognitive complexity 21 (threshold 15). Of this number, 19 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreator.cs"},"region":{"startLine":219}}}],"partialFingerprints":{"codehealthFindingId/v1":"a034136a3fc097d7ae63520ed3b975a81dadae80063596c33589108d60aded63"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodCreator.AddPostfixes (cognitive 21): MethodCreator.AddPostfixes has cognitive complexity 21 (threshold 15). Of this number, 19 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreator.cs"},"region":{"startLine":280}}}],"partialFingerprints":{"codehealthFindingId/v1":"d94a840d44f18a2c5f97c451702fe4c8d0b5c76f719a633bcd13c9d06c953718"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeInstructionExtensions.LocalIndex (cognitive 21): CodeInstructionExtensions.LocalIndex has cognitive complexity 21 (threshold 15). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":621}}}],"partialFingerprints":{"codehealthFindingId/v1":"17c2d64517a77da9e232c976e0fcc1f4860b38dc5e5bbd2c52870ee04bb93d85"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodBodyReader.FinalizeILCodes (cognitive 18): MethodBodyReader.FinalizeILCodes has cognitive complexity 18 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCopier.cs"},"region":{"startLine":307}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bba3331f5a4e8b3ae238ace9ecf40dbcb86d1e0e290059c31f38d575145b184"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeInstructionExtensions.IsLdarg (cognitive 18): CodeInstructionExtensions.IsLdarg has cognitive complexity 18 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Extensions.cs"},"region":{"startLine":416}}}],"partialFingerprints":{"codehealthFindingId/v1":"02d77bb046fb0a951fd569b2dbe1f77a4c4e37e80563c15218ef76881d48dc88"}},{"ruleId":"D2","level":"warning","message":{"text":"Emitter.RestoreArgumentArray (cognitive 17): Emitter.RestoreArgumentArray has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/Emitter.cs"},"region":{"startLine":200}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed88d226bf15ee551bf8fa1cab9c4da7cb4fd14073d435b0c6c79eeeea370d98"}},{"ruleId":"D2","level":"warning","message":{"text":"MethodCreatorTools.RestoreArgumentArray (cognitive 17): MethodCreatorTools.RestoreArgumentArray has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCreatorTools.cs"},"region":{"startLine":492}}}],"partialFingerprints":{"codehealthFindingId/v1":"c43b4cd3289b7760618dee9183ddca06e2510171cc0816205e56a8e4b497135a"}},{"ruleId":"D2","level":"warning","message":{"text":"PatchClassProcessor.BulkPatch (cognitive 17): PatchClassProcessor.BulkPatch has cognitive complexity 17 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Public/PatchClassProcessor.cs"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e2d0fa7cff892829b6e5c095df005cd26c50320467bef8f639c7c6aece42303"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeMatch.Matches (cognitive 17): CodeMatch.Matches has cognitive complexity 17 (threshold 15). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/CodeMatch.cs"},"region":{"startLine":133}}}],"partialFingerprints":{"codehealthFindingId/v1":"03584a9d0e3d05612e1ca316c983318d54868707fa308ec3a43e6a75ddeb0f93"}},{"ruleId":"D2","level":"warning","message":{"text":"Code.ReplacementMethod (cognitive 16): Code.ReplacementMethod has cognitive complexity 16 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/execution_with.cs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"e1782080fd0bb43250b3877137fc97b1544bb8c66132f85e5c775a84d85e583e"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: AccessTools: TooManyMethods \u2014 831 significant lines (blank, comment-only and punctuation-only lines excluded), 118 methods. The type holds no instance state, so there is no shared data to group its members by. To reduce it, split it by area instead: give each cohesive family of members its own smaller type, so no one type has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8816f2225799c8a3868baa720d79e7ae822bd55a0a5f43cf6b46ca4d2cf4178"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: CodeMatcher: TooManyMethods \u2014 401 significant lines (blank, comment-only and punctuation-only lines excluded), 76 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":"Harmony/Tools/CodeMatcher.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"63b7831f2743c0f7aec768a29c32e98a7b4c598983264ce4eabd0182c891b0e5"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Tools/AccessTools.cs: FileTooLong \u2014 844 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"df37c93e8c88b1b0ebe37bec0b08fd56f42850cb66a3799cfe6107ffffb2d381"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Traverse: TooManyMethods \u2014 164 significant lines (blank, comment-only and punctuation-only lines excluded), 38 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":"Harmony/Tools/Traverse.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b046e2e7d25c9947d0a737244308e1cd4178216c65b9008590f5f38de64434e"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: CodeMatch: TooManyMethods \u2014 119 significant lines (blank, comment-only and punctuation-only lines excluded), 33 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":"Harmony/Tools/CodeMatch.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d482060dcbc13e2c2abd62cbfbc2d082f4a6d85204329de11882d6d6d98efcca"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): Harmony/Internal/MethodCopier.cs:562-577 | Harmony/Internal/MethodCopier.cs:584-599 \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 \u0060Harmony/Internal/MethodCopier.cs:562\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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/MethodCopier.cs"},"region":{"startLine":562}}}],"partialFingerprints":{"codehealthFindingId/v1":"150545c3f6f016877ee779cc21ec38bff5673f51c1e40a6481162821ba2e1aab"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 4): Harmony/Internal/CodeTranspiler.cs:68-76 | Harmony/Internal/CodeTranspiler.cs:85-93 | Harmony/Internal/CodeTranspiler.cs:106-114 | Harmony/Internal/CodeTranspiler.cs:123-131 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/CodeTranspiler.cs"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"e933f6e83c2e8c905e6818d29a014178f08693924f0b00efb158fcc28c2cb13c"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Lib.Harmony(net35): Lib.Harmony(net35): abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3e2da01c8af47a77392c755aa172bf40cc12d695d686b78ba0c50565f4a9d6fc"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Lib.Harmony.Ref: Lib.Harmony.Ref: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"430d64ca0f0b73ff66cd3a7b31903f1d5776c15aa1b79dbeb1bb59f7214972b9"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 no coverage collector is wired up: Coverage NOT MEASURED: the test suite built and its tests PASSED, but the run produced no coverage data \u2014 \u0060--collect:\u0022XPlat Code Coverage\u0022\u0060 found no data collector, which is what a test project with no \u0060coverlet.collector\u0060 PackageReference does. Nothing is wrong with the suite or the build; there is simply no coverage instrumentation wired up. Add a \u0060coverlet.collector\u0060 PackageReference to the test project(s) (or commit the Cobertura/OpenCover/lcov report your CI produces) and real coverage will be measured. It is excluded from the score rather than counted as a near-zero defect."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: Harmony/Tools/CodeMatch.cs: Harmony/Tools/CodeMatch.cs changed 2 times in last 90 days, max complexity 16. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/CodeMatch.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"23e991e0fc23b433e9808becdfddfd7aa143d10f394c731537028e8ec311c8ac"}},{"ruleId":"D17","level":"warning","message":{"text":"BareSuppressMessage repeated 64 times in 7 files: The identical BareSuppressMessage (\u0060SuppressMessage\u0060) appears 64 times in 7 files \u2014 one decision applied repeatedly (e.g. the same rule silenced on every overload of a fluent API), not 64 independent debts. Decide it once \u2014 at the type or project level, or by fixing the underlying pattern \u2014 rather than occurrence-by-occurrence. (Every occurrence still counts toward the score and metrics.)"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/GlobalSuppressions.cs"},"region":{"startLine":3}}}],"partialFingerprints":{"codehealthFindingId/v1":"128e7805b8331a828ad7d92da34c2b47fb366c692b3946f32b397f5f33354e2d"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/execution_with.cs"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c244aca1c17b32fe1ea91369796e9e32e2f6a05d754129c7aec9e12ed33ee71"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/execution_with.cs"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"4bdea0d82c4c65348ce21677d38d9c65cd34a5d2b7ae15bded80349f7673cf76"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Public/Harmony.cs"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"67b44f3fbd840e2bcb7c73179e812c7af4225adc45c49f06e8afa21361359295"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS9113 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/execution_with.cs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e50d3491e218c9fd14fc7fbac4193b36d697655f8ea75f0c744c7d6fd502fff"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS9113 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Tools/Assets/AccessToolsClass.cs"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f98119f75184ed9cce279a3a18f7974e1c4ec6aa383f34c74db6edda5f33434"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0649 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/patching-edgecases.cs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddda720ac655a2b118199e2665cf2e64c22a1372708feac97872782cd72471c5"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0649 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/examples/patching-transpiler.cs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc65692ac0740b798446f818975be7d7ac009a6919b84f58bb4721d6a0f5f7e9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: implement \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Internal/Infix.cs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd7073aa2279e6e357c44b6961dd8801b78b541179d08e99a1a99c01186024e3"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS1591 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1748}}}],"partialFingerprints":{"codehealthFindingId/v1":"8cd90768bd708055de6d80b1d2ad0bc5a4a137d481d4f9875f52b8705984f769"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS1591 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":1938}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b417f23198a81c3e323f43473c4cd43307495bb3a1dfa835139e776f9702077"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS1591 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/Code.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6936cabe0d9085b22182ea22df45568dceecadabbf4c4671c699f77babef807"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0060 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/AccessTools.cs"},"region":{"startLine":2338}}}],"partialFingerprints":{"codehealthFindingId/v1":"529a1d45ea4802fb3a59b8498bb306426827fa35e2e879f7e4c88fd5f258b596"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE1006 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Harmony/Tools/CodeMatch.cs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"0ea43195e4846c2243a371ba79c7299d2d4ad9beddf5b94677867ddbcc5b041f"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0067 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Patching/Assets/Specials.cs"},"region":{"startLine":336}}}],"partialFingerprints":{"codehealthFindingId/v1":"23150d06f318e1f7c5aaa8cc6e317c1eaa1ce2a8be83c88ad04d3bd474e0f84b"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0028 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Patching/GenericsPatches.cs"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"db6f3680d6e60ad782b84a66ca85410dbccb1c83877a797c817f6707b2de6bc1"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0028 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Patching/GenericsPatches.cs"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d27de4adf364ab62d64962de228abad9670b708ba1b0525130a7d7ba26c37b8"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0169, CS0414, IDE0044, IDE0051, IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Tools/Assets/AccessToolsClass.cs"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"0fe3576e3f240b6d10e5c0cf94838fd918c93db073af61c131e46df425afd9d2"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0414 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseProperties_AccessModifiers.cs"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"632fda40f0f3090e31424ac4efaac56fb56c651d048d179df0c9cbf82f7fb398"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0414 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"92f7f08a7ce0e30d6bbb23d190cb10a0caf53f11f344aeb723f3bee585c6338a"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0414 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"b1be8b657da939bde7ea152accb9c291fe662fc86d474424ab9caaa928118765"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0414 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"2c05b21777571f94f0cea8afb0ca3e3676c37b5997a4231d727efccadba17fce"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"0047530820b933323d6f32607a7824eedc681de0e931d032052ec636b67db607"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"c82be49f3c33d31f2f5322868342bb8b46e6b0fd30701ee94463f687ac1e45f4"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b5a13874bcef2c5cecfac0a408f3c0af4c4054d846d0eecfcd9d85460efbb98"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"f19d532209f659934b97d9db7f11c6d19f513fac565f299cb0dbeee94c158a4d"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable IDE0052 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/Assets/TraverseTypes.cs"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"e3c0c62dbfdbf601d0d9991d725461b708da9a90772b30d842149cbf0037e386"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable CS0169 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Traverse/TestTraverse_Basics.cs"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"53b10ba3476997653569e7a6fc43e8c8b6f26a5bb7f5ab609836687c5c47dbe4"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj: SYSLIB0011 \u2014 this warning is switched off for the WHOLE project, in every file it builds, including code written years from now: nothing at the call site records that the rule was ever silenced, so the next reader has no reason to look here. Fix what the rule is reporting and drop the code from the list, or \u2014 if some occurrences really are legitimate \u2014 narrow the suppression to those sites and give each one its reason, so the rule keeps protecting the rest of the project."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Build.props"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"e27c7ad75a133df3be78ce19d3d5fad66ffaea4184673bdc3ee42ab7436d6f4e"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj: NU5131 \u2014 this warning is switched off for the WHOLE project, in every file it builds, including code written years from now: nothing at the call site records that the rule was ever silenced, so the next reader has no reason to look here. Fix what the rule is reporting and drop the code from the list, or \u2014 if some occurrences really are legitimate \u2014 narrow the suppression to those sites and give each one its reason, so the rule keeps protecting the rest of the project."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Build.props"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"e0472f599130e5a076288858c9dd190791bd476a16ac8831fcdcf95db9b9fd34"}},{"ruleId":"D18","level":"note","message":{"text":"Thin analysable surface across projects: 1 project(s) carry only a thin slice of real code (e.g. \u0060TestLibrary(net35)\u0060 with 39 significant line(s)). The mean analysable-surface weight is 72 %, lowering Solution Shape by about 2.2 point(s). Consolidate thin projects or grow them into substantial, well-scoped assemblies."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dd0b92fd965c2065080f16843920964ee00f7696ea03d87cdf61aed4cd2c746a"}},{"ruleId":"D19","level":"warning","message":{"text":"Low XML-doc coverage: Documentation(net35): Documentation(net35): 0 % XML-doc coverage (0/94)."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Documentation/Documentation.csproj"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8685b324eafb485410fd79ecdedb1112987abf6021d70f05f59e70431d5d768a"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 15222 LoC spread over 63 projects the codebase is both large and multi-module, so explicit bounded contexts are warranted. Name this codebase\u0027s bounded contexts (\u22652 module groups, e.g. per subsystem) so cross-boundary type coupling can be assessed. Declare them in \u0060.codehealth/config.yaml\u0060 at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it \u2014 e.g. \u0060architecture:\u0060 \u2192 \u0060contexts:\u0060 \u2192 \u0060Billing: [\u0022src/billing\u0022, \u0022Acme.Billing\u0022]\u0060, \u0060Catalog: [\u0022src/catalog\u0022, \u0022Acme.Catalog\u0022]\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"1c7e276c9c682731f01819ed5378b90ca320cde1b583c90920172911598362b3"}},{"ruleId":"D24","level":"note","message":{"text":"redundant comment: \u0022ReSharper disable FieldCanBeMadeReadOnly.Global\u0022 \u2014 restates the de-obfuscation comment; remove"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HarmonyTests/Patching/Assets/Specials.cs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"89b1a70d2f1bb1a354444cd8ea5b529ef1b9578c16b1a68b59901e59ad1eccd6"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 3 of 6 project(s) overshoot their size bounds, lowering Project Cohesion to 0.0/10. The most over is \u0060Lib.Harmony(net35)\u0060 (13930 LoC, 294 public types across 1 namespaces). Review these for cohesion \u2014 draw the boundary inside the assembly first (group each responsibility into its own namespace/folder and keep the cross-boundary members internal), since splitting a published assembly moves types between packages and breaks consumers."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0d495ee619e44f936366740a050a9cf1cbb0105da01caf977eba29edb3551455"},"taxa":[{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e038d3df8fe6f6687370b7dc9f979218c2d8a234b0dd1993e989bad1162ce318"},"taxa":[{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ed6fdc226c4d3b560a6765bd753130fbd1dbb00e11437de41604f04caeeea5a7"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"055602a3a1cdc5b9e89218f502c9e61eeade13f5e070e071043a3f93072d5055"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"387f64c82ea22dbc380238c96ae13cd13defae0ef1588cbc5c640b357fbc182d"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"9658270ba49f37ed04e99ab1263b6393cd42aed8bdfb7aafd9fa1070f5491895"}}],"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-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"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-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.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":10,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}