The output-stream refactor in #194 / PR #195 dropped two blank-line separators that the original code emitted via Write-Host '':
- Before
-WhatIf specified; no changes written.
- Before
Already up to date.
The PR summary claimed the -WhatIf output was byte-identical, but it is not -- the footer is now flush against the planned-ops preview. Observed in a consumer (DbxProvider) running .\Pull-SDLC.ai.ps1 -WhatIf:
add README.md.template
-WhatIf specified; no changes written.
Expected (note the blank line):
add README.md.template
-WhatIf specified; no changes written.
Fix: restore a separator before each footer using Write-Information '' (same stream as the footer message, so they suppress/redirect together). Do NOT add it to Write-PlannedOpsPreview or the apply path -- the Applied N ops. line correctly had no leading blank line. Add a behavior-first test asserting the separator precedes the -WhatIf footer on the Information stream.
The output-stream refactor in #194 / PR #195 dropped two blank-line separators that the original code emitted via
Write-Host '':-WhatIf specified; no changes written.Already up to date.The PR summary claimed the
-WhatIfoutput was byte-identical, but it is not -- the footer is now flush against the planned-ops preview. Observed in a consumer (DbxProvider) running.\Pull-SDLC.ai.ps1 -WhatIf:Expected (note the blank line):
Fix: restore a separator before each footer using
Write-Information ''(same stream as the footer message, so they suppress/redirect together). Do NOT add it toWrite-PlannedOpsPreviewor the apply path -- theApplied N ops.line correctly had no leading blank line. Add a behavior-first test asserting the separator precedes the-WhatIffooter on the Information stream.