You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maka.apache.org by "liuxiaocs7 (via GitHub)" <gi...@apache.org> on 2026/09/07 03:53:49 UTC
[PR] fix(computer-use): enforce JSON line UTF-8 byte limit [maka]
liuxiaocs7 opened a new pull request, #4953:
URL: https://github.com/apache/maka/pull/4953
## Summary
Measure the accumulated stdio JSON-line tail in UTF-8 bytes before comparing it with `maxBufferBytes`. This prevents multibyte executor output from retaining substantially more memory than the negotiated byte budget.
Add focused regression coverage for an over-limit multibyte tail, an exact-limit multibyte tail, and unchanged ASCII behavior across chunks.
Fixes #4952
## Verification
- `npm --workspace @maka/computer-use run build` — passed
- `node --test packages/computer-use/dist/__tests__/stdio-json-rpc.test.js` — 3/3 passed
- `npm --workspace @maka/computer-use run test:dist` — 117/117 passed
- `npm --workspace @maka/computer-use run typecheck` — passed
- `npm run build` — passed
- `npm run typecheck` — passed after rebuilding workspace declaration output
- `npm run lint` — passed
- `npm run format:check` — passed
- `npm test` — the changed tests passed, but the repository-wide parallel run failed in unrelated timing-sensitive PTY/process tests under load (`packages/eval` lifecycle cancellation, Runtime PTY ordering/rendering, and three existing `maka-cu` process timeout cases). The affected `@maka/computer-use` suite passed when rerun independently.
## AI use
Select exactly one:
- [ ] No generative tool made a substantive contribution
- [x] Generative tooling made a substantive contribution
Tool(s) and scope:
Codex diagnosed the byte/code-unit mismatch, implemented the fix, added regression tests, and ran verification.
## Checklist
- [x] Tests cover the change and fail without it
- [x] Lint, format, typecheck and the affected suites pass locally
Does this PR entail a change in behavior?
- [x] Yes — described under Summary above
- [ ] No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@maka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@maka.apache.org
For additional commands, e-mail: commits-help@maka.apache.org
Re: [PR] fix(computer-use): enforce JSON line UTF-8 byte limit [maka]
Posted by "me2seeks (via GitHub)" <gi...@apache.org>.
me2seeks merged PR #4953:
URL: https://github.com/apache/maka/pull/4953
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@maka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@maka.apache.org
For additional commands, e-mail: commits-help@maka.apache.org