Skip to content

Avoid duplicate cached app list update notifications - #33640

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/54f2d25a0adf6e0707402328b4c81fa5f950cf9a
Jul 16, 2026
Merged

Avoid duplicate cached app list update notifications#33640
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/54f2d25a0adf6e0707402328b4c81fa5f950cf9a

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 16, 2026

Copy link
Copy Markdown

Avoid duplicate cached app list update notifications

What changed

  • Emit one final app/list/updated notification for an initial cached
    app/list request so other initialized clients can refresh.
  • Suppress duplicate notifications for unchanged cached continuation pages.
  • Preserve progressive update notifications for forceRefetch: true requests.

Testing

  • Cover notification behavior for cached initial and paginated requests.

## What changed

- Emit one final `app/list/updated` notification for an initial cached
  `app/list` request so other initialized clients can refresh.
- Suppress duplicate notifications for unchanged cached continuation pages.
- Preserve progressive update notifications for `forceRefetch: true` requests.

## Testing

- Cover notification behavior for cached initial and paginated requests.

GitOrigin-RevId: 54f2d25a0adf6e0707402328b4c81fa5f950cf9a
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/54f2d25a0adf6e0707402328b4c81fa5f950cf9a branch from a85fd50 to f722a7e Compare July 16, 2026 16:44
@copyberry
copyberry Bot merged commit f722a7e into main Jul 16, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/54f2d25a0adf6e0707402328b4c81fa5f950cf9a branch July 16, 2026 16:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 16, 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.

4 participants