carson 2.23.0 → 2.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e192c17c03b702b370cf76636a3c78674c719ecb4244ba779549301fa3564f0f
-  data.tar.gz: 72c7f78fe5cb5140f25aa50d0f4312a9e4cb3c704eb7f3d7b996a3da90f3298d
+  metadata.gz: f0834352d33c889752494f99e13d8a73cb3f0d08ad3c46f0c3c5d5ed276a4223
+  data.tar.gz: 50b67148ebfd901429f4b2d421b8c801dcabd835aabc775cd6596c8d3fdc0bbc
 SHA512:
-  metadata.gz: 9cad7952c709755faba1df36e1b247dea7ea21885c7d7987ec4c8aa42a4f2c1e2eb139ca02d8ff392129b61d4d821eeb57c2f768431aab73ec45b199d69c77c2
-  data.tar.gz: 6e95e4916ed3635bb1fb502d3e8068b02cd737803304ea1d54c5b8205c0d85b31700baf6bc2d00c76bdf66f5e27ab1ba11b9bc5d7e8522faa7791e003aad6f4e
+  metadata.gz: 56cb4e7ae94c7f593dede60394729e57da55f70605d317603a4e7f6c1448ab77213a5a46c743685e4597197bf088f39eb134d06ee22e94054d7274b7f9c3be4d
+  data.tar.gz: 4bb26dad28da8adb21dfe6a039402b58ea9e3e2a3abf0f6cfc76f4a2a12f71615b601c951ad4fe72d8dd312525a38eb1206a8e793dcea2feb75fe564a72623b3

data/.github/pull_request_template.md

@@ -1,8 +1,6 @@
 ## Shared Scope and Validation
 
 - [ ] `single_business_intent`: this PR is one coherent domain or feature intent.
-- [ ] `single_scope_group`: non-doc files stay within one scope group.
-- [ ] `cross-boundary_changes_justified`: any cross-boundary change has explicit rationale.
 - [ ] `carson audit` before commit.
 - [ ] `carson audit` before push.
 - [ ] `gh pr list --state open --limit 50` checked at session start (capture competing active PRs).

data/MANUAL.md

@@ -230,7 +230,7 @@ These are starting points chosen during `carson setup`. Every default has a reas
 
 How code reaches main.
 
-- **`branch`** (default) — every change goes through a PR. Hooks block direct commits and pushes to main/master. PRs enforce review, scope integrity, and CI gates before code reaches main.
+- **`branch`** (default) — every change goes through a PR. Hooks block direct commits and pushes to main/master. PRs enforce review and CI gates before code reaches main.
 - **`trunk`** — commit directly to main. Hooks allow all commits. Suits solo projects or flat teams that don't need PR-based review.
 
 Change: `carson setup` or `CARSON_WORKFLOW_STYLE`.
@@ -279,14 +279,6 @@ Where lint configuration files come from and where they land.
 
 Change: `carson lint policy --source <path-or-git-url>` or `lint.policy_source` in config. After changing policy, run `carson refresh --all` to propagate to all governed repositories.
 
-#### Scope integrity
-
-Whether cross-module changes are flagged.
-
-- Default: **advisory** (attention, not block). Carson informs you when staged files span multiple module groups (e.g. both `domain` and `ui`), but doesn't prevent the commit. Useful for awareness; not a hard gate.
-
-Customise groups: `scope.path_groups` in config.
-
 #### Review disposition
 
 Whether reviewer findings require acknowledgement.

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.24.0 — Remove Scope Integrity Guard
+
+### What changed
+
+- **Scope integrity guard removed from `carson audit`.** The guard classified changed files into path groups (tool, ui, test, domain, docs) and flagged commits crossing multiple groups. This required maintaining an explicit `scope.path_groups` list in `~/.carson/config.json` that went stale whenever a repository's directory structure changed. The maintenance burden outweighed the value.
+
+### Migration
+
+- The `scope.path_groups` config key is now ignored. Existing configs with this key will not cause errors — the data is silently unused.
+- The `path_groups` attribute has been removed from `Carson::Config`. Code referencing `config.path_groups` will raise `NoMethodError`.
+- PR template no longer includes `single_scope_group` and `cross-boundary_changes_justified` checklist items. The `single_business_intent` check remains — that is a human-level focus check, not mechanical path classification.
+
 ## 2.23.0 — Warm Onboard Welcome Guide
 
 ### What changed

