Skip to content

[codex] Fix ballin_update output spacing#203

Merged
JBallin merged 2 commits into
mainfrom
codex/fix-ballin-update-output-spacing
Jun 30, 2026
Merged

[codex] Fix ballin_update output spacing#203
JBallin merged 2 commits into
mainfrom
codex/fix-ballin-update-output-spacing

Conversation

@JBallin

@JBallin JBallin commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the unconditional blank line that ballin_update printed before setup output
  • print the first-run analytics notice as its own paragraph after setup/docs output
  • tighten output assertions for ballin_update and setup analytics notice spacing

Root cause

ballin_update always printed an empty line before invoking setup, even when setup had no immediate output. Separately, analytics setup wrote the notice directly through the generic line writer, so it could appear immediately after the previous setup/docs line instead of as a separated notice block.

Validation

  • npm run test:unit -- test/ballin_update.test.ts test/install_setup.test.ts
  • npm test

@JBallin JBallin marked this pull request as ready for review June 30, 2026 03:55
@JBallin JBallin merged commit 82ee69d into main Jun 30, 2026
3 checks passed
@JBallin JBallin deleted the codex/fix-ballin-update-output-spacing branch June 30, 2026 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant