Skip to content

Bound pre-output stream inspection#13

Merged
gitcommit90 merged 1 commit into
mainfrom
fix/bound-preoutput-inspection
Jul 18, 2026
Merged

Bound pre-output stream inspection#13
gitcommit90 merged 1 commit into
mainfrom
fix/bound-preoutput-inspection

Conversation

@gitcommit90

@gitcommit90 gitcommit90 commented Jul 18, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Owner

Summary

  • cap pre-output SSE inspection at 64 KiB
  • inspect only completed newly received events instead of rescanning the full buffer
  • cancel and reroute members that exhaust the inspection budget without usable output

Tests

  • npm test (301 passed)
  • focused oversized-preamble fallback regression
  • git diff --check
Open in Devin Review

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jul 18, 2026
View reviewed changes

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@gitcommit90
gitcommit90 merged commit 9a77ce2 into main Jul 18, 2026
2 checks passed
@gitcommit90
gitcommit90 deleted the fix/bound-preoutput-inspection branch July 18, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@gitcommit90