Fix Remaining Code Review Issues

Please address the following issues found in the code review:

1. Missing status_display population in instance_resize_get function

File: src/handlers/instances.rs (around line 395)

In the instance_resize_get function, after retrieving the status from the API, we need to populate the status_display field with the formatted status value. This is currently done correctly in other handler functions like instance_change_pass_get and instance_change_pass_post, but is missing in the resize handler.

Fix needed: After line 395 where instance.status is set, add:

instance.status_display = crate::utils::format_status(&instance.status);

2. Add documentation for format_status function

File: src/utils/status_formatter.rs

The public format_status function lacks documentation. Add a doc comment explaining:

• What the function does
• The parameter it accepts
• What it returns
• Examples of transformations

Example documentation:

/// Formats a raw status string into a human-readable display format.
///
/// This function converts status values from their API representation (e.g., "preparing_disk")
/// into user-friendly display strings (e.g., "Preparing Disk").
///
/// # Arguments
///
/// * `status` - A string slice containing the raw status value
///
/// # Returns
///
/// A formatted String suitable for display to users. If the status is not recognized,
/// it returns the original status unchanged.
///
/// # Examples
///
/// ```
/// let formatted = format_status("preparing_disk");
/// assert_eq!(formatted, "Preparing Disk");
///
/// let formatted = format_status("active");
/// assert_eq!(formatted, "Active");
/// ```
pub fn format_status(status: &str) -> String {

3. Refactor duplicated InstanceView initialization

Files: src/handlers/instances.rs (lines 241, 279, 388) and src/services/instance_service.rs (line 73)

The default InstanceView initialization pattern is duplicated in multiple locations. Extract this into a helper method on the InstanceView struct or a standalone function to improve maintainability.

Suggested approach: Add a method to the InstanceView struct in src/models/instance_view.rs:

impl InstanceView {
    /// Creates a new InstanceView with default values for the given instance ID.
    pub fn new_with_defaults(instance_id: String) -> Self {
        Self {
            id: instance_id,
            hostname: "(no hostname)".into(),
            region: "".into(),
            main_ip: None,
            main_ipv6: None,
            status: "".into(),
            status_display: "".into(),
            vcpu_count_display: "—".into(),
            ram_display: "—".into(),
            disk_display: "—".into(),
            os: None,
        }
    }
}

Then replace all occurrences of the manual initialization with:

let mut instance = InstanceView::new_with_defaults(instance_id.clone());

---

Priority: High for issue #1, Medium for issues #2 and #3.

Testing: After making these changes, verify that:

1. The resize page displays formatted status values correctly
2. All existing functionality continues to work
3. The code compiles without errors