feat: add Git::Repository::Configuring#global_config facade and deprecated aliases

Implements the global_config(name = nil, value = nil) method on
Git::Repository::Configuring with the same three-way dispatch as config():

- global_config()            => Hash of all global config entries
- global_config(name)        => String value of one entry
- global_config(name, value) => Git::CommandLineResult

Each mode is backed by a new private helper in the Private module
(global_config_get, global_config_list, global_config_set) that passes
global: true to the existing ConfigOptionSyntax command classes.

Also adds three deprecated forwarding methods for backward compatibility:
- global_config_get(name) - delegates to global_config(name)
- global_config_list      - delegates to global_config
- global_config_set(name, value) - delegates to global_config(name, value)

Each deprecated method emits Git::Deprecation.warn and delegates to
global_config; they will be removed in v6.

Git::Base delegators are added for all four methods.

Closes redesign/c1c2_bucket6_lib_orphans.md §3.3. Updates c1c2_audit.md
§7.3 to mark the three global_config_* rows as promoted, and decrements
the human-decision count from 16 to 13.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: jcouball

lib/git/base.rb

@@ -231,6 +231,22 @@ def config(name = nil, value = nil, options = {})
       facade_repository.config(name, value, options)
     end
 
+    def global_config(name = nil, value = nil)
+      facade_repository.global_config(name, value)
+    end
+
+    def global_config_get(name)
+      facade_repository.global_config_get(name)
+    end
+
+    def global_config_list
+      facade_repository.global_config_list
+    end
+
+    def global_config_set(name, value)
+      facade_repository.global_config_set(name, value)
+    end
+
     # Returns a reference to the working directory
     #
     # @example

lib/git/repository/configuring.rb

@@ -7,8 +7,11 @@ module Git
   class Repository
     # Facade methods for reading and writing git configuration
     #
-    # Provides the {#config} method, which dispatches to read a single entry,
-    # list all entries, or write a value depending on the arguments supplied.
+    # Provides the {#config} and {#global_config} methods, which dispatch to read a
+    # single entry, list all entries, or write a value depending on the arguments
+    # supplied. {#config} uses git's default config scope (reads from the full
+    # resolution chain; set operations write to the repository's `.git/config`);
+    # {#global_config} targets the git global config scope (`git config --global`).
     #
     # Included by {Git::Repository}.
     #
@@ -81,6 +84,89 @@ def config(name = nil, value = nil, options = {})
         end
       end
 
+      # Read or write a global git configuration entry
+      #
+      # Dispatches to one of three modes depending on the arguments supplied,
+      # targeting the git global config scope (`git config --global`):
+      #
+      # * **List** — `global_config()` returns all global config entries as a `Hash`.
+      # * **Get** — `global_config(name)` returns the value for a single key as a `String`.
+      # * **Set** — `global_config(name, value)` writes a value and returns the raw
+      #   command result.
+      #
+      # @overload global_config
+      #
+      #   @example List all global config entries
+      #     repo.global_config #=> { "user.name" => "Alice", "core.autocrlf" => "false" }
+      #
+      #   @return [Hash{String => String}] all global config entries, keyed by their
+      #     full dotted key names (e.g. `"user.name"`)
+      #
+      #   @raise [Git::FailedError] if git exits with a non-zero exit status
+      #
+      # @overload global_config(name)
+      #
+      #   @example Read the global committer name
+      #     repo.global_config('user.name') #=> "Alice"
+      #
+      #   @param name [String] the dotted config key to look up (e.g. `"user.name"`)
+      #
+      #   @return [String] the value of the global config entry
+      #
+      #   @raise [Git::FailedError] if git exits with a non-zero exit status
+      #
+      # @overload global_config(name, value)
+      #
+      #   @example Set the global committer name
+      #     repo.global_config('user.name', 'Alice')
+      #
+      #   @param name [String] the dotted config key to write (e.g. `"user.name"`)
+      #
+      #   @param value [#to_s] the value to assign; any object is accepted and
+      #     converted to a String via `#to_s` before being passed to git
+      #
+      #   @return [Git::CommandLineResult] the raw result of
+      #     `git config --global <name> <value>`
+      #
+      #   @raise [Git::FailedError] if git exits with a non-zero exit status
+      #
+      def global_config(name = nil, value = nil)
+        if !name.nil? && !value.nil?
+          Private.global_config_set(@execution_context, name, value)
+        elsif !name.nil?
+          Private.global_config_get(@execution_context, name)
+        else
+          Private.global_config_list(@execution_context)
+        end
+      end
+
+      # @deprecated Use {#global_config} instead.
+      def global_config_get(name)
+        Git::Deprecation.warn(
+          'Git::Repository#global_config_get is deprecated and will be removed in a future version. ' \
+          'Use global_config(name) instead.'
+        )
+        global_config(name)
+      end
+
+      # @deprecated Use {#global_config} instead.
+      def global_config_list
+        Git::Deprecation.warn(
+          'Git::Repository#global_config_list is deprecated and will be removed in a future version. ' \
+          'Use global_config instead.'
+        )
+        global_config
+      end
+
+      # @deprecated Use {#global_config} instead.
+      def global_config_set(name, value)
+        Git::Deprecation.warn(
+          'Git::Repository#global_config_set is deprecated and will be removed in a future version. ' \
+          'Use global_config(name, value) instead.'
+        )
+        global_config(name, value)
+      end
+
       # Private helpers local to {Git::Repository::Configuring}
       #
       # @api private
