Ids of new cells must be provided to actions

[DonJayamanne]

For #9340 (more PRs to come, this PR focuses on some minor refactoring).

• When adding a new cell, the ids must be provided to the redux store actions, instead of the actions generating their own ids.
• Any operation that results in creation of a cell will ensure a id for the cell is provided
• DeleteAllCells is an operation that has a side effect of creating a new cell, hence this must provide the id of the new cell that will be created.

[codecov-io]

Codecov Report

Merging #9833 into ds/custom_editor will decrease coverage by <.01%.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##           ds/custom_editor    #9833      +/-   ##
====================================================
- Coverage             60.64%   60.64%   -0.01%     
====================================================
  Files                   555      555              
  Lines                 29726    29726              
  Branches               4479     4479              
====================================================
- Hits                  18028    18027       -1     
- Misses                10697    10699       +2     
+ Partials               1001     1000       -1

Impacted Files	Coverage Δ
...ience/interactive-common/interactiveWindowTypes.ts	100% <ø> (ø)	⬆️
src/client/common/utils/platform.ts	70.58% <0%> (-5.89%)	⬇️
src/client/constants.ts	80% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74ec45f...ab3a09c. Read the comment docs.

[rchiodo]

[IanMatthewHuff]

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information