Skip to content

fix: flush large activity output#72

Merged
roodboi merged 1 commit into
mainfrom
codex/hack-932-activity-json-flush
Jul 15, 2026
Merged

fix: flush large activity output#72
roodboi merged 1 commit into
mainfrom
codex/hack-932-activity-json-flush

Conversation

@roodboi

@roodboi roodboi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • await completion of stdout writes for activity/history responses larger than the buffered console window
  • preserve existing console behavior for normal-sized command output
  • add a subprocess regression covering output above 128 KiB

Verification

  • ./scripts/test-safe.sh src/util/cli-output.test.ts src/ai-cli.test.ts (18 pass)
  • bun run type-check
  • bun run check
  • git diff --check
  • real machine-local source dogfood: aggregate activity emitted 184,632 bytes and parsed as version 2 JSON with 12 feeds

Follow-up for the published aggregate activity compatibility gate discovered during HACK-932 dogfood.


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@linear-code

linear-code Bot commented Jul 15, 2026

Copy link
Copy Markdown

HACK-932

@roodboi

roodboi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@codex review exact head 4414e2f

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 4414e2ff38

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@roodboi roodboi merged commit feee9de into main Jul 15, 2026
3 checks passed
@roodboi roodboi deleted the codex/hack-932-activity-json-flush branch July 15, 2026 22:14
github-actions Bot pushed a commit that referenced this pull request Jul 15, 2026
## [2.27.1](v2.27.0...v2.27.1) (2026-07-15)

### Bug Fixes

* flush large activity output ([#72](#72)) ([feee9de](feee9de))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.27.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant