{"$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":"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":"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":"D22","name":"Internal API Consistency","shortDescription":{"text":"Internal API Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D22"},{"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":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"id":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D33","name":"JS/npm Dependency Vulnerabilities","shortDescription":{"text":"JS/npm Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D33","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"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 (20 lines \u00D7 2): framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs:131-150 | framework/src/Qf.Core.Web/Authentication/WeChat/WeChatHandler.cs:136-155 \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":"framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"e032900fc12c355a0f8a166aca3490b71fb8ad0a77c49cf1bcc3e080ed374ee9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs:51-63 | framework/src/Qf.Core.Web/Authentication/WeChat/WeChatHandler.cs:48-60 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"daccb1b428d590db1322054b52d55a8a9abec853fe19e725807c6e5768317aee"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs:202-211 | framework/src/Qf.Core.Web/Authentication/WeChat/WeChatHandler.cs:207-216 \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 \u0060framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs:202\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":"framework/src/Qf.Core.Web/Authentication/AliPay/AliPayHandler.cs"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"4244cf0692a62269cd13f3ce3af01ac10716976ce2bcaa232b5f4b76a8a11134"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): framework/src/Qf.Extensions.Configuration.Encryption/AES.cs:66-73 | framework/src/Qf.Extensions.Configuration.Encryption/AES.cs:92-99 \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 \u0060framework/src/Qf.Extensions.Configuration.Encryption/AES.cs:66\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":"framework/src/Qf.Extensions.Configuration.Encryption/AES.cs"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"978361c052c7dad6e82a52936b71457203c0d791a7e7b4d5caf8d58acb9804ab"}},{"ruleId":"D5","level":"error","message":{"text":"Layer violation: Application \u2192 Infrastructure: Qf.SysTodoList.Application (Application) references Qf.SysTodoList.Infrastructure (Infrastructure) \u2014 dependencies must point inward (Web \u2192 Application \u2192 Domain; Infrastructure implements inner interfaces, nothing depends outward on it)."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e27aca6f791fcbe2465e67f515025cce032234649858060654062cf706cdb54c"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: UnitOfWork (LCOM4 4): UnitOfWork\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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"6ee9471e51c98de673ea2157e5b9691683aef28cab3fcc16b9747586c5c956c9"}},{"ruleId":"D8","level":"error","message":{"text":"No automated tests: No automated tests \u2014 no test code was found in this repository. Untested code is the largest single risk to changing it safely."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3132564c6310e5d01a231f252a02d5d2f5a542c0a8fa29a14c89693f1e3df871"}},{"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":"D16","level":"warning","message":{"text":"dormant codebase \u2014 no living knowledge left to concentrate: All 17 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":"warning","message":{"text":"TodoComment: //TODO: It can be slow to always check if service is available. Test it and optimize if necessary. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.AutoMapper/DefaultObjectMapper.cs"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"b1a085c910a5ea14fe72bbc857649f28d5c0cce2551f35bf9aff6325cc5337c0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Check if TDestination has a proper constructor which takes TSource \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.AutoMapper/DefaultObjectMapper.cs"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"bfd52be243cb7e11632b0f62edf1db8d748729255ae6ea59230d47396c0b0969"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Check if TDestination has an empty constructor (in this case, use MapFrom) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.AutoMapper/DefaultObjectMapper.cs"},"region":{"startLine":61}}}],"partialFingerprints":{"codehealthFindingId/v1":"24af5184dd1da01d2c54d7e3f8d8bb7fe5c72b012afd2f61c387c56780e507f0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Remove catch when TODOs are implemented above \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.AutoMapper/DefaultObjectMapper.cs"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"db752b74e5248ad98252f767937ae381a68cb8eaa75bff849d953573721f351b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Why not using the new created transaction? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.EFCore/Uow/UnitOfWorkDbContextProvider.cs"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6bd6b653a261593d0dc5dd74f4ec8cf59549f63f6f5006b571b73049213ce48"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Can this be done in one operation on the server side? Probably, the trick would just be the DateTimeOffset math. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":150}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b2508a7452bf0affa5a53c409fa8a58161c4aa04d04e123e7d39a256e6b723c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Can this be done in one operation on the server side? Probably, the trick would just be the DateTimeOffset math. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc38654f93c81bae43215afab3693c8ec3ec1e6c63d430ec8f79f0dd2ae4701e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb671f23e1814abef89eb02e8a86244b396c2b7955df7832b5fc5aa4922bda34"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":201}}}],"partialFingerprints":{"codehealthFindingId/v1":"3634a824917cdd1ad5722be5349fb6e4a7390b30f6c99149032178c6a70fcecc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":224}}}],"partialFingerprints":{"codehealthFindingId/v1":"952f7c9eb1368e2243facb43f58d8d41ffae1c3dbd76d8e8fb7baf43586cb6ef"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":235}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8f24e9e447169ed35b29e1df250feeea23871bb9d4fd224b1973ba161cb5c71"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":273}}}],"partialFingerprints":{"codehealthFindingId/v1":"8687d3fb28b9d8a45bea19af737f979a3e2cce01a8a82537310358be3b16093a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error handling \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Extension/CSRedisCache.cs"},"region":{"startLine":300}}}],"partialFingerprints":{"codehealthFindingId/v1":"3391b4483c87f5eec9013eea5a17613a94d6435b394c90172438233a98501104"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Can be optimized (by caching per entity type)? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core/Helper/EntityHelper.cs"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"a20ce402696cec848bec26ad08cb0505644f66565c95cd4074a90fd5a4329d1d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Ehhance summary \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core/Helper/ReflectionHelper.cs"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"40f8e1a47a33f2bb5dba92d6e43101815391ce8c9f4df0cee82ba28f0b848893"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Summary \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core/Helper/ReflectionHelper.cs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"0dce32fbc4ffeb9dd2ff7cb932d683c4265c346d6eae04c0fd3127e80d5d026c"}},{"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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":298}}}],"partialFingerprints":{"codehealthFindingId/v1":"54d660c95903dfe72b923d069e8ea2931e9ea8b059cc0c0f4183d2b2e716b91b"}},{"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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":320}}}],"partialFingerprints":{"codehealthFindingId/v1":"916b3cc660ea4bcc5c5e7ea7904a04e144d04a8d316dd6ffca1f0dbd05866084"}},{"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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":329}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a257ad1daa482eeb00b923b11e41efb3b78e8c764cbbbca2b61267f2b11c2c0"}},{"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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":343}}}],"partialFingerprints":{"codehealthFindingId/v1":"64dbd9bb39e97eefb5cb43429b06b6bd9dcd836e3e25ac37a719a430238cc0ab"}},{"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":"framework/src/Qf.Core/Uow/UnitOfWork.cs"},"region":{"startLine":355}}}],"partialFingerprints":{"codehealthFindingId/v1":"6197f18b655a5f77021db4bfa6a6b0879f68bf8b2901d329f097f258f74a9c70"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj: 1591 \u2014 this warning is switched off for the WHOLE project, in every file it builds, including code written years from now: nothing at the call site records that the rule was ever silenced, so the next reader has no reason to look here. Fix what the rule is reporting and drop the code from the list, or \u2014 if some occurrences really are legitimate \u2014 narrow the suppression to those sites and give each one its reason, so the rule keeps protecting the rest of the project."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"samples/Qf.SysTodoList.WebApi/Qf.SysTodoList.WebApi.csproj"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"007df08bc65ec5f9fe9223811bced7e9848433eca3955b101e059d3501661689"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: SetValueByPath: Method SetValueByPath \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core/Helper/ReflectionHelper.cs"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"6fa00325b3237d47a9d756cb4c4a99d8c376b99d857bce8dca6234d045cad044"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: GetUnionId: Method GetUnionId \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"framework/src/Qf.Core.Web/Authentication/WeChat/WeChatHandler.cs"},"region":{"startLine":295}}}],"partialFingerprints":{"codehealthFindingId/v1":"a50e413cfd0bebe252b63696a7e2e9e6a1fe5f583a280db7ff91e56af0b0d0f0"}},{"ruleId":"D18","level":"warning","message":{"text":"Monorepo: only 1 of 3 solutions was scored: This repository contains 3 .NET solutions, but a scan analyzes ONE. Every score, lens, and finding here reflects only \u0060framework/Qf.Core.sln\u0060 \u2014 the other 2 (\u0060gateway/Qf.APIGateway.sln\u0060, \u0060samples/Qf.SysTodoList.sln\u0060) were not analyzed and are not represented in the headline. To cover them, scan each solution as its own target and group them in a Solution or Product for a portfolio roll-up. If a secondary solution is an archived or vendored tree, declare it \u2014 \u0060.gitattributes\u0060 (\u0060path/** linguist-vendored\u0060) or \u0060.editorconfig\u0060 (\u0060[path/**] generated_code = true\u0060) \u2014 to exclude it from discovery the same way generated code is."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"447bb11e4227696119972b805fcb72d3ced918b47fcda508d1efcfafe347e404"}},{"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 transaction-related configuration properties. One uses a boolean flag \u0060IsTransactional\u0060 while the other uses \u0060IsolationLevel\u0060 for transaction settings, suggesting a lack of unified naming convention for transaction configuration.: Consider using a single \u0060TransactionOptions\u0060 or \u0060TransactionSettings\u0060 object or consistent boolean prefixes like \u0060IsTransactional\u0060 and \u0060IsolationLevel\u0060 should be grouped or named consistently. (symbols: Qf.Core.Uow.UnitOfWorkOptions.IsTransactional, Qf.Core.Uow.UnitOfWorkOptions.IsolationLevel)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c6381f327e10a0ab4b753917c58b93c3f54c97f3984df7797534a72a756fa182"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 8k LoC across 5 projects the codebase is mid-range size and multi-module, so explicit bounded contexts are warranted. Name this codebase\u0027s bounded contexts (\u22652 module groups, e.g. per subsystem) so cross-boundary type coupling can be assessed. Declare them in \u0060.codehealth/config.yaml\u0060 at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it \u2014 e.g. \u0060architecture:\u0060 \u2192 \u0060contexts:\u0060 \u2192 \u0060Billing: [\u0022src/billing\u0022, \u0022Acme.Billing\u0022]\u0060, \u0060Catalog: [\u0022src/catalog\u0022, \u0022Acme.Catalog\u0022]\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"1c7e276c9c682731f01819ed5378b90ca320cde1b583c90920172911598362b3"}},{"ruleId":"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":"e061ecba0e9525b2b71d6fba0e587582cf8279d3e0d73be6391723971f67af5a"}},{"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":"D31","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":"cb73f05a5e88ec330079bd45fc13ff4392a08b5da3d00fef2ad17b93d2a09cd9"}},{"ruleId":"D31","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":"4a6cc66a9154e52926710c4ab09a54f2f68512f474e8cd3e94325a3e8b3adb57"}}],"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-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"id":"CWE-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.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-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.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":4,"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."]}]}