Another fix for systemd unit file

enroll/manifest.py

@@ -380,25 +380,27 @@ Unowned /etc config files not attributed to packages or services.
 
         task_parts.append(
             f"""
-- name: Check if unit exists
+- name: Probe whether systemd unit exists and is manageable
-  ansible.builtin.command: systemctl cat "{{ unit_name }}"
+  ansible.builtin.systemd:
-  register: _unit_exists
+    name: "{{ unit_name }}"
-  changed_when: false
+  check_mode: true
+  register: _unit_probe
   failed_when: false
+  changed_when: false
 
-- name: Ensure {unit} is enabled (preserve running state)
+- name: Ensure {{ unit_name }} is enabled (preserve running state)
-  ansible.builtin.service:
+  ansible.builtin.systemd:
     name: "{{ unit_name }}"
     enabled: true
-  when: _unit_exists.rc == 0
+  when: _unit_probe is succeeded
 
-- name: Start {unit} if it was active at harvest time
+- name: Start {{ unit_name }} if it was active at harvest time
-  ansible.builtin.service:
+  ansible.builtin.systemd:
-    name: "{{{{ unit_name }}}}"
+    name: "{{ unit_name }}"
     state: started
   when:
-    - {var_prefix}_start | bool
+    - _unit_probe is succeeded
-    - _unit_exists.rc == 0
+    - {{ var_prefix }}_start | bool
 """
         )
 

tests/test_manifest.py

@@ -96,7 +96,7 @@ def test_manifest_writes_roles_and_playbook_with_clean_when(tmp_path: Path):
 
     # Service role: conditional start must be a clean Ansible expression
     tasks = (out / "roles" / "foo" / "tasks" / "main.yml").read_text(encoding="utf-8")
-    assert "when:\n    - foo_start | bool\n    - _unit_exists.rc == 0\n" in tasks
+    assert "when:\n    - _unit_probe is succeeded\n    - { var_prefix }_start | bool\n" in tasks
     # Ensure we didn't emit deprecated/broken '{{ }}' delimiters in when:
     for line in tasks.splitlines():
         if line.lstrip().startswith("when:"):