{"$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":"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":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"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":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D18","name":"Solution Shape","shortDescription":{"text":"Solution Shape"},"helpUri":"https://codehealth.canine.dev/dimensions/D18"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"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":"D23","name":"Boundary Type-Coupling","shortDescription":{"text":"Boundary Type-Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D23"},{"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"}]}},"results":[{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (35 lines \u00D7 2): ViperService/Program.cs:20-54 | Samples/HelloWorldService/Program.cs:20-54 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060ViperService/Program.cs:20\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":"ViperService/Program.cs"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"449a7375c0435ff811b02aef16416839d1b58e3e8b776aaba39a1ccab8bfe009"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): Trace/Anno.Model/BaseModel/Entity.cs:37-57 | Trace/Anno.Domain/BaseModel/Entity.cs:36-56 \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":"Trace/Anno.Model/BaseModel/Entity.cs"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"34ea76e33aa676b2abc9ce891e42b41e44574be345691b6451bd3a8caa6fc5c3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): Anno.Plugs.TraceService/TraceModule.cs:40-56 | Anno.Plugs.TraceService/TraceModule.cs:111-127 \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 \u0060Anno.Plugs.TraceService/TraceModule.cs:40\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":"Anno.Plugs.TraceService/TraceModule.cs"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b9a813744e9afa2d719954da8a8c46edb84f85f77d658d1484dad3bcc4160cc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): Trace/Anno.Model/BaseModel/Entity.cs:19-33 | Trace/Anno.Domain/BaseModel/Entity.cs:18-32 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Trace/Anno.Model/BaseModel/Entity.cs:19\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":"Trace/Anno.Model/BaseModel/Entity.cs"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"86e1d05da178caf1e268ce4d5cf87a11d1416b68d09021c236d1ea794ff91a66"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 3): Trace/Anno.Model/BaseModel/Entity.cs:174-185 | Trace/Anno.Domain.Dto/BaseDto.cs:95-106 | Trace/Anno.Domain/BaseModel/Entity.cs:173-184 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made 3 times. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Trace/Anno.Model/BaseModel/Entity.cs:174\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":"Trace/Anno.Model/BaseModel/Entity.cs"},"region":{"startLine":174}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff1f0a68d7e75cd0af247e96b7b86f5c8e2276b9f0743eff50f7698e4078b7ef"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): Viper/Extensions/Utils/UtilService.cs:128-139 | Anno.Plugs.TraceService/RoutingInfomationDto.cs:48-59 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Viper/Extensions/Utils/UtilService.cs:128\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":"Viper/Extensions/Utils/UtilService.cs"},"region":{"startLine":128}}}],"partialFingerprints":{"codehealthFindingId/v1":"d25b7b936b3d56f549e9e07b3bb0fefc9699a2a5876873d7c15196dca15b2ea0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): Viper/Extensions/Utils/UtilService.cs:150-161 | Anno.Plugs.TraceService/RoutingInfomationDto.cs:70-81 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Viper/Extensions/Utils/UtilService.cs:150\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":"Viper/Extensions/Utils/UtilService.cs"},"region":{"startLine":150}}}],"partialFingerprints":{"codehealthFindingId/v1":"c57fe63147d9152c2eb31f268bcbc032ed4191b122bdbeb56fd81c1be93a70ff"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): Trace/Anno.Model/BaseModel/Entity.cs:102-109 | Trace/Anno.Domain.Dto/BaseDto.cs:23-30 | Trace/Anno.Domain/BaseModel/Entity.cs:101-108 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made 3 times. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Trace/Anno.Model/BaseModel/Entity.cs:102\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":"Trace/Anno.Model/BaseModel/Entity.cs"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"ffe5da1b37bf7362af1151eb906c06f104e7198d2a851e56f9ae5dcd94533b8d"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Plugs.ViperService: Anno.Plugs.ViperService: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b3e431cf2662a4f54861bb4665fb5713fa4682521d1101e419195f18b289349d"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Infrastructure: Anno.Infrastructure: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 4 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"9e0c4113e7cde97e49d602e32378fb97f3232e246f8df5ad991c8bb6e4ca46fb"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Common: Anno.Common: abstractness 0.00, instability 0.00, distance 1.00 \u2014 the shape a shared-kernel / building-block library has BY DESIGN \u2014 concrete and widely depended-on is what makes it useful, and this dimension does not penalise it (the distance is reported for completeness, not as a defect). Worth a look only if it has grown past one coherent kernel into an everything-bucket."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"5b7d33e621c9d3f6a1ed6bfdd0cb87500185b019ebdcd20f84ae514b32bf85e8"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Domain.Dto: Anno.Domain.Dto: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 6 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"789f37c7f6a212f8ac8b87e521c2e05230c4b4b32ed796be9834a875a7f520fb"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Redis: Anno.Redis: 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":"3b9bfb7fbf7e896be03ee4eec2b60079fbc3eef9696d91445c89457a01f6306f"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Plugs.MonitorService(netstandard2.0): Anno.Plugs.MonitorService(netstandard2.0): abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"09cfe0689bab7224243b2e042c973a0d71c636eb4bd3bf47e5b3762e8ad7be4a"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Plugs.DLockService: Anno.Plugs.DLockService: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"55d3b0fd0ca885ef409d425bc716284b64e677cced6cf1afe20efed58d0be44a"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Anno.Command: Anno.Command: abstractness 0.00, instability 0.29, distance 0.71 \u2014 zone of pain \u2014 concrete and depended on by 5 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"37680645dcbbc3131de32688cfb8f84f251c2ec5cc5fe5871d21cdc19c925549"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: PlatformModule (LCOM4 16): PlatformModule\u0027s methods form 16 groups that share no state and don\u0027t call each other \u2014 a sign it may have several responsibilities. Review whether it splits into focused classes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Anno.Plugs.LogicService/PlatformModule.cs"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"acea09a59024923e487b6f6844218c09510ee225e1287c101be5118efa0fff89"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SysMember (LCOM4 4): SysMember\u0027s methods form 4 groups that share no state and don\u0027t call each other \u2014 a sign it may have several responsibilities. Review whether it splits into focused classes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Trace/Anno.Domain/Member/SysMember.cs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"67695c4f6d6dd30f8498aafa5c16738f4258f6a3da527d73115adf7a62dfb6af"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 test suite did not build: Coverage NOT MEASURED: this repository did not build in our analyzer environment (a C#/MSBuild compiler error), so no coverage could be collected. It is excluded from the score rather than counted as a near-zero defect. We did not read WHERE the failing diagnostic is, so this does not claim the fault is in your test code \u2014 a repository written for an older SDK band can compile for you and not for us. Run \u0060dotnet build\u0060 on this commit; if it succeeds, the gap is ours. Committing the Cobertura/OpenCover/lcov report your CI already produces also lets us measure real coverage without building anything."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test suite could be collected \u2014 nothing here references a test framework (xUnit, NUnit or MSTest), so there were no discoverable tests to count. Tests written as plain executables or shell/PowerShell harnesses are not collectible this way and are not scored here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c9bf64cbb5a4ae13d6bcd01fa3bc8d2879d860c73bc67f176ee3c2cecb8adce3"}},{"ruleId":"D11","level":"warning","message":{"text":"Test reliability not measured \u2014 no test run produced results: Test reliability NOT MEASURED: the test run produced no results for any test tier, so no test ever ran and flakiness could not be exercised. The cause could not be attributed, so it is excluded from the score rather than read as an absence of tests."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"7d83a505f7fa19aeeece76141ae1e468d77e2a3fea2e88604b8140ebad47a6fe"}},{"ruleId":"D16","level":"warning","message":{"text":"dormant codebase \u2014 no living knowledge left to concentrate: All 15 significant source file(s) were last meaningfully changed so long ago that no living knowledge remains \u2014 nothing since has been substantial enough to re-establish ownership (a broad, mechanical sweep that touches many files shallowly does not count, and neither does no activity at all). There is no concentration to measure, so the bus factor is not scored. This is not a clean bill: nobody currently holds working knowledge of this code (see D34 Knowledge Freshness)."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"569d55ea5cb0330f13c125b289d07e5abdfcceac947a2c2645277937942e7d5f"}},{"ruleId":"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":"Anno.Common/AnnoMapper.cs"},"region":{"startLine":248}}}],"partialFingerprints":{"codehealthFindingId/v1":"85cfcbc69310f3805ca89b7f8ef7aefabe4b15e037adb0b192934dbfd9c882c5"}},{"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":"Anno.Plugs.TraceService/RouterModule.cs"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b0ac5624179b022c330656204770a5c382ac2f7581906a86cf1a6f34b67e2f9"}},{"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":"Viper/Extensions/Utils/UtilService.cs"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"3846b391f7aca02cc7c6cb5619aa5ca2d243b9769f717019e56750bebb12d351"}},{"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":"Viper/Hubs/TaskManager.cs"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"07b78a051de9b4076c69ff7f68c97de886470d12e189aa9701fad5e533799153"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 6 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Anno.Common/CryptoHelper.cs"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa2c35967e63c79c5dad1ac2791161f2961dd1bf6fb3fa26b70c345602aaa8a3"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 6 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Anno.Redis/RedisHelper.cs"},"region":{"startLine":269}}}],"partialFingerprints":{"codehealthFindingId/v1":"afc0283dfdd1c57e51c7d5676c036f9b7ae95d90bdb633c93d9eec48ffeb49c3"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Anno.Redis/RedisHelper.cs"},"region":{"startLine":247}}}],"partialFingerprints":{"codehealthFindingId/v1":"722c5a556a219dd9d6f46929a8fe858fa6fbb840e9a5dc5e1b4bb0e10721e115"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Trace/Anno.Domain/Service/MemberService.cs"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9a8acd9526fa373411789ed346962085da0731442571f6d5de2b4c88e1d31cf"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 15 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Anno.Redis/RedisHelper.cs"},"region":{"startLine":340}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d7d5752ca274371023fdf90255eeb24c2cd1d73e679be25ab63aab1533d15ad"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 33 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Trace/Anno.Domain/Service/MemberService.cs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"0d6b1e9f50ad264565c1799c35d22f46371ffd76b922f7df3c0b845616af1444"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Viper.Test/Program.cs"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"3742f99cb98e58f8426e6a53b0927ba942bccec28193a78b643501df7e411cba"}},{"ruleId":"D18","level":"note","message":{"text":"Thin analysable surface across projects: 4 project(s) carry only a thin slice of real code (e.g. \u0060HelloWorldDto\u0060 with 12 significant line(s)). The mean analysable-surface weight is 92 %, lowering Solution Shape by about 0.66 point(s). Consolidate thin projects or grow them into substantial, well-scoped assemblies."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dd0b92fd965c2065080f16843920964ee00f7696ea03d87cdf61aed4cd2c746a"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found at common paths; consider documenting architectural decisions in Docs/ADL/ or similar."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D21","level":"note","message":{"text":"Inconsistent naming for the \u0027Member\u0027 entity and its related tables/DTOs. Some use \u0027SysMember\u0027 or \u0027sys_member\u0027 (e.g., \u0060Anno.Domain.Member.SysMember\u0060), while others use the raw table name \u0027sys_member\u0027 (e.g., \u0060Anno.Model.sys_member\u0060). Similarly, \u0027Role\u0027 is sometimes \u0027SysRole\u0027 or \u0027sys_roles\u0027. The model layer mixes POCO names with database table names.: Standardize on \u0027SysMember\u0027/\u0027SysRole\u0027 for domain models and \u0027SysMember\u0027/\u0027SysRole\u0027 for DTOs/Models, or consistently use the table names if that is the convention. Avoid mixing \u0027SysMember\u0027 and \u0027sys_member\u0027 for the same concept. (symbols: Anno.Domain.Member.SysMember.Pwd, Anno.Domain.Member.SysMember.State, Anno.Model.sys_member.state, Anno.Model.sys_member.Roles, Anno.Model.sys_func_roles_link.fid, Anno.Model.sys_func.fcode, Anno.Model.sys_log.AppName, Anno.Model.sys_log.Content, Anno.Model.sys_log.Uname, Anno.Model.sys_trace.Askrouter, Anno.Model.sys_trace.Rlt, Anno.Model.sys_trace.Request, Anno.Model.bif_company.email, Anno.Model.bif_company.rdt, Anno.Model.sys_roles, Anno.Model.sys_member, Anno.Model.sys_member_roles_link, Anno.Model.sys_func_roles_link)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"03059371cf9e58cdb353dd0fe3bdf5ec5901878475148539440755f776007ea8"}},{"ruleId":"D21","level":"note","message":{"text":"Inconsistent naming for bash command wrappers. Some are named after the command itself (e.g., \u0060Mv\u0060, \u0060Cat\u0060, \u0060Grep\u0060), while others use a generic \u0060Command\u0060 or \u0060Output\u0060. This makes it hard to distinguish between specific command wrappers and generic utilities.: Either name all wrappers after the specific command (e.g., \u0060Mkdir\u0060, \u0060Chmod\u0060) or use a consistent pattern like \u0060BashCommand\u0060 or \u0060BashExecutor\u0060. Avoid mixing specific command names with generic ones. (symbols: Anno.Common.Bash.Command, Anno.Common.Bash.Mv, Anno.Common.Bash.Cat, Anno.Common.Bash.Grep, Anno.Common.Bash.Rm, Anno.Common.Bash.Output)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"301451d99eb53c4012c28b74b5719e6eeae247e85ff622ad91f5a7e14bedba21"}},{"ruleId":"D21","level":"note","message":{"text":"Inconsistent naming for Member and Role entities. The domain uses \u0027SysMember\u0027 and \u0027Role\u0027, while the model layer uses \u0027sys_member\u0027 and \u0027sys_roles\u0027. This creates a disconnect between the domain model and the data model/DTOs.: Align the model/DTO names with the domain names (e.g., \u0060Member\u0060 and \u0060Role\u0060) or ensure the model layer consistently uses the \u0027Sys\u0027 prefix if that is the convention. (symbols: Anno.Domain.Member.SysMember, Anno.Model.sys_member, Anno.Domain.Member.Role, Anno.Model.sys_roles)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a01bfb5f41fcbf45e50cc3a42d7aa3e7a61d191f243c53dfaa3f7a61844620d3"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 10k LoC across 26 projects the codebase is large and multi-module, so explicit bounded contexts are needed. 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":"D29","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":"e69fa36d2e8961655cd8b4efb21f5ecdaa0450d1c651601d42dd977f1a1b3e6c"},"taxa":[{"id":"CWE-862","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","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":"5d2f5c975f2d9e0904e2cc3d2e937d3ac92a963dc48770a7ba63b44204e7f003"},"taxa":[{"id":"CWE-862","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]}],"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-862","guid":"2d96ecd7-f7f1-7f55-9f3a-43bb5bafdf33","name":"CWE-862","shortDescription":{"text":"CWE-862"},"helpUri":"https://cwe.mitre.org/data/definitions/862.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."]}]}