{"$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":"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"},{"id":"D44","name":"Platform End-of-Life","shortDescription":{"text":"Platform End-of-Life"},"helpUri":"https://codehealth.canine.dev/dimensions/D44"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX3","name":"Project dependency cycles","shortDescription":{"text":"Project dependency cycles"},"helpUri":"https://codehealth.canine.dev/dimensions/AX3"},{"id":"AX4","name":"Dependency direction","shortDescription":{"text":"Dependency direction"},"helpUri":"https://codehealth.canine.dev/dimensions/AX4"},{"id":"AX5","name":"Architecture \u0026 structure","shortDescription":{"text":"Architecture \u0026 structure"},"helpUri":"https://codehealth.canine.dev/dimensions/AX5"},{"id":"AX6","name":"Interface segregation","shortDescription":{"text":"Interface segregation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX6"},{"id":"C1","name":"Data Protection","shortDescription":{"text":"Data Protection"},"helpUri":"https://codehealth.canine.dev/dimensions/C1"},{"id":"ED5","name":"Idempotency","shortDescription":{"text":"Idempotency"},"helpUri":"https://codehealth.canine.dev/dimensions/ED5"},{"id":"GD1","name":"Unfinished \u0026 placeholder code","shortDescription":{"text":"Unfinished \u0026 placeholder code"},"helpUri":"https://codehealth.canine.dev/dimensions/GD1"},{"id":"IC1","name":"Incompleteness \u0026 stubs","shortDescription":{"text":"Incompleteness \u0026 stubs"},"helpUri":"https://codehealth.canine.dev/dimensions/IC1"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"PF1","name":"Benchmark discipline","shortDescription":{"text":"Benchmark discipline"},"helpUri":"https://codehealth.canine.dev/dimensions/PF1"},{"id":"PF2","name":"Allocation hygiene","shortDescription":{"text":"Allocation hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/PF2"},{"id":"PF3","name":"Async \u0026 latency hygiene","shortDescription":{"text":"Async \u0026 latency hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/PF3"},{"id":"SC1","name":"Supply-chain hygiene","shortDescription":{"text":"Supply-chain hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/SC1"},{"id":"X1","name":"Async correctness","shortDescription":{"text":"Async correctness"},"helpUri":"https://codehealth.canine.dev/dimensions/X1"},{"id":"X10","name":"Duplicated predicate","shortDescription":{"text":"Duplicated predicate"},"helpUri":"https://codehealth.canine.dev/dimensions/X10"},{"id":"X12","name":"Unreachable branch","shortDescription":{"text":"Unreachable branch"},"helpUri":"https://codehealth.canine.dev/dimensions/X12"},{"id":"X13","name":"Undrained process stream","shortDescription":{"text":"Undrained process stream"},"helpUri":"https://codehealth.canine.dev/dimensions/X13"},{"id":"X14","name":"Bypassable address classification","shortDescription":{"text":"Bypassable address classification"},"helpUri":"https://codehealth.canine.dev/dimensions/X14"},{"id":"X15","name":"Unvalidated length from an untrusted reader","shortDescription":{"text":"Unvalidated length from an untrusted reader"},"helpUri":"https://codehealth.canine.dev/dimensions/X15"},{"id":"X16","name":"Unfloored truncation loop","shortDescription":{"text":"Unfloored truncation loop"},"helpUri":"https://codehealth.canine.dev/dimensions/X16"},{"id":"X17","name":"Uncapped recursion over a caller-supplied document","shortDescription":{"text":"Uncapped recursion over a caller-supplied document"},"helpUri":"https://codehealth.canine.dev/dimensions/X17"},{"id":"X18","name":"Disposal-pattern correctness","shortDescription":{"text":"Disposal-pattern correctness"},"helpUri":"https://codehealth.canine.dev/dimensions/X18"},{"id":"X19","name":"Unrestored process-global state","shortDescription":{"text":"Unrestored process-global state"},"helpUri":"https://codehealth.canine.dev/dimensions/X19"},{"id":"X20","name":"Mistyped argument guard","shortDescription":{"text":"Mistyped argument guard"},"helpUri":"https://codehealth.canine.dev/dimensions/X20"},{"id":"X21","name":"Side-effecting pattern guard","shortDescription":{"text":"Side-effecting pattern guard"},"helpUri":"https://codehealth.canine.dev/dimensions/X21"},{"id":"X22","name":"Contradicted release guard","shortDescription":{"text":"Contradicted release guard"},"helpUri":"https://codehealth.canine.dev/dimensions/X22"},{"id":"X23","name":"Unguarded diagnostic materialisation","shortDescription":{"text":"Unguarded diagnostic materialisation"},"helpUri":"https://codehealth.canine.dev/dimensions/X23"},{"id":"X24","name":"Document value interpolated into markup unescaped","shortDescription":{"text":"Document value interpolated into markup unescaped"},"helpUri":"https://codehealth.canine.dev/dimensions/X24"},{"id":"X25","name":"Inert configuration knob","shortDescription":{"text":"Inert configuration knob"},"helpUri":"https://codehealth.canine.dev/dimensions/X25"},{"id":"X26","name":"Unsynchronised callback handoff","shortDescription":{"text":"Unsynchronised callback handoff"},"helpUri":"https://codehealth.canine.dev/dimensions/X26"},{"id":"X27","name":"Collection changed while being enumerated","shortDescription":{"text":"Collection changed while being enumerated"},"helpUri":"https://codehealth.canine.dev/dimensions/X27"},{"id":"X28","name":"Index access outside its own emptiness guard","shortDescription":{"text":"Index access outside its own emptiness guard"},"helpUri":"https://codehealth.canine.dev/dimensions/X28"},{"id":"X29","name":"Per-element action decided by a fixed element","shortDescription":{"text":"Per-element action decided by a fixed element"},"helpUri":"https://codehealth.canine.dev/dimensions/X29"},{"id":"X3","name":"Exception handling","shortDescription":{"text":"Exception handling"},"helpUri":"https://codehealth.canine.dev/dimensions/X3"},{"id":"X30","name":"Support guard that admits what it rejects","shortDescription":{"text":"Support guard that admits what it rejects"},"helpUri":"https://codehealth.canine.dev/dimensions/X30"},{"id":"X32","name":"Type resolved by simple name across every loaded assembly","shortDescription":{"text":"Type resolved by simple name across every loaded assembly"},"helpUri":"https://codehealth.canine.dev/dimensions/X32"},{"id":"X4","name":"Structured logging","shortDescription":{"text":"Structured logging"},"helpUri":"https://codehealth.canine.dev/dimensions/X4"},{"id":"X5","name":"Nullable reference types","shortDescription":{"text":"Nullable reference types"},"helpUri":"https://codehealth.canine.dev/dimensions/X5"},{"id":"X6","name":"Hand-rolled structured-format parsing","shortDescription":{"text":"Hand-rolled structured-format parsing"},"helpUri":"https://codehealth.canine.dev/dimensions/X6"},{"id":"X7","name":"Silent fallback defaults","shortDescription":{"text":"Silent fallback defaults"},"helpUri":"https://codehealth.canine.dev/dimensions/X7"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"NavTreeDropHandler.IDropTarget_DragOver (cyclomatic 53): NavTreeDropHandler.IDropTarget_DragOver has cyclomatic complexity 53 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: CustomControls/NavigationTree/ExperimentTree.xaml.vb holds 2 of the 19 methods over the threshold \u2014 including the worst \u2014 and 51 of the 125 points over it (41%), 2.7\u00D7 the next-largest file (CustomControls/Sequences/SequenceGraph.xaml.vb at 19). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":757}}}],"partialFingerprints":{"codehealthFindingId/v1":"b7f04d33b9d3b823ff511e9d46ebd1a5d975673fb2e16bf809eb5084c0e6fcd0"}},{"ruleId":"D1","level":"warning","message":{"text":"DbUpgradeServer.UpdateRev1ServerFieldTypes (cyclomatic 31): DbUpgradeServer.UpdateRev1ServerFieldTypes has cyclomatic complexity 31 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/DbUpgradeServer.vb"},"region":{"startLine":142}}}],"partialFingerprints":{"codehealthFindingId/v1":"26c466cb19793338a9083e34bcff81557b93109296b882edfa8f66d52967552f"}},{"ruleId":"D1","level":"warning","message":{"text":"NavTreeDropHandler.IDropTarget_Drop (cyclomatic 28): NavTreeDropHandler.IDropTarget_Drop has cyclomatic complexity 28 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: CustomControls/NavigationTree/ExperimentTree.xaml.vb holds 2 of the 19 methods over the threshold \u2014 including the worst \u2014 and 51 of the 125 points over it (41%), 2.7\u00D7 the next-largest file (CustomControls/Sequences/SequenceGraph.xaml.vb at 19). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":954}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bef2887c05b238a5729473eadc96075fc671ca311d7d06fb56976b5e9541de9"}},{"ruleId":"D1","level":"warning","message":{"text":"SequenceGraph.BuildSequences (cyclomatic 26): SequenceGraph.BuildSequences has cyclomatic complexity 26 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":472}}}],"partialFingerprints":{"codehealthFindingId/v1":"a131ac9f850261df3f54fd4d37353953d48ddd186fa5657d7846dd99662391fb"}},{"ruleId":"D1","level":"warning","message":{"text":"MainWindow.Me_Loaded (cyclomatic 25): MainWindow.Me_Loaded has cyclomatic complexity 25 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e874cf8257895ee082f55b8da60e8ad8932d3a89cb9cb1fefca5072078c381e"}},{"ruleId":"D1","level":"warning","message":{"text":"MainWindow.Me_PreviewKeyDown (cyclomatic 23): MainWindow.Me_PreviewKeyDown has cyclomatic complexity 23 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1084}}}],"partialFingerprints":{"codehealthFindingId/v1":"f14d92df3b8e6de5531a4de61fb840357ff13e5023258d55e09718b35103299f"}},{"ruleId":"D1","level":"warning","message":{"text":"SequenceGraph.ComputeLayout (cyclomatic 21): SequenceGraph.ComputeLayout has cyclomatic complexity 21 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":712}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d069d9f908b1d11ec7d2d9cbdc06349bfb4b479c2fe3f721171cbfb41f399e1"}},{"ruleId":"D1","level":"warning","message":{"text":"SequenceSchemeView.DrawConnectors (cyclomatic 19): SequenceSchemeView.DrawConnectors has cyclomatic complexity 19 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":883}}}],"partialFingerprints":{"codehealthFindingId/v1":"f31feae4c54cc1e32ee96ae99fe3dbb995c3278d0944e9b30b6a74a75faf0ddb"}},{"ruleId":"D1","level":"warning","message":{"text":"Protocol.ProtocolItem_PreviewMouseLeftButtonDown (cyclomatic 18): Protocol.ProtocolItem_PreviewMouseLeftButtonDown has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"37883c19c582700addcfe47d90025434e7a5eab5346e07a4f2791a14569a8bfa"}},{"ruleId":"D1","level":"warning","message":{"text":"dlgConnectGraph.PopulateSequenceScheme (cyclomatic 18): dlgConnectGraph.PopulateSequenceScheme has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/dlgConnectGraph.xaml.vb"},"region":{"startLine":254}}}],"partialFingerprints":{"codehealthFindingId/v1":"399eb65de82f957c707e1d4fb8a9a592610365a08e18292d76fa98ee67ee8b13"}},{"ruleId":"D1","level":"warning","message":{"text":"SpellCheckerRtb.HighlightAllWordOccurrences (cyclomatic 18): SpellCheckerRtb.HighlightAllWordOccurrences has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SpellChecker.vb"},"region":{"startLine":432}}}],"partialFingerprints":{"codehealthFindingId/v1":"9054ce96afa40aecf3374d47e528061779396ec22b617f6a8ef4049758529f11"}},{"ruleId":"D1","level":"warning","message":{"text":"SequenceGraph.CollapseCycles (cyclomatic 17): SequenceGraph.CollapseCycles has cyclomatic complexity 17 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":599}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e5ec5cf652e1e2b8aaeb45fbec3d079bd2b14b607e64fb135ae429564467d55"}},{"ruleId":"D1","level":"warning","message":{"text":"ELNCalculations.RecalculateMaterials (cyclomatic 17): ELNCalculations.RecalculateMaterials has cyclomatic complexity 17 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ELNCalculations.vb"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"71443cad4b036d471123d9628974b431ce3301e8cbb7f26876c0cc2ecbf8e1ce"}},{"ruleId":"D1","level":"warning","message":{"text":"dlgSearch.LoadQueryInfo (cyclomatic 16): dlgSearch.LoadQueryInfo has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgSearch.xaml.vb"},"region":{"startLine":427}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c4caed6785e5ddc5bd7e14a49aa25e8605335055bcfc542c0250c02551dfb25"}},{"ruleId":"D1","level":"warning","message":{"text":"ExperimentContent.CreateExperiment (cyclomatic 16): ExperimentContent.CreateExperiment has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/ExperimentContent.xaml.vb"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a87ede8223897319f84d4f1e75f812e8cc97054ebd0f5f068861d79c174d836"}},{"ruleId":"D1","level":"warning","message":{"text":"SketchArea.RemoveDeletedSideProducts (cyclomatic 16): SketchArea.RemoveDeletedSideProducts has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":197}}}],"partialFingerprints":{"codehealthFindingId/v1":"73aca2d38dc9ff341c82c238bdef6fdd550e5b02358a4a76bb346f8a9fdfebf2"}},{"ruleId":"D1","level":"warning","message":{"text":"TransferPackage.InitializeImport (cyclomatic 16): TransferPackage.InitializeImport has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/TransferPack/TransferPackage.vb"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"d370f7bc170f4c86857bf19874a207b2b31dcd575122277acab4dd3ae8659071"}},{"ruleId":"D1","level":"warning","message":{"text":"ExperimentBase.CloneExperiment (cyclomatic 16): ExperimentBase.CloneExperiment has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ExperimentBase.vb"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"a95744d6f0599b38b77176fa374b12b81603df623293f818cd91463cb3ee02cf"}},{"ruleId":"D1","level":"warning","message":{"text":"RestoreFromServer.ProcessChildren (cyclomatic 16): RestoreFromServer.ProcessChildren has cyclomatic complexity 16 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/RestoreFromServer.vb"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc718ef9287d790074ec74a87aaccbda613fffe7aedeb70ec4bc8952ee948ee1"}},{"ruleId":"D2","level":"warning","message":{"text":"NavTreeDropHandler.IDropTarget_DragOver (cognitive 77): NavTreeDropHandler.IDropTarget_DragOver has cognitive complexity 77 (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":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":757}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5e4cf5892bfab353e83d38c9bcc00b32bfcaa9e713ffd2679d0cc9a72862c55"}},{"ruleId":"D2","level":"warning","message":{"text":"NavTreeDropHandler.IDropTarget_Drop (cognitive 67): NavTreeDropHandler.IDropTarget_Drop has cognitive complexity 67 (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":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":954}}}],"partialFingerprints":{"codehealthFindingId/v1":"ae3e9579e8b5824b95f3687d66415130186b56ba953555c5ede704244be00534"}},{"ruleId":"D2","level":"warning","message":{"text":"SpellCheckerRtb.HighlightAllWordOccurrences (cognitive 55): SpellCheckerRtb.HighlightAllWordOccurrences has cognitive complexity 55 (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":"CustomControls/SpellChecker.vb"},"region":{"startLine":432}}}],"partialFingerprints":{"codehealthFindingId/v1":"6afd05df9775c264be2d4fc265f516b524b65674abc401f33fd4788fc0d26252"}},{"ruleId":"D2","level":"warning","message":{"text":"MainWindow.expNavTree_ExperimentSelected (cognitive 50): MainWindow.expNavTree_ExperimentSelected has cognitive complexity 50 (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":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1462}}}],"partialFingerprints":{"codehealthFindingId/v1":"ebf3478e01f1e11d755505d92da113936c8a6b08df01d0f546988b55cf2c5220"}},{"ruleId":"D2","level":"warning","message":{"text":"ExperimentBase.CloneExperiment (cognitive 48): ExperimentBase.CloneExperiment has cognitive complexity 48 (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":"ElnBase/ExperimentBase.vb"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"46ac9f8ae31bd7719a8175198f524f21746789b7dec6b5c8b9a798a7f28631be"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceGraph.CollapseCycles (cognitive 45): SequenceGraph.CollapseCycles has cognitive complexity 45 (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":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":599}}}],"partialFingerprints":{"codehealthFindingId/v1":"e37a75d6837e9564ddfffd8818aa2ecc2c6fb5b3bf9c11c41c8fb278bd957393"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceGraph.BuildSequences (cognitive 44): SequenceGraph.BuildSequences has cognitive complexity 44 (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":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":472}}}],"partialFingerprints":{"codehealthFindingId/v1":"f40b8a3cbfd4b1ea27c101842585911c3c4717031db6b4a2022125f649037ac2"}},{"ruleId":"D2","level":"warning","message":{"text":"ExperimentContent.CreateExperiment (cognitive 41): ExperimentContent.CreateExperiment has cognitive complexity 41 (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":"CustomControls/ExperimentContent.xaml.vb"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"592356d0b22ab53487688ef8436f6ddc53501a75096c3c4d76ec08137ea19bf8"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceSchemeView.DrawConnectors (cognitive 40): SequenceSchemeView.DrawConnectors has cognitive complexity 40 (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":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":883}}}],"partialFingerprints":{"codehealthFindingId/v1":"66c5b085cac6837eb7431dbb54bedcb156bf3b73625f3f0a2ebb7a1f0ba26520"}},{"ruleId":"D2","level":"warning","message":{"text":"dlgSearch.LoadQueryInfo (cognitive 39): dlgSearch.LoadQueryInfo has cognitive complexity 39 (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":"CustomControls/dlgSearch.xaml.vb"},"region":{"startLine":427}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b6897f1392996e460ae62ab60ec266010bb9f42c6ad6ec69a4e4e8798852abc"}},{"ruleId":"D2","level":"warning","message":{"text":"MainWindow.Me_Loaded (cognitive 38): MainWindow.Me_Loaded has cognitive complexity 38 (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":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"323b802d1695c069190eaa2becf72eca64cc11f4703285a256ea035e6d332ea1"}},{"ruleId":"D2","level":"warning","message":{"text":"RestoreFromServer.ProcessChildren (cognitive 37): RestoreFromServer.ProcessChildren has cognitive complexity 37 (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":"ElnBase/RestoreFromServer.vb"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"557d14e63add7f6d02de332e3c718f3c3bd73c9b7ee5a7921a00eca87800b0b4"}},{"ruleId":"D2","level":"warning","message":{"text":"MainWindow.Me_PreviewKeyDown (cognitive 35): MainWindow.Me_PreviewKeyDown has cognitive complexity 35 (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":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1084}}}],"partialFingerprints":{"codehealthFindingId/v1":"e0abf203b263351a3b5026e4b44d250bf1047cb40b65b8522c6d8a6b526932f2"}},{"ruleId":"D2","level":"warning","message":{"text":"MainWindow.TryOpenExperiment (cognitive 31): MainWindow.TryOpenExperiment has cognitive complexity 31 (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":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1363}}}],"partialFingerprints":{"codehealthFindingId/v1":"689426a917de468a3472c26c073b07643855de55aa7ffe5399f3a83519f3d563"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceGraph.ComputeLayout (cognitive 30): SequenceGraph.ComputeLayout has cognitive complexity 30 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":712}}}],"partialFingerprints":{"codehealthFindingId/v1":"31283b51756558c4b1f8f65b77080b43561225af580125bbe55da04908d196fb"}},{"ruleId":"D2","level":"warning","message":{"text":"DbUpgradeServer.UpdateRev1ServerFieldTypes (cognitive 30): DbUpgradeServer.UpdateRev1ServerFieldTypes has cognitive complexity 30 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/DbUpgradeServer.vb"},"region":{"startLine":142}}}],"partialFingerprints":{"codehealthFindingId/v1":"4191de5e570ef9823f0ce6d462237f1a4c25a23176242eee9aa4c0c79c41c84d"}},{"ruleId":"D2","level":"warning","message":{"text":"TransferPackage.InitializeImport (cognitive 28): TransferPackage.InitializeImport has cognitive complexity 28 (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":"CustomControls/TransferPack/TransferPackage.vb"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"1bf8b50f22d9152761264fd4216304e7008dc1398f66a2ee926449e46659a705"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceSchemeView.CreateSequencePanel (cognitive 27): SequenceSchemeView.CreateSequencePanel has cognitive complexity 27 (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":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":677}}}],"partialFingerprints":{"codehealthFindingId/v1":"faf63a6a8a9f77097c916f0d536a1d545e6c57fda9f16014daaa5a7bce539c37"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceGraph.BfsGraph (cognitive 25): SequenceGraph.BfsGraph has cognitive complexity 25 (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":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":389}}}],"partialFingerprints":{"codehealthFindingId/v1":"f1875bbeef8244021f99d12b24384592b47ed866fce50d16c87190a639fde423"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.ProtocolItem_PreviewMouseLeftButtonDown (cognitive 24): Protocol.ProtocolItem_PreviewMouseLeftButtonDown has cognitive complexity 24 (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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"66a1b5fabffa9e200bbf5e9cd9ead924a5d967e5e2bacf05aef653b76ec1001d"}},{"ruleId":"D2","level":"warning","message":{"text":"SketchArea.SetComponentLabels (cognitive 23): SketchArea.SetComponentLabels has cognitive complexity 23 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":405}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa448bbc7154beb8e1169673a9c314b51b1a7640696af6a7b63bc30d1f20e0df"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.AutoSave (cognitive 22): Protocol.AutoSave has cognitive complexity 22 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"58f17b61611538736efa86a671fc17ec1737d454c8930511a6a01d6beb133235"}},{"ruleId":"D2","level":"warning","message":{"text":"dlgConnectGraph.PopulateSequenceScheme (cognitive 22): dlgConnectGraph.PopulateSequenceScheme has cognitive complexity 22 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/dlgConnectGraph.xaml.vb"},"region":{"startLine":254}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f3062b0b5c20ef79a0c6d15651e4f00152fa6e1627e173b6f8a5eb6364f7f2d"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.UpdateProductsResinInfo (cognitive 21): Protocol.UpdateProductsResinInfo has cognitive complexity 21 (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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1607}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c54446ce4c812a5d685e2b5926c42b572b102558ce22860dbb7dd66bc8c6d73"}},{"ruleId":"D2","level":"warning","message":{"text":"ExperimentPrint.Print (cognitive 20): ExperimentPrint.Print has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/ExperimentPrint.vb"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"10876471f2adafaa4ee627fc85b26687987874669bf2cabb0cc0b21fc7971aa4"}},{"ruleId":"D2","level":"warning","message":{"text":"ExperimentPrint.ExperimentsToPdfZip (cognitive 20): ExperimentPrint.ExperimentsToPdfZip has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/ExperimentPrint.vb"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a06ca1307e9abc44228f9925bbae3ca954b066ab0c013967ae59ce45cadf944"}},{"ruleId":"D2","level":"warning","message":{"text":"ProtocolDropHandler.RearrangeDropSequence (cognitive 20): ProtocolDropHandler.RearrangeDropSequence has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1715}}}],"partialFingerprints":{"codehealthFindingId/v1":"44e6fc04df0c57b620ec5c829572cf9f3ae229d6aad20f2cd24b6aab322ae69a"}},{"ruleId":"D2","level":"warning","message":{"text":"SequenceSchemeView.PlaceSequencePanels (cognitive 20): SequenceSchemeView.PlaceSequencePanels has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":750}}}],"partialFingerprints":{"codehealthFindingId/v1":"c58574d6861cd0f0db62d0baa2952e879e4f95739b3540f68393850fbb3c1d1b"}},{"ruleId":"D2","level":"warning","message":{"text":"TransferPackage.ImportMySettings (cognitive 20): TransferPackage.ImportMySettings has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/TransferPack/TransferPackage.vb"},"region":{"startLine":295}}}],"partialFingerprints":{"codehealthFindingId/v1":"4569842fcfcb5b3d786f2aaec8c4764adc755d662aa5bc4620076c3817e304a6"}},{"ruleId":"D2","level":"warning","message":{"text":"ELNCalculations.RecalculateMaterials (cognitive 20): ELNCalculations.RecalculateMaterials has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ELNCalculations.vb"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"45ec06c3285cc960b275a20b7556975fe6b4c7b21e19758a9e1554b152ab8549"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.EmbedDocument (cognitive 19): Protocol.EmbedDocument 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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":977}}}],"partialFingerprints":{"codehealthFindingId/v1":"b75b83769a88f48633985b30a6c4a0bbad298689f666aaacbc024f274e7a7d38"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.SerializeFile (cognitive 19): Protocol.SerializeFile 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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1061}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb469694cfc5ebe5b2a5bef8b0f3bb5a29eafacc49eb5405d56ee50aa871b247"}},{"ruleId":"D2","level":"warning","message":{"text":"SketchArea.EditSketch (cognitive 19): SketchArea.EditSketch 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":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"da78487c9fe1219edeea90a210e6c108ec212ef8d45b85ce206136b358110a63"}},{"ruleId":"D2","level":"warning","message":{"text":"SketchArea.RemoveDeletedSideProducts (cognitive 19): SketchArea.RemoveDeletedSideProducts 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":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":197}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b3e44013fba79735b8067a585a6f11d3462272d93ef587d2afe618ff380f269"}},{"ruleId":"D2","level":"warning","message":{"text":"ExperimentBase.ImportExperiment (cognitive 19): ExperimentBase.ImportExperiment 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":"ElnBase/ExperimentBase.vb"},"region":{"startLine":205}}}],"partialFingerprints":{"codehealthFindingId/v1":"c4881326ffd79163d08a0c7c36d02a316e147cf91f7faa682f3f7fd3281925c4"}},{"ruleId":"D2","level":"warning","message":{"text":"FullTextSearch.ExtendHighlightPastClosingBrackets (cognitive 19): FullTextSearch.ExtendHighlightPastClosingBrackets 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":"ElnBase/FullTextSearch.vb"},"region":{"startLine":1650}}}],"partialFingerprints":{"codehealthFindingId/v1":"ecb3fda30ed0672f2eadb28190546c8b66604f6fc86c474f901da8a9901a58a8"}},{"ruleId":"D2","level":"warning","message":{"text":"ServerSync.SynchronizeAsync (cognitive 18): ServerSync.SynchronizeAsync has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ServerSync.vb"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"66bc1124e948173da8df7e97dbb5641cdf9877f07419129b654776b2f9ae07e9"}},{"ruleId":"D2","level":"warning","message":{"text":"NumericTextBox.Me_PreviewTextInput (cognitive 17): NumericTextBox.Me_PreviewTextInput 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":"CustomControls/CoreControls/NumericTextBox.vb"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"a24f0306e4ace5b95a0acdb269091660a0759558d1ba587f442023180b163cfd"}},{"ruleId":"D2","level":"warning","message":{"text":"MaterialsSummary.GetReagentGroups (cognitive 17): MaterialsSummary.GetReagentGroups 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":"CustomControls/Material Summary/MaterialsSummary.vb"},"region":{"startLine":61}}}],"partialFingerprints":{"codehealthFindingId/v1":"21bd725d3f0bb004ce7fa49785ab947b9097ddca49892a5d1e99818835f3aa2e"}},{"ruleId":"D2","level":"warning","message":{"text":"CommentContent.rtbComments_Pasting (cognitive 17): CommentContent.rtbComments_Pasting 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":"CustomControls/Protocol Elements/Others/CommentContent.xaml.vb"},"region":{"startLine":205}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ee7baa35d9113c4ba19e7b55d7db16fbfb95a9d125045f994f794b155dcc0fd"}},{"ruleId":"D2","level":"warning","message":{"text":"ELNCalculations.SignificantDigitsString (cognitive 17): ELNCalculations.SignificantDigitsString 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":"ElnBase/ELNCalculations.vb"},"region":{"startLine":718}}}],"partialFingerprints":{"codehealthFindingId/v1":"71ab8a2031b40704c3e590b156d0956acf9c6b95af9aea680d58d7079532c4ec"}},{"ruleId":"D2","level":"warning","message":{"text":"MySqlBulkUpload.ConvertDbValueToString (cognitive 17): MySqlBulkUpload.ConvertDbValueToString 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":"ElnBase/MySqlBulkUpload.vb"},"region":{"startLine":252}}}],"partialFingerprints":{"codehealthFindingId/v1":"3452c2d78160503eb7a9f1b01f87559a4816192ecab4035193bd2efb30f2ef91"}},{"ruleId":"D2","level":"warning","message":{"text":"Users.MergeConflictingUserExp (cognitive 16): Users.MergeConflictingUserExp has cognitive complexity 16 (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":"ELN-Main/Users.vb"},"region":{"startLine":244}}}],"partialFingerprints":{"codehealthFindingId/v1":"d96619783219070a2b690dbe85c6c36ae0bca606bd9c1189306959041296f369"}},{"ruleId":"D2","level":"warning","message":{"text":"Protocol.DuplicateProtocolItem (cognitive 16): Protocol.DuplicateProtocolItem has cognitive complexity 16 (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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1256}}}],"partialFingerprints":{"codehealthFindingId/v1":"06b55aa8c538cb819d0dcbce83a0f49035ba9fcfc9b81d7296c1da4cdc7ed45f"}},{"ruleId":"D2","level":"warning","message":{"text":"ELNCalculations.RecalculateReagent (cognitive 16): ELNCalculations.RecalculateReagent has cognitive complexity 16 (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":"ElnBase/ELNCalculations.vb"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"c254689ee3aef7dd768a269188d05d20e82a8d66ce048ae592e455695e467b01"}},{"ruleId":"D2","level":"warning","message":{"text":"ReactionQuery.FilterRssHits (cognitive 16): ReactionQuery.FilterRssHits has cognitive complexity 16 (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":"ElnBase/ReactionQuery.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"1bfd454043a2c41b3d17922d40a202238c38df42ba46e2e965e900c9e864447c"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ElnDataContext.OnModelCreating: MethodTooLong \u2014 OnModelCreating runs 281 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 181 over it, 2.81\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnCoreModel/ELNModel/ElnDataContext.cs"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"2489fd3226eb1d816190a0b51de2d3cf4645b096f8e28921d04016f0577a3945"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: MainWindow: TooManyMethods \u2014 1211 significant lines (blank, comment-only and punctuation-only lines excluded), 83 methods. The bar is 30 methods; this is 53 over it, 2.77\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b83ebd7c1dc966fb5f7b018c2fee970f5d2c5b35e0cff6868bad993635b59b6"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: ELN-Main/MainWindow.xaml.vb: FileTooLong \u2014 1221 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 721 over it, 2.44\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d301ff80fee65d95ba458c58f3ab6c37aa75e6e9e44822d5a8df7a3e5064824"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: CustomControls/Protocol.xaml.vb: FileTooLong \u2014 1000 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 500 over it, 2.00\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"22d1b9cf8ec7bca512b3732a50ea6277fdc9908a6bcbd9f45dd09d20b1b069db"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: SequenceGraph: ClassTooLong \u2014 729 significant lines (blank, comment-only and punctuation-only lines excluded), 24 methods. The bar is 400 significant lines; this is 329 over it, 1.82\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e10d5555314f940d968c9ecfdc634fc14d4bea3e58ffa39a232301d635af1e8"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Sequences/SequenceSchemeView.xaml.vb: FileTooLong \u2014 851 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 351 over it, 1.70\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8379a3db73282081e199a57ea933b582efd755f9ab7d03d946c9bd405e3a0f39"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: MainWindow.Me_Loaded: MethodTooLong \u2014 Me_Loaded runs 158 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 58 over it, 1.58\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"f74c84a57e8b6d61144ee322f4f6b8e0540513720595db33fce3f70f985c5c6b"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Protocol: TooManyMethods \u2014 917 significant lines (blank, comment-only and punctuation-only lines excluded), 47 methods. The bar is 30 methods; this is 17 over it, 1.57\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce44ab12d6c95151d920334da104b9d35ab13f812866219c1f2224887a929cbe"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: ElnBase/FullTextSearch.vb: FileTooLong \u2014 773 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 273 over it, 1.55\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/FullTextSearch.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3624f06259d16501fd1704475de3ba51b15c68c1c80f2a99fdb46258d1f4e42e"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Sequences/SequenceGraph.xaml.vb: FileTooLong \u2014 736 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 236 over it, 1.47\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c72b15628129f527cb25df0816f525f9092970a272d8c62f697ff636f5eb19c"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: SpellChecker: ClassTooLong \u2014 568 significant lines (blank, comment-only and punctuation-only lines excluded), 10 methods. The bar is 400 significant lines; this is 168 over it, 1.42\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SpellChecker.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e51038181a8322a8cf22c3c22640481c38823435199fe7760bbaaf20f08f8c7"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: FullTextSearch: TooManyMethods \u2014 752 significant lines (blank, comment-only and punctuation-only lines excluded), 41 methods. The bar is 30 methods; this is 11 over it, 1.37\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/FullTextSearch.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"80b84d9f03460ba14d5801ee80f0c4ce3cb16df2905cc5b203ec941b05cccad3"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: ExperimentTree: TooManyMethods \u2014 325 significant lines (blank, comment-only and punctuation-only lines excluded), 40 methods. The bar is 30 methods; this is 10 over it, 1.33\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"627c287f5595fd1f0cb14f07bdd321189b534d2f4da9ce62a0695a6b1ca8d74d"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ThemedComboBox: ClassTooLong \u2014 493 significant lines (blank, comment-only and punctuation-only lines excluded), 9 methods. The bar is 400 significant lines; this is 93 over it, 1.23\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/CoreControls/ThemedComboBox.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"01cc992dad6ca6ac255ed6dcecb5f9b8d06b9c70d753452efd9016734988ea64"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: dlgSearch: TooManyMethods \u2014 405 significant lines (blank, comment-only and punctuation-only lines excluded), 35 methods. The bar is 30 methods; this is 5 over it, 1.17\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgSearch.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5f467d698dbc79061415d81d3407b01dead6081c6f4c164c2aa0b14f24c819b"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: CustomControls/SpellChecker.vb: FileTooLong \u2014 577 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 77 over it, 1.15\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SpellChecker.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9987d96c08bf4bf23bef422a1e4c207f2473e14ec266fc026eeffa78a6a6fd08"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: NavigationTree/ExperimentTree.xaml.vb: FileTooLong \u2014 553 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 53 over it, 1.11\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c0a6b4f75c0ff9ec62dd13f74ec4575aa1f518198c893e8f9fc27a9543fcc67d"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: SequenceSchemeView: TooManyMethods \u2014 837 significant lines (blank, comment-only and punctuation-only lines excluded), 33 methods. The bar is 30 methods; this is 3 over it, 1.10\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"46b11b42218c948ed5ee7d3dc611df609a2b25136a78075c7a9ce21c3f91df54"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WPFToolbox: TooManyMethods \u2014 455 significant lines (blank, comment-only and punctuation-only lines excluded), 32 methods. The bar is 30 methods; this is 2 over it, 1.07\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/WPFToolbox.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b5024b740b0d7cec25e8b01d47def50c7b7e7f226efea3cc66d7c246927e366"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: CoreControls/ThemedComboBox.xaml.vb: FileTooLong \u2014 529 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 29 over it, 1.06\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/CoreControls/ThemedComboBox.xaml.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2bf6054b7f0a64dfd341b35864b2c509a2a7a419bba3aae116a2c6de2f8c4888"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ExperimentContent.CreateExperiment: MethodTooLong \u2014 CreateExperiment runs 104 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 4 over it, 1.04\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/ExperimentContent.xaml.vb"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"f6072e25504a9408c77d0a53421ce8ce103b52dda68ad20088ea36b91d325c7a"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ELNCalculations: ClassTooLong \u2014 408 significant lines (blank, comment-only and punctuation-only lines excluded), 24 methods. The bar is 400 significant lines; this is 8 over it, 1.02\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ELNCalculations.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5a5b658bd36a5699f035fd430772ec0c61183c91118152f8e48ef4e14032a6df"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: NavTreeDropHandler.IDropTarget_DragOver: MethodTooLong \u2014 IDropTarget_DragOver runs 101 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 1 over it, 1.01\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":757}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a752c7b6c742298d71eb0f9f30043ca6233438edcce2fd89f19ac8784731fab"}},{"ruleId":"D4","level":"warning","message":{"text":"Near-duplicate member pair (45 shared lines): CustomControls/Sequences/SequenceSchemeView.xaml.vb:259-346 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:358-472 \u2014 These two members are variants of one another: 45 of their lines are already reported as duplicated blocks below, spread through both bodies rather than gathered into one. Read them as a single construct written twice. The repair is at the members\u0027 grain \u2014 factor the shared pipeline into one implementation the two call with their differences as parameters or as an injected step, or, where the difference is systematic (sync against async, one transport against another), generate one from the other. Extracting the individual blocks below is not the same fix: it leaves the two bodies in place and the next edit still has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":259}}}],"partialFingerprints":{"codehealthFindingId/v1":"40f3fab5aa5be43af92088f8b49fb733287936f529204d60b5011894ecb2c735"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 3): CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:135-152 | CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:174-191 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:135-152 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 50 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2016241b85a1c7f6b80b7d3905714c61f2a85b4b96345ff1678224f2314461d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:331-348 | CustomControls/Material Dialogs/dlgEditRefReactant.xaml.vb:159-176 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditReagent.xaml.vb"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed28ac1f49379e06b6b2bf5c06bf3dbd355eeec268ae8537af7700f305f5c1fe"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:92-108 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:92-108 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 56 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"83dc30539557953452936eac1b801ff5c09f42c086b61d9c37541b383ad2ada9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201316 lines \u00D7 2): CustomControls/Sequences/SequenceSchemeView.xaml.vb:329-344 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:423-436 \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 \u0060CustomControls/Sequences/SequenceSchemeView.xaml.vb:329\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":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":329}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a7b5e8382355073583b4735c1a5cab29d1e7b2e4ac3408873356b717dcb40d5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13\u201314 lines \u00D7 2): CustomControls/Sequences/SequenceSchemeView.xaml.vb:285-298 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:386-398 \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. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060CustomControls/Sequences/SequenceSchemeView.xaml.vb:298\u0060 calls \u0060SolidColorBrush\u0060, \u0060FromRgb\u0060 and \u0060CustomControls/Sequences/SequenceSchemeView.xaml.vb:398\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":285}}}],"partialFingerprints":{"codehealthFindingId/v1":"84ea53e78539fc1f4ea2cf74f8b364d3ebf37c6251bc84d0fde5152b0b8c30a3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12\u201314 lines \u00D7 2): CustomControls/Sequences/SequenceSchemeView.xaml.vb:312-325 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:408-419 \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 \u0060CustomControls/Sequences/SequenceSchemeView.xaml.vb:312\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":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":312}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e67fcea06af59b2cf44c83ad3284bb87e0d87858cf07b726c855e7dc26ebf33"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): ElnCoreModel/ELNModel/tblAuxiliaries.cs:7-20 | ElnCoreModel/ELNModel/tblRefReactants.cs:7-20 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060ElnCoreModel/ELNModel/tblAuxiliaries.cs:7\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":"ElnCoreModel/ELNModel/tblAuxiliaries.cs"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"72e14417b982724c20defb6f4b8eda3e5f336ef0ee67345d31671adc9abd451f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 3): CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:116-128 | CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:124-136 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:116-128 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 50 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf808442ed65551768a3c7be57346d981b66ec50ae5b7063fb44031184bcec3d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 3): CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:331-343 | CustomControls/Material Dialogs/dlgEditRefReactant.xaml.vb:159-171 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:280-292 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 63 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditReagent.xaml.vb"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2e92ec73c68edf8f900e9d1363a1934a866fa6a15fcf6405a1f241afdceb0ef"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11\u201312 lines \u00D7 2): CustomControls/Protocol Elements/Materials/ReagentContent.xaml.vb:21-32 | CustomControls/Protocol Elements/Materials/RefReactantContent.xaml.vb:13-23 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol Elements/Materials/ReagentContent.xaml.vb"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac841ff43f4f2264d4fa42e202511687d33a3c44b4144031ff688361523605ef"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:269-279 | CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:386-396 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 50 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:269\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. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:399\u0060 calls \u0060GetEnumVal\u0060, \u0060ConvertToML\u0060 and \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:281\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb"},"region":{"startLine":269}}}],"partialFingerprints":{"codehealthFindingId/v1":"a6f094f03d9a4743b2407540189b6894a2eeb44f0b4dab8c6a7b0d1b1007f37a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): CustomControls/Material Dialogs/dlgEditProduct.xaml.vb:180-190 | CustomControls/Material Dialogs/dlgEditRefReactant.xaml.vb:147-157 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditProduct.xaml.vb"},"region":{"startLine":180}}}],"partialFingerprints":{"codehealthFindingId/v1":"cade2a76a97754ea58850487298706a14ea37f3dd86f6e90d0f5eaefbcc241d3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:314-324 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:273-283 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 63 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditReagent.xaml.vb"},"region":{"startLine":314}}}],"partialFingerprints":{"codehealthFindingId/v1":"7bc7fc471231817923805f2386ce36e1520a6fbe263c0910823bcf9c5c4b341b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): ElnCoreModel/ELNModel/ElnDataContext.cs:80-87 | ElnCoreModel/ELNModel/ElnDataContext.cs:339-346 | ElnCoreModel/ELNModel/ElnDataContext.cs:357-364 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just before the matched lines, \u0060ElnCoreModel/ELNModel/ElnDataContext.cs:337\u0060 calls \u0060Property\u0060, \u0060HasColumnType\u0060 and \u0060ElnCoreModel/ELNModel/ElnDataContext.cs:77\u0060 does not \u2014 after which the two agree again for 3 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnCoreModel/ELNModel/ElnDataContext.cs"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"1330ddd800fb03f123ed3ecd18caad9cf59f983df0fc8c2ccec44cd8903eedbe"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): ElnCoreModel/ELNModel/ElnDataContext.cs:265-271 | ElnCoreModel/ELNModel/ElnDataContext.cs:393-399 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnCoreModel/ELNModel/ElnDataContext.cs"},"region":{"startLine":265}}}],"partialFingerprints":{"codehealthFindingId/v1":"e98544ec6e01f6a5ccbac9daa3ac9a505549d89d5d0d383a28480f20fc87827f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): CustomControls/Sequences/SequenceSchemeView.xaml.vb:306-311 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:402-407 \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 \u0060CustomControls/Sequences/SequenceSchemeView.xaml.vb:306\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":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":306}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e90c24c2db69be4e37aa69794d1b4b48ec0d19240ddaed8f3383c9363039395"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): CustomControls/CoreControls/ListBoxEditLabel.xaml.vb:132-139 | CustomControls/CoreControls/TreeViewEditLabel.xaml.vb:192-199 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/CoreControls/ListBoxEditLabel.xaml.vb"},"region":{"startLine":132}}}],"partialFingerprints":{"codehealthFindingId/v1":"b24318620e8b066950845745f1ef507dbb3cf19ba39fb7be967971ee76960775"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): CustomControls/dlgFinalize.xaml.vb:50-55 | CustomControls/dlgRestoreServer.xaml.vb:52-57 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgFinalize.xaml.vb"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e5ce2e974b8c99ba3668f05694e889171a75f671bafdfb16a6c8105cd97c6e7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb:296-303 | CustomControls/Material Dialogs/dlgEditReagent.xaml.vb:209-216 | CustomControls/Material Dialogs/dlgEditSolvent.xaml.vb:303-310 \u2014 before extracting anything, compare \u0060CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb\u0060 and \u0060CustomControls/Material Dialogs/dlgEditReagent.xaml.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 50 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Material Dialogs/dlgEditAuxiliary.xaml.vb"},"region":{"startLine":296}}}],"partialFingerprints":{"codehealthFindingId/v1":"663e0e2fe2a9490dbdc5c1cc8912d2117d36b8a82c88125b5a5e3a5bb672f94b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): CustomControls/Material Dialogs/MatDocControl.xaml.vb:312-318 | CustomControls/Protocol Elements/Others/FileContent.xaml.vb:226-232 \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":"CustomControls/Material Dialogs/MatDocControl.xaml.vb"},"region":{"startLine":312}}}],"partialFingerprints":{"codehealthFindingId/v1":"c17f2654495bb498963de12c5cfe51ed24505671a70f86dfeecb4b056e6a5b21"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): CustomControls/Sequences/SequenceGraph.xaml.vb:315-321 | CustomControls/Sequences/SequenceSchemeView.xaml.vb:595-600 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceGraph.xaml.vb"},"region":{"startLine":315}}}],"partialFingerprints":{"codehealthFindingId/v1":"7cf53b9be816588ee596cc54cd49b5222628281717f6ffeec710de4fe2e331d2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): CustomControls/SketchArea.xaml.vb:503-513 | CustomControls/SketchAreaRSS.xaml.vb:171-181 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":503}}}],"partialFingerprints":{"codehealthFindingId/v1":"406d9111d4dfd496ea0cf57a7cb311c5ac6c589b2dd77a5723621798cb9cb48a"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: ElnCoreModel: ElnCoreModel: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 3 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"40bf5cb349f66aa50068902670e84edc6380fa0a000c5983cdcfc1799096a543"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: MainWindow (LCOM4 16): MainWindow\u0027s methods fall into 16 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 16 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"6743ff3d5cbfaf6742c944e88397970a65ad68c771c26ac7c22a87679ee539b3"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ExperimentTree (LCOM4 12): ExperimentTree\u0027s methods fall into 12 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 12 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c79a4267a95daabc591d5e865e290b75fc13fa443f0175b28e344808886847d"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ExpTreeHeader (LCOM4 6): ExpTreeHeader\u0027s methods fall into 6 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 6 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExpTreeHeader.xaml.vb"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"d862723ad287deb288799143565151a6c2d10f997656501dc236bd0ee9e25f44"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: dlgNewUser (LCOM4 5): dlgNewUser\u0027s methods fall into 5 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 5 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/dlgNewUser.xaml.vb"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"fdc2bf72789ef95075d4c4358ecd28a193dabde37343c7c61aca782d0fba04ed"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: dlgServerConnection (LCOM4 5): dlgServerConnection\u0027s methods fall into 5 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 5 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgServerConnection.xaml.vb"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7684fc9d4f3b96d5885b071211ef5ba326bbb3b8960210bc893e1de31ae2ca6"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: StepSummary (LCOM4 5): StepSummary\u0027s methods fall into 5 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 5 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/StepSummary.xaml.vb"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7e826034f6a10faa8ae54c902f0bb6eef33ddcc6950955552bba08ea49cc5c1"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: dlgFullTextSearch (LCOM4 4): dlgFullTextSearch\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgFullTextSearch.xaml.vb"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"793f6c142bb4794a9fe8541d7d286b80114434ceb0eebb8b1d162451a7cc4e2e"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: dlgSearch (LCOM4 4): dlgSearch\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgSearch.xaml.vb"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"e25c26dc2933a382b6e5e8fa8e808a1b7b4760b3ba925102ca55e892728905b2"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: dlgTagSearch (LCOM4 4): dlgTagSearch\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/dlgTagSearch.xaml.vb"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"22b3093d3989ded5d08f6854117ef8654e490e6c0ba051ae89895424eba23dcc"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ExperimentContent (LCOM4 4): ExperimentContent\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/ExperimentContent.xaml.vb"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"13d84993181c5a6dd68bd6d1d9b70e8bf5af64abd4359f9d7be50e40c26608a1"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ImageContent (LCOM4 4): ImageContent\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol Elements/Others/ImageContent.xaml.vb"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff416bf13662c7ec51d15276d0704a5cf6d10eccbd162f5648856fe828fe2e1a"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SequenceStructure (LCOM4 4): SequenceStructure\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceStructure.xaml.vb"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"2bbddbd78e873964634262afc1bc5e5e38eb15402842422807cf074ffb5fec28"}},{"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. Start with the code you change most often: add a suite in a framework a runner can collect (xUnit, NUnit or MSTest), and run it in CI so the gap cannot reopen."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3132564c6310e5d01a231f252a02d5d2f5a542c0a8fa29a14c89693f1e3df871"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test suite could be collected \u2014 no discoverable tests to count. If this repository does test, wiring the suite to a framework a runner can collect (xUnit, NUnit or MSTest) is what makes it countable here; a pipeline step that invokes a runner is not evidence on its own, because a runner over an empty suite passes. 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":"D15","level":"warning","message":{"text":"Hotspot: ELN-Main/MainWindow.xaml.vb: ELN-Main/MainWindow.xaml.vb changed 35 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 25 in MainWindow.Me_Loaded at line 135. 9 of those changes were fix/bug commits, and the other 26 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- ELN-Main/MainWindow.xaml.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"7ff1ec31e317f2f75bc248e898edca1153d4d3ed44d313a4d7d28f6aaef7dd90"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: CustomControls/NavigationTree/ExperimentTree.xaml.vb: CustomControls/NavigationTree/ExperimentTree.xaml.vb changed 16 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 53 in NavTreeDropHandler.IDropTarget_DragOver at line 757. 2 of those changes were fix/bug commits, and the other 14 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- CustomControls/NavigationTree/ExperimentTree.xaml.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/NavigationTree/ExperimentTree.xaml.vb"},"region":{"startLine":757}}}],"partialFingerprints":{"codehealthFindingId/v1":"5cc95d3f7880988e5aed3d5e340313d9ae8accd8b1bf2ec6f2cd82c825723d51"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: CustomControls/Sequences/dlgConnectGraph.xaml.vb: CustomControls/Sequences/dlgConnectGraph.xaml.vb changed 6 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 18 in dlgConnectGraph.PopulateSequenceScheme at line 254. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- CustomControls/Sequences/dlgConnectGraph.xaml.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/dlgConnectGraph.xaml.vb"},"region":{"startLine":254}}}],"partialFingerprints":{"codehealthFindingId/v1":"cee912981fc0aecafed95c901953fb8ea2d1e3ad2de253cb9aa883d6ee9cea52"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: ElnBase/ExperimentBase.vb: ElnBase/ExperimentBase.vb changed 3 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 16 in ExperimentBase.CloneExperiment at line 68. 2 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- ElnBase/ExperimentBase.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/ExperimentBase.vb"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fa9a9f41cce8f8fa5f45c67cdc65381f095df92e0ced95a1024bd66e91d9b4f"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: ElnBase/RestoreFromServer.vb: ElnBase/RestoreFromServer.vb changed 3 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 16 in RestoreFromServer.ProcessChildren at line 79. 2 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- ElnBase/RestoreFromServer.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/RestoreFromServer.vb"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"b1acbbf93340735d72cfa68610b7c9f6de551ac8c02155e4594161d0f9073fdf"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: CustomControls/Sequences/SequenceSchemeView.xaml.vb: CustomControls/Sequences/SequenceSchemeView.xaml.vb changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 19 in SequenceSchemeView.DrawConnectors at line 883. 1 of those changes was a fix/bug commit, and the other 1 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- CustomControls/Sequences/SequenceSchemeView.xaml.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Sequences/SequenceSchemeView.xaml.vb"},"region":{"startLine":883}}}],"partialFingerprints":{"codehealthFindingId/v1":"e75c29f0c77cf237b6fa22ea95c393687f012cce0d4f306ae268faaa2f650fde"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: CustomControls/SketchArea.xaml.vb: CustomControls/SketchArea.xaml.vb changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 16 in SketchArea.RemoveDeletedSideProducts at line 197. 1 of those changes was a fix/bug commit, and the other 1 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 13:02:07 \u002B02:00\u0027 --until=\u00272026-09-24 13:02:07 \u002B02:00\u0027 --full-history --no-merges -- CustomControls/SketchArea.xaml.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/SketchArea.xaml.vb"},"region":{"startLine":197}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0bdaf3ab015f8c97aab1b4fca34edecfd70f377a25cfb5689fc11e67d618ecb"}},{"ruleId":"D16","level":"warning","message":{"text":"bus factor 1 \u2014 one contributor carries this repository: 2 contributor(s) have commits in this repository (automation and bot accounts excluded), but the commits of a single one of them cover most of its history \u2014 85% of the 543 commit(s) sampled \u2014 so the number of people who actually carry it is 1. This is not the ambient state of a solo project: there ARE other contributors, and if the one who carries the repository becomes unavailable, the knowledge behind most of its history goes with them. Spread the next changes deliberately \u2014 have a second contributor review or pair on them \u2014 and write down the parts only one person can currently change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a99f114ab4dfee37ae0bf0665b44c7468d85b809d361949e2b6258101fa1e5a2"}},{"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":"CustomControls/ExperimentPrint.vb"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"d46db1674ab2f3a827fa53bcaf465428f3350e64447361e788513dc0bdc318de"}},{"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":"CustomControls/ExperimentPrint.vb"},"region":{"startLine":199}}}],"partialFingerprints":{"codehealthFindingId/v1":"b451d1faf297a2725d5e95b83c0aee9fbeba190ff77f841d989fee2c442f20ec"}},{"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":"CustomControls/Protocol Elements/Others/FileContent.xaml.vb"},"region":{"startLine":453}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec90b252b91b716ef1ac6a06638bedd41ed419f5eac9789efcd80f4c9e1b14b1"}},{"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":"CustomControls/Protocol Elements/Others/ImageContent.xaml.vb"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"694d70affe0fe907708190561c2bd623d9250273cf095bee861c411cfd61f684"}},{"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":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"01f7dbc1bf32a6aa230ba906c31cedf7bd98a678d7a4368c716c495f09562b25"}},{"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":"CustomControls/Sequences/dlgConnectGraph.xaml.vb"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"23034aa9820814c5ddff8c6f9b5e98154031288178e28671e9a5b07d09f8d7f0"}},{"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":"CustomControls/Sequences/dlgConnectGraph.xaml.vb"},"region":{"startLine":533}}}],"partialFingerprints":{"codehealthFindingId/v1":"23b8e3916d756f2fe15eff5502b36db680f8d299aaa6388cd0d172450130fc43"}},{"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":"CustomControls/SpellChecker.vb"},"region":{"startLine":216}}}],"partialFingerprints":{"codehealthFindingId/v1":"bdc783d40af88580e2b49b69c498f9cf25a31c60c9ff23fb72ef1983eabf87c6"}},{"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":"CustomControls/SpellChecker.vb"},"region":{"startLine":236}}}],"partialFingerprints":{"codehealthFindingId/v1":"3aa28fe2b46a8467955d744e2b43dbfb9d4fe946f0aad11fbcc87c0250ccd329"}},{"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":"CustomControls/SpellChecker.vb"},"region":{"startLine":832}}}],"partialFingerprints":{"codehealthFindingId/v1":"dce6e3526158dfad6bd12b9b70edb02465d3f32e517f93927f60f842f77940d9"}},{"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":"CustomControls/WPFToolbox.vb"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"1cfad486c4113978f91b7c3ca0085f7e1a2f3416c20307dc952bef9580e8e10b"}},{"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":"ELN-Main/Application.xaml.vb"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"6ef7f54fe232b9b2bbf32a18ebb8a01ef84054ea7a846dc5ab353a70fc7c915c"}},{"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":"ELN-Main/MainWindow.xaml.vb"},"region":{"startLine":1547}}}],"partialFingerprints":{"codehealthFindingId/v1":"b20d365c86a727e5d69feeb8929c686bbf224d510c2a1dcc3b4d49ccd66af353"}},{"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":"ElnBase/RestoreFromServer.vb"},"region":{"startLine":162}}}],"partialFingerprints":{"codehealthFindingId/v1":"f813b39ad2e172c68c69437f78717545c46fd07e0b01c98447358871f17c329c"}},{"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":"ElnBase/ServerSync.vb"},"region":{"startLine":309}}}],"partialFingerprints":{"codehealthFindingId/v1":"fdf65cf6ca538467cf77d1a02da6a5fe529e585433fc5ac74ba366507b8d2236"}},{"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":"ElnBase/ServerSync.vb"},"region":{"startLine":392}}}],"partialFingerprints":{"codehealthFindingId/v1":"0aa63252c0228908301f42571ca1e32539231c48aeb9d72ac9fc0aca369f7f17"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027TODO: Add actual link target \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Notifications/StatusInfoPanel.xaml.vb"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ae435bce4fefacaea7391aed5b0b765bdef29fe9035f5f76197d1c3d6dc1805"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027TODO: Consider multiple refProduct portions AND multiple *additional* (i.e. non-ref.) products! \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CustomControls/Protocol.xaml.vb"},"region":{"startLine":1609}}}],"partialFingerprints":{"codehealthFindingId/v1":"0430df38de95e9616e441017194d37a3b645051bec6503ca21f110c9f2d3fa54"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027TODO -- replace LOCAL projEntry by SERVER projEntry \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ELN-Main/Users.vb"},"region":{"startLine":290}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2d1f6ee60452766becb1749c0a45440153649b2df930a55e7bfbccb31576b6c"}},{"ruleId":"D17","level":"error","message":{"text":"BlanketAnalyzerSeverityNone: dotnet_analyzer_diagnostic.severity = none \u2014 every analyzer rule \u2014 this is not one rule switched off, it is a whole set the file never lists: every rule the entry covers is silenced for every file the section matches, in every project under this directory, and any rule added to that set by a future analyzer upgrade is silenced too, without anybody deciding so. It reads as narrower than it is \u2014 the analyzers still run and the build still looks configured \u2014 and because no rule is named there is nothing for a reader to look up and no list that will ever disclose what is covered. Replace it with the rules you actually mean to switch off, each on its own line, so the set is finite, visible and reviewable, and the ones nobody chose keep protecting the tree. If the blanket entry is genuinely right, link the decision from a comment above it: a comment that only restates the entry is what the editor generates when it writes this line, so it is a reference to the decision \u2014 an issue or a ticket \u2014 that clears this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":".editorconfig"},"region":{"startLine":4}}}],"partialFingerprints":{"codehealthFindingId/v1":"802593472310deeb53d8b4c6969d3f7626b4107cc3f24974bb571842169118f6"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 1 of 4 project(s) overshoot their size bounds, lowering Project Cohesion to 5.0/10. The most over is \u0060CustomControls\u0060 (16966 LoC, 203 public types across 1 namespaces). Review these for cohesion \u2014 split a project that spans unrelated responsibilities."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DbUpgradeLocal.vb \u2194 DbUpgradeServer.vb: \u0060ElnBase/DbUpgradeLocal.vb\u0060 and \u0060ElnBase/DbUpgradeServer.vb\u0060 change together 50% of the time (6 of the 12 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u00602ebfd7c4\u0060 Implemented database infrastructure for experiment tags.; \u00601d5fab58\u0060 Introduced tblUsers.SequenceNr. Fixed iswsue with missing default val\u2026; \u00608f5bbab3\u0060 Now remembers last selected user. \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"ElnBase/DbUpgradeLocal.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b6dd530a44cf7ff853785631a91438ca416c6e8241e7481ec187d9c41dc1953c"}},{"ruleId":"C1","level":"warning","message":{"text":"No data-protection/encryption: No data-protection or encryption usage (ASP.NET Data Protection, AES, column encryption, PBKDF2) was found \u2014 sensitive data at rest may be unprotected. If TDE/KMS/vault is delegated to infrastructure, ignore."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3674448ef29fbd006d82bb49ec9615cf1ee508105aa1d952555aabaf3e10a9f9"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M2","level":"note","message":{"text":"No architecture diagram/doc: No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named \u0060architecture\u0060 or \u0060design\u0060 in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape \u2014 no \u0060## Architecture\u0060 heading, no \u0022architecture overview\u0022/\u0022high-level design\u0022 phrasing, no \u0022the architecture is \u2026\u0022 introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document \u2014 not evidence that nobody wrote the shape down."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0c190e7c159d1850ee706c3ac4486e151e8a4fe169de4bf61436b9f399654f06"}},{"ruleId":"M3","level":"note","message":{"text":"No src/ separation: Production code isn\u0027t grouped under a src/ folder \u2014 it\u0027s spread across several top-level directories, so there\u0027s no one place that says \u0027this is the product\u0027."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fdf7f5b24e313364d10170a5c2542959902fe0d26ed70edef3eaa10ec5bcdb1d"}},{"ruleId":"M3","level":"note","message":{"text":"Inconsistent root namespaces: Only 1/4 projects share a common root namespace \u2014 the code\u0027s module identity is inconsistent."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c8031c9fb77ad97c2e4a8ddb29978f4bf30140d481913bd7fde8b5adbbb83ede"}},{"ruleId":"P1","level":"warning","message":{"text":"No CI pipeline: No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, \u2026) \u2014 changes aren\u0027t gated by an automated build/test."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"44f01af96e50474fba2df84d95ddf4f7e1d34c29c307830b9efc9057daa6b670"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add a Roslyn security analyzer package referenced from the project (the analyzer packages do analyse VB.NET), plus gitleaks for committed secrets \u2014 CodeQL has no VB.NET extractor, so its csharp pack would extract nothing from this tree and then report it clean \u2014 this repository has no CI pipeline yet, so run it locally to clear the existing findings, then make it a step of the first workflow you add so a regression fails the build. What was searched, so you can tell an absence from a miss: the 0 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}},{"ruleId":"P6","level":"note","message":{"text":"No changelog: No CHANGELOG/HISTORY/RELEASES file \u2014 what shipped when isn\u0027t easy to reconstruct for support or audit. (Versioning/tagging makes releases traceable, but a changelog records the what.)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dda5aa5aed8cbb292c3ef2b733bc138f614293ae6426c85e98bf31ef330d9415"}},{"ruleId":"PF1","level":"note","message":{"text":"No performance benchmarks: No benchmark suite was found by the two searches this check runs. FIRST, a BenchmarkDotNet package reference in any project file \u2014 every project the workspace loaded, plus a walk of project files on disk that never loaded, because a benchmark suite is exactly the project a partial load drops. SECOND, a script harness: a file whose name contains \u0060benchmark\u0060 and ends \u0060.py\u0060, \u0060.sh\u0060, \u0060.ps1\u0060, \u0060.bash\u0060, \u0060.rb\u0060, \u0060.js\u0060 or \u0060.mjs\u0060, credited ONLY when this repository\u0027s CI text also mentions benchmarks \u2014 a harness no pipeline runs is read as a fixture, deliberately. Neither search can see a benchmark suite in another ecosystem\u0027s idiom (a Go \u0060testing.B\u0060 file, a JMH or criterion project, a pytest-benchmark run), so this is \u0027no benchmark found by those two searches\u0027, not a verdict that the repository has none. Where code is performance-sensitive, a benchmark guards against silent regressions \u2014 but it\u0027s a bonus here, not a deduction."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fd90d18ee3bb127e8033d41e8efe4e131487a6d794a5730a72c56105ef7485ea"}},{"ruleId":"PF2","level":"note","message":{"text":"No allocation-aware APIs detected: No Span/Memory, pooling (ArrayPool/ObjectPool), stackalloc, ValueTask or buffer-writer usage was found. If this code sits on a hot path, these reduce the GC pressure it creates \u2014 a bonus, not a requirement."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62ddee2db21224079bdf78941df83f75eb632dc1a76b0da3669d47246d8251ea"}},{"ruleId":"SC1","level":"warning","message":{"text":"NuGet dependencies are not locked: No packages.lock.json and no central package management \u2014 restores aren\u0027t reproducible or pinned (SSDF PW.4.4). Enable \u003CRestorePackagesWithLockFile\u003Etrue\u003C/RestorePackagesWithLockFile\u003E (commit the lockfile) or adopt Directory.Packages.props. Advisory \u2014 never scored."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0a97b4f69ccac509400ece7eedefb06d3ede0522cd4d8bcb5c068bea719545a6"}},{"ruleId":"X5","level":"note","message":{"text":"Null-forgiving operator (\u0060!\u0060) suppressions reduce the NRT score: ~12.5 \u0060!\u0060 suppressions per 1k syntax nodes \u2014 78 suppression(s) across the 6230 syntax node(s) in code where nullable warnings are ENABLED, which is the only code a \u0060!\u0060 can suppress anything in (a \u0060!\u0060 under \u0060#nullable disable\u0060 is inert and is not counted, and its file\u0027s nodes are not in the denominator). Each one tells the compiler to trust you about null, suppressing the very safety NRTs provide."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"93cfe05d4ad8c8c171267603b23dfe289b74842e38edd1b7bfed59cc32bda337"}}],"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}}}]}