data/VERSION

@@ -1 +1 @@
-2.23.0
+2.24.0

data/lib/carson/config.rb

@@ -8,7 +8,7 @@ module Carson
 	class Config
 		attr_accessor :git_remote
 		attr_reader :main_branch, :protected_branches, :hooks_base_path, :required_hooks,
-			:path_groups, :template_managed_files, :template_superseded_files, :template_canonical,
+			:template_managed_files, :template_superseded_files, :template_canonical,
 			:lint_policy_source,
 			:review_wait_seconds, :review_poll_seconds, :review_max_polls, :review_sweep_window_days,
 			:review_sweep_states, :review_disposition_prefix, :review_risk_keywords,
@@ -38,15 +38,6 @@ module Carson
 					"base_path" => "~/.carson/hooks",
 					"required_hooks" => [ "pre-commit", "prepare-commit-msg", "pre-merge-commit", "pre-push" ]
 				},
-				"scope" => {
-					"path_groups" => {
-						"tool" => [ "exe/**", "bin/**", "lib/**", "script/**", ".github/**", "templates/.github/**", "hooks/**", "install.sh", "README.md", "RELEASE.md", "VERSION", "carson.gemspec" ],
-						"ui" => [ "app/views/**", "app/assets/**", "app/javascript/**", "docs/ui_*.md" ],
-						"test" => [ "test/**", "spec/**", "features/**" ],
-						"domain" => [ "app/**", "db/**", "config/**" ],
-						"docs" => [ "docs/**", "*.md" ]
-					}
-				},
 				"template" => {
 					"managed_files" => [ ".github/carson.md", ".github/copilot-instructions.md", ".github/CLAUDE.md", ".github/AGENTS.md", ".github/pull_request_template.md" ],
 					"superseded_files" => [ ".github/carson-instructions.md", ".github/workflows/carson-lint.yml", ".github/.mega-linter.yml" ],
@@ -215,8 +206,6 @@ module Carson
 			@hooks_base_path = fetch_string( hash: fetch_hash( hash: data, key: "hooks" ), key: "base_path" )
 			@required_hooks = fetch_string_array( hash: fetch_hash( hash: data, key: "hooks" ), key: "required_hooks" )
 
-			@path_groups = fetch_hash( hash: fetch_hash( hash: data, key: "scope" ), key: "path_groups" ).transform_values { |value| normalize_patterns( value: value ) }
-
 			@template_managed_files = fetch_string_array( hash: fetch_hash( hash: data, key: "template" ), key: "managed_files" )
 			@template_superseded_files = fetch_optional_string_array( hash: fetch_hash( hash: data, key: "template" ), key: "superseded_files" )
 			@template_canonical = fetch_optional_path( hash: fetch_hash( hash: data, key: "template" ), key: "canonical" )
@@ -267,7 +256,6 @@ module Carson
 				raise ConfigError, "git.protected_branches must include #{main_branch}" unless protected_branches.include?( main_branch )
 				raise ConfigError, "hooks.base_path cannot be empty" if hooks_base_path.empty?
 				raise ConfigError, "hooks.required_hooks cannot be empty" if required_hooks.empty?
-				raise ConfigError, "scope.path_groups cannot be empty" if path_groups.empty?
 				raise ConfigError, "review.required_disposition_prefix cannot be empty" if review_disposition_prefix.empty?
 				raise ConfigError, "review.risk_keywords cannot be empty" if review_risk_keywords.empty?
 				raise ConfigError, "review.sweep.states must contain one or both of open, closed" if ( review_sweep_states - [ "open", "closed" ] ).any? || review_sweep_states.empty?
@@ -328,13 +316,6 @@ module Carson
 			rescue ArgumentError, TypeError
 				raise ConfigError, "config key #{key} must be an integer"
 			end
-
-			def normalize_patterns( value: )
-				patterns = Array( value ).map { |entry| entry.to_s.strip }.reject( &:empty? )
-				raise ConfigError, "scope.path_groups entries must contain at least one glob" if patterns.empty?
-				patterns
-			end
-
 			def fetch_optional_boolean( hash:, key:, default:, key_path: nil )
 				value = hash.fetch( key, default )
 				return true if value == true

data/lib/carson/runtime/audit.rb

@@ -92,15 +92,6 @@ module Carson
 				elsif baseline_st == "skipped"
 					audit_concise_problems << "Baseline: skipped (#{default_branch_baseline.fetch( :skip_reason )})."
 				end
-				scope_guard = print_scope_integrity_guard
-				audit_state = "attention" if audit_state == "ok" && scope_guard.fetch( :status ) == "attention"
-				if scope_guard.fetch( :status ) == "attention"
-					if scope_guard.fetch( :split_required )
-						audit_concise_problems << "Scope: multiple module groups touched."
-					else
-						audit_concise_problems << "Scope: unmatched paths — classify via scope.path_groups."
-					end
-				end
 				if config.template_canonical.nil? || config.template_canonical.to_s.empty?
 					puts_verbose ""
 					puts_verbose "[Canonical Templates]"
@@ -554,112 +545,6 @@ module Carson
 				lines.join( "\n" )
 			end
 
-			# Evaluates scope integrity using staged paths first, then working-tree paths as fallback.
-			def print_scope_integrity_guard
-				staged = staged_files
-				files = staged.empty? ? changed_files : staged
-				files_source = staged.empty? ? "working_tree" : "staged"
-				return { status: "ok", split_required: false } if files.empty?
-
-				scope = scope_integrity_status( files: files, branch: current_branch )
-				puts_verbose ""
-				puts_verbose "[Scope Integrity Guard]"
-				puts_verbose "scope_file_source: #{files_source}"
-				puts_verbose "scope_file_count: #{files.count}"
-				puts_verbose "branch: #{scope.fetch( :branch )}"
-				puts_verbose "scope_basis: changed_paths_only"
-				puts_verbose "detected_groups: #{scope.fetch( :detected_groups ).sort.join( ', ' )}"
-				puts_verbose "core_groups: #{scope.fetch( :core_groups ).empty? ? 'none' : scope.fetch( :core_groups ).sort.join( ', ' )}"
-				puts_verbose "non_doc_groups: #{scope.fetch( :non_doc_groups ).empty? ? 'none' : scope.fetch( :non_doc_groups ).sort.join( ', ' )}"
-				puts_verbose "docs_only_changes: #{scope.fetch( :docs_only )}"
-				puts_verbose "unmatched_paths_count: #{scope.fetch( :unmatched_paths ).count}"
-				scope.fetch( :unmatched_paths ).each { |path| puts_verbose "unmatched_path: #{path}" }
-				puts_verbose "violating_files_count: #{scope.fetch( :violating_files ).count}"
-				scope.fetch( :violating_files ).each { |path| puts_verbose "violating_file: #{path} (group=#{scope.fetch( :grouped_paths ).fetch( path )})" }
-				puts_verbose "checklist_single_business_intent: pass"
-				puts_verbose "checklist_single_scope_group: #{scope.fetch( :split_required ) ? 'advisory' : 'pass'}"
-				puts_verbose "checklist_cross_boundary_changes_justified: #{( scope.fetch( :split_required ) || scope.fetch( :misc_present ) ) ? 'advisory' : 'pass'}"
-				if scope.fetch( :split_required )
-					puts_verbose "ACTION: multiple module groups detected (informational only)."
-				elsif scope.fetch( :misc_present )
-					puts_verbose "ACTION: unmatched paths detected; classify via scope.path_groups for stricter module checks."
-				else
-					puts_verbose "ACTION: scope integrity is within commit policy."
-				end
-				{ status: scope.fetch( :status ), split_required: scope.fetch( :split_required ) }
-			end
-
-			# Evaluates whether changed files stay within one core module group.
-			def scope_integrity_status( files:, branch: )
-				grouped_paths = files.map { |path| [ path, scope_group_for_path( path: path ) ] }.to_h
-				detected_groups = grouped_paths.values.uniq
-				non_doc_groups = detected_groups - [ "docs" ]
-				# Tests are supporting changes; they may travel with one core module group.
-				core_groups = non_doc_groups - [ "test", "misc" ]
-				mixed_core_groups = core_groups.length > 1
-				misc_present = non_doc_groups.include?( "misc" )
-				split_required = mixed_core_groups
-				unmatched_paths = files.select { |path| grouped_paths.fetch( path ) == "misc" }
-				violating_files = if split_required
-					files.select do |path|
-						group = grouped_paths.fetch( path )
-						next false if [ "docs", "test", "misc" ].include?( group )
-						core_groups.include?( group )
-					end
-				else
-					[]
-				end
-				{
-				branch: branch,
-				grouped_paths: grouped_paths,
-				detected_groups: detected_groups,
-				non_doc_groups: non_doc_groups,
-				core_groups: core_groups,
-				docs_only: non_doc_groups.empty?,
-				mixed_core_groups: mixed_core_groups,
-				misc_present: misc_present,
-				split_required: split_required,
-				unmatched_paths: unmatched_paths,
-				violating_files: violating_files,
-				status: ( split_required || misc_present ) ? "attention" : "ok"
-				}
-			end
-
-			# Resolves a path to configured scope group; unmatched paths become misc.
-			def scope_group_for_path( path: )
-				config.path_groups.each do |group, patterns|
-					return group if patterns.any? { |pattern| pattern_matches_path?( pattern: pattern, path: path ) }
-				end
-				"misc"
-			end
-
-			# Supports directory-wide /** prefixes and fnmatch for other patterns.
-			def pattern_matches_path?( pattern:, path: )
-				if pattern.end_with?( "/**" )
-					prefix = pattern.delete_suffix( "/**" )
-					return path == prefix || path.start_with?( "#{prefix}/" )
-				end
-				File.fnmatch?( pattern, path, File::FNM_PATHNAME | File::FNM_DOTMATCH )
-			end
-
-			# Uses index-only paths so commit hooks evaluate exactly what is being committed.
-			def staged_files
-				git_capture!( "diff", "--cached", "--name-only" ).lines.map do |line|
-					raw_path = line.to_s.strip
-					next if raw_path.empty?
-					raw_path.split( " -> " ).last
-				end.compact
-			end
-
-			# Parses `git status --porcelain` and normalises rename targets.
-			def changed_files
-				git_capture!( "status", "--porcelain" ).lines.map do |line|
-					raw_path = line[ 3.. ].to_s.strip
-					next if raw_path.empty?
-					raw_path.split( " -> " ).last
-				end.compact
-			end
-
 			# True when there are no staged/unstaged/untracked file changes.
 		end
 

data/templates/.github/pull_request_template.md

@@ -1,8 +1,6 @@
 ## Shared Scope and Validation
 
 - [ ] `single_business_intent`: this PR is one coherent domain or feature intent.
-- [ ] `single_scope_group`: non-doc files stay within one scope group.
-- [ ] `cross-boundary_changes_justified`: any cross-boundary change has explicit rationale.
 - [ ] `carson audit` before commit.
 - [ ] `carson audit` before push.
 - [ ] `gh pr list --state open --limit 50` checked at session start (capture competing active PRs).

metadata

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