BUG-2: Fix empty slos:[] when slo_type=error_rate in generate_slo_manifest

Co-authored-by: chefgs <7605658+chefgs@users.noreply.github.com>

Author: Copilot

CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
+## [0.4.2] - 2026-03-18
+
+### Fixed
+- **`scaffold sre` empty `slos: []` when `--slo-type error_rate`** — `generate_slo_manifest()` in
+  `cli/scaffold_sre.py` had no branch for the `error_rate` SLO type, leaving `slo.yaml` with an
+  empty `slos: []` list. Sloth/OpenSLO tooling therefore received no SLO objectives. A dedicated
+  `error_rate` SLO entry (with matching SLI queries and alerting config) is now generated when
+  `--slo-type error_rate` is specified.
+
+---
+
 ## [0.4.1] - 2026-03-17
 
 ### Changed

cli/scaffold_sre.py

@@ -347,6 +347,28 @@ def generate_slo_manifest(args):
             },
         })
 
+    if args.slo_type in ("error_rate",):
+        slos.append({
+            "name": "error_rate",
+            "description": f"{name} error rate SLO — error rate stays below {round(100 - args.slo_target, 4)}%",
+            "objective": args.slo_target,
+            "sli": {
+                "events": {
+                    "error_query": f"rate(http_requests_total{{job=\"{name}\",status=~\"(5..)\"}}[{{{{.window}}}}])",
+                    "total_query": f"rate(http_requests_total{{job=\"{name}\"}}[{{{{.window}}}}])",
+                }
+            },
+            "alerting": {
+                "name": f"{name.title()}ErrorRateSLO",
+                "labels": {"team": args.team},
+                "annotations": {
+                    "runbook": f"https://wiki.example.com/runbooks/{name}/error-rate",
+                },
+                "page_alert": {"labels": {"severity": "critical"}},
+                "ticket_alert": {"labels": {"severity": "warning"}},
+            },
+        })
+
     if args.slo_type in ("latency", "all"):
         slos.append({
             "name": "latency",

tests/test_comprehensive.py

@@ -854,16 +854,6 @@ def test_slo_manifest_latency_entry(self):
         slo_names = [s["name"] for s in manifest["slos"]]
         assert "latency" in slo_names
 
-    # BUG-2: error_rate SLO type produces empty slos list in manifest
-    @pytest.mark.xfail(
-        strict=True,
-        reason=(
-            "BUG-2: generate_slo_manifest() only checks for 'availability' and "
-            "'latency' slo_type values. When slo_type='error_rate', neither "
-            "condition matches so the slos list is empty. An error_rate SLO "
-            "entry should be generated for slo_type='error_rate'."
-        ),
-    )
     def test_slo_manifest_error_rate_bug(self):
         """
         BUG-2: When slo_type='error_rate', generate_slo_manifest() should