Skip to content

runtime-node: surface explicit TYWRAP_CODEC_MAX_BYTES startup errors#183

Merged
bbopen merged 1 commit intomainfrom
codex/52-codec-max-bytes-explicit-env-error
Feb 12, 2026
Merged

runtime-node: surface explicit TYWRAP_CODEC_MAX_BYTES startup errors#183
bbopen merged 1 commit intomainfrom
codex/52-codec-max-bytes-explicit-env-error

Conversation

@bbopen
Copy link
Owner

@bbopen bbopen commented Feb 12, 2026

Summary

  • include captured stderr context in ProcessIO stdin/write stream errors so startup failures retain root-cause diagnostics
  • add a deterministic transport regression test for write-time EPIPE/error paths carrying TYWRAP_CODEC_MAX_BYTES details
  • add adversarial coverage for invalid TYWRAP_CODEC_MAX_BYTES values

Testing

  • npx eslint src/runtime/process-io.ts test/transport.test.ts test/adversarial_playground.test.ts
  • npm run typecheck
  • npm test -- test/transport.test.ts
  • npm test -- test/runtime_node.test.ts -t "TYWRAP_CODEC_MAX_BYTES"
  • TYWRAP_ADVERSARIAL=1 npm test -- test/adversarial_playground.test.ts -t "invalid TYWRAP_CODEC_MAX_BYTES"

Closes #52

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Warning

Rate limit exceeded

@bbopen has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 16 minutes and 35 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 2cf0e27 and 139cffa.

📒 Files selected for processing (3)
  • src/runtime/process-io.ts
  • test/adversarial_playground.test.ts
  • test/transport.test.ts
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/52-codec-max-bytes-explicit-env-error

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bbopen bbopen added area:runtime-node Area: Node runtime bridge priority:p2 Priority P2 (medium) labels Feb 12, 2026
@bbopen bbopen merged commit b6c3065 into main Feb 12, 2026
20 checks passed
@bbopen bbopen deleted the codex/52-codec-max-bytes-explicit-env-error branch February 12, 2026 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:runtime-node Area: Node runtime bridge priority:p2 Priority P2 (medium)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid TYWRAP_CODEC_MAX_BYTES should surface explicit error

1 participant