feat(#99): enhance adapter architecture for external dependencies

- Isolate external workflow execution behind smaller internal adapters for clearer change points.
- Refactor CLI environment access and Git command interactions for improved maintainability.
- Introduce shared JSON file handling functions for better configuration management.

Author: stiwicourage

src/private/cli/ConfirmNovaCliAction.ps1

@@ -37,7 +37,15 @@ function Invoke-NovaCliConsoleReadKey {
     [CmdletBinding()]
     param()
 
-    return [Console]::ReadKey($true)
+    $consoleReader = Get-NovaCliConsoleReadKeyReader
+    return & $consoleReader
+}
+
+function Get-NovaCliConsoleReadKeyReader {
+    [CmdletBinding()]
+    param()
+
+    return {[Console]::ReadKey($true)}
 }
 
 function Read-NovaCliPromptKey {
@@ -140,4 +148,3 @@ function Confirm-NovaCliCommandAction {
         return
     } while ($true)
 }
-

tests/CoverageGaps.Cli.Tests.ps1

@@ -852,6 +852,23 @@ Describe 'Coverage gaps for CLI and installed-version internals' {
         }
     }
 
+    It 'Invoke-NovaCliConsoleReadKey invokes the shared console reader delegate' {
+        InModuleScope $script:moduleName {
+            Mock Get-NovaCliConsoleReadKeyReader {
+                $delegate = {
+                    [pscustomobject]@{KeyChar = [char]'y'}
+                }
+
+                return $delegate
+            }
+
+            $result = Invoke-NovaCliConsoleReadKey
+
+            $result.KeyChar | Should -Be ([char]'y')
+            Assert-MockCalled Get-NovaCliConsoleReadKeyReader -Times 1
+        }
+    }
+
     It 'Invoke-NovaCliConsoleReadKey executes the console read path when standard input is redirected' {
         $runnerPath = Join-Path $TestDrive 'Invoke-NovaCliConsoleReadKey.Runner.ps1'
         $stdinPath = Join-Path $TestDrive 'Invoke-NovaCliConsoleReadKey.stdin.txt'

tests/CoverageGaps.ReleaseInternals.Tests.ps1

@@ -470,6 +470,31 @@ Describe 'Coverage gaps for release and git internals' {
         }
     }
 
+    It 'Get-NovaVersionLabelForBump returns Patch when the repository has commits but no tags yet' {
+        InModuleScope $script:moduleName {
+            $projectRoot = Join-Path $TestDrive 'mocked-git-without-tags'
+            New-Item -ItemType Directory -Path (Join-Path $projectRoot '.git') -Force | Out-Null
+
+            Mock Invoke-NovaGitCommand {
+                switch ($Arguments[0]) {
+                    'rev-parse' {
+                        return [pscustomobject]@{ExitCode = 0; Output = @('.git')}
+                    }
+                    'describe' {
+                        return [pscustomobject]@{ExitCode = 1; Output = @()}
+                    }
+                    default {
+                        throw "Unexpected git args: $( $Arguments -join ' ' )"
+                    }
+                }
+            }
+
+            Get-NovaVersionLabelForBump -ProjectRoot $projectRoot | Should -Be 'Patch'
+            Assert-MockCalled Invoke-NovaGitCommand -Times 1 -ParameterFilter {$Arguments[0] -eq 'describe' -and $Arguments[1] -eq '--tags'}
+            Assert-MockCalled Invoke-NovaGitCommand -Times 0 -ParameterFilter {$Arguments[0] -eq 'rev-list'}
+        }
+    }
+
     It 'Get-NovaVersionLabelForBump throws a clear error when the repository has no commits yet' {
         if (-not (Get-Command git -ErrorAction SilentlyContinue)) {
             Set-ItResult -Skipped -Because 'git is not available in this environment'
@@ -601,5 +626,3 @@ Describe 'Coverage gaps for release and git internals' {
         }
     }
 }
-
-

tests/RemainingCommandCoverage.Tests.ps1

@@ -131,6 +131,48 @@ Describe 'Coverage for remaining command and filesystem branches' {
         }
     }
 
+    It 'New-InitiateGitRepo stops with a default failure message when git init returns no details' {
+        $repoPath = Join-Path $TestDrive 'git-init-nonzero'
+        New-Item -ItemType Directory -Path $repoPath -Force | Out-Null
+
+        InModuleScope $script:moduleName -Parameters @{RepoPath = $repoPath} {
+            param($RepoPath)
+
+            Mock Get-Command {[pscustomobject]@{Name = 'git'}} -ParameterFilter {$Name -eq 'git'}
+            Mock Invoke-NovaGitCommand {
+                [pscustomobject]@{
+                    ExitCode = 1
+                    Output = @()
+                }
+            }
+
+            $thrown = $null
+            try {
+                New-InitiateGitRepo -DirectoryPath $RepoPath -Confirm:$false
+            }
+            catch {
+                $thrown = $_
+            }
+
+            $thrown | Should -Not -BeNullOrEmpty
+            $thrown.Exception.Message | Should -Be 'Failed to initialize Git repo.'
+            $thrown.FullyQualifiedErrorId | Should -Be 'Nova.Dependency.GitRepositoryInitializationFailed'
+            $thrown.CategoryInfo.Category | Should -Be ([System.Management.Automation.ErrorCategory]::OpenError)
+            $thrown.TargetObject | Should -Be $RepoPath
+        }
+    }
+
+    It 'Get-NovaGitInitializationFailureMessage includes git output details when they are available' {
+        InModuleScope $script:moduleName {
+            $result = [pscustomobject]@{
+                ExitCode = 1
+                Output = @('permission denied')
+            }
+
+            Get-NovaGitInitializationFailureMessage -Result $result | Should -Be 'Failed to initialize Git repo: permission denied'
+        }
+    }
+
     It 'Get-NovaTestWorkflowContext prepares the Pester workflow state and resolves the report writers' -ForEach @(
         @{Build = $false; ExpectedOperation = 'Run Pester tests and write test results'}
         @{Build = $true; ExpectedOperation = 'Build project, run Pester tests, and write test results'}

tests/RemainingHelperCoverage.Tests.ps1

@@ -464,6 +464,17 @@ function Get-Second {
         }
     }
 
+    It 'Read-NovaJsonFileData returns null when the file content is not valid JSON' {
+        $jsonPath = Join-Path $TestDrive 'invalid-settings.json'
+        '{ invalid json }' | Set-Content -LiteralPath $jsonPath -Encoding utf8
+
+        InModuleScope $script:moduleName -Parameters @{JsonPath = $jsonPath} {
+            param($JsonPath)
+
+            Read-NovaJsonFileData -LiteralPath $JsonPath | Should -BeNullOrEmpty
+        }
+    }
+
     It 'Write-ProjectJsonData preserves nested objects, arrays, and unicode text when writing project.json' {
         $projectJsonPath = Join-Path $TestDrive 'written-project.json'