carson 2.8.0 → 2.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c3f3bf2f0ea58a2d460c36fb57b6313d5dab97c08b76ec94ae1c1e9b685b24f
-  data.tar.gz: 48562197450233540c28a6fa912c38eaa636910ed3bab57e32e95d7b0b66c10a
+  metadata.gz: 76c0a5589cad84c4aaba1fee42bd667a21a55cea4840092b2984ae9eaeb84f6c
+  data.tar.gz: 656462b041f6764fd38780b500945ca0389b8235d741805367613051a27bd9da
 SHA512:
-  metadata.gz: 63a4d9fed728d78b2fcaf0d0d6ffb158a93e464af24997d661b24d77a2764e4a84014f45d4cfe2e204c7bac337c796f8e69e3722fe30a135f98650176271dba3
-  data.tar.gz: aba034301bccbf786198a1a44bd767d3f239370807a68841a89b6ffc40340719ef37dea2ddae9a0d54eedadf1b42c2446d02614721ab946586b809de0438a2d4
+  metadata.gz: 14cc66d4582c9ddfb9c5e87ec8e550d1f97cbcf6701a499e730dea4c06b5598ccdc6e8f61057a921ddba2d6ac9799fed4b3191a1a6dc1e440c4bc7249eec06bd
+  data.tar.gz: ce5e6f40342c08378830563ae0188cf61964d1f4650cc0be47a8cf52cb72f02b4f6413fa2b8d15a3dc70e3714628897446ab852ab7b276ac68a98cff0e5d1d54

data/RELEASE.md

@@ -5,6 +5,48 @@ 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.8.1 — Onboard UX and Install Cleanup
+
+### What changed
+
+- **Concise onboard output.** `carson onboard` now prints a clean 8-line summary instead of verbose internal state (hook paths, template statuses, config lines). Tells users what happened, what needs attention, and what to do next.
+- **Graceful handling of fresh repos.** Onboard no longer fails with a fatal error on repositories with no commits yet.
+- **Suppressed RubyGems PATH warning.** The misleading `WARNING: You don't have ... in your PATH, gem executables will not run` message from `gem install --user-install` is now suppressed during installation. Carson symlinks the executable to `~/.carson/bin`, making the gem bin directory irrelevant.
+
+### What users must do now
+
+1. Upgrade Carson to `2.8.1`.
+
+### Breaking or removed behaviour
+
+- None.
+
+### Upgrade steps
+
+```bash
+cd ~/Dev/carson
+git pull
+bash install.sh
+carson version
+```
+
+### Engineering Appendix
+
+#### Modified components
+
+- `lib/carson/runtime.rb` — added `concise?` flag and `with_captured_output` helper for suppressing sub-command detail during onboard.
+- `lib/carson/runtime/local.rb` — rewrote `onboard!` to use concise orchestration (`onboard_apply!`), added `onboard_report_remote!` and `onboard_run_audit!` helpers, added `unless concise?` guards to `prepare!` and `template_apply!`.
+- `install.sh` — capture `gem install` stderr and filter out RubyGems PATH warning.
+- `script/install_global_carson.sh` — same PATH warning suppression.
+- `test/runtime_govern_test.rb` — updated onboard output assertions.
+
+#### Public interface and config changes
+
+- No new CLI commands or config keys.
+- Exit status contract unchanged.
+
+---
+
 ## 2.8.0 — Interactive Setup and Remote Detection
 
 ### What changed

data/VERSION

@@ -1 +1 @@
-2.8.0
+2.8.1

data/lib/carson/runtime/local.rb

@@ -153,26 +153,31 @@ module Carson
 					target_path = File.join( hooks_dir, hook_name )
 					FileUtils.cp( source_path, target_path )
 					FileUtils.chmod( 0o755, target_path )
-					puts_line "hook_written: #{relative_path( target_path )}"
+					puts_line "hook_written: #{relative_path( target_path )}" unless concise?
 				end
 				git_system!( "config", "core.hooksPath", hooks_dir )
 				File.write( File.join( hooks_dir, "workflow_style" ), config.workflow_style )
-				puts_line "configured_hooks_path: #{hooks_dir}"
+				puts_line "configured_hooks_path: #{hooks_dir}" unless concise?
+				return EXIT_OK if concise?
+
 				inspect!
 			end
 
 			# One-command onboarding for new repositories: detect remote, install hooks,
-			# apply templates, and produce a first audit report.
+			# apply templates, and run initial audit.
 			def onboard!
 				fingerprint_status = block_if_outsider_fingerprints!
 				return fingerprint_status unless fingerprint_status.nil?
 
-				print_header "Onboard"
 				unless inside_git_work_tree?
 					puts_line "ERROR: #{repo_root} is not a git repository."
 					return EXIT_ERROR
 				end
 
+				repo_name = File.basename( repo_root )
+				puts_line ""
+				puts_line "Onboarding #{repo_name}..."
+
 				unless global_config_exists?
 					if self.in.respond_to?( :tty? ) && self.in.tty?
 						setup_status = setup!
@@ -182,21 +187,9 @@ module Carson
 					end
 				end
 
-				report_detected_remote!
-				hook_status = prepare!
-				return hook_status unless hook_status == EXIT_OK
-
-				template_status = template_apply!
-				return template_status unless template_status == EXIT_OK
-
-				audit_status = audit!
-				if audit_status == EXIT_OK
-					puts_line "OK: Carson onboard completed for #{repo_root}."
-				elsif audit_status == EXIT_BLOCK
-					puts_line "BLOCK: Carson onboard completed with policy blocks; resolve and rerun carson audit."
-				end
-				print_onboarding_guidance
-				audit_status
+				onboard_apply!
+			ensure
+				@concise = false
 			end
 
 			# Re-applies hooks, templates, and audit after upgrading Carson.
