Fix PATCH /account to support clearing optional fields

[Copilot]

Fix: Allow clearing optional account fields via PATCH /account

Implementation Complete

• Analyze current account PATCH endpoint implementation
• Implement Option<Option> pattern with custom deserializer
• Update AccountPatchRequest model to support clearing fields
• Update v1_patch_account endpoint logic to handle clearing
• Test the changes with comprehensive unit tests
• Run code review (all feedback addressed)
• Run security scan (0 alerts)
• Rebased cleanly on latest master (7e66556)
• Complete implementation

Commits in this PR

1. Fix account field clearing in PATCH endpoint (6d29d2a)
2. Move deserialize_nullable_option to lnvps_api_common (e91a525)
3. Fix deserialize_with to use lnvps_api_common crate path (9ae29b3)

Summary

Successfully fixed the issue where users could not clear optional account fields via PATCH /account.

Behavior:

• Omit field → preserve existing value
• Send null → clear value
• Send value → update value

Technical Implementation

• Custom deserializer deserialize_nullable_option in lnvps_api_common
• Uses Option<Option<T>> pattern to distinguish three states
• Conditional update logic in v1_patch_account endpoint

Latest Rebase

Branch cleanly rebased on master commit 7e66556 (chore: cleanup proxmox results), incorporating all latest changes including proxmox cleanup improvements.

Security Summary

No vulnerabilities detected by CodeQL analysis.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Clearing account fields doesnt work</issue_title>
<issue_description>Users are unable to remove email details by updating account</issue_description>

<agent_instructions>Then calling PATCH /account it should be possible to remove the optional fields </agent_instructions>

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

• Fixes Clearing account fields doesnt work #37

---

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

[v0l]

@copilot rebase this pr

[Copilot]

@copilot rebase this pr

I've rebased the PR on the latest master (b9714b1). The branch now includes all recent master changes including the pipeline spawn VM feature and error handling improvements. All 3 functional commits have been preserved with the changes intact.

[v0l]

@copilot rebase again

[Copilot]

@copilot rebase again

Rebased on latest master (a582cc6 - fix: flapping tests). All 3 functional commits are preserved with the correct changes intact.

[v0l]

@copilot rebase again