{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"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":"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":"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":"D1","level":"warning","message":{"text":"SxsManifest.ExtractDependenciesFromSxsElement (cyclomatic 30): SxsManifest.ExtractDependenciesFromSxsElement has cyclomatic complexity 30 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"46f6b312f59732c6485c275c0c2ad4730bae05c7115a717e2e766d47304ab88b"}},{"ruleId":"D1","level":"warning","message":{"text":"DependencyWindow.ProcessClrImports (cyclomatic 16): DependencyWindow.ProcessClrImports has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":768}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa8e620bd324fa1b35c1d9fe8d0eae818bd39f8cf92310eedd86356aafd8808a"}},{"ruleId":"D1","level":"warning","message":{"text":"DependencyWindow.ConstructDependencyTree (cyclomatic 16): DependencyWindow.ConstructDependencyTree has cyclomatic complexity 16 (threshold 15). Most of this is not in the body itself: 1 of the 16 points is its own statement and the rest belongs to one function literal inside it that branches (line 990). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":975}}}],"partialFingerprints":{"codehealthFindingId/v1":"861f3e73507f6eb63baf75d73887d7c69c1a7ff1d575e381b7d17414265463da"}},{"ruleId":"D2","level":"warning","message":{"text":"SxsManifest.ExtractDependenciesFromSxsElement (cognitive 53): SxsManifest.ExtractDependenciesFromSxsElement has cognitive complexity 53 (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":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"70c1b01d45b29bc05671d209ed8d6529a161062d198a94c5bb403088fab89055"}},{"ruleId":"D2","level":"warning","message":{"text":"DependencyWindow.ProcessClrImports (cognitive 42): DependencyWindow.ProcessClrImports has cognitive complexity 42 (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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":768}}}],"partialFingerprints":{"codehealthFindingId/v1":"3cb1a74bb228904df95de0310c8ce6606b3a1b0dea20ac066caf7c3f7e86f982"}},{"ruleId":"D2","level":"warning","message":{"text":"DependencyWindow.ConstructDependencyTree (cognitive 41): DependencyWindow.ConstructDependencyTree has cognitive complexity 41 (threshold 15). Most of this is not in the body itself: 0 of the 41 points are its own statements and the rest belongs to one function literal inside it that branches (line 990). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":975}}}],"partialFingerprints":{"codehealthFindingId/v1":"8abad9381ad791bfe257f712963419292d63c9fdd2372f3f2187baa9173b4401"}},{"ruleId":"D2","level":"warning","message":{"text":"StackOrganiser.Measure (cognitive 23): StackOrganiser.Measure has cognitive complexity 23 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"third_party/Dragablz/Dragablz/StackOrganiser.cs"},"region":{"startLine":219}}}],"partialFingerprints":{"codehealthFindingId/v1":"8595634bc79b284b17eb9ef1d9114cec5c3671eb5aa6032507afa21375d1ae12"}},{"ruleId":"D2","level":"warning","message":{"text":"TabablzControl.ReceiveDrag (cognitive 20): TabablzControl.ReceiveDrag has cognitive complexity 20 (threshold 15). Most of this is not in the body itself: 5 of the 20 points are its own statements and the rest belongs to one function literal inside it that branches (line 1330). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"third_party/Dragablz/Dragablz/TabablzControl.cs"},"region":{"startLine":1301}}}],"partialFingerprints":{"codehealthFindingId/v1":"93595a47dd257bcb8be8d40b4ee46255876d1f81653a3b38ec5d5561397ca51d"}},{"ruleId":"D2","level":"warning","message":{"text":"BinaryCache.LookupImports (cognitive 19): BinaryCache.LookupImports 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":"DependenciesLib/BinaryCache.cs"},"region":{"startLine":187}}}],"partialFingerprints":{"codehealthFindingId/v1":"317b82fef098cc3060dfc42e8dfb40fed44c06ac53fbde5397689a615f4be058"}},{"ruleId":"D2","level":"warning","message":{"text":"GridViewSort.ColumnHeader_Click (cognitive 17): GridViewSort.ColumnHeader_Click 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":"DependenciesGui/GridViewSort.cs"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac1bd299edea2ce8eea96f9a76abc5fcf07948417b7841fea67e649b939aa505"}},{"ruleId":"D2","level":"warning","message":{"text":"BinaryCache.LookupImport (cognitive 17): BinaryCache.LookupImport has cognitive complexity 17 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesLib/BinaryCache.cs"},"region":{"startLine":150}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b3a093e509b805eac7cabf9ebbc5a82a55a595a5d56ce5364e6c603d3fcae21"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: DependenciesGui/DependencyWindow.xaml.cs: FileTooLong \u2014 664 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f1fb205062da8480cf26a5658548d3aa2b901edf825d8ef9d70f509e3390a81"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: DependencyWindow: ClassTooLong \u2014 439 significant lines (blank, comment-only and punctuation-only lines excluded), 20 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"17be6a3e669560ac3d7f187e30192007ab5ae94e72c141c32e345b23dbe8ac05"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (23 lines \u00D7 3): DependenciesGui/Models/ModuleInfo.cs:449-471 | DependenciesGui/Models/PeExport.cs:150-172 | DependenciesGui/Models/PeImport.cs:178-200 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060DependenciesGui/Models/ModuleInfo.cs:449\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":"DependenciesGui/Models/ModuleInfo.cs"},"region":{"startLine":449}}}],"partialFingerprints":{"codehealthFindingId/v1":"ef5b12016e5cba87884f82ca9c99b0f6d8bf6d3cc313b8726f71dfdec2e7faf7"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Dragablz.net45: Dragablz.net45: abstractness 0.10, instability 0.00, distance 0.90 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a8cfb02deb6ad5f96d6cb32ccbf1d7fbdf5c99b51825ebfbec5a21a3f60d6b70"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: DependenciesLib: DependenciesLib: abstractness 0.14, instability 0.00, distance 0.86 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0662913f9f53a8af5c5c262a16a5f2665a6f6d57c2323d27e4d92fcc93197c6d"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: DragablzItemsControl (LCOM4 5): DragablzItemsControl\u0027s methods form 5 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":"third_party/Dragablz/Dragablz/DragablzItemsControl.cs"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"3413ab7f8dfc89ce3aff888a43997f2e4ee8ebec021b6e0ae7f92db4bd62889f"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: DependencyWindow (LCOM4 4): DependencyWindow\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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":513}}}],"partialFingerprints":{"codehealthFindingId/v1":"3972dad401f8dd2cc589a0860ff80217068149fad1f3226c02f6ffb1ba0e11ef"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: MainWindow (LCOM4 4): MainWindow\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":"DependenciesGui/MainWindow.xaml.cs"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"56e640fa53255b21a62402c118c31d4ddedd3f3ac5d15eee72cb3b6fea10f1dd"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured: The test suite couldn\u0027t be built/run in-image and no coverage report is committed, so line coverage was not measured \u2014 and it is EXCLUDED from the score rather than scored on a LoC-ratio proxy. No coverage collector was found in your CI either, so there is no existing report to hand us: add a coverage collector to your test run and commit (or publish) its Cobertura/OpenCover/lcov output anywhere in the repo, or make the suite runnable in-image, and real coverage will be measured."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test 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 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":"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":"DependenciesGui/About.xaml.cs"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d752221c723feca98b463041d392d1528518cb407077932b5492f7b361e08f1"}},{"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":"DependenciesGui/DependencyCustomListView.xaml.cs"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"824c0e857a78df678b382d4c61935bdaa1ef5bccb4ce0a3212fb6b999e63c336"}},{"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":"DependenciesGui/DependencyExportList.xaml.cs"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"7da6eac94dc1fc57e6534c1832c61b966b674229259675cc78477a5f38c7149f"}},{"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":"DependenciesGui/DependencyImportList.xaml.cs"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2a6193d70f0a137981d4338dd39942332e1948250483b11b83231c6d2cc6031"}},{"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":"DependenciesGui/Models/ModuleInfo.cs"},"region":{"startLine":466}}}],"partialFingerprints":{"codehealthFindingId/v1":"068ca8fee2801c83b219e100cc33acf1dc74856a323cc6ca8428adfb137c5ce2"}},{"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":"DependenciesGui/Models/PeExport.cs"},"region":{"startLine":166}}}],"partialFingerprints":{"codehealthFindingId/v1":"81a8589da28b04ce99ee0af8f32a51a8df6809eb526ff973ce4f272b71eb809d"}},{"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":"DependenciesGui/Models/PeImport.cs"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fb898b123a405cb586adceeb899a4d1a6dc58ebf5bff45edaa3c0a2f8de86ed"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : Find a way to properly bind commands instead of using this hack \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":"DependenciesGui/DependencyModuleList.xaml.cs"},"region":{"startLine":94}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7e74e46efdfa02b2d22f7fdd14c1aa69f863d93753203ffadead3e52b50f29d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : Find a way to properly bind commands instead of using this hack \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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":581}}}],"partialFingerprints":{"codehealthFindingId/v1":"347b4b2d6f5fb11c771fcfd0d2f366b9ea67df96b0b39c0d377cc207da3ca4ce"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // Extremely crude parser. TODO : Add support for quotes wrapped paths with spaces \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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":721}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea1245a281232d9f50dfba49d18df153e0395e2d93c6862411693af7830a435e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Improve resolution predicate \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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1135}}}],"partialFingerprints":{"codehealthFindingId/v1":"de93770fffa7b23601beaf97df080fb54190626cc64719c065da6204f870a048"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : do a proper refresh instead of asking the user to do it \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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1182}}}],"partialFingerprints":{"codehealthFindingId/v1":"057c5e2900a1d5d3ee86a0a239e038d98464ded7f934f5cda1b47fcb3cbbffa5"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : Reduce memory pressure by storing tree nodes data context in a HashSet and find an async trick \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":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1320}}}],"partialFingerprints":{"codehealthFindingId/v1":"955be2abdd451649d7b886c276ed5612d0eb62b133e80e7a04a9a5f159ef3d3c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : understand how to reliably bind in xaml \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":"DependenciesGui/MainWindow.xaml.cs"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"62ee8cb4260b4c8dc70334a996ea9054410f1a6bd409076002aa8565e1902b96"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/MainWindow.xaml.cs"},"region":{"startLine":217}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c7351edb0bc3c44da04bbc73f6b468b23675cfa22c788f420cd3902924a380a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : update LRU cache \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":"DependenciesLib/BinaryCache.cs"},"region":{"startLine":415}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0951b9ff3c329718b90870170c503240d89a0b06e7073fbb195ac6a86359c20"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : find dll search order \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":"DependenciesLib/FindPeModule.cs"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"a25bcf7b80fc8d005e1245bf6348f89230b1413677dda549e48e7b9e5f7665d8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : Add a user supplied path emulating %cwd% \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":"DependenciesLib/FindPeModule.cs"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"860a9ac2bdbd86c6a8bb21f2a42384d32dcdab222327236fe0e69fe1588a32fc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : DLL search order ? \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":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f94ec456ba4e25080c4425b5c8c46d037e64444917979f2dce19efe1a5ef3db"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : support localized sxs redirection \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":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"cbdd317ca8361dfa29ec061dc000c8a9e0ed922d516fbe5404a7f086d13cd5c3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : do the same thing for localization \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":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":307}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9ac1edaa72ba3b5e350597b70da3c75323f06d9a3f09927e9f1b361d2e36888"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO : also take into account Multilanguage User Interface (MUI) when \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":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":315}}}],"partialFingerprints":{"codehealthFindingId/v1":"76632188a1933018752b9cff8e8a60f9e85bf61fccb45e5e53cc3310ea7ec46b"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"379794117e79a7b202df1579d351163f579589858ce6fdc8fc9e417afdeb7d2e"}},{"ruleId":"D19","level":"note","message":{"text":"The phnt header collection README is a short \u0027This collection of Native API header files has been maintained since 2009...\u0027 entry with no usage example and only PHNT_VERSION defines; it does not explain how to use the headers or what symbols are exposed.: Add a Usage section showing the exact #include directives (phnt_windows.h, phnt.h) and the default Windows XP symbol set. Mention which PHNT_VERSION options expose newer APIs."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"third_party/phnt/README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c425cf9684905b156e359e4e18bfacfab800599d123a29944df17b0d6bf83968"}},{"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 file path properties: some use \u0027Filepath\u0027 (camelCase) while others use \u0027ModuleFilePath\u0027 (PascalCase) or \u0027ModuleName\u0027 (different concept). Specifically, \u0027Filepath\u0027 vs \u0027ModuleFilePath\u0027 represents a naming inconsistency for the same or similar concept (file path).: Standardize to \u0027FilePath\u0027 (PascalCase) for all file path properties. (symbols: Dependencies.NotFoundModuleInfo.Filepath, Dependencies.ApiSetModuleInfo.Filepath, Dependencies.ApiSetNotFoundModuleInfo.ModuleName, Dependencies.ModuleSearchResult.ModuleFilePath)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c025b9008d84b2459bc4fac5808ca4ecdea6a3babdbf3808d2944195eb2aa2dc"}},{"ruleId":"D21","level":"note","message":{"text":"Inconsistent naming for dependency-related concepts: \u0027PeDependencyItem\u0027 vs \u0027PeDependencies\u0027. One is singular, the other plural, suggesting the same or related concepts are named differently.: Standardize to \u0027PeDependency\u0027 (singular) or \u0027PeDependencies\u0027 (plural) consistently across the codebase. (symbols: Dependencies.PeDependencyItem.Dependencies, Dependencies.PeDependencies.ResolveModule(string))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"391e4427732b2c47a691bf6e918a05b5a20869b297af7dc57ad34852ef8c3a14"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 13k LoC across 6 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":"D34","level":"note","message":{"text":"Dormant codebase: 20 of 20 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 20 separate risks. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D34","level":"note","message":{"text":"Largest orphaned file: One of the largest files with no living knowledge remaining \u2014 a reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesGui/DependencyWindow.xaml.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"db07b2e79445af5b520afa9f6c0074e02808af37c8bd32eecdd232de3b11f475"}},{"ruleId":"D34","level":"note","message":{"text":"Largest orphaned file: One of the largest files with no living knowledge remaining \u2014 a reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Dependencies/Program.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d2bdfe6ef875ba2c3cc80593ac24723287609cc0225e3375bf37ce62c66eef6"}},{"ruleId":"D34","level":"note","message":{"text":"Largest orphaned file: One of the largest files with no living knowledge remaining \u2014 a reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"DependenciesLib/SxsManifest.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b95d3cbcc0c5b7e6324fef5bad0e5647e51549a16ba11596a198adeba0ac47a3"}}],"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":0,"secretScannerRunsExcluded":0}}}]}