SDK - Components - Fix - Stop serializing string values

[Ark-kun]

This can happen with Lightweight component outputs if they've already been serialized manually.

---

This change is 

[numerology]

/lgtm

Just to confirm, this won't actually change the behavior because json.dump a string still returns the same string, right?

[Ark-kun]

Just to confirm, this won't actually change the behavior because json.dump a string still returns the same string, right?

It changes/fixes the behavior of the serializer since json.jumps('string') == '"string"' - the quotes are added.

Fortunately, the problem was never triggerable in most code paths, because the serialize_value was checking the object type and never run serializers on strings. The only code path where this became a problem was introduced just 2 days ago in #2198 .

[Ark-kun]

/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Ark-kun

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• sdk/OWNERS [Ark-kun]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Ark-kun]

/retest