Greptile Summary

This PR closes a crash bug (#693) by wrapping showEditDialog() in a try-catch, making it consistent with every other entityToConfig call site in the ViewModel. The fix is minimal, targeted, and follows the established error-handling pattern already used by saveInterface, deleteInterface, and other methods.

Key changes:

• showEditDialog() now catches any Exception thrown by entityToConfigState() (e.g. IllegalArgumentException from a corrupted type field in Room DB)
• On failure: logs the error via Log.e and surfaces a user-visible message via showError() — the dialog stays closed, no crash
• On success: behaviour is identical to before

Confidence Score: 5/5

• Safe to merge — single-function change with no behavioural impact on the happy path.
• The change is a one-function, one-site addition of a try-catch that exactly mirrors the existing error-handling pattern used throughout the rest of the ViewModel. There is no change to the success path, no new state introduced, and the error path correctly leaves the dialog closed and config state untouched for the next operation. All existing tests pass.
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant UI as UI / Composable
    participant VM as InterfaceManagementViewModel
    participant Repo as InterfaceRepository
    participant DB as Room DB

    UI->>VM: showEditDialog(interfaceEntity)

    alt Happy path (valid entity)
        VM->>Repo: entityToConfig(entity)
        Repo->>DB: lookup type field
        DB-->>Repo: valid type string
        Repo-->>VM: InterfaceConfig (typed)
        VM->>VM: _configState.value = entityToConfigState(...)
        VM->>VM: _state.value.copy(showAddDialog=true, editingInterface=entity)
        VM-->>UI: dialog opens
    else Corrupted entity (e.g. bad `type` field) — NEW behaviour
        VM->>Repo: entityToConfig(entity)
        Repo-->>VM: throws IllegalArgumentException
        VM->>VM: Log.e(TAG, ...)
        VM->>VM: showError("Failed to load interface: …")
        VM->>VM: _state.value.copy(errorMessage=…)
        VM-->>UI: error toast / snackbar shown, dialog stays closed
    end

_{Reviews (1): Last reviewed commit: "fix: catch exception in showEditDialog t..." | Re-trigger Greptile}

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!