{"$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":"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":"AX9","name":"CQS / query purity","shortDescription":{"text":"CQS / query purity"},"helpUri":"https://codehealth.canine.dev/dimensions/AX9"},{"id":"AXB1","name":"Runtime evidence locked \u2014 no reproducible boot","shortDescription":{"text":"Runtime evidence locked \u2014 no reproducible boot"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB1"},{"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":"X10","name":"Duplicated predicate","shortDescription":{"text":"Duplicated predicate"},"helpUri":"https://codehealth.canine.dev/dimensions/X10"},{"id":"X32","name":"Type resolved by simple name across every loaded assembly","shortDescription":{"text":"Type resolved by simple name across every loaded assembly"},"helpUri":"https://codehealth.canine.dev/dimensions/X32"},{"id":"X6","name":"Hand-rolled structured-format parsing","shortDescription":{"text":"Hand-rolled structured-format parsing"},"helpUri":"https://codehealth.canine.dev/dimensions/X6"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D2","level":"warning","message":{"text":"Networking.handle (cognitive 30): Networking.handle has cognitive complexity 30 (threshold 15). Drivers by points: if/else 9 (23 pts), boolean chains 6, error handling 1 (nesting depth added 14). 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. This file is where this pass\u0027s cognitive complexity CONCENTRATES: Sources/Networking/Networking.swift holds 4 of the 7 methods over the threshold \u2014 including the worst \u2014 and 26 of the 32 points over it (81%), 8.7\u00D7 the next-largest file (Sources/Networking/CacheStore.swift at 3). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking.swift"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4423c28fa82893e8842f240824c07d6373b1dee604f13198e11222ad6d79b2e"}},{"ruleId":"D2","level":"warning","message":{"text":"Networking.logCompletion (cognitive 19): Networking.logCompletion has cognitive complexity 19 (threshold 15). Drivers by points: if/else 6 (8 pts), ternaries 4 (7 pts), loops 2 (3 pts), match/switch 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. This file is where this pass\u0027s cognitive complexity CONCENTRATES: Sources/Networking/Networking.swift holds 4 of the 7 methods over the threshold \u2014 including the worst \u2014 and 26 of the 32 points over it (81%), 8.7\u00D7 the next-largest file (Sources/Networking/CacheStore.swift at 3). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking.swift"},"region":{"startLine":250}}}],"partialFingerprints":{"codehealthFindingId/v1":"64db9add3a843c2bbc991613866bc4a94aec604960630fc1ef3f881bf2668b43"}},{"ruleId":"D2","level":"warning","message":{"text":"Networking.handleImageRequest (cognitive 19): Networking.handleImageRequest has cognitive complexity 19 (threshold 15). Drivers by points: if/else 10 (17 pts), boolean chains 1, error handling 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. This file is where this pass\u0027s cognitive complexity CONCENTRATES: Sources/Networking/Networking.swift holds 4 of the 7 methods over the threshold \u2014 including the worst \u2014 and 26 of the 32 points over it (81%), 8.7\u00D7 the next-largest file (Sources/Networking/CacheStore.swift at 3). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking.swift"},"region":{"startLine":200}}}],"partialFingerprints":{"codehealthFindingId/v1":"bd09e2aa7ad37923093fdb0eda8171ddb3b629a668436b48bc3f9dbd31036227"}},{"ruleId":"D2","level":"warning","message":{"text":"Networking.handleDataRequest (cognitive 18): Networking.handleDataRequest has cognitive complexity 18 (threshold 15). Drivers by points: if/else 9 (16 pts), boolean chains 1, error handling 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. This file is where this pass\u0027s cognitive complexity CONCENTRATES: Sources/Networking/Networking.swift holds 4 of the 7 methods over the threshold \u2014 including the worst \u2014 and 26 of the 32 points over it (81%), 8.7\u00D7 the next-largest file (Sources/Networking/CacheStore.swift at 3). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking.swift"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"c1a259ac9987fb54d3e23bad2cc9f83e89aa6f82a34c024a5e2c63297dd10d89"}},{"ruleId":"D2","level":"warning","message":{"text":"DecodingError.detailedMessage (cognitive 18): DecodingError.detailedMessage has cognitive complexity 18 (threshold 15). Drivers by points: if/else 9 (17 pts), match/switch 1 (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":"Sources/Networking/NetworkingError.swift"},"region":{"startLine":170}}}],"partialFingerprints":{"codehealthFindingId/v1":"369f481e2521940bd06bcd434961978dc5232ba8521c29f471839a42bf8517d1"}},{"ruleId":"D2","level":"warning","message":{"text":"CacheStore.object (cognitive 17): CacheStore.object has cognitive complexity 17 (threshold 15). Drivers by points: if/else 6 (13 pts), ternaries 1 (3 pts), match/switch 1 (nesting depth added 9). 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":"Sources/Networking/CacheStore.swift"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"8246f31b292c50f12889c9950268c3bc158f730444598c43e39610e1f78e5c26"}},{"ruleId":"D2","level":"warning","message":{"text":"CacheStore.sweepExpired (cognitive 16): CacheStore.sweepExpired has cognitive complexity 16 (threshold 15). Drivers by points: if/else 6 (12 pts), loops 2 (4 pts) (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":"Sources/Networking/CacheStore.swift"},"region":{"startLine":231}}}],"partialFingerprints":{"codehealthFindingId/v1":"9231490d2433ec5d6213019d2187f8662a0edcc0e98761c81754a87bdf9eb539"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (34 lines \u00D7 2): Sources/Networking/Networking\u002BPrivate.swift:103-136 | Sources/Networking/Networking\u002BPrivate.swift:201-234 \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 \u0060Sources/Networking/Networking\u002BPrivate.swift:103\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking\u002BPrivate.swift"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"9daef09e95bf96511a14e0cb691181501f27e03d2ecb1d2ddde6c0d94d9b2194"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 4): Sources/Networking/NetworkingError.swift:176-182 | Sources/Networking/NetworkingError.swift:185-191 | Sources/Networking/NetworkingError.swift:194-200 | Sources/Networking/NetworkingError.swift:203-209 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/NetworkingError.swift"},"region":{"startLine":176}}}],"partialFingerprints":{"codehealthFindingId/v1":"6bef2d1647e19db234eed7ccde12fd81545ae54a93b540f9fa26ca8017d1b986"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: Networking (LCOM4 4): Networking\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Sources/Networking/Networking.swift"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff41932322464bd6daeabfbda7d51db25fc1f6170c5d71caa06d2742962516c5"}},{"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":"Skipped test: testGETCacheKeyDistinguishesHost: Skipped \u2014 throws XCTSkip"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Tests/NetworkingTests/GETIntegrationTests.swift"},"region":{"startLine":210}}}],"partialFingerprints":{"codehealthFindingId/v1":"270e871396b1509f46095e77c9a09ce5391ad1a2df74534f98252129723e8e71"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelGET: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d3a21f8a84e07c2a36c44186bc8628a506dfadaa492861fda8d531af0a0880f"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelPOST: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"52bf04da7a21a96c88ee5341fb0add16c6c4b439d70dffb38e7e276e6d699a27"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelPUT: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"9352ad6cc24f74f3cae68b5a1ec537ae1d3b1f930ecf095554b41dc7bffdcb90"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelPATCH: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"02915d4c16005d6270f1bd465696a698dbc4789b8dd55751d92199e7eb254604"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelDELETE: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"03a8436cf8f636dd86b0fafa9275836bd5dd04a97214dc32e5cce87006b4a03f"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelImageDownload: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"290c8454f9662eaf5f9140060b6c015728c836ee5a7d82794dc00caac8632609"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testCancelAllRequests: 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/NetworkingTests/CancellationIntegrationTests.swift"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"f89fbb53f7d046f88c7b85161f1d67a482d15682e135994ef60496a4d6a39fe3"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testNewGET: 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/NetworkingTests/NewNetworkingIntegrationTests.swift"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"0020bbcd7e9b041a2120612b23c73adca21aa71e91d8c2954fd3b0ea8b828152"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testNewGETWithParams: 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/NetworkingTests/NewNetworkingIntegrationTests.swift"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"3abb4d0074f4d188ca0c72dc3dae42a79e9c1488d9b715727bb7c7db39b68e3f"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testNewPOST: 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/NetworkingTests/NewNetworkingIntegrationTests.swift"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"36400ae9b7599f13001bea488cbdd0c1d12ea3c35bc61b57155427d0a481c070"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: testNetworkingJSON: 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/NetworkingTests/NewNetworkingIntegrationTests.swift"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"0591666595819908e1cdde0ef9e85ae273c1ba15a3731726883bf95de5d70016"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: Sources/Networking/Networking.swift: Sources/Networking/Networking.swift changed 5 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 15 in Networking.handle at line 24. 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-19..2026-09-17, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-19 21:34:09 \u002B02:00\u0027 --until=\u00272026-09-17 21:34:09 \u002B02:00\u0027 --full-history --no-merges -- Sources/Networking/Networking.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":"Sources/Networking/Networking.swift"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"3e359d7f6a3d000c443f5ecfdbd3987525086c0df83a626301bbed04897a6d55"}},{"ruleId":"D22","level":"warning","message":{"text":"Inconsistent authorization configuration API. The first two methods imply specific auth schemes (Basic and Bearer/Token), while the third is a generic header setter. This forces users to know the internal implementation details of how \u0027token\u0027 auth is handled versus generic headers, and creates ambiguity about precedence or behavior.: Unify into a single method \u0060setAuthorizationHeader(value: String)\u0060 or \u0060setAuthorizationScheme(scheme: String, credentials: String)\u0060 to handle all cases uniformly, or remove the specific token method in favor of the generic one. (signatures: Networking.setAuthorizationHeader(username: String, password: String) | Networking.setAuthorizationHeader(token: String) | Networking.setAuthorizationHeader(headerKey: String, headerValue: String))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"79ea43c120c400e17fad8b4f83cfea977326e684a952280509f8afad3a7fa25f"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent with different generic constraints. One overload takes a concrete array of URLQueryItems, the other takes a generic type Q. Without seeing the implementation of Q, it is unclear if this is a convenience for encoding structs vs manual query building, or if it\u0027s redundant. If Q is just a wrapper or protocol implemented by [URLQueryItem], it\u0027s confusing. If Q is a distinct encoding mechanism, the naming doesn\u0027t reflect that distinction clearly.: If Q is a generic encoder, name the method to reflect encoding (e.g., \u0060getEncoded\u0060). If it\u0027s just a convenience, ensure the distinction is obvious or remove one if it adds no value. (signatures: Networking.get(String, query: [URLQueryItem]?, cachingLevel: CachingLevel): Result\u003CT, NetworkingError\u003E | Networking.get(String, query: Q, cachingLevel: CachingLevel): Result\u003CT, NetworkingError\u003E)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"11229b0ca34fec59e90d3cc24fa67f2302659a1089b52faad32603a6e00a0401"}},{"ruleId":"D22","level":"warning","message":{"text":"Inconsistent return types for similar operations. \u0060downloadImage\u0060 returns \u0060Result\u003CT, NetworkingError\u003E\u0060 (where T is likely Image based on context, but the signature is generic T), while \u0060downloadData\u0060 also returns \u0060Result\u003CT, NetworkingError\u003E\u0060. However, looking at \u0060ImageResponse\u0060 and \u0060DataResponse\u0060, there are specific response types. The generic \u0060T\u0060 in the download methods is ambiguous. Does \u0060downloadImage\u0060 return \u0060Result\u003CImage, ...\u003E\u0060? If so, why is it generic? If \u0060downloadData\u0060 returns \u0060Result\u003CData, ...\u003E\u0060, why is it generic? This suggests a lack of specific typing for these domain-specific downloads.: Make \u0060downloadImage\u0060 return \u0060Result\u003CImage, NetworkingError\u003E\u0060 and \u0060downloadData\u0060 return \u0060Result\u003CData, NetworkingError\u003E\u0060 explicitly, removing the generic T to clarify intent and prevent type inference errors. (signatures: Networking.downloadImage(String, cacheName: String?, cachingLevel: CachingLevel): Result\u003CT, NetworkingError\u003E | Networking.downloadData(String, cacheName: String?, cachingLevel: CachingLevel): Result\u003CT, NetworkingError\u003E)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"742c12d656a62241cc2ad94de1298cb594f612dd6028984ea5a6974133513115"}},{"ruleId":"D22","level":"warning","message":{"text":"Inconsistent fake/mock API design. The fake methods have three distinct overloads for different response sources (Encodable object, raw headers/status, file path). This is inconsistent with the real API which uses a unified \u0060post/get\u0060 with body/query parameters. The fake API forces users to choose a response format at the call site in a way that doesn\u0027t mirror the production API\u0027s structure, making mocking harder to reason about.: Unify the fake API to accept a \u0060Response\u0060 object or a closure that returns a response, similar to how real requests are structured, or at least ensure the fake methods mirror the real method signatures more closely (e.g., \u0060fakeGET(path, body: Encodable)\u0060). (signatures: Networking.fakeGET(String, response: some Encodable, ...) | Networking.fakeGET(String, headerFields: ..., ...) | Networking.fakeGET(String, fileName: String, ...))"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"2e762dfa8f8ff779cbd46ca288d7ff6637754ad048564d221469e9dc1bd3fb1b"}},{"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":"65681534483db00fb92867d3a27acce61bd10291a78aa09b2cff348c81dba4df"},"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":"warning","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":"9658270ba49f37ed04e99ab1263b6393cd42aed8bdfb7aafd9fa1070f5491895"}},{"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":"M4","level":"note","message":{"text":"README/code drift: README advertises Docker containerisation, but no Dockerfile/compose file exists \u2014 searched for: \u0060dockerfile\u0060, \u0060docker-compose\u0060, \u0060compose.yaml\u0060, \u0060compose.yml\u0060. Each was matched case- and separator-insensitively against file and directory NAMES anywhere in the tree, and against the CONTENTS of manifest files (package.json, *.csproj, *.props, *.slnx, *.yml, Dockerfile); the README\u0027s own prose never counts, so a claim is never refuted by merely being made. Nothing outside that search was read \u2014 a footprint living only in a submodule, in a file type not listed here, or under a name none of those terms matches is not seen, and this row is then wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"343bbbd51ceb8de7a9ff88d79f7c797abf9b5921f8727b67b4d0f4f8a22c5e49"}},{"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 3844 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":3,"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."]}]}