@@ -148,6 +234,58 @@ def config_list(execution_context)
             hsh[key] = value || ''
           end
         end
+
+        # Retrieve a global config value by key name
+        #
+        # @param execution_context [Git::ExecutionContext] the execution context
+        #
+        # @param name [String] the dotted config key to look up (e.g. `"user.name"`)
+        #
+        # @return [String] the value of the global config entry
+        #
+        # @raise [Git::FailedError] if git exits with a non-zero exit status
+        #
+        def global_config_get(execution_context, name)
+          result = Git::Commands::ConfigOptionSyntax::Get.new(execution_context).call(name, global: true)
+          raise Git::FailedError, result if result.status.exitstatus != 0
+
+          result.stdout
+        end
+
+        # Retrieve all global config entries as a hash
+        #
+        # @param execution_context [Git::ExecutionContext] the execution context
+        #
+        # @return [Hash{String => String}] all global config entries, keyed by their full
+        #   dotted key names (e.g. `"user.name"`)
+        #
+        # @raise [Git::FailedError] if git exits with a non-zero exit status
+        #
+        def global_config_list(execution_context)
+          lines = Git::Commands::ConfigOptionSyntax::List.new(execution_context).call(global: true).stdout.split("\n")
+          lines.each_with_object({}) do |line, hsh|
+            key, value = line.split('=', 2)
+            hsh[key] = value || ''
+          end
+        end
+
+        # Set a global config value by key name
+        #
+        # @param execution_context [Git::ExecutionContext] the execution context
+        #
+        # @param name [String] the dotted config key to write (e.g. `"user.name"`)
+        #
+        # @param value [#to_s] the value to assign; any object is accepted and
+        #   converted to a String via `#to_s` before being passed to git
+        #
+        # @return [Git::CommandLineResult] the raw result of
+        #   `git config --global <name> <value>`
+        #
+        # @raise [Git::FailedError] if git exits with a non-zero exit status
+        #
+        def global_config_set(execution_context, name, value)
+          Git::Commands::ConfigOptionSyntax::Set.new(execution_context).call(name, value, global: true)
+        end
       end
 
       private_constant :Private

redesign/c1c2_audit.md

@@ -401,9 +401,9 @@ These require a new facade method before a base.rb delegator can be added.
 | `config_get(name)` | Returns a single config value. Used by tooling. Part of the existing `config()` facade which reads/writes. | 🔍 human decision — expose as `config_get` or fold into `config(name)`? |
 | `config_list` | Returns full config hash. Used by tooling. | 🔍 human decision — expose separately or fold into `config()`? |
 | `config_set(name, value, options)` | Sets a config value. | 🔍 human decision — expose separately or fold into `config(name, value)`? |
-| `global_config_get(name)` | Gets a global config value. | 🔍 human decision |
-| `global_config_list` | Returns the full global config hash. | 🔍 human decision |
-| `global_config_set(name, value)` | Sets a global config value. | 🔍 human decision |
+| `global_config_get(name)` | Gets a global config value. | ✅ promoted — `global_config` facade + deprecated aliases in `Git::Repository::Configuring` + `Git::Base` delegators added (PR 5h-3) |
+| `global_config_list` | Returns the full global config hash. | ✅ promoted — `global_config` facade + deprecated aliases in `Git::Repository::Configuring` + `Git::Base` delegators added (PR 5h-3) |
+| `global_config_set(name, value)` | Sets a global config value. | ✅ promoted — `global_config` facade + deprecated aliases in `Git::Repository::Configuring` + `Git::Base` delegators added (PR 5h-3) |
 | `git_version` | Returns `Git::Version` for the current binary. Useful for tooling that conditionally enables features. Not a repository concern; `Git.git_version` is the canonical API. | ✅ delegator added to `Git::Base` — delegates to `Git.git_version` |
 | `list_files(ref_dir)` | Lists files under `.git/refs/{ref_dir}`. Internal ref-filesystem access. No plausible clean public use. | ❌ remove — internal plumbing; direct callers should migrate to `Git::Repository` ref-inspection methods |
 | `ls_remote(location = nil, opts = {})` | Lists remote refs. Clearly useful externally. | ✅ promoted — facade in `Git::Repository::RemoteOperations` + `Git::Base` delegator added (PR 5f) |
