{"$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":"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":"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":"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":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX5","name":"Architecture \u0026 structure","shortDescription":{"text":"Architecture \u0026 structure"},"helpUri":"https://codehealth.canine.dev/dimensions/AX5"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"DM1","name":"Aggregate boundaries","shortDescription":{"text":"Aggregate boundaries"},"helpUri":"https://codehealth.canine.dev/dimensions/DM1"},{"id":"DM11","name":"Constructible invalid state","shortDescription":{"text":"Constructible invalid state"},"helpUri":"https://codehealth.canine.dev/dimensions/DM11"},{"id":"DM12","name":"Ambient inputs in the domain","shortDescription":{"text":"Ambient inputs in the domain"},"helpUri":"https://codehealth.canine.dev/dimensions/DM12"},{"id":"DM4","name":"Rich vs anemic model","shortDescription":{"text":"Rich vs anemic model"},"helpUri":"https://codehealth.canine.dev/dimensions/DM4"},{"id":"DM5","name":"Encapsulated state","shortDescription":{"text":"Encapsulated state"},"helpUri":"https://codehealth.canine.dev/dimensions/DM5"},{"id":"DM6","name":"Domain \u2194 infrastructure boundary","shortDescription":{"text":"Domain \u2194 infrastructure boundary"},"helpUri":"https://codehealth.canine.dev/dimensions/DM6"},{"id":"DM7","name":"Repository granularity","shortDescription":{"text":"Repository granularity"},"helpUri":"https://codehealth.canine.dev/dimensions/DM7"},{"id":"DM8","name":"Value-object opportunities","shortDescription":{"text":"Value-object opportunities"},"helpUri":"https://codehealth.canine.dev/dimensions/DM8"},{"id":"DM9","name":"Scattered domain decisions","shortDescription":{"text":"Scattered domain decisions"},"helpUri":"https://codehealth.canine.dev/dimensions/DM9"},{"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":"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":"P4","name":"Deployment \u0026 Rollback","shortDescription":{"text":"Deployment \u0026 Rollback"},"helpUri":"https://codehealth.canine.dev/dimensions/P4"},{"id":"P5","name":"DR \u0026 Backup","shortDescription":{"text":"DR \u0026 Backup"},"helpUri":"https://codehealth.canine.dev/dimensions/P5"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java:24-35 | core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/ESBankAccount.java:25-36 \u2014 before extracting anything, compare \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java\u0060 and \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/ESBankAccount.java\u0060 as WHOLE FILES: 84% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 2 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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 \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java:24\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":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"77823cb733cd703f381a81010a6cd157f06e8a6f2954dafd9a46f1d034caca42"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): adapter-persistence-jpa/src/main/java/com/optivem/kata/banking/adapter/driven/persistence/jpa/JpaBankAccountStorage.java:56-63 | adapter-persistence-jpa/src/main/java/com/optivem/kata/banking/adapter/driven/persistence/jpa/JpaBankAccountStorage.java:68-75 \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 \u0060adapter-persistence-jpa/src/main/java/com/optivem/kata/banking/adapter/driven/persistence/jpa/JpaBankAccountStorage.java:56\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. The \u0060return\u0060 at the foot of the matched lines is the enclosing body\u0027s own terminal exit, not an early one: it moves with them unchanged, and each site calls the extracted unit from the position that \u0060return\u0060 occupied \u2014 no decision has to be handed back and re-acted on."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-jpa/src/main/java/com/optivem/kata/banking/adapter/driven/persistence/jpa/JpaBankAccountStorage.java"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"24b6a8d5d66f3ed1f732e159a75f82a0c44a2e7afefadb933efd98b07c44fa06"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/common/guards/LongGuard.java:4-15 | core/src/main/java/com/optivem/kata/banking/core/internal/crud/common/guards/LongGuard.java:4-15 \u2014 before extracting anything, compare \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/common/guards/LongGuard.java\u0060 and \u0060core/src/main/java/com/optivem/kata/banking/core/internal/crud/common/guards/LongGuard.java\u0060 as WHOLE FILES: 88% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. 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":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/common/guards/LongGuard.java"},"region":{"startLine":4}}}],"partialFingerprints":{"codehealthFindingId/v1":"df3603b2110da1c681634c66936195ed1279928187b7c3ff42bbc7104eb8aa68"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java:65-71 | core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/ESBankAccount.java:66-72 \u2014 before extracting anything, compare \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java\u0060 and \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/ESBankAccount.java\u0060 as WHOLE FILES: 84% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 2 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a0cd699f131de5b4b8bb77ea6676d68a11bd9022f89225d80f682b263ce19f1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/acl/FundsDepositedConverter.java:7-13 | core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/acl/FundsWithdrawnConverter.java:7-13 \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. 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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060event\u0060 to \u0060FundsDeposited\u0060 in one and \u0060FundsWithdrawn\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":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/acl/FundsDepositedConverter.java"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8b4ea19c3ea7394624754ab99c0e8ab39c8c421b203a50ee2a1494226f4063d"}},{"ruleId":"D8","level":"warning","message":{"text":"Coverage not measured \u2014 no coverage collector is wired up: Coverage NOT MEASURED: \u0060--collect:\u0022XPlat Code Coverage\u0022\u0060 names a data collector that ships in the \u0060coverlet.collector\u0060 package, and this repository wires up none \u2014 no test project references it and no runsettings declares one. The absence of coverage here is therefore not evidence about the suite or about our analyzer environment: without a collector, \u0060--collect\u0060 produces nothing even from a suite that builds and passes. Add a \u0060coverlet.collector\u0060 PackageReference to the test project(s) (or commit the Cobertura/OpenCover/lcov report your CI produces) and real coverage will be measured. It is excluded from the score rather than counted as a near-zero defect."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"62e63e619c28f057e129f2997c965d32a457366a9ce18ca019ffb6bdfba85c99"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_find_multiple_added_bank_accounts: 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":"adapter-base/src/test/java/com/optivem/kata/banking/adapter/driven/base/BankAccountStorageTest.java"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"b315bc98f5e89caf7c8e152624986c6d13193a0bd3432cdd1663a3ea1d9a7779"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_added_bank_account: 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":"adapter-base/src/test/java/com/optivem/kata/banking/adapter/driven/base/BankAccountStorageTest.java"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"b9ffbc0c3da1cf0fe37bdd4bfbdc542397c592d30db4548329c7ac283126fd78"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_execute_without_error: 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":"adapter-base/src/test/java/com/optivem/kata/banking/adapter/driven/base/EventBusTest.java"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a0e66079bb1847b6b83d9d88dcf4223759f404abacf95d056539bea5fe43cef"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_next_element_when_there_is_one_element: No conventional assertion call was detected, and 4 of 6 tests in \u0060adapter-generation-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-generation-fake/src/test/java/com/optivem/kata/banking/adapter/driven/generation/fake/internal/FakeLongGeneratorTest.java"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"92755cfa9d7ecd44caf6172c1b2fcb7d86cad861939360b0404a6e00f8ae3b8a"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_next_elements_when_there_are_multiple_elements: No conventional assertion call was detected, and 4 of 6 tests in \u0060adapter-generation-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-generation-fake/src/test/java/com/optivem/kata/banking/adapter/driven/generation/fake/internal/FakeLongGeneratorTest.java"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"437625f62a544f4935edfc199cda3d720cfaa151cd376868398172f44850b74d"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_next_element_when_there_is_one_element: No conventional assertion call was detected, and 4 of 6 tests in \u0060adapter-generation-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-generation-fake/src/test/java/com/optivem/kata/banking/adapter/driven/generation/fake/internal/FakeStringGeneratorTest.java"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"e0c6f74a5cfa93414aebf64351277dfa598c6bef9391c140c02854178ca1df73"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_next_elements_when_there_are_multiple_elements: No conventional assertion call was detected, and 4 of 6 tests in \u0060adapter-generation-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-generation-fake/src/test/java/com/optivem/kata/banking/adapter/driven/generation/fake/internal/FakeStringGeneratorTest.java"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5ddcb19b1a1ba4189db0def0f64c6a53701034b2cb1a4c1300f86528433b43e"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_false_when_customer_not_exists: 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":"adapter-microservice-real/src/test/java/com/optivem/kata/banking/adapter/driven/microservice/real/RealCustomerGatewayConsumerTest.java"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"7984a330b418de11f1139d3d311d31846b3630eb7321c969ebd46a9cefe49554"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_true_when_user_is_blacklisted: 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":"adapter-microservice-real/src/test/java/com/optivem/kata/banking/adapter/driven/microservice/real/RealCustomerGatewayConsumerTest.java"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d58d7a3265956fb738223a0a358bd0612bfa86201ee6593141769221a118354"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_true_when_user_is_whitelisted: 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":"adapter-microservice-real/src/test/java/com/optivem/kata/banking/adapter/driven/microservice/real/RealCustomerGatewayConsumerTest.java"},"region":{"startLine":108}}}],"partialFingerprints":{"codehealthFindingId/v1":"467c733535f9766b123d39850856440307e81b4ed09c1b58064f234b3e459bad"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_not_be_able_to_change_bank_account_after_add: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"7380655fb531784539305f59a37acdef58e40317b429f7009142b034ab24fa37"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_not_be_able_to_change_bank_account_after_find: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"b2008e49ace872e840a482b971b6441f9f3f471e27cf5f61b3c1d2e8b4927855"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_not_be_able_to_change_bank_account_after_update: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a5aa7620ae172fcc6b808629a7c16d094bdd1f8c830a62f7831054a2dba8066"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_retrieve_updated_bank_account_after_update: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"e446669add5ba28f706f6bd73f58bfa4022d27cdb3aa4e02739bf40dd09887ad"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_bank_account_when_account_number_exists: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"8815bc02dad2e4cad80ab5b4297fb3470b6fef93d5d7b431d50e360d909f3457"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_empty_result_when_account_number_does_not_exist: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ee1aedd43890da04723077f44f7b844df527823d286129c40b75813ce3c69ab"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_when_attempt_add_bank_account_with_same_account_number_twice: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":133}}}],"partialFingerprints":{"codehealthFindingId/v1":"a76c751439b4c3c16d58f2c8e936c07dac79243dae9243dd2da8b01880492882"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_when_attempt_to_update_non_existent_bank_account: No conventional assertion call was detected, and 8 of 8 tests in \u0060adapter-persistence-fake\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"a106e5064b517b95a905432691b4015c7b1aa4c07af4124e4472e2e36831ede8"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_false_when_user_not_exists: 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":"adapter-thirdparty-real/src/test/java/com/optivem/kata/banking/adapter/driven/thirdparty/real/RealNationalIdentityGatewayConsumerTest.java"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"66ff795a9f50ce939b0ac82ce59cd5646cde6817c2cc1469de79ab5914010134"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_true_when_user_exists: 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":"adapter-thirdparty-real/src/test/java/com/optivem/kata/banking/adapter/driven/thirdparty/real/RealNationalIdentityGatewayConsumerTest.java"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"70d73ccc331fb641edff6ee43e9b0274de74844b5fc97552deb6a482acd138d1"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: pactVerificationTestTemplate: 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":"z-thirdparty-provider-contracts/src/test/java/com/optivem/kata/banking/adapter/driven/thirdparty/real/RealNationalIdentityGatewayProviderTest.java"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"56477489741bc8d57cff0661dcc15df8f88fd4133aed0fd03d770e2ee6defc81"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_return_current_date_time: 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":"adapter-time-fake/src/test/java/com/optivem/kata/banking/adapter/driven/time/fake/FakeDateTimeServiceTest.java"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"7656b0e3614f273f2ad94544d58af6d0ae7247e621dc1928a91999802dce63d0"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: pactVerificationTestTemplate: 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":"startup/src/test/java/com/optivem/kata/banking/adapter/restapi/spring/BankingProviderContractTest.java"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"e1f675e5881954cb975969c6c1d7706ac2a56ecf1af3c3e84d88b0844317efdb"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_positive_value: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/AccountIdTest.java"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"68f3b58320929745da72356107d33d72b86825e749b638b3b8ab522e446e54fe"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_account_holder_name: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4ffa32579af0dc8e17247696d027c87b94f566e32ef95da525f32fad1ba297d"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_account_id: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d221ed99e32b4aafecb0fc044dacac12538d27724ee9c99421c2d0eeac0ab22"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"817ffaf8932471c74e9fb5ad5791960f6f3ed43d66eca431d68e7735b93c427c"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_balance: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"78ff4e129122a603a5ece704140538f6688f34d6d5685eb4ea811e95e25a7b6a"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_national_identity_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2ac2ad54c5e4cb56d802be9e95f270af3b073786043192ea98d01a1ea0132f7"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_null_opening_date: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/BankAccountTest.java"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"3881c0c0079df25ef9eab38b294a035099e40b2668e7e7609fca1db07d51667e"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_score_A_given_positive_odd_aggregation: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/ScoreCalculatorTest.java"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"5a394bedcdc1bdbf222d64d72d2ab2a58cece7e7248c992e746228e202e77b99"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_score_B_given_positive_even_aggregation: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/ScoreCalculatorTest.java"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb264a689ae668429c8404a93239d8cb7f4d23f4d028d017790f28a51098186b"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_return_score_C_given_non_positive_aggregation: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/domain/ScoreCalculatorTest.java"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e426c5e5da31648d5d01e5d7ab39183b5320b04efb208375817a3a8feb10353"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_deposit_funds_given_valid_request: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/DepositFundsUseCaseTest.java"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ae90e32e8f3ab56a822a2d8ea8ff529a10f2c071eb635120aa466271dacfd34"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/DepositFundsUseCaseTest.java"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"38de2ae225b0dd6626397ed33665ab8a34b57b6ae5ca6100caba3045b7afced8"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_existent_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/DepositFundsUseCaseTest.java"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"6238976e746dc3e220f58341bdcf3c919f98065976cfb53ca8a99bbb142d2b6f"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_positive_amount: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/DepositFundsUseCaseTest.java"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"467e68aebce4b55969276e84121876f68a5d1e2ae47efde2ca1889088ed8e1c0"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_open_account_given_valid_request: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"952177dd19df541367ba56f888ce49ad157db15b96f5d402eb8911a6686a4b94"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_blacklisted_national_identity_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"ecc3e9ac623d734d21a0216e59a19b035cae21414f656b4095fc828e6eeeb515"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_first_name: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":152}}}],"partialFingerprints":{"codehealthFindingId/v1":"998ca566b5e6c01385c41db324d8815060aaf7a759de0f1ec50d1b4f39a14c68"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_last_name: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":162}}}],"partialFingerprints":{"codehealthFindingId/v1":"88978383143942c0b664fe1a699de250828d540063a162b8536de13e84e4bff0"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_national_identity_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":110}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc782b5a373aeb9afa0e35214e7314d28c009d17c3cd16fdeda37eec5b63e7d6"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_negative_balance: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":172}}}],"partialFingerprints":{"codehealthFindingId/v1":"9acf2f5bdeb465b7bf8312e4925bba3a8a3a0a080e0df650ee55e8eb084c7cff"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_nonexistent_national_identity_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"8aead2f8b87b8199d6750841bd53bed02407a6b771a6112c4fef17de8a20a744"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/ViewAccountUseCaseTest.java"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"73efac29d3b49ad6daa7af6f771be924378699e0840407b6a47f3631d7ada0fa"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_existent_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/ViewAccountUseCaseTest.java"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f3f1b0c26474cdafe29cfffadb6cc73b88180c0edc898602e238e3d9ec0bbb6"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_view_account_given_valid_request: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/ViewAccountUseCaseTest.java"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"caea398a5bb16303a14d14a0a8babcd887d36e2565b736388e932b5bdf8ce6a5"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_empty_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/WithdrawFundsUseCaseTest.java"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"bacb0a6b7860dfe4c7922e5b7c635d10b126bef60c135d11baad6bb9bf216784"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_insufficient_funds: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/WithdrawFundsUseCaseTest.java"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"5a053abf13a6eac0de870a50250eadbd90afc5b7a36768939ae7dffe278eda36"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_existent_account_number: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/WithdrawFundsUseCaseTest.java"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"82aef28a26821c0e2e96e6e078ba807a132fdc9e8d0c4c84e82ed39026920954"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_throw_exception_given_non_positive_amount: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/WithdrawFundsUseCaseTest.java"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f4802fe50c20dac05c88f00e0e5e4aecf0851d25c216b172ba0d00d9672ee37"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: should_withdraw_funds_given_valid_request: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/usecases/WithdrawFundsUseCaseTest.java"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"fe09e43b83cf0a729d404b771721634724794e9482594ed462c2661d555fb3a4"}},{"ruleId":"D10","level":"note","message":{"text":"No direct assertions: testConstructor: No conventional assertion call was detected, and 30 of 37 tests in \u0060core\u0060 read the same way \u2014 so this is treated as that project\u0027s convention rather than a broken test, and it does not drag the score. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a project of runnable samples compiled as tests, where a run that does not throw is the only check. If it is the latter, these methods genuinely verify nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/test/java/com/optivem/kata/banking/core/util/UtilityClassTest.java"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"9d713336bacd17d310a74e6a66b82e13a9496a0b51391d9692ddebc3dc1d740b"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: contextLoads: 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":"startup/src/test/java/com/optivem/kata/banking/startup/BankingApplicationTests.java"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2147b9d25e38360e1203e99a0487128c61a90fa89889eb8b5beb469784a4634"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_consume_message_from_queue: 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":"adapter-messaging-rabbitmq/src/test/java/com/optivem/kata/banking/RabbitMQEventBusTest.java"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"916849d3b556c8f5ec6d5a6309cf9376ba0148b7ff510a5de86566e97c94016a"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: should_send_dto_to_default_queue: 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":"adapter-messaging-rabbitmq/src/test/java/com/optivem/kata/banking/RabbitMQEventBusTest.java"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"c47ebef3da0b0b6d4cb7415c9ab46f454dc933991a06e5b074fd0a7a3e2b96fe"}},{"ruleId":"D10","level":"warning","message":{"text":"No assertions: pactVerificationTestTemplate: 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":"z-customer-microservice/src/test/java/RealCustomerGatewayProviderTest.java"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ba2524f1b0cc7538dc7b37ff7e30d9dda1afc28a880829cdd920ac07ead4f41"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: adapter-messaging-rabbitmq: No conventional assertion call was detected in 2 of 2 tests in \u0060adapter-messaging-rabbitmq\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-messaging-rabbitmq/src/test/java/com/optivem/kata/banking/RabbitMQEventBusTest.java"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf9da09d2111960ab7ee605a7f0d0a3c080ab30ffd020bf411957139ed76e84b"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: adapter-microservice-real: No conventional assertion call was detected in 3 of 3 tests in \u0060adapter-microservice-real\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-microservice-real/src/test/java/com/optivem/kata/banking/adapter/driven/microservice/real/RealCustomerGatewayConsumerTest.java"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc3ce14c1b4fa9a8952b4d4a7f6189f6289291cb1bc1adc7cd34b2bebfdb2535"}},{"ruleId":"D10","level":"note","message":{"text":"Test project verifies nothing: adapter-persistence-fake: No conventional assertion call was detected in 8 of 8 tests in \u0060adapter-persistence-fake\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"556faedd25c7f41cfe8c71612e06ba274feef4488673974e2517c5e355a2f65c"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: adapter-thirdparty-real: No conventional assertion call was detected in 2 of 2 tests in \u0060adapter-thirdparty-real\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-thirdparty-real/src/test/java/com/optivem/kata/banking/adapter/driven/thirdparty/real/RealNationalIdentityGatewayConsumerTest.java"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"4c60157a34378bb882a6291cc4c5f3ecc55821382da4aa3334dbbf5263e29caf"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: adapter-time-fake: No conventional assertion call was detected in 1 of 1 tests in \u0060adapter-time-fake\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-time-fake/src/test/java/com/optivem/kata/banking/adapter/driven/time/fake/FakeDateTimeServiceTest.java"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2cebfeb2a4601b0ff69c2ec082191df91cb4dbad61090b7335131b7ae0bedde"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: z-customer-microservice: No conventional assertion call was detected in 1 of 1 tests in \u0060z-customer-microservice\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"z-customer-microservice/src/test/java/RealCustomerGatewayProviderTest.java"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ddb8d179f71ce89a459719ef57880ce55761e8415a87e7ad41e482dc96ef6f3"}},{"ruleId":"D10","level":"warning","message":{"text":"Test project verifies nothing: z-thirdparty-provider-contracts: No conventional assertion call was detected in 1 of 1 tests in \u0060z-thirdparty-provider-contracts\u0060 \u2014 the project as a whole, not one method. Two things look like this: verification that happens indirectly (an approval/verifier harness or BDD step methods), or a suite that genuinely checks nothing. If it is the latter, this project passes unconditionally and cannot fail."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"z-thirdparty-provider-contracts/src/test/java/com/optivem/kata/banking/adapter/driven/thirdparty/real/RealNationalIdentityGatewayProviderTest.java"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"b780776b999213811d1ca88fbf90d2910518c805d5f0ee35c19fc3bb05936681"}},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ae7cffd7f9420c42dbed9168fa312a13fc7cf494b48c1acd1954bd98847e6067"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"383be03e3c3b678f94063c4bb7b7575f83e6eba779817d29ed5752ba06260d0b"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"3d994f029914761828cee9318b2a09883aab5043b9f88033f836c371a5bbe595"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4f421d138d3e02f1ec96458f7ae79a652a9336c9ebb32101e9a902a9d1961d7a"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ad6e6cef5c98beb838c555db94d534a26e33e86735bf43dcc38bfcd46dd5c3bc"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"728bf081badd70efd2121429b17be6dc78f908b27f11058c10f4280912b78293"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"041b3369677038e212e27f59300aab5b8617364a916692a275c3a19b8cdd4d39"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //todo - do some assert \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":"adapter-messaging-rabbitmq/src/test/java/com/optivem/kata/banking/adminrabbitmq/AdminAMQPTest.java"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"f96abfaa2ef1d67bea09d75e13c41999f089d06e69b026b5b44f0ad08874b263"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //todo - do some assert \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":"adapter-messaging-rabbitmq/src/test/java/com/optivem/kata/banking/adminrabbitmq/AdminAMQPTest.java"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca71cc8d8b70c5b6c72d08cca403f6a54164c24f085aaf93c387b3816a975ab2"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Incorporate this into main test later \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":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"61d8803bfd4ed09d441da733e99de92d2d417e431bcd3485386f2a8b46aac007"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Bring back \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":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":150}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7e87979a9530f40f379494b5cd5e28b1e034db6e444f154310a92a418ccf926"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Bring back \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":"adapter-persistence-fake/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/fake/FakeBankAccountStorageExtendedTest.java"},"region":{"startLine":162}}}],"partialFingerprints":{"codehealthFindingId/v1":"177750bd5f048f3f0c51d056a8fa8a5c657f2b4182229fcf83655cb1f27017c8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // @ContextConfiguration // TODO: VC: Probably ring this back and delete ActiveProfiles; ActiveProfiles was temporarily added \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":"adapter-persistence-redis/src/test/java/com/optivem/kata/banking/adapter/driven/persistence/redis/RedisBankAccountStorageTest.java"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"7eb09558ffa429165718b7329a201dc2f8eccc8e3941f89d3a52b774de7943bc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: This should be in a separate module, it\u0027s some consumer, e.g. loan \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":"adapter-restapi-spring/src/main/java/com/optivem/kata/banking/adapter/driver/restapi/spring/clients/BankingClient.java"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff2d70a2d69c0353a1d469f9681f169f9ca94372df408ce1ef06f99c367fba97"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Refactor to centralized error handling \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":"adapter-restapi-spring/src/main/java/com/optivem/kata/banking/adapter/driver/restapi/spring/controllers/BankAccountController.java"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"e64e05f39b556a6da579df8585012eebf9b800db04aefb0dae5abb3aa1792c8d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Probably move to separate module? \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":"adapter-restapi-spring/src/main/java/com/optivem/kata/banking/adapter/driver/restapi/spring/fake/FakeAuthenticationProvider.java"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"63cbf6e12e4e81e7be8a16dd6a5890beba44cacf737f3d843e94a7e886959f2f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Value object for national identity number \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":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/OpenAccountUseCase.java"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"217c9048b058f928053703802165a6361d7eebe3e30d2c7de4c5152efb0ceed4"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: DELETE \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":"core/src/test/java/com/optivem/kata/banking/core/facade/DepositFundsUseCaseFacadeTest.java"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"785a4b7fc7713778da6040f838eaeb18e9aa0e60696661480525ade072f0d9a9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: This is temporary, re-design to remove this \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":"core/src/test/java/com/optivem/kata/banking/core/facade/common/TestFacade.java"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"094c2092a9e55cddb71164b6c56a3ed05915a1a35af2b338d9349c0d13c6d1c5"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Make configurable so that we can run same test twice \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":"core/src/test/java/com/optivem/kata/banking/core/usecases/OpenAccountUseCaseTest.java"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"67b16c2790440c7a7c1f6dd1549a37894e9db183d8dae1df21fc8299d1386266"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Use Pact Broker \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":"startup/src/test/java/com/optivem/kata/banking/adapter/restapi/spring/BankingProviderContractTest.java"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"652cc972c28c0a80c574a669a9d0ee44eddfce5b8df186bf6626ab576f1dcd53"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: This dependency should only exist in E2E, and maybe demos? \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":"startup/src/test/java/com/optivem/kata/banking/startup/BankAccountControllerSystemTest.java"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa9f03b12e2ed2f79150f8c4ca7f1b64cb5e27dfd5ff4fa8e18e000b779d39b2"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Test with invalid data - e.g. empty fields \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":"startup/src/test/java/com/optivem/kata/banking/startup/BankAccountControllerSystemTest.java"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"1673712f875bec73340b45cf3042cd9f0d72b55e6fa1d9316a0214c6f579abb0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Test with non-existent national identity number \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":"startup/src/test/java/com/optivem/kata/banking/startup/BankAccountControllerSystemTest.java"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce747603f263f9b1b4f1c12d07656b0bd104bccd35fd9ead5314166760093e0e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Consider separate file \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-fixtures/src/main/java/com/optivem/kata/banking/core/common/builders/ports/driven/BankAccountDtoTestBuilder.java"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"3bf520057937c20f5b6f144789f23fa9a0f11a22a26e694ca030cb921c480349"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: VC: Remove this after moving dependencies into specific projects \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":"z-customer-microservice/src/main/java/com/optivem/kata/banking/CustomerApplication.java"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e269c2d7c17b3f91e32e61691f3c824009fe333f1cf9b7ffb46c8a8fd3f7082"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no installation or build instructions: No build/run/gradle commands and no prerequisites (e.g. Keycloak, RabbitMQ) are described in any order. Add a short prerequisites section listing Keycloak client setup, RabbitMQ/docker, and the required Java version. Then add instructions for running Gradle tests, integration, and system tests."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fa6bfcd5c6ae231d1b8ee99bced50b832d1f61cde9da6cbb533467b879a57db"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no usage examples: No \u0027how to run it\u0027 or usage examples. Add a short runnable example showing how to open an account and deposit funds (or the relevant use case) so new readers can try it themselves."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1355eb4e127a4bc9236772ce1c46f09510aa9286e7a06d1e47f308229df19049"}},{"ruleId":"D28","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":"1e646b004b6623e6eb4231ccf8d4aa36435e01e8c8b72afdf347e5b349e97cd6"},"properties":{"commitSha":"214e6fd062ee53b8ba5b3633056dff46ca9807d4"}},{"ruleId":"D28","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":"8d03170d5cdce0c5d61fc9ed6dceb1df1a2f2d3eedb1496f95347ff2b1f91c36"},"properties":{"commitSha":"214e6fd062ee53b8ba5b3633056dff46ca9807d4"}},{"ruleId":"D28","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":"5f2ad38365ac99c3da8d7350ba503695ddfc147aae4f5913f637153099423062"},"properties":{"commitSha":"b8e07a61b101194a1f2962a6173f5e05c60ba1dc"}},{"ruleId":"D28","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":"57d8b45e273af329afc4ffe15dd8b3fe30ba8ec53a433f1f53af1dc2fbb5709a"},"properties":{"commitSha":"c78ee76904b768ded775a3d7dff75e99b07863ec"}},{"ruleId":"D28","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":"817e30748124cde8c257e71cb722dbf40c7471145664e9e602fc986c0469010e"},"properties":{"commitSha":"c78ee76904b768ded775a3d7dff75e99b07863ec"}},{"ruleId":"D28","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":"1528908090a785616e9ccd674d0d20516d0804eabebbf62e3f13cf83f197161d"}},{"ruleId":"D28","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":"e7352a584d832aaf0aed1cecc1bb00f6187640c05147a8a2229e20495424772c"},"properties":{"commitSha":"119403f4f9ed9f80297faf58c1c65eb377eff7a9"}},{"ruleId":"D28","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":"cf9c3953e4bcec64cab7a299246a9bbfecc8e8b2514fd43e034d06356438196e"},"properties":{"commitSha":"119403f4f9ed9f80297faf58c1c65eb377eff7a9"}},{"ruleId":"D28","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":"1f8cc196fb3b611ed2b537c423bd4f644d574c151ae48818307b4c138669433a"},"properties":{"commitSha":"8061fb3342aaeca8e741773b904e89ba57344dc3"}},{"ruleId":"D28","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":"7ba80fbc8baab59ce0456919d5679bd11136db79276b419c0c55b4d045bad0be"},"properties":{"commitSha":"8977b1378a6c6dedcffd6d0492daea7efaedf610"}},{"ruleId":"D28","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":"e54f12f272b988b4530f2caf940d03359ffb5188917e72df6b71ca3732d5cb0a"},"properties":{"commitSha":"119403f4f9ed9f80297faf58c1c65eb377eff7a9"}},{"ruleId":"D28","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":"4d087a9f8fa68b92f92ac77ea0318f3ad8315ce9cc688f4d2bd254294eafcecd"},"properties":{"commitSha":"8061fb3342aaeca8e741773b904e89ba57344dc3"}},{"ruleId":"D28","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":"9a92f7e63736db1a2b72f9317e560be7a5674910e4350f2f2bf4e0a0b73e1d0f"},"properties":{"commitSha":"119403f4f9ed9f80297faf58c1c65eb377eff7a9"}},{"ruleId":"D28","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":"e03f2f28c65227fbb564768fc86b36ae582852f521492980c4875cac412330d9"},"properties":{"commitSha":"214e6fd062ee53b8ba5b3633056dff46ca9807d4"}},{"ruleId":"D28","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":"63fb6e73ad20665b2c89690383a56008a4e9a21e4985140eb597f3548e766e20"}},{"ruleId":"D28","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":"b56fcd64eb99837d33c24b1b9a2037b6b7d3b8117f4257689b740c68b4bc12c1"},"properties":{"commitSha":"8061fb3342aaeca8e741773b904e89ba57344dc3"}},{"ruleId":"D28","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":"3cdfd7beb31b791ae18dcb425e21eb157c842e7bbcd522cc330573c1a6538db1"}},{"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":"77acac134f1162cc6c17e6baf468faf6043bbf5869d3f7bdefbaae2a9ffcc6b0"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4b1ebb9590d0846c1e9b341b950e85e7c0baa2d2da726c8a3e10462672270205"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"659c3ab40914cb81ec4f1f439c93bbd6a2ec899f3bc54e04cf442a8ab3155757"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"49ae853406171c5f6daddd8eda355794d7d0425a395c153d7d702e33edded18a"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"12a55a49dd18d6fb927914eadc9ca515a4cbcab0ab61a611619b904242c1eee9"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"143e822cde2a6a4e30da131748c2cfb2fd690bdfbf64fa79007141eefaf5ab40"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"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":"9cd0c5ac1d3e2f01f55785562b761163b0fa7f0f80c11f4aa497be27850e4648"},"taxa":[{"id":"CWE-1104","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-1329","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","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":"5a6a30d07974e85dde3091ec10545440009318e99d0265adad3acb374dc7e01c"},"taxa":[{"id":"CWE-1104","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-1329","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","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":"5146943cc0159418c23eccb27c4cb243a97f1eed8cb545025881bf777aeac31a"},"taxa":[{"id":"CWE-1104","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-1329","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","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":"3d220bc113cb0d990770d3e8f0a053270dc4b5fb65ab83a8db28cbdb2b9b19bf"},"taxa":[{"id":"CWE-352","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","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":"5d25722e4e84fa1b502c137b668bf8fdd78c8446b6d8bf1f71ce97016082e015"},"taxa":[{"id":"CWE-352","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","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":"2def96c7527a0760956db9f4b7ea49bcf60e6b53d845d94c1ad28e66d1c4c6e6"},"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":"D31","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":"d08bdaf5ea6479c4ec2d7400a3789303b8d50486a55da384c479080e6e30d1b5"},"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":"D31","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":"fc36f54e5ff5dbc8f9d72aad503feaa804fb141d7530891ccedc439e655fafa4"},"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":"D31","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":"2c43aa1090cc8bdf0ba2fecde49736302480b56a1c6d949a3c6b3e49e88a8203"},"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":"D31","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":"bab0cba1a6b550d2929fb58413dccd3cc54bd20bba2c51acee6de03295d4353b"},"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":"D31","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":"3df67cdfb1b809e5ea1a3c24f74325d658ff4496d078d3fc746c444786f6c148"},"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":"D31","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":"c613e6d8a735746e6e643979d15cd36a642960c321bf88afed3114c6c6ff6a17"},"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":"D34","level":"note","message":{"text":"Orphaned files with no living knowledge: 14 of 15 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 15 of the 177 production source files in this repository met that bar). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 most significant first: core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/OpenAccountUseCase.java, core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/ESBankAccount.java, core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/domain/accounts/BankAccount.java, core/src/main/java/com/optivem/kata/banking/core/internal/crud/usecases/OpenAccountUseCase.java, test-fixtures/src/main/java/com/optivem/kata/banking/core/common/factories/CleanArchUseCaseFactory.java, test-fixtures/src/main/java/com/optivem/kata/banking/core/common/builders/ports/driven/BankAccountDtoTestBuilder.java, core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/acl/BankAccountRepositoryImpl.java, core/src/main/java/com/optivem/kata/banking/core/Facade.java (and 6 more). Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D35","level":"error","message":{"text":"Boundary-crossing change coupling: ApplicationEventBus.java \u2194 FacadeFactory.java: \u0060adapter-messaging-inmemory/src/main/java/com/optivem/kata/banking/adapter/driven/messaging/inmemory/ApplicationEventBus.java\u0060 (context adapter-messaging-inmemory) and \u0060test-facade/src/main/java/com/optivem/kata/banking/core/common/factories/FacadeFactory.java\u0060 (context test-facade) sit in DIFFERENT parts of the tree yet change together 50% of the time (5 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) \u2014 the bounded-context boundary may be in the wrong place, or one context is leaking into the other. This is the behavioural boundary violation a static scan can\u0027t see. You can check this without leaving the row: of the 5 shared commits counted here, the most recent 3 are \u0060fc81eb29\u0060 Package Names - Cleanup; \u00603c42b428\u0060 #44 Refactoring - Driver Ports with plain DTOs - Event Bus (at that commit the files were still \u0060src/main/java/com/optivem/kata/banking/infra/real/events/EventPublisherToQueue.java\u0060 and \u0060src/test/java/com/optivem/kata/banking/core/common/factories/FacadeFactory.java\u0060); \u00607aa02689\u0060 #33 OpenAccountUseCase - Publishing events (brought back changes from\u2026 (at that commit the files were still \u0060src/main/java/com/optivem/kata/banking/infra/real/events/EventPublisherToQueue.java\u0060 and \u0060src/test/java/com/optivem/kata/banking/core/common/factories/FacadeFactory.java\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-messaging-inmemory/src/main/java/com/optivem/kata/banking/adapter/driven/messaging/inmemory/ApplicationEventBus.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"59cc25db79689247f8161f3b85b91b5c1d8e97a65b716166d6a6c0975482626a"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DepositFundsUseCase.java \u2194 WithdrawFundsUseCase.java: \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/DepositFundsUseCase.java\u0060 and \u0060core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/WithdrawFundsUseCase.java\u0060 change together 62% of the time (18 of the 29 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 18 shared commits counted here, the most recent 3 are \u00609b6e34af\u0060 Refactoring; \u006023152748\u0060 #44 Refactoring - Driver Ports with plain DTOs (at that commit the files were still \u0060src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/DepositFundsUseCase.java\u0060 and \u0060src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/WithdrawFundsUseCase.java\u0060); \u00609c8c4356\u0060 #30 Application - Pipelinr (refactoring) (at that commit the files were still \u0060src/main/java/com/optivem/kata/banking/core/usecases/depositfunds/DepositFundsUseCase.java\u0060 and \u0060src/main/java/com/optivem/kata/banking/core/usecases/withdrawfunds/WithdrawFundsUseCase.java\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/main/java/com/optivem/kata/banking/core/internal/cleanarch/usecases/DepositFundsUseCase.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b74e5fd21efd562e3e1f2124399a981c1b2eae505e4ad2e23ae2ee1a31075862"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: RandomAccountIdGenerator.java \u2194 RandomAccountNumberGenerator.java: \u0060adapter-generation-random/src/main/java/com/optivem/kata/banking/adapter/driven/generation/random/RandomAccountIdGenerator.java\u0060 and \u0060adapter-generation-random/src/main/java/com/optivem/kata/banking/adapter/driven/generation/random/RandomAccountNumberGenerator.java\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u00602d22001f\u0060 Standardize ProfilesName static variables and make available all requ\u2026; \u0060e57ea931\u0060 Package Names - Cleanup; \u0060ea978f8e\u0060 #77 REST API - Fake Authentication Provider (at that commit the files were still \u0060adapter-generation-random/src/main/java/com/optivem/kata/banking/adapters/generation/random/RandomAccountIdGenerator.java\u0060 and \u0060adapter-generation-random/src/main/java/com/optivem/kata/banking/adapters/generation/random/RandomAccountNumberGenerator.java\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-generation-random/src/main/java/com/optivem/kata/banking/adapter/driven/generation/random/RandomAccountIdGenerator.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e748c4adbf7fc46c59d34f3f50083f12b0803f3187cc6e77a91deee5dbb81eb3"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DepositFundsRequestBuilder.java \u2194 OpenAccountRequestBuilder.java: \u0060test-fixtures/src/main/java/com/optivem/kata/banking/core/common/builders/requests/DepositFundsRequestBuilder.java\u0060 and \u0060test-fixtures/src/main/java/com/optivem/kata/banking/core/common/builders/requests/OpenAccountRequestBuilder.java\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u0060ae7ccab1\u0060 Refactoring (at that commit the files were still \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/DepositFundsRequestBuilder.java\u0060 and \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/OpenAccountRequestBuilder.java\u0060); \u00606151ebf6\u0060 Refactoring - Test Builders (at that commit the files were still \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/DepositFundsRequestBuilder.java\u0060 and \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/OpenAccountRequestBuilder.java\u0060); \u006055c4d580\u0060 Refactoring - Test Builders (at that commit the files were still \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/DepositFundsRequestBuilder.java\u0060 and \u0060src/test/java/com/optivem/kata/banking/core/common/builders/requests/OpenAccountRequestBuilder.java\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"test-fixtures/src/main/java/com/optivem/kata/banking/core/common/builders/requests/DepositFundsRequestBuilder.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"939c1a142ea8ecc21776c9c95d41fd6a3ed489e9382b49de88de04718df9121b"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: OpenAccountUseCase.java \u2194 FacadeFactory.java: \u0060core/src/main/java/com/optivem/kata/banking/core/internal/crud/usecases/OpenAccountUseCase.java\u0060 and \u0060test-facade/src/main/java/com/optivem/kata/banking/core/common/factories/FacadeFactory.java\u0060 change together 55% of the time (6 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u00603b97d40a\u0060 Refactoring - Contract Tests - Renamed CustomerProvider to CustomerGa\u2026; \u00600520e6a8\u0060 Refactoring - Contract Tests - Renamed CustomerProvider to CustomerGa\u2026; \u0060954953be\u0060 #75 Contract Testing - Customer Microservice - Use Case \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/main/java/com/optivem/kata/banking/core/internal/crud/usecases/OpenAccountUseCase.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"214c3b0c617c8bc922747326084cca7cafd92220d761c467db6cd7547b9b57b2"}},{"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":"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":"d657599f6f99da1927d3377533dfc0888b34c4d84aa7d5579841fd72d0e39bce"}},{"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":"DM7","level":"warning","message":{"text":"Repository for a non-root entity: JpaBankAccountDataAccessor \u2192 BankAccountRecord: \u0060JpaBankAccountDataAccessor\u0060 is a repository over \u0060BankAccountRecord\u0060, which is an entity but not an aggregate root. Repositories should be per aggregate ROOT \u2014 loading/saving a child entity independently lets callers bypass the root\u0027s invariants. Access \u0060BankAccountRecord\u0060 through its owning aggregate instead."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"adapter-persistence-jpa/src/main/java/com/optivem/kata/banking/adapter/driven/persistence/jpa/internal/JpaBankAccountDataAccessor.java"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"36f1ffaf0fde81c81a0d368f51f474616d7dd767f938bf105c81f1bf6fd05c5e"}},{"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":"P12","level":"warning","message":{"text":"Coverage collected but not gated: CI collects a coverage report but no step enforces a minimum \u2014 coverage could halve and CI stays green. Add a step that fails the build when coverage drops below a floor (your coverage tool\u0027s minimum-threshold flag, or a coverage-gate action) so the number guards something. What was searched, so you can tell an absence from a miss: this repository\u0027s CI files AND its coverage configuration \u2014 the well-known coverage and test-runner config files, read at the repository root and inside workspace package directories two levels down, so a floor declared beside the tests rather than in the pipeline is credited \u2014 matched against the threshold settings this check knows by name. A floor set in your coverage service\u0027s web UI rather than in a committed file, or under a setting whose name is not one of those, is not seen here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"7f63c06044cf998d9a0698692df30d8873e29bc9b0c98ee829255017c1193d33"}},{"ruleId":"P4","level":"note","message":{"text":"No rollback/health safety: Deployment is orchestrated by compose, but no service declares a \u0060healthcheck:\u0060 and nothing pins a previous image to fall back to \u2014 the runtime can tell that the container is up, not that it is serving, so a bad release is harder to detect and reverse."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"db6bab8a28a2145f47e5a4e6683cda39d2ba0296c723238e8057da979ae1c706"}},{"ruleId":"P5","level":"warning","message":{"text":"No DR/backup evidence: A persistence guard (data volume / purge-protection) was found, but no backup, geo-recovery or RTO/RPO controls were evidenced \u2014 a volume that survives a container recreate is not a tested restore from catastrophic loss."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"5daabb38c90b07af65b8738cf94ce884e90a7d8fef810d341de5c186188c8fe0"}},{"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-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"id":"CWE-1104","guid":"4c918cb5-b2a6-6c55-9963-a44ee464305e","name":"CWE-1104","shortDescription":{"text":"CWE-1104"},"helpUri":"https://cwe.mitre.org/data/definitions/1104.html"},{"id":"CWE-1329","guid":"f70f1c3f-4ccf-cb5e-bdd3-03ba4868d36d","name":"CWE-1329","shortDescription":{"text":"CWE-1329"},"helpUri":"https://cwe.mitre.org/data/definitions/1329.html"},{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.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-352","guid":"2afe7bdb-7330-195f-b3bf-8cf5b789b3ae","name":"CWE-352","shortDescription":{"text":"CWE-352"},"helpUri":"https://cwe.mitre.org/data/definitions/352.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-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-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":45,"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."]}]}