Skip to content

Release v1.0.10-beta#233

Merged
debuggerone merged 2015 commits into
mainfrom
develop/v1.0.10-beta
Jul 1, 2026
Merged

Release v1.0.10-beta#233
debuggerone merged 2015 commits into
mainfrom
develop/v1.0.10-beta

Conversation

@debuggerone

Copy link
Copy Markdown
Contributor

Release v1.0.10-beta from develop/v1.0.10-beta.

  • Canonical baseline is green on a891426.
  • Local inference runner is green on a891426.
  • Release downgrade compatibility is green.

Merge method must preserve commits; no squash.

@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review


P1 Badge Advance past the full IIBIN header before slicing payload

When deserializing any IIBIN message with a payload, this only advances from byte 4 to byte 20 after reading the 4-byte payload length plus two 8-byte fields, so payload = data.slice(offset, ...) starts four bytes inside the message id instead of at byte 24. Round-tripping a TEXT_MESSAGE through serialize() then deserializePayload() shifts every field and corrupts the decoded message, so published consumers cannot reliably read their own encoded payloads.


$continue = ($this->onChunk)($slice, $this->cursor(), false);

P2 Badge Signal MCP source completion before returning

For McpByteSource::pumpLines() on a transfer whose final record is not delimiter-terminated, every chunk is delivered with sourceComplete=false here, and AbstractStreamingSource::pumpLines() only emits the trailing carry when that flag is true inside the callback. The final partial line is therefore left in line_buffer while the result reports complete=true, silently dropping the last record for common line-delimited MCP payloads.

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

@debuggerone
debuggerone merged commit b98b862 into main Jul 1, 2026
35 of 47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants