carson 2.28.0 → 2.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7117079168b4f8472a4324f954bc087122e0ded0d253252b73702c41d3996283
-  data.tar.gz: 21484878bcafe5799dfd87a18c223b5982d6b2ff1ab988c78f929de9cd642bdd
+  metadata.gz: 4db94cc8153a2add26aa8e34ee4d1232d2211bf0ecc81ed0a806b1b5608b0550
+  data.tar.gz: da51fe2ad074433bd52d83c22c06ed95187ac96745ffe66152a2f01120fdd8ad
 SHA512:
-  metadata.gz: bb4b347d7ace09997a93b40fc7ec179baf4eb8aee75a620e3511d4ac7b9c46545380f7157ac442807e09cc1a679e8709a62df1a237a41618501688162f99c511
-  data.tar.gz: 91aaf60433a82548170eb66eef14ce90c0eb730a3a56e06f7c154449d2355ca50f9a1e282d1b26e372c34b1df2946cb804587854875c14d1ffec3da5d6658bff
+  metadata.gz: 6a1aeea9070859c0781734ed69e7c984054f40dc6c3572b063614c81b10ed75c933989e99c65de749a7d5bcbe0b16e65702020d30fb69d3d8c8d8d9ecbcbab3a
+  data.tar.gz: e2e54a61d3e4d8acf3267f192482d0cc0358b1476f06037933e298a3150282a5c112c7de2ab14c057c930dfe20e05416eececc53824ee54f96872eafdbeead39

data/RELEASE.md

@@ -5,6 +5,18 @@ Release-note scope rule:
 - `RELEASE.md` records only version deltas, breaking changes, and migration actions.
 - Operational usage guides live in `MANUAL.md` and `API.md`.
 
+## 2.29.0 — Audit Concise Output UX
+
+### What changed
+
+- **"Checks: skipped" and "Baseline: skipped" messages are no longer shown in concise audit output.** If checks cannot run (no PR, no required checks, `gh` unavailable), the audit stays silent instead of surfacing a misleading "skipped" line. Verbose mode still logs the skip reason for debugging.
+- **Audit state no longer bumps to "attention" for skipped checks.** A skip is a non-event, not a concern.
+- **Failing and pending check names are now included in concise messages.** Before: `Baseline (main): 1 failing — fix before merge.` After: `Baseline (main): 1 failing (lint) — fix before merge.` Same treatment applies to PR required checks and advisory baseline messages.
+
+### Migration
+
+- No action required. This is a display-only change to concise audit output.
+
 ## 2.28.0 — Non-Interactive CLI Flags for Setup
 
 ### What changed

data/VERSION

@@ -1 +1 @@
-2.28.0
+2.29.0

data/lib/carson/runtime/audit.rb

@@ -54,42 +54,49 @@ module Carson
 				puts_verbose ""
 				puts_verbose "[PR and Required Checks (gh)]"
 				monitor_report = pr_and_check_report
-				audit_state = "attention" if audit_state == "ok" && monitor_report.fetch( :status ) != "ok"
-				if monitor_report.fetch( :status ) == "skipped"
-					audit_concise_problems << "Checks: skipped (#{monitor_report.fetch( :skip_reason )})."
-				elsif monitor_report.fetch( :status ) == "attention"
+				audit_state = "attention" if audit_state == "ok" && !%w[ok skipped].include?( monitor_report.fetch( :status ) )
+				if monitor_report.fetch( :status ) == "attention"
 					checks = monitor_report.fetch( :checks )
 					fail_n = checks.fetch( :failing_count )
 					pend_n = checks.fetch( :pending_count )
 					total = checks.fetch( :required_total )
+					fail_names = checks.fetch( :failing ).map { |e| e.fetch( :name ) }.join( ", " )
 					if fail_n.positive? && pend_n.positive?
