[WIP] Inline identity function and remove its definition

[Copilot]

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

Inline the identity function at its only call site and remove its definition from value/contained/types.go.

Steps:

• The generic function identity is currently defined as:
  func identity[T comparable](t T) (T, error) {
  return t, nil
  }
• It is only used once, as FromString: identity[string], in the String() TypeInfo constructor for string.
• Replace this use with an inline anonymous function: FromString: func(t string) (string, error) { return t, nil },
• Remove the definition of identity entirely.

Ensure no other code references to identity remain in value/contained/types.go. Do not make changes outside this file.

This pull request was created as a result of the following prompt from Copilot chat.

Inline the identity function at its only call site and remove its definition from value/contained/types.go.

Steps:

• The generic function identity is currently defined as:
  func identity[T comparable](t T) (T, error) {
  return t, nil
  }
• It is only used once, as FromString: identity[string], in the String() TypeInfo constructor for string.
• Replace this use with an inline anonymous function: FromString: func(t string) (string, error) { return t, nil },
• Remove the definition of identity entirely.

Ensure no other code references to identity remain in value/contained/types.go. Do not make changes outside this file.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.