fix(#194): standardize catch block formatting across multiple scripts

- Adjusted catch blocks to maintain consistent formatting
- Improved readability by aligning else statements

Author: stiwicourage

scripts/build/Sync-AgenticCopilotScaffold.ps1

@@ -56,8 +56,7 @@ function Test-AgenticPathExcluded {
             if ( $normalizedRelativePath.StartsWith($normalizedExcludedPath, [System.StringComparison]::Ordinal)) {
                 return $true
             }
-        }
-        elseif ($normalizedRelativePath -eq $normalizedExcludedPath) {
+        } elseif ($normalizedRelativePath -eq $normalizedExcludedPath) {
             return $true
         }
     }
@@ -80,8 +79,7 @@ function Get-AgenticMirrorSourceFile {
 
         if (Test-Path -LiteralPath $resolvedSourcePath -PathType Leaf) {
             Get-Item -LiteralPath $resolvedSourcePath
-        }
-        else {
+        } else {
             Get-ChildItem -LiteralPath $resolvedSourcePath -File -Recurse -Force
         }
     }
@@ -206,8 +204,7 @@ function Invoke-AgenticCopilotScaffoldSync {
             MirroredFileCount = $sourceFileList.Count
             ScaffoldOwnedPathCount = @($manifest.ScaffoldOwnedPaths).Count
         }
-    }
-    finally {
+    } finally {
         if (Test-Path -LiteralPath $stagingParent) {
             Remove-Item -LiteralPath $stagingParent -Recurse -Force
         }

scripts/build/ci/CodeSceneCoverageMap.ps1

@@ -91,8 +91,7 @@ function Get-CoberturaLineStat {
     $coveredLineCount = @($Bucket.Values | Where-Object {$_ -gt 0}).Count
     $lineRate = if ($validLineCount -eq 0) {
         0
-    }
-    else {
+    } else {
         $coveredLineCount / $validLineCount
     }
 

scripts/build/ci/CodeSceneCoverageXml.ps1

@@ -74,8 +74,7 @@ function Get-CoberturaPackageElement {
 
     $packageLineRate = if ($packageValidLineCount -eq 0) {
         0
-    }
-    else {
+    } else {
         $packageCoveredLineCount / $packageValidLineCount
     }
 
@@ -97,8 +96,7 @@ function Get-CoberturaCoverageAttributeMap {
 
     $lineRate = if ($ValidLineCount -eq 0) {
         0
-    }
-    else {
+    } else {
         $CoveredLineCount / $ValidLineCount
     }
 
@@ -176,8 +174,7 @@ function Convert-CoberturaCoverageToSourcePath {
     $xmlWriter = [System.Xml.XmlWriter]::Create($CoveragePath, $xmlWriterSettings)
     try {
         $newCoverageXml.Save($xmlWriter)
-    }
-    finally {
+    } finally {
         $xmlWriter.Dispose()
     }
 }

scripts/build/ci/Invoke-NovaModuleToolsCI.ps1

@@ -76,16 +76,13 @@ $novaModuleToolsTestFailed = $false
 try {
     if (@($ExcludeTag).Count -gt 0) {
         Test-NovaBuild -ExcludeTagFilter $ExcludeTag
-    }
-    else {
+    } else {
         Test-NovaBuild
     }
-}
-catch {
+} catch {
     $novaModuleToolsTestFailed = $true
     Write-Warning "Test-NovaBuild failed: $( $_.Exception.Message )"
-}
-finally {
+} finally {
     Copy-NovaModuleToolsTestResultIfPresent -ProjectRoot $projectInfo.ProjectRoot -ArtifactsDirectory $OutputDirectory
 }
 

src/private/build/AssertNovaPublicFunctionFileLayout.ps1

@@ -22,8 +22,7 @@ function Get-NovaInvalidPublicFunctionFileList {
         [pscustomobject]@{
             FilePath = if ($ProjectInfo.PSObject.Properties.Name -contains 'ProjectRoot') {
                 [System.IO.Path]::GetRelativePath($ProjectInfo.ProjectRoot, $publicFunctionFile.FullName)
-            }
-            else {
+            } else {
                 $publicFunctionFile.FullName
             }
             FunctionNameList = $functionNameList
@@ -48,8 +47,7 @@ function Format-NovaPublicFunctionFileValidationMessage {
     foreach ($invalidFile in $InvalidFileList) {
         $functionText = if (@($invalidFile.FunctionNameList).Count -eq 0) {
             '<none>'
-        }
-        else {
+        } else {
             @($invalidFile.FunctionNameList) -join ', '
         }
 

src/private/build/Format-ProjectJsonValue.ps1

@@ -8,8 +8,7 @@ function Format-ProjectJsonValue {
 
     try {
         return ($Value | ConvertTo-Json -Compress -Depth 10)
-    }
-    catch {
+    } catch {
         return [string]$Value
     }
 }

src/private/build/GetOrderedScriptFileForDirectory.ps1

@@ -12,8 +12,7 @@ function Get-OrderedScriptFileForDirectory {
 
     $items = if ($Recurse) {
         Get-ChildItem -Path $Directory -Filter '*.ps1' -File -Recurse -ErrorAction SilentlyContinue
-    }
-    else {
+    } else {
         Get-ChildItem -Path $Directory -Filter '*.ps1' -File -ErrorAction SilentlyContinue
     }
 

src/private/build/InvokeNovaBuildWorkflow.ps1

@@ -41,8 +41,7 @@ function Invoke-NovaBuildUpdateNotificationSafely {
 
     try {
         Invoke-NovaBuildUpdateNotification
-    }
-    catch {
+    } catch {
         $null = $_
     }
 }

src/private/build/TestProjectSchema.ps1

@@ -20,8 +20,7 @@ function Test-ProjectSchema {
                 Test-Json -Path 'project.json' -Schema (Get-Content $SchemaPath.Pester -Raw)
             }
         }
-    }
-    catch {
+    } catch {
         Stop-NovaOperation -Message "Invalid project.json for the $Schema schema: $( $_.Exception.Message )" -ErrorId 'Nova.Configuration.ProjectSchemaValidationFailed' -Category InvalidData -TargetObject 'project.json'
     }
 

src/private/build/manifest/GetFunctionNameFromFile.ps1

@@ -5,8 +5,7 @@ function Get-FunctionNameFromFile {
         $ast = [System.Management.Automation.Language.Parser]::ParseInput($moduleContent, [ref]$null, [ref]$null)
         $functionName = $ast.FindAll({$args[0] -is [System.Management.Automation.Language.FunctionDefinitionAst]}, $false) | ForEach-Object {$_.Name}
         return $functionName
-    }
-    catch {
+    } catch {
         return ''
     }
 }