{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"id":"D44","name":"Platform End-of-Life","shortDescription":{"text":"Platform End-of-Life"},"helpUri":"https://codehealth.canine.dev/dimensions/D44"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX3","name":"Project dependency cycles","shortDescription":{"text":"Project dependency cycles"},"helpUri":"https://codehealth.canine.dev/dimensions/AX3"},{"id":"AX4","name":"Dependency direction","shortDescription":{"text":"Dependency direction"},"helpUri":"https://codehealth.canine.dev/dimensions/AX4"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P12","name":"CI test-gate honesty","shortDescription":{"text":"CI test-gate honesty"},"helpUri":"https://codehealth.canine.dev/dimensions/P12"},{"id":"P2","name":"Observability","shortDescription":{"text":"Observability"},"helpUri":"https://codehealth.canine.dev/dimensions/P2"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P4","name":"Deployment \u0026 Rollback","shortDescription":{"text":"Deployment \u0026 Rollback"},"helpUri":"https://codehealth.canine.dev/dimensions/P4"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"P8","name":"Schema migrations","shortDescription":{"text":"Schema migrations"},"helpUri":"https://codehealth.canine.dev/dimensions/P8"},{"id":"S1","name":"Web-Security Posture","shortDescription":{"text":"Web-Security Posture"},"helpUri":"https://codehealth.canine.dev/dimensions/S1"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"ValueEvaluator.evaluate (cyclomatic 85): ValueEvaluator.evaluate has cyclomatic complexity 85 (threshold 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"63a6926290e8d5ec3caad0e6c4748591f61bdd9cc16701b2a6c2951da3fafe04"}},{"ruleId":"D1","level":"warning","message":{"text":"TypeEvaluator.evaluate (cyclomatic 84): TypeEvaluator.evaluate has cyclomatic complexity 84 (threshold 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"f5591c713f326b8728f393132845fd4fdf2cc94de8a21d0797105aab561e74cb"}},{"ruleId":"D1","level":"warning","message":{"text":"WdlNamespace.apply (cyclomatic 54): WdlNamespace.apply has cyclomatic complexity 54 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlNamespace.scala"},"region":{"startLine":198}}}],"partialFingerprints":{"codehealthFindingId/v1":"dd2c6ad3ed26b507ac9dcd4c5b553a3fcd9219020d5e576e5cdabc2311cb419d"}},{"ruleId":"D1","level":"warning","message":{"text":"CallElementToGraphNode.convert (cyclomatic 36): CallElementToGraphNode.convert has cyclomatic complexity 36 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/CallElementToGraphNode.scala"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f622ffc4d69c1907b77d4bfccb33f8249deba871da9ecf35ba06c1b6b8b07b6"}},{"ruleId":"D1","level":"warning","message":{"text":"FileEvaluator.evaluateRecursive (cyclomatic 35): FileEvaluator.evaluateRecursive has cyclomatic complexity 35 (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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0e5ddf49c557a8ca73f96b974c9a3256143b57613fb621a4ad8501aeeba6937"}},{"ruleId":"D1","level":"warning","message":{"text":"EnhancedAstNode.womValue (cyclomatic 27): EnhancedAstNode.womValue has cyclomatic complexity 27 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/AstTools.scala"},"region":{"startLine":179}}}],"partialFingerprints":{"codehealthFindingId/v1":"20af315aef15638cbf7d7b112181e03e41f0d0ba529313e8caa92312269e3c14"}},{"ruleId":"D1","level":"warning","message":{"text":"AstNodeToExpressionElement.convert (cyclomatic 27): AstNodeToExpressionElement.convert has cyclomatic complexity 27 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/ast2wdlom/AstNodeToExpressionElement.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"40967b04d8d4cb1636cf9240ce8ab7b68f919cea110f5ed268173b52de99ca0f"}},{"ruleId":"D1","level":"warning","message":{"text":"WomArrayType.coercion (cyclomatic 26): WomArrayType.coercion has cyclomatic complexity 26 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/types/WomArrayType.scala"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a1f08d62c404910c79e175fba3e5b3f54319ab54cf7c1d755f2c7a00180ff6c"}},{"ruleId":"D1","level":"warning","message":{"text":"StandardAsyncExecutionActor.handleExecutionResult (cyclomatic 25): StandardAsyncExecutionActor.handleExecutionResult 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":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1426}}}],"partialFingerprints":{"codehealthFindingId/v1":"4249bf863bc6da1e12671754942667e08b97154104a3beb6f31b754dbbe4a1a4"}},{"ruleId":"D1","level":"warning","message":{"text":"PartialWorkflowSources.fromSubmitRoute (cyclomatic 24): PartialWorkflowSources.fromSubmitRoute has cyclomatic complexity 24 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/PartialWorkflowSources.scala"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c03f638c6d6d191c34b5b68f9eae874dccf90128f667fe1d30d85347f78e20d"}},{"ruleId":"D1","level":"warning","message":{"text":"Scope.lookupFunction (cyclomatic 21): Scope.lookupFunction has cyclomatic complexity 21 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/Scope.scala"},"region":{"startLine":234}}}],"partialFingerprints":{"codehealthFindingId/v1":"58eca7db59f266cada4653c424b9c5c784babe56d1165b2f88d8f8f99f578ffd"}},{"ruleId":"D1","level":"warning","message":{"text":"WdlDraft2WomCallNodeMaker.toWomCallNode (cyclomatic 21): WdlDraft2WomCallNodeMaker.toWomCallNode has cyclomatic complexity 21 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/draft2/src/main/scala/wdl/transforms/draft2/wdlom2wom/WdlDraft2WomCallNodeMaker.scala"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"c05b6810f94b3b1935752c1a29317eb6a7356f7a0bb8c84cc7101c36dd67d521"}},{"ruleId":"D1","level":"warning","message":{"text":"GcpBatchConfigurationAttributes.apply (cyclomatic 20): GcpBatchConfigurationAttributes.apply has cyclomatic complexity 20 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpBatchConfigurationAttributes.scala"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"f14c2599ccfc65d0dca74a854ccc8c41e168934e0154625bc226ef50059b68b2"}},{"ruleId":"D1","level":"warning","message":{"text":"EnhancedAstNode.womType (cyclomatic 19): EnhancedAstNode.womType has cyclomatic complexity 19 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/AstTools.scala"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc2cd4aefd0d532160e16d8bcd206d19abc1d2b92785a479d55886004954444f"}},{"ruleId":"D1","level":"warning","message":{"text":"WdlCall.lookupFunction (cyclomatic 18): WdlCall.lookupFunction has cyclomatic complexity 18 (threshold 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlCall.scala"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"b50616fc48a7af8928c8b1b1b5b268ac9a80c30bc5ab67b2fc109479e6277f99"}},{"ruleId":"D1","level":"warning","message":{"text":"WdlNamespace.validateCallInputSection (cyclomatic 18): WdlNamespace.validateCallInputSection has cyclomatic complexity 18 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlNamespace.scala"},"region":{"startLine":600}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc28e70f7a23a96d0517fcf0c1eddc98c13e291aaf2e33aaa29499a75f00336f"}},{"ruleId":"D1","level":"warning","message":{"text":"WorkflowExecutionActor.startRunnableNodes (cyclomatic 17): WorkflowExecutionActor.startRunnableNodes has cyclomatic complexity 17 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/WorkflowExecutionActor.scala"},"region":{"startLine":620}}}],"partialFingerprints":{"codehealthFindingId/v1":"e49ba30c177168be6794f596d9acac526fe23c612010b19709a158162c5765e5"}},{"ruleId":"D1","level":"warning","message":{"text":"CromwellApiService.submitRequest (cyclomatic 17): CromwellApiService.submitRequest has cyclomatic complexity 17 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":217}}}],"partialFingerprints":{"codehealthFindingId/v1":"e6de2edfef049d7a5b33a32f89b37896b37485ee0266067967695a106599e3b9"}},{"ruleId":"D1","level":"warning","message":{"text":"WesRouteSupport.wesSubmitRequest (cyclomatic 17): WesRouteSupport.wesSubmitRequest has cyclomatic complexity 17 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b9f8e834ed840510fe15f4e3782163a98540656cf8262a9d5c006d80eeb7d2a"}},{"ruleId":"D1","level":"warning","message":{"text":"OutputEvaluator.evaluateOutputs (cyclomatic 16): OutputEvaluator.evaluateOutputs has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/OutputEvaluator.scala"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"051306c9992a48d69b86124855b1a791ff6748207527501533790fab1c6f2ee9"}},{"ruleId":"D1","level":"warning","message":{"text":"WomValueBuilder.toWomValue (cyclomatic 16): WomValueBuilder.toWomValue has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/main/scala/cromwell/core/simpleton/WomValueBuilder.scala"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"c0bcfde84eda753273436225a12a723e4ae41c7ca78610611bbac699c02ffa84"}},{"ruleId":"D1","level":"warning","message":{"text":"ParameterCommandPart.instantiate (cyclomatic 16): ParameterCommandPart.instantiate has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/command/ParameterCommandPart.scala"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"b28ab7d13c43c227b8563232b2209015935e49ecb4bc867070eedcf2231443a1"}},{"ruleId":"D2","level":"warning","message":{"text":"ValueEvaluator.evaluate (cognitive 157): ValueEvaluator.evaluate has cognitive complexity 157 (threshold 15). Drivers by points: loops 28 (95 pts), match/switch 14 (41 pts), if/else 17 (21 pts) (nesting depth added 98). 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"016d931ae5df49c9237e34724e80a1f95eb60612943ac7ddce1996e33dc0ee47"}},{"ruleId":"D2","level":"warning","message":{"text":"TypeEvaluator.evaluate (cognitive 147): TypeEvaluator.evaluate has cognitive complexity 147 (threshold 15). Drivers by points: loops 32 (109 pts), match/switch 12 (32 pts), if/else 6 (nesting depth added 97). 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"481c1c3fd82e8a0cbdbfc8a2b59182664c77f5383dcd65df621dedf763531613"}},{"ruleId":"D2","level":"warning","message":{"text":"FileEvaluator.evaluateRecursive (cognitive 79): FileEvaluator.evaluateRecursive has cognitive complexity 79 (threshold 15). Drivers by points: loops 9 (32 pts), match/switch 13 (31 pts), if/else 15 (16 pts) (nesting depth added 42). 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d833661a9fcc0f6ed4ee06d5694c22c19aff4443a521ef5582821007d9fdc43"}},{"ruleId":"D2","level":"warning","message":{"text":"WdlNamespace.apply (cognitive 61): WdlNamespace.apply has cognitive complexity 61 (threshold 15). Drivers by points: loops 17 (28 pts), match/switch 20 (26 pts), if/else 6, boolean chains 1 (nesting depth added 17). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlNamespace.scala"},"region":{"startLine":198}}}],"partialFingerprints":{"codehealthFindingId/v1":"5724f0fb722a072750df6267a95403b3bfca507393d0996001b3c408ff53e27b"}},{"ruleId":"D2","level":"warning","message":{"text":"MaterializeWorkflowDescriptorActor.buildWorkflowDescriptor (cognitive 53): MaterializeWorkflowDescriptorActor.buildWorkflowDescriptor has cognitive complexity 53 (threshold 15). Drivers by points: loops 8 (36 pts), match/switch 2 (10 pts), if/else 3 (7 pts) (nesting depth added 40). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/materialization/MaterializeWorkflowDescriptorActor.scala"},"region":{"startLine":304}}}],"partialFingerprints":{"codehealthFindingId/v1":"02412ca35bddbf86a5d608a1e1e91c79dabd4ace655b61996fb1d1482965e9ce"}},{"ruleId":"D2","level":"warning","message":{"text":"CallElementToGraphNode.convert (cognitive 52): CallElementToGraphNode.convert has cognitive complexity 52 (threshold 15). Drivers by points: if/else 15 (22 pts), match/switch 10 (17 pts), loops 5 (9 pts), boolean chains 4 (nesting depth added 18). 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":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/CallElementToGraphNode.scala"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"5271a061c318d23a8f2284d4a2e791f5267c31d9d43dbe4e83e928488e5371cf"}},{"ruleId":"D2","level":"warning","message":{"text":"GcpBatchAsyncBackendJobExecutionActor.executeAsync (cognitive 46): GcpBatchAsyncBackendJobExecutionActor.executeAsync has cognitive complexity 46 (threshold 15). Drivers by points: loops 9 (45 pts), match/switch 1 (nesting depth added 36). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":928}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f235d59905b8d4a9302104b651dabd57ad07bc3d1673c8186573c3c8c990ea5"}},{"ruleId":"D2","level":"warning","message":{"text":"CallCacheDiffActor.extractCallMetadata (cognitive 45): CallCacheDiffActor.extractCallMetadata has cognitive complexity 45 (threshold 15). Drivers by points: loops 9 (45 pts) (nesting depth added 36). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/callcaching/CallCacheDiffActor.scala"},"region":{"startLine":170}}}],"partialFingerprints":{"codehealthFindingId/v1":"88e26457e83e6b7798468bebfbe6aa8e0a81d2716350941f86dd207bddb3f9f3"}},{"ruleId":"D2","level":"warning","message":{"text":"ArchiveMetadataConfig.parseConfig (cognitive 45): ArchiveMetadataConfig.parseConfig has cognitive complexity 45 (threshold 15). Drivers by points: loops 9 (45 pts) (nesting depth added 36). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/archiver/ArchiveMetadataConfig.scala"},"region":{"startLine":28}}}],"partialFingerprints":{"codehealthFindingId/v1":"0260db31ed3235b6be0831ba95dff9f6bf451c0956c20855c27a769284ee98c3"}},{"ruleId":"D2","level":"warning","message":{"text":"WdlCall.lookupFunction (cognitive 34): WdlCall.lookupFunction has cognitive complexity 34 (threshold 15). Drivers by points: loops 12 (27 pts), match/switch 4, if/else 3 (nesting depth added 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlCall.scala"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc8160c600b06f02382119c55d4b6102fdd3801e46345fee30ef31832b69ae01"}},{"ruleId":"D2","level":"warning","message":{"text":"StandardAsyncExecutionActor.handleExecutionResult (cognitive 33): StandardAsyncExecutionActor.handleExecutionResult has cognitive complexity 33 (threshold 15). Drivers by points: if/else 10 (11 pts), match/switch 5 (10 pts), loops 5 (9 pts), boolean chains 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":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1426}}}],"partialFingerprints":{"codehealthFindingId/v1":"8cd9cf64cbbf317c943a32bb1af11092b582eb9310fae7fa8682aafb64f396d7"}},{"ruleId":"D2","level":"warning","message":{"text":"comparer.error_checks (cognitive 29): comparer.error_checks has cognitive complexity 29 (threshold 15). Drivers by points: if/else 11 (22 pts), loops 4 (7 pts) (nesting depth added 14). 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":"scripts/metadata_comparison/metadata_comparison/comparer.py"},"region":{"startLine":321}}}],"partialFingerprints":{"codehealthFindingId/v1":"18b7b4120e52fc11d6143ebb21ad5b1cf7ecaa3a8c69b8d4bfd07be973a79d49"}},{"ruleId":"D2","level":"warning","message":{"text":"MetadataSlickDatabase.summarizeDecreasing (cognitive 26): MetadataSlickDatabase.summarizeDecreasing has cognitive complexity 26 (threshold 15). Drivers by points: loops 6 (21 pts), match/switch 2 (5 pts) (nesting depth added 18). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/MetadataSlickDatabase.scala"},"region":{"startLine":318}}}],"partialFingerprints":{"codehealthFindingId/v1":"371227e4bf08d15cd8003e0313d894119d197417f244c0be5150d7c6e58bab6c"}},{"ruleId":"D2","level":"warning","message":{"text":"MetadataBuilderActor.computeCost (cognitive 24): MetadataBuilderActor.computeCost has cognitive complexity 24 (threshold 15). Drivers by points: loops 3 (12 pts), if/else 5 (8 pts), match/switch 2 (4 pts) (nesting depth added 14). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala"},"region":{"startLine":355}}}],"partialFingerprints":{"codehealthFindingId/v1":"9d06436fa3de2a229dd423bf57e6bd33c1757291b0b5ca9361f4815ce3113f43"}},{"ruleId":"D2","level":"warning","message":{"text":"WdlDraft2LanguageFactory.validateNamespace (cognitive 23): WdlDraft2LanguageFactory.validateNamespace has cognitive complexity 23 (threshold 15). Drivers by points: loops 6 (21 pts), match/switch 2 (nesting depth added 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"languageFactories/wdl-draft2/src/main/scala/languages/wdl/draft2/WdlDraft2LanguageFactory.scala"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"91c66bf5061d4a2827d11127efd74cfe0e47e01804a3d0c22eb514938a81403d"}},{"ruleId":"D2","level":"warning","message":{"text":"CostCatalogKey.apply (cognitive 21): CostCatalogKey.apply has cognitive complexity 21 (threshold 15). Drivers by points: loops 6 (21 pts) (nesting depth added 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline. This shape REPEATS in the file: one other method here (GcpCostCatalogService.calculateVmCostPerHour) has the same decision points, in the same order, at the same nesting depths \u2014 so this is one pattern written twice rather than two separate problems. Splitting this body alone leaves the other exactly as it is. Where these are variations on one operation, the change that clears both is the shared one: lift the common shape into a single routine the variants call, parameterised by whatever genuinely differs between them, and keep in each method only the part that is not shared."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/cost/GcpCostCatalogService.scala"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"726f880e544131627c5644f538dbf53316633745be62b88a9e6a4b7094ee1837"}},{"ruleId":"D2","level":"warning","message":{"text":"GcpCostCatalogService.calculateVmCostPerHour (cognitive 21): GcpCostCatalogService.calculateVmCostPerHour has cognitive complexity 21 (threshold 15). Drivers by points: loops 6 (21 pts) (nesting depth added 15). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline. This shape REPEATS in the file: one other method here (CostCatalogKey.apply) has the same decision points, in the same order, at the same nesting depths \u2014 so this is one pattern written twice rather than two separate problems. Splitting this body alone leaves the other exactly as it is. Where these are variations on one operation, the change that clears both is the shared one: lift the common shape into a single routine the variants call, parameterised by whatever genuinely differs between them, and keep in each method only the part that is not shared."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/cost/GcpCostCatalogService.scala"},"region":{"startLine":215}}}],"partialFingerprints":{"codehealthFindingId/v1":"41c96d9bdadd83570c7086010556f07cf39db8c35b8e9f26682581a1a2d225bf"}},{"ruleId":"D2","level":"warning","message":{"text":"MetadataBuilderActor.buildCostAndStop (cognitive 21): MetadataBuilderActor.buildCostAndStop has cognitive complexity 21 (threshold 15). Drivers by points: loops 5 (15 pts), match/switch 4, if/else 2 (nesting depth added 10). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala"},"region":{"startLine":624}}}],"partialFingerprints":{"codehealthFindingId/v1":"29d0d6248c8ad5315a7172ab7009429fb40df6621cbfb997703a1a434efbf50a"}},{"ruleId":"D2","level":"warning","message":{"text":"JobStoreWriterActor.process (cognitive 20): JobStoreWriterActor.process has cognitive complexity 20 (threshold 15). Drivers by points: match/switch 6 (13 pts), loops 2 (5 pts), if/else 2 (nesting depth added 10). 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":"engine/src/main/scala/cromwell/jobstore/JobStoreWriterActor.scala"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"31abf1f70f8f36daa4f390a67b00e098a92e0a0e55876eb13fd3ac16a9ff25f2"}},{"ruleId":"D2","level":"warning","message":{"text":"AstNodeToExpressionElement.convert (cognitive 19): AstNodeToExpressionElement.convert has cognitive complexity 19 (threshold 15). Drivers by points: loops 4 (9 pts), match/switch 4 (7 pts), if/else 2, boolean chains 1 (nesting depth added 8). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/ast2wdlom/AstNodeToExpressionElement.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"21cc5659cca263964fc35baaee63644ffeb2c29a1e132680320d4c299e5602e4"}},{"ruleId":"D2","level":"warning","message":{"text":"EnhancedAstNode.womValue (cognitive 18): EnhancedAstNode.womValue has cognitive complexity 18 (threshold 15). Drivers by points: boolean chains 11, if/else 4, match/switch 2 (3 pts) (nesting depth added 1). 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/AstTools.scala"},"region":{"startLine":179}}}],"partialFingerprints":{"codehealthFindingId/v1":"47f5105e96cdbd0e30e918ef6e702b69db46df7260fe867247398f02cff00832"}},{"ruleId":"D2","level":"warning","message":{"text":"WdlDraft2WomCallNodeMaker.toWomCallNode (cognitive 18): WdlDraft2WomCallNodeMaker.toWomCallNode has cognitive complexity 18 (threshold 15). Drivers by points: match/switch 8 (9 pts), loops 4 (6 pts), if/else 2, boolean chains 1 (nesting depth added 3). 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":"wdl/transforms/draft2/src/main/scala/wdl/transforms/draft2/wdlom2wom/WdlDraft2WomCallNodeMaker.scala"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"a094797605e443d45a0dc0df131646269935293ce0f6dd318a9819ca92d639b8"}},{"ruleId":"D2","level":"warning","message":{"text":"ReadMetadataRegulatorActor.receive (cognitive 17): ReadMetadataRegulatorActor.receive has cognitive complexity 17 (threshold 15). Drivers by points: if/else 5 (12 pts), match/switch 3 (5 pts) (nesting depth added 9). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/ReadMetadataRegulatorActor.scala"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c53904750b559ae94edce672a4cb065a7195962d649fe217f4daf67180fab76"}},{"ruleId":"D2","level":"warning","message":{"text":"ArchiveMetadataSchedulerActor.streamMetadataToGcs (cognitive 17): ArchiveMetadataSchedulerActor.streamMetadataToGcs has cognitive complexity 17 (threshold 15). Drivers by points: loops 4 (10 pts), if/else 4 (7 pts) (nesting depth added 9). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/archiver/ArchiveMetadataSchedulerActor.scala"},"region":{"startLine":257}}}],"partialFingerprints":{"codehealthFindingId/v1":"77996ea089d96384daccdb5d0df82317190b92260466eee7022ebeb4d48e1f37"}},{"ruleId":"D2","level":"warning","message":{"text":"GcpBatchAsyncBackendJobExecutionActor.createBatchParameters (cognitive 17): GcpBatchAsyncBackendJobExecutionActor.createBatchParameters has cognitive complexity 17 (threshold 15). Drivers by points: match/switch 4 (8 pts), loops 2 (5 pts), if/else 3 (4 pts) (nesting depth added 8). 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":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":600}}}],"partialFingerprints":{"codehealthFindingId/v1":"804143bcf918b70abfe391f01f4b371e21b4f85131a3f6cd01f73b2a862ac3bd"}},{"ruleId":"D2","level":"warning","message":{"text":"Scope.lookupFunction (cognitive 17): Scope.lookupFunction has cognitive complexity 17 (threshold 15). Drivers by points: match/switch 7 (10 pts), if/else 6, boolean chains 1 (nesting depth added 3). 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/Scope.scala"},"region":{"startLine":234}}}],"partialFingerprints":{"codehealthFindingId/v1":"7790d2f7d2f271175bd389ce5e8e1469b37df1209f38185a488594604cac27fc"}},{"ruleId":"D2","level":"warning","message":{"text":"UnixPath.normalize (cognitive 16): UnixPath.normalize has cognitive complexity 16 (threshold 15). Drivers by points: if/else 7 (12 pts), match/switch 1 (2 pts), boolean chains 1, loops 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":"cloud-nio/cloud-nio-spi/src/main/scala/cloud/nio/spi/UnixPath.scala"},"region":{"startLine":201}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc82812b424cb65d8326bca171a99493b3f2194c429c7336710c0e180f1735a7"}},{"ruleId":"D2","level":"warning","message":{"text":"DrsPathBuilder.maybeCreateOtherPath (cognitive 16): DrsPathBuilder.maybeCreateOtherPath has cognitive complexity 16 (threshold 15). Drivers by points: loops 5 (15 pts), match/switch 1 (nesting depth added 10). To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"filesystems/drs/src/main/scala/cromwell/filesystems/drs/DrsPathBuilder.scala"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"4fa08fdc1738145197a5d65e856be49ae088d3a8c32d794d7182ecece295da69"}},{"ruleId":"D2","level":"warning","message":{"text":"ArchiveMetadataSchedulerActor.receive (cognitive 16): ArchiveMetadataSchedulerActor.receive has cognitive complexity 16 (threshold 15). Drivers by points: if/else 5 (13 pts), match/switch 2 (3 pts) (nesting depth added 9). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/archiver/ArchiveMetadataSchedulerActor.scala"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb7910aa9b0b5eff3b21fef820952eb481cc34afa052250478e3187a1e42ff3e"}},{"ruleId":"D2","level":"warning","message":{"text":"GcpLabel.safeGoogleName (cognitive 16): GcpLabel.safeGoogleName has cognitive complexity 16 (threshold 15). Drivers by points: if/else 8 (11 pts), match/switch 2 (3 pts), boolean chains 2 (nesting depth added 4). 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":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpLabel.scala"},"region":{"startLine":28}}}],"partialFingerprints":{"codehealthFindingId/v1":"d8b54c024c4110e591ece53b2388861c7f2447f9ad7eb2b589b1a16265d8527f"}},{"ruleId":"D2","level":"warning","message":{"text":"WomArrayType.coercion (cognitive 16): WomArrayType.coercion has cognitive complexity 16 (threshold 15). Drivers by points: boolean chains 13, match/switch 2 (3 pts) (nesting depth added 1). 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":"wom/src/main/scala/wom/types/WomArrayType.scala"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"8e3d3f2cc6c1cb05e11b4beba3a0f123fb0d19e9c5f6b0a2a7f3d7cb33f61dc9"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BetterFileMethods: TooManyMethods \u2014 148 methods. The bar is 30 methods; this is 118 over it, 4.93\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/main/scala/cromwell/core/path/BetterFileMethods.scala"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6a22de1d4aa1a9ea0225d2366a7dba14e92313394ce3dc575574a7a55596881"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: StandardAsyncExecutionActor: TooManyMethods \u2014 69 methods. The bar is 30 methods; this is 39 over it, 2.30\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff135e2b76a483a63b023ff561c139706ac790c416e0f2c385e39ec6e59231b2"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: actors/GcpBatchAsyncBackendJobExecutionActor.scala: FileTooLong \u2014 960 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 460 over it, 1.92\u00D7 the bar. 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":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5215df6e0f58f2d2fca35a6028d58b9f9d8c162da42fce2f5d4cb6ca5ba4e896"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: job/EngineJobExecutionActor.scala: FileTooLong \u2014 851 significant lines (blank, comment-only and punctuation-only lines excluded), about 78% of them inside a single declaration: EngineJobExecutionActor (64-1034). The bar is 500 significant lines; this is 351 over it, 1.70\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/job/EngineJobExecutionActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"05bbc52562fc5eb2f4ef9587bfee0e14687c620396a3286ee4106d86fc1d4a75"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: standard/StandardAsyncExecutionActor.scala: FileTooLong \u2014 801 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 301 over it, 1.60\u00D7 the bar. 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":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e81c5bbb601b70dcb4afde68a9cd6e1d658c01af2c56416ed5b5699be07b725d"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: EngineJobExecutionActor: TooManyMethods \u2014 41 methods. The bar is 30 methods; this is 11 over it, 1.37\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/job/EngineJobExecutionActor.scala"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"395f1d675a7ecd7c0b3dd059f9192d5a98bc396e1d06a6cd619328665d8580c3"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: execution/WorkflowExecutionActor.scala: FileTooLong \u2014 682 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 182 over it, 1.36\u00D7 the bar. 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":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/WorkflowExecutionActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c41b7f893b366350ca03db63ba55e0ec549cbac3add0fc5030d19bda57e9cc0"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: values/EngineFunctionEvaluators.scala: FileTooLong \u2014 678 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 178 over it, 1.36\u00D7 the bar. 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":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/values/EngineFunctionEvaluators.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3758c6f634265764de30d24296f67dbe739876e73bd9574309fa4080546caed1"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WdlStandardLibraryFunctions: TooManyMethods \u2014 36 methods. The bar is 30 methods; this is 6 over it, 1.20\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/WdlStandardLibraryFunctions.scala"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"040e9aa1ebc3c3b8abbb467b996811beda1c8802670a58ffd0e3d12a6ccac1a3"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WdlStandardLibraryFunctionsType: TooManyMethods \u2014 35 methods. The bar is 30 methods; this is 5 over it, 1.17\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/WdlStandardLibraryFunctions.scala"},"region":{"startLine":526}}}],"partialFingerprints":{"codehealthFindingId/v1":"471ebd77386b0fc471914abc1bbdd0f8972c2472c873c43dae541e6da12895b3"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: workflow/WorkflowActor.scala: FileTooLong \u2014 567 significant lines (blank, comment-only and punctuation-only lines excluded), about 71% of them inside a single declaration: WorkflowActor (262-860). The bar is 500 significant lines; this is 67 over it, 1.13\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/WorkflowActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"dce014087824b2dbe2f38d732a39c9f44f9c031c0af9114517c4c1a0d55d1768"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: GcpBatchAsyncBackendJobExecutionActor: TooManyMethods \u2014 34 methods. The bar is 30 methods; this is 4 over it, 1.13\u00D7 the bar. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":153}}}],"partialFingerprints":{"codehealthFindingId/v1":"cdc305c6d124f510ee458aefcf0b09c9263c78aec0258f949e5d7a6d5546f604"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: model/WdlNamespace.scala: FileTooLong \u2014 533 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 33 over it, 1.07\u00D7 the bar. 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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlNamespace.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d854384f49d96a31731fc016497f43f389b4a1f4e9f8db6b235a58aee1186955"}},{"ruleId":"D4","level":"warning","message":{"text":"Near-duplicate member pair (34 shared lines): engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:217-270 | engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala:139-192 \u2014 These two members are variants of one another: 34 of their lines are already reported as duplicated blocks below, spread through both bodies rather than gathered into one. Read them as a single construct written twice. The repair is at the members\u0027 grain \u2014 factor the shared pipeline into one implementation the two call with their differences as parameters or as an injected step, or, where the difference is systematic (sync against async, one transport against another), generate one from the other. Extracting the individual blocks below is not the same fix: it leaves the two bodies in place and the next edit still has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":217}}}],"partialFingerprints":{"codehealthFindingId/v1":"d010be27f6e09deece615814b9d2ca87cd86466a610194f5c339f3d46b14ba16"}},{"ruleId":"D4","level":"warning","message":{"text":"Edited copy of a member (19 corresponding lines): wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlExpression.scala:260-293 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/WdlStandardLibraryFunctions.scala:398-460 \u2014 These two members are one piece of code written twice and then edited apart: 19 consecutive lines correspond almost exactly, broken only by small local edits. Most of that correspondence is NOT reported as duplicated blocks below \u2014 the edits cut it into fragments and only the largest of them clear the block floor, so the rows below understate it. The repair is at the members\u0027 grain \u2014 factor the shared implementation into one the two call with their differences as parameters or as an injected step, or, where the difference is systematic (an extra return value, one transport against another), generate one from the other. Left alone, the next edit has to be made twice and the two will drift further apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlExpression.scala"},"region":{"startLine":260}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a30664869a2d0bb4137c9df22977e5ced15250a84fd9a8da829ffc0f35ebd05"}},{"ruleId":"D4","level":"warning","message":{"text":"Edited copy of a member (19 corresponding lines): scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py:192-211 | scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py:262-280 \u2014 These two members are one piece of code written twice and then edited apart: 19 consecutive lines correspond almost exactly, broken only by small local edits. Most of that correspondence is NOT reported as duplicated blocks below \u2014 the edits cut it into fragments and only the largest of them clear the block floor, so the rows below understate it. The repair is at the members\u0027 grain \u2014 factor the shared implementation into one the two call with their differences as parameters or as an injected step, or, where the difference is systematic (an extra return value, one transport against another), generate one from the other. Left alone, the next edit has to be made twice and the two will drift further apart."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"e96a34a9f73355f6a8abd216f47078da41904517f65a7fadc90efba878a16db9"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (4 members, 50\u002B identical tokens): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:301-323 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendCacheHitCopyingActor.scala:75-97 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendFileHashingActor.scala:84-106 | wom/src/main/scala/wom/expression/WomExpression.scala:38-57 \u2014 These 4 members share a duplicated core: a run of at least 50 identical tokens appears in every one of them. That run is NOT broken out as duplicated-block rows below \u2014 it is what admitted this row, and the blocks below cover only the part of it that clears the block floor, so they understate the correspondence. Read the members as one construct written 4 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 4 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/expression/WomExpression.scala"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c7b926074cc7a61047528eac5a7805b85d4bd655ac6be91d295a21217575eed"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (29\u201331 lines \u00D7 3): supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/AbortRequestHandler.scala:54-82 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/GetRequestHandler.scala:33-61 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/RunRequestHandler.scala:34-64 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from all 3 call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/AbortRequestHandler.scala:54\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/AbortRequestHandler.scala"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"72107d816b6092e6b990fcd7cff2892e12ffc555baec7effed6d541491f3ad76"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (29 lines \u00D7 2): wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/wdlom2wom/package.scala:49-77 | wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/wdlom2wom/package.scala:49-77 \u2014 before extracting anything, compare \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/wdlom2wom/package.scala\u0060 and \u0060wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/wdlom2wom/package.scala\u0060 as WHOLE FILES: 91% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/wdlom2wom/package.scala"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"b231d4754283f1414d323d55180d8947b952d8452ed6427d4e296631350aa80b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (28 lines \u00D7 2): languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala:69-96 | languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala:69-96 \u2014 before extracting anything, compare \u0060languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala\u0060 and \u0060languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 49 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2f2fcd8d72500cb79f43f9cc29ff641949ccca5f2ed9809801f103e68d18421"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (26 lines \u00D7 2): cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/TryWithResource.scala:10-35 | core/src/main/scala/cromwell/util/TryWithResource.scala:10-35 \u2014 before extracting anything, compare \u0060cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/TryWithResource.scala\u0060 and \u0060core/src/main/scala/cromwell/util/TryWithResource.scala\u0060 as WHOLE FILES: 96% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/TryWithResource.scala"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"9adf2e2548ac5b113149e0ab22f4da787aaaf4be2017f57a2bcbc955ec49d737"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24\u201325 lines \u00D7 2): CromIAM/src/main/scala/cromiam/webservice/SwaggerUiHttpService.scala:45-69 | engine/src/main/scala/cromwell/webservice/SwaggerUiHttpService.scala:54-77 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/scala/cromiam/webservice/SwaggerUiHttpService.scala"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"95d6a6db610d779715af72d6e856a6ecfb5604857d65b793320a664da7d094f7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19\u201322 lines \u00D7 4): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:302-323 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendCacheHitCopyingActor.scala:76-97 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendFileHashingActor.scala:85-106 | wom/src/main/scala/wom/expression/WomExpression.scala:39-57 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 4 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 4 times. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/expression/WomExpression.scala"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"23daa133235e2b137b69a57c0bd51dab9e1fb970b402f6cbac7d5cc787baa7d1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (20 lines \u00D7 2): services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala:489-508 | services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala:520-539 \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 \u0060services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala:489\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/builder/MetadataBuilderActor.scala"},"region":{"startLine":489}}}],"partialFingerprints":{"codehealthFindingId/v1":"20b752a99555389f8010752616086ab9872f12ef3b7505f1fe52818876774594"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/SubWorkflowExecutionActor.scala:428-445 | engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/WorkflowExecutionActor.scala:1019-1037 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/SubWorkflowExecutionActor.scala:428\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/SubWorkflowExecutionActor.scala"},"region":{"startLine":428}}}],"partialFingerprints":{"codehealthFindingId/v1":"bfcab372ede1b0e36d8a647c28cdaadd9422769e7b7e373806f27f870d8ee9cf"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201317 lines \u00D7 2): backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1478-1487 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:862-878 \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 \u0060backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1478\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":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1478}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e69fde9520bc8b765d17e85006628d407afe75539fc3d3ca59471c43ce4352e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15\u201316 lines \u00D7 3): languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala:44-59 | languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala:44-59 | languageFactories/wdl-draft3/src/main/scala/languages/wdl/draft3/WdlDraft3LanguageFactory.scala:44-58 \u2014 before extracting anything, compare \u0060languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala\u0060 and \u0060languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 49 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8e5bad6e6ffd2e38e15dd09521912a886865c93ba631f2d0a6a6feaec2be883"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/VersionUtil.scala:64-79 | common/src/main/scala/common/util/VersionUtil.scala:64-79 \u2014 before extracting anything, compare \u0060cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/VersionUtil.scala\u0060 and \u0060common/src/main/scala/common/util/VersionUtil.scala\u0060 as WHOLE FILES: 96% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/VersionUtil.scala"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"d09e355a1117d0f45e79c6ee0c8f1b416dce228100beeb9537dce24b0688c9f3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:86-100 | backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:110-124 \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 \u0060backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:86\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":"backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"ffe313c16a7108b709b40d7ea3f8c9edbe3bf11f2f70b4aee82299b61171d040"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13\u201314 lines \u00D7 2): backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1535-1547 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:973-986 \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 \u0060backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1535\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1535}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e6988f9aeeb46bc295611501af5089670788a7836f1063860d982c1c185c884"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): database/sql/src/main/scala/cromwell/database/slick/tables/CallCachingAggregationEntryComponent.scala:88-101 | database/sql/src/main/scala/cromwell/database/slick/tables/CallCachingAggregationEntryComponent.scala:116-129 \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 \u0060database/sql/src/main/scala/cromwell/database/slick/tables/CallCachingAggregationEntryComponent.scala:88\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":"database/sql/src/main/scala/cromwell/database/slick/tables/CallCachingAggregationEntryComponent.scala"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"e83583efe0f7361b70a149c697e9efd4a6a55fcad31f95064185b72562a3f3b3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13\u201314 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:75-87 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala:106-119 \u2014 before extracting anything, compare \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala\u0060 and \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:75\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala"},"region":{"startLine":75}}}],"partialFingerprints":{"codehealthFindingId/v1":"0cd7228d7a6b744060c6557cac4d1df7249328affd4f7d0e8e00930ffbbec929"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12\u201313 lines \u00D7 2): services/src/main/scala/cromwell/services/metadata/impl/MetadataDatabaseAccess.scala:264-275 | services/src/main/scala/cromwell/services/metadata/impl/MetadataDatabaseAccess.scala:283-295 \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. 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":"services/src/main/scala/cromwell/services/metadata/impl/MetadataDatabaseAccess.scala"},"region":{"startLine":264}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddcbb92c3db69bb6301f01992d9ce939f876236b9993f61c7b76210b58a2f31b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlExpression.scala:266-278 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/WdlStandardLibraryFunctions.scala:402-414 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlExpression.scala"},"region":{"startLine":266}}}],"partialFingerprints":{"codehealthFindingId/v1":"b55c74acabf869115506e356bcc29f70094b15c3fbeed60322262456f4f581c9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): wdl/transforms/draft2/src/main/scala/wdl/transforms/draft2/wdlom2wom/WdlDraft2WomCallNodeMaker.scala:108-119 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/CallElementToGraphNode.scala:145-156 \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 \u0060wdl/transforms/draft2/src/main/scala/wdl/transforms/draft2/wdlom2wom/WdlDraft2WomCallNodeMaker.scala:108\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/draft2/src/main/scala/wdl/transforms/draft2/wdlom2wom/WdlDraft2WomCallNodeMaker.scala"},"region":{"startLine":108}}}],"partialFingerprints":{"codehealthFindingId/v1":"e59508d71aeeb9d1bb34226c9019dfdfd9b14f9557042ddde1a9987c89b363cd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:60-71 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala:90-101 \u2014 before extracting anything, compare \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala\u0060 and \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:60\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":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"6087c49cf36fcf3cc070c79021250efcb0d85e84015bfd55186486cbe1c2b551"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1111-1121 | supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:1187-1197 \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 \u0060backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala:1111\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":1111}}}],"partialFingerprints":{"codehealthFindingId/v1":"9677ff21b501479883191df949928fe4d2e32a6c47d54e921c0a577e647711c4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:92-102 | backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:140-150 \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":"backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"f6b06a6f6f89b1462cf1cc0faf35821e506fd46203860a449d9d5ebd274fef58"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201311 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/files/EngineFunctionEvaluators.scala:204-214 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/files/EngineFunctionEvaluators.scala:222-231 \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 \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/files/EngineFunctionEvaluators.scala:204\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. 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":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/files/EngineFunctionEvaluators.scala"},"region":{"startLine":204}}}],"partialFingerprints":{"codehealthFindingId/v1":"975bb4323df12609bf7c06e477222fb934a7cc223737fb3382dff90559f09697"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/TaskDefinitionElementToWomTaskDefinition.scala:135-145 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/WorkflowDefinitionElementToWomWorkflowDefinition.scala:199-209 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/TaskDefinitionElementToWomTaskDefinition.scala:135\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/TaskDefinitionElementToWomTaskDefinition.scala"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"93ccae348e27d1b8a2a6e5220bb3a7193dec59140811720b2a93e935cfef89b6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:49-59 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala:79-89 \u2014 before extracting anything, compare \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala\u0060 and \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/ScatterElementToGraphNode.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala:49\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/graph/IfElementToGraphNode.scala"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"a1aabe5e3bb77abb48c63b22b6a3647f46e7086878584f2d20570a19070ad2af"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:250-259 | engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala:172-181 \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 \u0060engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:250\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":250}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b1c347b06e056cb1c897618de367287883d2b4f23859f975930b362e8dc3330"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:480-489 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala:720-730 \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 \u0060supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:480\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":720}}}],"partialFingerprints":{"codehealthFindingId/v1":"30e26816013d6060660c7df86dc2171c536ac7054e4e24b3f84efd84ae1df028"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 3): backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:92-100 | backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:116-124 | backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala:140-148 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/validation/RuntimeAttributesValidation.scala"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c29f66f37554db42024c4bd3555552768f7b23f133f73a1a69579cdc94d17b0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:240-248 | engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala:162-170 \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 \u0060engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:240\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":240}}}],"partialFingerprints":{"codehealthFindingId/v1":"c4b55ae23ba12b05ae1f9c1fdbd3c8cf378b94454bc30f9a27d051a145d6dd36"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendCacheHitCopyingActor.scala:241-249 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/callcaching/BatchBackendCacheHitCopyingActor.scala:55-63 \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 \u0060supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/callcaching/AwsBatchBackendCacheHitCopyingActor.scala:241\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":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/callcaching/BatchBackendCacheHitCopyingActor.scala"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"6fe356277bcbf911c05471f5da6b60826968bfe96709bd1cbf8b6f9aef76a7ec"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:225-232 | engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala:146-153 \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 \u0060engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:225\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":225}}}],"partialFingerprints":{"codehealthFindingId/v1":"09f465e4875cf96465925060b4e7db75733d8e429eff1aaa6f9eb05a807bd19e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/monitoring/BatchInstrumentation.scala:41-48 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/monitoring/BatchInstrumentation.scala:50-57 \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":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/monitoring/BatchInstrumentation.scala"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a90aa8dc186c047bc5bf12ba87825f8e59bc6dc43db281c3a049478949f36aa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/ast2wdlom/BiscayneAstNodeToKvPair.scala:11-18 | wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/ast2wdlom/CascadesAstNodeToKvPair.scala:11-18 \u2014 before extracting anything, compare \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/ast2wdlom/BiscayneAstNodeToKvPair.scala\u0060 and \u0060wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/ast2wdlom/CascadesAstNodeToKvPair.scala\u0060 as WHOLE FILES: 88% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/ast2wdlom/BiscayneAstNodeToKvPair.scala:11\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/ast2wdlom/BiscayneAstNodeToKvPair.scala"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"144ead8522cffb7321f6abcc9f8ef59d2d71860862ec3fde404fc2fece9668b0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/types/BiscayneTypeEvaluators.scala:73-80 | wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/linking/expression/types/CascadesTypeEvaluators.scala:73-80 \u2014 before extracting anything, compare \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/types/BiscayneTypeEvaluators.scala\u0060 and \u0060wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/linking/expression/types/CascadesTypeEvaluators.scala\u0060 as WHOLE FILES: 98% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/types/BiscayneTypeEvaluators.scala"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa0633f1a411c232ea86d8f61f83c5cf9eccb8962fd21b51c3c3da4dc0767796"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): cloudSupport/src/main/scala/cromwell/cloudsupport/aws/AwsConfiguration.scala:152-158 | cloudSupport/src/main/scala/cromwell/cloudsupport/gcp/GoogleConfiguration.scala:107-113 \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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloudSupport/src/main/scala/cromwell/cloudsupport/gcp/GoogleConfiguration.scala"},"region":{"startLine":107}}}],"partialFingerprints":{"codehealthFindingId/v1":"60b46a4970ac16343ab797523b00194b6addbe00f72c3c209df2c917d4be5eed"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): cromwellApiClient/src/main/scala/cromwell/api/CromwellClient.scala:254-260 | cromwellApiClient/src/main/scala/cromwell/api/CromwellClient.scala:276-282 \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 \u0060cromwellApiClient/src/main/scala/cromwell/api/CromwellClient.scala:254\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cromwellApiClient/src/main/scala/cromwell/api/CromwellClient.scala"},"region":{"startLine":254}}}],"partialFingerprints":{"codehealthFindingId/v1":"69d5448417240a2e6875e8cc8b31315d3fec0da9d0866cfe5c88613f152bbe9a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:261-267 | engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala:183-189 \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 \u0060engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala:261\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/CromwellApiService.scala"},"region":{"startLine":261}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa98a7c1ce2507eedac73e196c973c28daa152e40c9cc2a4aff7a362b751f803"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala:84-90 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala:108-114 \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 \u0060wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala:84\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"c3967f645188c9ef9e92eef6a14ad90f9935015d03a544b61c643be3d2d163c5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:138-145 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala:111-117 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:138\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":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"378d4e5c4646cd603a937514faf20ce677142b4d84edd00657785f8e1dbaff81"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6\u20137 lines \u00D7 2): wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala:80-86 | wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala:92-97 \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 \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala:80\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. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala:97\u0060 calls \u0060renameIdentifiers\u0060 and \u0060wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala:86\u0060 does not \u2014 after which the two agree again for 3 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/wdlom2wom/expression/renaming/EngineFunctionEvaluators.scala"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"32640de2612bc883603aca8c82a00b2771db7c1fad8ddf47e42c925cc9908b70"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala:163-168 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:75-80 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala:163\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/FileEvaluator.scala"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2863502ac0371889d34508b117359e13fc8d497eaf691d159cfc95e9774abe5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:44-49 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala:83-88 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:44\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"41a28386997a94c34d28206a7fadbaea8c94fef5040c9df8655ba727c72caf8f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:50-55 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala:89-94 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:50\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"a86d87f972e4ecdbaf0020ccdb7e905580809283b77088e01e1076ca62fb9049"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/values/BiscayneValueEvaluators.scala:155-160 | wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/linking/expression/values/CascadesValueEvaluators.scala:155-160 \u2014 before extracting anything, compare \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/values/BiscayneValueEvaluators.scala\u0060 and \u0060wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/linking/expression/values/CascadesValueEvaluators.scala\u0060 as WHOLE FILES: 98% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/values/BiscayneValueEvaluators.scala:155\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/linking/expression/values/BiscayneValueEvaluators.scala"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"a6281b445dd96b7867dd63e79c247aa9f8946bf494aadb7b6043710211014686"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala:260-264 | database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala:273-277 \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 \u0060database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala:260\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":"database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala"},"region":{"startLine":260}}}],"partialFingerprints":{"codehealthFindingId/v1":"a63e9043c248cd0d2ff4a606dd5f8e8afa0290bf8eac0e1cc119111acc14aeac"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala:72-76 | wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/ValueEvaluator.scala:123-127 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/expression/TypeEvaluator.scala"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f8b854b278a28205f57f1900e2f21e72f30ef17c32eced19e7fb6e969dcf29e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (58 lines \u00D7 3): wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/parsing/WdlBiscayneSyntaxErrorFormatter.scala:10-67 | wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/parsing/WdlCascadesSyntaxErrorFormatter.scala:10-67 | wdl/transforms/draft3/src/main/scala/wdl/draft3/transforms/parsing/WdlDraft3SyntaxErrorFormatter.scala:9-66 \u2014 before extracting anything, compare \u0060wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/parsing/WdlBiscayneSyntaxErrorFormatter.scala\u0060 and \u0060wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/parsing/WdlCascadesSyntaxErrorFormatter.scala\u0060 as WHOLE FILES: 90% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/parsing/WdlBiscayneSyntaxErrorFormatter.scala"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"7b69b0b69e68d938338c9a31e5da928da7c0bfcd96c905eb25cc4df6b754b303"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): cloudSupport/src/main/scala/cromwell/cloudsupport/aws/AwsConfiguration.scala:53-58 | cloudSupport/src/main/scala/cromwell/cloudsupport/gcp/GoogleConfiguration.scala:22-27 \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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060authsByName\u0060 to \u0060Map[String, AwsAuthMode]\u0060 in one and \u0060Map[String, GoogleAuthMode]\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloudSupport/src/main/scala/cromwell/cloudsupport/gcp/GoogleConfiguration.scala"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"37410e31dc372ac27deb5773cd4300d65f4ef691b1d4f736406a948e43947e66"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 3): languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala:102-106 | languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala:102-106 | languageFactories/wdl-draft3/src/main/scala/languages/wdl/draft3/WdlDraft3LanguageFactory.scala:119-123 \u2014 before extracting anything, compare \u0060languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala\u0060 and \u0060languageFactories/wdl-cascades/src/main/scala/languages/wdl/cascades/WdlCascadesLanguageFactory.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 49 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"languageFactories/wdl-biscayne/src/main/scala/languages/wdl/biscayne/WdlBiscayneLanguageFactory.scala"},"region":{"startLine":102}}}],"partialFingerprints":{"codehealthFindingId/v1":"694b09d6588f9973f6db87902c619ad932c00e9dee54f1ec9a4e4fc5ef09f411"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchAsyncBackendJobExecutionActor.scala:1127-1131 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala:1145-1149 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":1145}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c47998a34f094a7d7e2a3e86e7e063498130c2c06221bddaf319d83172002ce"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchRuntimeAttributes.scala:597-601 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpBatchRuntimeAttributes.scala:271-275 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpBatchRuntimeAttributes.scala"},"region":{"startLine":271}}}],"partialFingerprints":{"codehealthFindingId/v1":"1eee4075d01cb4e52db9f1a93d00df7cc1c2c4a0bb53aaf41d68dece5648d2de"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): supportedBackends/aws/src/main/scala/cromwell/backend/impl/aws/AwsBatchRuntimeAttributes.scala:612-616 | supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpBatchRuntimeAttributes.scala:286-290 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/models/GcpBatchRuntimeAttributes.scala"},"region":{"startLine":286}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5aff781a3ac46de9b703e9b20bd97d7b1d77d43f64cde244862d2a43493981f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): wom/src/main/scala/wom/types/WomFloatType.scala:31-36 | wom/src/main/scala/wom/types/WomIntegerType.scala:39-44 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/types/WomFloatType.scala"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"8462f247e8b58276f64c9a6cca2124139bc2ddf3cd972ad09e9eb94affeb0440"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): wom/src/main/scala/wom/types/WomFloatType.scala:38-42 | wom/src/main/scala/wom/types/WomIntegerType.scala:46-50 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/types/WomFloatType.scala"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"2c2c3f44baeead512c5f227283273e075d1047847a42579fc14b0957031a9b65"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py:192-202 | scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py:262-272 \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":"scripts/metadata_comparison/metadata_comparison/lib/operations_digesters.py"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"adfd55d9ed4a22e8c3c887b6d6371a14b9c0c822c63469b5f41f7f4d4f9fb404"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: TesAsyncBackendJobExecutionActor (LCOM4 7): TesAsyncBackendJobExecutionActor\u0027s methods fall into 7 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 7 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesAsyncBackendJobExecutionActor.scala"},"region":{"startLine":222}}}],"partialFingerprints":{"codehealthFindingId/v1":"db5f91e9b57b289b58f37bf9be9900250193b149da7a94b2b5518eb00c1d3f21"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: WorkflowDockerLookupActor (LCOM4 6): WorkflowDockerLookupActor\u0027s methods fall into 6 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 6 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/WorkflowDockerLookupActor.scala"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"7cf04034a23bccf936b1493f2a05bd9ae72b3ad7aee0cfed84389a33b4e3be99"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: SubWorkflowExecutionActor (LCOM4 6): SubWorkflowExecutionActor\u0027s methods fall into 6 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 6 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/SubWorkflowExecutionActor.scala"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"0feb3e182a5354c91382d6b5f6a2d3827d7fa17d6444391e9caa5428223a7923"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: CloudNioFileSystemProvider (LCOM4 4): CloudNioFileSystemProvider\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-spi/src/main/scala/cloud/nio/spi/CloudNioFileSystemProvider.scala"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc6106336e2b15b6a5ad7c16e43f0f6db461175167951bd0b80aab0d92824797"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: CallCacheHashingJobActor (LCOM4 4): CallCacheHashingJobActor\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/callcaching/CallCacheHashingJobActor.scala"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"015193d7414473c869244d921939cc0e53e553fb8120b32c3c68524b422090a3"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: EngineJobExecutionActor (LCOM4 4): EngineJobExecutionActor\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/job/EngineJobExecutionActor.scala"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"431ce03dd15a72fda7e9c0dda8593de775b3b2124700fee483a5afa0f7fc28e0"}},{"ruleId":"D13","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d4ff50a7aebc43bb5067b09d2fa43f9aa455c7ce500cc5705633c76863098952"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 7 significant file(s) lose their only recent owner: engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/stores/ExecutionStore.scala, services/src/main/scala/cromwell/services/metadata/MetadataService.scala, services/src/main/scala/cromwell/services/metadata/impl/MetadataStatisticsRecorder.scala, supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/runnable/Delocalization.scala, engine/src/main/scala/cromwell/engine/workflow/lifecycle/finalization/CopyWorkflowOutputsActor.scala, engine/src/main/scala/cromwell/engine/workflow/lifecycle/OutputsLocationHelper.scala, supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/runnable/CheckpointingRunnable.scala. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"22a02489bf932cb66193cb9f10681a194cb90c0a75c8c946ba30acaeda4f714c"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 2 other contributor(s) are each the sole owner of a small amount of code below the off-boarding threshold \u2014 folded into the bus-factor score and metrics (10 single-owned of 489 analysed files in total, counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 489 of the 1152 production source files in this repository met that bar). They are anonymized user #2 (2 file(s)), anonymized user #3 (1 file(s)) \u2014 spread or document their files in the same way, at lower priority than the named off-boarding risks above."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6eac528f2429e724e1a97ed868f79eefbcf1888dab4af9a9e673429369bb5b2f"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME - getting pathPrefix to shrink this keeps hosing up, there\u0027s gotta be some way to do this \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/scala/cromiam/webservice/SubmissionSupport.scala"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"2527301f9bb39e9835785ca1b4bb775c01aeeecea5f4e182b8149e727485d132"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: Much like CromwellClient see if there are ways of unifying this a bit w/ the mothership \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/scala/cromiam/webservice/SubmissionSupport.scala"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"9dfd8fb026cb5c876d759b0787eff3a9d0c9ed21e92bae54cb936cdf1d05736c"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: Unify these w/ Cromwell.PartialWorkflowSources (via common?) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/scala/cromiam/webservice/SubmissionSupport.scala"},"region":{"startLine":183}}}],"partialFingerprints":{"codehealthFindingId/v1":"7410fabcff2af4025f4845dc51e46a24c118880c5f909a6516a550cbb4fb1ef4"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: use workbench-model ErrorReport \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/scala/cromiam/webservice/CromIamApiService.scala"},"region":{"startLine":61}}}],"partialFingerprints":{"codehealthFindingId/v1":"109bdbbe16385673f892eb6bef646ba5bf304ec37b4cb11c46a8e08beb640a3e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Re-common-ize swagger out of cromwell\u0027s engine and reuse. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/test/scala/cromiam/webservice/SwaggerUiHttpServiceSpec.scala"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"87cbe211ad0ae8e341c400859d7154986f6b7815e93c674cd6dbe7d9c866447d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: coerceRawValue should return an ErrorOr \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/OutputEvaluator.scala"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a82e83f62170cf5cf005b297b22882608014546f7482e96eaab38187b54b741"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: If a workflow executes jobs using multiple backends, \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/BackendWorkflowInitializationActor.scala"},"region":{"startLine":139}}}],"partialFingerprints":{"codehealthFindingId/v1":"5cfcc091402685a1868a83692f78b8b1e22151f5c1bd199b7fd864493b77bef9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: WOM: WOMFILE: How did a glob get here? Should this link into glob functions to list the globs? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/io/DirectoryFunctions.scala"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"9fb3081b3d9cb45ebfe619715fe689c100224169c227bc6af9ce79a9374ecb50"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Once we figure out premapping and postmapping, maybe we can standardize that behavior. Currently that\u0027s the most important feature that subclasses override. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardExpressionFunctions.scala"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"b81333869e4accfa7700e089e41b93df45e7b74ecef58b0bea56e0f6d7363b94"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Asynchronify \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":617}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3a2163c14d5ff9d73bdec9f25989ce14a483bd57ed45a237069f588b197b108"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO CWL: Is throwing an exception the best way to indicate command generation failure? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala"},"region":{"startLine":767}}}],"partialFingerprints":{"codehealthFindingId/v1":"7cbfedcf5801023c37ead27fadac528f9d84f89050634a14c25521be3fbca6bc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: this mechanism here is very close to the one in CallCacheHashingJobActorData \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/standard/callcaching/StandardCacheHitCopyingActor.scala"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4526320866f202d22d8c4846249d6944b882fc0fd822c6dbdc903a643d98ebd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO enhance to select the best container from the list if multiple are provided. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"backend/src/main/scala/cromwell/backend/validation/Containers.scala"},"region":{"startLine":61}}}],"partialFingerprints":{"codehealthFindingId/v1":"75afb42726a9f70685ed35921681b19824ba71aa1a59d0d5d69a4fd9fb480138"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Do NOT convert to a URI. Do soft parsing. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-spi/src/main/scala/cloud/nio/spi/CloudNioFileSystemProvider.scala"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ddf262d36cc6dbbb1f2c128e495ef7fcc40f11005237e368a3bf6364d3c5366"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Use stronger type for CloudNioFileSystemProvider to avoid casting \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-spi/src/main/scala/cloud/nio/spi/CloudNioFileSystem.scala"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"90909ad2757bb0c8513eb6cfda11d223e28b6e10384be5ee080d9fdda198b70d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: softer parsing using Guava UrlEscapers. May also be better to list the providers ourselves if possible. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cloud-nio/cloud-nio-util/src/main/scala/cloud/nio/util/CloudNioPaths.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"6fa33c100cb58e87485a9ec63891781bc2e271d2d0857ceefcb4cd1745f9aac1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: we could move this into WOM as it\u0027s already there for GraphNodes \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"core/src/main/scala/cromwell/core/CromwellGraphNode.scala"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f4bae63b717c84bbbe15618995001359c21558ea0501e6cbd66335df29e0049"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: rename column via liquibase \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"a12676c8225a52f9ed6f9704b7499fed5011e4eb635ceb6f4057e35bad590fdd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: rename column via liquibase \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"04b63be510bfb32b5fc84b4a44ef80f3a214774ac4f6523a3447cdf55b39e280"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: rename column via liquibase \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc13139ce2406d1a96601f3a29090b92e4d7df68e96ab1b6b0db3c33d10b9d96"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: rename column via liquibase \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/tables/MetadataEntryComponent.scala"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"be80b0bcb58bb18be99960190fc2ba9e44300b640429007bd583cc24febaff98"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Storing times relative to system zone. Look into db/slick using OffsetDateTime, or storing datetimes as UTC? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/sql/SqlConverters.scala"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d2ed23c80c565acd3744afbd3a0da93fb223700f5cf4ba5127e115ff6dbedf9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is not suited for multi backend / multi filesystem use. Keep local for now to not break local CWL conf tests \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/EngineIoFunctions.scala"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"9f897a7618a6c4d3454542a11f6ce567ac9a5fddc423db32c7d8a31296463f57"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // * TODO: enforce a terminal state ? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/instrumentation/WorkflowInstrumentation.scala"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"d74962e0cedfc272be507aed13dd85ab2a36396dc4a9021c503c119b88273d03"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Instead of simply logging to Kibana, figure out a way to tell the user what were the errors somehow (maybe through metadata?) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/WorkflowActor.scala"},"region":{"startLine":587}}}],"partialFingerprints":{"codehealthFindingId/v1":"6c4ff9c442ea95d3b7b0d3421bcf41386ee8ebb644d57e48b69d36d3076fa0f5"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: need to decide where to draw the line between language specific initialization and WOM \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/materialization/MaterializeWorkflowDescriptorActor.scala"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"6d60283ba11be2153b69c6c73881c61b18b318fe81e340d9dd9c704bfc351ad9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: need access to a \u0022source string\u0022 for WomExpressions \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/materialization/MaterializeWorkflowDescriptorActor.scala"},"region":{"startLine":575}}}],"partialFingerprints":{"codehealthFindingId/v1":"c62f1b31ce300843f36f562f3812e53e1e6d0013cf76dbaeafbd88922d92d7ae"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: ErrorOrify this ? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/lifecycle/materialization/MaterializeWorkflowDescriptorActor.scala"},"region":{"startLine":576}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ce82de88044bb0c17ebc025b0b548835691eba2313eb6ce7c6214d2844fa414"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: move constants to WebServiceUtils, adopt in PartialWorkflowSources \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/WomtoolRouteSupport.scala"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"c6abe7f87b496c65ca9e0ec6b8a3ed6f62b7c162279da881c4363b89ad957ee4"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: to handle - page_size, page_token \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala"},"region":{"startLine":241}}}],"partialFingerprints":{"codehealthFindingId/v1":"a339b7c3cb6f4fc0545720d0945767f911e2da032dec89bbcf86927ac21e083e"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: How to handle next_page_token in response? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/wes/WesRouteSupport.scala"},"region":{"startLine":242}}}],"partialFingerprints":{"codehealthFindingId/v1":"5dfed9d3bad4f9de59d69c1c34263d5d948b553132484cc0b54bdbd85822245a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: validate that the probe has left the queue \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/test/scala/cromwell/engine/workflow/tokens/JobTokenDispenserActorSpec.scala"},"region":{"startLine":509}}}],"partialFingerprints":{"codehealthFindingId/v1":"1bc34e37afa57572c71f39ad56fa03be34446ddc0ca3fa5b7a3927d4668f62b4"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate these next two. DefaultPathBuilder does something different from \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"filesystems/s3/src/test/scala/cromwell/filesystems/s3/S3PathBuilderSpec.scala"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"86f43b2855d48303f751ea8cc4ef47ae22f9dd5e542ce8ad72ff685b9f3742e0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: In order for this to pass tests, S3Path needs to implement the \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"filesystems/s3/src/test/scala/cromwell/filesystems/s3/S3PathBuilderSpec.scala"},"region":{"startLine":170}}}],"partialFingerprints":{"codehealthFindingId/v1":"7ecde9221d2d2bf39e8487a84b46c0ffd06b862a5fdbd0151ed0f5764151bd3b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Determine if this is critcal. Note \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"filesystems/s3/src/test/scala/cromwell/filesystems/s3/S3PathBuilderSpec.scala"},"region":{"startLine":216}}}],"partialFingerprints":{"codehealthFindingId/v1":"75a6a93589010b3de2e3d84336f51e26fe20b944b50d6fe3cec190783be25511"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: \u0060getm\u0060 is abandonware and should be removed \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"project/Settings.scala"},"region":{"startLine":129}}}],"partialFingerprints":{"codehealthFindingId/v1":"002799346e284a7342704a3087f8432afcb754042d92abb168099ec1fccc329b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO remove this once we upgrade Akka past 2.5 \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"project/Publishing.scala"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"37aa88c6905bc88d80d852f751eab859e373d437b0fbe9b8713b5afc57d6782a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: We\u0027re not using the \u0022F\u0022 in slf4j. Core only supports logback, specifically the WorkflowLogger. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"project/Dependencies.scala"},"region":{"startLine":420}}}],"partialFingerprints":{"codehealthFindingId/v1":"979cedeb59f967e3b342da2fd04c8fe2bffc76e353e53ac66ba4be40728f9428"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: These tests are (and were) somewhat unsatisfactory. They\u0027d be much better if we use TestFSMRefs and TestProbes to simulate job completions against the WorkflowActor and make sure it only completes the workflow at the appropriate time. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/WorkflowFailSlowSpec.scala"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"56836750da046bcc785280208a8de31eab9a80b5844bb2c7adeef9e3bd264e8e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: restore offending offensive input name \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/SimpleWorkflowActorSpec.scala"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"99af3557b608309fda4aea3df0f51829fc735557ac82b99661e877a7eee9785a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: should be fixed once call input evaluation is properly implemented \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/FilePassingWorkflowSpec.scala"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"1db0ead634c292eea9a34efab2b3a4fc5c4cdbd129d7100c00854fa9b91ae706"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: currently assuming all keys are String. But that\u0027s not WDL-complete... \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/CromwellTestKitSpec.scala"},"region":{"startLine":475}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ae1f2997c88fe561855add1fdc2fe65f2c42a8a61d2d78a7c535337766eb30b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: When re-enabled, this test also needs to check that child processes have actually been stopped. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/engine/WorkflowAbortSpec.scala"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"1dff1cf22281458ded4fdc7dd582dd1bcf1a335d1853b95c33c62fc61af43e50"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: move to wdl4s ? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/engine/EngineFunctionsSpec.scala"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"294cc23242a77bc2ac768575486c224ab6000eac4660589bbe3c6f913c35ffc9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Yes, this might be slow... I\u0027d advocate for refactoring away from the run-a-wdl style, but (shrug) \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/test/scala/cromwell/engine/workflow/lifecycle/execution/WorkflowExecutionActorSpec.scala"},"region":{"startLine":157}}}],"partialFingerprints":{"codehealthFindingId/v1":"48ed45833fd985188e34bec552d7ea9da68a00ba0381b51f2458698019362ab7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO add GCP auth provider \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/auth/GithubAuthVendingSupport.scala"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"e3ae95b0319fbadaf628f99698a9a5b6e78e8959c295bad93f59d10ce4069ec4"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO consider caching this, answers won\u0027t change until we reload the SKUs \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/cost/GcpCostCatalogService.scala"},"region":{"startLine":214}}}],"partialFingerprints":{"codehealthFindingId/v1":"4cc8450f4d7a1922b3ca6dbea610e728b3882a05b2317aa4a26d8df8da79de8e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: cancel any streaming that might be happening? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/archiver/ArchiveMetadataSchedulerActor.scala"},"region":{"startLine":118}}}],"partialFingerprints":{"codehealthFindingId/v1":"27d0d725d841817737bd03a60d74790623065bd0881c2c0ceaef2525e87bec40"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: cancel any deletion action that might be happening? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/main/scala/cromwell/services/metadata/impl/deleter/DeleteMetadataActor.scala"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3226e1e39e90ee87af47ceccc1b279333236b6089748952e3064794c5cb0eb6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // Oops. This column is nullable. TODO: make a changelog to fix, and then remove it from the list \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"services/src/test/scala/cromwell/services/database/LiquibaseComparisonSpec.scala"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"b54c4da8dc794a7795d1331c6f5c661ae2f168d539ea5070cc88114d395d06a8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO is it okay that this would silently ignore user auths if there isn\u0027t one defined in the config list of auths? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchInitializationActor.scala"},"region":{"startLine":96}}}],"partialFingerprints":{"codehealthFindingId/v1":"644f1292e49b398441e4a4ece1b0ae9f93890508d09d0e46d22c9cc7446ee109"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: There is an AWS version of this that looks functionally identical. Consider unifying. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":283}}}],"partialFingerprints":{"codehealthFindingId/v1":"991378d813068c07f57032b9e42c5e390859ea84d1fc9b34a97e41dd3c8aec66"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: FSMify this actor? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/BatchApiRequestWorker.scala"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d51d95c08ab7ec742e98fdc3093093fe653c0dedd972612526d2405d4533bbc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Review this formula, raising the number of workers also raises the delay for a worker to ask for work \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/BatchApiRequestManager.scala"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c2c6283f37a529bc5745a4aa8764f8da95267731a4c4104ada6644641afcceb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: There is a chance we can monitor this operation with \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/BatchRequestExecutor.scala"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"e95910b966c65bafeea2b263783f1ace314830b35081864a577891ac184d4993"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO unused metadata keys below, this is probably not right and there may not be tests to cover these \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/runnable/GcpBatchMetadataKeys.scala"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d2bc0608acba09e9f5414c2845477c0aabfe35cde590246bc8bf1abf78945c8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/test/scala/cromwell/backend/google/batch/actors/GcpBatchInitializationActorSpec.scala"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"8eafa553fc900af957dc3c60ff6e3733d7128226d24e07d2337161d5a9060539"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/test/scala/cromwell/backend/impl/tes/TesInitializationActorSpec.scala"},"region":{"startLine":175}}}],"partialFingerprints":{"codehealthFindingId/v1":"693b05e72bf01d700901e8745ac9f86812c0d5d2953799e72fe6d31a554f0e3e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: this could be done more generally in the StandardCacheHitCopyingActor \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/sfs/src/main/scala/cromwell/backend/sfs/SharedFileSystemCacheHitCopyingActor.scala"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"06f68163bbc75646ab9136b220f133fdf9357a538bbcf29ba17b3ecb94b10d53"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO validate \u0022project\u0022 field of workflowOptions \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesTask.scala"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"18e3ef35aa676a54aba940909da6859ef1b4e4be2e73b7c2d56823097a672725"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO add TES logs to standard outputs \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesTask.scala"},"region":{"startLine":93}}}],"partialFingerprints":{"codehealthFindingId/v1":"66bb7bea95610ea1533bc1e77989e9e8750f4278847c2c8e08178c35b7f2637e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO extract output file variable names and match with Files below \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesTask.scala"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b5fbb2d6791ffc30a5601961182ca1b74be3b23399bf5c49e3554edfaf8cbd3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: this should be pushed back into WOM. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesTask.scala"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"7022bd4e9d87f48233a63e581fab904de3f5c0736e141611df119f6dab9b0302"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO this could be used to create a separate directory for outputs e.g. \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesJobPaths.scala"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c7c772bb73fd7a2e0f5204b8b5615e935937d58c074ee18c5dff4b3e423cf33"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO we would like to use TesRuntimeAttributes.makeBackendParameters to determine which \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesInitializationActor.scala"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"dc66ba9b1d9602c596201949eee8e9f4acb90f2eb84d0ac2de7bc1108e458bdc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: needs runtime attributes validation working again \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/tes/src/test/scala/cromwell/backend/impl/tes/TesInitializationActorSpec.scala"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"fe7630b6ebbe48b5ccc7e261ddf87b1a60f16048b99ef9364cdccb0d02e73523"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO WOM: Unused except in Specs \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlTask.scala"},"region":{"startLine":154}}}],"partialFingerprints":{"codehealthFindingId/v1":"66e1f58a5235a461b58985f648d1d4997ebf4afeb2ab74d01f9814cb141d5b2c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: these next two methods won\u0027t be called by the parser because there are no lists in the WDL grammar that \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/WdlSyntaxErrorFormatter.scala"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"688b764ee2ca87526bb567bb3276329bb3fe4e1b6bdd7e5165d0d211989090ba"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: these next two methods won\u0027t be called by the parser because there are no lists in the WDL grammar that \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/main/scala/wdl/transforms/biscayne/parsing/WdlBiscayneSyntaxErrorFormatter.scala"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"8d5860cb3e717dc80f29867d6e35359d150d9dfd3a71834cc56fa1c9132fbfdd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: these next two methods won\u0027t be called by the parser because there are no lists in the WDL grammar that \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/cascades/src/main/scala/wdl/transforms/cascades/parsing/WdlCascadesSyntaxErrorFormatter.scala"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d3b557f04147f34c73d5199c6a6cbbd1efad087fd4f4fd1cc217460df7bc4f6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: these next two methods won\u0027t be called by the parser because there are no lists in the WDL grammar that \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/draft3/src/main/scala/wdl/draft3/transforms/parsing/WdlDraft3SyntaxErrorFormatter.scala"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"28059df0fd2fecb3f16697ba058762e3106223c82ef9079c66b9111b4bff0c03"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: The below cases, ArrayLiteral and MapLiteral, ObjectLiteral are brittle. They recursively call this womValue(). \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/AstTools.scala"},"region":{"startLine":248}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec3949ebdf54d0cdf67c12c8e865c7ee30ab692a377edb4c5944558773ced6f1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: rename this in the grammar \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/AstTools.scala"},"region":{"startLine":295}}}],"partialFingerprints":{"codehealthFindingId/v1":"733a835247cdd3ab9754b139f964ef73e5806c6427f332affa11b00a70bbcdfc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: return a Try ? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/main/scala/wdl/draft2/model/types/WdlFlavoredWomType.scala"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"85491814508962bc70a6ec7f51a78fb4cf15ab5805c5437eb8bdca42b32d52fc"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: Figure out if anything can be removed from cromwell completely or pulled from here \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/test/scala/wdl/SampleWdl.scala"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"c12c744213fe065206c2aef3cfbd7cd1428990fbbd8d006008e5b0b75334a2f4"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Can replace with \u0022OnlyPureFunctions when that branch merges...\u0022 \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/model/draft2/src/test/scala/wdl/expression/WdlStandardLibraryFunctionsSpec.scala"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc2bdc577142141cb374ed8a19e9136a9ba1f0a8882f76ef5396e52afceccb3f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Sort out stable map ordering \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/biscayne/src/test/scala/wdl/transforms/biscayne/linking/expression/values/BiscayneValueEvaluatorSpec.scala"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2d7416e978ebb1807484f20ea49642c80ca87dabd1184e5afc1a635d2c2b70f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Sort out stable map ordering \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/cascades/src/test/scala/wdl/transforms/cascades/linking/expression/values/CascadesValueEvaluatorSpec.scala"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"7df06f7a3cd83bb68e5a81e390ceeea8368174be81eb409b9854aaee7ec7d65f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO fill in other expression types \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/draft3/src/main/scala/wdl/draft3/transforms/linking/expression/consumed/package.scala"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"96ce1e9feff7e68e0f2a6072e45ca0103708acd18622fe697c85f18fde2c0ba8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO fill in other expression types \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/graph/package.scala"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8ef31cb2c3d67452122f2298666697b275770fa86f639165852259b49fb2c69"}},{"ruleId":"D17","level":"warning","message":{"text":"FixmeComment: // FIXME: how to get a meaningful order from the node set ? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/callable/WorkflowDefinition.scala"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"39ecfbcb08ac992e77b3535072da51f2bf4806f2855a95ffae4e7cc4b7a49dbb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO ErrorOrify this ? Throw for now \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/callable/CommandTaskDefinition.scala"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"48f27776ae4287f4c9dfe20bcb9ca78c17ef2aac737f8c78fd660e0cd840f744"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: It\u0027d be really cool if these could be typed (eg InputPort[WomString], OutputPort[WomInteger] but \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/graph/GraphNodePort.scala"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"084a013a1cb633da069a05fe7479eb0161a8b5d733b857a3a9bf1a2d5c018924"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: we\u0027d have to think about coercion... maybe some sort of implicit CoercionSocket[WomString, WomInteger]...? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/graph/GraphNodePort.scala"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"3534ccd00244f4ead4d281e881b11a77414eb59e2e11f774ba6f9dc67bdc7196"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: FIXME: Do something for PortBasedGraphOutputNodes \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/graph/GraphNode.scala"},"region":{"startLine":117}}}],"partialFingerprints":{"codehealthFindingId/v1":"a646f98df66490eae9f3cea96e5f2e7062f9c9c2c49883083c6adfca70c90037"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This should return an EitherT or whatever we decide we want to use to package Exceptions \u002B Nel[String] \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/graph/expression/ExpressionNode.scala"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"f30c878bebe3154efc051610428c22927f02ca46a3f2ad34e63a29f108a642b9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Try to stream this out to avoid memory overhead \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wom/src/main/scala/wom/values/WomObject.scala"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"994ffbef77be186889013f2793ab887e581c2476e13aba05a5c41020afa3369b"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no licence statement: The BSD 3-Clause license is cited in the README but not shown anywhere else in the documentation. Add a LICENSE file or a link to the LICENSE.txt document so readers can confirm it applies."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"docs/index.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1cce6347862a2ac5f34f14a9abc51f40fe400c407e090ada9691242dc61799b3"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 1 of 2 project(s) overshoot their size bounds, lowering Project Cohesion to 0.0/10. The most over is \u0060(repository root)\u0060 (84917 LoC, 3586 public types across 248 directories). Review these for cohesion \u2014 draw the boundary inside the module first (group each responsibility into its own package or directory and keep the cross-boundary members non-public), since splitting a published package moves types between packages and breaks consumers."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8b96c9276a75f2dbad902ff2520979f36d92ea6ba331b2c685bb9b753ca0513b"},"properties":{"commitSha":"438de9ee40ba3751205008afb916ec1cb96f0c3e"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"19933888ecb1bbd6db61d7e9487d6266381688f50d63f15666ba2cb330be2d8d"},"properties":{"commitSha":"061e3c77ccdca087d1d40198cccc7b3a689e6a90"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d60a3e1e0ad914bfe97d21f3fcd58190544739a87fc67d31d07d0fdfa7b723c4"},"properties":{"commitSha":"6a1b78f1d78156f7b2a2ed105104d90fcb4359f9"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ab133b5370c2392122dedbd8f6868bbc0c9219b14d72a6d152af727d54c3da0f"},"properties":{"commitSha":"b59f065410ec1f6e6fb7d52bc6f5050c4a5f2abe"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"cb89e7fb6d4a38906c371d0fbed442d3790fdf75e4f4c7c1d71d89707efd1e37"},"properties":{"commitSha":"894e10ce0139f8b7dc55f214fca9aa9154654342"}},{"ruleId":"D28","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"5937ee78a2fa56d8d3cac9acefe39fb556df1f81367b19571fba6dfd218f4d70"},"properties":{"commitSha":"7098a662bed58147231234b4eb21f08d53ec9a35"}},{"ruleId":"D28","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"3cdfd7beb31b791ae18dcb425e21eb157c842e7bbcd522cc330573c1a6538db1"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"3acb4fb0b9539542a6b28d0a5e6640ebd160fd98b9d62728e6612fbc3895bece"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"2eb280ff3b4c809d9b59a4e040dbb61c26c0a4b2201d515347385b3ccebbf355"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"622882335896b7cee5a1f2ce07a52328b59f4609f2fbbbb1b2cd9f257fb92028"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"dd1a21fff2e27c07071717d31ff761502daada10b5a6dd34a956123d6a72cf2a"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"da3b2fbb6e13c4d4e60aabc1b66482edd442fe3e893e4d95ff027d8b9e127660"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f4688cf8695f714c6f8592611762869bfb5b22ffdcd325547efd468ab8ce1e50"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7810475ef750505bfd8d2bf44e453c2d0488ee7729cca702456ad1eaf0adb37b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f09045a36cb9ea29980c07fb3a4e2a2ebc4d08bdba9b66325b8bf76bf764f094"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f5b97dab1d8f79b7aedba1d94d971e65ee6d2e700c7f7381c9bbb0af80d1682f"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"30c1f3e4b7f70cffe31089d19c487e65d00ba3842ad51db2955250f561026543"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"985f03277d332782ac46c8ce326c79dce37cbf4cf0dec49066f929f47a50066a"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ef9fda43172baeb1fb854238e9ca84b2bad2316fb1a336901c19d6ad133026d0"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fce27e24cb66421e822902b52fceed67c27f754c218d8a619c73ed616c7f3e54"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f782e4997f43872ef51690f59b33fe3f42ce398916e1d1bb25979dfff8fe8a43"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8bc70c829e14beeb4644ec11167cc743a630568979fa8fd2aab20b6b754628bd"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"774cfe94527683f76083e597ac00b40a9efb7fa3e0c3f1ee0b4eca4589ade258"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d08ede2c4ac7a55f79a4b83b27d40263478dc99aa0af0b779fe410f6afe5abbb"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"24e19c30fac74a6b69d43bdca87b2777df824d8632c69e0d1eb23c57fc0e3955"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d405856519c1425d1266d5f19f9acf86026f25c2a5be73df7e4dec39330e9432"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"cd8e6bc9f31a24f78e043cfe8f8bf5e729ce88b5dcc7af4dfb1a50174e54b4f3"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fba87b3aeffbd331be842bd79c4407990795f59149d74fc17e3d131be789b758"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e78d7266e3da02b40af4fa9e30c4b770da88d24e9aa888e38c35aa5685ecd073"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"44eb3c818864be59bb64b97b83992a01d363e4df0d05bd352d5681eccf400263"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"24a5291230f23a45ccc63ee621e4c9f09433205388787e48cedb75c6d51212cd"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4644e7d15437e507f4e27aa4e1b7b2f6ec354765acee079f45d2faeb93c74daf"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fbc8b200ea39238624f74308836a75c6445397400474a6958fbe68f0599fefa8"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"676429f45fbd245010085269848547cbed7f89cdfa218e4064103f0196a7a727"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"12abb9f530b6b42424d707c3c88913783314500f8c9a3c395fb49de70b390b85"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"6991cfe6a186a4d41455cf59fe8c5b69054fb796f758ce01fe09d16803d885e8"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"66fe0567278a0dfb4c45aa6bd35cf8650ec68d09b06d4ef3bdbacc4fafe1a2c3"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d25355c73e5af413853d5a7a4b6152642dea9e3b835eac05a2f484f4eb116faa"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e23d950ccb0314adbf974479f3572e93a8c86874070463016f852515c8efe630"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b29a023efea6839dedf1ea1135f224cedec8a57d1b6a240549cb97320ebf6fcf"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7d6d0abe400e7ff5451724c2407d213c4a5d4ba0c7f2526bdb012a7910a39cdd"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"30555e0505744bca9a6a2a3bb974c69861a31cfe12a39526574a29bbd5d0b718"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"12673fed1c54644f902b7547f038a5ea34a543e0dd8f443f3bf9dba60afeebfe"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f1066332b699154f2f5aad4d076cb1df17ff0d4b3e511050b7e0a4de268c2d3e"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"11b4930d02acdb87275d51305f3fefddbce9a00d3f71ed6d9dec3fd6b7a63d39"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"67fa6441cd3126b8e58687adef843d7fe113823dce9f2e5e9fd262aec1c9b70e"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b30c0974aa502428c31ff7c4e96054b3526d94c854b77bee76276120fab41216"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"115c3bdcf47ed35a103e36a84104ad27a90ebe5de5cb796c57dbe599f449423c"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"564f16b06edebb962a6c7da6ebdf303d434a66377bff046b644636dc48bc517c"},"taxa":[{"id":"CWE-1104","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-1329","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"be485158bf34369eeee91105177df6290f51102be050b555dd9e2ed7af318385"},"taxa":[{"id":"CWE-1104","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-1329","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"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":"8aea92f3f0e9cafc1864edb07119f209afd18c320f97c2ecdff9c34ce6300195"},"taxa":[{"id":"CWE-319","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":"8067346d5769f59a95fd15872c4e62bcc82598478e418189b7106ab0d5dc5d9e"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-668","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":"63e0ba30c7c63226d940d7dd6672d940366e89e64dd5e72439b04f83a0507a84"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-668","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":"24ce348a92c49a82b766d2f45b8b7cdb0e06a51e71039b5642cefe3737a8484c"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-653","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":"575c5a2cb942b60ae40270ba682a5dde0770f1593ca7d6ef185aa2a242bf1264"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-653","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":"4cb6e998f2a7a2c74e823eb96b7812b95aec80295503ffc3abe5b402c957e057"},"taxa":[{"id":"CWE-250","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-653","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":"da1bf4ea7f39d684eb2946eac2159b5ff953e91f005e6208c84b1689420da422"}},{"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":"bfa438b3e9c536a5be254172382accfd45ea11bcf028d28c3b439bf87363b6be"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"510766656d66a7388647216d8a89045b35ade00ba9ea00fcd6df4ae20ba74203"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4f111da27865cba75c653db97fed151c4922b8cb370fdda3e71fb3cbdf6c3e08"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7fea319677b4536cb919be783dc2dc4e7a4d5e205303204933c91bd645ffc708"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b385964966237ae6204531183ea5d3b6c40c8a53f448e09a768011385a38714"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"9c6bbae010f75608429db3ceb7fe00ea40bfccc48cf9b6f27447b74a4f2238cf"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"40dc3c8b043a48e2a75dee6396dbbdd79e253e548606c78e669fd0106826152f"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"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":"9a12a2df40f79f502c544b956bea122f79aa8065f26a89ad481f4d1eb162ef71"}},{"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":"3ee9ad5903564c6a37804fb94062d404c65371defe2f84a764b481ed83d24671"}},{"ruleId":"D34","level":"note","message":{"text":"Concentrated knowledge decay: 409 of 500 significant files have no living knowledge, while the repository is still being changed at a low rate (10 commit(s) in the last 90 days) \u2014 so this is one repo-wide knowledge-decay state, not 409 separate risks. Counted over 500 of the 1152 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over. The code moved on without the people who understood these files: document them or schedule a read-through before the next change lands in them."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"008bd8d3156a7f5206da709f59f1f87af76671c6b9becea72c8c9cffbcbd4f8e"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"wdl/transforms/new-base/src/main/scala/wdl/transforms/base/linking/expression/values/EngineFunctionEvaluators.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"71ac1aa291d2e7da73f53102c8b6e75100630c518e73286173d86b5ed25ffd38"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/engine/workflow/WorkflowActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fb929497ad3e603a3491c000f34a2f98b40ce4eb91d73236c6926d13e933e7d"}},{"ruleId":"D34","level":"note","message":{"text":"Most significant orphaned file: One of the orphaned files carrying the most lost knowledge \u2014 ranked by size weighted by the file\u0027s role in the codebase, the same weighting behind the score above, so core code outranks equally large plumbing. A reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"filesystems/s3/src/main/java/org/lerch/s3fs/S3FileSystemProvider.java"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"71e98801e3167bf3ceac70a47380d2e3d82397d77ced75a5f98b9b88425a8f99"}},{"ruleId":"D35","level":"error","message":{"text":"Boundary-crossing change coupling: MetadataSqlDatabase.scala \u2194 MetadataDatabaseAccess.scala: \u0060database/sql/src/main/scala/cromwell/database/sql/MetadataSqlDatabase.scala\u0060 (context database) and \u0060services/src/main/scala/cromwell/services/metadata/impl/MetadataDatabaseAccess.scala\u0060 (context services) sit in DIFFERENT parts of the tree yet change together 100% of the time (41 of the 41 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) \u2014 the bounded-context boundary may be in the wrong place, or one context is leaking into the other. This is the behavioural boundary violation a static scan can\u0027t see. You can check this without leaving the row: of the 41 shared commits counted here, the most recent 3 are \u0060f0bf524f\u0060 CTM-4 Remove metadata query row limit (#7878); \u0060c4093a0a\u0060 AN-214 Redirect workflow existence queries from metadata to summary (\u2026; \u0060af9660ee\u0060 WX-1110[risk=low] Added endpoint to fetch failed tasks by root workfl\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/sql/MetadataSqlDatabase.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e83a7e02e1df9463f42bf3b33f4479169ee1cb1b281d117fba44d6f7c5a92c5"}},{"ruleId":"D35","level":"error","message":{"text":"Boundary-crossing change coupling: CallCachingSlickDatabase.scala \u2194 CallCache.scala: \u0060database/sql/src/main/scala/cromwell/database/slick/CallCachingSlickDatabase.scala\u0060 (context database) and \u0060engine/src/main/scala/cromwell/engine/workflow/lifecycle/execution/callcaching/CallCache.scala\u0060 (context engine) sit in DIFFERENT parts of the tree yet change together 64% of the time (9 of the 14 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) \u2014 the bounded-context boundary may be in the wrong place, or one context is leaking into the other. This is the behavioural boundary violation a static scan can\u0027t see. You can check this without leaving the row: of the 9 shared commits counted here, the most recent 3 are \u006039c03e5d\u0060 [CTM-133] Reverse call-caching order and not check old entries (#7834); \u00609b396aac\u0060 Fix call cache checks in case of cache invalidation w/ test [CROM-660\u2026; \u00602d8a0a1b\u0060 WIP \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/slick/CallCachingSlickDatabase.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"64019389ca3915368933ad870c834db75f502d7a49cfdf1ded4e740578d05ccf"}},{"ruleId":"D35","level":"error","message":{"text":"Boundary-crossing change coupling: MetadataSqlDatabase.scala \u2194 ReadDatabaseMetadataWorkerActor.scala: \u0060database/sql/src/main/scala/cromwell/database/sql/MetadataSqlDatabase.scala\u0060 (context database) and \u0060services/src/main/scala/cromwell/services/metadata/impl/ReadDatabaseMetadataWorkerActor.scala\u0060 (context services) sit in DIFFERENT parts of the tree yet change together 58% of the time (7 of the 12 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) \u2014 the bounded-context boundary may be in the wrong place, or one context is leaking into the other. This is the behavioural boundary violation a static scan can\u0027t see. You can check this without leaving the row: of the 7 shared commits counted here, the most recent 3 are \u0060f0bf524f\u0060 CTM-4 Remove metadata query row limit (#7878); \u0060af9660ee\u0060 WX-1110[risk=low] Added endpoint to fetch failed tasks by root workfl\u2026; \u006032d5d0cb\u0060 Archiving scale fixes: Pin fetchSize for DB stream, better metric rep\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"database/sql/src/main/scala/cromwell/database/sql/MetadataSqlDatabase.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"291d208e1934f5fff8fcef107390bccdf506a11b9ed0fcf43c345e89beecf9a3"}},{"ruleId":"D35","level":"error","message":{"text":"Boundary-crossing change coupling: MetadataRouteSupport.scala \u2194 ReadDatabaseMetadataWorkerActor.scala: \u0060engine/src/main/scala/cromwell/webservice/routes/MetadataRouteSupport.scala\u0060 (context engine) and \u0060services/src/main/scala/cromwell/services/metadata/impl/ReadDatabaseMetadataWorkerActor.scala\u0060 (context services) sit in DIFFERENT parts of the tree yet change together 50% of the time (6 of the 12 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) \u2014 the bounded-context boundary may be in the wrong place, or one context is leaking into the other. This is the behavioural boundary violation a static scan can\u0027t see. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u0060dde735a8\u0060 WX-1784 Compute cost from metadata (#7537); \u0060f0a05ca4\u0060 WX-1783 Dummy Cost API (#7508); \u0060af9660ee\u0060 WX-1110[risk=low] Added endpoint to fetch failed tasks by root workfl\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/scala/cromwell/webservice/routes/MetadataRouteSupport.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"94ff9a5ff76e6d64d5adbd90d9091b22d2dada871821a42d1ae3bb014929acdb"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: Dependencies.scala \u2194 Merging.scala: \u0060project/Dependencies.scala\u0060 and \u0060project/Merging.scala\u0060 change together 90% of the time (9 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 9 shared commits counted here, the most recent 3 are \u006082d90116\u0060 CTM-563 Upgrade Jackson, remove Bard et al. (#7894); \u00608cb78dfa\u0060 WX-1763 Update to jakarta Bard client (#7475); \u006023e39191\u0060 WX-1122 Use legacy AppInsights to get better control over logging (#7\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"project/Dependencies.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5aee156d1129164be9832ad0d2d1e07ee542347a4e04620fc214017304d50e14"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: GcpBatchAsyncBackendJobExecutionActor.scala \u2194 BatchRequestExecutor.scala: \u0060supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala\u0060 and \u0060supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/api/request/BatchRequestExecutor.scala\u0060 change together 60% of the time (9 of the 15 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 9 shared commits counted here, the most recent 3 are \u00609f610ee3\u0060 CTM-503 Fix confusing Batch task error message (#7877); \u0060ae5763e7\u0060 [AN-695] Don\u0027t submit duplicate Batch jobs upon restart (#7782); \u0060a1bee937\u0060 AN-428 Automatically resubmit some types of Batch failures (#7706) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"supportedBackends/google/batch/src/main/scala/cromwell/backend/google/batch/actors/GcpBatchAsyncBackendJobExecutionActor.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c78348134a83b2a4d519ee23fe7ebf3a3aa6fcd352a289fcc123e9a9b1a44aa"}},{"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":"759dca709f1a032fb6f624ce672e6afb5558fce53ecf01fb73618c4d33923164"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0752d0df7434000fcabf1048e2de30a7a1a8b982b3db9a19898c4dec199856b4"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d068c977b62d9a977df1bb6f53e8b00b586c3abee955f91d715f92b8e019d624"}},{"ruleId":"M1","level":"warning","message":{"text":"README may be stale: 122 code files changed in the last 6 months but the README was not touched \u2014 it may no longer reflect the system."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e973e7a8d5d866995cbf78b120250fe7f132bf928812d48ca62e69aba11b9910"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add scalafix or scapegoat (or \u0060semgrep --config=auto\u0060, which runs on any language) as a CI step. What was searched, so you can tell an absence from a miss: the 20131 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}},{"ruleId":"S1","level":"warning","message":{"text":"Third-party script without Subresource Integrity: \u0060https://www.gstatic.com/charts/loader.js\u0060 is executed by this page with no Subresource Integrity. Whoever can answer that request \u2014 the CDN, anyone who compromises it, anyone on the network path \u2014 runs arbitrary script in this page\u0027s origin, with its session. The URL also names no version, so it resolves to whatever that origin serves at fetch time \u2014 the executed bytes can change with nobody touching this repository. 2 such include(s) across the repository\u0027s markup."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"engine/src/main/resources/workflowTimings/workflowTimings.html"},"region":{"startLine":4}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c1814831e942c67a1859e398371a2e7838b85335f4889caea67db662980efad"}},{"ruleId":"S1","level":"note","message":{"text":"No security response headers detected: No Content-Security-Policy / X-Frame-Options / X-Content-Type-Options configuration found \u2014 defense in depth, even when a reverse proxy could set them. This is reported because \u0060application.conf\u0060 is committed to this repository and declares the server that serves it, so the configuration that would carry these headers is in this repository and was read in full. (\u22122.0 on this card.)"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"CromIAM/src/main/resources/application.conf"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2041871f9732186de88a057b74e1419c77c628a8644d171f1ee9a8b8cd49e4c"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"id":"CWE-1104","guid":"4c918cb5-b2a6-6c55-9963-a44ee464305e","name":"CWE-1104","shortDescription":{"text":"CWE-1104"},"helpUri":"https://cwe.mitre.org/data/definitions/1104.html"},{"id":"CWE-1329","guid":"f70f1c3f-4ccf-cb5e-bdd3-03ba4868d36d","name":"CWE-1329","shortDescription":{"text":"CWE-1329"},"helpUri":"https://cwe.mitre.org/data/definitions/1329.html"},{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-250","guid":"52ee12e8-390a-7351-b1e6-388afa694e54","name":"CWE-250","shortDescription":{"text":"CWE-250"},"helpUri":"https://cwe.mitre.org/data/definitions/250.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-319","guid":"7af97476-9f0d-4458-9163-3d2043918a1a","name":"CWE-319","shortDescription":{"text":"CWE-319"},"helpUri":"https://cwe.mitre.org/data/definitions/319.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-522","guid":"71fb233e-ce6a-ae57-9419-ef8373540b09","name":"CWE-522","shortDescription":{"text":"CWE-522"},"helpUri":"https://cwe.mitre.org/data/definitions/522.html"},{"id":"CWE-653","guid":"aa3e371b-590e-225d-a56c-cd490a1dca63","name":"CWE-653","shortDescription":{"text":"CWE-653"},"helpUri":"https://cwe.mitre.org/data/definitions/653.html"},{"id":"CWE-668","guid":"e2cb99db-6d24-b056-b0d0-885a733ec403","name":"CWE-668","shortDescription":{"text":"CWE-668"},"helpUri":"https://cwe.mitre.org/data/definitions/668.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-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":74,"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."]}]}