Skip to content

Conversation

@ibgreen
Copy link
Collaborator

@ibgreen ibgreen commented Dec 14, 2025

Summary

  • update thrift serialization helpers to emit Uint8Array outputs
  • accept ArrayBuffer and Uint8Array inputs when decoding thrift structures while keeping buffered transport support
  • add typed-array concatenation and buffer conversion helpers for compatibility

Testing

  • yarn lint fix (fails: missing node_modules state; install blocked by 403 request errors)
  • yarn test node (fails: missing node_modules state; install blocked by 403 request errors)
  • yarn install (fails: RequestError 403 fetching packages)

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

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.

2 participants