Issue #194: preserve Codex marketplace metadata - #212
Closed
NWYLZW wants to merge 8 commits into
Closed
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
There was a problem hiding this comment.
Experience Review
Before merge, make sure the PR body contains this completed checklist:
## Experience Review
- [x] 已判断是否需要沉淀经验
- [x] 如需要,已运行 `$post-task-experience-review`
- [x] reviewer PASS 后才进入 mergeChecklist to confirm:
- Decide whether this PR produced reusable project experience.
- If experience should be captured, run
$post-task-experience-review. - Merge only after the independent reviewer reports PASS.
NWYLZW
marked this pull request as ready for review
August 3, 2026 05:52
NWYLZW
force-pushed
the
codex/issue-194-codex-marketplace-metadata
branch
from
August 3, 2026 07:56
470f7f3 to
208e854
Compare
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Root cause and user impact
Codex marketplace conversion did not preserve a stable manifest or the full app/capability/connection metadata needed by runtime discovery. Runtime and client fallbacks could therefore use an absolute install directory as the plugin request ID or display name, while installed plugins appeared with empty metadata. The same loose internal-to-public projection could expose private install roots through API, UI, diagnostics, or logs.
For users, this meant a newly installed Codex plugin could display a machine-specific absolute path, omit its useful capabilities and connection requirements, and behave inconsistently across duplicate installs or upgrades.
Current implementation
.app.jsonmetadata into strict runtime fields, including apps, capabilities, permissions, authentication, and connection requirements.root,rootDir,sourceRoot, orpluginRoot.Changelog
Included in the current head at
changelog/0.1.0-beta.10/readme.md. The entry covers hardened marketplace metadata handling, actionable source errors, retryable install-source failures, and installed runtime metadata after install or refresh.Screenshots
Privacy-reviewed macOS evidence is committed in evidence commit
be8fdfbdc224dfa4e2425ab29c72231eb2c11b06, parented directly on the reviewed PR head. The three files contain no private filesystem paths, credentials, tokens, usernames, or secrets.Screenshot SHA-256 values:
airtable-installed.jpg:aa86d121899993f0ccd0147898145d6993113fcdf3a8e8698e47608deb3e02a1airtable-runtime-detail.jpg:f0ccf814820f523e11a73fc8b2ef96aa6599505738aefe87390d5e6b937d9c05plugin-list.jpg:d5c2f91052d59f43f55d623aa8cc425987b850182f5a2407d0a78b749acf01d2Validation
The reviewed source/test tree is exact at current PR head
279696755634ceee562de8aa0201f72b11d6e148: all eight candidate paths match byte-for-byte. The evidence commitbe8fdfbdc224dfa4e2425ab29c72231eb2c11b06adds only the three screenshots above. The framed eight-path candidate fingerprint isac7fe11344cb789860559e96ff1342129a2970d2b9bcc72dc35a36b8fc525e6a.git diff --checkpassed.airtable, app idasdk_app_693ca6ce2db08191bb52d66743c65184, and capabilitiesRead, Write.codex-openai-plugins-airtable-52fa4877979453b87dbb90a4, sourcemarketplace/codex/openai-plugins/airtable, version0.1.3, enabledtrue; permissions, authentication, and connection requirements werenulland no values were synthesized in the UI.openai-curated-remotesource-not-found message remained inline while the Airtable card and installed runtime detail stayed usable. This PR does not claim that external source availability is fixed.Latest independent review: PASS
Independent high-risk code review reproduced the exact base, eight-path fingerprint, evidence artifact identities, and focused gates; no blocking security or correctness finding remained. Independent live visual review is recorded as PASS for the exact current candidate and evidence. Experience Review is therefore complete for this delivery scope.
Experience Review
$post-task-experience-reviewCloses #194