{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D12","name":"Dependency Hygiene","shortDescription":{"text":"Dependency Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/D12"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D18","name":"Solution Shape","shortDescription":{"text":"Solution Shape"},"helpUri":"https://codehealth.canine.dev/dimensions/D18"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D24","name":"Comment Value","shortDescription":{"text":"Comment Value"},"helpUri":"https://codehealth.canine.dev/dimensions/D24"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D27","name":"Navigability","shortDescription":{"text":"Navigability"},"helpUri":"https://codehealth.canine.dev/dimensions/D27"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"YtdlpExecutableService.GetDownloadProcess (cyclomatic 128): YtdlpExecutableService.GetDownloadProcess has cyclomatic complexity 128 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/YtdlpExecutableService.cs"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"fe46a877ae1eae701830e80a3b43f1201297b26d232d082ab6b988ef2fec2774"}},{"ruleId":"D1","level":"warning","message":{"text":"Media.Media.ctor (cyclomatic 102): Media.Media.ctor has cyclomatic complexity 102 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Media.cs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e478473e7f517edec69d3bbe6aaeda5d2b94b1322031d1a270551b86f5c516f"}},{"ruleId":"D1","level":"warning","message":{"text":"Format.Format.ctor (cyclomatic 53): Format.Format.ctor has cyclomatic complexity 53 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function, or replace a long branch ladder over a single value with a data-driven lookup or dispatch table."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"f247b507b6e9ff6721d8668eac7c9d1de9adea2c59ddee82a1f956b45735403b"}},{"ruleId":"D1","level":"warning","message":{"text":"Format.ToString (cyclomatic 53): Format.ToString has cyclomatic complexity 53 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":224}}}],"partialFingerprints":{"codehealthFindingId/v1":"fe7cbae7826e85c2bb9b171b533d0fbd1cdd3a286b05bd6b5e2f984e81a6e487"}},{"ruleId":"D1","level":"warning","message":{"text":"Download.Process_OutputDataReceived (cyclomatic 30): Download.Process_OutputDataReceived has cyclomatic complexity 30 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Download.cs"},"region":{"startLine":233}}}],"partialFingerprints":{"codehealthFindingId/v1":"df7a0cd65b2d56c3b00b672a9cfd58379658ccc0cc187b494fa085254015e1e8"}},{"ruleId":"D1","level":"warning","message":{"text":"AddDownloadDialogController.DiscoverAsync (cyclomatic 29): AddDownloadDialogController.DiscoverAsync has cyclomatic complexity 29 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/AddDownloadDialogController.cs"},"region":{"startLine":253}}}],"partialFingerprints":{"codehealthFindingId/v1":"b8d7ed32ea33b4180ff75997c7c5c17819886865dfb788b3bccccc4fd4793742"}},{"ruleId":"D1","level":"warning","message":{"text":"AddDownloadDialogController.AddPlaylistDownloadsAsync (cyclomatic 28): AddDownloadDialogController.AddPlaylistDownloadsAsync has cyclomatic complexity 28 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/AddDownloadDialogController.cs"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"1e2b9b3da3ffcded5af197f4c8057d9a09456cbccdf38be84d7ce7738429ab00"}},{"ruleId":"D1","level":"warning","message":{"text":"Download.Process_Exited (cyclomatic 24): Download.Process_Exited has cyclomatic complexity 24 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Download.cs"},"region":{"startLine":141}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa18525831f146ae5246fd636453b8ecf357da6f2932f32ad190f5917433c297"}},{"ruleId":"D1","level":"warning","message":{"text":"DiscoveryService.GetForUrlAsync (cyclomatic 17): DiscoveryService.GetForUrlAsync has cyclomatic complexity 17 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/DiscoveryService.cs"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf123204e73373c7dfa4fd12601517619a678fa40cbd738df392fe9a4289ff03"}},{"ruleId":"D1","level":"warning","message":{"text":"MainWindow.UpdateDownloadsList (cyclomatic 17): MainWindow.UpdateDownloadsList has cyclomatic complexity 17 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.WinUI/Views/MainWindow.xaml.cs"},"region":{"startLine":550}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bf1876470d3299f7c31c19b77362a0d7ef1b0c6de881e54fb20d5c423f5766c"}},{"ruleId":"D1","level":"warning","message":{"text":"AddDownloadDialog.DiscoverUrlButton_OnClicked (cyclomatic 16): AddDownloadDialog.DiscoverUrlButton_OnClicked has cyclomatic complexity 16 (threshold 15). Of this number, 14 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. 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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":329}}}],"partialFingerprints":{"codehealthFindingId/v1":"f2959cb20fabdd29b4118262bf0b8282964d539ae2c27925ae21b399cb4af7d9"}},{"ruleId":"D1","level":"warning","message":{"text":"MainWindowController.CheckForUpdatesAsync (cyclomatic 16): MainWindowController.CheckForUpdatesAsync has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"98bf0ba48765ef0bf3be01141fc8f241c451fa2922e8e4ac390d1714560cd4a2"}},{"ruleId":"D2","level":"warning","message":{"text":"Media.Media.ctor (cognitive 177): Media.Media.ctor has cognitive complexity 177 (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":"Nickvision.Parabolic.Shared/Models/Media.cs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"097a8fc93b3c0d90492e501487638ac503162ce4f86199c716c02d0bb19f5b6f"}},{"ruleId":"D2","level":"warning","message":{"text":"YtdlpExecutableService.GetDownloadProcess (cognitive 132): YtdlpExecutableService.GetDownloadProcess has cognitive complexity 132 (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":"Nickvision.Parabolic.Shared/Services/YtdlpExecutableService.cs"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e2634742f8517c68de3fb71907f3754db8da4260015a05849d5e80ea40e0b0d"}},{"ruleId":"D2","level":"warning","message":{"text":"Download.Process_Exited (cognitive 65): Download.Process_Exited has cognitive complexity 65 (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":"Nickvision.Parabolic.Shared/Models/Download.cs"},"region":{"startLine":141}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac2fd2dff7e3bdeea68ced06680ddd36f9049bd8a57cc617ee5fa60022ceb186"}},{"ruleId":"D2","level":"warning","message":{"text":"AddDownloadDialogController.DiscoverAsync (cognitive 47): AddDownloadDialogController.DiscoverAsync has cognitive complexity 47 (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":"Nickvision.Parabolic.Shared/Controllers/AddDownloadDialogController.cs"},"region":{"startLine":253}}}],"partialFingerprints":{"codehealthFindingId/v1":"706c29c36790fedf110ce725949e57f407f981ada37781ead696abd22397c8b9"}},{"ruleId":"D2","level":"warning","message":{"text":"Download.Process_OutputDataReceived (cognitive 45): Download.Process_OutputDataReceived has cognitive complexity 45 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Download.cs"},"region":{"startLine":233}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f86a26bd42b8c6428a4b9b680139d25cfec1555da4f44bdf5f6225119917279"}},{"ruleId":"D2","level":"warning","message":{"text":"AddDownloadDialogController.AddPlaylistDownloadsAsync (cognitive 44): AddDownloadDialogController.AddPlaylistDownloadsAsync has cognitive complexity 44 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/AddDownloadDialogController.cs"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"f783162678325bda892e3de8d7516491e5b339066b5bc6255de0a5e196849870"}},{"ruleId":"D2","level":"warning","message":{"text":"Format.ToString (cognitive 38): Format.ToString has cognitive complexity 38 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":224}}}],"partialFingerprints":{"codehealthFindingId/v1":"93ff037576a7d79b2afd9eadccd9e19a3fc5819f444f76d303f8864ef9555a60"}},{"ruleId":"D2","level":"warning","message":{"text":"Format.Format.ctor (cognitive 34): Format.Format.ctor has cognitive complexity 34 (threshold 15). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"2319d8b08e428a81133f33e0b6d9f713c4485b03d0e9ea6e14e9d060249e7110"}},{"ruleId":"D2","level":"warning","message":{"text":"KeyringDialog.ShowAsync (cognitive 33): KeyringDialog.ShowAsync has cognitive complexity 33 (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":"Nickvision.Parabolic.WinUI/Views/KeyringDialog.xaml.cs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed7d447a4c00f236959e7955e560d2ceb31ab25a5590e5a27267c43a5289ff90"}},{"ruleId":"D2","level":"warning","message":{"text":"AddDownloadDialog.DiscoverUrlButton_OnClicked (cognitive 31): AddDownloadDialog.DiscoverUrlButton_OnClicked has cognitive complexity 31 (threshold 15). Of this number, 24 points are the body\u0027s own statements and 7 belong to one function literal inside it that branches. 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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":329}}}],"partialFingerprints":{"codehealthFindingId/v1":"d68dbc30e741be26aec69583146ded1dade3b578883889e9d668f476db42a9ff"}},{"ruleId":"D2","level":"warning","message":{"text":"DiscoveryService.ParseBatchFileAsync (cognitive 29): DiscoveryService.ParseBatchFileAsync has cognitive complexity 29 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/DiscoveryService.cs"},"region":{"startLine":117}}}],"partialFingerprints":{"codehealthFindingId/v1":"70b535ce5b065bb8a6b5598c3345925d00cbaa876a142ed6940f657da58cbff5"}},{"ruleId":"D2","level":"warning","message":{"text":"DiscoveryService.GetForUrlAsync (cognitive 24): DiscoveryService.GetForUrlAsync 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":"Nickvision.Parabolic.Shared/Services/DiscoveryService.cs"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0403558ea64e40a06ce4e5593e878c3946e7874e1da721125e651c39ec7537a"}},{"ruleId":"D2","level":"warning","message":{"text":"DiscoveryResult.DiscoveryResult.ctor (cognitive 22): DiscoveryResult.DiscoveryResult.ctor has cognitive complexity 22 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/DiscoveryResult.cs"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7cb1a54e3b2d7bc242fc175a3a2d71e8dfa5b204a9df7df0ed8ca1ccfdef59c"}},{"ruleId":"D2","level":"warning","message":{"text":"MainWindowController.CheckForUpdatesAsync (cognitive 20): MainWindowController.CheckForUpdatesAsync has cognitive complexity 20 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"26ec44523fff0469da98262313200b46d51f3ace50846b538e8261aa1fddd110"}},{"ruleId":"D2","level":"warning","message":{"text":"Format.CompareTo (cognitive 18): Format.CompareTo has cognitive complexity 18 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"53bf76633ab8a9063d1c6462208349f6f94b68bbc849b27e936361e883f212fd"}},{"ruleId":"D2","level":"warning","message":{"text":"AddDownloadDialog.DiscoverMediaAsync (cognitive 16): AddDownloadDialog.DiscoverMediaAsync has cognitive complexity 16 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.WinUI/Views/AddDownloadDialog.xaml.cs"},"region":{"startLine":204}}}],"partialFingerprints":{"codehealthFindingId/v1":"d94b0178e7e9ec8102cb25f3aed52170d06f2a797f10cd1c6a34f7ea43daa93e"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: MainWindow: TooManyMethods \u2014 359 significant lines (blank, comment-only and punctuation-only lines excluded), 38 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":"Nickvision.Parabolic.WinUI/Views/MainWindow.xaml.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"410e6152f65c4f91c76ec254db602b669bbc62fae505cb033315c8a5641acfed"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: AddDownloadDialog: ClassTooLong \u2014 450 significant lines (blank, comment-only and punctuation-only lines excluded), 22 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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"377b7b849896436e073143f27dab1c2169f70ee369ffdd60750c3e74d01228b5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19 lines \u00D7 2): Nickvision.Parabolic.WinUI/Views/SettingsPage.xaml.cs:219-237 | Nickvision.Parabolic.WinUI/Views/SettingsPage.xaml.cs:282-300 \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 \u0060Nickvision.Parabolic.WinUI/Views/SettingsPage.xaml.cs:219\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":"Nickvision.Parabolic.WinUI/Views/SettingsPage.xaml.cs"},"region":{"startLine":219}}}],"partialFingerprints":{"codehealthFindingId/v1":"8736f552fab26ee9c21daed77984e54fb0b475a77afd1b4ae2c5bcb540cf286c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): Nickvision.Parabolic.Shared/Models/Format.cs:273-284 | Nickvision.Parabolic.Shared/Models/Format.cs:306-317 \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 \u0060Nickvision.Parabolic.Shared/Models/Format.cs:273\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":"Nickvision.Parabolic.Shared/Models/Format.cs"},"region":{"startLine":273}}}],"partialFingerprints":{"codehealthFindingId/v1":"d835448c103bf87e9705f90739a5fc75996e94f0d3c52da32d42f691c5d66177"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs:366-376 | Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs:398-408 \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 \u0060Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs:366\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":"Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs"},"region":{"startLine":366}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca19e0ef4ff895f3dee1e2a365e51ad123cc3a21f77038e0ae42c765c79a05c4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): Nickvision.Parabolic.Shared/Services/ConfigurationMigrationService.cs:40-47 | Nickvision.Parabolic.Shared/Services/ConfigurationMigrationService.cs:48-55 \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":"Nickvision.Parabolic.Shared/Services/ConfigurationMigrationService.cs"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"a5617a8028fbb749f7e897d780ab2447ffc388befe9b52da00edbafd999d67ac"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): Nickvision.Parabolic.Shared/Services/DownloadService.cs:297-304 | Nickvision.Parabolic.Shared/Services/DownloadService.cs:307-314 \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 \u0060Nickvision.Parabolic.Shared/Services/DownloadService.cs:297\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":"Nickvision.Parabolic.Shared/Services/DownloadService.cs"},"region":{"startLine":297}}}],"partialFingerprints":{"codehealthFindingId/v1":"b601e34ec8610180b93d288713c51ca3858ace8f0126fbd5d3d19c3290025b5c"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Nickvision.Parabolic.Shared: Nickvision.Parabolic.Shared: abstractness 0.16, instability 0.00, distance 0.84 \u2014 the shape a shared-kernel / building-block library has BY DESIGN \u2014 concrete and widely depended-on is what makes it useful, and this dimension does not penalise it (the distance is reported for completeness, not as a defect). Worth a look only if it has grown past one coherent kernel into an everything-bucket."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"08b7108301b87965b5f706d4767844b9220ec46481d579ebd1a20fe1456dccb6"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: AddDownloadDialog (LCOM4 7): AddDownloadDialog\u0027s methods form 7 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":"Nickvision.Parabolic.WinUI/Views/AddDownloadDialog.xaml.cs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"0dfbd70a85a81736f92bf7b98dad94e8b785c7c8f09f7539c4d7fb5e552e9ee8"}},{"ruleId":"D8","level":"error","message":{"text":"No automated tests: No automated tests \u2014 no test code was found in this repository. Untested code is the largest single risk to changing it safely."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3132564c6310e5d01a231f252a02d5d2f5a542c0a8fa29a14c89693f1e3df871"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test suite could be collected \u2014 nothing here references a test framework (xUnit, NUnit or MSTest), so there were no discoverable tests to count. Tests written as plain executables or shell/PowerShell harnesses are not collectible this way and are not scored here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c9bf64cbb5a4ae13d6bcd01fa3bc8d2879d860c73bc67f176ee3c2cecb8adce3"}},{"ruleId":"D12","level":"warning","message":{"text":"Prerelease dependency: CommunityToolkit.Labs.WinUI.Controls.TokenView: CommunityToolkit.Labs.WinUI.Controls.TokenView resolves to 0.1.251217-build.2433, a prerelease build. Prerelease packages carry no support policy, may change breaking between previews and can be unlisted \u2014 pin a stable release before shipping, or record the reason this preview is required."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d339936993fd35b7a97b954d9323ca35f408f6efc66a5b5bf82629af1d5877f4"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: Nickvision.Parabolic.Shared/Services/YtdlpExecutableService.cs: Nickvision.Parabolic.Shared/Services/YtdlpExecutableService.cs changed 2 times in last 90 days, max complexity 128. 1 of those changes was a fix/bug commit, so the churn is repair rather than feature work. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/YtdlpExecutableService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fb3cd13ea0890711c19b8cba9986211efa9c37d8a3dd9ba78cad44a312554fe"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 31 significant file(s) lose their only recent owner: Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs, Nickvision.Parabolic.WinUI/Views/AddDownloadDialog.xaml.cs, Nickvision.Parabolic.WinUI/Views/MainWindow.xaml.cs, Nickvision.Parabolic.Shared/Controllers/AddDownloadDialogController.cs, Nickvision.Parabolic.WinUI/Views/SettingsPage.xaml.cs, Nickvision.Parabolic.GNOME/Views/MainWindow.cs, Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs, Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs (\u002B23 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"5f2680c8a283d5428f5853c75dfe57970f716c830e811715f91c53a018481538"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":224}}}],"partialFingerprints":{"codehealthFindingId/v1":"48f6e6aff336078ccce9f9888ae2e0928ab7c83b00493b199e5a6b73e0e2f858"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":300}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a4933a528deacb4b6201dbf05b247feccb13d5d23598c4cdabd3c7aaad97432"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":363}}}],"partialFingerprints":{"codehealthFindingId/v1":"96cb7d04a1c88ab524d6a5b6df75fa4d14767e7f0e80405fd962ffd2ed7d8cd7"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":402}}}],"partialFingerprints":{"codehealthFindingId/v1":"ab54d41fe4a32b8ae332b7e3f2522bcbaca5832eeb104feccb616c50a1f90da0"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":520}}}],"partialFingerprints":{"codehealthFindingId/v1":"0d6dc7924efffb28bc70a266eb77fe92bad3fd659b35b39d8152b651332fb7a3"}},{"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":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":572}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd951ae075051f7162f17f5afb0b499d44f96927c1d14ab39c2685880635d288"}},{"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":"Nickvision.Parabolic.GNOME/Views/PreferencesDialog.cs"},"region":{"startLine":284}}}],"partialFingerprints":{"codehealthFindingId/v1":"bcb82c0a4d913a5c92f1e43945753c99d52179feda094ead8711004f9005bd26"}},{"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":"Nickvision.Parabolic.Shared/Models/Download.cs"},"region":{"startLine":219}}}],"partialFingerprints":{"codehealthFindingId/v1":"63c82c1a38a7dbbe9f737664176978c3a1d4a853623a767648d4ec0b63596883"}},{"ruleId":"D19","level":"note","message":{"text":"The legal copyright disclaimer is a CAUTION heading with no actionable links to DMCA policy or reporting channels.: Add a link to the official YouTube DMCA policy and an option to file a complaint directly from the page."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed3ef24e53d56d7f9b69259c8d894986442beb05ae33850eaac640b0b0c8f9e3"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found at common paths; consider documenting architectural decisions in Docs/ADL/ or similar."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D21","level":"note","message":{"text":"The concept of \u0027History Length\u0027 is named inconsistently. The service interface and implementation use the property name \u0027Length\u0027, while the view controller uses \u0027HistoryLength\u0027.: Standardize on \u0027HistoryLength\u0027 or \u0027Length\u0027 across all layers. (symbols: Nickvision.Parabolic.Shared.Services.HistoryService.Length, Nickvision.Parabolic.Shared.Services.IHistoryService.Length, Nickvision.Parabolic.Shared.Controllers.PreferencesViewController.HistoryLength)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"f9b7d60ee13e88e7c6be4a39e90e9bb22d27eff73044c4caa4c4c545bcf91ec3"}},{"ruleId":"D21","level":"note","message":{"text":"The action of clearing/cleaning up data is named inconsistently. \u0027ClearAsync\u0027 is used for recovery, \u0027ClearCompleted\u0027 for downloads, and \u0027Clear...\u0027 for controllers. The verb choice (Clear vs Remove) and scope (Completed vs All) vary.: Standardize on \u0027Clear\u0027 for bulk removal operations, e.g., \u0027ClearCompletedAsync\u0027 or \u0027RemoveCompleted\u0027. (symbols: Nickvision.Parabolic.Shared.Services.RecoveryService.ClearAsync(), Nickvision.Parabolic.Shared.Services.DownloadService.ClearCompleted(), Nickvision.Parabolic.Shared.Controllers.MainWindowController.ClearQueuedDownloads(), Nickvision.Parabolic.Shared.Controllers.MainWindowController.ClearRecoverableDownloadsAsync())"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0ceb001a7141bc2a9fd7c411de942dcbea8950ad8f185ab8a7f1145d1f407ea8"}},{"ruleId":"D21","level":"note","message":{"text":"Inconsistent naming for adding/starting operations. \u0027AddAsync\u0027 is used for History and Recovery, while \u0027Resume\u0027 is used for Download. \u0027Start\u0027 or \u0027Begin\u0027 might be more consistent with \u0027Add\u0027 or \u0027Create\u0027.: Use \u0027Add\u0027 for creating new entities (History, Recovery) and \u0027Start\u0027 or \u0027Resume\u0027 for existing ones (Download). (symbols: Nickvision.Parabolic.Shared.Services.HistoryService.AddAsync(System.Collections.Generic.IReadOnlyList\u003CNickvision.Parabolic.Shared.Models.HistoricDownload\u003E), Nickvision.Parabolic.Shared.Services.HistoryService.AddAsync(Nickvision.Parabolic.Shared.Models.HistoricDownload), Nickvision.Parabolic.Shared.Services.IHistoryService.RemoveAsync(Nickvision.Parabolic.Shared.Models.HistoricDownload), Nickvision.Parabolic.Shared.Services.RecoveryService.AddAsync(System.Collections.Generic.IReadOnlyList\u003CNickvision.Parabolic.Shared.Models.RecoverableDownload\u003E), Nickvision.Parabolic.Shared.Services.RecoveryService.RemoveAsync(int), Nickvision.Parabolic.Shared.Services.DownloadService.AddAsync(Nickvision.Parabolic.Shared.Models.DownloadOptions, bool), Nickvision.Parabolic.Shared.Services.DownloadService.Resume(int), Nickvision.Parabolic.Shared.Services.DownloadService.StopAsync(int), Nickvision.Parabolic.Shared.Services.DownloadService.RetryAsync(int))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"eb02b06d7272697c4216f003a6fcc425322ec8c588935642548fce4e578710d1"}},{"ruleId":"D21","level":"note","message":{"text":"The identifier for a download is named \u0027Id\u0027 in the model and event args, which is consistent, but the parameter name in \u0027DownloadService.RetryAsync\u0027 and \u0027DownloadService.StopAsync\u0027 uses \u0027int\u0027 (likely an ID), while other services use specific types.: Ensure the \u0027int\u0027 parameter in DownloadService methods is explicitly typed or named \u0027downloadId\u0027 for clarity. (symbols: Nickvision.Parabolic.Shared.Models.Download.Id, Nickvision.Parabolic.Shared.Events.DownloadEventArgs.Id)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"adeba0d24ac192f8e292e81d25b97efe3358493c6947100c4afee45c6dd9f400"}},{"ruleId":"D24","level":"note","message":{"text":"redundant comment: \u0022Events\u0022 \u2014 delete - the \u0027Events\u0027 comment restates OnClosed/OnShow and is repeated elsewhere"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.GNOME/Controls/DownloadRow.cs"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"36e1843a12485319f678605578a99c64ed5c09ea25e23f74fa6f52ddd3eae7c6"}},{"ruleId":"D24","level":"note","message":{"text":"redundant comment: \u0022Load\u0022 \u2014 delete - load is self-evident; only keep Load on rows, not everywhere"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.GNOME/Views/AddDownloadDialog.cs"},"region":{"startLine":172}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec416cb8b3e3284966221d16d66cf386c6eb84c42d3f9264eb901eb20e84bb07"}},{"ruleId":"D24","level":"note","message":{"text":"redundant comment: \u0022Config\u0022 \u2014 delete - \u0027AppWindow\u0027 is a concrete type name"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.WinUI/Views/MainWindow.xaml.cs"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"cfdad57dd17942f2c2abbcef8b0c2afcfd15884a8873c126bb94e582637d5136"}},{"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":"88208889bccf007f8f034db67d66b3a3459326a004fdb0be7e7f2daefe0c692d"},"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":"13bac7659b510d10aeba1efd64fffb75c20b61311e88a4aa8f2e0a0d7608d7ad"},"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":"82dee28fa684bb19360ec281414d3d1dfe6ba70aec76d1456517aa92d5b10559"},"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":"d60ee64b531a0ea1cc411154f5c4c4f1cc01ab98ee7cc5a336f0c5cbe39a6b22"},"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":"dc00a3771acb4cffac1cf29b889d64df350372870ea28568bd5010184a8c5b90"},"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":"0906f49159752f30778aaca9758f2b47aeda54ee30fb8e6080d5274cd4072af3"},"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":"e7cc70881a0bc0b1ed0c2ce555e5d0e310efed004feb043a8c8b02703ef09a00"},"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":"40dfbbf77739285274592a804ab81404c6448d13857d62ceb79e412fc0918405"},"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":"df99806b623d3f6e998573413b0c6739f255bf85f529a965b8c91035c8d1b70f"},"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":"fb450dbc1d7a06374b39fcf2e2171af3eb67bf203dd005d9b781e74d232d14a7"},"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":"7a86eca052c21f67df4c846b34fb1fd10289152d091107d466c6c4a3223b46e7"},"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":"b3c50c08ad6f3d428aec340978bebb0fe4aa1b77bfa463dd2912063d919cb437"},"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":"D35","level":"warning","message":{"text":"Change coupling: HistoryService.cs \u2194 RecoveryService.cs: \u0060Nickvision.Parabolic.Shared/Services/HistoryService.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/RecoveryService.cs\u0060 change together 64% of the time (7 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/HistoryService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"af1782bfdc650b2daf7dad0e64a1042583e87ee37fdb85e1bbd33a00cff8069c"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: HistoryService.cs \u2194 IDownloadService.cs: \u0060Nickvision.Parabolic.Shared/Services/HistoryService.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/IDownloadService.cs\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/HistoryService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f69c34b5e6cddb68172ffee8f23735d11ad74c03ad52da4b1f5525909f1a4acb"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: MainWindowController.cs \u2194 PreferencesViewController.cs: \u0060Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Controllers/PreferencesViewController.cs\u0060 change together 57% of the time (13 of the 23 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b96a833d294bded059ac4224c3d63f75d43a67946904afe98bbe2a9580de300f"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DownloadService.cs \u2194 RecoveryService.cs: \u0060Nickvision.Parabolic.Shared/Services/DownloadService.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/RecoveryService.cs\u0060 change together 55% of the time (6 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/DownloadService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"27df0abab56d4dadb73c4d509608712fee189737d1cf89148dc17e3adf80f841"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: MainWindowController.cs \u2194 DiscoveryResult.cs: \u0060Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Models/DiscoveryResult.cs\u0060 change together 55% of the time (6 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"62d262b37bb19013450ea1db55e83a46e8a81f2efb38023200c42edbafd5bfc9"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: MainWindowController.cs \u2194 HistoryService.cs: \u0060Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/HistoryService.cs\u0060 change together 54% of the time (7 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7f217b9c5c9e8c12007d2ad2e1e78c93126a0a752cdca9d0b6f30ef2a330e54c"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DownloadService.cs \u2194 HistoryService.cs: \u0060Nickvision.Parabolic.Shared/Services/DownloadService.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/HistoryService.cs\u0060 change together 54% of the time (7 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/DownloadService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"eac2b2f7298be15eb63f243076dbd605be42aea5ab0c717e1a411ed9f91ed0da"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: MainWindowController.cs \u2194 DownloadService.cs: \u0060Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/DownloadService.cs\u0060 change together 50% of the time (9 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Controllers/MainWindowController.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"35a5a337471b6f2ac9a0ff97ba996de3ef97b894c28dd82394371d40557f76a5"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: IDownloadService.cs \u2194 RecoveryService.cs: \u0060Nickvision.Parabolic.Shared/Services/IDownloadService.cs\u0060 and \u0060Nickvision.Parabolic.Shared/Services/RecoveryService.cs\u0060 change together 50% of the time (5 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Nickvision.Parabolic.Shared/Services/IDownloadService.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1171e5a5007c727ee614c6a89e9bbfa9d83b6a0690c056a0fed03724a02e3060"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-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-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":12,"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."]}]}