{"$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":"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":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"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":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"ScalaInterpreter.prepareResult (cyclomatic 19): ScalaInterpreter.prepareResult has cyclomatic complexity 19 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala"},"region":{"startLine":199}}}],"partialFingerprints":{"codehealthFindingId/v1":"414da1a564cfd2a5955f0501efa8538c8d3f4c3280f609ce414674c17fbd79f6"}},{"ruleId":"D2","level":"warning","message":{"text":"JupyterSparkMonitorListener.onTaskEnd (cognitive 26): JupyterSparkMonitorListener.onTaskEnd has cognitive complexity 26 (threshold 15). Drivers by points: if/else 8 (9 pts), loops 3 (9 pts), match/switch 2 (7 pts), boolean chains 1 (nesting depth added 12). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"spark-monitor-plugin/src/main/scala/org/apache/toree/plugins/sparkmonitor/JupyterSparkMonitorListener.scala"},"region":{"startLine":518}}}],"partialFingerprints":{"codehealthFindingId/v1":"92c05fb1e2228f671eb1b579fbc58637cec1f871d2f61734a0d3de582a14cd07"}},{"ruleId":"D2","level":"warning","message":{"text":"ScalaInterpreter.prepareResult (cognitive 25): ScalaInterpreter.prepareResult has cognitive complexity 25 (threshold 15). Drivers by points: if/else 11 (19 pts), match/switch 2 (4 pts), boolean chains 2 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala"},"region":{"startLine":199}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c5bde0abd4c8ab77ebf3c958de34d270a3886aa8d76b0a81f00239e4b36b410"}},{"ruleId":"D2","level":"warning","message":{"text":"JupyterSparkMonitorListener.onStageCompleted (cognitive 18): JupyterSparkMonitorListener.onStageCompleted has cognitive complexity 18 (threshold 15). Drivers by points: if/else 5 (12 pts), loops 3 (6 pts) (nesting depth added 10). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"spark-monitor-plugin/src/main/scala/org/apache/toree/plugins/sparkmonitor/JupyterSparkMonitorListener.scala"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b294af5e846cfdafb265173285ec5f4c05050665da293f2c76efb8165c27c73"}},{"ruleId":"D2","level":"warning","message":{"text":"AddJar.execute (cognitive 17): AddJar.execute has cognitive complexity 17 (threshold 15). Drivers by points: if/else 12 (15 pts), boolean chains 2 (nesting depth added 3). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"kernel/src/main/scala/org/apache/toree/magic/builtin/AddJar.scala"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"20477bcfbd76fdc1ec1b39ccb77e69bcd2d4d30945b0606d8827f9e77dffc2e9"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: CommRegistrar: TooManyMethods \u2014 39 methods. The bar is 30 methods; this is 9 over it, 1.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":"protocol/src/main/scala/org/apache/toree/comm/CommRegistrar.scala"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"10f90d917d2db32c1357c3b149ad3e0c7253517b7111e02386b675e2978af107"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: JupyterSparkMonitorListener: ClassTooLong \u2014 478 significant lines (blank, comment-only and punctuation-only lines excluded), 8 methods. The bar is 400 significant lines; this is 78 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":"spark-monitor-plugin/src/main/scala/org/apache/toree/plugins/sparkmonitor/JupyterSparkMonitorListener.scala"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7e0488614a7469808b5b45739ba7031c0ecd130ced453da7230bd47da48ee72"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: sparkmonitor/JupyterSparkMonitorListener.scala: FileTooLong \u2014 521 significant lines (blank, comment-only and punctuation-only lines excluded), about 92% of them inside a single declaration: JupyterSparkMonitorListener (52-731). The bar is 500 significant lines; this is 21 over it, 1.04\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":"spark-monitor-plugin/src/main/scala/org/apache/toree/plugins/sparkmonitor/JupyterSparkMonitorListener.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a3fef48aa74193aa82ca2c6beeeafc8938233cb95077dd39cf5b7a743305fb1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (37 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:404-440 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:408-444 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":404}}}],"partialFingerprints":{"codehealthFindingId/v1":"734070fd59d726a9d9f3aba2c9d98f6b9705f0af60cd9e96c66282a7f474d05a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:221-238 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:223-240 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":221}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0ef52b14c236ad95c0c671414a5945682aeeede1cea0f539e02919eefff6523"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:131-147 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:133-149 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:131\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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c1828ef39ab4c4d0668a0ed736c33785a36884dfa4a624b8ba63ecbb8dabbe4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/socket/SocketConfig.scala:40-51 | kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/socket/SocketConfig.scala:40-51 \u2014 before extracting anything, compare \u0060client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/socket/SocketConfig.scala\u0060 and \u0060kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/socket/SocketConfig.scala\u0060 as WHOLE FILES: 97% 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. 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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/socket/SocketConfig.scala"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"a5fabc35a83be3a8be1f1d81f7e88330105cb596da889b4d711f1622e1745d97"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): plugins/src/main/scala/org/apache/toree/plugins/PluginManager.scala:167-177 | plugins/src/main/scala/org/apache/toree/plugins/PluginManager.scala:204-214 \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 \u0060plugins/src/main/scala/org/apache/toree/plugins/PluginManager.scala:167\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":"plugins/src/main/scala/org/apache/toree/plugins/PluginManager.scala"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"77ed9692f94c31b5eb90c88ab27dd915060512735dd9e46268d4876b64bd9c7e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala:70-79 | kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/Utilities.scala:69-78 \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 \u0060client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala:70\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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"58d08600a6e6733356ef4424e46ca50473bc811ab413d474e99a409ef1c7a661"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala:58-65 | kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/Utilities.scala:56-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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"8fd88fa1231535dec5967d0339ffc00e75161708a1119852699ba14f4672987e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala:84-91 | kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/Utilities.scala:83-90 \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 \u0060client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala:84\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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7dba503f344683cffd61736cae56ad0075bbba7fb08349b8c70c7ddd5baf468"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:77-84 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:79-86 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:77\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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba0cb89d2a664e1da34b5e24417a6d086793512bad1996d5dfbf013c0ecb4001"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 4): communication/src/main/scala/org/apache/toree/communication/actors/DealerSocketActor.scala:40-48 | communication/src/main/scala/org/apache/toree/communication/actors/RepSocketActor.scala:40-48 | communication/src/main/scala/org/apache/toree/communication/actors/ReqSocketActor.scala:40-48 | communication/src/main/scala/org/apache/toree/communication/actors/RouterSocketActor.scala:40-48 \u2014 before extracting anything, compare \u0060communication/src/main/scala/org/apache/toree/communication/actors/DealerSocketActor.scala\u0060 and \u0060communication/src/main/scala/org/apache/toree/communication/actors/ReqSocketActor.scala\u0060 as WHOLE FILES: 85% 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. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"communication/src/main/scala/org/apache/toree/communication/actors/DealerSocketActor.scala"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"573bfbb683c6f8e57e3ef69d4dff63f421541a2c154cc146acfa595d3fa59d5b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:50-57 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:52-59 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"063d0d26476f80a7c0514004ce2cfc9fadefdcefc299c1e75cddfd2ecd7d1341"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:64-73 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:66-75 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"2cb7b0b54ee77c2092b06c8ef56e0b2a17e7e28ec3ae31b7deb98dbee617bbc9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:175-190 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:177-192 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":175}}}],"partialFingerprints":{"codehealthFindingId/v1":"babf24075b5ff3d1629da5af9245f04fb44bfbe2bcdadb0c6fc4c0cb2eed8a3f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:196-206 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:198-208 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"30ed93e43bbcd2526eb3872facadd066bb823aa2df0d22d93d97a0e3ca767dfb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:247-260 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:249-262 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":247}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b61cefcefcae0c99da2868029da95cfb0bfa7e4bd6dc11d9cd3f4379ed72f4f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:345-353 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:349-357 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":345}}}],"partialFingerprints":{"codehealthFindingId/v1":"78638dde4998a7c3f4aa251e285dd050f67d9fa78e23302f7003d9c3455b74ab"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:355-368 | scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala:359-372 \u2014 before extracting anything, compare \u0060scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 and \u0060scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala\u0060 as WHOLE FILES: this scan already matched 11 separate duplicated blocks between them, totalling at least 162 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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":355}}}],"partialFingerprints":{"codehealthFindingId/v1":"64e6000019716b3593388550603f423fac4b65b8a5eaa5a0f4bf274cf8079ae0"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: Kernel (LCOM4 5): Kernel\u0027s methods fall into 5 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 5 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":"kernel/src/main/scala/org/apache/toree/kernel/api/Kernel.scala"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"38550343733bc14116d634152067b60ae46cc35f8a8ee152f413b75204439b8f"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 1 significant file(s) lose their only recent owner: scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ccac9295b5e29d9215688320d12bca75239dd0cd9f89a4b38c6b4f858b19cfdb"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 1 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 (2 single-owned of 72 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; 72 of the 269 production source files in this repository met that bar). They are anonymized user #2 (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":"TodoComment: //  TODO Handle the case where there is no delimiter \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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"b57e54e1464274489d36ec75cefcadac4260cac211cd05a73f1d8628b10f9686"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //  TODO Handle the case where there is no delimiter \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/Utilities.scala"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"bacd9c8be2be423d78e16c770df93ed9ad640ae8442875367e419606342ac9bd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate better solution than setting parentHeader to null for {} \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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/Utilities.scala"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"227d8e3ba21389119ae1dff671fa1d93f679cd893d5d4f17247b9e0a98d7664f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate better solution than setting parentHeader to null for {} \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/Utilities.scala"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"c852e22a408c40ed7a57c59649ad6532c4fb8ba38208825abf40c4a203e1c954"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: hide this? just heartbeat to see if kernel is reachable? \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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/SparkKernelClient.scala"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"1be1327ed4bcb843319212a3b8a49f060b784bead1a627f3131f1fe5163e3547"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Validate incoming message signature \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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/socket/ShellClient.scala"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"a06adae086f8ad9fe30ff639d02304a8849008529af752087dfbac156698db14"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Validate incoming message signature \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":"client/src/main/scala/org/apache/toree/kernel/protocol/v5/client/socket/IOPubClient.scala"},"region":{"startLine":182}}}],"partialFingerprints":{"codehealthFindingId/v1":"a71886604550587eedcf0b7c2be70c0460c34db47fb8820336aed296e2a975f9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is this test necessary? It does nothing. \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":"client/src/test/scala/org/apache/toree/comm/ClientCommWriterSpec.scala"},"region":{"startLine":245}}}],"partialFingerprints":{"codehealthFindingId/v1":"a143badd4748575c37e83bc275aad6210accedf9192f41923f1bb2953b3a7624"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is this test necessary? It does nothing. \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":"kernel/src/test/scala/org/apache/toree/comm/KernelCommWriterSpec.scala"},"region":{"startLine":244}}}],"partialFingerprints":{"codehealthFindingId/v1":"3e69d7f6888290ff6afb39cc28a889cb5704fb122af1446942bf7784a6368631"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is this test necessary? It does nothing. \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":"protocol/src/test/scala/org/apache/toree/comm/CommWriterSpec.scala"},"region":{"startLine":183}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f352389047b42b27f20bb0f9af0760b52346ec329bb173ea66076aded7b4970"}},{"ruleId":"D17","level":"warning","message":{"text":"HackComment: // Hack to allow callbacks to occur after meeting our assertion value \u2014 a workaround marked in source: record what it is compensating for and what would allow its removal (the upstream fix, the API it is waiting on, the invariant it restores), so the next reader can judge whether it is still needed rather than rediscovering why it is there."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/test/scala/org/apache/toree/kernel/protocol/v5/client/execution/DeferredExecutionTest.scala"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb6572508601c409b988b481e7568a6980811cda0bd8f83210107fe269b3add8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Proper error handling for possible exception from mapTo \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":"communication/src/main/scala/org/apache/toree/communication/security/SignatureManagerActor.scala"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b9be11ff570e56901927e1d60792138bd9f6852e56ed7e8502fd3c5f2254158"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This should really be mocked since we don\u0027t care about 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":"communication/src/test/scala/org/apache/toree/communication/security/HmacSpec.scala"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"52ba07385bea4bfd06cbe1c7629cca505487378baf8a8dff0038dff68da19325"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This should really be mocked since we don\u0027t care about 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":"communication/src/test/scala/org/apache/toree/communication/security/HmacSpec.scala"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f2f50e7397f0791e4ef49eec122c3626549dd8c22c6f157be27b310e08339e7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: figure out why this is not excluded. It\u0027s in our build.sbt file \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"kernel-api/src/main/scala/org/apache/toree/dependencies/IvyDependencyDownloader.scala"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"a23e4ef4b4d8cde1c0d2d644f4eb2f415c4c83252f2f63584a61652d03a91969"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: and we exclude all deps there. Need to get rid of this hard-code \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":"kernel-api/src/main/scala/org/apache/toree/dependencies/IvyDependencyDownloader.scala"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"c9adfcd588cd7270a20e4a5e296f91c8fabc90c09d1fbe645f99700830f2098b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Deprecate and remove this package object as it is difficult to \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":"kernel-api/src/main/scala/org/apache/toree/interpreter/package.scala"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"070829852388d6875fe58cd784032354901d72bf5af4686b36a45cb2175b9fb7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Replace with object? \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":"kernel-api/src/main/scala/org/apache/toree/interpreter/ExecuteFailure.scala"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"5253e35806fbd7a9d01a4123fc1064128240e62f98ba2ad1f79a460285ff53b6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Figure out how to dynamically update the output stream used \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":"kernel-api/src/main/scala/org/apache/toree/interpreter/broker/BrokerProcess.scala"},"region":{"startLine":185}}}],"partialFingerprints":{"codehealthFindingId/v1":"473cfcb2eae38529f0d3a7d59e6ab09b4cfb842be14472b0a29cbeabcd7ca556"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Provides an exposed reference to the super loadClass to be stubbed \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":"kernel-api/src/main/scala/org/apache/toree/magic/InternalClassLoader.scala"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a766b08b15c4789f24d144343930678c06459b86e40a9ad679859a136ee5a46"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate why the StackOverflowError occurs in IntelliJ without \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":"kernel-api/src/main/scala/org/apache/toree/security/KernelSecurityManager.scala"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"fbf3db5ec6ca47d9def0efb56a98dec5fdfd0a8cc21b0320835f372c9f8fed11"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate why the StackOverflowError occurs in IntelliJ without \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":"kernel-api/src/main/scala/org/apache/toree/security/KernelSecurityManager.scala"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"826ac9d8e84614ed900fc6eb699b544903b1659dcf6e2291af04d800c51a64d1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Timing issue on Travis CI needs to be resolved \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":"kernel-api/src/test/scala/org/apache/toree/utils/TaskManagerSpec.scala"},"region":{"startLine":211}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7f45659955618d0a8996cdc1f83caa664240f6eecda2de4ace2e5953b2ed723"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Need better way to ensure tasks are still running while \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":"kernel-api/src/test/scala/org/apache/toree/utils/TaskManagerSpec.scala"},"region":{"startLine":233}}}],"partialFingerprints":{"codehealthFindingId/v1":"dcc3205194538d1e17d9aa78352aa7a03f78989686bb9ce44dc6690dbf7a561b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Refactoring task manager to be parallelizable broke this ability \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":"kernel-api/src/test/scala/org/apache/toree/utils/TaskManagerSpec.scala"},"region":{"startLine":289}}}],"partialFingerprints":{"codehealthFindingId/v1":"ee4dc27479289cd816f558406121cb2bc241f849b26dc586dbf14d8d5653a923"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is failing frequently due to some sort of timing problem \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":"kernel-api/src/test/scala/org/apache/toree/utils/ScheduledTaskManagerSpec.scala"},"region":{"startLine":97}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b13a8ab112be0893efa8feaf7de62343e4ca94cc636626e5a387cf433ba9927"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate potential to initialize System out/err/in to capture \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":"kernel/src/main/scala/org/apache/toree/boot/KernelBootstrap.scala"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec2442b1488948dfe84e979d21d9944879953f6dcffed9f2ee0fd2edafa82ca2"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Signals are not a good way to handle this since JVM only has 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":"kernel/src/main/scala/org/apache/toree/boot/layer/HookInitialization.scala"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"924423ce572ef698fbed82d80bd705618ac8180bd1c3734ace53917bc3a7f588"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Cancel group representing current code execution \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":"kernel/src/main/scala/org/apache/toree/boot/layer/HookInitialization.scala"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"d981afccaeebdd5e9080e9f49ec868111700273a8594c42ff7b2e4886098c048"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Cancel group representing current code execution \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":"kernel/src/main/scala/org/apache/toree/boot/layer/HookInitialization.scala"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"237b285976a3d7cedab0082b74e7f1faae1c04e3a81b243b9b73493c54408c3b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Figure out how to pass variable number of arguments to 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":"kernel/src/main/scala/org/apache/toree/boot/layer/HandlerInitialization.scala"},"region":{"startLine":142}}}],"partialFingerprints":{"codehealthFindingId/v1":"41101d13933c6cdbc4938cc3fe09e721b5009684697d97f95681c7bdfe03d488"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Move SparkIMain to private and insert in a different way \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":"kernel/src/main/scala/org/apache/toree/boot/layer/ComponentInitialization.scala"},"region":{"startLine":173}}}],"partialFingerprints":{"codehealthFindingId/v1":"2d169fee787da1949f83e0d98b2c23a946c0bed11a2b3bc035346f42577dbb96"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Construct class server outside of SparkIMain \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":"kernel/src/main/scala/org/apache/toree/boot/layer/ComponentInitialization.scala"},"region":{"startLine":176}}}],"partialFingerprints":{"codehealthFindingId/v1":"c6d620c5bb689486e81f4be1e9c8ba3263ffdd72664585f3a46e4bac85b702cb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Think of a better way to test without exposing 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":"kernel/src/main/scala/org/apache/toree/kernel/api/Kernel.scala"},"region":{"startLine":361}}}],"partialFingerprints":{"codehealthFindingId/v1":"164131d8a5edeee0213e8ae8584c25be1a3349c8918a90f063c1cc906e527587"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Think of a better way to test without exposing 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":"kernel/src/main/scala/org/apache/toree/kernel/api/Kernel.scala"},"region":{"startLine":374}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba2b9ffd56417c73245c87853cacd5de7598482a8baef7f9f1f13b210259eb44"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Inject stream redirect headers in Spark dynamically \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":"kernel/src/main/scala/org/apache/toree/kernel/api/Kernel.scala"},"region":{"startLine":378}}}],"partialFingerprints":{"codehealthFindingId/v1":"f451b6f4eaf7a1ed5153e9f1b66171d6f594ea2b0976ad1262ef5c991c399494"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Exposed for testing purposes. \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":"kernel/src/main/scala/org/apache/toree/kernel/api/Kernel.scala"},"region":{"startLine":395}}}],"partialFingerprints":{"codehealthFindingId/v1":"4885d823290a1968dfe925773344fcafff374ee2fae2340593b8195c94af4f71"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //  TODO Determine if this should be null or an empty parent header \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/dispatch/StatusDispatch.scala"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"eab4d519d900da7c7de2c85411d9a816049e9d8957c256f08b60acdfeede48ff"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO could we use HeaderBuilder 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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/KernelInfoRequestHandler.scala"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"b12c7ab7b08af0a22ff4fd8606e420fe54cfed7cbf2450e7d5e39f3a1f5d6b7f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better way than storing actor refs? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/InputRequestReplyHandler.scala"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7e2197b1c894b66b7f94a54dff9a272fe8f3cc568e9350e043e260b7546bf1b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Add support for error streams \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/ExecuteRequestHandler.scala"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0418657b1d701c2a86c3cb5d2b64379e220002097a2e9cefc9a9530bafcf97c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Should we be checking the return values? Probably not. \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommOpenHandler.scala"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"35ce00fedca6d368803184b7dbe370dd194af3d0648a797ec36591a4301366fe"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Should we be checking the return values? Probably not. \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommMsgHandler.scala"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"739276b6280313302156dcc0a527a2f0b6243327cfd8d6484317a306d80b313b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Should we be checking the return values? Probably not. \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommCloseHandler.scala"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"f10dd7dbecc28c4e249c6bb9ab44f4d80957762891593b193b162eec3fd29166"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Determine proper response for a parse 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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommOpenHandler.scala"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"f052f71d043e9ef47f50d171bc67a1f82d40b0a3b0e0c17de55a8777d4c77e08"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Determine proper response for a parse 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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommMsgHandler.scala"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"d8e4db202b0b5e5deb79e67c2ff3845a9f33b74b167c5c5534ba6f6088025fc6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Determine proper response for a parse 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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/handler/CommCloseHandler.scala"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca1949352c2fac91da988dd7778ba809fa0db9af009af5a1297b514a4451c3bd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate restart sequence \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/interpreter/InterpreterActor.scala"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"517d688ab587848bcede40d9d1f5ce7d255bff722d56bf89728c73bd2c430e3a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Provide a failure message type to be passed around? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/interpreter/tasks/IsCompleteTaskActor.scala"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"e4b27b02d510940a7dff0344844fac129cc107517f6fe9b1c1b3b3e3ee89e2db"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Provide a failure message type to be passed around? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/interpreter/tasks/ExecuteRequestTaskActor.scala"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"4cc9863054d9f9234998b52f7c4736c01099cc7204d0ed59288be166f419866e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Provide a failure message type to be passed around? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/interpreter/tasks/CodeCompleteTaskActor.scala"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3f7b0ae720c4ddf13d81859772f8f07f5d81d03dc721d6b050dd8e579f24123"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Think of a cleaner wrapper to handle updating the Console \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/interpreter/tasks/ExecuteRequestTaskActor.scala"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"051810bb9ad407761c2a4bf01c8ba186c05873e879f055d116a8cba1101d8879"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This assumes NO extra buffers, refactor? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/kernel/socket/ZeromqKernelMessageSocket.scala"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"526b85fdc64e7ad131c9914e117a683125f76f6cbf060a466c689a63b0e9223e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: How to restore this when the actor dies? \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/relay/KernelMessageRelay.scala"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a3833532d0d36cc1e4d5cb801b27e91948d7ca3e5fd31f708b7dc35ff7d0274"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Figure out what the failure message structure should be! \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/relay/KernelMessageRelay.scala"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"842c7b1ebc19fdffd4e5620211655a03b49f191812457c9a4dc3e66f3a782ce8"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Handle error case for mapTo and non-present onFailure \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":"kernel/src/main/scala/org/apache/toree/kernel/protocol/v5/relay/KernelMessageRelay.scala"},"region":{"startLine":171}}}],"partialFingerprints":{"codehealthFindingId/v1":"114cf56f58ab5eb8ab21754eed7119847654d4f7013f1434482a0c20e24f329a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Decide if this.getClass.getClassLoader should just be 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":"kernel/src/main/scala/org/apache/toree/magic/builtin/BuiltinLoader.scala"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"a66c34fae2afe086ca051332e07a8083a20c5adfddabe9eed3101f4b25a32573"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(emcmanus): consider whether to log other failure cases too. \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":"kernel/src/main/scala/org/apache/toree/utils/ClassPath.java"},"region":{"startLine":475}}}],"partialFingerprints":{"codehealthFindingId/v1":"34d1db984468ffcc3dbc64f88c21a4ffa329b16f49631fe37eceb396edd08901"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(b/65488446): Make this a public API. \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":"kernel/src/main/scala/org/apache/toree/utils/ClassPath.java"},"region":{"startLine":657}}}],"partialFingerprints":{"codehealthFindingId/v1":"594de1ca63d15a2e64a893e2925eac194b2ad45b95f8ae6ad334cf45e863c740"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(benyu): Try java.nio.file.Paths#get() when Guava drops JDK 6 support. \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":"kernel/src/main/scala/org/apache/toree/utils/ClassPath.java"},"region":{"startLine":691}}}],"partialFingerprints":{"codehealthFindingId/v1":"b320cf5b3fe57f05b79ec3bff68e09913a4f7de8fd94222f26424a8b555d1019"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Migrate this to a helper method in MessageType.Incoming \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":"kernel/src/main/scala/org/apache/toree/utils/MessageLogSupport.scala"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"4fa7fc2c0c459c9f6698d5f61594db4166a92e871280ee32ba4b3afad8a5a64d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Access the underlying streamType field to assert stderr? \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":"kernel/src/test/scala/org/apache/toree/kernel/api/KernelSpec.scala"},"region":{"startLine":142}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a3a42f6f24ccb38f7ea4213c356e118d43f5cf2b1f480d6ea05310db82efb33"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate if this is still relevant at all \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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/handler/ExecuteRequestHandlerSpec.scala"},"region":{"startLine":207}}}],"partialFingerprints":{"codehealthFindingId/v1":"27842657ab7321ac55d7e9860b1329dcba08ff7f5f30aa2f70f276bea320a517"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate if this is still relevant at all \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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/handler/ExecuteRequestHandlerSpec.scala"},"region":{"startLine":255}}}],"partialFingerprints":{"codehealthFindingId/v1":"c940e12b1686cfe26d4bedde7d7ba70245dd90657f4c3d030cf56b74a7a771db"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better way to test for this without an upper 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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/handler/CommOpenHandlerSpec.scala"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"1118f00c0b6c248cb1ef3df704a2f035c3bb734c59ef931b9ed845107fc54972"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better way to test for this without an upper 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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/handler/CommMsgHandlerSpec.scala"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"384d802f26894b1d19cac1e2ace7b5db842f5d39338fd8ad31dc51398ad96200"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better way to test for this without an upper 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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/handler/CommCloseHandlerSpec.scala"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a69bb3e71e61d0b3307341e72a00768489d12d91263993a3c8570f147935d7d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO test that the response type changed \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":"kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/kernel/socket/IOPubSpec.scala"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"09c115a5e01f227d834e8d0d1be74e60b1cfe7edb4790e44f360beb680ea1fc5"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Stub out UUID method to test id was generated \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":"plugins/src/test/scala/org/apache/toree/plugins/dependencies/DependencySpec.scala"},"region":{"startLine":87}}}],"partialFingerprints":{"codehealthFindingId/v1":"07cc77c5307ad7c08adf31fc82c91370d4a4f4d385f5ae5b22e3ee3b10d7a5bc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Find some class that is in a different classloader and \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":"plugins/src/test/scala/org/apache/toree/plugins/dependencies/DependencyManagerSpec.scala"},"region":{"startLine":310}}}],"partialFingerprints":{"codehealthFindingId/v1":"6fed1d82ee2fd2a9e9042c5578c02729030bde977290bcd797b088639afba139"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is potentially bad design considering appending methods to \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":"protocol/src/main/scala/org/apache/toree/comm/CommManager.scala"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"8895d74f4a2b92705988b3a41df6701d67c2822a4a58c06adcab0494ff5d7fb7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Split this into client/server socket types and move them to their \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/package.scala"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"578d3a6d40ec6e921dda0e85e54ec16150f5adba7a6c2ccc2cbb46f62e5159fb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This can be an empty json object of {} \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/KernelMessage.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad35ccc13da353a95a1332bd53ab78636c66ee191ccac33bc904c8b16b8db2ef"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a way to wrap the Option arguments in Some(...)? \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/content/package.scala"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2dc366ba94f0fd58c7375846f43d96ac2ca8f327cef395db4e6160d74c01b6f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is currently either 0 or 1... should we \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/content/InspectRequest.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"53d27be1a21fa994ed27d5d0a2af26f2de8a40301a94d2c77fd6a8ba88fe82cf"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: look into enforcing that in our schema? \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/content/InspectRequest.scala"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"c02d9a3031aaa4aa23b8e6605158ab30c7967fdde1331ffafcdd08e5bd22f210"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is really (String, String, String | (String, String)), look \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/content/HistoryReply.scala"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"060fe5104489a50c3699eb68fd20c1bac7da25dc21af43dc507eea0a55e9bfc0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: into writing implicits to handle tuples \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":"protocol/src/main/scala/org/apache/toree/kernel/protocol/v5/content/HistoryReply.scala"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e845fb108b1327044fe2107393cd31a11e6fd9a88ab5f3c6bef140bf2701dd2"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better/cleaner way to assert the contents of the callback? \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":"protocol/src/test/scala/org/apache/toree/comm/CommManagerSpec.scala"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"b8e65bd94c72ba216395b781a1e7f768edb946b2164f9c7f8b0a6402082665e3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is there a better/cleaner way to assert the contents of the callback? \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":"protocol/src/test/scala/org/apache/toree/comm/CommManagerSpec.scala"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4a002bb37ece2e7fe70f49c158cfeb63920ad27d5a0f4bbfd904e5a6e2514a3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Move duplicate code to separate project (kernel and client) \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":"protocol/src/test/scala/org/apache/toree/comm/CommCallbacksSpec.scala"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"e9a9eaca447c0acc95f4866c71bd9bb6371286d672da6bb398c8e9dc6e7c371d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Get this API method changed back to public in Apache Spark \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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":199}}}],"partialFingerprints":{"codehealthFindingId/v1":"d52246ab971da3b821a5fbdd356f338851ed2770307e1127f9dcaeade2125f10"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Get this API method changed back to public in Apache Spark \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":"scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":201}}}],"partialFingerprints":{"codehealthFindingId/v1":"75d30e5548af14a87d70ffa69fe50ca0682c36eb4d2592d578eeac8f4410f23f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This wrapper is not needed when just getting compile \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":"scala-interpreter/src/main/scala-2.12/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":430}}}],"partialFingerprints":{"codehealthFindingId/v1":"c4e05425c3b7de10d43bae735ae78bf205e2fd72fc00c67775494eb6c2480562"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This wrapper is not needed when just getting compile \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":"scala-interpreter/src/main/scala-2.13/org/apache/toree/kernel/interpreter/scala/ScalaInterpreterSpecific.scala"},"region":{"startLine":434}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5787fa4e75321a11df1788bad03262c64e3cb7b0d56a1afa6761f6c31c7ac47"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: use SparkContext.setJobGroup to avoid killing all jobs \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":"scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala"},"region":{"startLine":171}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f050f44e32a8952edb9123b619dcf139c333b3bf8855cf1d8e9e434ff39f820"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Investigate why the cluster has to be initialized in the kernel \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":"scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala"},"region":{"startLine":351}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b563b9fa66aa812db3acf6c980cd6f0d0da0ed8ecec3331755fd7c346625cb1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This only adds the context to the main interpreter AND \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":"scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala"},"region":{"startLine":371}}}],"partialFingerprints":{"codehealthFindingId/v1":"458b364da5babca421c67a6302eddd9dd160527745031d8c24e33dc6d25b0d27"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // Todo: rebinding is kinda finicky in Scala 2.11 \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":"scala-interpreter/src/test/scala/integration/interpreter/scala/AddExternalJarMagicSpecForIntegration.scala"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"4a3826100f72d6e003034d888fe87c27d2c1648eaa9417646feac48317ba2d79"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: There is an internal method used for show called showString 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":"sql-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/sql/SqlService.scala"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd091ddfed8b7696a874a972347c6131316904aca916fad8eabd35467eca07aa"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Provide some way to change the number of records shown \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":"sql-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/sql/SqlService.scala"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"27283522da569c5c5a0ecb5d315c5b0b1d84faee79a7f5a14b643db82db0bd91"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Enable once client and kernel have Akka versions that are matching \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":"src/test/scala/system/StdinForSystemSpec.scala"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"87fa3de79032efc392be743b71cbb34927cd341f5ca45eca7b097247302814f3"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d5b8f11381d61a58459ffc7c595fd52d09c03c711894a63a9578043d2338412b"},"taxa":[{"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":"1542ac657294a7960a3ca68607ef7a17d9ed3d2457fb06dcd777067445c6a3cf"},"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":"188b0fa7f5d9a6fe39b67b6c9d2c56717a9c8395e8dfaee45f9a83ed7f988f9a"},"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":"7caadedfbd73389d2832a0eaa411bf995def243204ee30a0483117d44605b75a"},"taxa":[{"id":"CWE-95","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":"c5c6ecb2b0c7e79f4cb591e1e9fa5417659c73a5d89bd63f25c85bcad1dbaae6"}},{"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":"be9c57e2fb625c43c271d051f76b97f27b6976f4ec358ef8fea1d9cb5205f460"},"taxa":[{"id":"CWE-494","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":"d71b791cf9bfa3342aa65392552b2ca117730b949a2b811819d7f4d9e78e196c"}},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"becbc5502f94878197b05d70c7ef15f1353fd99a90efcb824b9c43f11ae5ce7a"}},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fad1e7703c3dc7a8ed264246c27a440927bd557a8a33180ecf25550265e383fb"},"taxa":[{"id":"CWE-1357","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":"b13e8a9414075670ac190ef8c16b893f484962289d123cd949353aecacd322f5"},"taxa":[{"id":"CWE-1357","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":"1ef917966379b04d31f5b5ddacfa29455f5c509294cb1f3c5aadc05fd50215d3"}},{"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":"6f14498c40d014ff296c91cf606cbe4e80389e637657da15f988a030e19d1d4f"}},{"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":"938443bfd56658a1dc4ca28a31056c86bcad72ef912248ad8b76c49a3cb455e5"}},{"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":"be24ce29c415e8fa7166103d78c993ee2fdade4ce64762ac0834b94318e49650"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"9658270ba49f37ed04e99ab1263b6393cd42aed8bdfb7aafd9fa1070f5491895"}},{"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 2338 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":"P4","level":"note","message":{"text":"No release approval gate: The release is automated and no gate that pauses it for a human is DECLARED IN THIS REPOSITORY\u0027S PIPELINE FILES. What was read: every file under \u0060.github/workflows/\u0060, \u0060.forgejo/workflows/\u0060, \u0060.gitea/workflows/\u0060, \u0060.azuredevops/\u0060 and \u0060.azure-pipelines/\u0060, plus \u0060.gitlab-ci*\u0060 and \u0060azure-pipelines*\u0060 \u2014 with comment text stripped, so documenting a gate is not declaring one. What would have counted: GitLab\u0027s \u0060when: manual\u0060, CircleCI\u0027s \u0060type: approval\u0060, an Azure \u0060ManualValidation@\u0060 task or an \u0060approvals:\u0060 block, a Jenkins \u0060input\u0060 step, a \u0060uses:\u0060 step naming an approval action, an \u0060environment:\u0060 paired with \u0060reviewers\u0060 / \u0060required_reviewers\u0060 / \u0060protection\u0060 / \u0060wait-timer\u0060 / \u0060deployment_branch_policy\u0060, a draft-release step, a \u0060workflow_dispatch\u0060 promotion, or a release-event gate. \u2605 What this cannot see, because none of it is a file: a GitHub environment whose required reviewers are configured in repo SETTINGS, a branch protection rule, or an organisation deployment policy \u2014 all of them real, enforced gates that live outside the repository. If yours is one of those, this row is wrong and nothing in the tree could have told us. Otherwise: whatever the release trigger points at is published to users unreviewed, so a mistagged or unverified commit ships and the only remedy is a follow-up release."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6c11e2dbd483b4b423b95ac61bfcc7af1d55351b28a20d2b1105b31cfe38cc60"}}],"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-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-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"},{"id":"CWE-95","guid":"400bba4e-483c-5c50-a695-114fa9168003","name":"CWE-95","shortDescription":{"text":"CWE-95"},"helpUri":"https://cwe.mitre.org/data/definitions/95.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":18,"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."]}]}