feat: added exit code 8 for successfully fixed violations (#4674)

Author: Jkhall81

src/ansiblelint/app.py

@@ -198,7 +198,10 @@ def report_outcome(
     ) -> int:
         """Display information about how to skip found rules.
 
-        Returns exit code, 2 if errors were found, 0 when only warnings were found.
+        Returns exit code:
+          - 0: when no errors were found
+          - 2: if errors were found
+          - 8: if all errors were fixed automatically
         """
         msg = ""
 
@@ -260,6 +263,8 @@ def report_outcome(
                 files_count,
                 is_success=mark_as_success,
             )
+        if not summary.failures and changed_files_count > 0:
+            return RC.FIXED_VIOLATIONS
         if mark_as_success:
             if not files_count:
                 # success without any file being analyzed is reported as failure

src/ansiblelint/constants.py

@@ -37,6 +37,7 @@ class RC:  # pylint: disable=too-few-public-methods
     INVALID_CONFIG = 3
     LOCK_TIMEOUT = 4
     NO_FILES_MATCHED = 5
+    FIXED_VIOLATIONS = 8
     EXIT_CONTROL_C = 130
 
 

src/ansiblelint/schemas/__store__.json

@@ -4,7 +4,7 @@
     "url": "https://raw.githubusercontent.com/ansible/ansible-lint/main/src/ansiblelint/schemas/ansible-lint-config.json"
   },
   "ansible-navigator-config": {
-    "etag": "730a0572f872af3ce271fb5116bc9d4609da10dc2e35448b5ec1050f12b72e2b",
+    "etag": "5f454fdaeb3d92c4bb2cc15dcc48faa67925fecfbdb72efffeddee01860d6bcb",
     "url": "https://raw.githubusercontent.com/ansible/ansible-navigator/main/src/ansible_navigator/data/ansible-navigator.json"
   },
   "changelog": {
@@ -24,7 +24,7 @@
     "url": "https://raw.githubusercontent.com/ansible/ansible-lint/main/src/ansiblelint/schemas/inventory.json"
   },
   "meta": {
-    "etag": "fa83d61b00b254d54a7c9a69e4fb8b722f356adec906da7beb733cadda1342bc",
+    "etag": "2f5c7deb174136b48a4bc1d8d4399b1a175379dd4b8a863baa9c9bd56ffda489",
     "url": "https://raw.githubusercontent.com/ansible/ansible-lint/main/src/ansiblelint/schemas/meta.json"
   },
   "meta-runtime": {

src/ansiblelint/schemas/ansible-navigator-config.json

@@ -1,7 +1,7 @@
 {
     "$schema": "http://json-schema.org/draft-07/schema",
     "additionalProperties": false,
-    "description": "See https://docs.ansible.com/projects/navigator/settings/",
+    "description": "See https://ansible.readthedocs.io/projects/navigator/settings/",
     "properties": {
         "ansible-navigator": {
             "additionalProperties": false,
@@ -525,7 +525,7 @@
     "required": [
         "ansible-navigator"
     ],
-    "title": "ansible-navigator settings v25",
+    "title": "ansible-navigator settings v26",
     "type": "object",
-    "version": "25"
+    "version": "26"
 }

test/test_app.py

@@ -52,3 +52,24 @@ def test_with_inventory_concurrent_syntax_checks(tmp_path: Path) -> None:
         # https://github.com/ansible/ansible-lint/issues/4446.
         assert "AttributeError" not in result.stderr
         counter += 1
+
+
+def test_app_fixed_violations(tmp_path: Path) -> None:
+    """Validate that linter returns RC.FIXED_VIOLATIONS (8) when all issues are fixed."""
+    # 1. Create a playbook with an auto-fixable error (FQCN)
+    # Note: We include 'changed_when: false' so no unfixable errors remain
+    lintable = Lintable(tmp_path / "fixable_playbook.yml")
+    lintable.content = """---
+- name: Test Autofix
+  hosts: localhost
+  tasks:
+    - name: Use shell instead of FQCN
+      command: echo "Hello World"
+      changed_when: false
+"""
+    lintable.write(force=True)
+
+    result = run_ansible_lint(lintable.filename, "--fix", cwd=tmp_path)
+
+    assert result.returncode == RC.FIXED_VIOLATIONS
+    assert "Modified 1 file." in result.stderr