@@ -438,14 +438,14 @@ upgrade notes as "unsupported; remove any `g.lib.X` calls."
 
 | Status | Count |
 |--------|-------|
-| ✅ promote (repo already had it, `Git::Base` delegator added — PR 2d; or alias added) | 26 |
+| ✅ promote (repo already had it, `Git::Base` delegator added — PR 2d; or alias added) | 29 |
 | ⬜ promote (new facade work required) | 1 |
 | ❌ remove (internal plumbing) | 12 |
-| 🔍 human decision | 14 |
+| 🔍 human decision | 11 |
 | **Total orphaned methods** | **56** |
 
 > **Recommendation:** The 24 "trivial wiring" promotions can be handled in PR 5a
-> as a batch. The 4 "new facade" promotions and 16 human-decision items should be
+> as a batch. The 1 "new facade" promotion and 11 human-decision items should be
 > addressed in a companion document (`redesign/c1c2_bucket6_lib_orphans.md`)
 > before PR 5b begins.
 

redesign/c1c2_bucket6_lib_orphans.md

@@ -326,6 +326,8 @@ These can be removed in v6 after a full deprecation cycle.
 
 **Decision:** Accepted. Add `global_config(name = nil, value = nil)` to `Git::Repository::Configuring` with three-way dispatch (list/get/set). Add deprecated forwarding methods `global_config_get`, `global_config_list`, `global_config_set` with `Git::Deprecation.warn` calls. Add `Git::Base` delegators for all four. Remove deprecated aliases in v6.
 
+**Implemented in PR 5h-3.** `global_config` facade added to `Git::Repository::Configuring` with three-way dispatch; private helpers `global_config_get/list/set` in the `Private` module; deprecated forwarding methods on the public API; `Git::Base` delegators for all four methods.
+
 ---
 
 ### 3.4 `parse_config(file)`

spec/integration/git/repository/configuring_spec.rb

@@ -45,4 +45,41 @@
       end
     end
   end
+
+  describe '#global_config' do
+    around do |example|
+      with_isolated_global_config { example.run }
+    end
+
+    context 'when called with no arguments' do
+      before do
+        described_instance.global_config('user.name', 'GlobalUser')
+        described_instance.global_config('user.email', 'global@example.com')
+      end
+
+      it 'returns a Hash containing the written global config entries' do
+        result = described_instance.global_config
+        expect(result).to be_a(Hash)
+        expect(result).to include('user.name' => 'GlobalUser', 'user.email' => 'global@example.com')
+      end
+    end
+
+    context 'when called with a name' do
+      before { described_instance.global_config('user.name', 'GlobalUser') }
+
+      it 'returns the String value for the named key from global config' do
+        expect(described_instance.global_config('user.name')).to eq('GlobalUser')
+      end
+    end
+
+    def with_isolated_global_config
+      global_config = File.join(repo_dir, 'global.config')
+      FileUtils.touch(global_config)
+      saved = ENV.fetch('GIT_CONFIG_GLOBAL', nil)
+      ENV['GIT_CONFIG_GLOBAL'] = global_config
+      yield
+    ensure
+      saved.nil? ? ENV.delete('GIT_CONFIG_GLOBAL') : ENV['GIT_CONFIG_GLOBAL'] = saved
+    end
+  end
 end

spec/unit/git/base_spec.rb

@@ -460,4 +460,42 @@
       expect(described_instance.stash_list).to eq('stash@{0}: WIP')
     end
   end
+
+  describe '#global_config' do
+    include_context 'with a stubbed facade_repository'
+
+    it 'delegates to facade_repository.global_config with name and value' do
+      expect(facade_repository).to receive(:global_config).with('user.name', 'Alice').and_return(nil)
+      described_instance.global_config('user.name', 'Alice')
+    end
+  end
+
+  describe '#global_config_get' do
+    include_context 'with a stubbed facade_repository'
+
+    it 'delegates to facade_repository.global_config_get with name' do
+      expect(facade_repository).to receive(:global_config_get).with('user.name').and_return('Alice')
+      expect(described_instance.global_config_get('user.name')).to eq('Alice')
+    end
+  end
+
+  describe '#global_config_list' do
+    include_context 'with a stubbed facade_repository'
+
+    it 'delegates to facade_repository.global_config_list' do
+      result = { 'user.name' => 'Alice' }
+      expect(facade_repository).to receive(:global_config_list).and_return(result)
+      expect(described_instance.global_config_list).to eq(result)
+    end
+  end
+
+  describe '#global_config_set' do
+    include_context 'with a stubbed facade_repository'
+
+    it 'delegates to facade_repository.global_config_set with name and value' do
+      set_result = instance_double(Git::CommandLineResult)
+      expect(facade_repository).to receive(:global_config_set).with('user.name', 'Alice').and_return(set_result)
+      expect(described_instance.global_config_set('user.name', 'Alice')).to be(set_result)
+    end
+  end
 end