{"$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":"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":"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":"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":"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":"D22","name":"Internal API Consistency","shortDescription":{"text":"Internal API Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D22"},{"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":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"id":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"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"]}},{"id":"D43","name":"Malicious Dependencies","shortDescription":{"text":"Malicious Dependencies"},"helpUri":"https://codehealth.canine.dev/dimensions/D43","relationships":[{"target":{"id":"CWE-506","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-506"]}},{"id":"D44","name":"Platform End-of-Life","shortDescription":{"text":"Platform End-of-Life"},"helpUri":"https://codehealth.canine.dev/dimensions/D44"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX3","name":"Project dependency cycles","shortDescription":{"text":"Project dependency cycles"},"helpUri":"https://codehealth.canine.dev/dimensions/AX3"},{"id":"AX4","name":"Dependency direction","shortDescription":{"text":"Dependency direction"},"helpUri":"https://codehealth.canine.dev/dimensions/AX4"},{"id":"AX8","name":"Test isolation","shortDescription":{"text":"Test isolation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX8"},{"id":"AX9","name":"CQS / query purity","shortDescription":{"text":"CQS / query purity"},"helpUri":"https://codehealth.canine.dev/dimensions/AX9"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P10","name":"Library API \u0026 versioning","shortDescription":{"text":"Library API \u0026 versioning"},"helpUri":"https://codehealth.canine.dev/dimensions/P10"},{"id":"P12","name":"CI test-gate honesty","shortDescription":{"text":"CI test-gate honesty"},"helpUri":"https://codehealth.canine.dev/dimensions/P12"},{"id":"P2","name":"Observability","shortDescription":{"text":"Observability"},"helpUri":"https://codehealth.canine.dev/dimensions/P2"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P4","name":"Deployment \u0026 Rollback","shortDescription":{"text":"Deployment \u0026 Rollback"},"helpUri":"https://codehealth.canine.dev/dimensions/P4"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"X10","name":"Duplicated predicate","shortDescription":{"text":"Duplicated predicate"},"helpUri":"https://codehealth.canine.dev/dimensions/X10"},{"id":"X32","name":"Type resolved by simple name across every loaded assembly","shortDescription":{"text":"Type resolved by simple name across every loaded assembly"},"helpUri":"https://codehealth.canine.dev/dimensions/X32"},{"id":"X6","name":"Hand-rolled structured-format parsing","shortDescription":{"text":"Hand-rolled structured-format parsing"},"helpUri":"https://codehealth.canine.dev/dimensions/X6"},{"id":"X7","name":"Silent fallback defaults","shortDescription":{"text":"Silent fallback defaults"},"helpUri":"https://codehealth.canine.dev/dimensions/X7"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"SudoOptions::try_parse_from (cyclomatic 33): SudoOptions::try_parse_from has cyclomatic complexity 33 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":640}}}],"partialFingerprints":{"codehealthFindingId/v1":"57206481a706241b148f7311ad597196267fa625ac269448e1f1f82e42f4ac1c"}},{"ruleId":"D1","level":"warning","message":{"text":"sudo_rs::exec::use_pty::parent::exec_pty (cyclomatic 31): sudo_rs::exec::use_pty::parent::exec_pty has cyclomatic complexity 31 (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":"src/exec/use_pty/parent.rs"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ab0e26b25124ed913b168a4e3bf07f03c4970b3f5775ec95dc5ecdf9d8ec497"}},{"ruleId":"D1","level":"warning","message":{"text":"SuContext::from_env (cyclomatic 27): SuContext::from_env has cyclomatic complexity 27 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/context.rs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4cbb851dbc57696dfffb95ec166b2c3a63ba9313a023b41bf658f6508e2e60b"}},{"ruleId":"D1","level":"warning","message":{"text":"Error::fmt (cyclomatic 23): Error::fmt has cyclomatic complexity 23 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/common/error.rs"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"95bf1ae66312205c34ec3339a8753486504f41ee4c593c60bf9078ad37df976b"}},{"ruleId":"D1","level":"warning","message":{"text":"sudo_rs::sudoers::analyze::process (cyclomatic 20): sudo_rs::sudoers::analyze::process has cyclomatic complexity 20 (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":"src/sudoers/mod.rs"},"region":{"startLine":860}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d8d9f32d7c2f96863032c337aeb179beb77d12ba528fb63030634dcfa64e18d"}},{"ruleId":"D1","level":"warning","message":{"text":"sudo_rs::pam::rpassword::read_unbuffered (cyclomatic 19): sudo_rs::pam::rpassword::read_unbuffered has cyclomatic complexity 19 (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":"src/pam/rpassword.rs"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"da46b09aed8e975bc1e4fd7628edac02f084a2511bb77ed9bbcb3c9f2881897f"}},{"ruleId":"D1","level":"warning","message":{"text":"sudo_rs::exec::use_pty::monitor::exec_monitor (cyclomatic 19): sudo_rs::exec::use_pty::monitor::exec_monitor has cyclomatic complexity 19 (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":"src/exec/use_pty/monitor.rs"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"e4e90e2fbf89c6eaca221c695c62003bf1d945f6d106cb3d7fd80e80f981da10"}},{"ruleId":"D1","level":"warning","message":{"text":"ParentClosure::suspend_pty (cyclomatic 17): ParentClosure::suspend_pty 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":"src/exec/use_pty/parent.rs"},"region":{"startLine":562}}}],"partialFingerprints":{"codehealthFindingId/v1":"54f07c86e8b0532b0cdfb7064a87df48511c75e9d7dc4fd8d25dbd74b456f794"}},{"ruleId":"D1","level":"warning","message":{"text":"SudoArg::normalize_arguments (cyclomatic 17): SudoArg::normalize_arguments 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":"src/sudo/cli/mod.rs"},"region":{"startLine":524}}}],"partialFingerprints":{"codehealthFindingId/v1":"134fa29da773798bac95d41af4957f52a31427c13c2591bbd4ee630ff16de777"}},{"ruleId":"D1","level":"warning","message":{"text":"SuOptions::parse_arguments (cyclomatic 16): SuOptions::parse_arguments 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":"src/su/cli.rs"},"region":{"startLine":368}}}],"partialFingerprints":{"codehealthFindingId/v1":"0d29dd14a3036b4223c76f3ad7a331908a0863cd7b55dae384e42e89cdf7091f"}},{"ruleId":"D2","level":"warning","message":{"text":"SuOptions::parse_arguments (cognitive 53): SuOptions::parse_arguments has cognitive complexity 53 (threshold 15). Drivers by points: if/else 22 (49 pts), loops 2 (4 pts) (nesting depth added 29). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/cli.rs"},"region":{"startLine":368}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0aa4b355240692354cf402e4f5a749ad3b6ccb1624822601ec46b04c5c303d4"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::exec::use_pty::parent::exec_pty (cognitive 53): sudo_rs::exec::use_pty::parent::exec_pty has cognitive complexity 53 (threshold 15). Drivers by points: if/else 22 (35 pts), match/switch 7 (15 pts), boolean chains 3 (nesting depth added 21). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/use_pty/parent.rs"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b3053deb3ce7bd4070c8aa06876ebb0a62b88da21cab8a3b73cb00f80712edf"}},{"ruleId":"D2","level":"warning","message":{"text":"VisudoOptions::parse_arguments (cognitive 51): VisudoOptions::parse_arguments has cognitive complexity 51 (threshold 15). Drivers by points: if/else 19 (46 pts), loops 2 (4 pts), boolean chains 1 (nesting depth added 29). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/visudo/cli.rs"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"99d6235621b00133af6098ef60fabf25156c2fd78b5045d2c7db63c6e5034670"}},{"ruleId":"D2","level":"warning","message":{"text":"SudoArg::normalize_arguments (cognitive 43): SudoArg::normalize_arguments has cognitive complexity 43 (threshold 15). Drivers by points: if/else 18 (38 pts), loops 2 (4 pts), boolean chains 1 (nesting depth added 22). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":524}}}],"partialFingerprints":{"codehealthFindingId/v1":"95f058cee8b890f7e7dd6756f38aaf69f79f239adbb4e572475c349915134d35"}},{"ruleId":"D2","level":"warning","message":{"text":"SuContext::from_env (cognitive 37): SuContext::from_env has cognitive complexity 37 (threshold 15). Drivers by points: if/else 20 (26 pts), boolean chains 7, loops 3 (4 pts) (nesting depth added 7). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/context.rs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"20959505d6d7c4db6c856bcffadbc26fb86087b1e22bbb0c6d388d059faec178"}},{"ruleId":"D2","level":"warning","message":{"text":"ExecClosure::suspend_parent (cognitive 35): ExecClosure::suspend_parent has cognitive complexity 35 (threshold 15). Drivers by points: if/else 15 (34 pts), boolean chains 1 (nesting depth added 19). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/no_pty.rs"},"region":{"startLine":160}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d826da96d9150bced0758581a87451fb4d63f5bbbd35e4c3a8608ddb9c403e1"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::sudoers::analyze::process (cognitive 34): sudo_rs::sudoers::analyze::process has cognitive complexity 34 (threshold 15). Drivers by points: if/else 8 (23 pts), loops 2 (5 pts), match/switch 2 (5 pts), boolean chains 1 (nesting depth added 21). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/mod.rs"},"region":{"startLine":860}}}],"partialFingerprints":{"codehealthFindingId/v1":"781f9c40068cd464227d5492d367f961e6c48a6d2a326fc2c3d648715e3e2337"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::pam::rpassword::read_unbuffered (cognitive 32): sudo_rs::pam::rpassword::read_unbuffered has cognitive complexity 32 (threshold 15). Drivers by points: if/else 13 (26 pts), match/switch 2 (3 pts), boolean chains 2, loops 1 (nesting depth added 14). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/pam/rpassword.rs"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"d45ecc54a8619f45df2d8cf2a8e0003bba7b5c6af8a59fd9997975339f826afa"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::exec::use_pty::monitor::exec_monitor (cognitive 30): sudo_rs::exec::use_pty::monitor::exec_monitor has cognitive complexity 30 (threshold 15). Drivers by points: if/else 11 (17 pts), match/switch 5 (8 pts), loops 2 (5 pts) (nesting depth added 12). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/use_pty/monitor.rs"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"1502554e72e75f1d873bb78c1b7e88fe42f29e7f0d0f304576a828afa74b3f54"}},{"ruleId":"D2","level":"warning","message":{"text":"ParentClosure::suspend_pty (cognitive 27): ParentClosure::suspend_pty has cognitive complexity 27 (threshold 15). Drivers by points: if/else 13 (21 pts), boolean chains 4, match/switch 1 (2 pts) (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/use_pty/parent.rs"},"region":{"startLine":562}}}],"partialFingerprints":{"codehealthFindingId/v1":"354f0a089609fff6cbcc7d0ce1eaa278c0968f59a3dd917b4e06e53cad6b3f60"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::sudoers::entry::write_spec (cognitive 27): sudo_rs::sudoers::entry::write_spec has cognitive complexity 27 (threshold 15). Drivers by points: if/else 5 (13 pts), loops 3 (9 pts), match/switch 4 (5 pts) (nesting depth added 15). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/entry.rs"},"region":{"startLine":232}}}],"partialFingerprints":{"codehealthFindingId/v1":"8945a0c631583924590ec4890ac4b5c11471bd678039f9745d35e8be20595559"}},{"ruleId":"D2","level":"warning","message":{"text":"SudoOptions::try_parse_from (cognitive 22): SudoOptions::try_parse_from has cognitive complexity 22 (threshold 15). Drivers by points: match/switch 4 (12 pts), if/else 2 (7 pts), boolean chains 2, loops 1 (nesting depth added 13). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":640}}}],"partialFingerprints":{"codehealthFindingId/v1":"30c4f6a4d3cd37ff29756237828294970a0cd3432883cfc51c27af1149c998c3"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::visudo::edit_sudoers_file (cognitive 22): sudo_rs::visudo::edit_sudoers_file has cognitive complexity 22 (threshold 15). Drivers by points: if/else 6 (9 pts), match/switch 3 (8 pts), loops 2 (4 pts), boolean chains 1 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/visudo/mod.rs"},"region":{"startLine":241}}}],"partialFingerprints":{"codehealthFindingId/v1":"1aaab7f7a4b8cba9a96602c0d706f761bc4aebcb2d1c3a75b0409203eb0d7b6f"}},{"ruleId":"D2","level":"warning","message":{"text":"Sudo::parse (cognitive 19): Sudo::parse has cognitive complexity 19 (threshold 15). Drivers by points: if/else 12 (18 pts), boolean chains 1 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/ast.rs"},"region":{"startLine":518}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad1310606f51c180c35038513d97258f51891d2f525d27d2f5728e3dd4973b2a"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::exec::run_command (cognitive 18): sudo_rs::exec::run_command has cognitive complexity 18 (threshold 15). Drivers by points: if/else 10 (13 pts), match/switch 3 (5 pts) (nesting depth added 5). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/mod.rs"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"c0721db1a123acdc652bf576d56944504380657e4b19cc295f7ae3a8863a2713"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::sudoers::select_editor (cognitive 18): sudo_rs::sudoers::select_editor has cognitive complexity 18 (threshold 15). Drivers by points: if/else 7 (15 pts), loops 2, boolean chains 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/mod.rs"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"8be69ffb962329fb914d2bf157fffb84d897662c7170913ea2346037da7db273"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::sudoers::entry::write_users (cognitive 16): sudo_rs::sudoers::entry::write_users has cognitive complexity 16 (threshold 15). Drivers by points: match/switch 5 (12 pts), if/else 2 (3 pts), loops 1 (nesting depth added 8). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/entry.rs"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"9099f14e827433ad38db309f2af798b51ff34dcb5a228495d0ca92366ea5ce3b"}},{"ruleId":"D2","level":"warning","message":{"text":"sudo_rs::sudo::edit::edit_files (cognitive 16): sudo_rs::sudo::edit::edit_files has cognitive complexity 16 (threshold 15). Drivers by points: if/else 7 (10 pts), loops 4, match/switch 1 (2 pts) (nesting depth added 4). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/edit.rs"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"0965fa49b15a9f2dc5f2fef91fe5345df2be3eae3ecb4fdda60340290eb1872b"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: sudo_rs::sudoers::analyze: FunctionTooLong \u2014 sudo_rs::sudoers::analyze runs 214 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 114 over it, 2.14\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/mod.rs"},"region":{"startLine":723}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7acfdeaa7d3c2d47ac0efb55ec532a9d72ee221890b1777d35af94da8f4adf9"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: sudo_rs::exec::use_pty::parent::exec_pty: FunctionTooLong \u2014 sudo_rs::exec::use_pty::parent::exec_pty runs 157 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 57 over it, 1.57\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/use_pty/parent.rs"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"f89d044eb2411a1922755dcff7a878943e421915279c26bdc4bfa99dada4c480"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: sudoers/mod.rs: FileTooLong \u2014 692 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 192 over it, 1.38\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/mod.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"319d26bc91edaf291bee331a1ef97f5b7aac4e29b46a92ef011e47e734f1ac9d"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: cli/mod.rs: FileTooLong \u2014 549 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 49 over it, 1.10\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0812e4834ace149d6c34f2ad36cb41c26f405ae624dfbc792e5a5445ccac7b78"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: sudoers/ast.rs: FileTooLong \u2014 508 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 8 over it, 1.02\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/ast.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"172979ae7cb5b734a444daa8140c396dd818150d6c8e27834cf88b0dbf1ead88"}},{"ruleId":"D4","level":"warning","message":{"text":"Near-duplicate member family (4 members, 14 shared lines): src/system/mod.rs:419-456 | src/system/mod.rs:488-522 | src/system/mod.rs:573-613 | src/system/mod.rs:625-662 \u2014 These 4 members are variants of one another: a block of 14 lines reported below appears in every one of them, and the pairwise near-duplicate rows they would otherwise produce are collapsed into this row. Read them as one construct written 4 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 4 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/mod.rs"},"region":{"startLine":419}}}],"partialFingerprints":{"codehealthFindingId/v1":"64580a4487066614c1030b7844012b826f295023bb23b0d93c1d0a2119c9fa70"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (4 members, 50\u002B identical tokens): src/sudo/cli/mod.rs:137-171 | src/sudo/cli/mod.rs:200-242 | src/sudo/cli/mod.rs:275-322 | src/sudo/cli/mod.rs:361-428 \u2014 These 4 members share a duplicated core: a run of at least 50 identical tokens appears in every one of them. That run is NOT broken out as duplicated-block rows below \u2014 it is what admitted this row, and the blocks below cover only the part of it that clears the block floor, so they understate the correspondence. Read the members as one construct written 4 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 4 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":137}}}],"partialFingerprints":{"codehealthFindingId/v1":"b311fcc76eeb12c6deecc34bd3327b3a574f7f9595a5da2514d754eab4d89af9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): src/system/mod.rs:436-456 | src/system/mod.rs:502-522 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/mod.rs"},"region":{"startLine":436}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e3416193cfc61b7b4be570201219d878e4de825d78dfed438ff7e13ad452e1c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): src/exec/noexec.rs:198-214 | src/exec/noexec.rs:241-257 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/noexec.rs"},"region":{"startLine":198}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f1ebeb5c590b78382c45713ff55df1d7cd4d208ed21eba5bfa920e7dec3b411"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): src/exec/use_pty/monitor.rs:47-62 | src/exec/use_pty/parent.rs:52-67 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/use_pty/monitor.rs"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2c42b2ab3d528bd91ba8f4a66ef1ce41ce36a8ae0f08e276e5742f92c7c79e9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201316 lines \u00D7 2): src/system/mod.rs:589-604 | src/system/mod.rs:640-653 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/mod.rs"},"region":{"startLine":589}}}],"partialFingerprints":{"codehealthFindingId/v1":"8288342a744c3e0e75451fdaf7f39d100929780f121a14ef97c993a2df47bfd7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/sudoers/entry.rs:112-126 | src/sudoers/entry.rs:155-169 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/entry.rs"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"b081e029858a9dc563427f66a3fbab8799356704973549debfa5c3ca25beb630"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11\u201312 lines \u00D7 2): src/su/mod.rs:53-64 | src/sudo/pam.rs:108-118 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/mod.rs"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0fa99dbf3f6b1709bd3616cc3c1719b3bd5ae83d475ca9be6f300d01bcb055b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 3): src/sudo/cli/mod.rs:147-157 | src/sudo/cli/mod.rs:212-222 | src/sudo/cli/mod.rs:284-294 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb3524a4b525528b33d567a24fc796ff42e76fab2a8494867b460d7ef5f54718"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/common/context.rs:192-202 | src/common/context.rs:239-249 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/common/context.rs"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"04f91b1b941fb9458fbc13b64238c24613d55eee68c598709f8a53eed8b0be9e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/system/timestamp.rs:353-363 | src/system/timestamp.rs:365-375 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/timestamp.rs"},"region":{"startLine":353}}}],"partialFingerprints":{"codehealthFindingId/v1":"d38eceaad644858f5887caf47f2c918f3ca694eac104bef177aa70b7cb0d34a0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8\u201310 lines \u00D7 2): src/su/cli.rs:395-402 | src/visudo/cli.rs:136-145 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/cli.rs"},"region":{"startLine":395}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a8aca7eadebbd852ce89baf2593f90bdbbfdd8fdfb827139118da054d192026"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 4): src/system/mod.rs:435-443 | src/system/mod.rs:501-509 | src/system/mod.rs:588-599 | src/system/mod.rs:639-648 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/mod.rs"},"region":{"startLine":435}}}],"partialFingerprints":{"codehealthFindingId/v1":"872c8a6c3e03c7e2584260079c2fad4fbd8c7a5faf73355f437c9ebfffa33cae"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): src/exec/no_pty.rs:240-251 | src/exec/use_pty/monitor.rs:366-374 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/no_pty.rs"},"region":{"startLine":240}}}],"partialFingerprints":{"codehealthFindingId/v1":"af98f621c11dda218ae4928917b21d79f053a7493381b0269a587748b409e96e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): src/su/cli.rs:407-415 | src/visudo/cli.rs:149-157 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/su/cli.rs"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e519f6ace3bf302790bd0d4980011e802268293cf2e51537d3eecdcd99526ef"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 4): src/sudo/cli/mod.rs:147-153 | src/sudo/cli/mod.rs:212-218 | src/sudo/cli/mod.rs:284-290 | src/sudo/cli/mod.rs:373-379 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbddac44eaf60764e1108430973f54de873d42d0984b37d9591f8c426c3085c4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): src/sudo/cli/mod.rs:139-144 | src/sudo/cli/mod.rs:202-207 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudo/cli/mod.rs"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed04372ecaf21bae005b3c95b60aa4c3ccd1788304768727fced09017805deea"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 4): src/system/mod.rs:422-429 | src/system/mod.rs:491-495 | src/system/mod.rs:578-582 | src/system/mod.rs:629-633 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/mod.rs"},"region":{"startLine":422}}}],"partialFingerprints":{"codehealthFindingId/v1":"962bae58fe6001a54d5af57d27b90186634523f64abc67afcbe0a3579d9f44a3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): src/system/timestamp.rs:100-104 | src/system/timestamp.rs:110-114 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/system/timestamp.rs"},"region":{"startLine":100}}}],"partialFingerprints":{"codehealthFindingId/v1":"1617251196b49d42d55b541415298ac0dbda658ef06ac011e6bfe1a6aa87406c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): src/exec/no_pty.rs:143-157 | src/exec/use_pty/parent.rs:504-518 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/no_pty.rs"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"a3a091ec286c109c6593471d5ee19cd0e1380fcb26d595046d3413a2e097d86f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201316 lines \u00D7 3): test-framework/e2e-tests/src/pty.rs:51-64 | test-framework/e2e-tests/src/su/flag_pty.rs:20-35 | test-framework/sudo-compliance-tests/src/sudo/use_pty.rs:21-36 \u2014 before extracting anything, compare \u0060test-framework/e2e-tests/src/su/flag_pty.rs\u0060 and \u0060test-framework/sudo-compliance-tests/src/sudo/use_pty.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/e2e-tests/src/pty.rs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"33d4932a34cf7473915a1f1798c10371f64ff6bcad0c133bdf3586d40cd0b0d6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:615-629 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:645-659 \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":"test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs"},"region":{"startLine":615}}}],"partialFingerprints":{"codehealthFindingId/v1":"360f59f956acb43b7feaa9202fb2938d4be8da9aa9b068c69b94d8057baa4f5f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): test-framework/e2e-tests/src/su/flag_pty.rs:47-60 | test-framework/sudo-compliance-tests/src/sudo/use_pty.rs:48-61 \u2014 before extracting anything, compare \u0060test-framework/e2e-tests/src/su/flag_pty.rs\u0060 and \u0060test-framework/sudo-compliance-tests/src/sudo/use_pty.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/e2e-tests/src/su/flag_pty.rs"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"a6c1ca0007c8a2784700c31337bcf3dc4c95a6c65b6d9e018a31342b865fefd6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 5): test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:137-148 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:179-190 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:266-277 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:399-410 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:467-478 \u2014 all 5 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs"},"region":{"startLine":137}}}],"partialFingerprints":{"codehealthFindingId/v1":"227ac50392463a244172a80000603331aa1b6918ecdeae0793dbc338e45c3fa7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:134-145 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:377-388 \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":"test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"08942d278c3885df91d85e3a06c84622c5810b1a8578fa52c071dc72f5df7f4f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201311 lines \u00D7 7): test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:158-167 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:201-210 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:244-253 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:489-498 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:536-546 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:562-572 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:588-598 \u2014 all 7 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs"},"region":{"startLine":158}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c8deeef671b673d6747e618408864038c4ade24db7fc78b74949f53ec3570c7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 3): test-framework/sudo-compliance-tests/src/sudo/child_process/signal_handling/mod.rs:65-74 | test-framework/sudo-compliance-tests/src/sudo/child_process/signal_handling/mod.rs:189-198 | test-framework/sudo-compliance-tests/src/sudo/child_process/signal_handling/mod.rs:218-227 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/child_process/signal_handling/mod.rs"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"2c1c57f8eb857f8deaa34dcf316f21d6afcfa522418c25d5ee7cc7db822cd03b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7\u20138 lines \u00D7 6): test-framework/sudo-compliance-tests/src/sudo/pam/env.rs:190-196 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:164-170 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:250-256 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:622-629 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:652-659 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:707-714 \u2014 there are 6 copies across 2 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 6 sites; resolving a subset leaves the remainder to drift apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/pam/env.rs"},"region":{"startLine":190}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddde299327aaa878c4f7f4406a89ffd2135e77f5ccca02d1847346a278354df7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:346-353 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:365-372 \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":"test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs"},"region":{"startLine":346}}}],"partialFingerprints":{"codehealthFindingId/v1":"ebb48a92480a268c89f0f55274082bc28e08a0d25a8558f3f28455d003deacad"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): test-framework/sudo-test/src/lib.rs:853-857 | test-framework/sudo-test/src/lib.rs:876-880 \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":"test-framework/sudo-test/src/lib.rs"},"region":{"startLine":853}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff130290d8386fcb4eac19e1d5c3a4fa11f13558e814aa3755f24d8d2cc0d1dc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): test-framework/sudo-test/src/lib.rs:859-863 | test-framework/sudo-test/src/lib.rs:882-886 \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":"test-framework/sudo-test/src/lib.rs"},"region":{"startLine":859}}}],"partialFingerprints":{"codehealthFindingId/v1":"4fb4442fe4a023973f3535323fd3b7734c142ca05449545805f48a36caf1a4fc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 10): test-framework/sudo-compliance-tests/src/sudo/sudo_ps1.rs:55-63 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:137-145 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:179-187 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:222-230 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:266-274 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:341-349 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:360-368 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:399-407 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:418-426 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:467-475 \u2014 there are 10 copies across 2 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 10 sites; resolving a subset leaves the remainder to drift apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/sudo_ps1.rs"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd756e4feda40e0f3e6a2b798272867aa23965852db9a1c8fd5b24df789037e7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 14): test-framework/sudo-compliance-tests/src/sudo/pam/env.rs:189-195 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:141-147 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:163-169 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:183-189 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:226-232 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:249-255 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:270-276 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:403-409 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:471-477 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:494-500 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:621-628 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:651-658 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:706-713 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:735-742 \u2014 there are 14 copies across 2 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 14 sites; resolving a subset leaves the remainder to drift apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/pam/env.rs"},"region":{"startLine":189}}}],"partialFingerprints":{"codehealthFindingId/v1":"17a55e5e09401fcc055cefbbcd59a9a898eadcddd5e1024dc61d47d6d3d7f70b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): test-framework/e2e-tests/src/su/flag_pty.rs:36-42 | test-framework/sudo-compliance-tests/src/sudo/use_pty.rs:37-43 \u2014 before extracting anything, compare \u0060test-framework/e2e-tests/src/su/flag_pty.rs\u0060 and \u0060test-framework/sudo-compliance-tests/src/sudo/use_pty.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/e2e-tests/src/su/flag_pty.rs"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"072807906c9e7b151afce7d7733fc5c4416f3fc9104216b9d42a60b4e7791f03"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 4): test-framework/sudo-compliance-tests/src/sudo/sudo_ps1.rs:60-65 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:297-302 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:346-351 | test-framework/sudo-compliance-tests/src/sudo/sudoers/env.rs:365-370 \u2014 there are 4 copies across 2 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 4 sites; resolving a subset leaves the remainder to drift apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-compliance-tests/src/sudo/sudo_ps1.rs"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f43f47d45ca34637817877dea1f39060abbb8611f98ee9881c1c88e3beb4fd0"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SudoString (LCOM4 4): SudoString\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/common/string.rs"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"dbb163b05499a851dbbfb6031fd53b7bfebb6c300b230ebddb716b2f318343d8"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: Command (LCOM4 4): Command\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-framework/sudo-test/src/docker/command.rs"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb31152757d33913915d4ff74d5bcfad1b9d35673f48d18e2d7019c1db3b53ce"}},{"ruleId":"D13","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":"813768978dce6ee0ee5b5ca785936c6f29191416f6d5c83b265fb52850ea4dee"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: src/sudoers/mod.rs: src/sudoers/mod.rs changed 6 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 30 in sudo_rs::sudoers::analyze at line 723. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-07-01..2026-09-29, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-01 15:27:15 \u002B02:00\u0027 --until=\u00272026-09-29 15:27:15 \u002B02:00\u0027 --full-history --no-merges -- src/sudoers/mod.rs\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/sudoers/mod.rs"},"region":{"startLine":723}}}],"partialFingerprints":{"codehealthFindingId/v1":"ccf82799e7c8a00ee397a64ee825e8d52e4a110722ea838bd5e9fb3421e9a8d3"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 7 significant file(s) lose their only recent owner: src/sudoers/entry.rs, src/exec/use_pty/pipe/mod.rs, test-framework/sudo-compliance-tests/src/sudo/sudoers/timestamp_type.rs, src/sudoers/ast_names.rs, test-framework/sudo-compliance-tests/src/visudo/sudoers/editor.rs, src/common/string.rs, test-framework/sudo-compliance-tests/src/sudo/flag_preserve_environment.rs. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"adbcf7517ec09a02d78b2081ef5772e163df9ea2923ab2dd8e50b3d2239c453d"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #2: If anonymized user #2 becomes unavailable, 4 significant file(s) lose their only recent owner: test-framework/sudo-compliance-tests/src/sudo/timestamp/reset.rs, test-framework/sudo-compliance-tests/src/sudo/pass_auth/stdin.rs, test-framework/sudo-compliance-tests/src/sudo/pass_auth/tty.rs, test-framework/sudo-compliance-tests/src/sudo/timestamp/remove.rs. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ecadb09ce2d58c2da6cbb7dd471b2f9186091d08913df1ce3bb07bb462edcb18"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME replace with OsStr::display() once our MSRV is 1.87 \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/common/mod.rs"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"b26ce4f1a6e8e5c7f805c5cab81990ff9011ad98e1ca9fa63615000836157d8d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: the more Rust way of doing things would be to create an alternative for sudoedit instead; \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/common/context.rs"},"region":{"startLine":151}}}],"partialFingerprints":{"codehealthFindingId/v1":"89d13efea37a5be0b7d51ea56e09339a54e9429b09af25929218c97f80d7a0fa"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Initialize the policy plugin\u0027s session here. \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/exec/no_pty.rs"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"33a0ac02df6bc1c32dbb62fe0f80be143fbad5808e27f7df147fe76830614b4c"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Some extra config happens here if selinux is available. \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/exec/no_pty.rs"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb0385bc42c9bf91736c5008a8300278c5ce60e70ed77b757a0c765851b9b6f1"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Some extra config happens here if selinux is available. \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/exec/use_pty/monitor.rs"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"4353ebc6a039f24587d3959576510ca7fc94f2c8c4e59f134000909d13cdaf1d"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: we should handle SIGWINCH here if we want to support I/O plugins that \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/exec/no_pty.rs"},"region":{"startLine":248}}}],"partialFingerprints":{"codehealthFindingId/v1":"98c5a6e0de85fcb2e2378b586c8734dae644cec5d422af5e15d45197c40dab95"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: should we pipe the stdio streams? \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/exec/mod.rs"},"region":{"startLine":99}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5b871d7d92620da544f0dac3836d19a8b469fa16be5e114479b2aeb84659bb1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: this mechanism could perhaps also be used to set the arg0 for login shells, as below \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/exec/mod.rs"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"61c27f8fabce5c97b89be0d8c5284a680f64b03f23a8927e792e7778443aa84a"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME ensure that the fd is not closed while the event registry is still open \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/exec/event.rs"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2fcbc40c3d473aaf02cd222772235b47885dd4f94cc8d2bd6807c691cd52291"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: we should set either a timeout or use ppoll when available. \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/exec/event.rs"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"9d4ea1fa216603158ca92cabdd70fe185dd6b33ecf79830647ffb5bdc3088a89"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: maybe we should return the IO error instead. \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/exec/event.rs"},"region":{"startLine":232}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc59b06d1db9167302882e2419dd9ad0b702fd4e02946ab4d273ac5f40e96432"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Initialize the policy plugin\u0027s session here by calling \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/exec/use_pty/parent.rs"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"0aa21786f549558a69381d3a24a001cebbf5e86e254513386cd4dccd7eb43c4f"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): initializes ttyblock sigset here by calling \u0060init_ttyblock\u0060 \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/exec/use_pty/parent.rs"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e74457f3780e5a7ba55debd41d543c2ac04a150661d09eba0b7ac2fdd262086"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: maybe all these boolean flags should be on a dedicated type. \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/exec/use_pty/parent.rs"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"be66b429e4507a37eb3c2c9040ecc9bd59ed6a49a7f8adbc929bd9294f6b1f5c"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: Here\u0027s where we should intercept the IO streams if we want to implement IO logging. \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/exec/use_pty/parent.rs"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"2077622d6650fb93487a8b6c0b6d2ce2d987f2adc9afca983e3290243c078805"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: ogsudo creates pipes for the IO streams and uses events to read from the strams to \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/exec/use_pty/parent.rs"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"2485c1407ecf0e52e775d0937e06afa8937f7bc34b87249b9af69d22772411bd"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Retry if \u0060/dev/tty\u0060 is revoked. \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/exec/use_pty/parent.rs"},"region":{"startLine":293}}}],"partialFingerprints":{"codehealthFindingId/v1":"edd2c83a81aee5cbf7be15ec7f934bf56a821f9948f92cbf211d7b07808ad7a7"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: sync the terminal size here. \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/exec/use_pty/parent.rs"},"region":{"startLine":675}}}],"partialFingerprints":{"codehealthFindingId/v1":"957b3072401c0a84d9a16326b6dc0137eb0296607783afed0d207d649413c123"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: check \u0060send_command_status\u0060 \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/exec/use_pty/parent.rs"},"region":{"startLine":735}}}],"partialFingerprints":{"codehealthFindingId/v1":"61e7237e9dc918a6eec4d1c8a611255d3df4456746bd06fb03c187aa7f102094"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Do any additional configuration that needs to be run after \u0060fork\u0060 but before \u0060exec\u0060 \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/exec/use_pty/monitor.rs"},"region":{"startLine":99}}}],"partialFingerprints":{"codehealthFindingId/v1":"49c2a982e68017d5a24b6d2dc392e06d1e814bd7203779aee0f462de8d80cab5"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): Here\u0027s where the signal mask is removed because the handlers for the signals \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/exec/use_pty/monitor.rs"},"region":{"startLine":151}}}],"partialFingerprints":{"codehealthFindingId/v1":"520bb17fbde963284668433553ba83cf0813e440ce98624e752ed8999176735f"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME (ogsudo): The tty is restored here if selinux is available. \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/exec/use_pty/monitor.rs"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"e386c46e44ffed25d35b36a3ee57d7905d08247694587cc45c1f6af721bfa1f3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: add missing doc-comments \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/pam/error.rs"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"36d630b625335dc87f31da1b5aacbf710e6670cf45f5d677d52a64d082e01ed2"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME make it configurable by the packager \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/su/mod.rs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"e1d8bb10c5a8325377b22f03e1d00004608dc6bd66f92ffb2294828ff53ceefe"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: use _PATH_STDPATH and _PATH_DEFPATH_ROOT from paths.h \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/su/context.rs"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"bdcbdde66eeffb64d0e120c7b6eaf45136b5c1be040a5f6415ed6aa8a28d54cc"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME check if modified since reading and if so ask user what to do \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/sudo/edit.rs"},"region":{"startLine":149}}}],"partialFingerprints":{"codehealthFindingId/v1":"236270b04bfc238ecc09a11c2e6aea1a37752967c103270750098b4a6f2aa3e6"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME maybe use pipes once std::io::pipe has been stabilized long enough. \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/sudo/edit.rs"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"6257e7d02463176c875e8ecae1d54274df61c334e1f715868339be541e2bcde2"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME preserve temporary file if the original couldn\u0027t be written to \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/sudo/edit.rs"},"region":{"startLine":293}}}],"partialFingerprints":{"codehealthFindingId/v1":"7f5477028fe35047a64742360358c46473ad2d1f625678b9814dfb1c7df79923"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: use _PATH_STDPATH from paths.h \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/sudo/env/environment.rs"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"270add1ea0ea428df7f435f01e68246d0725b4193fbf3179c8f3e8714a92fefd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO Netgroups \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/sudoers/ast.rs"},"region":{"startLine":276}}}],"partialFingerprints":{"codehealthFindingId/v1":"4af950d4c640c3acc5194810bc22393b9bb6e50ba3bb6d8c9c9abc3462d5ed28"}},{"ruleId":"D17","level":"warning","message":{"text":"HackComment: // HACK #1: no space is allowed between \u0022Defaults\u0022 and \u0027!\u003E@:\u0027. The below avoids having to \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/sudoers/ast.rs"},"region":{"startLine":708}}}],"partialFingerprints":{"codehealthFindingId/v1":"50efdf7742f49056487256cbeed1ef1f754086ecdd071229e69ea6e06108ffc0"}},{"ruleId":"D17","level":"warning","message":{"text":"HackComment: // HACK #2: \u0027@\u0027 can be part of a username, so it will already have been parsed; \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/sudoers/ast.rs"},"region":{"startLine":711}}}],"partialFingerprints":{"codehealthFindingId/v1":"be43ee3e1b5b9b18379bcb8fc097cf84d7f95d2101dd702093823620cae74452"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: remove unused attribute when system is cleaned up \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/system/mod.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0480209b302ba64f30a45c3e538a137baa005add9528753eba01e5645d0ad197"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME use /dev/fd on macOS \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/system/mod.rs"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"d3097bf3dfa4b0669f1c818b17d4370971fe73e46124effa6e2a485a4cf87eff"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME add debug assertion that we are not currently using multiple threads. \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/system/mod.rs"},"region":{"startLine":127}}}],"partialFingerprints":{"codehealthFindingId/v1":"596a0e222b1e8b81373b70c56bf4301285609430ffe621b777909be3c542dafa"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME use UnixStream::peer_cred() once stable in our MSRV. \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/system/audit.rs"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e7afd0d14dcb2737eb492f0cdfe2373cfd6fd85e1315b72fe8130380ea4d773"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // | IUCLC /* FIXME: not in libc */ \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/system/term/user_term.rs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e7ac726e4b075c45f98fec33040878bb2d7b3627e5dd8dc7c6a16499f82dc11"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // | XCASE /* FIXME: not in libc */ \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/system/term/user_term.rs"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed62765e9210a756ca8f2664c170e28b424cb49ef2821cc1bdf7b17472ef2f6a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: this is incorrect, will be fixed in a future sudo; the \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":"test-framework/sudo-compliance-tests/src/visudo.rs"},"region":{"startLine":160}}}],"partialFingerprints":{"codehealthFindingId/v1":"2649669a679e2faab3a15cbcc308927c417e25a4466c29e2354a6d1975dd84b7"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME workaround for gh506. change to \u0060SHELL=my-shell\u0060 \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":"test-framework/sudo-compliance-tests/src/su/env.rs"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e8f716d0596830050a88e0402837759cc4d8db2887dc3c975376579eacf80aa"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX unclear why this and the tests that follow need a pseudo-TTY allocation to pass \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":"test-framework/sudo-compliance-tests/src/sudo/timestamp.rs"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"9077c215bb64b6ac8f9e83dde2d3371ea666172e2e48905a67054d1a4b35da81"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: sudo 1.9.16p2 has strange behaviour and doesn\u0027t allow the user to run -U; whereas \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":"test-framework/sudo-compliance-tests/src/sudo/flag_list/needs_auth.rs"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"c47b511ab04cfe2c6e532405eb064dae77ea5ad8d38a96d9de6c1ef75bbe66d0"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: a recent change in ogsudo has broken this error message, \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":"test-framework/sudo-compliance-tests/src/sudo/sudoers/includedir.rs"},"region":{"startLine":120}}}],"partialFingerprints":{"codehealthFindingId/v1":"bdc11cfa2c0767e37f51f2aaf7e424487bb7d89afda956e85ffb9eb9373102ea"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: a recent change in ogsudo has broken this error message, \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":"test-framework/sudo-compliance-tests/src/sudo/sudoers/includedir.rs"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5dce90fecd38390b45b96b431bb9207a138d149f5199ad5207501eb21c156b8"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: sudo 1.9.16p2 has a different interaction with a bare \u0022-g\u0022 and the runas specifier \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":"test-framework/sudo-compliance-tests/src/sudoedit/sudoers.rs"},"region":{"startLine":117}}}],"partialFingerprints":{"codehealthFindingId/v1":"66f4593be863e14c3f44a8303da8e300d58269c0893ebd47228eb6d4877377e9"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no licence statement: No licence statement appears in any document, despite the project being a Rust library. Add a LICENSE file or a one-line licence line describing the permitted use."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc71d4465cafafc3df7a1b646a8cd5f9a306761b307dc199a6de8c1c35d8b21d"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D22","level":"warning","message":{"text":"Two top-level factory methods with nearly identical signatures and return types, differing only by a suffix (\u0027NoImplicit\u0027). The semantic distinction is not obvious from the signature alone and requires external documentation to understand.: Rename to reflect the specific behavior difference (e.g., \u0027EnvWithImplicitDefaults\u0027 vs \u0027EnvWithoutImplicitDefaults\u0027) or consolidate into a single \u0027Env\u0027 method with an optional configuration parameter if the \u0027NoImplicit\u0027 behavior is the standard use case. (signatures: sudo_test.Env(sudoers: impl Into\u003CTextFile\u003E): EnvBuilder | sudo_test.EnvNoImplicit(sudoers: impl Into\u003CTextFile\u003E): EnvBuilder)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d1651c09895c4c52a9521faeec678f1964644007a07395c00949147253e94056"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 1 of 4 project(s) overshoot their size bounds, lowering Project Cohesion to 5.0/10. The most over is \u0060(repository root)\u0060 (19436 LoC, 183 public types across 22 directories). Review these for cohesion \u2014 draw the boundary inside the module first (group each responsibility into its own package or directory and keep the cross-boundary members non-public), since splitting a published package moves types between packages and breaks consumers."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"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":"89e44872616047bf8d867c607e5035a6c23e8ba0e062cb153649783099b5c43b"},"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":"dc483d61446b706b5e0251ebbe66bc87cd49090d6e8c0eb71d3a3665373a9763"},"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":"b82a2289f2e988b0078710ce06d0d72f5a96645903fb36e3513f9bfab3800a6c"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","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":"a4ef1e8ad685b40c17f68e1a356d865101a8bab96ef7f19763cbab7c68c27810"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D34","level":"note","message":{"text":"Orphaned files with no living knowledge: 15 of 146 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 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 146 of the 204 production source files in this repository met that bar). 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 most significant first: test-framework/sudo-compliance-tests/src/su/flag_login.rs, test-framework/sudo-compliance-tests/src/sudo/sudoers/cwd.rs, test-framework/sudo-compliance-tests/src/su/flag_whitelist_environment.rs, src/common/bin_serde.rs, test-framework/e2e-tests/src/pty.rs, test-framework/sudo-compliance-tests/src/su/flag_preserve_environment.rs, test-framework/sudo-compliance-tests/src/su/env.rs, test-framework/sudo-compliance-tests/src/sudo/flag_list/not_allowed.rs (and 7 more). Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: no_pty.rs \u2194 parent.rs: \u0060src/exec/no_pty.rs\u0060 and \u0060src/exec/use_pty/parent.rs\u0060 change together 77% of the time (37 of the 48 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 37 shared commits counted here, the most recent 3 are \u0060e2098c38\u0060 Remove untracked signal modification functions; \u00602ab4a4f4\u0060 Signal state (#1420); \u006065576dd2\u0060 Fix compilation on FreeBSD \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/no_pty.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5611324f52c36a47ceda270d3eb77e4dfebe33abeafbb1f1f72a3582bb3c8cb0"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: no_pty.rs \u2194 monitor.rs: \u0060src/exec/no_pty.rs\u0060 and \u0060src/exec/use_pty/monitor.rs\u0060 change together 69% of the time (33 of the 48 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 33 shared commits counted here, the most recent 3 are \u0060e2098c38\u0060 Remove untracked signal modification functions; \u00602ab4a4f4\u0060 Signal state (#1420); \u0060c28c87f8\u0060 Extract exec_command function out of no_pty and use_pty \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/exec/no_pty.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"54d15c4ad709481423a63da67aea4f3b39c54ed1a7598be69d63b9fc05634a50"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4f1e7684f8281bd255ddef97105c7055c2ebae06964c8745716b83e174debe83"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c593d4dbd23724f337bfb8d2c1653812f143ae255bd9b59d4b3e9c37779fa6f4"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M2","level":"note","message":{"text":"No architecture diagram/doc: No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named \u0060architecture\u0060 or \u0060design\u0060 in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape \u2014 no \u0060## Architecture\u0060 heading, no \u0022architecture overview\u0022/\u0022high-level design\u0022 phrasing, no \u0022the architecture is \u2026\u0022 introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document \u2014 not evidence that nobody wrote the shape down."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0c190e7c159d1850ee706c3ac4486e151e8a4fe169de4bf61436b9f399654f06"}},{"ruleId":"P12","level":"warning","message":{"text":"Coverage collected but not gated: CI collects a coverage report but no step enforces a minimum \u2014 coverage could halve and CI stays green. Add a step that fails the build when coverage drops below a floor (your coverage tool\u0027s minimum-threshold flag, or a coverage-gate action) so the number guards something. What was searched, so you can tell an absence from a miss: this repository\u0027s CI files AND its coverage configuration \u2014 the well-known coverage and test-runner config files, read at the repository root and inside workspace package directories two levels down, so a floor declared beside the tests rather than in the pipeline is credited \u2014 matched against the threshold settings this check knows by name. A floor set in your coverage service\u0027s web UI rather than in a committed file, or under a setting whose name is not one of those, is not seen here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"7f63c06044cf998d9a0698692df30d8873e29bc9b0c98ee829255017c1193d33"}}],"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-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"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-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-250","guid":"52ee12e8-390a-7351-b1e6-388afa694e54","name":"CWE-250","shortDescription":{"text":"CWE-250"},"helpUri":"https://cwe.mitre.org/data/definitions/250.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-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-506","guid":"401d6455-56e3-0552-9a39-f77461673e3f","name":"CWE-506","shortDescription":{"text":"CWE-506"},"helpUri":"https://cwe.mitre.org/data/definitions/506.html"},{"id":"CWE-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.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-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.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."]}]}