Please address the comments from this code review:
## Overall Comments
- Consider extracting the GBK/UTF-8 decoding fallback logic into a helper function to improve readability and reuse.
- Remove the flutter/material.dart import from the server provider since it’s not used in this context.
- Use the existing Logger instead of debugPrint for error output to keep logging consistent across the app.

## Individual Comments

### Comment 1
<location> `lib/data/provider/server.dart:469` </location>
<code_context>
+              Loggers.app.warning('GBK decoding failed', e2);
+            }
+          }
+          raw = rawStr;
+        } else {
+          raw = execResult.toString();
</code_context>

<issue_to_address>
Potential for raw to be null if both decoding attempts fail.

If decoding fails for both encodings, rawStr stays null and may cause issues later. Please add explicit handling or a fallback to prevent silent errors.
</issue_to_address>

### Comment 2
<location> `lib/data/provider/server.dart:496` </location>
<code_context>
       sv.status.err = SSHErr(type: SSHErrType.getStatus, message: e.toString());
       _setServerState(s, ServerConn.failed);
       Loggers.app.warning('Get status from ${spi.name} failed', e);
+      debugPrint('错误 $raw');

       // Update SSH session status to disconnected on status error
</code_context>

<issue_to_address>
Consider removing or gating debugPrint for production builds.

Debug statements like debugPrint should be removed or conditionally executed to avoid cluttering production logs.
</issue_to_address>

Please address the comments from this code review:
## Overall Comments
- Remove devtools_options.yaml from the commit as it’s an IDE-specific config and not relevant to the GBK decoding fix.
- Consider extracting the UTF-8/GBK decoding fallback logic into a separate utility or extension method for improved readability and easier testing.
- Handle the null execResult case explicitly instead of calling toString() (which yields "null"), to avoid confusion when parsing downstream.

## Individual Comments

### Comment 1
<location> `lib/data/provider/server.dart:454` </location>
<code_context>
+          bool needGbk = false;
+          try {
+            rawStr = utf8.decode(execResult, allowMalformed: true);
+            if (rawStr.runes.where((c) => c == 0xfffd).length > 5 || rawStr.contains('��')) {
+              needGbk = true;
+            }
</code_context>

<issue_to_address>
Threshold for malformed UTF-8 detection may need adjustment.

The threshold for replacement characters may not suit all cases. Consider making it configurable or clarifying why this value was chosen.

Suggested implementation:

```
          // The threshold for malformed UTF-8 detection is configurable via malformedThreshold.
          try {
            rawStr = utf8.decode(execResult, allowMalformed: true);
            final malformedCount = rawStr.runes.where((c) => c == 0xfffd).length;
            if (malformedCount > malformedThreshold || rawStr.contains('��')) {
              needGbk = true;
            }

```

1. Add an integer parameter `malformedThreshold` (with a default value of 5) to the containing function or class.
2. Pass `malformedThreshold` where this function is called, if you want to override the default.
3. Document the parameter in the function/class docstring to clarify its purpose.
</issue_to_address>

### Comment 2
<location> `lib/data/provider/server.dart:463` </location>
<code_context>
+          }
+          if (needGbk) {
+            try {
+              rawStr = gbk.decode(execResult);
+            } catch (e2) {
+              Loggers.app.warning('GBK decoding failed', e2);
</code_context>

<issue_to_address>
GBK decoding fallback is a good idea, but may mask other encoding issues.

Consider logging the original bytes or adding more context to error messages to help diagnose cases where both UTF-8 and GBK decoding fail.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
            } catch (e2) {
              Loggers.app.warning('GBK decoding failed', e2);
              rawStr = 'decode failed, please submit the issues with the log';
            }
=======
            } catch (e2) {
              Loggers.app.warning(
                'GBK decoding failed after UTF8 fallback. Original bytes: ${execResult}',
                e2,
              );
              rawStr = 'decode failed, please submit the issues with the log. Original bytes: ${execResult}';
            }
>>>>>>> REPLACE

</suggested_fix>

### Comment 3
<location> `lib/data/provider/server.dart:469` </location>
<code_context>
+              rawStr = 'decode failed, please submit the issues with the log';
+            }
+          }
+          raw = rawStr ?? 'decode failed, please submit the issues with the log';
+        } else {
+          raw = execResult.toString();
</code_context>

<issue_to_address>
Error message could be more actionable for users.

Specify which log users should submit and provide instructions or a path to help them locate it.

Suggested implementation:

```
              rawStr = 'Decode failed. Please submit an issue and attach the application log file (see Help > View Log File in the menu) for troubleshooting.';

```

```
          raw = rawStr ?? 'Decode failed. Please submit an issue and attach the application log file (see Help > View Log File in the menu) for troubleshooting.';

```
</issue_to_address>