Skip to content

feat: carry compression transport upgrades from upstream #98205 - #101

Merged
mrkillbob merged 7 commits into
mainfrom
codex/fork-compression-transport-20260913
Sep 13, 2026
Merged

mrkillbob merged 7 commits into
mainfrom
codex/fork-compression-transport-20260913

Conversation

@mrkillbob

Copy link
Copy Markdown
Owner

Ports the genuinely missing compression portion of NousResearch/hermes-agent PR NousResearch#98205 onto the current fork runtime.

  • Adds selective gzip compression for eligible dashboard JSON/text responses while excluding secret-bearing routes.
  • Stores new trajectories as gzip JSONL by default and keeps the compressor readable for plain or gzip JSONL.
  • Focused validation: scripts/run_tests.sh tests/test_response_compression.py

…rch#98205

Salvages the upstream response-compression and gzip trajectory changes onto the fork's current decomposed runtime.

Co-authored-by: Mike DeMott <25466867+mrkillbob@users.noreply.github.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-13T18:59:11.304464Z 1973ec7 New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

૮ >ﻌ< ა ci review

running on b21d4cd — Merge branch 'main' into codex/fork-compression-transport-20


Still running 1 job: CI timing report

⚠️ Warnings

OSV vulnerability scan · View job

10 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 12m15s vs 12m31s (-2.1%). 13 job(s) slower, 5 faster, 3 unchanged.

  • Python tests / Run tests slice 4/8: +287.0s
  • Python tests / Run tests slice 8/8: -76.0s
  • Python tests / Run tests slice 3/8: +67.0s
  • Python tests / Run tests slice 2/8: -48.0s
  • Python tests / Run tests slice 7/8: +34.0s

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6651f505b3

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

Comment thread agent/trajectory.py Outdated
Comment thread agent/trajectory.py Outdated
Comment thread hermes_cli/web_server.py Outdated
Comment thread hermes_cli/response_compression.py
Comment thread agent/trajectory.py
@mrkillbob mrkillbob added the ci-reviewed Maintainer reviewed CI-sensitive changes label Sep 13, 2026

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1add46f285

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

Comment thread agent/trajectory.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 58b63af11d

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

Comment thread agent/trajectory.py Outdated
Comment thread hermes_cli/response_compression.py

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dfe1243af5

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

Comment thread agent/trajectory.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8077ddb190

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

Comment thread agent/trajectory.py Outdated
Comment thread agent/trajectory.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1973ec726b

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

Comment thread agent/trajectory.py
Comment thread agent/trajectory.py
@mrkillbob
mrkillbob enabled auto-merge September 13, 2026 18:59
@mrkillbob
mrkillbob merged commit f750e95 into main Sep 13, 2026
78 of 80 checks passed
@mrkillbob
mrkillbob deleted the codex/fork-compression-transport-20260913 branch September 13, 2026 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-reviewed Maintainer reviewed CI-sensitive changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant