From 760bc1beb2e42b6ddc90382c3be16396d854f2e3 Mon Sep 17 00:00:00 2001
From: pitboss <pitboss@local>
Date: Fri, 15 May 2026 18:58:53 -0500
Subject: [PATCH] [pitboss] sweep after phase 28: 2 deferred items resolved

---
 src/utils/config.rs | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/utils/config.rs b/src/utils/config.rs
index e88f19a1..b956e511 100644
--- a/src/utils/config.rs
+++ b/src/utils/config.rs
@@ -1707,6 +1707,17 @@ fn runs_config_defaults() {
     assert!(cfg.save_code_snippets);
 }
 
+#[test]
+fn output_config_preserves_chain_constituents_by_default() {
+    // Phase 25 deferred decision (b): the default keeps every constituent
+    // finding in the `findings: [...]` array so existing pipelines see no
+    // behavioural change. Flipping this to `false` is a deliberate breaking
+    // change and must be done explicitly, not silently. Guarding both the
+    // `Default` impl and the serde-default getter so neither drifts alone.
+    assert!(OutputConfig::default().show_chain_constituents);
+    assert!(default_show_chain_constituents());
+}
+
 #[test]
 fn server_config_toml_roundtrip() {
     let toml_str = r#"