refactor(@sanity): remove warning for when child within PTE has no key

[RitaDias]

Description

Working on removing some warnings from PTE when we can self fix it ourselves.
In this case we are removing the warning when a child doesn't have a _key then the PTE should automatically create one for them.

What to review

Check that it makes sense, there are automated tests.

Testing

If you choose to do it manually you can create a document via de CLI such as:

{
  "_type": "simpleBlock",
  "body": [
    {
      "_key": "a03bf79532e8",
      "_type": "block",
      "children": [
        {
          "_type": "span",
          "marks": [],
          "text": "a test with text"
        }
      ],
      "markDefs": [],
      "style": "normal"
    }
  ],
}

when you enter that document in the test studio you will then be able to see that

1. the warning doesn't appear
2. that when inspecting the doc the child in question will have a key

Notes for release

When child within the PTE is created with a _key, we now automatically create one for it avoiding unnecessary warning.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
performance-studio	✅ Ready (Inspect)	Visit Preview		May 6, 2024 0:38am
test-next-studio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 6, 2024 0:38am
test-studio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 6, 2024 0:38am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
studio-workshop	⬜️ Ignored (Inspect)	Visit Preview		May 6, 2024 0:38am

[github-actions]

No changes to documentation

[github-actions]

Component Testing Report Updated May 6, 2024 12:43 PM (UTC)

File	Status	Duration	Passed	Skipped	Failed
comments/CommentInput.spec.tsx	✅ Passed (Inspect)	42s	14	0	0
formBuilder/ArrayInput.spec.tsx	✅ Passed (Inspect)	6s	3	0	0
formBuilder/inputs/PortableText/Annotations.spec.tsx	✅ Passed (Inspect)	26s	6	0	0
formBuilder/inputs/PortableText/copyPaste/CopyPaste.spec.tsx	✅ Passed (Inspect)	20s	6	3	0
formBuilder/inputs/PortableText/Decorators.spec.tsx	✅ Passed (Inspect)	14s	6	0	0
formBuilder/inputs/PortableText/FocusTracking.spec.tsx	✅ Passed (Inspect)	36s	15	0	0
formBuilder/inputs/PortableText/Input.spec.tsx	✅ Passed (Inspect)	1m 5s	15	0	0
formBuilder/inputs/PortableText/ObjectBlock.spec.tsx	✅ Passed (Inspect)	1m 5s	18	0	0
formBuilder/inputs/PortableText/PresenceCursors.spec.tsx	✅ Passed (Inspect)	7s	3	9	0
formBuilder/inputs/PortableText/RangeDecoration.spec.tsx	✅ Passed (Inspect)	20s	9	0	0
formBuilder/inputs/PortableText/Styles.spec.tsx	✅ Passed (Inspect)	14s	6	0	0
formBuilder/inputs/PortableText/Toolbar.spec.tsx	✅ Passed (Inspect)	22s	9	0	0

[RitaDias]

@skogsmaskin had to add the condition since the tests showed that the editor itself was falling in this condition (since it has no _key itself)

[skogsmaskin]

Great with tests for this, and good catch that we need to test the node for what it is.
I have some small comments on this which I'd like to change.

[skogsmaskin]

Thanks @RitaDias !