Skip to content

Require prefixes for outbound response item IDs - #32312

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0209fe430c826d0ae88bc4648652c4ebf390c4a4
Jul 11, 2026
Merged

Require prefixes for outbound response item IDs#32312
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/0209fe430c826d0ae88bc4648652c4ebf390c4a4

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 11, 2026

Copy link
Copy Markdown

Require prefixes for outbound response item IDs

What changed

  • Add a ResponseItemId type that generates item-specific prefixes with UUIDv7 suffixes and use it across response items.
  • Keep deserialization permissive for legacy histories, but omit empty or unprefixed item IDs from HTTP and WebSocket requests.
  • Export the new type in the generated TypeScript protocol schema.

Testing

  • Cover prefixed ID generation, legacy deserialization, prefix recognition, and outbound request filtering.

## What changed

- Add a `ResponseItemId` type that generates item-specific prefixes with UUIDv7 suffixes and use it across response items.
- Keep deserialization permissive for legacy histories, but omit empty or unprefixed item IDs from HTTP and WebSocket requests.
- Export the new type in the generated TypeScript protocol schema.

## Testing

- Cover prefixed ID generation, legacy deserialization, prefix recognition, and outbound request filtering.

GitOrigin-RevId: 0209fe430c826d0ae88bc4648652c4ebf390c4a4
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/0209fe430c826d0ae88bc4648652c4ebf390c4a4 branch from 6129218 to c9d52de Compare July 11, 2026 02:30
@copyberry
copyberry Bot merged commit c9d52de into main Jul 11, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/0209fe430c826d0ae88bc4648652c4ebf390c4a4 branch July 11, 2026 02:31
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 11, 2026
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.

5 participants