carson 2.27.0 → 2.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b07e53b67d21543ee131ce2a8bbc906938fbd657f79973aaa59008911dd14df7
-  data.tar.gz: 73d0811c01b3daa2f397dd545a266085950d93d4cda474ab3288b6b754cd5d20
+  metadata.gz: 4db94cc8153a2add26aa8e34ee4d1232d2211bf0ecc81ed0a806b1b5608b0550
+  data.tar.gz: da51fe2ad074433bd52d83c22c06ed95187ac96745ffe66152a2f01120fdd8ad
 SHA512:
-  metadata.gz: 9880a422abf831615af431cf9b32b6a28a0cd026f2154a46270664deb4d8d32dacf0dac4912a2e64984889372a43bd3cf36cd1cd87899c7b4e3c5cf7ae989fab
-  data.tar.gz: 11e37eb20e6f62dd632493c134218a714dab0a565b533681b1f57b78f28f01dd799786d2d461285dd7623d2832b29f4d3e72f2d95a273f8c80e5869d49f72cd6
+  metadata.gz: 6a1aeea9070859c0781734ed69e7c984054f40dc6c3572b063614c81b10ed75c933989e99c65de749a7d5bcbe0b16e65702020d30fb69d3d8c8d8d9ecbcbab3a
+  data.tar.gz: e2e54a61d3e4d8acf3267f192482d0cc0358b1476f06037933e298a3150282a5c112c7de2ab14c057c930dfe20e05416eececc53824ee54f96872eafdbeead39

data/RELEASE.md

@@ -5,6 +5,39 @@ 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
+
+- **`carson setup` now accepts optional CLI flags** to configure settings without interactive prompts:
+  - `--remote NAME` sets `git.remote`
+  - `--main-branch NAME` sets `git.main_branch`
+  - `--workflow STYLE` sets `workflow.style` (branch or trunk)
+  - `--merge METHOD` sets `govern.merge.method` (squash, rebase, or merge)
+  - `--canonical PATH` sets `template.canonical`
+- When any flag is present, interactive prompts are skipped and only the specified values are written.
+- When no flags are present, existing behaviour is preserved (interactive in TTY, silent detection in non-TTY).
+
+### Why
+
+Running `carson setup` in non-TTY environments (CI pipelines, agent scripts) previously required manually editing `config.json`. CLI flags allow fully automated configuration without file manipulation.
+
+### Migration
+
+No action required. Existing behaviour is unchanged when no flags are provided.
+
 ## 2.27.0 — Absorbed Branch Pruning
 
 ### What changed

data/VERSION

@@ -1 +1 @@
-2.27.0
+2.29.0

data/lib/carson/cli.rb

@@ -53,7 +53,7 @@ module Carson
 
 		def self.build_parser
 			OptionParser.new do |opts|
-				opts.banner = "Usage: carson [setup|audit|sync|prune|onboard [repo_path]|refresh [--all|repo_path]|offboard [repo_path]|template check|template apply|review gate|review sweep|govern [--dry-run] [--json] [--loop SECONDS]|version]"
+				opts.banner = "Usage: carson [setup [--remote NAME] [--main-branch NAME] [--workflow STYLE] [--merge METHOD] [--canonical PATH]|audit|sync|prune|onboard [repo_path]|refresh [--all|repo_path]|offboard [repo_path]|template check|template apply|review gate|review sweep|govern [--dry-run] [--json] [--loop SECONDS]|version]"
 			end
 		end
 
@@ -74,6 +74,8 @@ module Carson
 			when "version"
 				parser.parse!( argv )
 				{ command: "version" }
+			when "setup"
+				parse_setup_command( argv: argv, parser: parser, err: err )
 			when "onboard", "offboard"
 				parse_repo_path_command( command: command, argv: argv, parser: parser, err: err )
 			when "refresh"
@@ -90,6 +92,28 @@ module Carson
 			end
 		end
 
+		def self.parse_setup_command( argv:, parser:, err: )
+			options = {}
+			setup_parser = OptionParser.new do |opts|
+				opts.banner = "Usage: carson setup [--remote NAME] [--main-branch NAME] [--workflow STYLE] [--merge METHOD] [--canonical PATH]"
+				opts.on( "--remote NAME", "Git remote name" ) { |v| options[ "git.remote" ] = v }
+				opts.on( "--main-branch NAME", "Main branch name" ) { |v| options[ "git.main_branch" ] = v }
+				opts.on( "--workflow STYLE", "Workflow style (branch or trunk)" ) { |v| options[ "workflow.style" ] = v }
+				opts.on( "--merge METHOD", "Merge method (squash, rebase, or merge)" ) { |v| options[ "govern.merge.method" ] = v }
+				opts.on( "--canonical PATH", "Canonical template directory path" ) { |v| options[ "template.canonical" ] = v }
+			end
+			setup_parser.parse!( argv )
+			unless argv.empty?
+				err.puts "#{BADGE} Unexpected arguments for setup: #{argv.join( ' ' )}"
+				err.puts setup_parser
+				return { command: :invalid }
+			end
+			{ command: "setup", cli_choices: options }
+		rescue OptionParser::ParseError => e
+			err.puts "#{BADGE} #{e.message}"
+			{ command: :invalid }
+		end
+
 		def self.parse_repo_path_command( command:, argv:, parser:, err: )
 			parser.parse!( argv )
 			if argv.length > 1
@@ -209,7 +233,7 @@ module Carson
 
 			case command
 			when "setup"
-				runtime.setup!
+				runtime.setup!( cli_choices: parsed.fetch( :cli_choices, {} ) )
 			when "audit"
 				runtime.audit!
 			when "sync"

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 ""

data/lib/carson/runtime/setup.rb

@@ -6,13 +6,17 @@ module Carson
 		module Setup
 			WELL_KNOWN_REMOTES = %w[origin github upstream].freeze
 
-			def setup!
+			def setup!( cli_choices: {} )
 				puts_verbose ""
 				puts_verbose "[Setup]"
 
 				unless inside_git_work_tree?
 					puts_line "WARN: not a git repository. Skipping remote and branch detection."
-					return write_setup_config( choices: {} )
+					return write_setup_config( choices: cli_choices )
+				end
+
+				unless cli_choices.empty?
+					return write_setup_config( choices: cli_choices )
 				end
 
 				if self.in.respond_to?( :tty? ) && self.in.tty?

metadata

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