fix: make RecoveryAddress.ID optional to prevent UUID parsing error

[blackshady]

Related issue(s)

Checklist

• I have read the contributing guidelines.
• I have referenced an issue containing the design document if my change
  introduces a new feature.
• I am following the
  contributing code guidelines.
• I have read the security policy.
• I confirm that this pull request does not address a security
  vulnerability. If this pull request addresses a security vulnerability, I
  confirm that I got the approval (please contact
  [email protected]) from the maintainers to push
  the changes.
• I have added tests that prove my fix is effective or that my feature
  works.
• I have added or changed the documentation.

Further Comments

This PR fixes a bug where RecoveryAddress.ID is treated as required, causing the following error when omitted:

uuid: incorrect UUID length 0 in string ""

This breaks identity creation via kratos-client-go and direct API calls when the id field is not set.

The issue is explained here:
🔗 #4454

This fix makes the field optional to allow proper recovery address creation and aligns the behavior with VerifiableAddress, which had a similar bug.

[gaultier]

Thank you for your contribution!
It looks good to me, can you just run make sdk format to regenerate the SDK and auto-format? Thank you!

[blackshady]

Done! I’ve run make sdk format and pushed the updated files. Thanks!

[aeneasr]

@gaultier @jonas-jonas you need to approve these PRs in the internal repo, not here!

[icamys]

Hey folks! What's the status of this PR?
@gaultier @jonas-jonas

[gaultier]

I am working on getting this merged internally.

[gaultier]

Thank you for your contribution, this has been merged just now: 6f5e79a (I had to do it myself to fix merge conflicts with files OSS contributors do not have access to).