Skip to content

Keep image resize notices attached during remote compaction - #36956

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/82bb70a46eb68b07b4601e7fb345921d0789f442
Aug 4, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/82bb70a46eb68b07b4601e7fb345921d0789f442

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

Keep image resize notices attached during remote compaction

Why

Image resize notices describe the history item immediately before them, so
remote compaction must not preserve or discard them independently of that item.

What changed

  • Group recognized image_resize_notice developer messages with their source
    history items when filtering compacted history.
  • Account for each group together when trimming or truncating history to a token
    budget, preserving the notice only when its source is retained.

Testing

  • Extend the remote compaction v2 follow-up test to verify that a retained user
    image keeps its resize notice, while notices for discarded items and
    unrecognized developer notices are omitted.

## Why

Image resize notices describe the history item immediately before them, so
remote compaction must not preserve or discard them independently of that item.

## What changed

- Group recognized `image_resize_notice` developer messages with their source
  history items when filtering compacted history.
- Account for each group together when trimming or truncating history to a token
  budget, preserving the notice only when its source is retained.

## Testing

- Extend the remote compaction v2 follow-up test to verify that a retained user
  image keeps its resize notice, while notices for discarded items and
  unrecognized developer notices are omitted.

GitOrigin-RevId: 82bb70a46eb68b07b4601e7fb345921d0789f442
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/82bb70a46eb68b07b4601e7fb345921d0789f442 branch from d4279db to 4bd5b9f Compare August 4, 2026 19:34
@copyberry
copyberry Bot merged commit 4bd5b9f into main Aug 4, 2026
26 of 30 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/82bb70a46eb68b07b4601e7fb345921d0789f442 branch August 4, 2026 19:35
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2026

This branch was previously deployed

1 inactive deployment
issue-triage — 4bd5b9fd Deployed Aug 4, 2026 by sickdude-inc via Generate label suggestions #36085
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants