aelassas / servy / 26839761894

push

github

fix(service): Service.cs - Dispose(bool) has two '// 2.' numbered comment steps (base-class call should be '// 3.') (#2306)
fix(psm1): stale comment: EnvVars 28,000-char cap is justified by 'CLI argument' limit, but EnvVars is now passed via environment variable, not the command line (#2303)
fix(core): ResourceHelper.ShouldCopyResource - 'embeddedResourceTime' variable and debug log actually report the host executable's timestamp, not the embedded resource's (#2301)
fix(service): StartOptionsParser.cs - SafeResolvePath XML summary claims it falls back to 'empty values' but actually returns the original raw path (#2299)
fix(infra): ServiceRepository.cs - decrypt-then-handle-corruption block duplicated across 5 read methods (DRY) (#2293)
fix(notifications): ServySecurity.ps1 - header says masker parity is with 'Servy.Core' but the inline sync comment correctly points to Servy.Service/Helpers/ServiceHelper.cs (#2290)
fix(psm1): Stdout/Stderr/PreLaunch* path validators reject bare relative filenames (empty-parent), inconsistent with Export Path validator (#2289)
fix(core): ServiceManager.cs - AutomaticDelayedStart→Automatic start-type coercion duplicated in CreateService and UpdateServiceConfig (#2288)
fix(service): Service.cs - health-monitoring enable predicate duplicated in OnStart (_recoveryActionEnabled) and SetupHealthMonitoring (#2287)
fix(notifications): ServyFailureEmail.ps1 / ServyFailureNotification.ps1 - dangling, nonsensical comment inside the dependency-missing Write-EventLog block (#2284)

543 of 743 branches covered (73.08%)

Branch coverage included in aggregate %.

3 of 8 new or added lines in 2 files covered. (37.5%)

3704 of 5647 relevant lines covered (65.59%)

86.86 hits per line