{"$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":"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":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"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":"DM2","name":"Strongly-typed ids","shortDescription":{"text":"Strongly-typed ids"},"helpUri":"https://codehealth.canine.dev/dimensions/DM2"},{"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":"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":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"PduHeaders.setOctet (cyclomatic 66): PduHeaders.setOctet has cyclomatic complexity 66 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function so the top-level body reads as a short sequence of named decisions."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduHeaders.java"},"region":{"startLine":358}}}],"partialFingerprints":{"codehealthFindingId/v1":"d015106de882f02012505cb222f07b1b365c05095e6b22dc922702f9ddeef0ab"}},{"ruleId":"D1","level":"warning","message":{"text":"PduComposer.appendHeader (cyclomatic 37): PduComposer.appendHeader has cyclomatic complexity 37 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":544}}}],"partialFingerprints":{"codehealthFindingId/v1":"93f5b7476559722d30f102beedb11bf1b77d3bde35ecce5132ab7129142f7fc9"}},{"ruleId":"D1","level":"warning","message":{"text":"MessagesListFragment.onViewCreated (cyclomatic 33): MessagesListFragment.onViewCreated has cyclomatic complexity 33 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/MessagesListFragment.kt"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"39d02471284e589ceae31a569f5a61fdee933f0f7f7e3fea25d2e184295a8440"}},{"ruleId":"D1","level":"warning","message":{"text":"WebhookQueueListFragment.onViewCreated (cyclomatic 28): WebhookQueueListFragment.onViewCreated has cyclomatic complexity 28 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/settings/WebhookQueueListFragment.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"0de421b732b3e8ada966f55450bab1e8bed0897a963fc12819d62fef941a1e93"}},{"ruleId":"D1","level":"warning","message":{"text":"PduComposer.makeMessageBody (cyclomatic 28): PduComposer.makeMessageBody has cyclomatic complexity 28 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":919}}}],"partialFingerprints":{"codehealthFindingId/v1":"60b9bb66f3a3f65d44df45d3f0049ccc80b98c0ab639f7e45c02ffeb364fb3ea"}},{"ruleId":"D1","level":"warning","message":{"text":"IncomingMessagesListFragment.onViewCreated (cyclomatic 25): IncomingMessagesListFragment.onViewCreated has cyclomatic complexity 25 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"333f5b0aea6d7e35916602687d5c7d11ac1978af34dee29034528a25c2357814"}},{"ruleId":"D1","level":"warning","message":{"text":"MessagesSettings.import (cyclomatic 23): MessagesSettings.import has cyclomatic complexity 23 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function so the top-level body reads as a short sequence of named decisions."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"55a77a3a2bad0cb174ba338efcd6eb00611ac995898289e8bd4fb3b5edf1ba05"}},{"ruleId":"D1","level":"warning","message":{"text":"InboxRoutes.inboxRoutes (cyclomatic 22): InboxRoutes.inboxRoutes has cyclomatic complexity 22 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"8d51bb29b83bec23007f7c4a389032bba0417ab9cbc6c12e88d0d117cd1745da"}},{"ruleId":"D1","level":"warning","message":{"text":"MessagesRoutes.messagesRoutes (cyclomatic 20): MessagesRoutes.messagesRoutes has cyclomatic complexity 20 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function so the top-level body reads as a short sequence of named decisions."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea423cc86f80d7261bc70f3bd6e18a52760ea336e1c5f7d9a190d780fe5463ba"}},{"ruleId":"D1","level":"warning","message":{"text":"MessagesService.processStateIntent (cyclomatic 16): MessagesService.processStateIntent has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing. This is NOT this file\u0027s highest cyclomatic complexity: MessagesService.resultToErrorMessage (cyclomatic 59) is higher and carries no row of its own \u2014 it was excluded as a flat dispatcher (a long switch/match over independent cases: many branches, almost no nesting), which this dimension does not treat as a refactor obligation. It is named here so the ranking you see in this file is not mistaken for the whole of it; the excluded method is counted neither in this dimension\u0027s figures nor in its score."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":222}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e5c65d71d1f531a07c0dc39fe84b5cdad2373990697ba298684dc8c6a1e6fde"}},{"ruleId":"D2","level":"warning","message":{"text":"PduComposer.makeMessageBody (cognitive 61): PduComposer.makeMessageBody has cognitive complexity 61 (threshold 15). Drivers by points: if/else 22 (40 pts), error handling 5 (14 pts), loops 2 (4 pts), boolean chains 3 (nesting depth added 29). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":919}}}],"partialFingerprints":{"codehealthFindingId/v1":"94b84975e979ee1e02000aaacbe156ea3b7ed32fc5194732e5c5bc022223688a"}},{"ruleId":"D2","level":"warning","message":{"text":"PduHeaders.setOctet (cognitive 56): PduHeaders.setOctet has cognitive complexity 56 (threshold 15). Drivers by points: if/else 19 (33 pts), boolean chains 22, match/switch 1 (nesting depth added 14). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduHeaders.java"},"region":{"startLine":358}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ce2302a23ee6a8b451a1020eb24379ccf9a0c01c2cb1b827644fbc2d35d2a19"}},{"ruleId":"D2","level":"warning","message":{"text":"WebhookQueueListFragment.onViewCreated (cognitive 36): WebhookQueueListFragment.onViewCreated has cognitive complexity 36 (threshold 15). Drivers by points: if/else 17 (32 pts), match/switch 2 (4 pts) (nesting depth added 17). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/settings/WebhookQueueListFragment.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce511804195580e078098f59db94c7b4e92e866e2e16f76026f679ffa5c6d7fc"}},{"ruleId":"D2","level":"warning","message":{"text":"PduComposer.appendHeader (cognitive 36): PduComposer.appendHeader has cognitive complexity 36 (threshold 15). Drivers by points: if/else 18 (32 pts), loops 1 (2 pts), boolean chains 1, match/switch 1 (nesting depth added 15). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":544}}}],"partialFingerprints":{"codehealthFindingId/v1":"7af3c97aba87d152e5ad0de296e422f0158898256039f8c0aae769766001d2bf"}},{"ruleId":"D2","level":"warning","message":{"text":"MessagesListFragment.onViewCreated (cognitive 30): MessagesListFragment.onViewCreated has cognitive complexity 30 (threshold 15). Drivers by points: if/else 18 (26 pts), match/switch 3 (4 pts) (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/MessagesListFragment.kt"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b8027d33e5ab2e4b305f0cb027de8bf483a88a811d0e459dbefb41819268edd"}},{"ruleId":"D2","level":"warning","message":{"text":"MessagesService.processStateIntent (cognitive 27): MessagesService.processStateIntent has cognitive complexity 27 (threshold 15). Drivers by points: if/else 9 (14 pts), match/switch 5 (12 pts), boolean chains 1 (nesting depth added 12). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":222}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f624dcbd10f9b11bfaf5a61bd95d9ba27be710569960e457a8295719758e31d"}},{"ruleId":"D2","level":"warning","message":{"text":"IncomingMessagesListFragment.onViewCreated (cognitive 24): IncomingMessagesListFragment.onViewCreated has cognitive complexity 24 (threshold 15). Drivers by points: if/else 14 (20 pts), match/switch 3 (4 pts) (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b18bb88af5b158b6d33334b8930c4b470833c8a9bb0f9522c10014527bc987d"}},{"ruleId":"D2","level":"warning","message":{"text":"InboxRoutes.inboxRoutes (cognitive 21): InboxRoutes.inboxRoutes has cognitive complexity 21 (threshold 15). Drivers by points: if/else 11, error handling 9, boolean chains 1. To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4dd5576c5dbc54d64f221bc668be55236e29d724ee906069ae7b52599576b4c"}},{"ruleId":"D2","level":"warning","message":{"text":"MessagesSettings.import (cognitive 20): MessagesSettings.import has cognitive complexity 20 (threshold 15). Drivers by points: if/else 7 (13 pts), boolean chains 6, match/switch 1 (nesting depth added 6). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e0e433fa89af5edfa4845d8f9257b72d35bd9920d49099607975a5c3eb4ce6b"}},{"ruleId":"D2","level":"warning","message":{"text":"MessagesRoutes.messagesRoutes (cognitive 17): MessagesRoutes.messagesRoutes has cognitive complexity 17 (threshold 15). Drivers by points: if/else 8, error handling 7, boolean chains 1, match/switch 1. To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"419d69ee19cfdcf957661f2b1afa704ed213cbf0d593d774e032dadc81a942d5"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: MessagesRoutes.messagesRoutes: MethodTooLong \u2014 messagesRoutes runs 149 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 49 over it, 1.49\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"b91fdf41610b4eb4dc1ea3a9d2ff1735a2d8f7ba6173443c587f832db9f1c86c"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: PduComposer: ClassTooLong \u2014 529 significant lines (blank, comment-only and punctuation-only lines excluded), 24 methods. The bar is 400 significant lines; this is 129 over it, 1.32\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ffc36759258256464aff2b22a174af42a45b56a55cb27faa63e58f844848ea9"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: MessagesService: ClassTooLong \u2014 504 significant lines (blank, comment-only and punctuation-only lines excluded), 23 methods. The bar is 400 significant lines; this is 104 over it, 1.26\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"533e040e50736f031f18e765f6dc7785bca3f80f07b496b673a4356ecefbe4d6"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: InboxRoutes.inboxRoutes: MethodTooLong \u2014 inboxRoutes runs 125 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 25 over it, 1.25\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"a975608db79d2041455181f056a474bd1796606110dc8caf8aa2e6d503afd51c"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: PduComposer.appendHeader: MethodTooLong \u2014 appendHeader runs 120 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 20 over it, 1.20\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":544}}}],"partialFingerprints":{"codehealthFindingId/v1":"84c7ec37dce6bf4ed18a431b83b039ab1a8263874c862264352eedfbad226787"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: PduComposer.makeMessageBody: MethodTooLong \u2014 makeMessageBody runs 115 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 15 over it, 1.15\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":919}}}],"partialFingerprints":{"codehealthFindingId/v1":"479578ee44063c13534166636c58862747075d53f1117494f8ba06765940817c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: messages/MessagesService.kt: FileTooLong \u2014 549 significant lines (blank, comment-only and punctuation-only lines excluded), about 92% of them inside a single declaration: MessagesService (48-825). The bar is 500 significant lines; this is 49 over it, 1.10\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"51eed528876a32ce8819cf6cdd259549ec479342967d76c1aef6f78d02bcf073"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: aosp/PduComposer.java: FileTooLong \u2014 539 significant lines (blank, comment-only and punctuation-only lines excluded), about 98% of them inside a single declaration: PduComposer (31-1216). The bar is 500 significant lines; this is 39 over it, 1.08\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduComposer.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"74fd403a38cde6bf91f5d5adc0a4e9113088cdc1867c7b070f6599b1ed3f4fb3"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: HomeFragment.onViewCreated: MethodTooLong \u2014 onViewCreated runs 105 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 5 over it, 1.05\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/HomeFragment.kt"},"region":{"startLine":132}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f856dff07d9a7957f8a0fb724a777772bcaa77753c625a3adbd60f03097672a"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: PduHeaders.setOctet: MethodTooLong \u2014 setOctet runs 102 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 2 over it, 1.02\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduHeaders.java"},"region":{"startLine":358}}}],"partialFingerprints":{"codehealthFindingId/v1":"53bfcced8e742bfe97a428092194ac810cfaf0f531aafc4d556fd2e659033493"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: PduHeaders: ClassTooLong \u2014 402 significant lines (blank, comment-only and punctuation-only lines excluded), 12 methods. The bar is 400 significant lines; this is 2 over it, 1.01\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/PduHeaders.java"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"bac8de37ea81277ce9f3535acf3f3ee6cc90badd0b30d6cedb833353b40b044d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/SettingsUpdateWorker.kt:44-64 | app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/WebhooksUpdateWorker.kt:53-73 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/SettingsUpdateWorker.kt:44\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":"app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/SettingsUpdateWorker.kt"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa39b1678f0b57da57a2455cdf61fe27968e826639fb38c1284018b763998ba7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18\u201320 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt:108-125 | app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt:81-100 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt:108\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt"},"region":{"startLine":108}}}],"partialFingerprints":{"codehealthFindingId/v1":"12515b917da0690515e3f9f9f6d4a6f1081f6e825d5aa9df293d2cc70b7e4050"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:45-56 | app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt:60-74 \u2014 before extracting anything, compare \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 39 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:45\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e881d73142e88e0799cfc9082d7ec415c5b72a2090fc9705311b7af59fded7f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt:38-46 | app/src/main/java/me/capcom/smsgateway/ui/MessagesListFragment.kt:41-49 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt:38\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"921c36d5db2fd80f180cad4e9a5c2898500958f7c0aaf7dc12c8bebb5f570fa8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt:86-92 | app/src/main/java/me/capcom/smsgateway/ui/MessagesListFragment.kt:98-104 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt:86\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/ui/IncomingMessagesListFragment.kt"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"431724027711a855681f3b1df9e3b0eb455c31c14eaa4b441429024ac1fe16a7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/incoming/vm/IncomingMessagesListViewModel.kt:38-44 | app/src/main/java/me/capcom/smsgateway/modules/messages/vm/MessagesListViewModel.kt:40-46 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. 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":"app/src/main/java/me/capcom/smsgateway/modules/incoming/vm/IncomingMessagesListViewModel.kt"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"d271bc31e6f0b37195b41334d57c555daca934ad4b59091989a346c6707e900f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/receiver/MessagesReceiver.kt:96-102 | app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsReceiver.kt:110-116 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/receiver/MessagesReceiver.kt"},"region":{"startLine":96}}}],"partialFingerprints":{"codehealthFindingId/v1":"95d4f89b9f4ec6fc4ce430bd57b7f6776a81124a32316ebe925e15b5633c973a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:119-130 | app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt:147-158 \u2014 before extracting anything, compare \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 39 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:119\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt"},"region":{"startLine":119}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc1f8579d78312899de1b09b88594ebab5e045adb4cdaec5a9a66fd80620f80b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:84-95 | app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt:102-113 \u2014 before extracting anything, compare \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/SmsContentObserver.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 39 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt:84\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/receiver/MmsContentObserver.kt"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"489f85b08275341f44c4d90aa07254e3c764017a0b93e4106485395f8c1eab20"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt: app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt changed 9 times in last 90 days, max cyclomatic complexity 59 in MessagesService.resultToErrorMessage at line 762. 2 of those changes were fix/bug commits, and the other 7 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":762}}}],"partialFingerprints":{"codehealthFindingId/v1":"156538cbf86bfdd2970054dc3d1ed138da3b6de551e8d5d2af7b14cf97f1c11a"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt: app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt changed 4 times in last 90 days, max cyclomatic complexity 20 in MessagesRoutes.messagesRoutes at line 54. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/MessagesRoutes.kt"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a8e2eda012c38ee43646711ac5247f09d1de10290fec84f18e60963cae0d47a"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/receiver/parsers/MMSParser.kt: app/src/main/java/me/capcom/smsgateway/modules/receiver/parsers/MMSParser.kt changed 2 times in last 90 days, max cyclomatic complexity 29 in MMSParser.parseMNotificationInd at line 71. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/receiver/parsers/MMSParser.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/receiver/parsers/MMSParser.kt"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f916c6f11e1a5e001044b7177d8c84449f019c5f0f6d19776b1f68a7834c7dd"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt: app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt changed 2 times in last 90 days, max cyclomatic complexity 23 in MessagesSettings.import at line 188. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesSettings.kt"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e43de3cc52b9df2f07c740236b2caa6105d6b5d3a33a309b949ed4fe19cae5e"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/gateway/GatewayService.kt: app/src/main/java/me/capcom/smsgateway/modules/gateway/GatewayService.kt changed 3 times in last 90 days, max cyclomatic complexity 15 in GatewayService.processMessage at line 246. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/gateway/GatewayService.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/gateway/GatewayService.kt"},"region":{"startLine":246}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3a1204e234b993f41fa01604246d355c2ea660432cb95af01c801c19d642fea"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt: app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt changed 2 times in last 90 days, max cyclomatic complexity 22 in InboxRoutes.inboxRoutes at line 39. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 07:23:09 \u002B07:00\u0027 --until=\u00272026-09-25 07:23:09 \u002B07:00\u0027 --full-history --no-merges -- app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/routes/InboxRoutes.kt"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"53b30679768e1a199f4a7f560137976736ddb7df90e7e1cf958ac05e95bd00b4"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // todo: use MessagesRepository \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":"app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb15eeced971abc5dcde987302e42021bd7f8191f0768891d3dba8698fac026e"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: Content-type must be decided according to whether \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":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/SendReq.java"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f980b95d4ab5b3aa77863b295d40c72b3c69bf56c63bb68ba7122e286f5fc5d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: CharSet needs to be validated against MIBEnum. \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":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/EncodedStringValue.java"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"6acb49f6db5a0ed58db247af3c828c0de7e2f1f38e78b2598366bf3ee85498bf"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: CharSet needs to be validated against MIBEnum. \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":"app/src/main/java/me/capcom/smsgateway/modules/mms/pdu/aosp/EncodedStringValue.java"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"3e873f9a3bbeeb1c626cfccebe819dbcd87e32e0112eb3f1d761ed1967a08f9f"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no architecture or design documentation: The README does not reference any architecture or design documentation, despite having two markdown docs (not shown). Link to an architecture document covering the gateway\u0027s message-flow graph, rate limits, and cloud/server deployment."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"086460f2da112d94904fac5439f54a7780bc7778ff9375a067e428a1cc9a8a4c"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"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":"095555378dd835f84e259e9c9384262a5b261e18cf7865fa2090ba73b0eee205"},"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":"0bfe52d90d2bd8e3c66070eaad98517295083903617508985908d6bddc2e92c5"},"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":"60002d93f120ef55ea5918b40d2496031998d69b7340539b597d7ac775b09933"},"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":"e841a40bcd6d70751a4fc9a76d45b72db754eb7c3e4d1b5f0e1f1238c4d2a3cb"},"taxa":[{"id":"CWE-78","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":"9fdec05352c785d2bce98256908af13f863f9b593546eb15e0b68382c2b921f3"},"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":"38d513c0f2427739ec07812c646e7cd8cf3ad6e627b06ac79773262f554be127"},"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":"07e9f34a15fc0ce3ebc65fb0f7718e0356223d37862f168d351f1427cccaaad8"},"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":"3a8b170bd3043b9cb73fa52edb8c4d8865d21621f21cc92d37121d5d7f83a51b"},"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":"7d6deb8caf57ce9a94571c04d1fa66f73571af8105c6b364b3e9d6d989e606ab"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"4b9683954c927adde51f5b7ccac484a6528d3b649930ce673e6e6e78e7b2b5e9"},"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":"0ac7d87d3daa5ac6ebed8bd442b8d7878019315aa85ba789e3f8db08a6b1a1c5"},"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":"76a82b352ddcfd04465dfb9324cf52612d433123cb04dfc4b272b2079ee21ca7"},"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":"23caa47746d1bc00153080bd8a94b9013c68c417385163ed3fe8662f57150c68"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"a2f8934b73eb1fd702b381ead093d099cd7568b49e9700de272caf032448f5ac"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"e48718994286c7257a2bc132aa1b0c8b5a7829b547e6ba5c2cc6cacca48adad2"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"1764ef2a64375594c5c163a9726856cc9b5ce68e631857ad329f473ba8246c19"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"1f97830b1538643b300a7af863a3acae68fa877c2b15ff6f021702b311a6f471"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"2210f8469536350daf101d60c5c109494a0f5c74aafe99e31361ea992453353c"},"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":"14c17cd4cb2d4e85eea7a5e7965aea36b12e3cfe02863d5e0c1d80bc989d8753"},"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":"bb468f70abde72932b40ce5ba9a23b9a39388c811a65f967ab34968f4fd4535e"},"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":"82c6c4ae07e2f93f9433c967b5b9bcf49fcb87c78612c4bcc951fe9310c151f8"},"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":"180da4dbe22bf29194d1c31131f2964e91f5c9d4a5b86bf2b43934297b141e85"},"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":"f2fca1f5be7d0cbba460c006c990897e8227c66a615b16357af1bab85a992f3f"},"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":"679edc22363480901e5c957c35f01695e8152d5ec9a18687c36df28fd9f13f6e"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-552","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":"8bc18167b1c4af233d9507208469856c1d3f53f1e7566672e76e88057df169a1"},"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":"5a146052cc5a0e229c2e49ebb340c13b622f5172fa350f87c08a80754616154c"},"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":"D35","level":"warning","message":{"text":"Change coupling: AppDatabase.kt \u2194 MessagesService.kt: \u0060app/src/main/java/me/capcom/smsgateway/data/AppDatabase.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt\u0060 change together 64% of the time (16 of the 25 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 16 shared commits counted here, the most recent 3 are \u00603f82b13a\u0060 [messages] add scheduling; \u0060e38c6560\u0060 [webhooks] add parts count to the \u0060sms:sent\u0060 webhook payload; \u00609a8d3dea\u0060 [localserver] get messages history \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/AppDatabase.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3ca1ab31a320976a04cd960af293f6cf75d7fcda530d4d455d85142b1af22be7"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: WebService.kt \u2194 EventsReceiver.kt: \u0060app/src/main/java/me/capcom/smsgateway/modules/localserver/WebService.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/receivers/EventsReceiver.kt\u0060 change together 58% of the time (7 of the 12 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 7 shared commits counted here, the most recent 3 are \u0060960b37fd\u0060 Fixed: show only 50 last messages (at that commit the file was still \u0060app/src/main/java/me/capcom/smsgateway/services/WebService.kt\u0060); \u00606a1e466e\u0060 Added: messages list basic components (at that commit the file was still \u0060app/src/main/java/me/capcom/smsgateway/services/WebService.kt\u0060); \u006049e263e6\u0060 Added: get messages from server and send message status (at that commit the file was still \u0060app/src/main/java/me/capcom/smsgateway/services/WebService.kt\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/localserver/WebService.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd0b5549c57535888f52c5b0c57a4a0e2764e4bb378b29f60d10d332b333e6c8"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: PullMessagesWorker.kt \u2194 MessagesService.kt: \u0060app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/PullMessagesWorker.kt\u0060 and \u0060app/src/main/java/me/capcom/smsgateway/modules/messages/MessagesService.kt\u0060 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) 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 5 shared commits counted here, the most recent 3 are \u006053470165\u0060 [gateway] send message state by \u0060Worker\u0060; \u0060e8b439f6\u0060 Fixed: code cleanup (at that commit the file was still \u0060app/src/main/java/me/capcom/smsgateway/modules/gateway/PullMessagesWorker.kt\u0060); \u0060dd1b3734\u0060 Added: random delats between messages (at that commit the file was still \u0060app/src/main/java/me/capcom/smsgateway/modules/gateway/PullMessagesWorker.kt\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/gateway/workers/PullMessagesWorker.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"eaf0bb0826d2b9740d08f4476dd60a88f16799265183b0f5a218994b236d8612"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"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":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"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":"d068c977b62d9a977df1bb6f53e8b00b586c3abee955f91d715f92b8e019d624"}},{"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":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: Message: \u0060Message\u0060\u0027s public constructor takes \u0060id\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060Message\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/Message.kt"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"9a8f8f302ce4e758e6c8c25fa02f16a2f73e919b885f152ad88d5cd5a7fcb655"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: MessageRecipient: \u0060MessageRecipient\u0060\u0027s public constructor takes \u0060messageId\u0060, \u0060phoneNumber\u0060, \u0060error\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060MessageRecipient\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageRecipient.kt"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"905b36ec5dab7f7bd0cf665d33c1dd79b45ddb8cf0103f2679243a5ce8ce3d8d"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: MessageState: \u0060MessageState\u0060\u0027s public constructor takes \u0060messageId\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060MessageState\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageState.kt"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"f21b6544d114178ce858cfc2f4f0d0885983451c951152f30e450e54f5975f49"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: RecipientState: \u0060RecipientState\u0060\u0027s public constructor takes \u0060messageId\u0060, \u0060phoneNumber\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060RecipientState\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/RecipientState.kt"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"bade5e3a7cc921033070aa901c63259a115bff59668ce8879cb55d5920eeb380"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: Token: \u0060Token\u0060\u0027s public constructor takes \u0060id\u0060, \u0060tokenUse\u0060, \u0060parentJti\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060Token\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/Token.kt"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"8d45cb6c5e3627f9531451b78adaa409a0257de5f78c6aa785ebf9a4ac25e5df"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: InboxUploadEntity: \u0060InboxUploadEntity\u0060\u0027s public constructor takes \u0060messageId\u0060, \u0060type\u0060, \u0060senderEncrypted\u0060, \u0060recipientEncrypted\u0060, \u2026 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060InboxUploadEntity\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/gateway/inbox/InboxUploadEntity.kt"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"150cbac0cef5d0409277cb34c2f5be58f0a7f9438fe05c5bcf7813e131e120c8"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: IncomingMessage: \u0060IncomingMessage\u0060\u0027s public constructor takes \u0060id\u0060, \u0060sender\u0060, \u0060recipient\u0060, \u0060contentPreview\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060IncomingMessage\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/incoming/db/IncomingMessage.kt"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb9c39c3a9d99b72214efe3ce3fc803ac44a8c7a4dd14362cc55820f665ad09d"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: LogEntry: \u0060LogEntry\u0060\u0027s public constructor takes \u0060module\u0060, \u0060message\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060LogEntry\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/logs/db/LogEntry.kt"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb867799b7d430e216d47e143fa7a6ac8f0982b3f38bc49b836dd51dc512f9f0"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: WebHook: \u0060WebHook\u0060\u0027s public constructor takes \u0060id\u0060, \u0060url\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060WebHook\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/webhooks/db/WebHook.kt"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"d44dc55ca6aae3f1c64c9076b79071e2bdd2e476d9b98a73187659727fb431d8"}},{"ruleId":"DM11","level":"warning","message":{"text":"Constructible in an invalid state: WebhookQueueEntity: \u0060WebhookQueueEntity\u0060\u0027s public constructor takes \u0060id\u0060, \u0060url\u0060, \u0060payload\u0060, \u0060lastError\u0060 as raw primitives, and neither of the two things that would make that safe was found. What was searched, so you can tell an absence from a miss: for VALIDATION, the constructor body was scanned for \u0060throw\u0060, \u0060Guard.\u0060, \u0060Ensure.\u0060, \u0060ArgumentNullException\u0060, \u0060ArgumentException\u0060, \u0060CheckRule\u0060 or \u0060Requires.\u0060 \u2014 a guard reached through a helper method those tokens do not name is not seen; for a FACTORY, only this type\u0027s OWN static members were read, and only names beginning \u0060Create\u0060, \u0060Of\u0060, \u0060From\u0060, \u0060New\u0060 or \u0060Build\u0060 \u2014 one inherited from a base type, one on a companion builder, or one called \u0060Parse\u0060, \u0060Make\u0060 or \u0060Rehydrate\u0060 is not seen. If either exists, this row is wrong. If neither does, a caller can build a \u0060WebhookQueueEntity\u0060 that breaks the rules it exists to enforce, and every rule downstream then has to cope with a state the model said was impossible. Guard in the constructor, take value objects that validate themselves, or make the constructor private behind a \u0060Create\u0060 that can refuse."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/webhooks/db/WebhookQueueEntity.kt"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"b19a728f7ef1263e39e36bd9fdc7ad1e6fa67294838a192eaba19c6bab3c8cb1"}},{"ruleId":"DM2","level":"note","message":{"text":"Primitive id on a domain type: MessageRecipient.messageId: \u0060MessageRecipient.messageId\u0060 is a raw \u0060String\u0060 \u2014 give it a strongly-typed id: a dedicated single-field type wrapping the \u0060String\u0060, in whatever form your language spells that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageRecipient.kt"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"a20031e2d4f454cfced5c2946067110d0988fea37c887eaaa0a2d03cf4f29cca"}},{"ruleId":"DM2","level":"note","message":{"text":"Primitive id on a domain type: MessageState.messageId: \u0060MessageState.messageId\u0060 is a raw \u0060String\u0060 \u2014 give it a strongly-typed id: a dedicated single-field type wrapping the \u0060String\u0060, in whatever form your language spells that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageState.kt"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"735685299727fb5cac59118156c95f269546af112fe03021fc34cc0f08db6fa5"}},{"ruleId":"DM2","level":"note","message":{"text":"Primitive id on a domain type: RecipientState.messageId: \u0060RecipientState.messageId\u0060 is a raw \u0060String\u0060 \u2014 give it a strongly-typed id: a dedicated single-field type wrapping the \u0060String\u0060, in whatever form your language spells that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/RecipientState.kt"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"6c01441ab76a6448be6f6185e5069d2b61b9e1f1ee06f6d0cb8b316d51608473"}},{"ruleId":"DM2","level":"note","message":{"text":"Primitive id on a domain type: IncomingMessage.subscriptionId: \u0060IncomingMessage.subscriptionId\u0060 is a raw \u0060Int\u0060 \u2014 give it a strongly-typed id: a dedicated single-field type wrapping the \u0060Int\u0060, in whatever form your language spells that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/incoming/db/IncomingMessage.kt"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"989e5830533ce6019d16b848e6ad08f3c9835fcc03bf300c854173f5abac69c3"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: Message: \u0060Message\u0060 is an aggregate/entity with 18 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/Message.kt"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"dde1ce1ea4223525fbc42a7b8b94574f9c5defaace9e3c603a083347a20485b0"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: MessageRecipient: \u0060MessageRecipient\u0060 is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageRecipient.kt"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"7fe3b721e809894dc9e9dc3a5b168fa49c7bc79ab84787b76cd7394c333734a4"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: MessageState: \u0060MessageState\u0060 is an aggregate/entity with 3 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/MessageState.kt"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"e1c64b725338e1535c61fae8e6efab706cecd47a2ea9f583b8ffdcbde7811bda"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: RecipientState: \u0060RecipientState\u0060 is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/RecipientState.kt"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"3bfc2fb4f7479e637df9304027c55561309cc5a3aaac513105bee1fd04a2bd12"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: Token: \u0060Token\u0060 is an aggregate/entity with 5 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/data/entities/Token.kt"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"a05c1fc27242902816fbb1f6321f68ad62a7839557b98b54596010835cf2eab0"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: IncomingMessage: \u0060IncomingMessage\u0060 is an aggregate/entity with 8 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/incoming/db/IncomingMessage.kt"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"1828e62d17bdef57df557ded51fbf2041ab7f4a05695b10bb57869b58440fd66"}},{"ruleId":"DM4","level":"warning","message":{"text":"Anemic entity: WebHook: \u0060WebHook\u0060 is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) \u2014 the business logic lives in a service."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"app/src/main/java/me/capcom/smsgateway/modules/webhooks/db/WebHook.kt"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ded0159cc02182f6c1c9a32aa0c037e6bf22ad0dd4e3bce446f40338cd69bae"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M2","level":"note","message":{"text":"No architecture diagram/doc: No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named \u0060architecture\u0060 or \u0060design\u0060 in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape \u2014 no \u0060## Architecture\u0060 heading, no \u0022architecture overview\u0022/\u0022high-level design\u0022 phrasing, no \u0022the architecture is \u2026\u0022 introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document \u2014 not evidence that nobody wrote the shape down."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0c190e7c159d1850ee706c3ac4486e151e8a4fe169de4bf61436b9f399654f06"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add spotbugs with find-sec-bugs (or \u0060semgrep --config=auto\u0060, which runs on any language) as a CI step. What was searched, so you can tell an absence from a miss: the 10045 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-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-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-522","guid":"71fb233e-ce6a-ae57-9419-ef8373540b09","name":"CWE-522","shortDescription":{"text":"CWE-522"},"helpUri":"https://cwe.mitre.org/data/definitions/522.html"},{"id":"CWE-552","guid":"3492436b-eca2-9c54-9ba3-5dade427c903","name":"CWE-552","shortDescription":{"text":"CWE-552"},"helpUri":"https://cwe.mitre.org/data/definitions/552.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-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":31,"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."]}]}