{"$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":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D33","name":"JS/npm Dependency Vulnerabilities","shortDescription":{"text":"JS/npm Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D33","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"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":"D37","name":"Vulnerability-disclosure Policy","shortDescription":{"text":"Vulnerability-disclosure Policy"},"helpUri":"https://codehealth.canine.dev/dimensions/D37","relationships":[{"target":{"id":"CWE-1059","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1059"]}},{"id":"D38","name":"OSV Dependency Vulnerabilities","shortDescription":{"text":"OSV Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D38","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"Server.handleInternal (cyclomatic 145): Server.handleInternal has cyclomatic complexity 145 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":563}}}],"partialFingerprints":{"codehealthFindingId/v1":"d374f1e6122377ce4fe76acc05b8822ae532d6f2893178c5017ce91976451ee3"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execServe (cyclomatic 144): cmd.execServe has cyclomatic complexity 144 (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":"cmd/serve.go"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"1af7f8beb462abb188935faa7d7043da98f447480d357b675eb32d426aa39302"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.parsePublishParams (cyclomatic 47): Server.parsePublishParams has cyclomatic complexity 47 (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":"server/server.go"},"region":{"startLine":1128}}}],"partialFingerprints":{"codehealthFindingId/v1":"1e5ed6efcc46fdcb932ce0c4067f4a015014b75fa3be266a1dda108ade4d6b38"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execPublish (cyclomatic 41): cmd.execPublish has cyclomatic complexity 41 (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":"cmd/publish.go"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"4976e7e401b10268633d18bcd7aa9744e4b7ec912dde9de88f2de2519765ce1c"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handlePublishInternal (cyclomatic 34): Server.handlePublishInternal has cyclomatic complexity 34 (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":"server/server.go"},"region":{"startLine":858}}}],"partialFingerprints":{"codehealthFindingId/v1":"f247e2ce31ee636842414dd9037a6f60a19a9c5febcd4a88c6555f269add0b29"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handleSubscribeWS (cyclomatic 29): Server.handleSubscribeWS has cyclomatic complexity 29 (threshold 15). Of this number, 15 points are the body\u0027s own statements and 14 belong to 4 function literals inside it that branch. 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":"server/server.go"},"region":{"startLine":1510}}}],"partialFingerprints":{"codehealthFindingId/v1":"e6811437cd5067a0e2016acdc0fb10035629d68e807491acbacbfdcc807c57e3"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execTierChange (cyclomatic 28): cmd.execTierChange has cyclomatic complexity 28 (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":"cmd/tier.go"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4e7774aa4f05fe0a9ff2d19bc4b4efa8fba54a429b63cdfa1fb5e25e6a9875a"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handleAccountGet (cyclomatic 28): Server.handleAccountGet 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":"server/server_account.go"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"a828868ec757ca411d47c0d7315e5497bb3c38ee6278bbcf14ca4b94f7f285f5"}},{"ruleId":"D1","level":"warning","message":{"text":"main.execImport (cyclomatic 27): main.execImport has cyclomatic complexity 27 (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":"tools/pgimport/main.go"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"843b12a4794e7082c1665e0f95a39bf43f063669b30df84bccc2ad9d606e00b9"}},{"ruleId":"D1","level":"warning","message":{"text":"action.populateAction (cyclomatic 25): action.populateAction has cyclomatic complexity 25 (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":"action/action.go"},"region":{"startLine":161}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a91fbbb41aae961944140222848f558f7220fc704d5d6df537225ef08070570"}},{"ruleId":"D1","level":"warning","message":{"text":"server.New (cyclomatic 25): server.New 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":"server/server.go"},"region":{"startLine":178}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb2a773850e767d81f0cac9ef2dfa31c0d2c3477ffe0810460a6f2e631bfe14b"}},{"ruleId":"D1","level":"warning","message":{"text":"server.treeContainsDisallowedNode (cyclomatic 25): server.treeContainsDisallowedNode has cyclomatic complexity 25 (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":"server/server_template.go"},"region":{"startLine":183}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e70b8b5067ad647002ea18af9fcbeec1bc9608b172046ad65187c2c89e2e7f3"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handleSubscribeHTTP (cyclomatic 23): Server.handleSubscribeHTTP has cyclomatic complexity 23 (threshold 15). Of this number, 17 points are the body\u0027s own statements and 6 belong to 2 function literals inside it that branch. 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":"server/server.go"},"region":{"startLine":1410}}}],"partialFingerprints":{"codehealthFindingId/v1":"0d96c0743cfce9c5726f33e1eb4ece3c5a45d8986be641212aa6094c69840761"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.transformBodyJSON (cyclomatic 20): Server.transformBodyJSON has cyclomatic complexity 20 (threshold 15). Most of this is not in the body itself: 1 of the 20 points is its own statement and the rest belongs to one function literal inside it that branches (line 2018). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":2017}}}],"partialFingerprints":{"codehealthFindingId/v1":"57be01fc29cb7e48169c4658fdafb737cd21948336c47765362bdb08c6d4d0be"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handleUsersUpdate (cyclomatic 20): Server.handleUsersUpdate has cyclomatic complexity 20 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_admin.go"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ac97f276fc85f0480ecbb96b63458d770e31890fad2eb646de01f7fbd60b072"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execTierAdd (cyclomatic 18): cmd.execTierAdd has cyclomatic complexity 18 (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":"cmd/tier.go"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"64c75bd83d3d6e07f0e444a06ec1aefbeb308ff37b68036f2c67c509e750cc0b"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execTokenList (cyclomatic 18): cmd.execTokenList has cyclomatic complexity 18 (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":"cmd/token.go"},"region":{"startLine":165}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac68ed90841452dd1f43cddcffe572c0ca986164f51997cc282b368579542260"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.Run (cyclomatic 18): Server.Run has cyclomatic complexity 18 (threshold 15). Of this number, 15 points are the body\u0027s own statements and 3 belong to one function literal inside it that branches. 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":"server/server.go"},"region":{"startLine":349}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf02867a63fd378e6caee8003bb5b724db0b62d964c30a33230a26bd0517c857"}},{"ruleId":"D1","level":"warning","message":{"text":"main.importMessages (cyclomatic 18): main.importMessages has cyclomatic complexity 18 (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":"tools/pgimport/main.go"},"region":{"startLine":723}}}],"partialFingerprints":{"codehealthFindingId/v1":"313bdf6456df53d9e05d0901e2c92d6ddafb0feef059e023ff673fdf6aa67737"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.execSubscribe (cyclomatic 17): cmd.execSubscribe has cyclomatic complexity 17 (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":"cmd/subscribe.go"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"5dd6ef75baffd04755bc66e66293d904b5faf684d2dabd2575a0209fc2c4a7b2"}},{"ruleId":"D1","level":"warning","message":{"text":"schema.Migrate (cyclomatic 17): schema.Migrate has cyclomatic complexity 17 (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":"db/schema/schema.go"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"11ebba779ab491eff13f1fb0ced760a9c34dc9a1363df1bc09ccf1507410d54b"}},{"ruleId":"D1","level":"warning","message":{"text":"Server.handleAccountBillingSubscriptionCreateSuccess (cyclomatic 17): Server.handleAccountBillingSubscriptionCreateSuccess has cyclomatic complexity 17 (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":"server/server_payments.go"},"region":{"startLine":185}}}],"partialFingerprints":{"codehealthFindingId/v1":"85a46b271e9cfca3ebf7c5fa31397089b4ae9648d70e09e0973b180971717323"}},{"ruleId":"D1","level":"warning","message":{"text":"main.verifySampledMessages (cyclomatic 17): main.verifySampledMessages has cyclomatic complexity 17 (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":"tools/pgimport/main.go"},"region":{"startLine":1095}}}],"partialFingerprints":{"codehealthFindingId/v1":"8bcec8134839ffd1dd5bac58c65898b5ab2fdcc43c14e135d27390fa657ab683"}},{"ruleId":"D1","level":"warning","message":{"text":"Manager.maybeProvisionUsers (cyclomatic 17): Manager.maybeProvisionUsers has cyclomatic complexity 17 (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":"user/manager.go"},"region":{"startLine":1863}}}],"partialFingerprints":{"codehealthFindingId/v1":"76996c2f651664e2c16f51e506455fb5f4a2eef4d30c33660ad3a095389bd225"}},{"ruleId":"D1","level":"warning","message":{"text":"handleClick (cyclomatic 17): handleClick has cyclomatic complexity 17 (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":"web/public/sw.js"},"region":{"startLine":287}}}],"partialFingerprints":{"codehealthFindingId/v1":"e5f40909df72999716c34e1b11cd2df5f7107d0985d4a4d088b58e18077b6df9"}},{"ruleId":"D1","level":"warning","message":{"text":"cmd.showUsers (cyclomatic 16): cmd.showUsers has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/access.go"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"8decab62037617f5574cfd64963f199494454f42d8fc5b2b237f5b00330c1d8d"}},{"ruleId":"D1","level":"warning","message":{"text":"main.importWebPush (cyclomatic 16): main.importWebPush has cyclomatic complexity 16 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":838}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a96b6648ff39db03b88f92ed463eb50ac241fc8487b812390f724fc92b39ab1"}},{"ruleId":"D1","level":"warning","message":{"text":"accessCache.Reload (cyclomatic 16): accessCache.Reload has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/access_cache.go"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"796ba53245ae7c3d00c72983b838df50fb1a2756d6f0b3ebc7b18af35b1ba6b8"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleInternal (cognitive 141): Server.handleInternal has cognitive complexity 141 (threshold 15). Drivers by points: boolean chains 75, if/else 66. To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":563}}}],"partialFingerprints":{"codehealthFindingId/v1":"102154833b7d9bc2dfe85491b2448ed4d91f86b44e88816ed0c2268bbd538296"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execServe (cognitive 134): cmd.execServe has cognitive complexity 134 (threshold 15). Drivers by points: if/else 85, boolean chains 47, loops 2 (nesting depth added 5). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/serve.go"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"e9cdfab695279d07fb83ffdd646d563daadda84ff0a8cef1c16230039e00abbc"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleAccountGet (cognitive 72): Server.handleAccountGet has cognitive complexity 72 (threshold 15). Drivers by points: if/else 59, loops 13 (nesting depth added 44). 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":"server/server_account.go"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"579499308bec2efdd93558d345844273191520674cc13c9249638702fb67e24d"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.parsePublishParams (cognitive 66): Server.parsePublishParams has cognitive complexity 66 (threshold 15). Drivers by points: if/else 56, boolean chains 10 (nesting depth added 19). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1128}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e441ea56a0bc8e81bf3bd2137b7960993da60de80ee4280ebd47272752dd3ec"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execPublish (cognitive 56): cmd.execPublish has cognitive complexity 56 (threshold 15). Drivers by points: if/else 54, boolean chains 2 (nesting depth added 13). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/publish.go"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"3e533751bc873814b3081c7f786e657c4232f9b3ad349c1c2da47814f1ecb2b7"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handlePublishInternal (cognitive 40): Server.handlePublishInternal has cognitive complexity 40 (threshold 15). Drivers by points: if/else 36, boolean chains 4 (nesting depth added 8). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":858}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd3c78dd2771e8063a7ddeb18527dea2f2f11b9d0177147bf61b4f16f2833bda"}},{"ruleId":"D2","level":"warning","message":{"text":"server.New (cognitive 39): server.New has cognitive complexity 39 (threshold 15). Drivers by points: if/else 30, loops 7, boolean chains 2 (nesting depth added 13). 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":"server/server.go"},"region":{"startLine":178}}}],"partialFingerprints":{"codehealthFindingId/v1":"15b70eca93bac784ba3f294ed54cfa7935085201a7a412e9b2fdf92724f3d6b2"}},{"ruleId":"D2","level":"warning","message":{"text":"Manager.maybeProvisionUsers (cognitive 39): Manager.maybeProvisionUsers has cognitive complexity 39 (threshold 15). Drivers by points: if/else 35, boolean chains 2, loops 2 (nesting depth added 22). 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":"user/manager.go"},"region":{"startLine":1863}}}],"partialFingerprints":{"codehealthFindingId/v1":"d89b231724d2e8d05d3752972ea62b1fd1bb8a644f91ed5ba5512fcd39eb0efb"}},{"ruleId":"D2","level":"warning","message":{"text":"main.execImport (cognitive 36): main.execImport has cognitive complexity 36 (threshold 15). Drivers by points: if/else 35, boolean chains 1 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0c1c669dca8693d32dc2063a70c86b4242315d29bd17575b311afc0657e5d7d"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleSubscribeWS (cognitive 34): Server.handleSubscribeWS has cognitive complexity 34 (threshold 15). Drivers by points: if/else 20, loops 9, match/switch 4, boolean chains 1 (nesting depth added 8). Most of this is not in the body itself: 15 of the 34 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 1570, 1548, 1599, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1510}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c3e581a52b4bb7341f1b56df83d3299179b2e539e18310170708623fa2fa2aa"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.showUsers (cognitive 33): cmd.showUsers has cognitive complexity 33 (threshold 15). Drivers by points: if/else 29, loops 4 (nesting depth added 15). 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":"cmd/access.go"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5a6555b7ccebac14f16ab7fb75319bb9991471e7ab55d86d699acd29f9c4a3b"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execTierChange (cognitive 32): cmd.execTierChange has cognitive complexity 32 (threshold 15). Drivers by points: if/else 30, boolean chains 2 (nesting depth added 5). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/tier.go"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"09269b4dc624790c6991477081192b5c917a10d3049514144152d501e0861937"}},{"ruleId":"D2","level":"warning","message":{"text":"accessCache.Reload (cognitive 30): accessCache.Reload has cognitive complexity 30 (threshold 15). Drivers by points: if/else 26, loops 4 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/access_cache.go"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"c456d0e446e6062649b726382f077368d0e69ee02b635bd3ce1c71ebd226f783"}},{"ruleId":"D2","level":"warning","message":{"text":"handleClick (cognitive 30): handleClick has cognitive complexity 30 (threshold 15). 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":"web/public/sw.js"},"region":{"startLine":287}}}],"partialFingerprints":{"codehealthFindingId/v1":"b308d543620856ee9c7d68521d612ec3ad0babc2e9c8ecebcb2db5f417422ea9"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execTokenList (cognitive 29): cmd.execTokenList has cognitive complexity 29 (threshold 15). Drivers by points: if/else 24, loops 3, boolean chains 2 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/token.go"},"region":{"startLine":165}}}],"partialFingerprints":{"codehealthFindingId/v1":"c66910e14d3d455f5f5d4bb1271b13b32087e57bf7b246ef35987e176186e4ed"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleSubscribeHTTP (cognitive 29): Server.handleSubscribeHTTP has cognitive complexity 29 (threshold 15). Drivers by points: if/else 19, loops 8, match/switch 2 (nesting depth added 8). Of this number, 23 points are the body\u0027s own statements and 6 belong to 2 function literals inside it that branch. To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1410}}}],"partialFingerprints":{"codehealthFindingId/v1":"d1f9ae8c9f74006e25c121fd8359c836cbebe94015b0b4d869aa692c4cd5e692"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleUsersUpdate (cognitive 29): Server.handleUsersUpdate has cognitive complexity 29 (threshold 15). Drivers by points: if/else 27, boolean chains 2 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_admin.go"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"824b04ee3e311e87ce67409dd268a3493c7e3689f6315f369810aea6e25cd83f"}},{"ruleId":"D2","level":"warning","message":{"text":"main.importMessages (cognitive 29): main.importMessages has cognitive complexity 29 (threshold 15). Drivers by points: if/else 28, loops 1 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":723}}}],"partialFingerprints":{"codehealthFindingId/v1":"6ecd83a612db4727b962d383c1e5b1f295dab9e30d2dcd1d395c5eeb33dbe491"}},{"ruleId":"D2","level":"warning","message":{"text":"main.verifyContent (cognitive 24): main.verifyContent has cognitive complexity 24 (threshold 15). Drivers by points: if/else 21, loops 3 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":1023}}}],"partialFingerprints":{"codehealthFindingId/v1":"9592a60cc3a37d38abba5da6370c693b6e4fe8097beb0f0c992c42ba6cb57c13"}},{"ruleId":"D2","level":"warning","message":{"text":"server.treeContainsDisallowedNode (cognitive 23): server.treeContainsDisallowedNode has cognitive complexity 23 (threshold 15). Drivers by points: if/else 13, loops 6, boolean chains 3, match/switch 1 (nesting depth added 11). 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":"server/server_template.go"},"region":{"startLine":183}}}],"partialFingerprints":{"codehealthFindingId/v1":"12d9de48fe7d1d72f53e843112099ea8a214240202f951644f7e1ff730d0e9de"}},{"ruleId":"D2","level":"warning","message":{"text":"main.streamSubscription (cognitive 23): main.streamSubscription has cognitive complexity 23 (threshold 15). Drivers by points: if/else 18, loops 3, match/switch 2 (nesting depth added 13). 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":"tools/loadtest/main.go"},"region":{"startLine":383}}}],"partialFingerprints":{"codehealthFindingId/v1":"7bdc6afe3fd8f4baa9373c380f32b818608afbeb5f5154293a806a619272ac9e"}},{"ruleId":"D2","level":"warning","message":{"text":"main.verifySampledMessages (cognitive 22): main.verifySampledMessages has cognitive complexity 22 (threshold 15). Drivers by points: if/else 20, boolean chains 1, loops 1 (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":"tools/pgimport/main.go"},"region":{"startLine":1095}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff258fda224aed5abcd113cc608e673bd0c41fa4f9e7037155da826459dfe3fc"}},{"ruleId":"D2","level":"warning","message":{"text":"schema.Migrate (cognitive 21): schema.Migrate has cognitive complexity 21 (threshold 15). Drivers by points: if/else 19, boolean chains 1, loops 1 (nesting depth added 5). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"db/schema/schema.go"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e44b450fa22e0867e5d805f9ba743ac8a2fe85205fe0e8d0eacaff7f5729775"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.Run (cognitive 21): Server.Run has cognitive complexity 21 (threshold 15). Drivers by points: if/else 21 (nesting depth added 4). Of this number, 14 points are the body\u0027s own statements and 7 belong to one function literal inside it that branches. To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":349}}}],"partialFingerprints":{"codehealthFindingId/v1":"022630e7ff59362e40ffc4a643924bb3fcbe4b82e586963b396d38f1d3c9e207"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execSubscribe (cognitive 20): cmd.execSubscribe has cognitive complexity 20 (threshold 15). Drivers by points: if/else 17, boolean chains 3 (nesting depth added 3). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/subscribe.go"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"56a142d0ba1f8f1da56412626856a41fb6d179050256ddcd90aaaab54cf1f1e4"}},{"ruleId":"D2","level":"warning","message":{"text":"Cache.addMessages (cognitive 20): Cache.addMessages has cognitive complexity 20 (threshold 15). Drivers by points: if/else 18, boolean chains 1, loops 1 (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"message/cache.go"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"327998cdd33d8dfd34b3b6949e981a7cedbd89d548d7493df65fc5dfe37b3737"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.transformBodyJSON (cognitive 20): Server.transformBodyJSON has cognitive complexity 20 (threshold 15). Drivers by points: if/else 20 (nesting depth added 1). Most of this is not in the body itself: 0 of the 20 points are its own statements and the rest belongs to one function literal inside it that branches (line 2018). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":2017}}}],"partialFingerprints":{"codehealthFindingId/v1":"e8744595056595093d3a1b54ec67ab2aece91d9a24c66221dbaf278e508c6864"}},{"ruleId":"D2","level":"warning","message":{"text":"Manager.maybeProvisionTokens (cognitive 20): Manager.maybeProvisionTokens has cognitive complexity 20 (threshold 15). Drivers by points: if/else 12, loops 7, boolean chains 1 (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":"user/manager.go"},"region":{"startLine":1938}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a860e40bb7c3ca6c768493e5cab0ffafb289aa039a616b4742db904e133cdf1"}},{"ruleId":"D2","level":"warning","message":{"text":"main.wsSubscription (cognitive 20): main.wsSubscription has cognitive complexity 20 (threshold 15). Drivers by points: if/else 12, match/switch 4, loops 3, boolean chains 1 (nesting depth added 10). Of this number, 15 points are the body\u0027s own statements and 5 belong to one function literal inside it that branches. 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":"tools/loadtest/main.go"},"region":{"startLine":435}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c2cf33fac42e8400dab3234aad02fbee83572a204e0ad440301daa5aad3c193"}},{"ruleId":"D2","level":"warning","message":{"text":"main.importWebPush (cognitive 19): main.importWebPush has cognitive complexity 19 (threshold 15). Drivers by points: if/else 17, loops 2 (nesting depth added 4). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":838}}}],"partialFingerprints":{"codehealthFindingId/v1":"f84af396bb0920e69e0cf16e23f200f59d64017f9438902ff4df3012a79b097e"}},{"ruleId":"D2","level":"warning","message":{"text":"action.populateAction (cognitive 18): action.populateAction has cognitive complexity 18 (threshold 15). Drivers by points: if/else 11, boolean chains 5, match/switch 2 (nesting depth added 3). 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":"action/action.go"},"region":{"startLine":161}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ab13338501ae2815d4cd284811920ffc3ffa23ca25ef48fd8ea87fd5b6150db"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.parseAccess (cognitive 18): cmd.parseAccess has cognitive complexity 18 (threshold 15). Drivers by points: if/else 17, loops 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/serve.go"},"region":{"startLine":647}}}],"partialFingerprints":{"codehealthFindingId/v1":"3e2d1d5405e7ce1218c26359d634c26b046ca59c17fcc680f5606eef7a5ed8ed"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execTierAdd (cognitive 18): cmd.execTierAdd has cognitive complexity 18 (threshold 15). Drivers by points: if/else 16, boolean chains 2 (nesting depth added 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":"cmd/tier.go"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff8d316186e6fe5445392ab69d42fdd874fcd550445d35b5e97291baf5d5f511"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleError (cognitive 18): Server.handleError has cognitive complexity 18 (threshold 15). Drivers by points: if/else 15, boolean chains 3 (nesting depth added 4). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":516}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2edca73f512c00bb295d25716000f25a55d1be994976c8349fc2c127c794b88"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.maybeSetRateVisitors (cognitive 18): Server.maybeSetRateVisitors has cognitive complexity 18 (threshold 15). Drivers by points: if/else 15, loops 2, boolean chains 1 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1669}}}],"partialFingerprints":{"codehealthFindingId/v1":"cfb2a5a16bf4d82aded93178373e63306a8705634f0878a2d14063853d31c077"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleAccountCreate (cognitive 18): Server.handleAccountCreate has cognitive complexity 18 (threshold 15). Drivers by points: if/else 16, boolean chains 2 (nesting depth added 4). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_account.go"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"3790c342859a18d0a971e848100cf50afff3297716a4e8730058e3f23315deda"}},{"ruleId":"D2","level":"warning","message":{"text":"sprig.strslice (cognitive 18): sprig.strslice has cognitive complexity 18 (threshold 15). Drivers by points: if/else 10, loops 5, match/switch 3 (nesting depth added 11). 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":"util/sprig/strings.go"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"23a7e19c4f3e12ffcbbd8d228302fe4d5d671bf76db83e4969df6bb599c1446b"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.doSubscribe (cognitive 17): cmd.doSubscribe has cognitive complexity 17 (threshold 15). Drivers by points: if/else 13, loops 4 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/subscribe.go"},"region":{"startLine":178}}}],"partialFingerprints":{"codehealthFindingId/v1":"396cfc746239ce9ecbb924a2c0a42303dea0bacf4d04cd40f274d7eec73bb983"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleFile (cognitive 17): Server.handleFile has cognitive complexity 17 (threshold 15). Drivers by points: if/else 16, boolean chains 1 (nesting depth added 3). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":751}}}],"partialFingerprints":{"codehealthFindingId/v1":"7380f758152e24c8211171d62727c5921eaf15a262ef4b6fe346e076863fb485"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.dispatch (cognitive 17): Server.dispatch has cognitive complexity 17 (threshold 15). Drivers by points: if/else 12, boolean chains 5 (nesting depth added 3). Of this number, 14 points are the body\u0027s own statements and 3 belong to one function literal inside it that branches. 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":"server/server.go"},"region":{"startLine":826}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b920e4093c0525c4880eb4ca7dbf7cd9513ac8e532ae9b2ca6138c81385aa82"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.execManager (cognitive 17): Server.execManager has cognitive complexity 17 (threshold 15). Drivers by points: if/else 16, loops 1 (nesting depth added 6). Most of this is not in the body itself: 7 of the 17 points are its own statements and the rest belongs to one function literal inside it that branches (line 30). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_manager.go"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f029e0047be65338dd44502d263195c95de3e68340bbed94f38566cd3911fb0"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.renderTemplateFromFile (cognitive 17): Server.renderTemplateFromFile has cognitive complexity 17 (threshold 15). Drivers by points: if/else 17 (nesting depth added 5). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_template.go"},"region":{"startLine":75}}}],"partialFingerprints":{"codehealthFindingId/v1":"b7f942fab56d7923805aab75b552d7c4c974c8a8cb22a7312e13d5230b4a8bc1"}},{"ruleId":"D2","level":"warning","message":{"text":"cmd.execUserResetPass (cognitive 16): cmd.execUserResetPass has cognitive complexity 16 (threshold 15). Drivers by points: if/else 15, boolean chains 1 (nesting depth added 2). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/user.go"},"region":{"startLine":319}}}],"partialFingerprints":{"codehealthFindingId/v1":"d808fad1b15cc80f4b905608de701a7dd05335bd4662e4c93cf0d2912ebcdb8d"}},{"ruleId":"D2","level":"warning","message":{"text":"Client.listMultipartUploads (cognitive 16): Client.listMultipartUploads has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, loops 3 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"s3/client_multipart.go"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"2504caba73b605b15607d802f0e7f2066bb6f9c56b89e599fb0051f219e0aef9"}},{"ruleId":"D2","level":"warning","message":{"text":"server.toFirebaseMessage (cognitive 16): server.toFirebaseMessage has cognitive complexity 16 (threshold 15). Drivers by points: if/else 15, match/switch 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_firebase.go"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6076a160cd2634c92664564f964d172e328c0fe435b6ca32d46b89fe573b42c"}},{"ruleId":"D2","level":"warning","message":{"text":"server.readMultipartMailBodyParts (cognitive 16): server.readMultipartMailBodyParts has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, boolean chains 1, loops 1 (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/smtp_server.go"},"region":{"startLine":276}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b75af565cea2e26697be33dcf3d13c7716cd2120aa1b48066f5415406dff2a5"}},{"ruleId":"D2","level":"warning","message":{"text":"server.extractIPAddressFromHeader (cognitive 16): server.extractIPAddressFromHeader has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, loops 2, boolean chains 1 (nesting depth added 6). Of this number, 13 points are the body\u0027s own statements and 3 belong to one function literal inside it that branches. 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":"server/util.go"},"region":{"startLine":111}}}],"partialFingerprints":{"codehealthFindingId/v1":"236669aa9e7b23dd696280de8002df722bcc1a9636fb9314e1a1325c54b3446f"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleActionMessage (cognitive 16): Server.handleActionMessage has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, boolean chains 2 (nesting depth added 4). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1023}}}],"partialFingerprints":{"codehealthFindingId/v1":"dc1cb900dc1f6dddb28992187795c231a21ebd355034eedea6936bf270119c5f"}},{"ruleId":"D2","level":"warning","message":{"text":"Server.handleUsersAdd (cognitive 16): Server.handleUsersAdd has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, boolean chains 3 (nesting depth added 2). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_admin.go"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"50161c9a9cef66b27429555fe0b8dde42ba8fa26dd01058e8cb570bcac242463"}},{"ruleId":"D2","level":"warning","message":{"text":"Manager.AddReservation (cognitive 16): Manager.AddReservation has cognitive complexity 16 (threshold 15). Drivers by points: if/else 15, boolean chains 1 (nesting depth added 6). Most of this is not in the body itself: 3 of the 16 points are its own statements and the rest belongs to one function literal inside it that branches (line 901). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/manager.go"},"region":{"startLine":897}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b26c3f35c3666d4c465c6fb246c67969f6c2f369a4759e45fe15cc8d8476e0e"}},{"ruleId":"D2","level":"warning","message":{"text":"util.ReadPassword (cognitive 16): util.ReadPassword has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, boolean chains 1, loops 1 (nesting depth added 5). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"util/util.go"},"region":{"startLine":301}}}],"partialFingerprints":{"codehealthFindingId/v1":"662df4c71fecde47fa0b5d4b565f3f04591b9b75eb80073cf68ec6698fa1a8b4"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Server: TooManyMethods \u2014 155 methods, declared across 11 files: server/server.go (59), server/server_account.go (32), server/server_middleware.go (15), server/server_payments.go (12), \u002B7 more file(s). That list is where to read them, not a suggestion to split the file: the members belong to the type wherever they are declared, so moving them between files leaves the count unchanged. 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":"server/server.go"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"63963d5ca4be146122467be9ebcbe195afaaa5c60dcb09c0f117a2e884064cf7"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Manager: TooManyMethods \u2014 110 methods. 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":"user/manager.go"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd2bc54c8c2af443a7398342db0829a9a2986378623418f97834d0c905b8bc5f"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: server/server.go: FileTooLong \u2014 1519 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ae1a94109b305dacc30a7b8a22ee24b1a6c658512279cf1478e57603ed29c6b"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: user/manager.go: FileTooLong \u2014 1201 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/manager.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c08d86af7c566043d2e8ca5ed3fb4f5d65b6f3c0643ee19dde14de6b043d3a4"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: pgimport/main.go: FileTooLong \u2014 713 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ecc47ac3f929fdce02b031729adfdf6335c750f596b35660bdc99a1d8745474"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: server/server_account.go: FileTooLong \u2014 676 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_account.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"37b6d426d6291907ce496be4ea7b7e39f77c230576277d54a3f2e25c88b7b885"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: cmd/serve.go: FileTooLong \u2014 606 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/serve.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff22f098a1f9fbe8989131a7d75e92ed7269f58453309fdaa48c47b556cb5b2b"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: visitor: TooManyMethods \u2014 31 methods. 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":"server/visitor.go"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"203d37c1774767c0a96b8bd3575eb1319d74601169d1507d64abecccd8dc7e69"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): tools/pgimport/main.go:242-255 | tools/pgimport/main.go:329-342 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060tools/pgimport/main.go:242\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. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tools/pgimport/main.go"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"08d6ea60c087c701d9b4fc1a822a024012f1376fc8c7d13ddb207fdaedeb1c17"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): user/manager.go:1230-1241 | user/manager.go:1249-1260 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060user/manager.go:1230\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. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/manager.go"},"region":{"startLine":1230}}}],"partialFingerprints":{"codehealthFindingId/v1":"24f4747db596b3b9fe7efed6c9f6295cdacef12ce5514bab1266b6684b83e138"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): message/cache.go:556-566 | user/manager.go:1570-1580 \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":"message/cache.go"},"region":{"startLine":556}}}],"partialFingerprints":{"codehealthFindingId/v1":"c3022dab50e2fbb20fbf870ea150912979f6fad06261598d2b2bfb8e176f41ee"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): server/server_payments.go:122-132 | server/server_payments.go:283-293 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/server_payments.go:122\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":"server/server_payments.go"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"23a8c58f266e77361e13911f226ddec9d4f8b6d0fcf03e456b85520593161ddf"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 4): cmd/user.go:272-281 | cmd/user.go:297-306 | cmd/user.go:380-389 | cmd/user.go:417-426 \u2014 all 4 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060cmd/user.go:272\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. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/user.go"},"region":{"startLine":272}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ed8f01dd66cb28d349f56bfe77c58f777b76d9cd59b41d47fc01fce5506f844"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): tools/pgimport/main.go:406-415 | cmd/config_loader.go:43-52 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060tools/pgimport/main.go:406\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":"tools/pgimport/main.go"},"region":{"startLine":406}}}],"partialFingerprints":{"codehealthFindingId/v1":"3477f9654d7d7e9428af80e63a2892def310840a78b7b0672114d2163ecd49b7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): cmd/publish.go:177-186 | cmd/subscribe.go:118-127 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060cmd/publish.go:177\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":"cmd/publish.go"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"b2bba54a503a00e7a19001e36fdb3533d1e62f6bbdcc8ea0db1f0ea1125fed1d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): server/server_account.go:313-322 | server/server_account.go:354-363 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/server_account.go:313\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. 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":"server/server_account.go"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"54d6a7eb3aa6100d6d2f71b195c1176b239d2300c789832a7c5f570275022d3b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): server/server.go:1472-1480 | server/server.go:1621-1629 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/server.go:1472\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. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1472}}}],"partialFingerprints":{"codehealthFindingId/v1":"79622f076c995e8ef4c8143570a701405777ddf030cf9b21f774ed029d555d50"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): user/manager.go:668-676 | user/manager.go:1427-1435 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/manager.go"},"region":{"startLine":668}}}],"partialFingerprints":{"codehealthFindingId/v1":"364c20bcad385b3a3cee3e77e8450c6d1cc3d2f2ca85bc7427286dc313ddc685"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): util/sprig/list.go:202-210 | util/sprig/list.go:235-243 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060util/sprig/list.go:202\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":"util/sprig/list.go"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b86c9dc334205001aebfd9578c098048350c8d65f0eda28aa1bacf8df4ef35b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): util/sprig/list.go:312-320 | util/sprig/list.go:345-353 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060util/sprig/list.go:312\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":"util/sprig/list.go"},"region":{"startLine":312}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d5f86910423d19ff02a40f1bbacccd993d1ac63cbedd0ad957143a2c21efa9d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): server/server_payments.go:71-78 | server/server_account.go:78-85 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_payments.go"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"87e0686dce38332e71d7858365326b2ceae7410a88b45f58f8af7cb37dec8219"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): server/server_account.go:741-748 | server/server_account.go:762-769 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_account.go"},"region":{"startLine":741}}}],"partialFingerprints":{"codehealthFindingId/v1":"92892eee12a80a820fc2fee0d499c2476a098b906c74953f9d181b84b836a5ae"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 3): util/sprig/list.go:34-40 | util/sprig/list.go:63-69 | util/sprig/list.go:282-289 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060util/sprig/list.go:34\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":"util/sprig/list.go"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"632baae49cb00b5ad6948e97ed3d918296982074f45f012e45fb0f3a70716e3b"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/server.go: server/server.go changed 27 times in last 90 days, max complexity 145. 2 of those changes were fix/bug commits, and the other 25 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f67bc877ed54df09f812b32aa4c1c6a36a97f4554c1e3b93af74259e5f348f68"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/server_account.go: server/server_account.go changed 21 times in last 90 days, max complexity 28. 1 of those changes was a fix/bug commit, and the other 20 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_account.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"468724206a9d1813f9f6460ea06e6f879dde88f700e4af295c0cca89ee99f720"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: cmd/serve.go: cmd/serve.go changed 4 times in last 90 days, max complexity 144. 1 of those changes was a fix/bug commit, and the other 3 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/serve.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"52dc84083c374abeb72753a2e529b4fdd3d2a29a30dc19dddcfcefb67d8334bd"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/src/components/Account.jsx: web/src/components/Account.jsx changed 16 times in last 90 days, max complexity 30. 1 of those changes was a fix/bug commit, and the other 15 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/components/Account.jsx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7afa106a7b063edd1c2ad8715e1459faf62f9590d38e193a53a2d044b8364873"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: user/manager.go: user/manager.go changed 28 times in last 90 days, max complexity 17. 1 of those changes was a fix/bug commit, and the other 27 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/manager.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c35bc51dec8fd7ee4c1946b90973c9ed3d1eb4f352335b77d5cb3c77603928be"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: user/access_cache.go: user/access_cache.go changed 14 times in last 90 days, max complexity 16. 1 of those changes was a fix/bug commit, and the other 13 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"user/access_cache.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a81836c37d50e6bd93f39a2431babdb9f207f387df96a50919027a0fd064c74"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/src/components/PublishDialog.jsx: web/src/components/PublishDialog.jsx changed 6 times in last 90 days, max complexity 28. 2 of those changes were fix/bug commits, and the other 4 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/components/PublishDialog.jsx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7f0c8218b7ceb978ea10289ad7daf85577fe8c0e8f8357070c93998961cd1310"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/server_template.go: server/server_template.go changed 5 times in last 90 days, max complexity 25. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server_template.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"702e663e15b48a21aeace39d1b95b879d5ccef3cd4dce3787983beaf882f1e44"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/types.go: server/types.go changed 8 times in last 90 days, max complexity 15. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/types.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"614a17405a3901f11d390762ea8938831f6b9ea45c98d8af2634a52f495a3775"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/src/components/Notifications.jsx: web/src/components/Notifications.jsx changed 6 times in last 90 days, max complexity 18. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/components/Notifications.jsx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a336ad29f22027d80c8b844cb658aadd1b4c90e43c9c23b525a322674453fd7"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 50 significant file(s) lose their only recent owner: user/manager.go, tools/pgimport/main.go, server/server_account.go, server/server_payments.go, user/manager_sqlite.go, user/manager_postgres.go, server/visitor.go, server/config.go (\u002B42 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"f42b2b3db38a31ad6afe276d6c4cb0305a3c07d5083b7e81626a9bef405ee870"}},{"ruleId":"D19","level":"note","message":{"text":"The \u0027All the things\u0027 subsection lists features like behind a proxy, Firebase, cache, attachments, SMTP publishing \u0026 receiving, but no link or description explains what each means.: Link each feature in All the things to its own short explanation so readers can find them without scanning through the text."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"docs/config.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"97f05e3f95bdc355f254fc5e12149fe924d6f860c80f572a0c2b3a465b1494c3"}},{"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":"bbb570fc79beb12fb2dc4f25038ca4678e631704c3c7e302d8cfd4bfec761b0a"},"taxa":[{"id":"CWE-338","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":"e9fc05a47234e5ba05fc0b823e6bf0b947455c16776674fd7b676923d8c8dbb7"},"taxa":[{"id":"CWE-328","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":"47c6737c0b4cca92fcb829cd1c57a2fb9fa1f73ae85aa533f14e11206bbd7db0"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","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":"fe2d688610b51e0a5a02c7c5dd1724a4986e40f2689fb84510e2ec0164c27d6a"}},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"becbc5502f94878197b05d70c7ef15f1353fd99a90efcb824b9c43f11ae5ce7a"}},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ae2dfb06bfe0dcadfc949c881aa237cb2a4aac5b1cc69adf101d9e003549ef46"}},{"ruleId":"D31","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":"1f51a163f26d94ce6b6f473117dde579d091c8cbf9e59810475c94d9fc921120"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ConnectionManager.js \u2194 App.jsx: \u0060web/src/app/ConnectionManager.js\u0060 and \u0060web/src/components/App.jsx\u0060 change together 72% of the time (13 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/app/ConnectionManager.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6961d31eb7c6d70d06c7bf3691dcc4a61f49f5d14d4fc8cc3adfa44a5d98baad"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: server.go \u2194 routes.js: \u0060server/server.go\u0060 and \u0060web/src/components/routes.js\u0060 change together 67% of the time (12 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) and are written in DIFFERENT LANGUAGES, so no import can join them and they cannot be co-located into one unit \u2014 they compile and ship as separate artifacts. What binds them is a CONTRACT across that boundary \u2014 an event or message name, a route, a serialised shape \u2014 that each side currently spells out on its own, which is exactly why a change to one drags the other. Declare that contract once where both sides read it (a shared schema, a generated constants file, an interface-definition file) so a change on one side fails the other\u0027s build instead of drifting silently; where the surface is too small to be worth that, name the counterpart in a comment on both sides so the next reader finds it. There is nothing here to merge."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"667bf3919dc8e8b96401752735791c844d73af30ce0139711cd7aa26d754a8dc"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: Connection.js \u2194 App.jsx: \u0060web/src/app/Connection.js\u0060 and \u0060web/src/components/App.jsx\u0060 change together 67% of the time (12 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/app/Connection.js"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3ed17793d1d828a768314d49bd923cd15dac55bad7e8ad18907a15855b854e8a"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: tier.go \u2194 config.go: \u0060cmd/tier.go\u0060 and \u0060server/config.go\u0060 change together 55% of the time (6 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/tier.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3be606b9d7abe122d2bb3ca4f9bd2f35329cea79580f8e0652a4ad2e9183c0c"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: server.go \u2194 config.js: \u0060server/server.go\u0060 and \u0060web/public/config.js\u0060 change together 54% of the time (13 of the 24 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) and are written in DIFFERENT LANGUAGES, so no import can join them and they cannot be co-located into one unit \u2014 they compile and ship as separate artifacts. What binds them is a CONTRACT across that boundary \u2014 an event or message name, a route, a serialised shape \u2014 that each side currently spells out on its own, which is exactly why a change to one drags the other. Declare that contract once where both sides read it (a shared schema, a generated constants file, an interface-definition file) so a change on one side fails the other\u0027s build instead of drifting silently; where the surface is too small to be worth that, name the counterpart in a comment on both sides so the next reader finds it. There is nothing here to merge."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/server.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a109902527070318f975aab3af93c3fe97c09e30020c807b0e326f31e7d28d8f"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: config.go \u2194 server_twilio.go: \u0060server/config.go\u0060 and \u0060server/server_twilio.go\u0060 change together 50% of the time (11 of the 22 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/config.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd7fb51c99644e35b174dfb7edd9f65a6ba66037e56acb48965bf98ffbb136b7"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: Preferences.jsx \u2194 SubscriptionPopup.jsx: \u0060web/src/components/Preferences.jsx\u0060 and \u0060web/src/components/SubscriptionPopup.jsx\u0060 change together 50% of the time (9 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) with no explicit dependency between them. They sit in the same directory, but in this ecosystem each file is its own module \u2014 a sibling reference still needs an import \u2014 so the missing import edge is real: the coupling runs through shared behaviour, not a declared dependency. If they duplicate structure, extract the common part into one unit; otherwise the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/src/components/Preferences.jsx"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"525e8fcb5be70e18437f5a35bfb531db530889631f9b28d33b0f9b591c65c3ad"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: config.go \u2194 time.go: \u0060server/config.go\u0060 and \u0060util/time.go\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) 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/config.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8cb458d403b3a48e661dfdace421189dcc7522bbfe1da13c689122e735365254"}},{"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":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"}},{"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":"9658270ba49f37ed04e99ab1263b6393cd42aed8bdfb7aafd9fa1070f5491895"}},{"ruleId":"D38","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":"196e767899709fd49cb9b76f1686e6b44bd30d05446047c4892dec76125d87f9"}},{"ruleId":"D38","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":"355bedf9e40482ac137be454010e5c5e1b90059c53b8ebecea862385d9666a1f"}},{"ruleId":"D38","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":"de29dcd1988e28b3d16ad39acef38672e2f985e1705d5603e9a2c5570096bde6"}},{"ruleId":"D38","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":"50c379534161bb30611d421e82af3666b891a67b9a6a35df5a4fc4b4977e653b"}},{"ruleId":"D38","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":"57585607d4e49033f482f427d43ba3f01caab9b7d1cbe28bddcb2019042226f8"}},{"ruleId":"D38","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":"301ea32bb265e3ad6be81006558f037018e8ce234954576e2ea9a8f048d2487a"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"id":"CWE-1059","guid":"a2381a08-60f6-9554-a8b8-f3018cfaaca5","name":"Insufficient Technical Documentation","shortDescription":{"text":"Insufficient Technical Documentation"},"helpUri":"https://cwe.mitre.org/data/definitions/1059.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-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-328","guid":"f0ffc869-97c3-dc5c-8825-be1159f1fc5f","name":"CWE-328","shortDescription":{"text":"CWE-328"},"helpUri":"https://cwe.mitre.org/data/definitions/328.html"},{"id":"CWE-338","guid":"e89b7604-c3f7-8b53-bc8c-4d4718c28617","name":"CWE-338","shortDescription":{"text":"CWE-338"},"helpUri":"https://cwe.mitre.org/data/definitions/338.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.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":17,"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."]}]}