perf: reduce parse node allocations when deserializing primitive types

[Copilot]

Several TextParseNode methods caused unnecessary intermediate heap allocations. This PR eliminates those allocations across both numeric and string-based value getters.

Changes

Numeric getters — eliminate intermediate *float64 allocations

• GetFloat32Value: parses directly via strconv.ParseFloat(n.value, 32) — eliminates intermediate *float64
• GetFloat64Value: corrects bitSize from 0 → 64; returns &val directly
• GetInt32Value: parses directly via strconv.ParseInt(n.value, 10, 32) — eliminates intermediate *float64
• GetInt64Value: parses directly via strconv.ParseInt(n.value, 10, 64) — eliminates intermediate *float64

String-based getters — eliminate intermediate *string allocations

• GetTimeValue, GetISODurationValue, GetTimeOnlyValue, GetDateOnlyValue, GetUUIDValue, GetEnumValue, GetByteArrayValue: previously called GetStringValue() which allocated a *string on the heap before passing the value to the underlying parser. Each now works with strings.Trim(n.value, "\"") directly — eliminating that pointer allocation.

Before / After

// Before — two allocations (intermediate *float64 + result *int32)
func (n *TextParseNode) GetInt32Value() (*int32, error) {
    v, err := n.GetFloat64Value()   // allocs *float64
    ...
    cast := int32(*v)
    return &cast, nil               // allocs *int32
}

// After — single allocation
func (n *TextParseNode) GetInt32Value() (*int32, error) {
    val, err := strconv.ParseInt(n.value, 10, 32)
    ...
    result := int32(val)
    return &result, nil             // allocs *int32 only
}

// Before — intermediate *string allocation
func (n *TextParseNode) GetTimeValue() (*time.Time, error) {
    v, err := n.GetStringValue()    // allocs *string
    ...
    parsed, err := time.Parse(time.RFC3339, *v)
    return &parsed, err
}

// After — no intermediate allocation
func (n *TextParseNode) GetTimeValue() (*time.Time, error) {
    ...
    val := strings.Trim(n.value, "\"")
    parsed, err := time.Parse(time.RFC3339, val)
    return &parsed, err
}

Also adds unit tests for each modified numeric method.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>reduce parse node allocations when deserializing primitive types</issue_title>
<issue_description>This pull request contains changes that reduce parse node allocations when deserializing primitive types (or collection of primitive types). Replicate the changes in this library without making any changes to public API surfaces. Use conventional commits</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes reduce parse node allocations when deserializing primitive types #127

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[baywet]

@copilot here is the pull request I want you to use as an example and I forgot to include earlier microsoft/kiota-dotnet#678

[Copilot]

@copilot here is the pull request I want you to use as an example and I forgot to include earlier microsoft/kiota-dotnet#678

Applied the same pattern from that PR in commit c2b7755. Methods like GetTimeValue, GetISODurationValue, GetTimeOnlyValue, GetDateOnlyValue, GetUUIDValue, GetEnumValue, and GetByteArrayValue previously called GetStringValue() which allocated an intermediate *string on the heap. They now work with the raw string value directly (strings.Trim(n.value, "\"")) — eliminating that pointer allocation, matching the pattern from the reference PR where static helpers avoid creating intermediate objects.

[sonarqubecloud]

Quality Gate failed

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud