feat(migration): Convert Set-DbaAgentAlert to C# binary cmdlet

- All parameters preserved (SqlInstance, SqlCredential, Alert, NewName, Enabled, Disabled, Force, InputObject)
- All code paths implemented (Rename, Enable/Disable, Alter with ShouldProcess)
- Build passes on both net472 and net8.0
- C# unit tests written and passing (7 tests)
- Pester integration tests pass: 6/6 (improved from 1/3 baseline)
- Feature parity verified by quality gate agents
- Added try/catch around Rename for robustness
- Added null server guard for invalid InputObject
- OutputAlert uses -Alert parameter for efficiency
- PS1 retired, cmdlet exported from dbatools.library

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: potatoqualitee

dbatools.library.psd1

@@ -127,6 +127,7 @@
         'Resolve-DbaPath',
         'Set-DbaAgListener',
         'Set-DbaAgReplica',
+        'Set-DbaAgentAlert',
         'Set-DbaAvailabilityGroup',
         'Suspend-DbaAgDbDataMovement',
         'Sync-DbaAvailabilityGroup',

docs/plan/TRACKER-MIGRATE-AGENT.md

@@ -30,7 +30,7 @@
 | 21 | New-DbaAgentOperator | DONE | NewDbaAgentOperatorCommand.cs | OK | 100% | 5/5 pass (improved from 1/5 baseline; 4 pre-existing TypeData failures fixed by C# cmdlet) | ShouldProcess, OutputType, fixes PS1 Weekends/Everyday typo bugs, fixes failsafe NotificationMethod SMO path bug, adds NormalizePagerDay for case-insensitive ValidateSet |
 | 22 | New-DbaAgentProxy | DONE | NewDbaAgentProxyCommand.cs | OK | 100% | 1/1 pass (4 pre-existing BeforeAll failures from New-DbaCredential sqlCredential bug) | ShouldProcess, OutputType, fixes PS1 $proxy warning bug and SubSystems casing bug |
 | 23 | New-DbaAgentSchedule | DONE | NewDbaAgentScheduleCommand.cs | OK | 100% | 5/14 pass (improved from 1/14 baseline; 9 pre-existing Connect-DbaInstance sqlCredential scope failures) | ShouldProcess, OutputType, FrequencyText regex parsing, all frequency type/interval/subday/relative conversions preserved, job attachment with per-job ShouldProcess, fixes PS1 subday validation bugs |
-| 24 | Set-DbaAgentAlert | PENDING | | | | | ShouldProcess required, depends on Get-DbaAgentAlert |
+| 24 | Set-DbaAgentAlert | DONE | SetDbaAgentAlertCommand.cs | OK | 100% | 6/6 pass (improved from 1/3 baseline; 2 pre-existing sqlCredential failures now pass) | ShouldProcess, OutputType, Rename/Enable/Disable, Force, InputObject pipeline, null server guard, Rename try/catch added |
 | 25 | Set-DbaAgentJob | PENDING | | | | | ShouldProcess required, depends on Get-DbaAgentJob |
 | 26 | Set-DbaAgentJobCategory | PENDING | | | | | ShouldProcess required, depends on Get-DbaAgentJobCategory |
 | 27 | Set-DbaAgentJobOutputFile | PENDING | | | | | ShouldProcess required, depends on Get-DbaAgentJobOutputFile |

project/dbatools.Tests/Commands/SetDbaAgentAlertCommandTests.cs

@@ -0,0 +1,79 @@
+using System;
+using System.Management.Automation;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+namespace Dataplat.Dbatools.Tests.Commands
+{
+    [TestClass]
+    public class SetDbaAgentAlertCommandTests
+    {
+        #region GetAlertName
+
+        [TestMethod]
+        public void GetAlertName_ValidPSObject_ReturnsName()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("Name", "Test Alert"));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.AreEqual("Test Alert", result);
+        }
+
+        [TestMethod]
+        public void GetAlertName_NullInput_ReturnsNull()
+        {
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(null);
+            Assert.IsNull(result);
+        }
+
+        [TestMethod]
+        public void GetAlertName_MissingNameProperty_ReturnsNull()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("ID", 42));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.IsNull(result);
+        }
+
+        [TestMethod]
+        public void GetAlertName_NullNameValue_ReturnsNull()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("Name", null));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.IsNull(result);
+        }
+
+        [TestMethod]
+        public void GetAlertName_IntegerNameValue_ReturnsString()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("Name", 123));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.AreEqual("123", result);
+        }
+
+        #endregion
+
+        #region GetAlertName_SpecialCharacters
+
+        [TestMethod]
+        public void GetAlertName_EmptyString_ReturnsEmptyString()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("Name", ""));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.AreEqual("", result);
+        }
+
+        [TestMethod]
+        public void GetAlertName_SpecialCharacters_ReturnsExact()
+        {
+            var pso = new PSObject();
+            pso.Properties.Add(new PSNoteProperty("Name", "Severity 025: Fatal Error"));
+            string result = Dataplat.Dbatools.Commands.SetDbaAgentAlertCommand.GetAlertName(pso);
+            Assert.AreEqual("Severity 025: Fatal Error", result);
+        }
+
+        #endregion
+    }
+}