Skip to content

[codex] Reuse revision-stable plugin catalog snapshots#28453

Closed
mzeng-openai wants to merge 1 commit into
dev/mzeng/plugin-catalog-revisionsfrom
dev/mzeng/plugin-catalog-snapshots
Closed

[codex] Reuse revision-stable plugin catalog snapshots#28453
mzeng-openai wants to merge 1 commit into
dev/mzeng/plugin-catalog-revisionsfrom
dev/mzeng/plugin-catalog-snapshots

Conversation

@mzeng-openai

Copy link
Copy Markdown
Collaborator

Why

Marketplace listing rebuilds the same revision-stable catalogs repeatedly. Local and unrevisioned sources still need a fresh read.

What changed

  • Build a fresh aggregate PluginCatalogSnapshot for each request.
  • Reuse only source fragments backed by root-local revision markers.
  • Always rebuild home, additional, local, or otherwise unrevisioned sources.
  • Keep the fragment cache bounded and avoid manual invalidation calls.

This is 2/3 in the plugin catalog stack and depends on #28451.

Validation

  • just test -p codex-core-plugins — 226 passed
  • just fix -p codex-core-plugins

@mzeng-openai

Copy link
Copy Markdown
Collaborator Author

Consolidating this into #27812 so the revision, snapshot, and consumer changes can be reviewed together.

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.

1 participant