{"$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":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"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":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"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":"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":"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":"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":"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":"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":"PF2","name":"Allocation hygiene","shortDescription":{"text":"Allocation hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/PF2"},{"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":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"SectionModel.calculateElementFramesIfNecessary (cyclomatic 32): SectionModel.calculateElementFramesIfNecessary 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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":486}}}],"partialFingerprints":{"codehealthFindingId/v1":"933e46400c62bafda2dfdd949e0ac163d6ed118827d0edfcaf639a6fb7bf9387"}},{"ruleId":"D1","level":"warning","message":{"text":"LayoutState.targetContentOffsetAnchor (cyclomatic 22): LayoutState.targetContentOffsetAnchor has cyclomatic complexity 22 (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":"MagazineLayout/LayoutCore/LayoutState.swift"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"7095aa9d3bba099e95934395d4ace8b735e17fe57e8e3545829ac7cff60c0a7b"}},{"ruleId":"D1","level":"warning","message":{"text":"MagazineLayout.prepare (cyclomatic 16): MagazineLayout.prepare 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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"c65894654f45ea5481a7607ba0cbada2218c9a70d01199f1ff5dca5356794b75"}},{"ruleId":"D2","level":"warning","message":{"text":"SectionModel.calculateElementFramesIfNecessary (cognitive 45): SectionModel.calculateElementFramesIfNecessary has cognitive complexity 45 (threshold 15). Drivers by points: if/else 23 (32 pts), boolean chains 11, loops 2 (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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":486}}}],"partialFingerprints":{"codehealthFindingId/v1":"3875ff684cce41fb9edefaef7806614d2aa0b388513325dcf46d95cefe08567e"}},{"ruleId":"D2","level":"warning","message":{"text":"MagazineLayout.prepare (cognitive 39): MagazineLayout.prepare has cognitive complexity 39 (threshold 15). Drivers by points: if/else 16 (36 pts), boolean chains 2, loops 1 (nesting depth added 20). 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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"2aa01d0951848e8a90c456ee5bed33825462a0936207f08b94340c5474d01303"}},{"ruleId":"D2","level":"warning","message":{"text":"LayoutState.targetContentOffsetAnchor (cognitive 21): LayoutState.targetContentOffsetAnchor has cognitive complexity 21 (threshold 15). Drivers by points: match/switch 5 (9 pts), boolean chains 6, if/else 5, loops 1 (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":"MagazineLayout/LayoutCore/LayoutState.swift"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"a910aa18656e7847a7ead2affd02831f1bc0e0172a780a46d72f44d2afe067d1"}},{"ruleId":"D2","level":"warning","message":{"text":"MagazineLayout.prepare (cognitive 18): MagazineLayout.prepare has cognitive complexity 18 (threshold 15). Drivers by points: if/else 11 (14 pts), boolean chains 2, loops 1 (2 pts) (nesting depth added 4). 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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"25ea06df0fb3c9b3a544179ed766ebe748082b35abe671409311a64544123eca"}},{"ruleId":"D2","level":"warning","message":{"text":"MagazineLayout.invalidationContext (cognitive 17): MagazineLayout.invalidationContext has cognitive complexity 17 (threshold 15). Drivers by points: if/else 5 (9 pts), match/switch 4 (8 pts) (nesting depth added 8). 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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":745}}}],"partialFingerprints":{"codehealthFindingId/v1":"f099de6986c7a51a8ebc8e91a73657a2c83c1e4373cb51427b9be66c13bf181a"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: MagazineLayout: ClassTooLong \u2014 887 significant lines (blank, comment-only and punctuation-only lines excluded), 43 methods. The bar is 400 significant lines; this is 487 over it, 2.22\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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca6d522786744aca24b5b0bd61c4db6e7cfbdc987327bda4e1a74c0bcb63db5e"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: Public/MagazineLayout.swift: FileTooLong \u2014 984 significant lines (blank, comment-only and punctuation-only lines excluded), about 90% of them inside a single declaration: MagazineLayout (26-1397). The bar is 500 significant lines; this is 484 over it, 1.97\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c10bad36b843e417c3ece5630370977857499535ddea8c6e3f9770287715fa4"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: ModelState: TooManyMethods \u2014 54 methods. The bar is 30 methods; this is 24 over it, 1.80\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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"394e0607a607c94e30563a4903fc47268de6e157f950353aa88be1ffd63c5637"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: MagazineLayout: TooManyMethods \u2014 43 methods. The bar is 30 methods; this is 13 over it, 1.43\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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f9bf0a0329a5c4e3db3c369457c468e4eac8d3457d085595b798f64763b63a3"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ModelState: ClassTooLong \u2014 471 significant lines (blank, comment-only and punctuation-only lines excluded), 54 methods. The bar is 400 significant lines; this is 71 over it, 1.18\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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"62f9db5a8a6001f40538adeaf40ea662d34033e314d2747cfabfa9d2ba1f1568"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: SectionModel: TooManyMethods \u2014 35 methods. The bar is 30 methods; this is 5 over it, 1.17\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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"7be915724cf500ce9d686431ad16627c8e6ae4119fead2fdab0669078ae92ac5"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: SectionModel: ClassTooLong \u2014 416 significant lines (blank, comment-only and punctuation-only lines excluded), 35 methods. The bar is 400 significant lines; this is 16 over it, 1.04\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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a1d81f1b5b85f0c366c30d9ddc309e915f68462ae3e53dcfa12266022e042f4"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: SectionModel.calculateElementFramesIfNecessary: MethodTooLong \u2014 calculateElementFramesIfNecessary runs 104 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 4 over it, 1.04\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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":486}}}],"partialFingerprints":{"codehealthFindingId/v1":"c1d7421a37e22f71b7b8a24728abbefaf8f00e7fbd067dd424ed42d2217f034f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): MagazineLayout/LayoutCore/ModelState.swift:248-257 | MagazineLayout/LayoutCore/ModelState.swift:271-280 \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 \u0060MagazineLayout/LayoutCore/ModelState.swift:248\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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":248}}}],"partialFingerprints":{"codehealthFindingId/v1":"39ab52ba2307e5f3e33600ca173ba2a0c0f49bb9cd61197546f3fa9a32fcd01c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): MagazineLayout/LayoutCore/LayoutState.swift:158-165 | MagazineLayout/LayoutCore/LayoutState.swift:171-178 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"MagazineLayout/LayoutCore/LayoutState.swift"},"region":{"startLine":158}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f6a237958ece8826c6240c6d92a3c3bc5dec9bb21c7231de3225f05f046f024"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): MagazineLayout/LayoutCore/SectionModel.swift:247-254 | MagazineLayout/LayoutCore/SectionModel.swift:262-269 \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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":247}}}],"partialFingerprints":{"codehealthFindingId/v1":"29bf30f239bd81948d629d61065cae1a777320fe38a30cf74928ea469b996aaa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): MagazineLayout/LayoutCore/FooterModel.swift:23-28 | MagazineLayout/LayoutCore/HeaderModel.swift:24-29 \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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060heightMode\u0060 to \u0060MagazineLayoutFooterHeightMode\u0060 in one and \u0060MagazineLayoutHeaderHeightMode\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"MagazineLayout/LayoutCore/FooterModel.swift"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"191f7133e7e62ee02b6ff3d65605fb033133a639dca6dd47bffcbd54ef7c14c7"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (5 members, 50\u002B identical tokens): Example/MagazineLayoutExample/GridDemoViewController.swift:26-54 | Example/MagazineLayoutExample/ListDemoViewController.swift:26-54 | Example/MagazineLayoutExample/MessageThreadDemoViewController.swift:26-55 | Example/MagazineLayoutExample/PerformanceDemoViewController.swift:26-49 | Example/MagazineLayoutExample/RootMenuViewController.swift:26-42 \u2014 These 5 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 5 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 5 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"94016c2f12675bc4528638a9e918032c668112d5c6c09d0d0794e974ddf5cdb9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24 lines \u00D7 2): Example/MagazineLayoutExample/GridDemoViewController.swift:26-49 | Example/MagazineLayoutExample/ListDemoViewController.swift:26-49 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/GridDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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 \u0060Example/MagazineLayoutExample/GridDemoViewController.swift:26\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":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ef291fd940535ba8507296e81cc782f63d15ea704fe6e3be01f51f1c287d40b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 3): Example/MagazineLayoutExample/GridDemoViewController.swift:46-54 | Example/MagazineLayoutExample/ListDemoViewController.swift:46-54 | Example/MagazineLayoutExample/PerformanceDemoViewController.swift:41-49 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/GridDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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 \u0060Example/MagazineLayoutExample/GridDemoViewController.swift:46\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, 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":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"0748d52abdc7ef1682f4e03aac107cf36871ffe0ea4f49338729c62341f37b72"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 5): Example/MagazineLayoutExample/GridDemoViewController.swift:44-51 | Example/MagazineLayoutExample/ListDemoViewController.swift:44-51 | Example/MagazineLayoutExample/MessageThreadDemoViewController.swift:45-52 | Example/MagazineLayoutExample/PerformanceDemoViewController.swift:39-46 | Example/MagazineLayoutExample/RootMenuViewController.swift:32-39 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/GridDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e1c8e4a83b2f43cdb50e7e4d56a5f1915e673ed306e387be9c12608ea09acba"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 5): Example/MagazineLayoutExample/GridDemoViewController.swift:271-278 | Example/MagazineLayoutExample/ListDemoViewController.swift:326-333 | Example/MagazineLayoutExample/MessageThreadDemoViewController.swift:279-286 | Example/MagazineLayoutExample/PerformanceDemoViewController.swift:206-213 | Example/MagazineLayoutExample/RootMenuViewController.swift:165-172 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/GridDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":271}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ef86bdf185da0ac09d20ad4cb067aac7569b000db2950e3784725f293b38680"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 3): Example/MagazineLayoutExample/ListDemoViewController.swift:268-277 | Example/MagazineLayoutExample/MessageThreadDemoViewController.swift:223-232 | Example/MagazineLayoutExample/RootMenuViewController.swift:109-118 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/MessageThreadDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 34 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Example/MagazineLayoutExample/ListDemoViewController.swift"},"region":{"startLine":268}}}],"partialFingerprints":{"codehealthFindingId/v1":"adef46aff4f4bcad5f13a58bdd1436eac8bc212cee2112102dedc25c20023605"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): Example/MagazineLayoutExample/ListDemoViewController.swift:335-342 | Example/MagazineLayoutExample/MessageThreadDemoViewController.swift:288-295 | Example/MagazineLayoutExample/RootMenuViewController.swift:174-181 \u2014 before extracting anything, compare \u0060Example/MagazineLayoutExample/ListDemoViewController.swift\u0060 and \u0060Example/MagazineLayoutExample/MessageThreadDemoViewController.swift\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 34 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Example/MagazineLayoutExample/ListDemoViewController.swift"},"region":{"startLine":335}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7159a116b1e0430e049cd06fad382c1219379c70e07621fd0172f75e83199ea"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: PerformanceDemoViewController (LCOM4 10): PerformanceDemoViewController\u0027s methods fall into 10 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 10 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":"Example/MagazineLayoutExample/PerformanceDemoViewController.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"da713660c916050c9ce120060c75569ec8178333dd06c3c0dd715be6d3546fc2"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: MessageThreadDemoViewController (LCOM4 9): MessageThreadDemoViewController\u0027s methods fall into 9 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 9 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":"Example/MagazineLayoutExample/MessageThreadDemoViewController.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f4f7d0317b2a4de6ae61f1ba3eff7eb71757c336c68a684b7b8c403201634c8"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: RootMenuViewController (LCOM4 9): RootMenuViewController\u0027s methods fall into 9 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 9 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":"Example/MagazineLayoutExample/RootMenuViewController.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"e5a71ff9080821d03b046e4d2346fe33bfb5864acd86e1c8d541d16179f25ce2"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: GridDemoViewController (LCOM4 8): GridDemoViewController\u0027s methods fall into 8 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 8 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":"Example/MagazineLayoutExample/GridDemoViewController.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ea8d46c7881276db3152bcbc3affe8179fc0fc221f7f6c689b29c0842697363"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: ListDemoViewController (LCOM4 7): ListDemoViewController\u0027s methods fall into 7 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 7 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":"Example/MagazineLayoutExample/ListDemoViewController.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"447c00d04a6c22373f0736a66968ace8d5a28a0e25f0dc5be617723f0b0f37e1"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SceneDelegate (LCOM4 6): SceneDelegate\u0027s methods fall into 6 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 6 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":"Example/MagazineLayoutExample/SceneDelegate.swift"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea4c4ee03ec3c1a96a6e3acb6bab443b6007d4ca3eb38933fdc043b73daf32dc"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 Swift suite: Coverage NOT MEASURED: the Swift half could not be measured \u2014 the Swift suite in . produced no coverage export. Coverage is excluded from the score rather than counted as a near-zero. The named suite step is one the repository\u0027s maintainers can perform; once it passes, the real number is measured on the next scan. Alternatively, commit the lcov/Cobertura report your CI produces and it is read without a re-run."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testInitialLayout: 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/ModelStateLayoutTests.swift"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"18040b0221bea63d48f0290c11809e84237b538ee7e70d833b67ef6e3eb50a4b"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testUpdatingSectionMetrics: 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/ModelStateLayoutTests.swift"},"region":{"startLine":203}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b3dffe7e26beaf0980b99adaae41cbcf61865b09e54d0afbe99a205452eec5c"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testLayoutAfterInsertingItems: 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/ModelStateLayoutTests.swift"},"region":{"startLine":315}}}],"partialFingerprints":{"codehealthFindingId/v1":"05b9344a53b4b53a0985ed3b2677ae69372cf5f7958c53b1ad24257e9a6463e0"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testLayoutAfterDeletingItems: 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/ModelStateLayoutTests.swift"},"region":{"startLine":481}}}],"partialFingerprints":{"codehealthFindingId/v1":"c19b400b43dd376e8d94e1d8aa2143bc50d8d6a9ef7193053f6638347868785e"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testLayoutAfterMovingItems: 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/ModelStateLayoutTests.swift"},"region":{"startLine":610}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbe8b4f7cc52fba96eba8fa61fa50a69983c2b9ee47e357ce352ce13fe9520ae"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testReplacingHeader: 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/ModelStateLayoutTests.swift"},"region":{"startLine":754}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2bc134a9e03d515fe7f580c580c19b471b9ddd84fcddcb62b60edba456bdbb7"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testReplacingFooter: 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/ModelStateLayoutTests.swift"},"region":{"startLine":881}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a34e9fbb0e4ba0e413ccd5d6a9f20e3ed9ab03086bf295f0890599419782578"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testReplacingBackground: 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/ModelStateLayoutTests.swift"},"region":{"startLine":1010}}}],"partialFingerprints":{"codehealthFindingId/v1":"5cff6c23c33c4b89d11c5389e19b0816f5ec21f1686193a69c9a09d573a751e6"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testPinnedHeadersAndFooters: 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/ModelStateLayoutTests.swift"},"region":{"startLine":1132}}}],"partialFingerprints":{"codehealthFindingId/v1":"1695537f6704474e274dec0f90d102481c5a57e653716ac280e526b2dcd7466f"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: MagazineLayout/LayoutCore/SectionModel.swift: MagazineLayout/LayoutCore/SectionModel.swift changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 32 in SectionModel.calculateElementFramesIfNecessary at line 486. 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 11:14:40 -07:00\u0027 --until=\u00272026-09-24 11:14:40 -07:00\u0027 --full-history --no-merges -- MagazineLayout/LayoutCore/SectionModel.swift\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":"MagazineLayout/LayoutCore/SectionModel.swift"},"region":{"startLine":486}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c7dcce58531761eec964f6b15d4ebad5f3b519d903c99ee6293f7a0a8583872"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Only invalidate if something changes \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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":390}}}],"partialFingerprints":{"codehealthFindingId/v1":"2c0ff664de8556931cdcd2be7ea615c7499c348081bc027fa780afdcfa123d42"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Only invalidate if something changes \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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":401}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6fa30a80406ed11bac274877e9345c5df495e78a9ebd34dafc116fa1027ef21"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Only invalidate if something changes \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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":421}}}],"partialFingerprints":{"codehealthFindingId/v1":"dc32b24f37a072e34ca7101bd4c68f46d3323223353dcf8fe6a4a77a4403b9cf"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Only invalidate if something changes \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":"MagazineLayout/LayoutCore/ModelState.swift"},"region":{"startLine":441}}}],"partialFingerprints":{"codehealthFindingId/v1":"de36269e927d16f5ece2d9dc248a795cdbf15cc0e72ca5b5e7863bc1a049aa8c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(bryankeller): Look into whether this happens on iOS 10. It definitely does on iOS 9. \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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":1258}}}],"partialFingerprints":{"codehealthFindingId/v1":"341e61593714f0795d50a5b7f8d5c0a1fa27f7b5098e824ff6898779355f21e9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(bryankeller): Look into whether this happens on iOS 10. It definitely does on iOS 9. \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":"MagazineLayout/Public/MagazineLayout.swift"},"region":{"startLine":1296}}}],"partialFingerprints":{"codehealthFindingId/v1":"e8d2f1478a64f059963d47d78e5c8f00eaf85176c96b4c01c9870b9eddd725f3"}},{"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":"b38720db37cd540e3dc09fe3d5c0b3652d2f9d051cf0e85dbbd366b46c6f1a79"},"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":"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":"ace515990e7ee0f17b5c17e44dd168a5bdecf90804a3a3dd845cf05540978013"}},{"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":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add CodeQL\u0027s Swift pack (Swift/Xcode) (or \u0060semgrep --config=auto\u0060, which runs on any language) as a CI step. What was searched, so you can tell an absence from a miss: the 1312 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}},{"ruleId":"P6","level":"note","message":{"text":"No changelog: No CHANGELOG/HISTORY/RELEASES file \u2014 what shipped when isn\u0027t easy to reconstruct for support or audit. (Versioning/tagging makes releases traceable, but a changelog records the what.)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"dda5aa5aed8cbb292c3ef2b733bc138f614293ae6426c85e98bf31ef330d9415"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-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-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":2,"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."]}]}