feat: ✨ Add localization support and improve error handling

* Introduced `Messages.psd1` for localized messages.
* Updated functions to utilize localized messages for warnings and errors.
* Improved formatting and readability in several functions.
* Ensured consistent use of `[CmdletBinding()]` across all functions.

Author: HeyItsGilbert

PowerShellBuild/PowerShellBuild.psm1

@@ -1,6 +1,6 @@
 # Dot source public functions
 $private = @(Get-ChildItem -Path ([IO.Path]::Combine($PSScriptRoot, 'Private/*.ps1')) -Recurse)
-$public  = @(Get-ChildItem -Path ([IO.Path]::Combine($PSScriptRoot, 'Public/*.ps1')) -Recurse)
+$public = @(Get-ChildItem -Path ([IO.Path]::Combine($PSScriptRoot, 'Public/*.ps1')) -Recurse)
 foreach ($import in $public + $private) {
     try {
         . $import.FullName
@@ -9,6 +9,14 @@ foreach ($import in $public + $private) {
     }
 }
 
+$importLocalizedDataSplat = @{
+    BindingVariable = 'LocalizedData'
+    FileName        = 'Messages.psd1'
+    ErrorAction     = 'SilentlyContinue'
+}
+Import-LocalizedData @importLocalizedDataSplat
+
+
 Export-ModuleMember -Function $public.Basename
 
 # $psakeTaskAlias = 'PowerShellBuild.psake.tasks'

PowerShellBuild/Public/Build-PSBuildMAMLHelp.ps1

@@ -14,7 +14,7 @@ function Build-PSBuildMAMLHelp {
         Uses PlatyPS to generate MAML XML help from markdown files in ./docs
         and saves the XML file to a directory under ./output/MyModule
     #>
-    [cmdletbinding()]
+    [CmdletBinding()]
     param(
         [parameter(Mandatory)]
         [string]$Path,
@@ -23,7 +23,7 @@ function Build-PSBuildMAMLHelp {
         [string]$DestinationPath
     )
 
-    $helpLocales = (Get-ChildItem -Path $Path -Directory).Name
+    $helpLoc2ales = (Get-ChildItem -Path $Path -Directory).Name
 
     # Generate the module's primary MAML help file
     foreach ($locale in $helpLocales) {

PowerShellBuild/Public/Build-PSBuildMarkdown.ps1

@@ -25,7 +25,7 @@ function Build-PSBuildMarkdown {
 
         Analysis the comment-based help of the MyModule module and create markdown documents under ./docs/en-US.
     #>
-    [cmdletbinding()]
+    [CmdletBinding()]
     param(
         [parameter(Mandatory)]
         [string]$ModulePath,
@@ -56,7 +56,7 @@ function Build-PSBuildMarkdown {
 
     try {
         if ($moduleInfo.ExportedCommands.Count -eq 0) {
-            Write-Warning 'No commands have been exported. Skipping markdown generation.'
+            Write-Warning $LocalizedData.NoCommandsExported
             return
         }
 
@@ -93,7 +93,7 @@ function Build-PSBuildMarkdown {
         }
         New-MarkdownHelp @newMDParams > $null
     } catch {
-        Write-Error "Failed to generate markdown help. : $_"
+        Write-Error ($LocalizedData.FailedToGenerateMarkdownHelp -f $_)
     } finally {
         Remove-Module $moduleName
     }

PowerShellBuild/Public/Build-PSBuildModule.ps1

@@ -1,50 +1,58 @@
+# spell-checker:ignore modulename
 function Build-PSBuildModule {
     <#
     .SYNOPSIS
         Builds a PowerShell module based on source directory
     .DESCRIPTION
-        Builds a PowerShell module based on source directory and optionally concatenates
-        public/private functions from separete files into monolithic .PSM1 file.
+        Builds a PowerShell module based on source directory and optionally
+        concatenates public/private functions from separate files into
+        monolithic .PSM1 file.
     .PARAMETER Path
         The source module path.
     .PARAMETER DestinationPath
         Destination path to write "built" module to.
     .PARAMETER ModuleName
         The name of the module.
     .PARAMETER Compile
-        Switch to indicate if separete function files should be concatenated into monolithic .PSM1 file.
+        Switch to indicate if separate function files should be concatenated
+        into monolithic .PSM1 file.
     .PARAMETER CompileHeader
         String that will be at the top of your PSM1 file.
     .PARAMETER CompileFooter
         String that will be added to the bottom of your PSM1 file.
     .PARAMETER CompileScriptHeader
         String that will be added to your PSM1 file before each script file.
     .PARAMETER CompileScriptFooter
-        String that will be added to your PSM1 file beforeafter each script file.
+        String that will be added to your PSM1 file after each script file.
     .PARAMETER ReadMePath
-        Path to project README. If present, this will become the "about_<ModuleName>.help.txt" file in the build module.
+        Path to project README. If present, this will become the
+        "about_<ModuleName>.help.txt" file in the build module.
     .PARAMETER CompileDirectories
-        List of directories containing .ps1 files that will also be compiled into the PSM1.
+        List of directories containing .ps1 files that will also be compiled
+        into the PSM1.
     .PARAMETER CopyDirectories
         List of directories that will copying "as-is" into the build module.
     .PARAMETER Exclude
-        Array of files (regular expressions) to exclude from copying into built module.
+        Array of files (regular expressions) to exclude from copying into built
+        module.
     .PARAMETER Culture
-        UI Culture. This is used to determine what culture directory to store "about_<ModuleName>.help.txt" in.
+        UI Culture. This is used to determine what culture directory to store
+        "about_<ModuleName>.help.txt" in.
     .EXAMPLE
-        PS> $buildParams = @{
+        $buildParams = @{
             Path               = ./MyModule
             DestinationPath    = ./Output/MoModule/0.1.0
             ModuleName         = MyModule
             Exclude            = @()
             Compile            = $false
             Culture            = (Get-UICulture).Name
         }
-        PS> Build-PSBuildModule @buildParams
+        Build-PSBuildModule @buildParams
 
-        Build module from source directory './MyModule' and save to '/Output/MoModule/0.1.0'
+        Build module from source directory './MyModule' and save to
+        '/Output/MoModule/0.1.0'
     #>
-    [cmdletbinding()]
+    [CmdletBinding()]
     param(
         [parameter(Mandatory)]
         [string]$Path,
@@ -77,49 +85,80 @@ function Build-PSBuildModule {
     )
 
     if (-not (Test-Path -LiteralPath $DestinationPath)) {
-        New-Item -Path $DestinationPath -ItemType Directory -Verbose:$VerbosePreference > $null
+        $newItemSplat = @{
+            Path     = $DestinationPath
+            ItemType = 'Directory'
+            Verbose  = $VerbosePreference
+        }
+        New-Item @newItemSplat > $null
     }
 
     # Copy "non-processed files"
-    Get-ChildItem -Path $Path -Include '*.psm1', '*.psd1', '*.ps1xml' -Depth 1 | Copy-Item -Destination $DestinationPath -Force
+    $getChildItemSplat = @{
+        Path    = $Path
+        Include = '*.psm1', '*.psd1', '*.ps1xml'
+        Depth   = 1
+    }
+    Get-ChildItem @getChildItemSplat |
+        Copy-Item -Destination $DestinationPath -Force
     foreach ($dir in $CopyDirectories) {
         $copyPath = [IO.Path]::Combine($Path, $dir)
         Copy-Item -Path $copyPath -Destination $DestinationPath -Recurse -Force
     }
 
     # Copy README as about_<modulename>.help.txt
     if (-not [string]::IsNullOrEmpty($ReadMePath)) {
-        $culturePath     = [IO.Path]::Combine($DestinationPath, $Culture)
-        $aboutModulePath = [IO.Path]::Combine($culturePath, "about_$($ModuleName).help.txt")
-        if(-not (Test-Path $culturePath -PathType Container)) {
+        $culturePath = [IO.Path]::Combine($DestinationPath, $Culture)
+        $aboutModulePath = [IO.Path]::Combine(
+            $culturePath,
+            "about_$($ModuleName).help.txt"
+        )
+        if (-not (Test-Path $culturePath -PathType Container)) {
             New-Item $culturePath -Type Directory -Force > $null
-            Copy-Item -LiteralPath $ReadMePath -Destination $aboutModulePath -Force
+            $copyItemSplat = @{
+                LiteralPath = $ReadMePath
+                Destination = $aboutModulePath
+                Force       = $true
+            }
+            Copy-Item @copyItemSplat
         }
     }
 
-    # Copy source files to destination and optionally combine *.ps1 files into the PSM1
+    # Copy source files to destination and optionally combine *.ps1 files
+    # into the PSM1
     if ($Compile.IsPresent) {
         $rootModule = [IO.Path]::Combine($DestinationPath, "$ModuleName.psm1")
 
         # Grab the contents of the copied over PSM1
         # This will be appended to the end of the finished PSM1
         $psm1Contents = Get-Content -Path $rootModule -Raw
-        '' | Out-File -FilePath $rootModule -Encoding utf8
+        '' | Out-File -FilePath $rootModule -Encoding 'utf8'
 
         if ($CompileHeader) {
-            $CompileHeader | Add-Content -Path $rootModule -Encoding utf8
+            $CompileHeader | Add-Content -Path $rootModule -Encoding 'utf8'
         }
 
         $resolvedCompileDirectories = $CompileDirectories | ForEach-Object {
             [IO.Path]::Combine($Path, $_)
         }
-        $allScripts = Get-ChildItem -Path $resolvedCompileDirectories -Filter '*.ps1' -File -Recurse -ErrorAction SilentlyContinue
+        $getChildItemSplat = @{
+            Path        = $resolvedCompileDirectories
+            Filter      = '*.ps1'
+            File        = $true
+            Recurse     = $true
+            ErrorAction = 'SilentlyContinue'
+        }
+        $allScripts = Get-ChildItem @getChildItemSplat
 
         $allScripts = $allScripts | Remove-ExcludedItem -Exclude $Exclude
 
+        $addContentSplat = @{
+            Path     = $rootModule
+            Encoding = 'utf8'
+        }
         $allScripts | ForEach-Object {
             $srcFile = Resolve-Path $_.FullName -Relative
-            Write-Verbose "Adding [$srcFile] to PSM1"
+            Write-Verbose ($LocalizedData.AddingFileToPsm1 -f $srcFile)
 
             if ($CompileScriptHeader) {
                 Write-Output $CompileScriptHeader
@@ -130,15 +169,14 @@ function Build-PSBuildModule {
             if ($CompileScriptFooter) {
                 Write-Output $CompileScriptFooter
             }
+        } | Add-Content @addContentSplat
 
-        } | Add-Content -Path $rootModule -Encoding utf8
-
-        $psm1Contents | Add-Content -Path $rootModule -Encoding utf8
+        $psm1Contents | Add-Content @addContentSplat
 
         if ($CompileFooter) {
-            $CompileFooter | Add-Content -Path $rootModule -Encoding utf8
+            $CompileFooter | Add-Content @addContentSplat
         }
-    } else{
+    } else {
         # Copy everything over, then remove stuff that should have been excluded
         # It's just easier this way
         $copyParams = @{
@@ -157,13 +195,26 @@ function Build-PSBuildModule {
                 }
             }
         }
-        $toRemove | Remove-Item -Recurse -Force -ErrorAction Ignore
+        $toRemove | Remove-Item -Recurse -Force -ErrorAction 'Ignore'
     }
 
     # Export public functions in manifest if there are any public functions
-    $publicFunctions = Get-ChildItem $Path/Public/*.ps1 -Recurse -ErrorAction SilentlyContinue
+    $getChildItemSplat = @{
+        Recurse     = $true
+        ErrorAction = 'SilentlyContinue'
+        Path        = "$Path/Public/*.ps1"
+    }
+    $publicFunctions = Get-ChildItem @getChildItemSplat
     if ($publicFunctions) {
-        $outputManifest = [IO.Path]::Combine($DestinationPath, "$ModuleName.psd1")
-        Update-Metadata -Path $OutputManifest -PropertyName FunctionsToExport -Value $publicFunctions.BaseName
+        $outputManifest = [IO.Path]::Combine(
+            $DestinationPath,
+            "$ModuleName.psd1"
+        )
+        $updateMetadataSplat = @{
+            Path         = $OutputManifest
+            PropertyName = 'FunctionsToExport'
+            Value        = $publicFunctions.BaseName
+        }
+        Update-Metadata @updateMetadataSplat
     }
 }

PowerShellBuild/Public/Build-PSBuildUpdatableHelp.ps1

@@ -9,13 +9,14 @@ function Build-PSBuildUpdatableHelp {
     .PARAMETER OutputPath
         Path to create updatable help .cab file in.
     .PARAMETER Module
-        Name of the module to create a .cab file for. Defaults to the $ModuleName variable from the parent scope.
+        Name of the module to create a .cab file for. Defaults to the
+        $ModuleName variable from the parent scope.
     .EXAMPLE
         PS> Build-PSBuildUpdatableHelp -DocsPath ./docs -OutputPath ./Output/UpdatableHelp
 
         Create help .cab file based on PlatyPS markdown help.
     #>
-    [cmdletbinding()]
+    [CmdletBinding()]
     param(
         [parameter(Mandatory)]
         [string]$DocsPath,
@@ -27,26 +28,40 @@ function Build-PSBuildUpdatableHelp {
     )
 
     if ($null -ne $IsWindows -and -not $IsWindows) {
-        Write-Warning 'MakeCab.exe is only available on Windows. Cannot create help cab.'
+        Write-Warning $LocalizedData.MakeCabNotAvailable
         return
     }
 
     $helpLocales = (Get-ChildItem -Path $DocsPath -Directory).Name
 
     # Create updatable help output directory
     if (-not (Test-Path -LiteralPath $OutputPath)) {
-        New-Item $OutputPath -ItemType Directory -Verbose:$VerbosePreference > $null
+        $newItemSplat = @{
+            ItemType = 'Directory'
+            Verbose  = $VerbosePreference
+            Path     = $OutputPath
+        }
+        New-Item @newItemSplat > $null
     } else {
-        Write-Verbose "Directory already exists [$OutputPath]."
-        Get-ChildItem $OutputPath | Remove-Item -Recurse -Force -Verbose:$VerbosePreference
+        Write-Verbose ($LocalizedData.DirectoryAlreadyExists -f $OutputPath)
+        $removeItemSplat = @{
+            Recurse = $true
+            Force   = $true
+            Verbose = $VerbosePreference
+        }
+        Get-ChildItem $OutputPath | Remove-Item @removeItemSplat
     }
 
-    # Generate updatable help files.  Note: this will currently update the version number in the module's MD
-    # file in the metadata.
+    # Generate updatable help files. Note: this will currently update the
+    # version number in the module's MD file in the metadata.
     foreach ($locale in $helpLocales) {
         $cabParams = @{
             CabFilesFolder  = [IO.Path]::Combine($moduleOutDir, $locale)
-            LandingPagePath = [IO.Path]::Combine($DocsPath, $locale, "$Module.md")
+            LandingPagePath = [IO.Path]::Combine(
+                $DocsPath,
+                $locale,
+                "$Module.md"
+            )
             OutputFolder    = $OutputPath
             Verbose         = $VerbosePreference
         }

PowerShellBuild/Public/Clear-PSBuildOutputFolder.ps1

@@ -16,11 +16,11 @@ function Clear-PSBuildOutputFolder {
         # Maybe a bit paranoid but this task nuked \ on my laptop. Good thing I was not running as admin.
         [parameter(Mandatory)]
         [ValidateScript({
-            if ($_.Length -le 3) {
-                throw "`$Path [$_] must be longer than 3 characters."
-            }
-            $true
-        })]
+                if ($_.Length -le 3) {
+                    throw ($LocalizedData.PathLongerThan3Chars -f $_)
+                }
+                $true
+            })]
         [string]$Path
     )