-						audit_concise_problems << "Checks: #{fail_n} failing, #{pend_n} pending of #{total} required."
+						audit_concise_problems << "Checks: #{fail_n} failing (#{fail_names}), #{pend_n} pending of #{total} required."
 					elsif fail_n.positive?
-						audit_concise_problems << "Checks: #{fail_n} of #{total} failing."
+						audit_concise_problems << "Checks: #{fail_n} of #{total} failing (#{fail_names})."
 					elsif pend_n.positive?
 						audit_concise_problems << "Checks: pending (#{total - pend_n} of #{total} complete)."
-					elsif checks.fetch( :status ) == "skipped"
-						audit_concise_problems << "Checks: skipped (#{checks.fetch( :skip_reason )})."
 					end
 				end
 				puts_verbose ""
 				puts_verbose "[Default Branch CI Baseline (gh)]"
 				default_branch_baseline = default_branch_ci_baseline_report
-				audit_state = "attention" if audit_state == "ok" && default_branch_baseline.fetch( :status ) != "ok"
+				audit_state = "attention" if audit_state == "ok" && !%w[ok skipped].include?( default_branch_baseline.fetch( :status ) )
 				baseline_st = default_branch_baseline.fetch( :status )
 				if baseline_st == "block"
 					parts = []
-					parts << "#{default_branch_baseline.fetch( :failing_count )} failing" if default_branch_baseline.fetch( :failing_count ).positive?
-					parts << "#{default_branch_baseline.fetch( :pending_count )} pending" if default_branch_baseline.fetch( :pending_count ).positive?
+					if default_branch_baseline.fetch( :failing_count ).positive?
+						names = default_branch_baseline.fetch( :failing ).map { |e| e.fetch( :name ) }.join( ", " )
+						parts << "#{default_branch_baseline.fetch( :failing_count )} failing (#{names})"
+					end
+					if default_branch_baseline.fetch( :pending_count ).positive?
+						names = default_branch_baseline.fetch( :pending ).map { |e| e.fetch( :name ) }.join( ", " )
+						parts << "#{default_branch_baseline.fetch( :pending_count )} pending (#{names})"
+					end
 					parts << "no check-runs for active workflows" if default_branch_baseline.fetch( :no_check_evidence )
 					audit_concise_problems << "Baseline (#{default_branch_baseline.fetch( :default_branch, config.main_branch )}): #{parts.join( ', ' )} — fix before merge."
 				elsif baseline_st == "attention"
 					parts = []
-					parts << "#{default_branch_baseline.fetch( :advisory_failing_count )} advisory failing" if default_branch_baseline.fetch( :advisory_failing_count ).positive?
-					parts << "#{default_branch_baseline.fetch( :advisory_pending_count )} advisory pending" if default_branch_baseline.fetch( :advisory_pending_count ).positive?
+					if default_branch_baseline.fetch( :advisory_failing_count ).positive?
+						names = default_branch_baseline.fetch( :advisory_failing ).map { |e| e.fetch( :name ) }.join( ", " )
+						parts << "#{default_branch_baseline.fetch( :advisory_failing_count )} advisory failing (#{names})"
+					end
+					if default_branch_baseline.fetch( :advisory_pending_count ).positive?
+						names = default_branch_baseline.fetch( :advisory_pending ).map { |e| e.fetch( :name ) }.join( ", " )
+						parts << "#{default_branch_baseline.fetch( :advisory_pending_count )} advisory pending (#{names})"
+					end
 					audit_concise_problems << "Baseline (#{default_branch_baseline.fetch( :default_branch, config.main_branch )}): #{parts.join( ', ' )}."
-				elsif baseline_st == "skipped"
-					audit_concise_problems << "Baseline: skipped (#{default_branch_baseline.fetch( :skip_reason )})."
 				end
 				if config.template_canonical.nil? || config.template_canonical.to_s.empty?
 					puts_verbose ""

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: carson
 version: !ruby/object:Gem::Version
-  version: 2.28.0
+  version: 2.29.0
 platform: ruby
 authors:
 - Hailei Wang