Please address the comments from this code review:
## Overall Comments
- Consider setting isBusy.value = true at the beginning of getUnits to ensure the loading spinner appears while units are being fetched.
- In _buildUnitList you can drop the outer listenVal on units (allUnits) since it’s not used and simplify by listening only to scopeFilter or filteredUnits.
- Instead of nesting your filter chips and spinner inside a single SliverToBoxAdapter Column, consider using separate slivers (e.g., a SliverPadding or SliverToBoxAdapter per widget) for better scroll performance.

## Individual Comments

### Comment 1
<location> `lib/data/provider/systemd.dart:49` </location>
<code_context>
       final systemUnits = <String>[];
       for (final unit in units) {
-        if (unit.startsWith('/etc/systemd/system')) {
+        final maybeSystem = unit.contains('/systemd/system');
+        final maybeUser = unit.contains('/.config/systemd/user');
+        if (maybeSystem && !maybeUser) {
           systemUnits.add(unit);
-        } else if (unit.startsWith('~/.config/systemd/user')) {
+        } else {
           userUnits.add(unit);
-        } else if (unit.trim().isNotEmpty) {
</code_context>

<issue_to_address>
The logic for classifying system/user units may misclassify units with overlapping paths.

Using contains for path matching can lead to false positives if both substrings appear or if the path format changes. More precise matching, such as startsWith or regular expressions, is recommended.
</issue_to_address>

### Comment 2
<location> `lib/data/provider/systemd.dart:83` </location>
<code_context>
     final parsedUnits = <SystemdUnit>[];
-    for (final unit in units) {
-      final parts = unit.split('\n');
+    for (final unit in units.skipWhile((e) => e.trim().isEmpty)) {
+      final parts = unit.split('\n').skipWhile((e) => e.trim().isEmpty);
+      if (parts.isEmpty) continue;
       var name = '';
       var type = '';
</code_context>

<issue_to_address>
Skipping empty units and lines may inadvertently skip valid but empty entries.

Using skipWhile only removes leading empty entries. To exclude all empty entries, use where or another filtering method.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
    for (final unit in units.skipWhile((e) => e.trim().isEmpty)) {
      final parts = unit.split('\n').skipWhile((e) => e.trim().isEmpty);
      if (parts.isEmpty) continue;
      var name = '';
      var type = '';
      var state = '';
=======
    for (final unit in units.where((e) => e.trim().isNotEmpty)) {
      final parts = unit
          .split('\n')
          .where((e) => e.trim().isNotEmpty)
          .toList();
      if (parts.isEmpty) continue;
      var name = '';
      var type = '';
      var state = '';
>>>>>>> REPLACE

</suggested_fix>

### Comment 3
<location> `lib/data/provider/systemd.dart:158` </location>
<code_context>

 String _getIniVal(String line) {
-  return line.split('=').last;
+  final idx = line.indexOf('=');
+  if (idx < 0) return line;
+  return line.substring(idx + 1).trim();
 }
</code_context>

<issue_to_address>
The _getIniVal function may return the entire line if '=' is not found.

Consider returning an empty string or null when '=' is missing, or handle this case explicitly at the call site to avoid propagating unintended values.
</issue_to_address>

### Comment 4
<location> `lib/view/page/systemd.dart:80` </location>
<code_context>
+  Widget _buildUnitList() {
</code_context>

<issue_to_address>
Nesting listenVal calls may lead to unnecessary rebuilds.

Consider refactoring _buildUnitList to use a single listenVal with a derived value, reducing redundant rebuilds and improving performance.

Suggested implementation:

```
  Widget _buildUnitList() {
    return listenVal2(_pro.units, _pro.scopeFilter, (allUnits, filter) {
      final filteredUnits = _pro.getFilteredUnits(allUnits, filter);
      if (filteredUnits.isEmpty) {
        return SliverToBoxAdapter(child: CenterGreyTitle(libL10n.empty).paddingSymmetric(horizontal: 13));
      }
      return SliverList(
        delegate: SliverChildBuilderDelegate((context, index) {
          final unit = filteredUnits[index];
          return ListTile(

```

You will need to implement the `listenVal2` function if it does not already exist. It should take two listenable values and a callback, and rebuild when either changes.

Also, replace `_pro.filteredUnits` with a method like `_pro.getFilteredUnits(allUnits, filter)` that computes the filtered units from the provided values, rather than relying on possibly stale state.

If you have a utility for combining listenables, use that instead of `listenVal2`.
</issue_to_address>