{"$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":"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":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D18","name":"Solution Shape","shortDescription":{"text":"Solution Shape"},"helpUri":"https://codehealth.canine.dev/dimensions/D18"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D23","name":"Boundary Type-Coupling","shortDescription":{"text":"Boundary Type-Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D23"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D27","name":"Navigability","shortDescription":{"text":"Navigability"},"helpUri":"https://codehealth.canine.dev/dimensions/D27"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"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":"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":"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":"AX5","name":"Architecture \u0026 structure","shortDescription":{"text":"Architecture \u0026 structure"},"helpUri":"https://codehealth.canine.dev/dimensions/AX5"},{"id":"AX6","name":"Interface segregation","shortDescription":{"text":"Interface segregation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX6"},{"id":"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":"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":"R1","name":"Type Safety","shortDescription":{"text":"Type Safety"},"helpUri":"https://codehealth.canine.dev/dimensions/R1"},{"id":"R10","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/R10"},{"id":"R2","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/R2"},{"id":"R3","name":"Large Files","shortDescription":{"text":"Large Files"},"helpUri":"https://codehealth.canine.dev/dimensions/R3"},{"id":"R7","name":"Dead Code","shortDescription":{"text":"Dead Code"},"helpUri":"https://codehealth.canine.dev/dimensions/R7"},{"id":"R9","name":"Circular Imports","shortDescription":{"text":"Circular Imports"},"helpUri":"https://codehealth.canine.dev/dimensions/R9"},{"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":"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":"X6","name":"Hand-rolled structured-format parsing","shortDescription":{"text":"Hand-rolled structured-format parsing"},"helpUri":"https://codehealth.canine.dev/dimensions/X6"},{"id":"X7","name":"Silent fallback defaults","shortDescription":{"text":"Silent fallback defaults"},"helpUri":"https://codehealth.canine.dev/dimensions/X7"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"LLMClient.ChatRound (cyclomatic 34): LLMClient.ChatRound has cyclomatic complexity 34 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/LLMClient.vb"},"region":{"startLine":614}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d3c12a586eae024a109323b255714aa25b451f269e6675225990d6835b9216a"}},{"ruleId":"D1","level":"warning","message":{"text":"OpenAIProvider.ParseDeltaChunk (cyclomatic 28): OpenAIProvider.ParseDeltaChunk has cyclomatic complexity 28 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OpenAIProvider.vb"},"region":{"startLine":157}}}],"partialFingerprints":{"codehealthFindingId/v1":"48210d507ab921a88d5c84bff75e1d9a64fd4eea33c432bf3064e9e813c6bc60"}},{"ruleId":"D1","level":"warning","message":{"text":"ToolCallLogger.FillArguments (cyclomatic 27): ToolCallLogger.FillArguments has cyclomatic complexity 27 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":539}}}],"partialFingerprints":{"codehealthFindingId/v1":"3ce48fc0651d2e4568a8e4e0b7164e037a9dc07a00a9d0bbb35a99b7adf08526"}},{"ruleId":"D1","level":"warning","message":{"text":"HarnessLogReader.Keep (cyclomatic 27): HarnessLogReader.Keep has cyclomatic complexity 27 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":555}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2b95ecee82166f830b024e14ef726c191366fb500870b83ec94cb6a487b8243"}},{"ruleId":"D1","level":"warning","message":{"text":"harness-log.openDrawer (cyclomatic 27): harness-log.openDrawer has cyclomatic complexity 27 (threshold 15). Of this number, 24 points are the body\u0027s own statements and 3 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1158}}}],"partialFingerprints":{"codehealthFindingId/v1":"5875edf1c3d71803475ff87804a75b7b32c7f6c0a0e60dfffb2ef9fcf5eabf76"}},{"ruleId":"D1","level":"warning","message":{"text":"OllamaProvider.ParseOllamaStream (cyclomatic 24): OllamaProvider.ParseOllamaStream has cyclomatic complexity 24 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OllamaProvider.vb"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"130389bb472db6931d179ce8c587d1488d68c601d2d6c7b6712a2c484b26d27a"}},{"ruleId":"D1","level":"warning","message":{"text":"HiddenMarkovModel.estimateParametersUsingBaumWelchAlgorithm (cyclomatic 24): HiddenMarkovModel.estimateParametersUsingBaumWelchAlgorithm has cyclomatic complexity 24 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/HiddenMarkovModel.vb"},"region":{"startLine":354}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7fbbddbc14269e27e4ce7d9d351b83f21e64d60f79ef7c4855f4e1d782d40e9"}},{"ruleId":"D1","level":"warning","message":{"text":"harness-log.renderOverview (cyclomatic 24): harness-log.renderOverview has cyclomatic complexity 24 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":511}}}],"partialFingerprints":{"codehealthFindingId/v1":"ecfcaba42a2a273de7cc9fbed5edb50a39043e7847a3c14359f05829325340d7"}},{"ruleId":"D1","level":"warning","message":{"text":"harness-log.turnBodyHtml (cyclomatic 23): harness-log.turnBodyHtml has cyclomatic complexity 23 (threshold 15). Of this number, 16 points are the body\u0027s own statements and 7 belong to one function literal inside it that branches. To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":706}}}],"partialFingerprints":{"codehealthFindingId/v1":"187b040a3bd3988ea5b8f905dfea19950a8cdc7b05b964808d86c5a6450b4710"}},{"ruleId":"D1","level":"warning","message":{"text":"SkillManifest.Parse (cyclomatic 22): SkillManifest.Parse has cyclomatic complexity 22 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillManifest.vb"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"d8d7a988226d4f4d3951e0b0f63dc05aa9de7243114bd0f2fbfc16fc69da2319"}},{"ruleId":"D1","level":"warning","message":{"text":"ApplyPatchTool.ParseBlocks (cyclomatic 21): ApplyPatchTool.ParseBlocks has cyclomatic complexity 21 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":454}}}],"partialFingerprints":{"codehealthFindingId/v1":"68699f2dfd78440c5c44eddf041103a48565da11066a6d8f4aa84727b974782f"}},{"ruleId":"D1","level":"warning","message":{"text":"index.html.window.addEventListener(\u0022DOMContentLoaded\u0022) (cyclomatic 21): index.html.window.addEventListener(\u0022DOMContentLoaded\u0022) has cyclomatic complexity 21 (threshold 15). Of this number, 18 points are the body\u0027s own statements and 3 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/index.html"},"region":{"startLine":1597}}}],"partialFingerprints":{"codehealthFindingId/v1":"328487d990df746beef164e716dfa9c438f828f44dae8697eb3252b38286c542"}},{"ruleId":"D1","level":"warning","message":{"text":"SkillScriptExecutor.Execute (cyclomatic 20): SkillScriptExecutor.Execute has cyclomatic complexity 20 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"638f7413e5a7879626f3dee9705cd28c942623c2d9de5472e5048e9f1b0be183"}},{"ruleId":"D1","level":"warning","message":{"text":"GrepTool.grep (cyclomatic 20): GrepTool.grep has cyclomatic complexity 20 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"c6f242b630fab92b3f57fabdfca57b30d8b41cfdd2c04a4f851d40bb22d59027"}},{"ruleId":"D1","level":"warning","message":{"text":"MultiEditTool.multi_edit (cyclomatic 20): MultiEditTool.multi_edit has cyclomatic complexity 20 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/MultiEditTool.vb"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e9695f9c7c952d0e29c7111c0c2c0c8eeccd52ecc5ea31aa67a719ab073fcaf"}},{"ruleId":"D1","level":"warning","message":{"text":"AskUserQuestionTool.ParseQuestions (cyclomatic 19): AskUserQuestionTool.ParseQuestions has cyclomatic complexity 19 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":438}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7941055a176140e960cbe8f369a8ea03058807de188e047c93b0b06f595da36"}},{"ruleId":"D1","level":"warning","message":{"text":"LlmJsonExtractor.TryRepairJson (cyclomatic 19): LlmJsonExtractor.TryRepairJson has cyclomatic complexity 19 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ToolCalls/LlmJsonExtractor.vb"},"region":{"startLine":180}}}],"partialFingerprints":{"codehealthFindingId/v1":"5649c734e6f50cd6398d20fa5dbf7be4c75cb475d707538449923e41f7e5b96f"}},{"ruleId":"D1","level":"warning","message":{"text":"AskUserQuestionTool.Ask (cyclomatic 18): AskUserQuestionTool.Ask has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":173}}}],"partialFingerprints":{"codehealthFindingId/v1":"fbcfe2968929ae18551f51725b54eff4e9ba6d21b1600b75b74bffc9324f711a"}},{"ruleId":"D1","level":"warning","message":{"text":"GrepTool.CollectMatches (cyclomatic 18): GrepTool.CollectMatches has cyclomatic complexity 18 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":257}}}],"partialFingerprints":{"codehealthFindingId/v1":"529001cfe8cd1b9bff1d786ee4593c1ba8507719b7d77f4e89e0ca6f5c8d089b"}},{"ruleId":"D1","level":"warning","message":{"text":"HarnessLogReader.Records (cyclomatic 17): HarnessLogReader.Records has cyclomatic complexity 17 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":193}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a2d5bc6dc93c53588024f9ddfa510393829c0ea79784f6e33eb52d4fc7aab73"}},{"ruleId":"D1","level":"warning","message":{"text":"DeepSeekTokenizerAdapter.New.ctor (cyclomatic 17): DeepSeekTokenizerAdapter.New.ctor has cyclomatic complexity 17 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/DeepSeekTokenizerAdapter.vb"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"beb5a6c5eae8f0d8182344f7b15a70eca21731f45ebd629e594dc6ad47fb1cea"}},{"ruleId":"D2","level":"warning","message":{"text":"LLMClient.ChatRound (cognitive 77): LLMClient.ChatRound has cognitive complexity 77 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/LLMClient.vb"},"region":{"startLine":614}}}],"partialFingerprints":{"codehealthFindingId/v1":"58366023a48fca98366ca245282bf23b914661239b5a39752adac5c0296d9a6e"}},{"ruleId":"D2","level":"warning","message":{"text":"OllamaProvider.ParseOllamaStream (cognitive 55): OllamaProvider.ParseOllamaStream has cognitive complexity 55 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OllamaProvider.vb"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c9e1bdac0f484a1724e31972954eb843773d3e74e625e14136ecc0171e277b3"}},{"ruleId":"D2","level":"warning","message":{"text":"OpenAIProvider.ParseDeltaChunk (cognitive 53): OpenAIProvider.ParseDeltaChunk has cognitive complexity 53 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OpenAIProvider.vb"},"region":{"startLine":157}}}],"partialFingerprints":{"codehealthFindingId/v1":"d1472b041f7c94efee740449d3e596d851d17755db63db8734ca6841eaf51d8a"}},{"ruleId":"D2","level":"warning","message":{"text":"HiddenMarkovModel.estimateParametersUsingBaumWelchAlgorithm (cognitive 52): HiddenMarkovModel.estimateParametersUsingBaumWelchAlgorithm has cognitive complexity 52 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/HiddenMarkovModel.vb"},"region":{"startLine":354}}}],"partialFingerprints":{"codehealthFindingId/v1":"613eb81ce12cdef6c7ba6456c237aff455184bb5421be06c435c724ced45a60e"}},{"ruleId":"D2","level":"warning","message":{"text":"ApplyPatchTool.ParseBlocks (cognitive 42): ApplyPatchTool.ParseBlocks has cognitive complexity 42 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":454}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8879b075bc9e13ca1eaf8e9646f8dd815af56f9911948414d2c2dc1e0694db9"}},{"ruleId":"D2","level":"warning","message":{"text":"openDrawer (cognitive 40): openDrawer has cognitive complexity 40 (threshold 15). Drivers by points: ternaries 7 (17 pts), if/else 7 (12 pts), boolean chains 11 (nesting depth added 15). Of this number, 31 points are the body\u0027s own statements and 9 belong to 4 function literals inside it that branch. 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/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1158}}}],"partialFingerprints":{"codehealthFindingId/v1":"b99f8a13d2dc23afa871b67920decfc6d666109698308f7016f0a2e33467c67f"}},{"ruleId":"D2","level":"warning","message":{"text":"ToolCallLogger.FillArguments (cognitive 39): ToolCallLogger.FillArguments has cognitive complexity 39 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":539}}}],"partialFingerprints":{"codehealthFindingId/v1":"6eb7f31e2fbaa4a709dd75807280bea2c13025f28246d4d30767de33604c2762"}},{"ruleId":"D2","level":"warning","message":{"text":"SkillManifest.Parse (cognitive 39): SkillManifest.Parse has cognitive complexity 39 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillManifest.vb"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e990607fc40ce7c3a4b0f8926864c48725431d7bf49528f5ca7bd884988779a"}},{"ruleId":"D2","level":"warning","message":{"text":"HarnessLogReader.Keep (cognitive 38): HarnessLogReader.Keep has cognitive complexity 38 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":555}}}],"partialFingerprints":{"codehealthFindingId/v1":"4328047c445ea1a578bf0f33d516f0a28fdf824d8db5acbdf76d00c09282e767"}},{"ruleId":"D2","level":"warning","message":{"text":"AskUserQuestionTool.Ask (cognitive 32): AskUserQuestionTool.Ask has cognitive complexity 32 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":173}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba27d88fe1e419500b0de2d244e305891a3ed38aa1d1ac3bee4d0c0b22d81ca8"}},{"ruleId":"D2","level":"warning","message":{"text":"AskUserQuestionTool.ParseQuestions (cognitive 32): AskUserQuestionTool.ParseQuestions has cognitive complexity 32 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":438}}}],"partialFingerprints":{"codehealthFindingId/v1":"efffe826bf9928aeda3fcec3fcdde950c5c3e1df01eb104fa622a3c237966fe3"}},{"ruleId":"D2","level":"warning","message":{"text":"LlmJsonExtractor.TryRepairJson (cognitive 32): LlmJsonExtractor.TryRepairJson has cognitive complexity 32 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ToolCalls/LlmJsonExtractor.vb"},"region":{"startLine":180}}}],"partialFingerprints":{"codehealthFindingId/v1":"df25d1b8ade676d73a9e7a4374f69ec1f2b8ce5709411856f6653d8a3a1153fb"}},{"ruleId":"D2","level":"warning","message":{"text":"index.html.window.addEventListener(\u0022DOMContentLoaded\u0022) (cognitive 30): index.html.window.addEventListener(\u0022DOMContentLoaded\u0022) has cognitive complexity 30 (threshold 15). Drivers by points: if/else 10 (15 pts), boolean chains 9, error handling 3 (6 pts) (nesting depth added 8). Of this number, 25 points are the body\u0027s own statements and 5 belong to 2 function literals inside it that branch. 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/WebView2UI/index.html"},"region":{"startLine":1597}}}],"partialFingerprints":{"codehealthFindingId/v1":"235406ce1c77f2c84e4f9621c3277c645ae5fddee73fd6ecca91d0af6a4799d5"}},{"ruleId":"D2","level":"warning","message":{"text":"ApplyPatchTool.Apply (cognitive 29): ApplyPatchTool.Apply has cognitive complexity 29 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":344}}}],"partialFingerprints":{"codehealthFindingId/v1":"8fdd548314db265792efb1c4d2804e8ea701fb50d4a69889ce8f5de1ca2064ec"}},{"ruleId":"D2","level":"warning","message":{"text":"MultiEditTool.multi_edit (cognitive 29): MultiEditTool.multi_edit has cognitive complexity 29 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/MultiEditTool.vb"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e1ba2e4c4c3677087a2f6e7250af1a9b7e4723a1124a9a18758ffe5bd00219e"}},{"ruleId":"D2","level":"warning","message":{"text":"DeepSeekTokenizerAdapter.New.ctor (cognitive 29): DeepSeekTokenizerAdapter.New.ctor has cognitive complexity 29 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/DeepSeekTokenizerAdapter.vb"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"f1ace1f3e078e53eadae63da596b81f95bde64a2063cb1c5b5a3481b07931481"}},{"ruleId":"D2","level":"warning","message":{"text":"HiddenMarkovModel.getOptimalStateSequenceUsingViterbiAlgorithm (cognitive 28): HiddenMarkovModel.getOptimalStateSequenceUsingViterbiAlgorithm has cognitive complexity 28 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/HiddenMarkovModel.vb"},"region":{"startLine":293}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f75d42f6581be472b6e6b42311b15d37c1a6bc52c742bc18fcb61a1ee70ecb0"}},{"ruleId":"D2","level":"warning","message":{"text":"LLMClient.Chat (cognitive 26): LLMClient.Chat has cognitive complexity 26 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/LLMClient.vb"},"region":{"startLine":522}}}],"partialFingerprints":{"codehealthFindingId/v1":"a10fdc8b303d152e0989959927d64eca22780cb24f764e9ca1424436b01ac791"}},{"ruleId":"D2","level":"warning","message":{"text":"ToolCallLogger.RegisterTool (cognitive 25): ToolCallLogger.RegisterTool has cognitive complexity 25 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":795}}}],"partialFingerprints":{"codehealthFindingId/v1":"2cefc36b7f5bc5d5915f680dbb0bcf1e188adfe274b28b6e0a5473fe9e0d11fd"}},{"ruleId":"D2","level":"warning","message":{"text":"GrepTool.CollectMatches (cognitive 25): GrepTool.CollectMatches has cognitive complexity 25 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":257}}}],"partialFingerprints":{"codehealthFindingId/v1":"db0f0adf3a13bfb2a43be3fb5aa1c2b816e151f08bf8f4d50421313af1d81ef9"}},{"ruleId":"D2","level":"warning","message":{"text":"LlmJsonExtractor.ExtractBalancedSubstring (cognitive 24): LlmJsonExtractor.ExtractBalancedSubstring has cognitive complexity 24 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ToolCalls/LlmJsonExtractor.vb"},"region":{"startLine":120}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f6afcd58d22f0e8f36f8fb00e6e44674ee51a16095f39c5c32e678717e5dcda"}},{"ruleId":"D2","level":"warning","message":{"text":"Llama2.Talk (cognitive 24): Llama2.Talk has cognitive complexity 24 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2.vb"},"region":{"startLine":128}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc06ccaee27136978179daeb707f80e3fde62cc23b5bf1782db4e931b90d2dca"}},{"ruleId":"D2","level":"warning","message":{"text":"Validator.isValidEmissionMatrix (cognitive 24): Validator.isValidEmissionMatrix has cognitive complexity 24 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/Model/Validator.vb"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba07377862554c067585b23080c2d166aed32d342b5538037acca2ddda6e93c8"}},{"ruleId":"D2","level":"warning","message":{"text":"McpHttpTransport.ReadSseAsync (cognitive 23): McpHttpTransport.ReadSseAsync has cognitive complexity 23 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpHttpTransport.vb"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5def9f9356774274ec9181fa682036ad9369b6b9712995a6bf6e75867f284a2"}},{"ruleId":"D2","level":"warning","message":{"text":"MultiEditTool.ParseEdits (cognitive 23): MultiEditTool.ParseEdits has cognitive complexity 23 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/MultiEditTool.vb"},"region":{"startLine":172}}}],"partialFingerprints":{"codehealthFindingId/v1":"6215139fd0123ae8d01ef84c2d156facc18de36ecdc5c6d871ed6d63f805aeec"}},{"ruleId":"D2","level":"warning","message":{"text":"HarnessLogReader.Records (cognitive 23): HarnessLogReader.Records has cognitive complexity 23 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":193}}}],"partialFingerprints":{"codehealthFindingId/v1":"aaf00827472ed05513e64d54245c838b90626b14e25a1a7c3ed94eb38fed5c15"}},{"ruleId":"D2","level":"warning","message":{"text":"renderOverview (cognitive 23): renderOverview has cognitive complexity 23 (threshold 15). Drivers by points: boolean chains 9, if/else 6 (7 pts), ternaries 4 (7 pts) (nesting depth added 4). Of this number, 21 points are the body\u0027s own statements and 2 belong to 10 function literals inside it that branch. 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/WebView2UI/Logs/harness-log.js"},"region":{"startLine":511}}}],"partialFingerprints":{"codehealthFindingId/v1":"23b6f5979bc665f65a37b2c59d617b50a278e1fdce05a59e0a1c9b589ad5345f"}},{"ruleId":"D2","level":"warning","message":{"text":"McpConfig.ParseServer (cognitive 22): McpConfig.ParseServer has cognitive complexity 22 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpConfig.vb"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"771f45927a8ecd9bf1061e78ce012df7c461e512c9b89d0c3dffbc47b9c66003"}},{"ruleId":"D2","level":"warning","message":{"text":"turnBodyHtml (cognitive 22): turnBodyHtml has cognitive complexity 22 (threshold 15). Drivers by points: boolean chains 12, ternaries 8, if/else 2. Most of this is not in the body itself: 8 of the 22 points are its own statements and the rest belongs to 3 function literals inside it that branch (lines 733, 727, 750). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":706}}}],"partialFingerprints":{"codehealthFindingId/v1":"e12d107b1663fe225098f2624ab0214da484498f4a019f0ebc492ed9903041d4"}},{"ruleId":"D2","level":"warning","message":{"text":"DemoPipeline.DemoConstrainedDecoding (cognitive 21): DemoPipeline.DemoConstrainedDecoding has cognitive complexity 21 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/Pipeline.vb"},"region":{"startLine":766}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9325b907788ae9b64c91a284b9667170042e1395e63ca2d5bfb1a132d8d6f59"}},{"ruleId":"D2","level":"warning","message":{"text":"ApplyPatchTool.Validate (cognitive 20): ApplyPatchTool.Validate has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":119}}}],"partialFingerprints":{"codehealthFindingId/v1":"a3213fde7a9eb0577dc493b7b132b36f7c4d466d27557eeccfe11b74e83cf99d"}},{"ruleId":"D2","level":"warning","message":{"text":"ApplyPatchTool.ValidateBlock (cognitive 20): ApplyPatchTool.ValidateBlock has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":213}}}],"partialFingerprints":{"codehealthFindingId/v1":"d73afef33197deda04aabb75f4acecbafb5c2b94fafdb3d05ee6d6e9fa56f146"}},{"ruleId":"D2","level":"warning","message":{"text":"GrepTool.grep (cognitive 20): GrepTool.grep has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"b2028c444f4207a0d63aef6035865bcbd27c77521eae978a4270b9a654e6efd5"}},{"ruleId":"D2","level":"warning","message":{"text":"GrepTool.CollectSearchFiles (cognitive 20): GrepTool.CollectSearchFiles has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"8cee706a6c7a98cacf8a28a1053e7ce86b9cec2ad74869936ccc6fe4e0fa6398"}},{"ruleId":"D2","level":"warning","message":{"text":"DeepSeekWebClient.PostJsonAsync (cognitive 20): DeepSeekWebClient.PostJsonAsync has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/Web/DeepSeekWebClient.vb"},"region":{"startLine":200}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f0568adf0886d4eb8bcab4b06a4f7be7b0413d39456db8c23dac77ab327dcf6"}},{"ruleId":"D2","level":"warning","message":{"text":"ChatSessionLogger.BeginRound (cognitive 20): ChatSessionLogger.BeginRound has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/Logging/ChatSessionLogger.vb"},"region":{"startLine":338}}}],"partialFingerprints":{"codehealthFindingId/v1":"c40d73924a1c3af418ee9cc52d9299ae934f652f50bfdf3509782f9e3eb05e84"}},{"ruleId":"D2","level":"warning","message":{"text":"HarnessLogReader.HasKeyword (cognitive 20): HarnessLogReader.HasKeyword has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":641}}}],"partialFingerprints":{"codehealthFindingId/v1":"6d4bf8f19357f4ed40e48aa24a52597aee1b5974ae23a40e6324fb99d4fa02de"}},{"ruleId":"D2","level":"warning","message":{"text":"jsonview-shim.search (cognitive 20): jsonview-shim.search has cognitive complexity 20 (threshold 15). Drivers by points: loops 4 (8 pts), ternaries 2 (7 pts), if/else 2 (5 pts) (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/WebView2UI/Logs/jsonview-shim.js"},"region":{"startLine":100}}}],"partialFingerprints":{"codehealthFindingId/v1":"f68409cfce61ebf62fffc6bf1e09538c4652089d3ffc60f7385817ba3698bdaf"}},{"ruleId":"D2","level":"warning","message":{"text":"ChatContextMemory.CompressAsync (cognitive 19): ChatContextMemory.CompressAsync has cognitive complexity 19 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ContextMemory/ChatContextMemory.vb"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e6619c84d3fbb37c99a19d3d1ad0289320c41c85e572de50c0b2cc8dcee2a16"}},{"ruleId":"D2","level":"warning","message":{"text":"McpToolBridge.CollectCandidates (cognitive 18): McpToolBridge.CollectCandidates has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpToolBridge.vb"},"region":{"startLine":218}}}],"partialFingerprints":{"codehealthFindingId/v1":"0d9d46dfb14b2fa4e48904fa6052f356a707730c41a7959e485e1c0ab181a77c"}},{"ruleId":"D2","level":"warning","message":{"text":"ApplyPatchTool.ApplyHunks (cognitive 18): ApplyPatchTool.ApplyHunks has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":277}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b482f8b6f38520c36dfb10dfff0754622dc4bed1c894cfcf6a12e79ac7c0d2e"}},{"ruleId":"D2","level":"warning","message":{"text":"ShellTool.shell (cognitive 18): ShellTool.shell has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ShellTool.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"af69a902464848969ed3cd50d1f146a16564485d93626e3b60d364b7e7a358f8"}},{"ruleId":"D2","level":"warning","message":{"text":"LogHost.ListSessions (cognitive 18): LogHost.ListSessions has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/LogHost.vb"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"ef4b4099f2fe5ff48567da766016e193f749414bf55e4f1bf3f0e228192d4afc"}},{"ruleId":"D2","level":"warning","message":{"text":"DemoPipeline.MeasureToolCallAccuracy (cognitive 18): DemoPipeline.MeasureToolCallAccuracy has cognitive complexity 18 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/Pipeline.vb"},"region":{"startLine":421}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f68ad826c7d48c04255e34e37489fdd2e0f60812e577c894cc2beadaa9b64f8"}},{"ruleId":"D2","level":"warning","message":{"text":"McpHttpTransport.SendAsync (cognitive 17): McpHttpTransport.SendAsync has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpHttpTransport.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"3df0ee2b80dec39ffa57cc149b69328b986547fefc1bc145872532945c8553d6"}},{"ruleId":"D2","level":"warning","message":{"text":"GlobPattern.Create (cognitive 17): GlobPattern.Create has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GlobPattern.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c8a024ba648a02c212d3525a88f4823bd47d47d26d3661442e0525189e85a7b"}},{"ruleId":"D2","level":"warning","message":{"text":"GlobTool.glob (cognitive 17): GlobTool.glob has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GlobTool.vb"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"39768430634e8fea152c43ffc004ca28f98906241f62cc32ac643f5f84074cd3"}},{"ruleId":"D2","level":"warning","message":{"text":"OpenAIProvider.ConvertToOpenAIMessages (cognitive 17): OpenAIProvider.ConvertToOpenAIMessages has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OpenAIProvider.vb"},"region":{"startLine":330}}}],"partialFingerprints":{"codehealthFindingId/v1":"6437d8a0244f2cbc2948e2e0db253d443fcd44f3375c242c992a2c5c4ccd30ee"}},{"ruleId":"D2","level":"warning","message":{"text":"LlmJsonExtractor.ExtractJsonFromLlmResponse (cognitive 17): LlmJsonExtractor.ExtractJsonFromLlmResponse has cognitive complexity 17 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ToolCalls/LlmJsonExtractor.vb"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c97b45de7ffffc8a3574a8972cea52df693353062a659cdebb275f154ed3ff6"}},{"ruleId":"D2","level":"warning","message":{"text":"McpClient.ListPagesAsync (cognitive 16): McpClient.ListPagesAsync has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpClient.vb"},"region":{"startLine":255}}}],"partialFingerprints":{"codehealthFindingId/v1":"9467ec15c7c4236c2473cbd24d8fa7bb2397d779e0f13afe006e59f3eb667ce6"}},{"ruleId":"D2","level":"warning","message":{"text":"McpToolBridge.ConnectAllAsync (cognitive 16): McpToolBridge.ConnectAllAsync has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpToolBridge.vb"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7df893bfb4b6b567f02c58af12cce3a0700ebeecf17b6962e2839acaf0c8071"}},{"ruleId":"D2","level":"warning","message":{"text":"McpToolBridge.RebuildArguments (cognitive 16): McpToolBridge.RebuildArguments has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpToolBridge.vb"},"region":{"startLine":429}}}],"partialFingerprints":{"codehealthFindingId/v1":"1aec83f33c111195764cfc2d788ca7c563b3eec6efe11179f25ae30d92fccabc"}},{"ruleId":"D2","level":"warning","message":{"text":"SkillManifest.BuildSummary (cognitive 16): SkillManifest.BuildSummary has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillManifest.vb"},"region":{"startLine":261}}}],"partialFingerprints":{"codehealthFindingId/v1":"e36bd4f3aa0236e5e300320b5b4e52131ab5d74b08cf060a3d107649751f6816"}},{"ruleId":"D2","level":"warning","message":{"text":"SkillScriptExecutor.SplitArguments (cognitive 16): SkillScriptExecutor.SplitArguments has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":237}}}],"partialFingerprints":{"codehealthFindingId/v1":"822448e618113f85e962a5ff47954ad1333a67e2b40118ce3988d8dd81e26ff6"}},{"ruleId":"D2","level":"warning","message":{"text":"MoveTool.move (cognitive 16): MoveTool.move has cognitive complexity 16 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/MoveTool.vb"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"7036ab06303883d7da438d491d832f03d52e0dde10ceb4057eaf549428689ac3"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ToolCallLogger: ClassTooLong \u2014 731 significant lines (blank, comment-only and punctuation-only lines excluded), 20 methods. The bar is 400 significant lines; this is 331 over it, 1.83\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/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d4c42ba00af51e64eed2ce6d5862f26b579108b8e5d6795d2d3d93795ef852b4"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Logs/harness-log.js: FileTooLong \u2014 903 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 403 over it, 1.81\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/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"50e87f916bdaecfc661651d5fbb4df47e83f55fe0ff1d217c6a455c87c26580e"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ChatSessionLogger: ClassTooLong \u2014 640 significant lines (blank, comment-only and punctuation-only lines excluded), 25 methods. The bar is 400 significant lines; this is 240 over it, 1.60\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/Ollama/Logging/ChatSessionLogger.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"451b1b875c8d675de97f954b36730fb0a625cec3af6193d4c5b634b2ce513260"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: LLMClient.ChatRound: MethodTooLong \u2014 ChatRound runs 152 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 52 over it, 1.52\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/LLMClient.vb"},"region":{"startLine":614}}}],"partialFingerprints":{"codehealthFindingId/v1":"94a54aab6625404cd094434b5f40b3d9616b4f82982736586393e2354b97ea83"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Logging/ToolCallLogger.vb: FileTooLong \u2014 742 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 242 over it, 1.48\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/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e3f9ef629f833e2598ee9e29adf1df1dff7873a7a7f7b4e1817f5d76422734dd"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: LLMClient: ClassTooLong \u2014 540 significant lines (blank, comment-only and punctuation-only lines excluded), 21 methods. The bar is 400 significant lines; this is 140 over it, 1.35\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/Ollama/LLMClient.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ebf7bf963cc37b62aff9edca8c8a65e55b3698992e0c8af37d06fe2e687bb30"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: DemoPipeline: ClassTooLong \u2014 535 significant lines (blank, comment-only and punctuation-only lines excluded), 17 methods. The bar is 400 significant lines; this is 135 over it, 1.34\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/TalkBuddy/Pipeline.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"bacddeea9f5651f70a5ad3c28fefcdff068a94231bc2283f3ad62354350c7b0a"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WebView2LLMUI: TooManyMethods \u2014 499 significant lines (blank, comment-only and punctuation-only lines excluded), 40 methods. The bar is 30 methods; this is 10 over it, 1.33\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/WebView2LLMUI.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"53244ab550419939002f32f3f3cb6974849566cdaa23ce195bc125e5bb66bcf6"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Logging/ChatSessionLogger.vb: FileTooLong \u2014 650 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 150 over it, 1.30\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/Ollama/Logging/ChatSessionLogger.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"25f0d42cbbe22cb8125756dfbd2da46fb426c9a745c3b694b72fabeecca4f10a"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: HarnessLogReader: ClassTooLong \u2014 491 significant lines (blank, comment-only and punctuation-only lines excluded), 21 methods. The bar is 400 significant lines; this is 91 over it, 1.23\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed71af79d95db181aaa8934d15a920203a21b2d9f872fd4d26c3de1daa6bd4ea"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Logs/HarnessLogReader.vb: FileTooLong \u2014 570 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 70 over it, 1.14\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/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"aaf1f105b464391eaa35c946d5df084090ea806a890d75636cdfbfa235d986f4"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ApplyPatchTool: ClassTooLong \u2014 448 significant lines (blank, comment-only and punctuation-only lines excluded), 14 methods. The bar is 400 significant lines; this is 48 over it, 1.12\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/Harness/Tools/ApplyPatchTool.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd82484cd754bfeee7d71f6bcf8699ba0d5421e1af3d3cff060d162325adc2db"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Ollama/LLMClient.vb: FileTooLong \u2014 550 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 50 over it, 1.10\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/Ollama/LLMClient.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"455bfd759600be0b60db6089ae58c9e4b9eb715563f46407d64e554e39fd5d65"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: TalkBuddy/Pipeline.vb: FileTooLong \u2014 544 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 44 over it, 1.09\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/TalkBuddy/Pipeline.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"22a2df6356281d61a5ad56fa36e065353bc667f44614eed4b2b306ec36f7ce4a"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: WebView2UI/index.html: FileTooLong \u2014 525 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 25 over it, 1.05\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/WebView2UI/index.html"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"08af18af97f1b003013f794ba54ae4a3e055c217da1bc2830e751599f5293d9b"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: WebView2UI/WebView2LLMUI.vb: FileTooLong \u2014 510 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 10 over it, 1.02\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/WebView2UI/WebView2LLMUI.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9bfeff4c65c9f8f63da47c70c77bf208dcddff02c4b09ff7118f1a6aa68d34a6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (28\u201329 lines \u00D7 2): src/Harness/Logging/ToolCallLogger.vb:98-126 | src/Ollama/Logging/JsonArrayAppender.vb:67-94 \u2014 before extracting anything, compare \u0060src/Harness/Logging/ToolCallLogger.vb\u0060 and \u0060src/Ollama/Logging/JsonArrayAppender.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 74 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5c40dbdec6890d234ca57145874a9876aa77bc10dd36d1bf1ab53ed876c140c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24 lines \u00D7 2): src/Harness/Logging/ToolCallLogger.vb:160-183 | src/Ollama/Logging/JsonArrayAppender.vb:122-145 \u2014 before extracting anything, compare \u0060src/Harness/Logging/ToolCallLogger.vb\u0060 and \u0060src/Ollama/Logging/JsonArrayAppender.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 74 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":160}}}],"partialFingerprints":{"codehealthFindingId/v1":"28fac586b65f9ab49fae6095b37a32bcf8cec04d9d26d50984f35f3011dc917d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): src/Harness/Tools/GlobTool.vb:77-97 | src/Harness/Tools/GrepTool.vb:133-153 \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/Harness/Tools/GlobTool.vb"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"656d7e2cda721b3809d1f56b8573fff8c7079c82fb0679a0f31c601ac2e16ebc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 3): src/Harness/Tools/GlobTool.vb:77-93 | src/Harness/Tools/GrepTool.vb:133-149 | src/Harness/Tools/ReadTool.vb:59-75 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GlobTool.vb"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"0da3333da194839f23b9dea335020c59e0812c2b468530535c492b8a86acf9b2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201320 lines \u00D7 4): src/Harness/Tools/MoveTool.vb:44-57 | src/Harness/Tools/MoveTool.vb:59-72 | src/Harness/Tools/MultiEditTool.vb:53-69 | src/Harness/Tools/WriteTool.vb:58-77 \u2014 there are 4 copies across 3 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 4 sites; resolving a subset leaves the remainder to drift apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/MoveTool.vb"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"4618f7aebc399118be8c518b9e629a9051a17bd4a00283a845394003b5d536b4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (20 lines \u00D7 2): src/Harness/Job/JobStartTool.vb:129-148 | src/Harness/Tools/ShellTool.vb:57-76 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/JobStartTool.vb"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"47c553cd21e2650d471643515812c03fbec46b16ea57ab4e6d9be05a992c6ae0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): src/Harness/Logging/ToolBackend.vb:104-121 | src/Ollama/ToolCalls/CLRFunction.vb:166-183 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolBackend.vb"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b8d05c7c234f496faa8814ca4406ee0513326376e444156aaf56baace53f67b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/Harness/Logging/ToolCallLogger.vb:74-88 | src/Ollama/Logging/JsonArrayAppender.vb:44-58 \u2014 before extracting anything, compare \u0060src/Harness/Logging/ToolCallLogger.vb\u0060 and \u0060src/Ollama/Logging/JsonArrayAppender.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 74 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"69f2c5506a8cfcc4fee998f00bf5cac03235c74269dcba77ef2e468f9caa577d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (2\u201312 lines \u00D7 3): src/TalkBuddy/Pipeline.vb:248-249 | src/TalkBuddy/Pipeline.vb:324-335 | src/TalkBuddy/Pipeline.vb:374-385 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060src/TalkBuddy/Pipeline.vb:336\u0060 calls \u0060KeyValue\u0060 and \u0060src/TalkBuddy/Pipeline.vb:386\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/TalkBuddy/Pipeline.vb"},"region":{"startLine":248}}}],"partialFingerprints":{"codehealthFindingId/v1":"3df7f3a33863e7d95756c9fd182b545acdbec873c48b0788c770651371c44862"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): src/Harness/Logging/ToolCallLogger.vb:698-707 | src/Ollama/Logging/ChatSessionLogger.vb:956-965 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":698}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca75e285b93083dc98b3a404f87569783877e0c1f35b083f57d99b0e7ed27172"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): src/Harness/Logging/ToolCallLogger.vb:144-149 | src/Ollama/Logging/JsonArrayAppender.vb:112-117 \u2014 before extracting anything, compare \u0060src/Harness/Logging/ToolCallLogger.vb\u0060 and \u0060src/Ollama/Logging/JsonArrayAppender.vb\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 74 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":144}}}],"partialFingerprints":{"codehealthFindingId/v1":"86b6e8b1f9bd3ea860460798a9c751e7baacb7219adcd99cc4fd026c66a374a7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): src/Harness/Task/AskUserQuestionTool.vb:606-618 | src/Harness/Tools/MultiEditTool.vb:274-286 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":606}}}],"partialFingerprints":{"codehealthFindingId/v1":"a1b14bb42597ee85074941f1d3c6a907cba03afd771e75f162e1f0d288325aaa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): src/Harness/Tools/Web/SearchWebTool.vb:249-258 | src/Harness/Tools/Web/WebFetchTool.vb:207-216 \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/Harness/Tools/Web/SearchWebTool.vb"},"region":{"startLine":249}}}],"partialFingerprints":{"codehealthFindingId/v1":"65436c145998d7a4a924a7ec88a5933188d83367e44e457fdcb0e6366d0248b5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): src/Harness/Tools/Web/SearchWebTool.vb:308-312 | src/Harness/Tools/Web/WebFetchTool.vb:221-225 \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/Harness/Tools/Web/SearchWebTool.vb"},"region":{"startLine":308}}}],"partialFingerprints":{"codehealthFindingId/v1":"75747ad9c824c25beb9737aa6682fbfcf4c9508701c0a46f8ada122beb62b50a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): src/TalkBuddy/Corpus/SftSynthesizer.vb:56-63 | src/TalkBuddy/Corpus/ToolCallSynthesizer.vb:217-224 \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/TalkBuddy/Corpus/SftSynthesizer.vb"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"71af9d8b176a2779634f67c87406555e238a90a3e865e6450a1cc0830defecf5"}},{"ruleId":"D5","level":"warning","message":{"text":"Unstable project Harness: Harness has instability 0.50 with 1 dependents."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Harness.vbproj"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2cd6bae7f29babba6427fc54caa88f6cb15728233af4ffde17ed9ab6786fbe5d"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Ollama: Ollama: abstractness 0.02, instability 0.00, distance 0.98 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/Ollama.vbproj"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa2dd0ad468e7b1e68bf172f326b7ff1930f8d283cae92b1331692ad6160b2a6"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 no coverage collector is wired up: Coverage NOT MEASURED: \u0060--collect:\u0022XPlat Code Coverage\u0022\u0060 names a data collector that ships in the \u0060coverlet.collector\u0060 package, and this repository wires up none \u2014 no test project references it and no runsettings declares one. The absence of coverage here is therefore not evidence about the suite or about our analyzer environment: without a collector, \u0060--collect\u0060 produces nothing even from a suite that builds and passes. Add a \u0060coverlet.collector\u0060 PackageReference to the test project(s) (or commit the Cobertura/OpenCover/lcov report your CI produces) and real coverage will be measured. It is excluded from the score rather than counted as a near-zero defect."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Ollama/LLMClient.vb: src/Ollama/LLMClient.vb changed 34 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 34 in LLMClient.ChatRound at line 614. 4 of those changes were fix/bug commits, and the other 30 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-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Ollama/LLMClient.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/LLMClient.vb"},"region":{"startLine":614}}}],"partialFingerprints":{"codehealthFindingId/v1":"849f546cd1755c4cc7b5a08d45330bcd21e60da7d1bd2b395e1152a7647a73cb"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/WebView2UI/index.html: src/WebView2UI/index.html changed 11 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 21 in index.html.window.addEventListener(\u0022DOMContentLoaded\u0022) at line 1597. 1 of those changes was a fix/bug commit, and the other 10 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-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/WebView2UI/index.html\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/WebView2UI/index.html"},"region":{"startLine":1597}}}],"partialFingerprints":{"codehealthFindingId/v1":"e072ee81f95420442e6731c89d35914889578e1d1992f2a15328c83fd4e8a73a"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Ollama/ToolCalls/CLRFunction.vb: src/Ollama/ToolCalls/CLRFunction.vb changed 8 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 17 in CLRFunction.MakeTypeString at line 64. 2 of those changes were fix/bug commits, and the other 6 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-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Ollama/ToolCalls/CLRFunction.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ToolCalls/CLRFunction.vb"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b1793fc384fbe63276e3c6624d755b38c5aa806fca400901ab8d34964516206"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/TalkBuddy/DeepSeekTokenizerAdapter.vb: src/TalkBuddy/DeepSeekTokenizerAdapter.vb changed 4 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 17 in DeepSeekTokenizerAdapter.New.ctor at line 109. 3 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-07-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/TalkBuddy/DeepSeekTokenizerAdapter.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/DeepSeekTokenizerAdapter.vb"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"893dc569f9691ccf035488c3d08a60a48acef2508f0211b022d485ca77a809e6"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Harness/Tools/GrepTool.vb: src/Harness/Tools/GrepTool.vb changed 3 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 20 in GrepTool.grep at line 89. 3 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-07-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Harness/Tools/GrepTool.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/GrepTool.vb"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"bd8383258b0117a763a90f4983375fc2a77db972dc3526c2812bb83fea5fbaad"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Harness/MCP/McpHttpTransport.vb: src/Harness/MCP/McpHttpTransport.vb changed 3 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 15 in McpHttpTransport.SendAsync at line 43. 2 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-07-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Harness/MCP/McpHttpTransport.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpHttpTransport.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"aad48e65b7c3fbb0dc899f2e7ac9369731fd6ff229f69214697a35db9891df3a"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Harness/SkillSystem/SkillResource.vb: src/Harness/SkillSystem/SkillResource.vb changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 20 in SkillScriptExecutor.Execute at line 101. 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-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Harness/SkillSystem/SkillResource.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"5867dacc568164d3fb6a54f313411aa66e0391c6a59664b813d1b8f09d01291b"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/Harness/Tools/ShellTool.vb: src/Harness/Tools/ShellTool.vb changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 15 in ShellTool.shell at line 43. 1 of those changes was a fix/bug commit, and the other 1 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-07-20..2026-10-18, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-20 14:59:09 \u002B08:00\u0027 --until=\u00272026-10-18 14:59:09 \u002B08:00\u0027 --full-history --no-merges -- src/Harness/Tools/ShellTool.vb\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ShellTool.vb"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"36f1fb5572f50eab0fa1e936339df41305abdb7c67c50f75f8a6cfe37511655f"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 29 significant file(s) lose their only recent owner: src/TalkBuddy/Pipeline.vb, src/Ollama/LLMClient.vb, src/WebView2UI/WebView2LLMUI.vb, src/Harness/SkillSystem/SkillManager.vb, src/Ollama/OpenAIProvider.vb, src/Harness/SkillSystem/SkillAgent.vb, src/Ollama/ContextMemory/MemoryPersistsStorage.vb, src/ollama_rag.py (\u002B21 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"04de65b6cb6dd2870c7e6ad8e9c95c21b47e353c173e866174c0706aa17abe96"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Agent/ChatGLM.vb"},"region":{"startLine":166}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f1e762a32ed943c094cf908e6ad51b4029b19c6613b4849d8810297aabb0bef"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/BackgroundJob.vb"},"region":{"startLine":272}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6ea1687489144a51bfa0f20e638a65f73bf9d457781907578ac7640f23f4fa1"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/BackgroundJob.vb"},"region":{"startLine":332}}}],"partialFingerprints":{"codehealthFindingId/v1":"95364ce515a4fa17801792c7830fea9b9bf11674a2e8463cbe183b68d114bc88"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/ShellJobRunner.vb"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"9192c76776fab92f57e2da445eb99fc64fbd5d598d771673e42b51f6bed9cbaa"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/ShellJobRunner.vb"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"79af5bc026a3c0d0ddff45aea3f2e330fdf5a6996359fedbb395c71692db4a04"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/ShellJobRunner.vb"},"region":{"startLine":100}}}],"partialFingerprints":{"codehealthFindingId/v1":"cded101668a0aac05b93a5bbf700992fd47cddf9b4e21eb93d6438441d4d91ea"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Job/ShellJobRunner.vb"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"38e8a9d53aab0e38ceb7b968620fdf2b5b256a8b873d70fe7afabbb0125737cc"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Logging/ToolCallLogger.vb"},"region":{"startLine":197}}}],"partialFingerprints":{"codehealthFindingId/v1":"6af71c1e9cf5b1782e2353b9ae0166e3b0cbeb4fe3adc9006e02ac9ac473a206"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpClient.vb"},"region":{"startLine":355}}}],"partialFingerprints":{"codehealthFindingId/v1":"679ffed7e7705be9acaff2cf674f7db5e9cd2d296c70cb0f0702694c84eaeb89"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpStdioTransport.vb"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e9dcf366fe2e29c28eb80c167b6dbb2e205205a2736941fe31ad7ed89d5d851"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpStdioTransport.vb"},"region":{"startLine":127}}}],"partialFingerprints":{"codehealthFindingId/v1":"9753542eafb841f7ea05eb1996031587ae76b4e2578c240475ece02dc54a03d6"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpToolBridge.vb"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"564b1053decd28d9a0242f7f711b0b5ef7ece6d9fd3a3c246f24b1360edf2099"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/MCP/McpToolBridge.vb"},"region":{"startLine":534}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea56bbaeedfc295f6d692f5197d2a6cc5f3b302a50d187bff9a5f444669db147"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":193}}}],"partialFingerprints":{"codehealthFindingId/v1":"e66955f5c1653e52edc0f6263f855469062d0dfac72e3d7607091232aa664250"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":203}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f6c94ea6250ad37ecd3c59f002553cd0784b5f9a8ed9a0a7d3ff976ff58cf92"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":208}}}],"partialFingerprints":{"codehealthFindingId/v1":"16421b84fb616cba51605a0942178be2190d2630ab43a1a77722ca75b4314de5"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/SkillSystem/SkillResource.vb"},"region":{"startLine":282}}}],"partialFingerprints":{"codehealthFindingId/v1":"48ad67bac5f2e88faced5c4d2616a776be908d670cce1d8afade0f116d36b4e3"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Task/AskUserQuestionTool.vb"},"region":{"startLine":291}}}],"partialFingerprints":{"codehealthFindingId/v1":"043a0df4b7ebd4391f0e841b55fd49b6da49ad1912ae39027a4fb27db164ed0f"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Harness/Tools/ShellTool.vb"},"region":{"startLine":110}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b86dbd814d10fc73103d57758040287cc461020c8bf9020b36073dafd23ff49"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/Logging/JsonArrayAppender.vb"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba5375261048fd11bf4fbda1c5fdeccfb1b97e989160d96f0209be00a59ce518"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/OpenAIProvider.vb"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"467d9cf6ef50d1c34b974d9a691de3b67c0f488b9a22fb6791da94cc6541faef"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy/ConsoleReport.vb"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"886de263a7aa383e41e61f3409e13b9a1d85262f8ff0050e6fe18c497b7062ac"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/FormLLMUI.vb"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"d24de622fcda1d617a6c66a147979f416466279d7257002aeb6c6583b8aafe14"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogIndex.vb"},"region":{"startLine":227}}}],"partialFingerprints":{"codehealthFindingId/v1":"37db12e73b7a3c5bc6a456cfc816d5201bcc2dea8234358306f3f1b13152d088"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Record it through whatever this codebase already uses to report problems, or let it propagate. Where swallowing one specific failure really is correct \u2014 as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 catch exactly that exception type and write down WHY in a comment on the catch. Both are needed: a comment on a catch-all still swallows every failure nobody anticipated, and a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/HarnessLogReader.vb"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"7fb8a51ecd866c732690091ccb8d5da859dcd7bf603f804f069e14468e074198"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: dispose managed state (managed objects) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ContextMemory/ChatContextMemory.vb"},"region":{"startLine":390}}}],"partialFingerprints":{"codehealthFindingId/v1":"68c36e6edd17a501c662194ae095b955bd0d579269847f24911f1f2179b93e43"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: free unmanaged resources (unmanaged objects) and override finalizer \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ContextMemory/ChatContextMemory.vb"},"region":{"startLine":397}}}],"partialFingerprints":{"codehealthFindingId/v1":"e74fb31cf32a82300d0fa6f865ef6bbf778ab1eb06649b3ad07648073ee4022d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: set large fields to null \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ContextMemory/ChatContextMemory.vb"},"region":{"startLine":398}}}],"partialFingerprints":{"codehealthFindingId/v1":"508b29debbfff0ec81b6e6aec15a7a651a99410d688ecfcdee51059c84f5d295"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 \u0027 TODO: override finalizer only if \u0027Dispose(disposing As Boolean)\u0027 has code to free unmanaged resources \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama/ContextMemory/ChatContextMemory.vb"},"region":{"startLine":403}}}],"partialFingerprints":{"codehealthFindingId/v1":"c1b107a3a48c4f78f3bde6973361d12b5d45be1fa08fe0381379f0b3a955ec55"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: Replace with actual RAG logic (e.g., vector DB lookup, BM25, etc.) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/OllamaProxy/OllamaProxyServer.vb"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b9c1b153d9995aeb107a2b513ac6975a74cc896555a6a2edd64f9e817cb5a56"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: \u91CA\u653E\u6258\u7BA1\u72B6\u6001(\u6258\u7BA1\u5BF9\u8C61) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2Models/ModelReader.vb"},"region":{"startLine":96}}}],"partialFingerprints":{"codehealthFindingId/v1":"d103631357d7e6d9f479c5495970eda9651e052ce854404d05e9471944d49720"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: \u91CA\u653E\u672A\u6258\u7BA1\u7684\u8D44\u6E90(\u672A\u6258\u7BA1\u7684\u5BF9\u8C61)\u5E76\u91CD\u5199\u7EC8\u7ED3\u5668 \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2Models/ModelReader.vb"},"region":{"startLine":99}}}],"partialFingerprints":{"codehealthFindingId/v1":"51aef2267ecd327f223ebc3d1b07321b26b69888345e95452460bb47b063d792"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 TODO: \u5C06\u5927\u578B\u5B57\u6BB5\u8BBE\u7F6E\u4E3A null \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2Models/ModelReader.vb"},"region":{"startLine":100}}}],"partialFingerprints":{"codehealthFindingId/v1":"1de99f607eac091ef3a81ec859a462db7ac68b3b3e00cb191864913eea6f1893"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: \u0027 \u0027 TODO: \u4EC5\u5F53\u201CDispose(disposing As Boolean)\u201D\u62E5\u6709\u7528\u4E8E\u91CA\u653E\u672A\u6258\u7BA1\u8D44\u6E90\u7684\u4EE3\u7801\u65F6\u624D\u66FF\u4EE3\u7EC8\u7ED3\u5668 \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060\u0027 REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2Models/ModelReader.vb"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"df6359fab0f3d4715d73f11d03bcb7e85f53aa16d872d4913170c2c1fc691ead"}},{"ruleId":"D18","level":"warning","message":{"text":"Solution references 5 projects that do not exist: \u0060src/Ollama.slnx\u0060 declares 5 project file(s) that are missing from the tree (\u0060Core.vbproj\u0060, \u0060JSON-netcore5.vbproj\u0060, \u0060markdown.NET5.vbproj\u0060, \u0060text.yaml.vbproj\u0060, \u0060LiteJs.vbproj\u0060). A solution-level restore or build fails on a dangling reference, so anyone building the solution rather than an individual project is blocked. Delete the stale entry from the solution, or restore the missing project file."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Ollama.slnx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0b99ec33ac285338acea61b6e6796c700ad149aa1364847c8ba3e4b46111acc"}},{"ruleId":"D18","level":"warning","message":{"text":"Solution references 13 projects that do not exist: \u0060src/LLMs.slnx\u0060 declares 13 project file(s) that are missing from the tree (\u0060Rsharp-netcore5.vbproj\u0060, \u0060R-cmd.NET5.vbproj\u0060, \u0060Flute.NET5.vbproj\u0060, \u0060dataframework-netcore5.vbproj\u0060, \u0060Trinity.NET5.vbproj\u0060, \u0060graph-netcore5.vbproj\u0060, \u002B7 more). A solution-level restore or build fails on a dangling reference, so anyone building the solution rather than an individual project is blocked. Delete the stale entry from the solution, or restore the missing project file."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/LLMs.slnx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"bd1af9bc6329e27de85a9c9da76e271d94138d9f44ff4af0d58cba35dae6ac4c"}},{"ruleId":"D18","level":"warning","message":{"text":"Solution references 12 projects that do not exist: \u0060src/TalkBuddy.slnx\u0060 declares 12 project file(s) that are missing from the tree (\u0060dataframework-netcore5.vbproj\u0060, \u0060DataMining.NET5.vbproj\u0060, \u0060DeepLearning.NET6.vbproj\u0060, \u0060machine_learning-netcore5.vbproj\u0060, \u0060TensorFlow.vbproj\u0060, \u0060Math.NET5.vbproj\u0060, \u002B6 more). A solution-level restore or build fails on a dangling reference, so anyone building the solution rather than an individual project is blocked. Delete the stale entry from the solution, or restore the missing project file."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/TalkBuddy.slnx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"57032e3c7264e3098fa3d59bdc19ef22491ef1357537228bc73db8bb351d6db7"}},{"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":"D34","level":"error","message":{"text":"Orphaned knowledge: No living knowledge remains for this large file \u2014 its last meaningful change has decayed away; if it breaks, no one currently understands it. Schedule a read-through / add characterisation tests before it bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/HiddenMarkovModel.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"79e7fd634c592309fae077911244f1ed4a32abf386f07792ac9731e2863575a9"}},{"ruleId":"D34","level":"error","message":{"text":"Orphaned knowledge: No living knowledge remains for this large file \u2014 its last meaningful change has decayed away; if it breaks, no one currently understands it. Schedule a read-through / add characterisation tests before it bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3f24b686bbaa593997fd53f6718dac8de1c664a98670241a162adedf8ea1e58"}},{"ruleId":"D34","level":"note","message":{"text":"Further orphaned files (smaller): 5 smaller file(s) also have no living knowledge \u2014 folded into the freshness score and metrics rather than raised one row each \u2014 most significant first: src/Agent/ChatGLM.vb, src/Talk/Llama2Models/ModelReader.vb, src/HMM/Model/Validator.vb, src/Agent/GLMBatchTask.vb, src/Talk/HMM/Generator.vb (7 orphaned of 85 analysed files in total, counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 85 of the 118 production source files in this repository met that bar). Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned file: No one still active in this repository holds living knowledge of this file: everyone who made a meaningful change to it has gone quiet, or what the current team did to it (moves, reformatting, log-level or import edits, renames) never needed understanding it. If it breaks, no one currently understands it. Attach the read to the next change that touches it: have a second person review that change, and leave behind a short comment or test recording what the file is for."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Agent/ChatGLM.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b70896ea28d5177233efc9b10da5368aa3344e538f606a0f04ca0a28d14cb420"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned file: No one still active in this repository holds living knowledge of this file: everyone who made a meaningful change to it has gone quiet, or what the current team did to it (moves, reformatting, log-level or import edits, renames) never needed understanding it. If it breaks, no one currently understands it. Attach the read to the next change that touches it: have a second person review that change, and leave behind a short comment or test recording what the file is for."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/Llama2Models/ModelReader.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c6f435a7d023675d6ddc44f6001015ea0e8e432005031296aaa01c5ac1ebfa87"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned file: No one still active in this repository holds living knowledge of this file: everyone who made a meaningful change to it has gone quiet, or what the current team did to it (moves, reformatting, log-level or import edits, renames) never needed understanding it. If it breaks, no one currently understands it. Attach the read to the next change that touches it: have a second person review that change, and leave behind a short comment or test recording what the file is for."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/HMM/Model/Validator.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"80a7ddb78c8ac4b4218e3e1f2652a2986114aad35437643be736560758871ea7"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned file: No one still active in this repository holds living knowledge of this file: everyone who made a meaningful change to it has gone quiet, or what the current team did to it (moves, reformatting, log-level or import edits, renames) never needed understanding it. If it breaks, no one currently understands it. Attach the read to the next change that touches it: have a second person review that change, and leave behind a short comment or test recording what the file is for."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Agent/GLMBatchTask.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"40fb1c28755450bffaa6eeca960c1cec8c301aad33857b649be44eb2c4ef04e5"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned file: No one still active in this repository holds living knowledge of this file: everyone who made a meaningful change to it has gone quiet, or what the current team did to it (moves, reformatting, log-level or import edits, renames) never needed understanding it. If it breaks, no one currently understands it. Attach the read to the next change that touches it: have a second person review that change, and leave behind a short comment or test recording what the file is for."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Talk/HMM/Generator.vb"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"17acc91b377bc6574322554ad6a8eb0c8a6e396bc110fc79eb591d6471b54a9d"}},{"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 2/11 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: README.md names \u0060SKILL.md\u0060, but no file or folder \u0060SKILL.md\u0060 exists. 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":"README.md"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"b85864071a30ab325207b0475304e4f6aa1b79c8e2996b50e4173a6d35a328ea"}},{"ruleId":"P1","level":"warning","message":{"text":"No CI pipeline: No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, \u2026) \u2014 changes aren\u0027t gated by an automated build/test."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"44f01af96e50474fba2df84d95ddf4f7e1d34c29c307830b9efc9057daa6b670"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add a Roslyn security analyzer package referenced from the project (the analyzer packages do analyse VB.NET), plus gitleaks for committed secrets \u2014 CodeQL has no VB.NET extractor, so its csharp pack would extract nothing from this tree and then report it clean \u2014 this repository has no CI pipeline yet, so run it locally to clear the existing findings, then make it a step of the first workflow you add so a regression fails the build. What was searched, so you can tell an absence from a miss: the 0 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}},{"ruleId":"P6","level":"note","message":{"text":"Changelog looks stale/thin: The changelog this check read carries 0 versioned entries, against a bar of 3. Counted as either the distinct \u0060x.y.z\u0060 versions it names (a link\u0027s target is not counted) or its version-shaped section headings (\u0060## [1.2.0]\u0060, \u0060## V14.x.x\u0060, \u0060## Updates in 8.0.x\u0060), whichever is the larger \u2014 so a log sectioned by minor series counts its sections, not its individual patch numbers. Entries recorded somewhere this check does not read \u2014 release notes generated at tag time from a generator config it did not recognise, or a log kept outside the repository \u2014 are not counted here, and this row is about what is re-findable in the tree rather than about how often you release."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c9a74c249fe67853238d9c15085f7f97d31f51bb82455046aa10c2cef82f67b8"}},{"ruleId":"R1","level":"warning","message":{"text":"Type Safety: 0 typed \u00B7 5 plain JS \u2014 the untyped files are src/WebView2UI/Logs/harness-log.js, src/WebView2UI/Logs/jsonview-shim.js, test/request.js, vignettes/_assets/R_syntax.js, vignettes/_assets/highlights.js."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d00ee7953f55325a823d27e5db9657c80ca9b9861c0c0abf8fe487cd3fa586d3"}},{"ruleId":"R10","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2 locations): src/WebView2UI/Logs/harness-log.js:616 \u00B7 src/WebView2UI/Logs/harness-log.js:921 \u2014 all 2 copies are in the same file, and the CITED SPAN is not a self-contained block \u2014 it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead \u2014 the whole function, component or branch these lines sit in \u2014 and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":616}}}],"partialFingerprints":{"codehealthFindingId/v1":"eebb71fead8939564ddcc2b3cfa0e5615c441d41eff60789dae6f6d6ee33646b"}},{"ruleId":"R10","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2 locations): src/WebView2UI/Logs/harness-log.js:1123 \u00B7 src/WebView2UI/Logs/harness-log.js:1277 \u2014 all 2 copies are in the same file, and the CITED SPAN is not a self-contained block \u2014 it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead \u2014 the whole function, component or branch these lines sit in \u2014 and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1123}}}],"partialFingerprints":{"codehealthFindingId/v1":"a3e46154da948f44f3c0dc9ec14941456585997047b1c0334eaf13a1d7635fdd"}},{"ruleId":"R10","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 3 locations): src/WebView2UI/Logs/harness-log.js:1030 \u00B7 src/WebView2UI/Logs/harness-log.js:1123 \u00B7 src/WebView2UI/Logs/harness-log.js:1277 \u2014 all 3 copies are in the same file, and the CITED SPAN is not a self-contained block \u2014 it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead \u2014 the whole function, component or branch these lines sit in \u2014 and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1030}}}],"partialFingerprints":{"codehealthFindingId/v1":"2702d6c8e1543f58685a5df6a332a9e7cdd9f797f850a51367faf454ce9b3504"}},{"ruleId":"R10","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2 locations): src/WebView2UI/Logs/harness-log.js:769 \u00B7 src/WebView2UI/Logs/harness-log.js:1220 \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/WebView2UI/Logs/harness-log.js"},"region":{"startLine":769}}}],"partialFingerprints":{"codehealthFindingId/v1":"15e808e22abd51c35293a60054f20e9362f062385c3a26c47e1ea2749fa806a8"}},{"ruleId":"R2","level":"warning","message":{"text":"Complex function renderOverview (cyclomatic 22, cognitive 21): renderOverview has cyclomatic complexity 22 and cognitive complexity 21; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth \u2014 arms side by side rather than stacked \u2014 and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository\u0027s own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":511}}}],"partialFingerprints":{"codehealthFindingId/v1":"6af0d56359790ecc965020ba7342b98e20144fa20a9fb745b4bf157c3cf0264a"}},{"ruleId":"R2","level":"warning","message":{"text":"Complex function openDrawer (cyclomatic 21, cognitive 31): openDrawer has cyclomatic complexity 21 and cognitive complexity 31; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side \u2014 extracting each decision into its own named function is the change that pays. Measured by this repository\u0027s own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1158}}}],"partialFingerprints":{"codehealthFindingId/v1":"8aa4f509dc9ec769dc0827797c36e42afb4a8d8b5bed68ef02a9251b05d382cf"}},{"ruleId":"R2","level":"warning","message":{"text":"Complex function (anonymous) (cyclomatic 13, cognitive 12): (anonymous) has cyclomatic complexity 13 and cognitive complexity 12; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth \u2014 arms side by side rather than stacked \u2014 and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository\u0027s own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":733}}}],"partialFingerprints":{"codehealthFindingId/v1":"acaf3b298daacbfc4d9b713f3e97dc23b2bf73bb69af5f57d062bf701bbb66bc"}},{"ruleId":"R2","level":"warning","message":{"text":"Complex function (anonymous) (cyclomatic 12, cognitive 12): (anonymous) has cyclomatic complexity 12 and cognitive complexity 12; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side \u2014 extracting each decision into its own named function is the change that pays. Measured by this repository\u0027s own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":239}}}],"partialFingerprints":{"codehealthFindingId/v1":"57d5800a9dd33fa629ceb3f66b2d64e7853d3b284577ba6cdff1a6b1fef7182b"}},{"ruleId":"R2","level":"warning","message":{"text":"Complex function renderRaw (cyclomatic 12, cognitive 12): renderRaw has cyclomatic complexity 12 and cognitive complexity 12; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side \u2014 extracting each decision into its own named function is the change that pays. Measured by this repository\u0027s own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1008}}}],"partialFingerprints":{"codehealthFindingId/v1":"6cebca3a1b662b9024b83c3527cf143502fbc2271ab4a169e938aeac22c18071"}},{"ruleId":"R3","level":"warning","message":{"text":"src/WebView2UI/Logs/harness-log.js: 1098 lines of code: src/WebView2UI/Logs/harness-log.js has 1098 significant lines (blank lines excluded), over the 400-line bar, and contains logic \u2014 a file this long is usually several responsibilities in one module."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e10fff55d2402f5255e4a8e779551d3602b0f568ec8c8fbda705c3960fc0f6e"}},{"ruleId":"R7","level":"warning","message":{"text":"Dead file (~1309 LoC): no import path from any entry point (2 application, 3 tooling, 1 test roots considered), and no other file in the scanned tree imports it \u2014 nothing in-repo names this module at all, which is the strongest form of this claim the import graph can make"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/harness-log.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d4d4da963807f969f1c7ed4511360ca89e1b564c0c4960a531ad4b8ee5caf2fd"}},{"ruleId":"R7","level":"warning","message":{"text":"Dead file (~217 LoC): no import path from any entry point (2 application, 3 tooling, 1 test roots considered), and no other file in the scanned tree imports it \u2014 nothing in-repo names this module at all, which is the strongest form of this claim the import graph can make"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/WebView2UI/Logs/jsonview-shim.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"4997ee5f2607f1dcc2ee97803c93020785d7764552d58275944fba603b2bc750"}},{"ruleId":"SC1","level":"warning","message":{"text":"NuGet dependencies are not locked: No packages.lock.json and no central package management \u2014 restores aren\u0027t reproducible or pinned (SSDF PW.4.4). Enable \u003CRestorePackagesWithLockFile\u003Etrue\u003C/RestorePackagesWithLockFile\u003E (commit the lockfile) or adopt Directory.Packages.props. Advisory \u2014 never scored."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0a97b4f69ccac509400ece7eedefb06d3ede0522cd4d8bcb5c068bea719545a6"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"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-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":"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/"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":0,"secretScannerRunsExcluded":0}}}]}