{"$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":"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":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D10","name":"Test Quality","shortDescription":{"text":"Test Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D10"},{"id":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"id":"D12","name":"Dependency Hygiene","shortDescription":{"text":"Dependency Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/D12"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"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":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D33","name":"JS/npm Dependency Vulnerabilities","shortDescription":{"text":"JS/npm Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D33","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D37","name":"Vulnerability-disclosure Policy","shortDescription":{"text":"Vulnerability-disclosure Policy"},"helpUri":"https://codehealth.canine.dev/dimensions/D37","relationships":[{"target":{"id":"CWE-1059","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1059"]}}]}},"results":[{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: MonacoEditor: TooManyMethods \u2014 83 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Assets/javascript/monacoEditor.ts"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"82675f787262e1bc9aaf48303232cc506ba4642995f8503d69481259ee4055da"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: TextModel: TooManyMethods \u2014 105 significant lines (blank, comment-only and punctuation-only lines excluded), 51 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/MonacoEditor/Monaco/TextModel.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"eaf811029b79fc26dd3a6bb5a46a3a418a4209b74ab1278347ebfb53b62d2cae"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: javascript/monacoEditor.ts: FileTooLong \u2014 684 significant lines (blank, comment-only and punctuation-only lines excluded), about 99% of them inside a single declaration: MonacoEditor (14-1078). Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Assets/javascript/monacoEditor.ts"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"96f1f2b6f2fc0832ba71008a3c6148113e331d715c7c8f4e0f53bbb26ba4eaf0"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: RicherMonacoEditorBase: TooManyMethods \u2014 316 significant lines (blank, comment-only and punctuation-only lines excluded), 41 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/MonacoEditor/RicherMonacoEditorBase.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"674d8410d678e6704d73451813cc9beffc4a653c00683d6823bc3d9ff4fdbd15"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: GUI: TooManyMethods \u2014 128 significant lines (blank, comment-only and punctuation-only lines excluded), 41 methods, declared across 13 files: Components/IUIDropDownButton.cs (6), Components/IUIUtf32IconButton.cs (5), Components/IUIUtf32IconDropDownButton.cs (5), Components/IUIDataGridRow.cs (4), \u002B9 more file(s). The type holds no instance state, so there is no shared data to group its members by. To reduce it, split it by area instead: give each cohesive family of members its own smaller type, so no one type has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Api/Tool/GUI/Components/IUIDropDownButton.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8c6bb0f8257a3c708cee5053dc88dc068ab750fa2711f86f7b28a6d5151f930b"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: ObservableHashSet: TooManyMethods \u2014 141 significant lines (blank, comment-only and punctuation-only lines excluded), 34 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Api/Core/ObservableHashSet.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ae5465d3e0af0fe347968e9efd53a42ad9574bb743fd754870a9552fe26c3b5"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: MonacoEditorBase: TooManyMethods \u2014 124 significant lines (blank, comment-only and punctuation-only lines excluded), 33 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/MonacoEditor/MonacoEditorBase.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"23f372da0e53286023b7a36c49bfaa574dcf6d3c5728d414f2702f975435f28e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19 lines \u00D7 2): src/app/dev/DevToys.Blazor/Components/UIElements/UIFileSelectorPresenter.razor.cs:213-231 | src/app/dev/DevToys.Blazor/Components/UIElements/UIFileSelectorPresenter.razor.cs:240-258 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Blazor/Components/UIElements/UIFileSelectorPresenter.razor.cs:213\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIFileSelectorPresenter.razor.cs"},"region":{"startLine":213}}}],"partialFingerprints":{"codehealthFindingId/v1":"f09aeaa39feaed6cba56561ef35601ae6d78bf64b306d0878c309aeceedbc10e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): src/app/dev/DevToys.Api/Core/ObservableHashSet.cs:246-262 | src/app/dev/DevToys.Api/Core/ObservableHashSet.cs:272-288 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Api/Core/ObservableHashSet.cs:246\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Api/Core/ObservableHashSet.cs"},"region":{"startLine":246}}}],"partialFingerprints":{"codehealthFindingId/v1":"996d951a94b478c7d0a6d9b5463c2365c3af6d6be56a59034463c2491b7ce90d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/app/dev/platforms/desktop/DevToys.CLI/Core/FileStorage/FileStorage.cs:47-61 | src/app/dev/platforms/desktop/DevToys.Windows/Core/FileStorage.cs:46-60 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/platforms/desktop/DevToys.CLI/Core/FileStorage/FileStorage.cs:47\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.CLI/Core/FileStorage/FileStorage.cs"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"d56d4c5624de98749c34ee579ff66a8772346d6b7708d8d8d1a1e9a8efe9db7f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/app/dev/DevToys.Core/Tools/Metadata/GuiToolGroupMetadata.cs:31-45 | src/app/dev/DevToys.Core/Tools/Metadata/GuiToolMetadata.cs:77-91 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Core/Tools/Metadata/GuiToolGroupMetadata.cs:31\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Tools/Metadata/GuiToolGroupMetadata.cs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b2a8b4bfb90f50dad7752d2a5a17bc27a25b8650c12c9d8e87489e2b1829a31"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs:50-63 | src/app/dev/platforms/desktop/DevToys.Windows/Core/SettingsStorage.cs:51-64 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs:50\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"0da049d1773af95b017b796babdf3b8409ee0ab7e9a4112fe460f13cca198771"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): src/app/dev/DevToys.Blazor/Components/Feedback/ProgressBar/ProgressBar.razor.cs:24-37 | src/app/dev/DevToys.Blazor/Components/Feedback/ProgressRing/ProgressRing.razor.cs:30-43 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Blazor/Components/Feedback/ProgressBar/ProgressBar.razor.cs:24\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Feedback/ProgressBar/ProgressBar.razor.cs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"951d0d19ba1ada57b9d90f2a0ccb09aa64198397a538ec093061e627af155472"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs:78-88 | src/app/dev/platforms/desktop/DevToys.Windows/Core/SettingsStorage.cs:79-89 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs:78\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.CLI/Core/Settings/SettingsStorage.cs"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"dfefc937f3646245c342e68e3404b481fc66b68f4ab872cb9c27f843899d7e69"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/app/dev/DevToys.Core/Tools/SmartDetectionService.cs:67-77 | src/app/dev/DevToys.Core/Tools/SmartDetectionService.cs:84-94 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Core/Tools/SmartDetectionService.cs:67\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Tools/SmartDetectionService.cs"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"1948c904ef92e6fa9ecf6edad6d2683691124ff20017a030e967d8dae4d1b247"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): src/app/dev/DevToys.Blazor/Components/Collections/DataGrid/DataGrid.razor.cs:278-286 | src/app/dev/DevToys.Blazor/Components/Collections/ListBox/SelectBase.cs:219-227 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/app/dev/DevToys.Blazor/Components/Collections/DataGrid/DataGrid.razor.cs:278\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Collections/DataGrid/DataGrid.razor.cs"},"region":{"startLine":278}}}],"partialFingerprints":{"codehealthFindingId/v1":"d7887056052e6880cd81f5a8a9c9ee2231420a6d2a6562c7e0c752c80b87e14e"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: DevToys.Localization: DevToys.Localization: abstractness 0.00, instability 0.17, distance 0.83 \u2014 zone of pain \u2014 concrete and depended on by 5 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"21eb2c1a99e3f6a10d0c5a41a4c8df0c47ecfbec4e20a6cf9a5697492e29898d"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SettingsGuiTool (LCOM4 5): SettingsGuiTool\u0027s methods form 5 groups that share no state and don\u0027t call each other \u2014 a sign it may have several responsibilities. Review whether it splits into focused classes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/BuiltInTools/Settings/SettingsGuiTool.cs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"c8277cb89fba3409d082b480a1b29c5ad9854e1ff0cabb96e600cd45f4c81770"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: GuiToolProvider (LCOM4 4): GuiToolProvider\u0027s methods form 4 groups that share no state and don\u0027t call each other \u2014 a sign it may have several responsibilities. Review whether it splits into focused classes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Tools/GuiToolProvider.cs"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"d3aa57f81cf8023c10eef5adbea4569bcd928d2e0cd0ff8cbecd381bbbec0932"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: MicaWindowWithOverlay (LCOM4 4): MicaWindowWithOverlay\u0027s methods form 4 groups that share no state and don\u0027t call each other \u2014 a sign it may have several responsibilities. Review whether it splits into focused classes."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.Windows/Controls/MicaWindowWithOverlay.xaml.cs"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d60b454fe36c3abefb8fdf91abb46bdb883cbf1c61148bca1213d1633ca8917"}},{"ruleId":"D11","level":"warning","message":{"text":"Test reliability not measured \u2014 test suite did not build: Test reliability NOT MEASURED: this repository did not build in our analyzer environment (a C#/MSBuild compiler error), so no test ever ran and flakiness could not be exercised. It is excluded from the score rather than counted as a defect. We did not read WHERE the failing diagnostic is, so this does not claim the fault is in your test code \u2014 a repository written for an older SDK band can compile for you and not for us. Run \u0060dotnet build\u0060 on this commit: if it succeeds, the gap is ours and nothing here is a defect in your code."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"daf09aaa4e14533ca5ff2442790c6867a248b3a07dae410d48e7d6405d6fec30"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 2 significant file(s) lose their only recent owner: src/app/dev/DevToys.Core/Tools/GuiToolProvider.cs, src/app/dev/platforms/desktop/DevToys.Windows/Controls/MicaWindowWithOverlay.xaml.cs. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0555d9573518996f680ea17189859c23f34e7913922b928db1910e9b9eb09b2c"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 1 other contributor(s) are each the sole owner of a small amount of code below the off-boarding threshold \u2014 folded into the bus-factor score and metrics (3 single-owned of 167 analysed files in total, counted over production source files of roughly 100 lines or more, excluding tests, vendored, generated and example/demo trees, largest first). They are anonymized user #2 (1 file(s)) \u2014 spread or document their files in the same way, at lower priority than the named off-boarding risks above."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6eac528f2429e724e1a97ed868f79eefbcf1888dab4af9a9e673429369bb5b2f"}},{"ruleId":"D17","level":"error","message":{"text":"NoWarnInCsproj \u2014 6 warning codes suppressed in one element: A single \u003CNoWarn\u003E suppresses 6 warning codes (CS0649;CS0169;CA1050;CA1822;CA2211;IDE1006) in one stroke \u2014 one team-wide decision, not 6 independent debts. Review the set centrally rather than code-by-code; each code you can re-enable is one less blanket suppression. (Every code still counts toward the score and metrics.)"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/build/_build.csproj"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"3767039c36a7c7d70d572e8a7355b581dcfc67c7d22386da01e3d8dfd222655d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Finish this tool. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/BuiltInTools/SupportDevelopment/SupportDevelopmentGuidTools.cs"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"659fe10e9d3ef27142c732bd4390e8084244ba9aa254ef45abf176b6193a5fa7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Actually handle keyboard shortcuts? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Menu/ContextMenu/ContextMenuItem.cs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"93c3f203029bc64c1c65e20d6b02ac01850e9427bbfe0171b6dbe275c04f1144"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/MonacoEditor/MonacoEditorBase.cs"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"f063407c0137979fb779a6f36e7c8bf05eb82aeb32345656809d0a863429941b"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/TextBox/TextBox.razor.cs"},"region":{"startLine":128}}}],"partialFingerprints":{"codehealthFindingId/v1":"499205d839834da502a773bd260b200fc7a91e1f77b8bed7d775920a468601a9"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Core/Services/DocumentEventService.cs"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"21dd246b6bf80ca05bc227069d77decfd64b48975a7cdc6fe80ae25434633399"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/FileHelper.cs"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"2411af6282f81d775e02e374802c155c36b6683ad833a7499dca827066a6098a"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/FileHelper.cs"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"23a5aa0332198d37b338c7310b210fd3f6e994d2ea4c6489b77588e9036c1796"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Logging/FileLoggerProvider.cs"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"5231c7438cf9f6ad2d0fbb87ab54af78ee59d4b76e3c063a7480450f2f45f02d"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.CLI/Core/OneOfParser.cs"},"region":{"startLine":324}}}],"partialFingerprints":{"codehealthFindingId/v1":"770af7e826f7a1397266817796cc896579235c0ace3bba660928c61618e3219b"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.Windows/Controls/MicaWindowWithOverlay.xaml.cs"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb23a77973c5b3bc3a6352509fdf03a617a375f59a832fbbb388e34f01691dfc"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/platforms/desktop/DevToys.Windows/Controls/MicaWindowWithOverlay.xaml.cs"},"region":{"startLine":281}}}],"partialFingerprints":{"codehealthFindingId/v1":"75d312faee0a34a98dcb0229b5fc67568068a18850db975617161f470194ad11"}},{"ruleId":"D17","level":"warning","message":{"text":"BarePragmaDisable: #pragma warning disable BL0005 \u2014 the disable has no matching restore, so it does not end with the construct that needed it: it runs to the end of the file and silences the rule for everything written below, including code added years later. Close it with the matching restore directive immediately after the construct it covers, or fix the cause and drop the directive entirely."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/Text/MonacoEditor/RicherMonacoEditorDiffBase.cs"},"region":{"startLine":212}}}],"partialFingerprints":{"codehealthFindingId/v1":"3adc0904d2fd8c2af42e3bfafe4b69ceb1e936781f7f7ab89a1abdc5a2b6f6dc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Try Catch and log? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIButtonPresenter.razor.cs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"b571205b2e40ea92a6adb19d04006df5b4d369b30515d2cd59009b0d8403748d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Try Catch and log? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIInfoBarPresenter.razor.cs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"e14803d8172252861909c38330eaa223891a2d4cee5eae6f1a0e356f51c37b91"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Try Catch and log? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIInfoBarPresenter.razor.cs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"d68a2920efd52a0c08faa13e81de1744a2d314e1595160d7e6b3f765bb138269"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Add logs. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIImageViewerPresenter.razor.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8fbdaba64571ee749db76084143745beefc57d2b22e3096d969ef867ee778c09"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Log this. Also, maybe display a message to the user that the image cannot be displayed? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIImageViewerPresenter.razor.cs"},"region":{"startLine":262}}}],"partialFingerprints":{"codehealthFindingId/v1":"db5c5bded1a6d82ca78ef75632665ebbefe01078122015e3025852ddfe167bdc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: if language is plain text, then should we hide line numbers and few other things? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UIMultiLineTextInputPresenter.razor.cs"},"region":{"startLine":227}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3fc0c777ecb818c6bf57f9442866aedc57fa72b97200b7cb4c0e26bee16dbf6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Display ContentDialog \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UITextInputWrapper.razor.cs"},"region":{"startLine":252}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7589f8183c426d3de118fc5defabda2b2153b63e662604ddbc0d10018d1d319"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Display ContentDialog \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UITextInputWrapper.razor.cs"},"region":{"startLine":316}}}],"partialFingerprints":{"codehealthFindingId/v1":"756bc81f585babcc1ad54f3fa4318884bd0553e8eddec24f1f33a1216ea64804"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Detect encoding. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UITextInputWrapper.razor.cs"},"region":{"startLine":294}}}],"partialFingerprints":{"codehealthFindingId/v1":"03153ab5fac0822e4b84a3ef318c2f535afa6882e68936140377df8cd2a28ccb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Perf risk. Read in chunks. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Blazor/Components/UIElements/UITextInputWrapper.razor.cs"},"region":{"startLine":296}}}],"partialFingerprints":{"codehealthFindingId/v1":"32ca5b4caebd3efa0bd887ee04bb93844fc7091e19941ba5f7b952a169b889f6"}},{"ruleId":"D17","level":"warning","message":{"text":"HackComment: // hack because of this: https://github.com/dotnet/corefx/issues/10361 \u2014 a workaround marked in source: record what it is compensating for and what would allow its removal (the upstream fix, the API it is waiting on, the invariant it restores), so the next reader can judge whether it is still needed rather than rediscovering why it is there."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/OSHelper.cs"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"58eefb6a4e49e528e3200e52747fe2b08bb069beb5b52855d2d0576a6430051d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: We should let the user know that something went wrong. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Tools/GuiToolProvider.cs"},"region":{"startLine":436}}}],"partialFingerprints":{"codehealthFindingId/v1":"0ad8c5faf691e5cd1591e21f4ee88b6bf64def6a1ad32a5fb6c4091ae808c979"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: We should let the user know that something went wrong. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/dev/DevToys.Core/Tools/GuiToolProvider.cs"},"region":{"startLine":625}}}],"partialFingerprints":{"codehealthFindingId/v1":"895a49e37e5818afdee0cf6f9ff725a694ae2de63f2ba66b9829ff1b3c31081e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: prompt the user to type in the console a relative or absolute file path that has one of the file types indicated. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/tests/DevToys.UnitTests/Mocks/MockIFileStorage.cs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"b7cde58fe81d00fa5d858643896351c441919330a7411ad7c175ef8a4920c07c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: prompt the user to type in the console a relative or absolute file path that has one of the file types indicated. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/app/tests/DevToys.UnitTests/Mocks/MockIFileStorage.cs"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"b96b0b4cd1b371a1356f234cf9e67f3c1c8063a85152d6d2c54066e2a27d6301"}},{"ruleId":"D18","level":"warning","message":{"text":"Monorepo: only 1 of 4 solutions was scored: This repository contains 4 .NET solutions, but a scan analyzes ONE. Every score, lens, and finding here reflects only \u0060src/DevToys-Windows.sln\u0060 \u2014 the other 3 (\u0060src/Build.sln\u0060, \u0060src/DevToys-Linux.sln\u0060, \u0060src/DevToys-MacOS.sln\u0060) were not analyzed and are not represented in the headline. To cover them, scan each solution as its own target and group them in a Solution or Product for a portfolio roll-up. If a secondary solution is an archived or vendored tree, declare it \u2014 \u0060.gitattributes\u0060 (\u0060path/** linguist-vendored\u0060) or \u0060.editorconfig\u0060 (\u0060[path/**] generated_code = true\u0060) \u2014 to exclude it from discovery the same way generated code is."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62fa065c747a6832ac822748990f597a784f3014a4068b54197fecb1aecb274a"}},{"ruleId":"D18","level":"note","message":{"text":"Thin analysable surface across projects: 1 project(s) carry only a thin slice of real code (e.g. \u0060DevToys.Localization\u0060 with 44 significant line(s)). The mean analysable-surface weight is 98 %, lowering Solution Shape by about 0.17 point(s). Consolidate thin projects or grow them into substantial, well-scoped assemblies."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dd0b92fd965c2065080f16843920964ee00f7696ea03d87cdf61aed4cd2c746a"}},{"ruleId":"D19","level":"note","message":{"text":"The README is a one-line \u0027SDK for building an extension\u0027 with no links to the full DevToys app doc site (https://devtoys.app/doc).: Link this SDK README to the main devtoys.app/doc host so readers can find the complete API reference, examples, and usage."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/dev/DevToys.Api/README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"070661397634f55b772458632a283ef41657e11dc9c6cff19e17f3931bb12bbe"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found at common paths; consider documenting architectural decisions in Docs/ADL/ or similar."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 40k LoC across 9 projects the codebase is large and multi-module, so explicit bounded contexts are needed. Name this codebase\u0027s bounded contexts (\u22652 module groups, e.g. per subsystem) so cross-boundary type coupling can be assessed. Declare them in \u0060.codehealth/config.yaml\u0060 at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it \u2014 e.g. \u0060architecture:\u0060 \u2192 \u0060contexts:\u0060 \u2192 \u0060Billing: [\u0022src/billing\u0022, \u0022Acme.Billing\u0022]\u0060, \u0060Catalog: [\u0022src/catalog\u0022, \u0022Acme.Catalog\u0022]\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"1c7e276c9c682731f01819ed5378b90ca320cde1b583c90920172911598362b3"}},{"ruleId":"D26","level":"note","message":{"text":"Split DevToys.Blazor: A large Blazor project whose namespaces are unrelated Razor components and tools. Suggested: by namespace: DevToys.Blazor.Razor, DevToys.Blazor.Tools"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0b682e36ec4b041a2a73fc4fe55dbaef485ab7d233120d46b7eb6dc0323081df"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7ceee73d76663034321cbd3b5bbecae8b673163039b70b12462f979a331e73d3"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"98bc95744487d6f8bb36ba3a49535c74c1160f6846de0c05d0a36ca260c39063"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8eb50784ab958284eb3ecd7050384e4061ddeae0c76cf46d9d242ff3f816fc6b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"53e31539fd6c680bc524cd465b04ae19c17835bc13d80d46925869e82647ba77"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1bc5fbf1b4ba8fb339a230a0414d3202e917cff32ac780d26e635d43cd241bce"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4e5ba20d569d258ae6d0a24953bb1bf2096a834bc4a53403da7c12ca10d2411b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"3566a4535f32b9e3b41d1001b00837e415a5916934896370e7fc4877cd60d5f5"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"2dafe06ef62877e9f28dcce9e5ac43868f305fc70625d6fed6deeca47e9b1af8"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4df1f7c2949e8fe5de2db225c95da951068db0a034014ea1f8c0de77ac99bbd9"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4c92748b46b2da1c8a4a4921849a1c94eee11983bd424e8af286b0307ffbfafe"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D34","level":"note","message":{"text":"Further orphaned files (smaller): 2 of 167 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 100 lines or more, excluding tests, vendored, generated and example/demo trees, largest first). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 largest first: src/app/dev/DevToys.Api/Tool/GUI/Components/IUIUtf32IconButton.cs, src/app/dev/DevToys.Api/Tool/GUI/Components/IUIUtf32IconDropDownButton.cs. 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"}}],"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-1059","guid":"a2381a08-60f6-9554-a8b8-f3018cfaaca5","name":"Insufficient Technical Documentation","shortDescription":{"text":"Insufficient Technical Documentation"},"helpUri":"https://cwe.mitre.org/data/definitions/1059.html"},{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":10,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}