{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"id":"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"]},{"target":{"id":"A07:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"],"owasp":"A07:2025"}},{"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":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D27","name":"Navigability","shortDescription":{"text":"Navigability"},"helpUri":"https://codehealth.canine.dev/dimensions/D27"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"A07:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"],"owasp":"A07:2025"}},{"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"]},{"target":{"id":"A05:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"],"owasp":"A05:2025"}},{"id":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"],"owasp":"A03:2025"}},{"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":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"],"owasp":"A03:2025"}},{"id":"D39","name":"IL Efficiency","shortDescription":{"text":"IL Efficiency"},"helpUri":"https://codehealth.canine.dev/dimensions/D39"},{"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":"AX8","name":"Test isolation","shortDescription":{"text":"Test isolation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX8"},{"id":"AXB1","name":"Runtime evidence locked \u2014 no reproducible boot","shortDescription":{"text":"Runtime evidence locked \u2014 no reproducible boot"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB1"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"ED5","name":"Idempotency","shortDescription":{"text":"Idempotency"},"helpUri":"https://codehealth.canine.dev/dimensions/ED5"},{"id":"GD1","name":"Unfinished \u0026 placeholder code","shortDescription":{"text":"Unfinished \u0026 placeholder code"},"helpUri":"https://codehealth.canine.dev/dimensions/GD1"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P12","name":"CI test-gate honesty","shortDescription":{"text":"CI test-gate honesty"},"helpUri":"https://codehealth.canine.dev/dimensions/P12"},{"id":"P2","name":"Observability","shortDescription":{"text":"Observability"},"helpUri":"https://codehealth.canine.dev/dimensions/P2"},{"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":"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":"X10","name":"Duplicated predicate","shortDescription":{"text":"Duplicated predicate"},"helpUri":"https://codehealth.canine.dev/dimensions/X10"},{"id":"X20","name":"Mistyped argument guard","shortDescription":{"text":"Mistyped argument guard"},"helpUri":"https://codehealth.canine.dev/dimensions/X20"},{"id":"X23","name":"Unguarded diagnostic materialisation","shortDescription":{"text":"Unguarded diagnostic materialisation"},"helpUri":"https://codehealth.canine.dev/dimensions/X23"},{"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":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"ServerStateLoop.processServerEvent (cyclomatic 72): ServerStateLoop.processServerEvent has cyclomatic complexity 72 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/ServerStateLoop.fs"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba5b1b229fc22f5cadc81d81cf965524ae4c1cd4e811dcc47a0b3eb031d0265c"}},{"ruleId":"D1","level":"warning","message":{"text":"InlayHint.toInlayHint (cyclomatic 57): InlayHint.toInlayHint has cyclomatic complexity 57 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/InlayHint.fs"},"region":{"startLine":238}}}],"partialFingerprints":{"codehealthFindingId/v1":"37e83916f846bb906b0a181fc3b6efd0facfac47947b632ab0236b10e904cab0"}},{"ruleId":"D1","level":"warning","message":{"text":"JsonRpc.processEvent (cyclomatic 40): JsonRpc.processEvent has cyclomatic complexity 40 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/JsonRpc.fs"},"region":{"startLine":674}}}],"partialFingerprints":{"codehealthFindingId/v1":"69351be567c053de0e91b899f7cafb855610669ad89d5130a83330d5a363c9aa"}},{"ruleId":"D1","level":"warning","message":{"text":"ErasedUnionConverter\u00601.Read (cyclomatic 33): ErasedUnionConverter\u00601.Read has cyclomatic complexity 33 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/src/JsonUtils.fs"},"region":{"startLine":428}}}],"partialFingerprints":{"codehealthFindingId/v1":"054e07d55e72199b3f96a0ce8d6e4699d8478e1ac385dc9beb3d14e9b0a639ac"}},{"ruleId":"D1","level":"warning","message":{"text":"GenerateClientServer.generateClientServer (cyclomatic 33): GenerateClientServer.generateClientServer has cyclomatic complexity 33 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a94046c8fffbe6b009fdf2e1ea0304a6c739f58c62e9985f781412c334b6946"}},{"ruleId":"D1","level":"warning","message":{"text":"SymbolKind.fromSymbol (cyclomatic 23): SymbolKind.fromSymbol has cyclomatic complexity 23 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Roslyn/Conversions.fs"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b57259c28bf6af60c1cb1e9ce6fc643e393399848bd5ed41f6e2702c013b5c4"}},{"ruleId":"D1","level":"warning","message":{"text":"CallHierarchy.outgoingCalls (cyclomatic 20): CallHierarchy.outgoingCalls has cyclomatic complexity 20 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"86aa00efc446bf49b0b7446b62568748a5e81ce10ab8015598ff92a72c1e1184"}},{"ruleId":"D1","level":"warning","message":{"text":"GenerateTypes.createField (cyclomatic 20): GenerateTypes.createField has cyclomatic complexity 20 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":181}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b47aa601d6567e78268b1fc818d60f3d1c11a00b7e44b99957e96f75c6fe811"}},{"ruleId":"D1","level":"warning","message":{"text":"GenerateTypes.createStructure (cyclomatic 20): GenerateTypes.createStructure has cyclomatic complexity 20 (threshold 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"074def76e7d6767c9db2cceda7d7c8d512ed3761bc96b50a596c005dbeb355f5"}},{"ruleId":"D1","level":"warning","message":{"text":"GenerateTypes.createTypeAlias (cyclomatic 20): GenerateTypes.createTypeAlias has cyclomatic complexity 20 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":652}}}],"partialFingerprints":{"codehealthFindingId/v1":"bd7b49d88b901668575e7157577401a4fe99d45802540b1aecf22fe4ebe5e2b7"}},{"ruleId":"D1","level":"warning","message":{"text":"Program.entry (cyclomatic 18): Program.entry has cyclomatic complexity 18 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Program.fs"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"18d7f9c448f096c9c7b00938c79351078db9822370d5c30614ad30db23ce9330"}},{"ruleId":"D1","level":"warning","message":{"text":"PushDiagnostics.processPendingPushDiagnostics (cyclomatic 18): PushDiagnostics.processPendingPushDiagnostics has cyclomatic complexity 18 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/PushDiagnostics.fs"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"68e240a0e3f94e530740bab6ff40b06299dd8d55d72097b946dedbd82c51df12"}},{"ruleId":"D1","level":"warning","message":{"text":"WorkspaceFolder.workspaceFolderDocumentDetails (cyclomatic 16): WorkspaceFolder.workspaceFolderDocumentDetails has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs"},"region":{"startLine":447}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c64bf755febf124357ace36e431f83c3380fc1fc378c92decc65f31eb97a0b6"}},{"ruleId":"D2","level":"warning","message":{"text":"ServerStateLoop.processServerEvent (cognitive 60): ServerStateLoop.processServerEvent has cognitive complexity 60 (threshold 15). Drivers by points: match/switch 19 (44 pts), if/else 7 (14 pts), boolean chains 2 (nesting depth added 32). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/ServerStateLoop.fs"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7606c541e1015f5a6181d6196f0148e65fd3a02ff03c6296d0515806f328d48"}},{"ruleId":"D2","level":"warning","message":{"text":"ErasedUnionConverter\u00601.Read (cognitive 56): ErasedUnionConverter\u00601.Read has cognitive complexity 56 (threshold 15). Drivers by points: if/else 26 (33 pts), match/switch 8 (15 pts), error handling 5 (7 pts), boolean chains 1 (nesting depth added 16). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/src/JsonUtils.fs"},"region":{"startLine":428}}}],"partialFingerprints":{"codehealthFindingId/v1":"df70f067ab06ce1d8dcee32482f77068559b2f378ba9545611ff5473430158f3"}},{"ruleId":"D2","level":"warning","message":{"text":"InlayHint.toInlayHint (cognitive 50): InlayHint.toInlayHint has cognitive complexity 50 (threshold 15). Drivers by points: match/switch 20 (32 pts), boolean chains 9, if/else 6 (9 pts) (nesting depth added 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/InlayHint.fs"},"region":{"startLine":238}}}],"partialFingerprints":{"codehealthFindingId/v1":"f873694bbfdc461fbf4c307ebbf792394a2e83fd8bfb51d625adf1c64dfde46a"}},{"ruleId":"D2","level":"warning","message":{"text":"CallHierarchy.outgoingCalls (cognitive 50): CallHierarchy.outgoingCalls has cognitive complexity 50 (threshold 15). Drivers by points: match/switch 8 (40 pts), loops 2 (9 pts), boolean chains 1 (nesting depth added 39). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb6575135611b12e925c19094bcf8e7033466331153c771664b792e82bc59309"}},{"ruleId":"D2","level":"warning","message":{"text":"GenerateClientServer.generateClientServer (cognitive 48): GenerateClientServer.generateClientServer has cognitive complexity 48 (threshold 15). Drivers by points: if/else 12 (20 pts), match/switch 7 (18 pts), loops 6 (10 pts) (nesting depth added 23). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"2d8d43df5dfe90edb219d1010865c133dead28b9df26f7e519540fbb85641a55"}},{"ruleId":"D2","level":"warning","message":{"text":"JsonRpc.processEvent (cognitive 37): JsonRpc.processEvent has cognitive complexity 37 (threshold 15). Drivers by points: match/switch 12 (26 pts), loops 2 (6 pts), if/else 2 (3 pts), error handling 1 (2 pts) (nesting depth added 20). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/JsonRpc.fs"},"region":{"startLine":674}}}],"partialFingerprints":{"codehealthFindingId/v1":"e37c6a70da06305489d14079af0d562e9a6f9aba50efbba73bfdb8ab5f1c2363"}},{"ruleId":"D2","level":"warning","message":{"text":"PushDiagnostics.processPendingPushDiagnostics (cognitive 33): PushDiagnostics.processPendingPushDiagnostics has cognitive complexity 33 (threshold 15). Drivers by points: match/switch 8 (27 pts), if/else 2 (6 pts) (nesting depth added 23). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Runtime/PushDiagnostics.fs"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"76881ca2e756c9aa9b7728632ac532b93586e1fe454737cc539ad00473e4067e"}},{"ruleId":"D2","level":"warning","message":{"text":"GenerateTypes.createStructure (cognitive 32): GenerateTypes.createStructure has cognitive complexity 32 (threshold 15). Drivers by points: loops 9 (16 pts), match/switch 7 (15 pts), error handling 1 (nesting depth added 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"14e99647f9a1a6813cfc04579a8d8510806d6bb21a8fea755e45e46fabefe116"}},{"ruleId":"D2","level":"warning","message":{"text":"Diagnostic.getWorkspaceDiagnosticReports (cognitive 26): Diagnostic.getWorkspaceDiagnosticReports has cognitive complexity 26 (threshold 15). Drivers by points: loops 5 (11 pts), if/else 4 (6 pts), match/switch 2 (4 pts), error handling 1 (3 pts), boolean chains 2 (nesting depth added 12). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/Diagnostic.fs"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"a075f2fc346f1e2d11cc22ee0ba52c86e7fa4ad3748351aa4f921c3b3b960a25"}},{"ruleId":"D2","level":"warning","message":{"text":"Completion.getCompletionsForRazorDocument (cognitive 24): Completion.getCompletionsForRazorDocument has cognitive complexity 24 (threshold 15). Drivers by points: match/switch 4 (10 pts), if/else 3 (9 pts), loops 1 (3 pts), boolean chains 2 (nesting depth added 14). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/Completion.fs"},"region":{"startLine":239}}}],"partialFingerprints":{"codehealthFindingId/v1":"78745f3ab28bc44396f3a648043a15a88fa6157fc20a2b0337f1bc378d981680"}},{"ruleId":"D2","level":"warning","message":{"text":"GenerateTypes.createTypeAlias (cognitive 24): GenerateTypes.createTypeAlias has cognitive complexity 24 (threshold 15). Drivers by points: match/switch 5 (10 pts), if/else 6 (8 pts), loops 2 (6 pts) (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":652}}}],"partialFingerprints":{"codehealthFindingId/v1":"70fa48e7866c35205ae22b6654afa66e82b38fe43dd4278870cdc5567a626087"}},{"ruleId":"D2","level":"warning","message":{"text":"GenerateTypes.createField (cognitive 23): GenerateTypes.createField has cognitive complexity 23 (threshold 15). Drivers by points: if/else 8 (13 pts), match/switch 3 (5 pts), loops 1 (3 pts), boolean chains 1, error handling 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":181}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a16a4057cb35ad449ea52480e19ed45c197cdea65afc5264c5a98650d29c885"}},{"ruleId":"D2","level":"warning","message":{"text":"Document.normalizeDocumentEndOfFile (cognitive 22): Document.normalizeDocumentEndOfFile has cognitive complexity 22 (threshold 15). Drivers by points: if/else 9 (14 pts), boolean chains 6, loops 1 (2 pts) (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Roslyn/Document.fs"},"region":{"startLine":94}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c7f685532fc73c8eb376b9f44dc48ec6b3b35011e68da90d0db5f63a9602010"}},{"ruleId":"D2","level":"warning","message":{"text":"LspTraceLogger.Microsoft.Extensions.Logging.ILogger.Log (cognitive 19): LspTraceLogger.Microsoft.Extensions.Logging.ILogger.Log has cognitive complexity 19 (threshold 15). Drivers by points: if/else 5 (10 pts), match/switch 3 (9 pts) (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Logging.fs"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"46bf55ffdead2ac277584ea160b733981aafd9baab1c1a33f6803e845645efce"}},{"ruleId":"D2","level":"warning","message":{"text":"CodeAction.getRoslynCodeActions (cognitive 18): CodeAction.getRoslynCodeActions has cognitive complexity 18 (threshold 15). Drivers by points: error handling 2 (6 pts), loops 3 (4 pts), match/switch 2 (4 pts), if/else 1 (3 pts), boolean chains 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/CodeAction.fs"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"ebf2867d253d89f9b937e1cb8549c533271f8568786fd16c4b6d14d1b8acdbdc"}},{"ruleId":"D2","level":"warning","message":{"text":"CSharpMetadata.handle (cognitive 17): CSharpMetadata.handle has cognitive complexity 17 (threshold 15). Drivers by points: match/switch 6 (17 pts) (nesting depth added 11). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/CSharpMetadata.fs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"df3c61598eec4a2fc9f9c7188314297e6509e4bfd8a7b69ffbc00166bf50909e"}},{"ruleId":"D2","level":"warning","message":{"text":"DocumentOnTypeFormatting.getSyntaxNode (cognitive 16): DocumentOnTypeFormatting.getSyntaxNode has cognitive complexity 16 (threshold 15). Drivers by points: match/switch 3 (9 pts), if/else 4 (6 pts), boolean chains 1 (nesting depth added 8). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/DocumentOnTypeFormatting.fs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"58568172bdb81fcb611cf228514e9a738659dbe4a1228f777a846c7a2cab703c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/Types.cg.fs: FileTooLong \u2014 2480 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 1980 over it, 4.96\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":"src/Ionide.LanguageServerProtocol/src/Types.cg.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9f2aacf2d35f6685d534c1c2ed4e297a8e81b9f8710783501d7d4b5ff1cf3eb"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: LspServer: TooManyMethods \u2014 140 methods. The bar is 30 methods; this is 110 over it, 4.67\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/src/Server.fs"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"b342b393f4e21125d183a11d83843c2b86bf2611862b65c53bf9fd7cdcda3fdf"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Runtime/JsonRpc.fs: FileTooLong \u2014 735 significant lines (blank, comment-only and punctuation-only lines excluded), declaring 33 functions. The bar is 500 significant lines; this is 235 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":"src/CSharpLanguageServer/Runtime/JsonRpc.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"00c72b10e4d55c38a0f22b4e782c3a64da995760a52406453f9bd80442bab1e7"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: LspClient: TooManyMethods \u2014 42 methods. The bar is 30 methods; this is 12 over it, 1.40\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":"src/Ionide.LanguageServerProtocol/src/Client.fs"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0be2e7f9848ee53e432d521a58c129c51f7af6a4937d72448984d2f82f8ed21"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: MetaModelGenerator/GenerateTypes.fs: FileTooLong \u2014 675 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 175 over it, 1.35\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":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e41e6c8fc74ea2a718b6c3f023f9a339fda87d6a15bb4f16805ec58ed0013f93"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Lsp/WorkspaceFolder.fs: FileTooLong \u2014 597 significant lines (blank, comment-only and punctuation-only lines excluded), declaring 36 functions. The bar is 500 significant lines; this is 97 over it, 1.19\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":"src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"579ccc1b13d084087970d6f7388f0aafd32a70b686c9772808679fae5ca91ab2"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (7 members, 50\u002B identical tokens): src/CSharpLanguageServer/Handlers/CallHierarchy.fs:1-234 | src/CSharpLanguageServer/Handlers/Definition.fs:1-63 | src/CSharpLanguageServer/Handlers/DocumentHighlight.fs:1-110 | src/CSharpLanguageServer/Handlers/Implementation.fs:1-105 | src/CSharpLanguageServer/Handlers/Rename.fs:1-218 | src/CSharpLanguageServer/Handlers/TypeDefinition.fs:1-77 | src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:1-194 \u2014 These 7 members share a duplicated core: a run of at least 50 identical tokens appears in every one of them. That run is NOT broken out as duplicated-block rows below \u2014 it is what admitted this row, and the blocks below cover only the part of it that clears the block floor, so they understate the correspondence. Read the members as one construct written 7 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 7 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8413cd5841b2a36bf289fb166b392bfc9e8b39268530acb35b6658d64b466c22"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (43 lines \u00D7 2): src/Ionide.LanguageServerProtocol/src/Types.cg.fs:1738-1780 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:4187-4229 \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 \u0060src/Ionide.LanguageServerProtocol/src/Types.cg.fs:1738\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":"src/Ionide.LanguageServerProtocol/src/Types.cg.fs"},"region":{"startLine":1738}}}],"partialFingerprints":{"codehealthFindingId/v1":"40374d802cb4c2829ff1513deac6bb1d20683628a4efe7ad0c35fad8da1545c4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24\u201333 lines \u00D7 2): src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2241-2264 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:4515-4547 \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 \u0060src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2241\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":"src/Ionide.LanguageServerProtocol/src/Types.cg.fs"},"region":{"startLine":2241}}}],"partialFingerprints":{"codehealthFindingId/v1":"7778340f9ab1118cf0acc4452bcc77db2d87d8de21c55db77ef614d205eafd98"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24 lines \u00D7 6): src/Ionide.LanguageServerProtocol/src/Types.cg.fs:401-424 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:452-475 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:687-710 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:1212-1235 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2400-2423 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2493-2516 \u2014 all 6 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/Ionide.LanguageServerProtocol/src/Types.cg.fs:401\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/src/Types.cg.fs"},"region":{"startLine":401}}}],"partialFingerprints":{"codehealthFindingId/v1":"20425439fb799aa2cc717159d48cbe35760edd516eae399c834336c5b5f85b04"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 7): src/Ionide.LanguageServerProtocol/src/Types.cg.fs:409-428 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:460-475 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:695-710 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:1220-1240 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:1994-2013 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2408-2426 | src/Ionide.LanguageServerProtocol/src/Types.cg.fs:2501-2521 \u2014 all 7 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/Ionide.LanguageServerProtocol/src/Types.cg.fs:409\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. 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":"src/Ionide.LanguageServerProtocol/src/Types.cg.fs"},"region":{"startLine":409}}}],"partialFingerprints":{"codehealthFindingId/v1":"7624d169a44f2df365802c11e18f708a40ef93125c08ff9bb7d9ad8451f56894"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/CSharpLanguageServer/Roslyn/Conversions.fs:139-153 | src/CSharpLanguageServer/Roslyn/Conversions.fs:171-185 \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":"src/CSharpLanguageServer/Roslyn/Conversions.fs"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a7bb1a39a271b2965749a1739a0e4fa5c0f0ea33083e6784f706911d6fc4f50"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): src/CSharpLanguageServer/Handlers/CallHierarchy.fs:57-70 | src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:55-68 \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":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca4f4b03193a4b52d7965c4075bb2ffeb80573dcd3b266a8deb54382e4af9075"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11\u201313 lines \u00D7 2): src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:105-115 | src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:172-184 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/TypeHierarchy.fs"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"a21b20902b06fa0b6317ea68773d67d0e9151756054ebd3b442b0b48d759a2fa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): src/Ionide.LanguageServerProtocol/src/JsonUtils.fs:538-550 | src/Ionide.LanguageServerProtocol/src/JsonUtils.fs:561-573 \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":"src/Ionide.LanguageServerProtocol/src/JsonUtils.fs"},"region":{"startLine":538}}}],"partialFingerprints":{"codehealthFindingId/v1":"23ee5d02eab54fc079d26c78f7e60fe6015d3fd97b8c491d3022d8f0dd93e724"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): src/CSharpLanguageServer/Roslyn/Conversions.fs:124-135 | src/CSharpLanguageServer/Roslyn/Conversions.fs:156-167 \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 \u0060src/CSharpLanguageServer/Roslyn/Conversions.fs:124\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":"src/CSharpLanguageServer/Roslyn/Conversions.fs"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"d516a60c04df4cb195f087d83d542991f8ebf22f2fc8de5fd7bbc748c84bb223"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/CSharpLanguageServer/Handlers/CodeAction.fs:255-265 | src/CSharpLanguageServer/Handlers/Rename.fs:54-64 \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 \u0060src/CSharpLanguageServer/Handlers/CodeAction.fs:255\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":"src/CSharpLanguageServer/Handlers/CodeAction.fs"},"region":{"startLine":255}}}],"partialFingerprints":{"codehealthFindingId/v1":"719403b09c8847526a586495ce5c311c12387cdae3e0e86644f7036a293c2586"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/CSharpLanguageServer/Handlers/DocumentOnTypeFormatting.fs:77-87 | src/CSharpLanguageServer/Handlers/DocumentRangeFormatting.fs:48-58 \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 \u0060src/CSharpLanguageServer/Handlers/DocumentOnTypeFormatting.fs:77\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":"src/CSharpLanguageServer/Handlers/DocumentOnTypeFormatting.fs"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f547534b3b846a1ccae2a6759d2abb522ad835e0404c5439fb334b2ae901cab"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/CSharpLanguageServer/Handlers/SignatureHelp.fs:132-142 | src/CSharpLanguageServer/Handlers/SignatureHelp.fs:152-162 \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":"src/CSharpLanguageServer/Handlers/SignatureHelp.fs"},"region":{"startLine":132}}}],"partialFingerprints":{"codehealthFindingId/v1":"323fdeab68611e59f5245db5bcf83c7a1e5319ca9bb0c8fc22c0dc7e10b61b5b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs:111-121 | src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs:146-157 \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 \u0060src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs:111\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":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateClientServer.fs"},"region":{"startLine":111}}}],"partialFingerprints":{"codehealthFindingId/v1":"06726523d523adfecde4a71f7c2cd13eddda13855e0526b986114a99c72748f6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 3): src/CSharpLanguageServer/Handlers/InlayHint.fs:703-712 | src/CSharpLanguageServer/Handlers/Rename.fs:119-128 | src/CSharpLanguageServer/Handlers/SignatureHelp.fs:95-104 \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 all 3 call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/CSharpLanguageServer/Handlers/InlayHint.fs:703\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":"src/CSharpLanguageServer/Handlers/InlayHint.fs"},"region":{"startLine":703}}}],"partialFingerprints":{"codehealthFindingId/v1":"18faa95f7e91b5336a44fc46e43e202ca79dbf03d9e587a7fcf308e9cd1e7e5e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9\u201310 lines \u00D7 2): src/CSharpLanguageServer/Handlers/CallHierarchy.fs:138-147 | src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:131-139 \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 \u0060src/CSharpLanguageServer/Handlers/CallHierarchy.fs:138\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":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"e4f7f5c95c04fc84dafa2dc1dbc9866f590393cc19207c6521b7247fb8d23a90"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): src/CSharpLanguageServer/Handlers/DocumentSymbol.fs:325-334 | src/CSharpLanguageServer/Handlers/FoldingRange.fs:199-208 \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. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060src/CSharpLanguageServer/Handlers/DocumentSymbol.fs:336\u0060 calls \u0060GetSemanticModelAsync\u0060 and \u0060src/CSharpLanguageServer/Handlers/FoldingRange.fs:210\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":"src/CSharpLanguageServer/Handlers/DocumentSymbol.fs"},"region":{"startLine":325}}}],"partialFingerprints":{"codehealthFindingId/v1":"e216b8edcf1b2a274a46eb41e7cf1fbb7830051eded6429f1abc3a932e4640d7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 7): src/CSharpLanguageServer/Handlers/CallHierarchy.fs:56-64 | src/CSharpLanguageServer/Handlers/Definition.fs:44-52 | src/CSharpLanguageServer/Handlers/DocumentHighlight.fs:84-92 | src/CSharpLanguageServer/Handlers/Implementation.fs:87-95 | src/CSharpLanguageServer/Handlers/Rename.fs:184-192 | src/CSharpLanguageServer/Handlers/TypeDefinition.fs:47-55 | src/CSharpLanguageServer/Handlers/TypeHierarchy.fs:54-62 \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 all 7 call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/CSharpLanguageServer/Handlers/CallHierarchy.fs:56\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":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a6e70a6c9e98f4294267a56fa25945eb19acb48f757b5c980d75bc212069c6d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7\u20139 lines \u00D7 4): src/CSharpLanguageServer/Handlers/DocumentRangeFormatting.fs:53-59 | src/CSharpLanguageServer/Handlers/InlayHint.fs:704-712 | src/CSharpLanguageServer/Handlers/Rename.fs:120-128 | src/CSharpLanguageServer/Handlers/SignatureHelp.fs:96-104 \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 all 4 call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/DocumentRangeFormatting.fs"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"e8cf064717ac291325dc76c21e9de150ef055bd0d85816c8306134820ff60d5f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7\u20139 lines \u00D7 3): src/CSharpLanguageServer/Handlers/CodeLens.fs:125-133 | src/CSharpLanguageServer/Handlers/DocumentSymbol.fs:329-335 | src/CSharpLanguageServer/Handlers/FoldingRange.fs:203-209 \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 all 3 call sites, so a change lands once. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060src/CSharpLanguageServer/Handlers/DocumentSymbol.fs:336\u0060 calls \u0060GetSemanticModelAsync\u0060 and \u0060src/CSharpLanguageServer/Handlers/FoldingRange.fs:210\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":"src/CSharpLanguageServer/Handlers/CodeLens.fs"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"05e897064520c584358b316c4c431b8cb260e9a2ae4325ef6829a7ded969902c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8\u20139 lines \u00D7 2): src/CSharpLanguageServer/Handlers/DocumentSymbol.fs:55-62 | src/CSharpLanguageServer/Roslyn/Conversions.fs:107-115 \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. 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":"src/CSharpLanguageServer/Handlers/DocumentSymbol.fs"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"12d000dbf0d53daee29db9fe3675a36a7a044b9cd05cb45b3b61df20e5bde65d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7\u20139 lines \u00D7 2): src/CSharpLanguageServer/Handlers/DocumentSync.fs:182-188 | src/CSharpLanguageServer/Handlers/DocumentSync.fs:336-344 \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 \u0060src/CSharpLanguageServer/Handlers/DocumentSync.fs:336\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":"src/CSharpLanguageServer/Handlers/DocumentSync.fs"},"region":{"startLine":182}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbaf10db380177faff08e3ff6ff27b5fd350b2d7cb3563f81cb19f403d2420ee"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): src/CSharpLanguageServer/Handlers/DocumentFormatting.fs:55-62 | src/CSharpLanguageServer/Handlers/DocumentOnTypeFormatting.fs:76-83 | src/CSharpLanguageServer/Handlers/DocumentRangeFormatting.fs:47-54 \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 all 3 call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/CSharpLanguageServer/Handlers/DocumentFormatting.fs:55\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":"src/CSharpLanguageServer/Handlers/DocumentFormatting.fs"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"f14814e1421aa19b3885750a8cac26c12aea642b4ef3416c7440c298d4a8dc79"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): src/Ionide.LanguageServerProtocol/src/JsonUtils.fs:328-335 | src/Ionide.LanguageServerProtocol/src/JsonUtils.fs:339-346 \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":"src/Ionide.LanguageServerProtocol/src/JsonUtils.fs"},"region":{"startLine":328}}}],"partialFingerprints":{"codehealthFindingId/v1":"396da3e81f71e8d8e1c501779a86cdd244071ff7c4a42c15a87889bcca000eba"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): src/CSharpLanguageServer/Handlers/Workspace.fs:161-167 | src/CSharpLanguageServer/Handlers/Workspace.fs:176-182 \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":"src/CSharpLanguageServer/Handlers/Workspace.fs"},"region":{"startLine":161}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7d72ca7ec4faf11de39c52848ae25dd2f498735da1860dc82cbc6583ca77f22"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs:49-59 | src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs:60-72 \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 \u0060src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs:49\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it. 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":"src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator/GenerateTypes.fs"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"4884c7b53e375a214cef0028f153b50ca45ce0a44d6ef6dba50474dc3ff7c93b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): src/CSharpLanguageServer/Handlers/DocumentSync.fs:62-72 | src/CSharpLanguageServer/Handlers/DocumentSync.fs:100-108 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/DocumentSync.fs"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3abd074b2b8f03400f609ef47d919ec109adcb534e0e814f2f0f839cbe7a49a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 3): src/CSharpLanguageServer/Handlers/DocumentSync.fs:67-76 | src/CSharpLanguageServer/Handlers/DocumentSync.fs:80-89 | src/CSharpLanguageServer/Handlers/DocumentSync.fs:93-102 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/DocumentSync.fs"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"095373cd3c029be4d9c02ce758c3240526ad8bb36b08cfb83f04e1eb3a109035"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): src/Ionide.LanguageServerProtocol/src/FsLibLog.fs:449-457 | src/Ionide.LanguageServerProtocol/src/FsLibLog.fs:482-490 \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 \u0060src/Ionide.LanguageServerProtocol/src/FsLibLog.fs:449\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ionide.LanguageServerProtocol/src/FsLibLog.fs"},"region":{"startLine":449}}}],"partialFingerprints":{"codehealthFindingId/v1":"314e4c5bdc384999039bd6b6e89fe340d6856e816e3cae97c7a232c971a99152"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 test suite did not build: Coverage NOT MEASURED: this repository did not build in our analyzer environment (a C#/MSBuild compiler error), so no coverage could be collected. It is excluded from the score rather than counted as a near-zero defect. We did not read WHERE the failing diagnostic is, so this does not claim the fault is in your test code \u2014 a repository written for an older SDK band can compile for you and not for us. Run \u0060dotnet build\u0060 on this commit; if it succeeds, the gap is ours. Committing the Cobertura/OpenCover/lcov report your CI already produces also lets us measure real coverage without building anything."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D12","level":"note","message":{"text":"Outdated: coverlet.collector: coverlet.collector 6.0.4 \u2192 10.1.0 available (referenced by CSharpLanguageServer.Tests)."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Packages.props"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"3233e35c9af91a23826c2b9c32dd3eb3674bd1f0fa7f30f4742ae3349d69e42d"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/CSharpLanguageServer/Runtime/ServerStateLoop.fs: src/CSharpLanguageServer/Runtime/ServerStateLoop.fs changed 3 times in last 90 days, max cyclomatic complexity 72 in ServerStateLoop.processServerEvent at line 115. 1 of those changes was a fix/bug commit, and the other 2 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-07-25..2026-10-23, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-25 10:02:10 \u002B03:00\u0027 --until=\u00272026-10-23 10:02:10 \u002B03:00\u0027 --full-history --no-merges -- src/CSharpLanguageServer/Runtime/ServerStateLoop.fs\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":"src/CSharpLanguageServer/Runtime/ServerStateLoop.fs"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f378fbcb5b27286f8906bea9e8d13321028007fce651e8f0be69ecb796ebc34"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/CSharpLanguageServer/Handlers/CallHierarchy.fs: src/CSharpLanguageServer/Handlers/CallHierarchy.fs changed 5 times in last 90 days, max cyclomatic complexity 20 in CallHierarchy.outgoingCalls at line 134. 2 of those changes were fix/bug commits, and the other 3 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-07-25..2026-10-23, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-25 10:02:10 \u002B03:00\u0027 --until=\u00272026-10-23 10:02:10 \u002B03:00\u0027 --full-history --no-merges -- src/CSharpLanguageServer/Handlers/CallHierarchy.fs\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":"src/CSharpLanguageServer/Handlers/CallHierarchy.fs"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"84b11fc2a1987fc0f9333fc8756046f6b964a9d9172cacaeba5845b4408f4814"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs: src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs changed 5 times in last 90 days, max cyclomatic complexity 16 in WorkspaceFolder.workspaceFolderDocumentDetails at line 447. 2 of those changes were fix/bug commits, and the other 3 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-07-25..2026-10-23, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-25 10:02:10 \u002B03:00\u0027 --until=\u00272026-10-23 10:02:10 \u002B03:00\u0027 --full-history --no-merges -- src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs\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":"src/CSharpLanguageServer/Lsp/WorkspaceFolder.fs"},"region":{"startLine":447}}}],"partialFingerprints":{"codehealthFindingId/v1":"77e9f0d1cb23f8327c2d987aee41c992a0bf022afa8a1fa836e920858336aa5e"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Ionide.LanguageServerProtocol/src/JsonUtils.fs: src/Ionide.LanguageServerProtocol/src/JsonUtils.fs changed 2 times in last 90 days, max cyclomatic complexity 33 in ErasedUnionConverter\u00601.Read at line 428. 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-07-25..2026-10-23, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-25 10:02:10 \u002B03:00\u0027 --until=\u00272026-10-23 10:02:10 \u002B03:00\u0027 --full-history --no-merges -- src/Ionide.LanguageServerProtocol/src/JsonUtils.fs\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":"src/Ionide.LanguageServerProtocol/src/JsonUtils.fs"},"region":{"startLine":428}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e3eca54ff7edbbee8ce4c035f60f1f6ef8b950681cc55f7194d11a038f46dd6"}},{"ruleId":"D15","level":"warning","message":{"text":"Repeated repair: src/CSharpLanguageServer/Roslyn/Solution.fs: src/CSharpLanguageServer/Roslyn/Solution.fs changed 5 times in last 90 days and 4 of those changes were fix/bug commits, so repair is the majority of this file\u0027s churn. Its max cyclomatic complexity is 8 (its worst body is Solution.solutionGetRazorDocumentForPath at line 506), UNDER the 15 threshold, so this is deliberately not filed as a churn \u00D7 complexity hotspot \u2014 the difficulty here is in the behaviour the file has to get right, not in its control flow, and refactoring it for complexity would be the wrong move. The repairs counted were: \u201Cfix(solution): honor .slnf filters in the declared-project set\u201D; \u201Cfix(workspace): resolve multi-targeted project documents to a best-TFM flavor\u201D; \u201Cfix(workspace): only force a TargetFramework every project declares\u201D; \u201CRe-enable Razor tests: upgrade Roslyn to 5.9.0 and fix hover/reference token mapping\u201D. Each one is a case this code did not handle. Before the next change lands here, check that every one of them is pinned by a test that fails without its fix; where the same area keeps coming back, the durable fix is usually at the interface that keeps being misused rather than at the line that was last corrected. Counted over 2026-07-25..2026-10-23, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-25 10:02:10 \u002B03:00\u0027 --until=\u00272026-10-23 10:02:10 \u002B03:00\u0027 --full-history --no-merges -- src/CSharpLanguageServer/Roslyn/Solution.fs\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":"src/CSharpLanguageServer/Roslyn/Solution.fs"},"region":{"startLine":506}}}],"partialFingerprints":{"codehealthFindingId/v1":"edad4ae4364fe3f7654ce99c3c66a652dc40c70b68caf5c6e2d1e0a522de9942"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj \u2014 known-vulnerability alert suppressed: NU1902 \u2014 this is not a style rule being switched off: it is the dependency audit reporting that a package this project references has a KNOWN, already-published security vulnerability, and the entry deletes that report for the whole project \u2014 for this package and for every dependency the project takes from here on, including ones added long after whoever wrote the line has moved on. Nothing in the build will raise it again, so the next reader has no way to learn it was ever accepted. Upgrade the package to a version carrying the fix, or replace it, and drop the code from the list. Where the fix genuinely does not exist yet, keep the report and stop it FAILING the build \u2014 demote it to a warning without silencing it \u2014 and write down which advisory is being accepted and until when, so it is the acceptance that expires rather than the alert."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Packages.props"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed44f03e3a84ac8b32bd2a47200aff160d72cb123d71bc10e60285e0d1c058d8"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj \u2014 known-vulnerability alert suppressed: NU1903 \u2014 this is not a style rule being switched off: it is the dependency audit reporting that a package this project references has a KNOWN, already-published security vulnerability, and the entry deletes that report for the whole project \u2014 for this package and for every dependency the project takes from here on, including ones added long after whoever wrote the line has moved on. Nothing in the build will raise it again, so the next reader has no way to learn it was ever accepted. Upgrade the package to a version carrying the fix, or replace it, and drop the code from the list. Where the fix genuinely does not exist yet, keep the report and stop it FAILING the build \u2014 demote it to a warning without silencing it \u2014 and write down which advisory is being accepted and until when, so it is the acceptance that expires rather than the alert."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Packages.props"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"754cefd03fc8ad392c18833bef4e0652a4e555cea60fe1ea3a2f15915cb123a2"}},{"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":"D27","level":"note","message":{"text":"Scattered collaborators: 94 % of calls cross a namespace and only 57 % of collaborators are co-located \u2014 group each feature\u0027s code into a vertical slice so a call\u0027s collaborators sit together."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"5aad4a4530bac3928d3a065a664be89d023ecfc865f9bf79ac05754b3bffe322"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"52d1a8c43a2d53be9eb49e7b2748e19b9b0981141de452f8ec2aefc7ae34ab31"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"26318058c642462f51224e345d1c1f287c6019a5c4a968754ec93861b144e68a"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"feaa8bce29050234f02cb5719080cfa11dad1809fd01a49ff58cb83420030853"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0fd14044958507cd40fe734315d483e62e62b958ab3c7f95383f276fa91d6e9a"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f9709f875631203b0f359dc51165142e874ae52b91e9c357cedfb88f7b3c6b60"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ddd1ad03aace3c210a4d7b54eb15a714e06a30d23ccbcd0326fa98901ce32d5c"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"aaa8a9f351229202be42a6b922274fe2fd2a5a7255b2e12e899ecdedbc07e6f9"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"57d67f7e9b9641aa497716b0f96c9f60aebb6cc780d1f36a607a3c5c42fe50d6"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"A03:2025","toolComponent":{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61"}}]},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: Implementation.fs \u2194 TypeDefinition.fs: \u0060src/CSharpLanguageServer/Handlers/Implementation.fs\u0060 and \u0060src/CSharpLanguageServer/Handlers/TypeDefinition.fs\u0060 change together 60% of the time (6 of the 10 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 \u006057c4a492\u0060 chore: pass-in ServerSettings to workspaceFolderSymbolLocations to ma\u2026; \u0060937fa0c2\u0060 fix(Handlers.CSharpMetadata): make sure metadata URI works with no pr\u2026; \u0060ba04065d\u0060 chore(Lsp.Workspace): make \u0060project\u0060 arg non-optional for workspaceFo\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/Handlers/Implementation.fs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3525000e75cd54b7022e04aeaa4da5a71b25e54cd18a7f8ad2ea5f7869b5e9c9"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"},"taxa":[{"id":"CICD-SEC-9","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"},"taxa":[{"id":"CICD-SEC-9","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"},"taxa":[{"id":"CICD-SEC-9","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"},"taxa":[{"id":"CICD-SEC-8","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7e37904db2ccbc85e059aeb48973ed8e1f6e6831834f4b40800c83663672d401"},"taxa":[{"id":"CICD-SEC-5","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d339e23018d6ba57a64e7789bce515a9b72a731c2f4069852406dd1012e4a611"},"taxa":[{"id":"CICD-SEC-5","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ace515990e7ee0f17b5c17e44dd168a5bdecf90804a3a3dd845cf05540978013"},"taxa":[{"id":"CICD-SEC-3","toolComponent":{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35"}}]},{"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":"M3","level":"note","message":{"text":"Inconsistent root namespaces: Only 14/31 projects share a common root namespace \u2014 the code\u0027s module identity is inconsistent."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c8031c9fb77ad97c2e4a8ddb29978f4bf30140d481913bd7fde8b5adbbb83ede"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: src/CSharpLanguageServer/README.md links to \u0060LICENSE\u0060, but \u0060src/CSharpLanguageServer/LICENSE\u0060 does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every package.json, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents\u0027 own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/README.md"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f974aa8897efdc22bbdccb6c8e44c6c856a2536b84b210dfec3e621c667fc05"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: src/CSharpLanguageServer/README.md links to \u0060docs/features.md\u0060, but \u0060src/CSharpLanguageServer/docs/features.md\u0060 does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every package.json, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents\u0027 own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/README.md"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"560533813307a3b0b92d0a8ba22a8c02cab525ad0706fc82f87d3b72bf89bfa5"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: src/CSharpLanguageServer/README.md links to \u0060docs/troubleshooting.md\u0060, but \u0060src/CSharpLanguageServer/docs/troubleshooting.md\u0060 does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every package.json, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents\u0027 own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/README.md"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"a907593ec4c38cb89f31c81850da46b6e89fd7963bf02d1ba2f07a485e0e53bf"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: src/CSharpLanguageServer/README.md links to \u0060CONTRIBUTING.md\u0060, but \u0060src/CSharpLanguageServer/CONTRIBUTING.md\u0060 does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every package.json, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents\u0027 own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/README.md"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"3bd29a500613c0cfbba5ecf44d0fd1b62eaccf46703d8e3ad10542fc7aef6581"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: src/CSharpLanguageServer/README.md links to \u0060docs/codebase-architecture.md\u0060, but \u0060src/CSharpLanguageServer/docs/codebase-architecture.md\u0060 does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every package.json, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents\u0027 own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/CSharpLanguageServer/README.md"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"b42e0f8789ccc1fe09bac744210053dec9fb76dc889b7ea71036b06703b60907"}},{"ruleId":"P2","level":"note","message":{"text":"Logging is not universal: Only 2/3 runnable modules use logging (modules with no entry point or server are excluded \u2014 they are libraries a runnable module hosts). Silent: \u0060src/Ionide.LanguageServerProtocol/tools/MetaModelGenerator\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"91a94e21d6d4d0e6a2003f94505b0b02b157176f0b24c4820f3f82b4447a97fe"}},{"ruleId":"P3","level":"error","message":{"text":"NuGet vulnerability audit disabled (csproj): \u003CNuGetAudit\u003Efalse\u003C/NuGetAudit\u003E in MSBuild suppresses NuGet\u0027s vulnerability audit project-wide."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tests/CSharpLanguageServer.Tests/Fixtures/platformSpecificTfm/Project/Project.csproj"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5091dad71d12647fdcd1765845e303602b4ce84a8f0dc0b1b0f19d1253ef7c6"}},{"ruleId":"P3","level":"error","message":{"text":"NuGet vulnerability audit suppressed via NoWarn: \u0060NoWarn\u0060 silences NU1902, NU1903 \u2014 NuGet\u0027s vulnerability-audit warnings for moderate and high advisories. Suppressed advisories are still shipped; they are only invisible in the build log. Remove these codes from NoWarn and address or explicitly accept each advisory instead."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Directory.Packages.props"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"143157174852f2742e1483b321e0b0a2e3f57caccbe25b95588aa253a91416a2"}}],"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-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-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-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"Missing Support for Integrity Check","shortDescription":{"text":"Missing Support for Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Improper Neutralization of Special Elements used in a Command (\u0027Command Injection\u0027)","shortDescription":{"text":"Improper Neutralization of Special Elements used in a Command (\u0027Command Injection\u0027)"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"Improper Neutralization of Special Elements used in an OS Command (\u0027OS Command Injection\u0027)","shortDescription":{"text":"Improper Neutralization of Special Elements used in an OS Command (\u0027OS Command Injection\u0027)"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Improper Neutralization of Input During Web Page Generation (\u0027Cross-site Scripting\u0027)","shortDescription":{"text":"Improper Neutralization of Input During Web Page Generation (\u0027Cross-site Scripting\u0027)"},"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":"Improper Neutralization of Special Elements used in an SQL Command (\u0027SQL Injection\u0027)","shortDescription":{"text":"Improper Neutralization of Special Elements used in an SQL Command (\u0027SQL Injection\u0027)"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Improper Control of Generation of Code (\u0027Code Injection\u0027)","shortDescription":{"text":"Improper Control of Generation of Code (\u0027Code Injection\u0027)"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]},{"name":"OWASP Top 10","guid":"7d1f0c2e-4b6a-4e39-9a51-0b2e8c3d5f61","version":"2025","organization":"OWASP Foundation","informationUri":"https://top10.owasp.org/2025/","isComprehensive":false,"shortDescription":{"text":"The OWASP Top 10:2025 web application security risks."},"taxa":[{"id":"A03:2025","guid":"168e227c-7735-2e57-bc6e-997affe644c5","name":"Software Supply Chain Failures","shortDescription":{"text":"Software Supply Chain Failures"},"helpUri":"https://top10.owasp.org/2025/A03_2025-Software_Supply_Chain_Failures/"},{"id":"A05:2025","guid":"87e86033-3231-8458-8e4a-5cb82743553a","name":"Injection","shortDescription":{"text":"Injection"},"helpUri":"https://top10.owasp.org/2025/A05_2025-Injection/"},{"id":"A07:2025","guid":"334ddd78-539d-d35c-83dd-253647eb7eaf","name":"Authentication Failures","shortDescription":{"text":"Authentication Failures"},"helpUri":"https://top10.owasp.org/2025/A07_2025-Authentication_Failures/"}]},{"name":"OWASP Top 10 CI/CD Security Risks","guid":"3c9b2a6e-5d14-4f7b-8e02-6a1d9c4b7f35","organization":"OWASP Foundation","informationUri":"https://owasp.org/www-project-top-10-ci-cd-security-risks/","isComprehensive":false,"shortDescription":{"text":"The OWASP Top 10 CI/CD Security Risks."},"taxa":[{"id":"CICD-SEC-3","guid":"458c83a6-ebf7-cf55-8cc3-b4d949bbf271","name":"Dependency Chain Abuse","shortDescription":{"text":"Dependency Chain Abuse"},"helpUri":"https://owasp.github.io/www-project-top-10-ci-cd-security-risks/CICD-SEC-03-Dependency-Chain-Abuse"},{"id":"CICD-SEC-5","guid":"cc7f4610-8859-e654-9029-d5f97d992220","name":"Insufficient PBAC (Pipeline-Based Access Controls)","shortDescription":{"text":"Insufficient PBAC (Pipeline-Based Access Controls)"},"helpUri":"https://owasp.github.io/www-project-top-10-ci-cd-security-risks/CICD-SEC-05-Insufficient-PBAC"},{"id":"CICD-SEC-8","guid":"b6b2dacf-a168-4957-876e-a8c73ee00eb0","name":"Ungoverned Usage of 3rd Party Services","shortDescription":{"text":"Ungoverned Usage of 3rd Party Services"},"helpUri":"https://owasp.github.io/www-project-top-10-ci-cd-security-risks/CICD-SEC-08-Ungoverned-Usage-of-3rd-Party-Services"},{"id":"CICD-SEC-9","guid":"b3d36727-0395-5b51-94c4-5e173d62eab3","name":"Improper Artifact Integrity Validation","shortDescription":{"text":"Improper Artifact Integrity Validation"},"helpUri":"https://owasp.github.io/www-project-top-10-ci-cd-security-risks/CICD-SEC-09-Improper-Artifact-Integrity-Validation"}]}],"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":15,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}