carson 4.3.4 → 4.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9a188fed6453dc86ca1da87766cf9e594e3eb8a92fccdc091fa6d9daeca57fcf
-  data.tar.gz: a817693746cbd73a690c16de3f75e5e6b0ec6312d42a94f1cec0b02ed418ff89
+  metadata.gz: b22cfd5f4976a5367d0a815fac4f360049673aaa585a3783c7296f155be896a9
+  data.tar.gz: bdc3f70b4b74dcb489b8dde16ca0244846b4082fc6697713ce83490cb64319c4
 SHA512:
-  metadata.gz: cd7ce61eadfac8406738d2eb8bfaa15c18ec64fca01912e2e01866e91c362bec82b62708863fd6cffd8f7f1d7e3d55efd8240f911a141f29ec792af8e1d5341b
-  data.tar.gz: 83c23c3de5d30229a718c393201a9f0af70c8a3336266baacc0ac891b771fd8f27a6d924baa84a098363e2cb121cbebdfed24cbfac758eb10236e265b959e1d9
+  metadata.gz: a77aa24657d4863d5de2a06e14b14e0f339cd9b9ac709933f2b2ec084de721a101781867a7b3a4bb0557623968247ca4333cf5821660778ce0ddd70c20f780ba
+  data.tar.gz: 5113678e26302f1eb7b9a5c93e59f4a357434ddade3ec9b4eec00f412c1af8b77888b132ab5f849594e04621d0af57e8e9969f3e0292b23b18bb215894a1dac9

data/RELEASE.md

@@ -7,25 +7,36 @@ Release-note scope rule:
 
 ## Unreleased
 
-### Removed
+## 4.4.0
 
