🐛 fix(scripts): address PR 17 Copilot review comments

- Move PSSession creation into try/catch with terminating errors
- Ensure PSSession cleanup runs from finally when remote execution fails
- Use consistent PascalCase Result variable in Resolve-IPs

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

Author: SamErde

DDI/Resolve-IPs.ps1

@@ -18,9 +18,9 @@ foreach ($IP in $ListOfIPs) {
 
     Write-Host "Resolving $IP" -ForegroundColor Green
     try {
-        $result = [System.Net.Dns]::gethostentry($IP)
+        $Result = [System.Net.Dns]::GetHostEntry($IP)
     } catch {
-        $result = $null
+        $Result = $null
     }
 
     If ($Result) {

Snippets/RemoteServerSessionLoop2.ps1

@@ -15,24 +15,26 @@ if ($session) { Remove-PSSession $session }
 
 #Loop through each server in the list, open a PowerShell remoting session, then show the name and status of the session. Skip (continue) to the next server if a connection fails.
 foreach ($server in $servers) {
-    $session = New-PSSession -ComputerName $server -Name $server -Credential $Creds
+    $session = $null
 
     Try {
         Write-Information "Connecting to $server... " -InformationAction Continue
+        $session = New-PSSession -ComputerName $server -Name $server -Credential $Creds -ErrorAction Stop
         # Enter-PSSession is interactive-only; use Invoke-Command to run code in the remote session.
-        Invoke-Command -Session $session -ScriptBlock {
+        Invoke-Command -Session $session -ErrorAction Stop -ScriptBlock {
             <#
                 Code to be run on each remote server goes here.
             #>
             Write-Information 'Inner code.' -InformationAction Continue
         }
     } Catch {
-        Write-Warning "Failed to connect to $server. Skipping." -WarningAction Continue
+        Write-Warning "Failed to create or use the PSSession for $server. Skipping." -WarningAction Continue
         Continue
+    } Finally {
+        if ($session) {
+            # Cleanup and then show the current PSSession state.
+            Remove-PSSession $session -ErrorAction SilentlyContinue
+            Write-Information "$($session.ComputerName) $($session.State) `n`n" -InformationAction Continue
+        }
     }
-    Write-Output $session.State
-
-    #Cleanup and then show the current PSSession state.
-    Remove-PSSession $session
-    Write-Information "$($session.ComputerName) $($session.State) `n`n" -InformationAction Continue
 }