Implemented a Delete functionality for KeyValue

[nathannaveen]

Description of the PR

• Fixes [feature] KeyValue add delete node functionality  #1983

PR Checklist

• All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
• All new changes are covered by tests
• If GraphQL schema is changed, make generate has been run
• If GraphQL schema is changed, GraphQL client updates/additions have been made
• If OpenAPI spec is changed, make generate has been run
• If ent schema is changed, make generate has been run
• If collectsub protobuf has been changed, make proto has been run
• All CI checks are passing (tests and formatting)
• All dependent PRs have already been merged

[jeffmendoza]

This isn't correct. It is replacing the vuln and package structs/nodes with a string array. You need to get / modify / and set those structs. Should probably use the setkv() helper (https:/guacsec/guac/blob/main/pkg/assembler/backends/keyvalue/backend.go#L248)

[nathannaveen]

Thank you for this review! This review made me think deeply, and I realized that I was setting the key value wrong, so I have fixed that, as well as modifying the structs and not replacing them.

[jeffmendoza]

This is still not right and doing the same thing.

[nathannaveen]

After some thought, I realize that the code is going to require a lot of changes. So I am going to be converting this to a draft PR.

This is mainly because of a deep dive into the comment #2033 (comment) by @jeffmendoza.

[pxp928]

@nathannaveen this needs a rebase