-- **`carson audit` dissolved.** The audit command and its pre-commit gate have been removed entirely. Three of its seven checks were already enforced by the commands that need them (sealed workbench by `pack!`, working tree by Courier, main sync by `sync!`). The remaining four (PR status, CI baseline, hooks health, outsider fingerprints) were either unproven at commit time or already called by individual commands. Bare `carson` now defaults to `status` instead of `audit`. The pre-commit hook is now a no-op reserved for future use.
+### New
+
+- **Auto-chdir on checkout from inside worktree.** `carson checkout` no longer blocks when the current working directory is inside the worktree being removed. Carson `Dir.chdir`s to the main worktree root and completes the removal automatically. Both the warehouse (`assess_removal`) and legacy (`remove_check`) paths are fixed.
+- **Parent shell excluded from held-by-other-process check.** `held_by_other_process?` now excludes `Process.ppid` (the user's shell that launched Carson) in addition to `Process.pid`. Prevents false blocks when the user runs `carson checkout` from inside the worktree.
 
 ### Why
 
-Audit was a remote-centred pre-commit gate — it blocked commits based on delivery-time concerns (PR checks, CI baseline) that don't belong at commit time. In local-centred mode it was purely dead weight. Rather than relocate it to bureau, we verified that none of its unique checks had proven value at any gate, and dissolved the whole thing. Scars, not speculation.
+Running `carson checkout` from inside the target worktree is the most natural invocation — agents and users are already there. Blocking and printing a recovery command added friction for no safety benefit, since Carson can move its own process out.
 
-## 4.3.4
+## 4.3.5
 
 ### Fixed
 
-- **`carson checkin` is 13× faster.** Removed two redundant `git fetch` calls that hit GitHub on every checkin. `checkin` now branches from local main — zero network, 0.7s instead of 9.4s. Local main is the standard; GitHub is a backup, not a gating dependency.
+- **Story language purged from CLI output.** `checkin`/`checkout` (#509) shipped after the story-language purge (#458) and used "workbench" in every user-facing string — help text, status messages, error messages. All 9 output strings now use technical language ("worktree"). Source-code names unchanged.
+
+### New
+
+- **Story-language guard test.** `test/story_language_guard_test.rb` scans all output-producing lines in `lib/` for forbidden story-language terms (workbench, warehouse, parcel, courier, bureau, waybill, vault). Strips `#{}` interpolation before checking. Catches future leaks mechanically — the purge is now a permanent rule, not a one-time pass.
+
+### Removed
+
+- **`carson audit` dissolved.** The audit command and its pre-commit gate have been removed entirely. Three of its seven checks were already enforced by the commands that need them (sealed workbench by `pack!`, working tree by Courier, main sync by `sync!`). The remaining four (PR status, CI baseline, hooks health, outsider fingerprints) were either unproven at commit time or already called by individual commands. Bare `carson` now defaults to `status` instead of `audit`. The pre-commit hook is now a no-op reserved for future use.
 
 ### Why
 
-Local-centred mode means local main is the source of truth. Fetching from GitHub on every checkin was a remote-centred assumption baked into the command. The two sequential network round-trips accounted for 90% of the wall time.
+Story language in output has no mechanical enforcement — conventions fail on new code. The guard test closes that gap. Audit dissolution: audit was a remote-centred pre-commit gate; in local-centred mode it was dead weight.
 
-## 4.3.3
+## 4.3.4
 
 ### New
 
@@ -35,6 +46,16 @@ Local-centred mode means local main is the source of truth. Fetching from GitHub
 
 An agent edited four files in a worktree, ran `carson deliver` without committing, and lost all work. Carson merged a branch with zero commits ahead of main — a valid fast-forward, technically a no-op. The edits existed only in the working directory and were destroyed on workbench cleanup. The agent then wrote a confident delivery summary describing changes that did not exist on main.
 
+## 4.3.3
+
+### Fixed
+
+- **`carson checkin` is 13× faster.** Removed two redundant `git fetch` calls that hit GitHub on every checkin. `checkin` now branches from local main — zero network, 0.7s instead of 9.4s. Local main is the standard; GitHub is a backup, not a gating dependency.
+
+### Why
+
+Local-centred mode means local main is the source of truth. Fetching from GitHub on every checkin was a remote-centred assumption baked into the command. The two sequential network round-trips accounted for 90% of the wall time.
+
 ## 4.3.2
 
 ### Fixed

data/VERSION

@@ -1 +1 @@
-4.3.4
+4.4.0

data/lib/carson/warehouse/workbench.rb

@@ -166,7 +166,7 @@ module Carson
 						tracking = seal_check.sealed_tracking_number || "unknown"
 						return { command: "checkout", status: "block",
 							name: File.basename( workbench.path ), branch: workbench.branch,
-							error: "workbench is sealed — PR ##{tracking} is still in flight",
+							error: "worktree is sealed — PR ##{tracking} is still in flight",
 							recovery: "wait for CI checks to complete, or run carson deliver to check status" }
 					end
 				end
@@ -232,11 +232,8 @@ module Carson
 					return { status: :ok, missing: true }
 				end
 
-				if workbench.holds_cwd?
-					return { status: :block, result_status: "block",
-						error: "current working directory is inside this worktree",
-						recovery: "cd #{main_worktree_root} && carson checkout #{File.basename( workbench.path )}" }
-				end
+				# Auto-chdir out so removal can proceed.
+				Dir.chdir( main_worktree_root ) if workbench.holds_cwd?
 
 				if workbench.held_by_other_process?
 					return { status: :block, result_status: "block",

data/lib/carson/worktree.rb

@@ -270,18 +270,8 @@ module Carson
 			entry = find( path: resolved_path, runtime: runtime )
 			branch = entry&.branch
 
-			if entry&.holds_cwd?
-				safe_root = runtime.main_worktree_root
-				return {
-					status: :block,
-					result_status: "block",
-					exit_code: Runtime::EXIT_BLOCK,
-					resolved_path: resolved_path,
-					branch: branch,
-					error: "current working directory is inside this worktree",
-					recovery: "cd #{safe_root} && carson checkout #{File.basename( resolved_path )}"
-				}
-			end
+			# Auto-chdir out so removal can proceed.
+			Dir.chdir( runtime.main_worktree_root ) if entry&.holds_cwd?
 
 			if entry&.held_by_other_process?
 				return {
@@ -388,11 +378,12 @@ module Carson
 			return false if stdout.nil? || stdout.empty?
 
 			normalised = File.join( canonical, "" )
-			my_pid = Process.pid
+			# Exclude this process and its parent (the user's shell that launched Carson).
+			excluded_pids = [ Process.pid, Process.ppid ].to_set
 			stdout.lines.drop( 1 ).any? do |line|
 				fields = line.strip.split( /\s+/ )
 				next false unless fields.length >= 9
-				next false if fields[ 1 ].to_i == my_pid
+				next false if excluded_pids.include?( fields[ 1 ].to_i )
 				name = fields[ 8.. ].join( " " )
 				name == canonical || name.start_with?( normalised )
 			end

data/lib/cli.rb

@@ -134,7 +134,7 @@ module Carson
 				parser.separator "Keep agents from breaking main. Keep agents from breaking each other."
 				parser.separator ""
 				parser.separator "Agent workflow:"
-				parser.separator "    checkin      Get a fresh workbench"
+				parser.separator "    checkin      Get a fresh worktree"
 				parser.separator "    deliver      Accept into main and back up to remote"
 				parser.separator ""
 				parser.separator "Portfolio:"
@@ -528,15 +528,15 @@ module Carson
 			checkin_parser = OptionParser.new do |parser|
 				parser.banner = "Usage: carson checkin <name> [--json]"
 				parser.separator ""
-				parser.separator "Prepare a fresh workbench from the latest standard."
-				parser.separator "The agent names the workbench — it becomes the branch."
+				parser.separator "Prepare a fresh worktree from the latest main."
+				parser.separator "The name becomes the branch."
 				parser.separator ""
 				parser.separator "Options:"
 				parser.on( "--json", "Machine-readable JSON output" ) { options[ :json ] = true }
 				parser.separator ""
 				parser.separator "Examples:"
 				parser.separator "    carson checkin feature-auth"
-				parser.separator "    carson checkin oo/refactor-courier"
+				parser.separator "    carson checkin oo/refactor-auth"
 			end
 			checkin_parser.parse!( arguments )
 			name = arguments.shift.to_s.strip
@@ -560,7 +560,7 @@ module Carson
 			checkout_parser = OptionParser.new do |parser|
 				parser.banner = "Usage: carson checkout <name> [--json] [--force]"
 				parser.separator ""
-				parser.separator "Release a workbench when safe."
+				parser.separator "Release a worktree when safe."
 				parser.separator "Removes the directory and local branch."
 				parser.separator ""
 				parser.separator "Options:"
@@ -569,7 +569,7 @@ module Carson
 				parser.separator ""
 				parser.separator "Examples:"
 				parser.separator "    carson checkout feature-auth"
-				parser.separator "    carson checkout oo/refactor-courier --force"
+				parser.separator "    carson checkout oo/refactor-auth --force"
 			end
 			checkout_parser.parse!( arguments )
 			name = arguments.shift.to_s.strip
@@ -1036,7 +1036,7 @@ module Carson
 				name = parsed.fetch( :workbench_name )
 				result = { command: "checkout", status: "error",
 					name: name,
-					error: "#{name} is not a registered workbench",
+					error: "#{name} is not a registered worktree",
 					recovery: "carson worktree list" }
 				return report_workbench( result: result, json: parsed.fetch( :json, false ), output: runtime.output )
 			end
@@ -1155,11 +1155,11 @@ module Carson
 				when "ok"
 					case result[ :command ]
 					when "checkin"
-						output.puts "#{BADGE} Workbench ready: #{result[ :name ]}"
+						output.puts "#{BADGE} Worktree ready: #{result[ :name ]}"
 						output.puts "  path: #{result[ :path ]}"
 						output.puts "  branch: #{result[ :branch ]}"
 					when "checkout"
-						output.puts "#{BADGE} Workbench released: #{result[ :name ]}"
+						output.puts "#{BADGE} Worktree released: #{result[ :name ]}"
 					end
 				when "error"
 					output.puts "#{BADGE} #{result[ :error ]}"

metadata

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