aelassas / servy / 26751057100

85%

main: 85%

push

github

fix(infra): ServiceRepository.UpsertBatchAsync - PatchRuntimeState reads run outside the batch transaction (separate connection), breaking the documented snapshot isolation
fix(core): ProtectedKeyProvider.SaveProtected - crash between WriteAllBytes and File.Move leaves encrypted .tmp files orphaned with elevated ACLs; subsequent runs never clean them up
fix(notifications): ServyFailureEmail.xml - RunOnlyIfNetworkAvailable is false, so the task fires on every Servy error event even when no network is reachable
fix(core): ServiceManager.UninstallServiceAsync - pre-uninstall ChangeServiceConfig(DEMAND_START) is not rolled back when stop/delete fails, leaving the service silently switched to Manual
fix(publish): signpath.ps1 - failed signing leaves orphan $Path.signed on disk; a later run silently moves the stale artifact over the real binary
fix(service): Service.cs HandleLogWriters - stderr path is passed to Helper.Canonicalise before _pathValidator.IsValidPath runs, so a malformed StdErrPath throws and crashes OnStart
fix(core): Logger - RotatingStreamWriter - re-entrant rotation deadlock: Logger.Warn inside PerformPhysicalRotation re-enters the same writer and blocks forever on Monitor.Wait
fix(notifications): ServyFailureEmail.ps1 Send-NotificationEmail - generic catch classifies permanent errors (e.g. CRLF in Subject, MailMessage validation) as TransientFailure, freezing the email queue forever
fix(core): Helper.WriteFileAtomic / WriteFileAtomicCore - PrepareDestinationForMove runs outside the retry loop, so AV/EDR-induced AccessDenied on the destination file bypasses the documented retry policy

1616 of 2200 branches covered (73.45%)

Branch coverage included in aggregate %.

35 of 54 new or added lines in 5 files covered. (64.81%)

11245 of 16425 relevant lines covered (68.46%)

795.53 hits per line