@@ -287,29 +280,29 @@ module Carson
 				fingerprint_status = block_if_outsider_fingerprints!
 				return fingerprint_status unless fingerprint_status.nil?
 
-				print_header "Template Sync Apply"
+				print_header "Template Sync Apply" unless concise?
 				results = template_results
 				applied = 0
 				results.each do |entry|
 					if entry.fetch( :status ) == "error"
-						puts_line "template_file: #{entry.fetch( :file )} status=error reason=#{entry.fetch( :reason )}"
+						puts_line "template_file: #{entry.fetch( :file )} status=error reason=#{entry.fetch( :reason )}" unless concise?
 						next
 					end
 
 					file_path = File.join( repo_root, entry.fetch( :file ) )
 					if entry.fetch( :status ) == "ok"
-						puts_line "template_file: #{entry.fetch( :file )} status=ok reason=in_sync"
+						puts_line "template_file: #{entry.fetch( :file )} status=ok reason=in_sync" unless concise?
 						next
 					end
 
 					FileUtils.mkdir_p( File.dirname( file_path ) )
 					File.write( file_path, entry.fetch( :applied_content ) )
-					puts_line "template_file: #{entry.fetch( :file )} status=updated reason=#{entry.fetch( :reason )}"
+					puts_line "template_file: #{entry.fetch( :file )} status=updated reason=#{entry.fetch( :reason )}" unless concise?
 					applied += 1
 				end
 
 				error_count = results.count { |entry| entry.fetch( :status ) == "error" }
-				puts_line "template_apply_summary: updated=#{applied} error=#{error_count}"
+				puts_line "template_apply_summary: updated=#{applied} error=#{error_count}" unless concise?
 				error_count.positive? ? EXIT_ERROR : EXIT_OK
 			end
 
@@ -680,12 +673,66 @@ module Carson
 				end
 			end
 
-			def print_onboarding_guidance
-				puts_line ""
-				puts_line "Carson is ready. Current workflow: #{config.workflow_style}"
+			# Concise onboard orchestration: hooks, templates, remote, audit, guidance.
+			def onboard_apply!
+				@concise = true
+
+				hook_status = prepare!
+				return hook_status unless hook_status == EXIT_OK
+				puts_line "Hooks installed (#{config.required_hooks.count} hooks)."
+
+				template_drift_count = template_results.count { |entry| entry.fetch( :status ) != "ok" }
+				template_status = template_apply!
+				return template_status unless template_status == EXIT_OK
+				if template_drift_count.positive?
+					puts_line "Templates synced (#{template_drift_count} file#{plural_suffix( count: template_drift_count )} updated)."
+				else
+					puts_line "Templates in sync."
+				end
+
+				onboard_report_remote!
+				audit_status = onboard_run_audit!
+
 				puts_line ""
-				puts_line "Reconfigure anytime with: carson setup"
-				puts_line "Run carson refresh after changing config."
+				puts_line "Carson is ready. Workflow: #{config.workflow_style}"
+				puts_line "Reconfigure anytime: carson setup"
+				audit_status
+			end
+
+			# Friendly remote status for onboard output.
+			def onboard_report_remote!
+				if git_remote_exists?( remote_name: config.git_remote )
+					puts_line "Remote: #{config.git_remote} (connected)."
+				else
+					puts_line "Remote not configured yet — carson setup will walk you through it."
+				end
+			end
+
+			# Runs audit with captured output; reports summary instead of full detail.
+			def onboard_run_audit!
+				audit_error = nil
+				audit_status = with_captured_output { audit! }
+			rescue StandardError => e
+				audit_error = e
+				audit_status = EXIT_OK
+			ensure
+				return onboard_print_audit_result( status: audit_status, error: audit_error )
+			end
+
+			def onboard_print_audit_result( status:, error: )
+				if error
+					if error.message.to_s.match?( /HEAD|rev-parse/ )
+						puts_line "No commits yet — run carson audit after your first commit."
+					else
+						puts_line "Audit skipped — run carson audit for details."
+					end
+					return EXIT_OK
+				end
+
+				if status == EXIT_BLOCK
+					puts_line "Some checks need attention — run carson audit for details."
+				end
+				status
 			end
 
 			# Uses `git remote get-url` as existence check to avoid parsing remote lists.

data/lib/carson/runtime.rb

@@ -4,6 +4,7 @@
 require "fileutils"
 require "json"
 require "open3"
+require "stringio"
 require "time"
 
 module Carson
@@ -28,6 +29,7 @@ module Carson
 			@out = out
 			@err = err
 			@in = in_stream
+			@concise = false
 			@config = Config.load( repo_root: repo_root )
 			@git_adapter = Adapters::Git.new( repo_root: repo_root )
 			@github_adapter = Adapters::GitHub.new( repo_root: repo_root )
@@ -37,6 +39,22 @@ module Carson
 
 		attr_reader :repo_root, :tool_root, :out, :err, :in, :config, :git_adapter, :github_adapter
 
+		# Returns true when output should be minimal (used by onboard/refresh).
+		def concise?
+			@concise
+		end
+
+		# Runs a block with all output captured (suppressed from the user).
+		# Returns the block's return value; output is silently discarded.
+		def with_captured_output
+			saved_out, saved_err = @out, @err
+			@out = StringIO.new
+			@err = StringIO.new
+			yield
+		ensure
+			@out, @err = saved_out, saved_err
+		end
+
 		# Current local branch name.
 		def current_branch
 			git_capture!( "rev-parse", "--abbrev-ref", "HEAD" ).strip

metadata

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