{"$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":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"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":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"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":"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"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX5","name":"Architecture \u0026 structure","shortDescription":{"text":"Architecture \u0026 structure"},"helpUri":"https://codehealth.canine.dev/dimensions/AX5"},{"id":"AX6","name":"Interface segregation","shortDescription":{"text":"Interface segregation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX6"},{"id":"ED5","name":"Idempotency","shortDescription":{"text":"Idempotency"},"helpUri":"https://codehealth.canine.dev/dimensions/ED5"},{"id":"GD1","name":"Unfinished \u0026 placeholder code","shortDescription":{"text":"Unfinished \u0026 placeholder code"},"helpUri":"https://codehealth.canine.dev/dimensions/GD1"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"GameStatsJar.PackDataValue (cyclomatic 18): GameStatsJar.PackDataValue has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Protocol/GameStatsJar.vb"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"dce90220257ae201c895e95cfe811d6e22593f104d175f31dee94a5664f84c87"}},{"ruleId":"D1","level":"warning","message":{"text":"GameStats.Equals (cyclomatic 18): GameStats.Equals has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3GameStats.vb"},"region":{"startLine":264}}}],"partialFingerprints":{"codehealthFindingId/v1":"67cde3e54b6757904252fe7f0fa7db8075ed0846f13a7a5ce31db0c49550d92e"}},{"ruleId":"D1","level":"warning","message":{"text":"TestingCommon.ObjectEqual (cyclomatic 18): TestingCommon.ObjectEqual has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"HostBotTests/Utilities/Common.vb"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"2588dc29e8f88eeeb3b36780519281c9610e296c58c0cb0db693fdfd15d757ad"}},{"ruleId":"D1","level":"warning","message":{"text":"GameLobby.OnPlayerSetTeam (cyclomatic 17): GameLobby.OnPlayerSetTeam has cyclomatic complexity 17 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":438}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7adceb69a880923ce360c66c0bef712e512725ffd1b901a4162981c021d622e"}},{"ruleId":"D2","level":"warning","message":{"text":"TestingCommon.ObjectEqual (cognitive 30): TestingCommon.ObjectEqual has cognitive complexity 30 (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":"HostBotTests/Utilities/Common.vb"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"b786f49dd457dfa3a99f45e53690d783bb8ebcbfd9dc1354be6e72006f2f9eaa"}},{"ruleId":"D2","level":"warning","message":{"text":"GameLobby.OnPlayerSetTeam (cognitive 26): GameLobby.OnPlayerSetTeam has cognitive complexity 26 (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":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":438}}}],"partialFingerprints":{"codehealthFindingId/v1":"e29dc1b92369037168c74c96f8c6a3bf227a762fd07765b918ab21a45869cc90"}},{"ruleId":"D2","level":"warning","message":{"text":"CommandArgument.New.ctor (cognitive 26): CommandArgument.New.ctor has cognitive complexity 26 (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":"Commands/CommandArgument.vb"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"5feabaac72aa59127b0e8953fc3fac017ec98b22f38ac65ea1491076b3558196"}},{"ruleId":"D2","level":"warning","message":{"text":"CommandArgument.Tokenize (cognitive 26): CommandArgument.Tokenize has cognitive complexity 26 (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":"Commands/CommandArgument.vb"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"b26578f27d77679e5bfd39aeb54ff9fe0a25de0e55dcd3d1cc1657d552e9997f"}},{"ruleId":"D2","level":"warning","message":{"text":"GameLobby.New.ctor (cognitive 23): GameLobby.New.ctor 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":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"3be59c05d85563f78277d08db7f66a7659840485fd0a502d3212fe2af5b4a15c"}},{"ruleId":"D2","level":"warning","message":{"text":"GameLoadScreen.OnReceiveReady (cognitive 23): GameLoadScreen.OnReceiveReady 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":"Warcraft3/Game/Game Load Screen.vb"},"region":{"startLine":118}}}],"partialFingerprints":{"codehealthFindingId/v1":"06495ce2b464112a73e715e55787a7753b09b1db420a5cffa8cf05eafc830c57"}},{"ruleId":"D2","level":"warning","message":{"text":"CommandTemplate.TryFindMismatch (cognitive 22): CommandTemplate.TryFindMismatch has cognitive complexity 22 (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":"Commands/CommandTemplate.vb"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b61e4530e78111218fe858005bb5611914b4eee05216e90f2f87080cd2972aa"}},{"ruleId":"D2","level":"warning","message":{"text":"PoorlyCategorizedFunctions.LayoutPanel (cognitive 22): PoorlyCategorizedFunctions.LayoutPanel has cognitive complexity 22 (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":"Library/Common.vb"},"region":{"startLine":511}}}],"partialFingerprints":{"codehealthFindingId/v1":"af37e3a7029a7d9dd2043dd4dbf580f2d041068516817cf6615b4530ec461bc8"}},{"ruleId":"D2","level":"warning","message":{"text":"PoorlyCategorizedFunctions.SplitText (cognitive 20): PoorlyCategorizedFunctions.SplitText has cognitive complexity 20 (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":"Library/Common.vb"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"e47d6813262a4a3aa79ad8e57ec370ff4236bedb786c8ad3c3fca16278401e8c"}},{"ruleId":"D2","level":"warning","message":{"text":"SettingsForm.btnSave_Click (cognitive 20): SettingsForm.btnSave_Click has cognitive complexity 20 (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":"Interface/SettingsForm.vb"},"region":{"startLine":148}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4346d280d3226c540708200072b2b81e15ca4d520d3349f18a7548351c55230"}},{"ruleId":"D2","level":"warning","message":{"text":"GameMotor.UpdateLagScreen (cognitive 19): GameMotor.UpdateLagScreen has cognitive complexity 19 (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":"Warcraft3/Game/Game Motor.vb"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0637fd18564a40c325ffd688c01f3b713938e11ac05c0cb5ca821d89c685bf9"}},{"ruleId":"D2","level":"warning","message":{"text":"GameStats.Equals (cognitive 18): GameStats.Equals has cognitive complexity 18 (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":"Warcraft3/W3GameStats.vb"},"region":{"startLine":264}}}],"partialFingerprints":{"codehealthFindingId/v1":"54ae29e182536153d45b4fd33b5d20bf59710b202be0e95aedebde838965ac92"}},{"ruleId":"D2","level":"warning","message":{"text":"Manager.ClientLatencyDescription.Get (cognitive 17): Manager.ClientLatencyDescription.Get 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":"Warcraft3/Download/Download Manager.vb"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"b794d05b1a78d79919f83c17956d5d04d416a5ff719e6f660d2e1d246489edc1"}},{"ruleId":"D2","level":"warning","message":{"text":"LoggerControl.EmptyQueue (cognitive 17): LoggerControl.EmptyQueue 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":"Interface/LoggerControl.vb"},"region":{"startLine":209}}}],"partialFingerprints":{"codehealthFindingId/v1":"7cbd8667a3d51a5969254da894668e6afdca6d388186019b946886eb60bb4db8"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: GameActionProtocolTest: TooManyMethods \u2014 639 significant lines (blank, comment-only and punctuation-only lines excluded), 61 methods. The bar is 30 methods; this is 31 over it, 2.03\u00D7 the bar. 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":"HostBotTests/GameActionProtocolTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ab0ee9aea0f2ef524f66a18d7b5471b1f00fa798f5345148cf4f4aa7766ef52"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Game: TooManyMethods \u2014 614 significant lines (blank, comment-only and punctuation-only lines excluded), 50 methods. The bar is 30 methods; this is 20 over it, 1.67\u00D7 the bar. 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":"Warcraft3/Game/W3Game.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e6d28b6934248be3debce0a0a0aeefec9ae5b2adf65e22838229776ad68b631"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: Client: ClassTooLong \u2014 573 significant lines (blank, comment-only and punctuation-only lines excluded), 26 methods. The bar is 400 significant lines; this is 173 over it, 1.43\u00D7 the bar. 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":"Bnet/Bnet Client.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddf70352430a0449b79c0f3165ea33472aeaca86ba240b390852eed729a61648"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: PoorlyCategorizedFunctions: TooManyMethods \u2014 535 significant lines (blank, comment-only and punctuation-only lines excluded), 41 methods. The bar is 30 methods; this is 11 over it, 1.37\u00D7 the bar. 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":"Library/Common.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0efdeed5d7b748114630b0806b12d91583a0e56ea1fcc4bcbab20ed9637fe21"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: GameLobby: TooManyMethods \u2014 633 significant lines (blank, comment-only and punctuation-only lines excluded), 40 methods. The bar is 30 methods; this is 10 over it, 1.33\u00D7 the bar. 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":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5de999eba62d190dd37b9f529a45704efb5236f41ac9813c5061faeb2a9ca027"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WC3ProtocolTest: TooManyMethods \u2014 486 significant lines (blank, comment-only and punctuation-only lines excluded), 40 methods. The bar is 30 methods; this is 10 over it, 1.33\u00D7 the bar. 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":"HostBotTests/WC3ProtocolTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"047003cefbeeb0309cc65000b5c35533c53a70e5f18951c7d3ea60d54f5627d9"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: HostBotTests/GameActionProtocolTest.vb: FileTooLong \u2014 649 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 149 over it, 1.30\u00D7 the bar. 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":"HostBotTests/GameActionProtocolTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1e895660b4bd8b6895a7eb87bd9145c4641eae92087c6d67db3cc5e41d109ad3"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Game/Game Lobby.vb: FileTooLong \u2014 635 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 135 over it, 1.27\u00D7 the bar. 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":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"34b15d775919f6f5b4dcc6f6eaccd2e406ba096cfeb5e5f404993a0fe451ddce"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Game/W3Game.vb: FileTooLong \u2014 616 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 116 over it, 1.23\u00D7 the bar. 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":"Warcraft3/Game/W3Game.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5d535dba37c33162d10b482fa3563a53395bb2456579a4e55149869d3c82f0c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Bnet/Bnet Client.vb: FileTooLong \u2014 586 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 86 over it, 1.17\u00D7 the bar. 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":"Bnet/Bnet Client.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"352badd582290b32c5629ffe95050d39871d21d7fe95a6617a0ee634dc8eb947"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BnetProtocolTest: TooManyMethods \u2014 394 significant lines (blank, comment-only and punctuation-only lines excluded), 33 methods. The bar is 30 methods; this is 3 over it, 1.10\u00D7 the bar. 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":"HostBotTests/BnetProtocolTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"707e5487c3d384ce6e895b00d2ca519d1b3037ef9736ee994a8c3bbcc79ab89a"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Packers: TooManyMethods \u2014 338 significant lines (blank, comment-only and punctuation-only lines excluded), 33 methods. The bar is 30 methods; this is 3 over it, 1.10\u00D7 the bar. 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":"Warcraft3/Protocol/WC3 Protocol Packers.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d411e9460147ecd87221d737fc583673fb2fe356f21dc1c5b9ee4d8150dfdac9"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: GameCommands: ClassTooLong \u2014 440 significant lines (blank, comment-only and punctuation-only lines excluded), 7 methods. The bar is 400 significant lines; this is 40 over it, 1.10\u00D7 the bar. 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":"Warcraft3/Game/Game Commands.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cb376aa0e03d2cee1297de6e7a6255ab9f764c58c2d76d1da0c906128123e740"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Library/Common.vb: FileTooLong \u2014 536 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 36 over it, 1.07\u00D7 the bar. 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":"Library/Common.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd22c0a692810727f0a306effecee9ef1d0743cbef36ba82055d6c282ff5803f"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Player: TooManyMethods \u2014 421 significant lines (blank, comment-only and punctuation-only lines excluded), 32 methods. The bar is 30 methods; this is 2 over it, 1.07\u00D7 the bar. 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":"Warcraft3/Player/W3Player.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3782b16daea0fc838d647e263a0d1010ca82cbc99c001c2a005d8b4f3e56af7"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WC3ProtocolPackersTest: TooManyMethods \u2014 143 significant lines (blank, comment-only and punctuation-only lines excluded), 32 methods. The bar is 30 methods; this is 2 over it, 1.07\u00D7 the bar. 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":"HostBotTests/WC3ProtocolPackersTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c1e5c6891297f9549e218edf0dadd74b15b399dd93e6401a595cbe7ddfe9184"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: PicklingTest: ClassTooLong \u2014 407 significant lines (blank, comment-only and punctuation-only lines excluded), 23 methods. The bar is 400 significant lines; this is 7 over it, 1.02\u00D7 the bar. 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":"HostBotTests/PicklingTest.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"65f581b026cbf02951c7ad413b713f2d3280567be42350533da9ceabc8b76e42"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: GameSettings: ClassTooLong \u2014 401 significant lines (blank, comment-only and punctuation-only lines excluded), 11 methods. The bar is 400 significant lines; this is 1 over it, 1.00\u00D7 the bar. 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":"Warcraft3/Server/GameSettings.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b56a4d5caa3afae4bafe3fab76b47dd27b0a3ab3f56b15d4786c9f80bb14c18e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24 lines \u00D7 2): Bot/MainBot.vb:145-168 | Bot/MainBot.vb:177-200 \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 \u0060Bot/MainBot.vb:177\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":"Bot/MainBot.vb"},"region":{"startLine":145}}}],"partialFingerprints":{"codehealthFindingId/v1":"2067dc414d1f13d26e733e9017436959cdc319a89e149823fb3347e82a6c0a0d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 3): Warcraft3/Game/Game Commands.vb:27-37 | Warcraft3/Game/Game Commands.vb:40-65 | Warcraft3/Game/Game Commands.vb:68-80 \u2014 all 3 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":"Warcraft3/Game/Game Commands.vb"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad6456fcde428c513fd4a00b030489099918e652ab6d6fe8e1b7b6d9296f8ebd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): Library/Stream Extensions.vb:102-112 | Library/Stream Extensions.vb:120-130 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Library/Stream Extensions.vb"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"4383f1853e64a2a587231f0d4ab9eb1462bf000890c4d91012d7018e894eb2a3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): Warcraft3/W3Map.vb:575-584 | Warcraft3/W3Map.vb:585-594 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3Map.vb"},"region":{"startLine":575}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea04cae362fba986e3c33080ae19c98bf783b864dc683532c89fd98cb077b2e3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): Warcraft3/W3GameDescription.vb:224-230 | Warcraft3/W3GameDescription.vb:331-337 \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 \u0060Warcraft3/W3GameDescription.vb:224\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":"Warcraft3/W3GameDescription.vb"},"region":{"startLine":224}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7301d9e54f3ac245ef7a0cdbcff314e42fe26ed7b65e58f6db62bd3685fcd12"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 3): Warcraft3/W3GameDescription.vb:63-68 | Warcraft3/W3GameDescription.vb:216-221 | Warcraft3/W3GameDescription.vb:323-328 \u2014 all 3 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Warcraft3/W3GameDescription.vb:63\u0060 it does not close everything it opens, 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. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3GameDescription.vb"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a0f837239ed353f98c9937ba9769f8fb5882488d2532193e5bee56a4f8dd96c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Bnet/ClientCredentials.vb:167-172 | Bnet/ClientCredentials.vb:185-190 \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 \u0060Bnet/ClientCredentials.vb:167\u0060 it does not close everything it opens, 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":"Bnet/ClientCredentials.vb"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"e5c0d6d10155e504f645dd026896cf8f185e52b62e9a9a53f4007702fde94edc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Pickling/Pickling Base.vb:122-127 | Warcraft3/Protocol/ObjectTypeJar.vb:44-49 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Pickling/Pickling Base.vb"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e47fc0c395c18f3ca083b4f7b22e43e5cc1d6ff7b8a001cfcc1eef69679856e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): Warcraft3/Server/GameServer.vb:241-245 | Warcraft3/Server/GameSet.vb:173-177 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Warcraft3/Server/GameServer.vb:241\u0060 it does not close everything it opens, 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. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Server/GameServer.vb"},"region":{"startLine":241}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ba9b473c4456504b59d4acc4f37f816f1df4c7e3deb8317b16ea3c709a49892"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Tinker: Tinker: abstractness 0.16, instability 0.00, distance 0.84 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d952fd23866edc097358dceff2b877cd1125b2408df4973e60ba18143ca80e72"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ContractClass (LCOM4 4): ContractClass\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Download/IPlayerDownloadAspect.vb"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"e454ce1abd71b970613ce807c69ec67d76133479c61e1f4630c0153e2bdc5636"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 no coverage collector is wired up: Coverage NOT MEASURED: \u0060--collect:\u0022XPlat Code Coverage\u0022\u0060 names a data collector that ships in the \u0060coverlet.collector\u0060 package, and this repository wires up none \u2014 no test project references it and no runsettings declares one. The absence of coverage here is therefore not evidence about the suite or about our analyzer environment: without a collector, \u0060--collect\u0060 produces nothing even from a suite that builds and passes. 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":"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":"Bot/MainBot.vb"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"1502b778dc27bee17048bf3a73708044b3ecca8594d9fdb08742c8a40b3a6708"}},{"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":"HostBotTests/Utilities/Common.vb"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e1d2e0a1ae6597a976b8a9f45d2942e7d013e8587c7d0004974d5c3140a255e"}},{"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":"Library/Common Async.vb"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"b144ac005964cfc4b8dcf49ba1ed86fa68be3165401e4cd44e9aec1bd50d5c88"}},{"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":"Warcraft3/Player/W3Player.vb"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"67bb1e80c82282119df4148f9493d8327a49b52ab965e1b2544ad3792f9ce8c3"}},{"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":"Warcraft3/W3DummyPlayer.vb"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"c93b39ede92a0b25b2c615437792fa8050b505a40230b6bd0317c335643565fc"}},{"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":"Warcraft3/W3Relay.vb"},"region":{"startLine":185}}}],"partialFingerprints":{"codehealthFindingId/v1":"14926135c14168827cc4f23e3b1c90aae6de79f33786a744a8fe44fc0ffdb8c1"}},{"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":"Warcraft3/W3RemotePlayer.vb"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"7003aaf93ca88fb44d2353695bf822fa4391d262b87144ed0797140435a9b367"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027todo: check that these are correct for non-square sizes \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3Map.vb"},"region":{"startLine":285}}}],"partialFingerprints":{"codehealthFindingId/v1":"4af16d7f77c3ceb321bdd02f046618741517e8c5beddd348a79709e5ed16e1ee"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027todo: distinguish blizzard maps from user maps \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3Map.vb"},"region":{"startLine":309}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5212140b151ec3e8cd040958d4287085d45e1683e1776f9e9773b904affb7d4"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no installation or build instructions: No build/run/install steps for Tinker. Add a short setup guide: how to clone the repo, install dependencies (Python/PyO3), and run the bot."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fa6bfcd5c6ae231d1b8ee99bced50b832d1f61cde9da6cbb533467b879a57db"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no usage examples: No runnable example showing how to start or connect to BNET. Write a one-line usage snippet demonstrating login and lobby creation, with a link to the full API reference."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1355eb4e127a4bc9236772ce1c46f09510aa9286e7a06d1e47f308229df19049"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 1 of 2 project(s) overshoot their size bounds, lowering Project Cohesion to 0.0/10. The most over is \u0060Tinker\u0060 (23625 LoC, 373 public types across 20 namespaces). Review these for cohesion \u2014 split a project that spans unrelated responsibilities."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"ruleId":"D28","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":"d4b9b69f9e4d474b5a5d99630c9cd10b674fee3fbc94f165e5bfa200eedb900d"},"properties":{"commitSha":"922801837cb3ed7afd473c253b924f095961cef0"}},{"ruleId":"D28","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":"3cdfd7beb31b791ae18dcb425e21eb157c842e7bbcd522cc330573c1a6538db1"}},{"ruleId":"D34","level":"note","message":{"text":"Dormant codebase: 126 of 126 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 126 separate risks. Counted over 126 of the 152 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Game/Game Lobby.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f6cdc573d3e0d7b28c79942a30bdb2f929251d99816cb4f188d41e8decd450d5"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/Game/W3Game.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"77c2cc45e520dc5af47ded568cf459e9573f36c743a2a90b3d45400e14f27e7e"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Warcraft3/W3Map.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0102f8542b0e6f0dd79006c2858c7b80dddcbb45bbd36b36c63ea1cbcd7be315"}},{"ruleId":"AX5","level":"warning","message":{"text":"Appears unstructured (no recognised architecture): ~24.1k production LoC (488 types) across 2 project(s) / 24 namespace(s) with no recognised architectural style or layering \u2014 at this size, deliberate module boundaries would help. No DDD/clean/vertical-slice/CQRS/transaction-script organisation was detected \u2014 code is likely organised ad hoc, which makes change and onboarding harder as it grows."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"cf38333cec692da9187608fc8d12d547460b1d3964d6b4da78c9c51e859dd969"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M2","level":"note","message":{"text":"No architecture diagram/doc: No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named \u0060architecture\u0060 or \u0060design\u0060 in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape \u2014 no \u0060## Architecture\u0060 heading, no \u0022architecture overview\u0022/\u0022high-level design\u0022 phrasing, no \u0022the architecture is \u2026\u0022 introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document \u2014 not evidence that nobody wrote the shape down."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0c190e7c159d1850ee706c3ac4486e151e8a4fe169de4bf61436b9f399654f06"}},{"ruleId":"M3","level":"note","message":{"text":"No src/ separation: Production code isn\u0027t grouped under a src/ folder \u2014 it\u0027s spread across several top-level directories, so there\u0027s no one place that says \u0027this is the product\u0027."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fdf7f5b24e313364d10170a5c2542959902fe0d26ed70edef3eaa10ec5bcdb1d"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: README claims infinite lobbies but the evidence shows no lobby subsystem \u2014 searched for: \u0060infinite lobbies\u0060. Each was matched case- and separator-insensitively against file and directory NAMES anywhere in the tree, and against the CONTENTS of manifest files (package.json, *.csproj, *.props, *.slnx, *.yml, Dockerfile); the README\u0027s own prose never counts, so a claim is never refuted by merely being made. Nothing outside that search was read \u2014 a footprint living only in a submodule, in a file type not listed here, or under a name none of those terms matches is not seen, and this row is then wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d0971213e81929c69638e74c731450d3a7722934ee7af2c1c4aa4ee00c08b6a8"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: claims HCL game modes but there is no HCL project in the evidence \u2014 searched for: \u0060HCL game modes\u0060. Each was matched case- and separator-insensitively against file and directory NAMES anywhere in the tree, and against the CONTENTS of manifest files (package.json, *.csproj, *.props, *.slnx, *.yml, Dockerfile); the README\u0027s own prose never counts, so a claim is never refuted by merely being made. Nothing outside that search was read \u2014 a footprint living only in a submodule, in a file type not listed here, or under a name none of those terms matches is not seen, and this row is then wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"1c3b5869fe8ba2385bf8c75e0d2beb7d3b64b1be0cd30da42586137bdb800c61"}},{"ruleId":"P1","level":"warning","message":{"text":"No CI pipeline: No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, \u2026) \u2014 changes aren\u0027t gated by an automated build/test."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"44f01af96e50474fba2df84d95ddf4f7e1d34c29c307830b9efc9057daa6b670"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add a Roslyn security analyzer package referenced from the project (the analyzer packages do analyse VB.NET), plus gitleaks for committed secrets \u2014 CodeQL has no VB.NET extractor, so its csharp pack would extract nothing from this tree and then report it clean \u2014 this repository has no CI pipeline yet, so run it locally to clear the existing findings, then make it a step of the first workflow you add so a regression fails the build. What was searched, so you can tell an absence from a miss: the 0 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}},{"ruleId":"P6","level":"note","message":{"text":"No changelog: No CHANGELOG/HISTORY/RELEASES file \u2014 what shipped when isn\u0027t easy to reconstruct for support or audit. (Versioning/tagging makes releases traceable, but a changelog records the what.)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dda5aa5aed8cbb292c3ef2b733bc138f614293ae6426c85e98bf31ef330d9415"}}],"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":2,"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."]}]}