{"$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":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D10","name":"Test Quality","shortDescription":{"text":"Test Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D10"},{"id":"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":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"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":"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":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"CompiledUrlMatcherTrait.doMatch (cyclomatic 61): CompiledUrlMatcherTrait.doMatch has cyclomatic complexity 61 (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":"Matcher/Dumper/CompiledUrlMatcherTrait.php"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"57b478317448d1081f0a6384a1be898bbd6e85973391e340ea0215a9c4783aeb"}},{"ruleId":"D1","level":"warning","message":{"text":"UrlGenerator.doGenerate (cyclomatic 60): UrlGenerator.doGenerate has cyclomatic complexity 60 (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":"Generator/UrlGenerator.php"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"906a25df6dba9ae373ba82fe468bd512dc0ba6fb8d042a9d636576f0f95e64eb"}},{"ruleId":"D1","level":"warning","message":{"text":"RouteCompiler.compilePattern (cyclomatic 43): RouteCompiler.compilePattern has cyclomatic complexity 43 (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":"RouteCompiler.php"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"f1f1c0cf46fb5af7949ac57aa4a7567efe334a4e95651050628e195f2f3eea28"}},{"ruleId":"D1","level":"warning","message":{"text":"TraceableUrlMatcher.matchCollection (cyclomatic 32): TraceableUrlMatcher.matchCollection has cyclomatic complexity 32 (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":"Matcher/TraceableUrlMatcher.php"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"9fa650574b978fbd8bb06c5f2b5b2cfefa38d2505955825026bac34cbabb6eb8"}},{"ruleId":"D1","level":"warning","message":{"text":"AttributeClassLoader.addRoute (cyclomatic 29): AttributeClassLoader.addRoute has cyclomatic complexity 29 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/AttributeClassLoader.php"},"region":{"startLine":145}}}],"partialFingerprints":{"codehealthFindingId/v1":"c8181c7ce8cd7b6ce0b758c338e7b24e5de1f785cca80d15f1a771efcdca77df"}},{"ruleId":"D1","level":"warning","message":{"text":"UrlMatcher.matchCollection (cyclomatic 28): UrlMatcher.matchCollection has cyclomatic complexity 28 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Matcher/UrlMatcher.php"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"6be5b4addf722c860b966e85282629fa853992860c89f7f39d48d20b6f2a44a6"}},{"ruleId":"D1","level":"warning","message":{"text":"StaticPrefixCollection.getCommonPrefix (cyclomatic 26): StaticPrefixCollection.getCommonPrefix has cyclomatic complexity 26 (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":"Matcher/Dumper/StaticPrefixCollection.php"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"ef09bb255f1989ab4a5c81b585ab1646ca50148db3a50d388d704e36df3b0bdc"}},{"ruleId":"D1","level":"warning","message":{"text":"RedirectController.urlRedirectAction (cyclomatic 21): RedirectController.urlRedirectAction has cyclomatic complexity 21 (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":"Controller/RedirectController.php"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"4251770eb57433754fd56574edbf6f637dd632892112eb20790993f7dac6e967"}},{"ruleId":"D1","level":"warning","message":{"text":"AttributeClassLoader.load (cyclomatic 20): AttributeClassLoader.load 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":"Loader/AttributeClassLoader.php"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9c0c38ad5c2fc8e8db7e3c8e1d1fba614b9d4302f16f2376f1cd11dd40b9c6d"}},{"ruleId":"D1","level":"warning","message":{"text":"AttributeFileLoader.findClass (cyclomatic 20): AttributeFileLoader.findClass 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":"Loader/AttributeFileLoader.php"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"71db1ead91c8c57dd11ce5a82c1100f84b98d2450181477d824c7b1348ae4dc7"}},{"ruleId":"D1","level":"warning","message":{"text":"PrefixTrait.addPrefix (cyclomatic 17): PrefixTrait.addPrefix has cyclomatic complexity 17 (threshold 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/Configurator/Traits/PrefixTrait.php"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"c8c449987e22c15c5f7d1c99e37940cc87ed90af130168507a7f2f7b2ff87a89"}},{"ruleId":"D1","level":"warning","message":{"text":"ContentLoaderTrait.parseImport (cyclomatic 16): ContentLoaderTrait.parseImport 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":"Loader/ContentLoaderTrait.php"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"937c5bd0050bce2cf244985669b382f70f0e57fe9049d7363aa590ac249f5dea"}},{"ruleId":"D1","level":"warning","message":{"text":"ContentLoaderTrait.validate (cyclomatic 16): ContentLoaderTrait.validate has cyclomatic complexity 16 (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":"Loader/ContentLoaderTrait.php"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c00759c78d4dfdb506d04fa3a0882ed9c52bbf7a58c238da1a8aff830e751df"}},{"ruleId":"D2","level":"warning","message":{"text":"CompiledUrlMatcherTrait.doMatch (cognitive 121): CompiledUrlMatcherTrait.doMatch has cognitive complexity 121 (threshold 15). Drivers by points: if/else 21 (62 pts), boolean chains 26, ternaries 9 (22 pts), loops 5 (11 pts) (nesting depth added 60). 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":"Matcher/Dumper/CompiledUrlMatcherTrait.php"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"8da03a12b1cd905e860aded21975ad927a2681b59b7d75d0fefef93db530f863"}},{"ruleId":"D2","level":"warning","message":{"text":"UrlGenerator.doGenerate (cognitive 106): UrlGenerator.doGenerate has cognitive complexity 106 (threshold 15). Drivers by points: if/else 38 (72 pts), boolean chains 16, ternaries 4 (13 pts), loops 3 (5 pts) (nesting depth added 45). 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":"Generator/UrlGenerator.php"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"d107f76e106cb3221f434e19d42bd134af46b756a533f1768382c37a0f17dc39"}},{"ruleId":"D2","level":"warning","message":{"text":"RouteCompiler.compilePattern (cognitive 68): RouteCompiler.compilePattern has cognitive complexity 68 (threshold 15). Drivers by points: if/else 21 (38 pts), boolean chains 13, ternaries 5 (11 pts), loops 4 (6 pts) (nesting depth added 25). 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":"RouteCompiler.php"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"54f6a3a98f188342a251f279d15e65806f0df54fc0dc2e11b05876cfbfbe30c3"}},{"ruleId":"D2","level":"warning","message":{"text":"AttributeClassLoader.addRoute (cognitive 52): AttributeClassLoader.addRoute has cognitive complexity 52 (threshold 15). Drivers by points: if/else 13 (27 pts), loops 9 (16 pts), ternaries 3 (6 pts), boolean chains 3 (nesting depth added 24). 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":"Loader/AttributeClassLoader.php"},"region":{"startLine":145}}}],"partialFingerprints":{"codehealthFindingId/v1":"0db48ad35a1e6ed31122dd03376f1a07f0fe16292c3335b5d6eb51c2e24cfaa4"}},{"ruleId":"D2","level":"warning","message":{"text":"TraceableUrlMatcher.matchCollection (cognitive 51): TraceableUrlMatcher.matchCollection has cognitive complexity 51 (threshold 15). Drivers by points: if/else 14 (31 pts), boolean chains 15, loops 2 (4 pts), ternaries 1 (nesting depth added 19). 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":"Matcher/TraceableUrlMatcher.php"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"1de230151c3820f472d3760001289ce4411da2bb1c5a164229f1d23221b177f7"}},{"ruleId":"D2","level":"warning","message":{"text":"PrefixTrait.addPrefix (cognitive 41): PrefixTrait.addPrefix has cognitive complexity 41 (threshold 15). Drivers by points: loops 7 (21 pts), if/else 5 (9 pts), ternaries 2 (9 pts), boolean chains 2 (nesting depth added 25). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/Configurator/Traits/PrefixTrait.php"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"62bd9736bcb807881d849542b72d7b2ed05a38dc05dc28d52be4a20bb902b10b"}},{"ruleId":"D2","level":"warning","message":{"text":"UrlMatcher.matchCollection (cognitive 40): UrlMatcher.matchCollection has cognitive complexity 40 (threshold 15). Drivers by points: if/else 12 (24 pts), boolean chains 14, loops 1, ternaries 1 (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":"Matcher/UrlMatcher.php"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"ffee2ef2aa905ff957840a9ee91a4067eafa4ee251fc7351cfa822d122d768bb"}},{"ruleId":"D2","level":"warning","message":{"text":"AttributeFileLoader.findClass (cognitive 39): AttributeFileLoader.findClass has cognitive complexity 39 (threshold 15). Drivers by points: if/else 10 (27 pts), loops 3 (7 pts), boolean chains 5 (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":"Loader/AttributeFileLoader.php"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"975ec4b27c77b71c11595437f952f0354085c04f0b1c2b460699b70687c3ed9c"}},{"ruleId":"D2","level":"warning","message":{"text":"StaticPrefixCollection.getCommonPrefix (cognitive 38): StaticPrefixCollection.getCommonPrefix has cognitive complexity 38 (threshold 15). Drivers by points: if/else 7 (20 pts), boolean chains 12, loops 3 (6 pts) (nesting depth added 16). 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":"Matcher/Dumper/StaticPrefixCollection.php"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"0445c05e0d6185d7f02b10e95fa43a932f1b8a92dabf447f406e2a23c5c6d456"}},{"ruleId":"D2","level":"warning","message":{"text":"RedirectController.urlRedirectAction (cognitive 35): RedirectController.urlRedirectAction has cognitive complexity 35 (threshold 15). Drivers by points: if/else 17 (26 pts), ternaries 4 (7 pts), boolean chains 2 (nesting depth added 12). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Controller/RedirectController.php"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"42a35d290d211c35bd6ae4313ceb444d7b7d2781ce3a641dc06ed271d22d14a9"}},{"ruleId":"D2","level":"warning","message":{"text":"AttributeClassLoader.load (cognitive 31): AttributeClassLoader.load has cognitive complexity 31 (threshold 15). Drivers by points: if/else 12 (21 pts), loops 4 (7 pts), boolean chains 3 (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":"Loader/AttributeClassLoader.php"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ab097afa7c8bcfb35bf26ffaa8ce1b493858c7a63742af7b6dc3ae42a88c550"}},{"ruleId":"D2","level":"warning","message":{"text":"AttributeClassLoader.getGlobals (cognitive 25): AttributeClassLoader.getGlobals has cognitive complexity 25 (threshold 15). Drivers by points: if/else 12 (23 pts), loops 1 (2 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":"Loader/AttributeClassLoader.php"},"region":{"startLine":279}}}],"partialFingerprints":{"codehealthFindingId/v1":"5de92aa59e77e6dd57b0a9eb77609f980b507118c7914dc1c27bfda1360cfe87"}},{"ruleId":"D2","level":"warning","message":{"text":"StaticPrefixCollection.addRoute (cognitive 23): StaticPrefixCollection.addRoute has cognitive complexity 23 (threshold 15). Drivers by points: if/else 7 (17 pts), boolean chains 5, loops 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":"Matcher/Dumper/StaticPrefixCollection.php"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"6dd38602be2c7a246c4b76a0da87795cd7ff507cafcb98f14b027b49f5ad5aaf"}},{"ruleId":"D2","level":"warning","message":{"text":"ContentLoaderTrait.parseImport (cognitive 21): ContentLoaderTrait.parseImport has cognitive complexity 21 (threshold 15). Drivers by points: if/else 13 (19 pts), loops 1, ternaries 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":"Loader/ContentLoaderTrait.php"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"3644d5eca49cd9db814ca220a327eac5575fda60a2d4890b54b529d9a16b989c"}},{"ruleId":"D2","level":"warning","message":{"text":"RedirectController.redirectAction (cognitive 19): RedirectController.redirectAction has cognitive complexity 19 (threshold 15). Drivers by points: if/else 8 (12 pts), ternaries 3 (6 pts), boolean chains 1 (nesting depth added 7). 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":"Controller/RedirectController.php"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a055d07ade51f1b64f5d06643f5361a68ab1722ea3a249113388797faf02452"}},{"ruleId":"D2","level":"warning","message":{"text":"RedirectableUrlMatcher.match (cognitive 19): RedirectableUrlMatcher.match has cognitive complexity 19 (threshold 15). Drivers by points: if/else 4 (9 pts), error handling 3 (7 pts), ternaries 1 (3 pts) (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Matcher/RedirectableUrlMatcher.php"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"127bb8e859cc44c2f3b09f6a666153eb671e4796f67c70ab702b6589e8028c9e"}},{"ruleId":"D2","level":"warning","message":{"text":"Router.resolve (cognitive 16): Router.resolve has cognitive complexity 16 (threshold 15). Drivers by points: if/else 8 (10 pts), ternaries 1 (3 pts), loops 1 (2 pts), boolean chains 1 (nesting depth added 5). 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":"DependencyInjection/Router.php"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"37ce2d55dde25b24901fec4def3d73902b173fdbf3267a82b276c2e5b32cd3a7"}},{"ruleId":"D2","level":"warning","message":{"text":"CompiledUrlGeneratorDumper.getCompiledAliases (cognitive 16): CompiledUrlGeneratorDumper.getCompiledAliases has cognitive complexity 16 (threshold 15). Drivers by points: if/else 4 (10 pts), loops 2 (3 pts), ternaries 1 (2 pts), 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":"Generator/Dumper/CompiledUrlGeneratorDumper.php"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"a080544dc2afb32915b980dca724f187b6722297bd566a8027a86036d8a7bfcd"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Route: TooManyMethods \u2014 37 methods. The bar is 30 methods; this is 7 over it, 1.23\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Route.php"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"2cb6f90a6cf856a9cadbd678908560e456f0697b10d4d952c34235e4a526f239"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: UrlGenerator.doGenerate: MethodTooLong \u2014 doGenerate runs 109 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 9 over it, 1.09\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":"Generator/UrlGenerator.php"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"a09617e24cc423984137a6ba64865940ea3b1a26b1511d6f7bef196bd6db3910"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): Matcher/TraceableUrlMatcher.php:56-68 | Matcher/UrlMatcher.php:117-129 \u2014 before extracting anything, compare \u0060Matcher/TraceableUrlMatcher.php\u0060 and \u0060Matcher/UrlMatcher.php\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 31 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Matcher/TraceableUrlMatcher.php:56\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060Matcher/TraceableUrlMatcher.php:69\u0060 calls \u0060addTrace\u0060, \u0060sprintf\u0060, \u0060getPath\u0060 and \u0060Matcher/UrlMatcher.php:130\u0060 does not \u2014 after which the two agree again for 4 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Matcher/TraceableUrlMatcher.php"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"873c88365b9544717c020f820424af210b91e0a9290c1e165d1df6d6f88cb3aa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): Loader/ContentLoaderTrait.php:91-100 | Loader/ContentLoaderTrait.php:145-154 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Loader/ContentLoaderTrait.php:91\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/ContentLoaderTrait.php"},"region":{"startLine":91}}}],"partialFingerprints":{"codehealthFindingId/v1":"1569815da4255b2d33ecdc70a77a63f03e4b6eca9374f1502e0ccb3467d9719d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): Matcher/TraceableUrlMatcher.php:70-78 | Matcher/UrlMatcher.php:130-138 \u2014 before extracting anything, compare \u0060Matcher/TraceableUrlMatcher.php\u0060 and \u0060Matcher/UrlMatcher.php\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 31 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Matcher/TraceableUrlMatcher.php:70\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just before the matched lines, \u0060Matcher/TraceableUrlMatcher.php:69\u0060 calls \u0060addTrace\u0060, \u0060sprintf\u0060, \u0060getPath\u0060 and \u0060Matcher/UrlMatcher.php:129\u0060 does not \u2014 after which the two agree again for 4 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Matcher/TraceableUrlMatcher.php"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e4beb3860459e9e85766803c257862258a06bd8d39385145abd91399c4977ce"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): Matcher/TraceableUrlMatcher.php:105-113 | Matcher/UrlMatcher.php:145-153 \u2014 before extracting anything, compare \u0060Matcher/TraceableUrlMatcher.php\u0060 and \u0060Matcher/UrlMatcher.php\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 31 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Matcher/TraceableUrlMatcher.php:105\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060Matcher/TraceableUrlMatcher.php:114\u0060 calls \u0060addTrace\u0060, \u0060sprintf\u0060, \u0060requirement\u0060 and \u0060Matcher/UrlMatcher.php:154\u0060 does not \u2014 after which the two agree again for 4 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Matcher/TraceableUrlMatcher.php"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"c59a53e858125c7cdcb96b45c82d141b62177d4ef24937fa52bfb3cc9215a253"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): Loader/ContentLoaderTrait.php:101-108 | Loader/ContentLoaderTrait.php:155-162 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Loader/ContentLoaderTrait.php:101\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/ContentLoaderTrait.php"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"5816d8bf4ca9ce15b8162f850d8de27f2a53d287cc6313610a2d9cdb1d0c35c6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Generator/CompiledUrlGenerator.php:62-67 | Generator/UrlGenerator.php:132-137 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060Generator/CompiledUrlGenerator.php:62\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Generator/CompiledUrlGenerator.php"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"890c1269dea950588f8414b41bd3eb2e1b08bf0038856e6d49abddcb3f22b04c"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testWarmUpWithWarmableInterfaceWithBuildDir: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/CacheWarmer/RouterCacheWarmerTest.php"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"4ec3028a2c29b53e888fbdc57f7b337ed8b1dfc6dc36258fd02b4d1b386e2f80"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testDeprecatedAlias: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/Dumper/CompiledUrlGeneratorDumperTest.php"},"region":{"startLine":359}}}],"partialFingerprints":{"codehealthFindingId/v1":"219854611220aaea007a16f85188b80f99b29558cee084b530f84ec1ec0ccee5"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testDeprecatedAliasWithCustomMessage: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/Dumper/CompiledUrlGeneratorDumperTest.php"},"region":{"startLine":375}}}],"partialFingerprints":{"codehealthFindingId/v1":"9fed468193d349c9baf46072330bc90ec42a6914fe0ab06f0734c199cc518a62"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testTargettingADeprecatedAliasShouldTriggerDeprecation: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/Dumper/CompiledUrlGeneratorDumperTest.php"},"region":{"startLine":391}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e3c7af3da576cc3040e8a07a7e29f700241142480d36b32eb46e692cd0db0cb"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testDeprecatedAlias: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/UrlGeneratorTest.php"},"region":{"startLine":877}}}],"partialFingerprints":{"codehealthFindingId/v1":"367976c26190bf8d1c47cc6dc12a5d284f87002501062b56fd47f4459938c108"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testDeprecatedAliasWithCustomMessage: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/UrlGeneratorTest.php"},"region":{"startLine":890}}}],"partialFingerprints":{"codehealthFindingId/v1":"5dbd0248df2f3362f3691171c25881e4fc84911aa475a91b997c9e3fd4b3d515"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testTargettingADeprecatedAliasShouldTriggerDeprecation: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Generator/UrlGeneratorTest.php"},"region":{"startLine":903}}}],"partialFingerprints":{"codehealthFindingId/v1":"446d4470e310f23908917e65e183a80ce6a34f5d1eddf9df4e473b696a3ac0d4"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testLoadDirectory: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Loader/DirectoryLoaderTest.php"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ff28217ac7d33a21db17f086ba367a8094812115f98ace93312e8eb546d4dcb"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testImportDirectory: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Loader/DirectoryLoaderTest.php"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"494f3401e0d85a6b5366fa40a21722063bba705a0a439c0974c9459190a1cff0"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testRedirectPreservesUrlEncoding: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/Dumper/CompiledUrlMatcherDumperTest.php"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"c3a4681007f06e2ca16046f65becf6d9c148b5c3925f3d5f39328e70cd671710"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testMissingTrailingSlash: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"bca664d461e18b43fc8669b40414d1d82789d09cd7738753aebedeed80116a4f"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testExtraTrailingSlash: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"d99f24e18fcc3a82b59e255467770ff2ac0154f92ea4ebebec2b1d0d5f83380a"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testSchemeRedirectRedirectsToFirstScheme: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"f37b300352555bc90e8d0aeb5b0435d9d1db0d2c46bf92409f72208e2bce2db9"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testNoSchemaRedirectIfOneOfMultipleSchemesMatches: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"083cf08313005985a9fedf842f2f175bf0ecede37176c43a3af2abbae773155b"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testRedirectPreservesUrlEncoding: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":127}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2f587ae894e9127228f71621f113a64ee64a1b3365b0e34bc3b718d1e0c4f88"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testMissingTrailingSlashAndScheme: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/Matcher/RedirectableUrlMatcherTest.php"},"region":{"startLine":165}}}],"partialFingerprints":{"codehealthFindingId/v1":"465fd5379f097e08715a51585e94bd8ed41b16b5fc31fe966e06894a052aae42"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testMatchRequestWithUrlMatcherInterface: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/RouterTest.php"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ddcb8292084b3644b564ce57eeb6533238d0583bb05a5376776f7f73d42b2ed"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testMatchRequestWithRequestMatcherInterface: This method\u0027s body runs code, and no assertion call was recognised in it. Recognised by name: Assert*, *Should*/ShouldBe*, Verify, Expect, Throws, Record, Received/DidNotReceive, MustHaveHappened/MustNotHaveHappened, EnsureSuccessStatusCode and *AndEnsure* \u2014 so verification routed through a helper of your own naming, through a base-class or callback object whose members hold the assertions, or through a harness that fails by throwing under some other name, is not visible to this check and is not counted here. Read it as \u0027no assertion this check knows how to see\u0027, and if that is right, add one."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/RouterTest.php"},"region":{"startLine":166}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd74045d867f187d824f128649b25baa9dff1dd3d4a5a41c6ca38d15208e5fdd"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: Generator/UrlGenerator.php: Generator/UrlGenerator.php changed 4 times in last 90 days, max cyclomatic complexity 60 in UrlGenerator.doGenerate at line 147. 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-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 18:20:30 \u002B02:00\u0027 --until=\u00272026-09-24 18:20:30 \u002B02:00\u0027 --full-history --no-merges -- Generator/UrlGenerator.php\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":"Generator/UrlGenerator.php"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"73563eb7c6ad4716001138eaf95965f101c5a033fc4f2ac1841dede6287c5ab2"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: RouteCompiler.php: RouteCompiler.php changed 3 times in last 90 days, max cyclomatic complexity 43 in RouteCompiler.compilePattern at line 107. 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-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 18:20:30 \u002B02:00\u0027 --until=\u00272026-09-24 18:20:30 \u002B02:00\u0027 --full-history --no-merges -- RouteCompiler.php\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":"RouteCompiler.php"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4875d3e33b174615669730fdb847224448bbf0d9131b6b111cdedcc918d7cc4"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: Loader/ContentLoaderTrait.php: Loader/ContentLoaderTrait.php changed 2 times in last 90 days, max cyclomatic complexity 16 in ContentLoaderTrait.parseImport at line 129. 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-06-26..2026-09-24, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-26 18:20:30 \u002B02:00\u0027 --until=\u00272026-09-24 18:20:30 \u002B02:00\u0027 --full-history --no-merges -- Loader/ContentLoaderTrait.php\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":"Loader/ContentLoaderTrait.php"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"2fb782d8520778ddfb7868cc7dc7d81a78c021d89979823225c234a61b142c18"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 2 significant file(s) lose their only recent owner: CompiledRoute.php, Loader/YamlFileLoader.php. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"bfb28f8176ae10f67d8c4ebfc36c57a92e7a5bc9ab84f0240cf50166042ca1b2"}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned files with no living knowledge: 6 of 35 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; 35 of the 85 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: Router.php, Matcher/TraceableUrlMatcher.php, Generator/UrlGeneratorInterface.php, Loader/ObjectLoader.php, Loader/Configurator/Traits/LocalizedRouteTrait.php, Alias.php. 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: CollectionConfigurator.php \u2194 ImportConfigurator.php: \u0060Loader/Configurator/CollectionConfigurator.php\u0060 and \u0060Loader/Configurator/ImportConfigurator.php\u0060 change together 65% of the time (11 of the 17 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 11 shared commits counted here, the most recent 3 are \u00606fc4c445\u0060 Replace __sleep/wakeup() by __(un)serialize() for throwing and intern\u2026; \u00601b8f64bf\u0060 Replace __sleep/wakeup() by __(un)serialize() when BC isn\u0027t a concern; \u006072826485\u0060 Add missing return types to magic methods \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Loader/Configurator/CollectionConfigurator.php"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"76a9d6c63153a74ae40daecd75be0f2e2ddfc8503ed0ee21074ab1fb7b76ddad"}},{"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":"M3","level":"note","message":{"text":"No src/ separation: Production code isn\u0027t grouped under a src/ folder \u2014 it\u0027s spread across several top-level directories, so there\u0027s no one place that says \u0027this is the product\u0027."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fdf7f5b24e313364d10170a5c2542959902fe0d26ed70edef3eaa10ec5bcdb1d"}},{"ruleId":"P1","level":"warning","message":{"text":"No CI pipeline: No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, \u2026) \u2014 changes aren\u0027t gated by an automated build/test."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"44f01af96e50474fba2df84d95ddf4f7e1d34c29c307830b9efc9057daa6b670"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add phpstan or psalm (or \u0060semgrep --config=auto\u0060, which runs on any language) \u2014 this repository has no CI pipeline yet, so run it locally to clear the existing findings, then make it a step of the first workflow you add so a regression fails the build. What was searched, so you can tell an absence from a miss: the 0 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":0,"secretScannerRunsExcluded":0}}}]}