Skip to content

fix(readme): correct Codex marketplace install command#106

Merged
kantorcodes merged 1 commit into
mainfrom
fix/codex-marketplace-install-url
May 15, 2026
Merged

fix(readme): correct Codex marketplace install command#106
kantorcodes merged 1 commit into
mainfrom
fix/codex-marketplace-install-url

Conversation

@kantorcodes
Copy link
Copy Markdown
Member

@kantorcodes kantorcodes commented May 15, 2026

Summary:

  • replace raw marketplace JSON URL with Git repo marketplace command Codex expects
  • include sparse paths for marketplace manifest and mirrored plugin bundles
  • document why raw GitHub file URLs fail with remote 404

Closes #64.

Verification:

  • README install command check passed
  • marketplace JSON validates with jq
  • git diff --check passed
  • README guidance guard script passed

Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 15, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • README.md - Documentation-only changes fixing marketplace command instructions

Reviewed by laguna-m.1-20260312:free · 167,866 tokens

@kantorcodes kantorcodes merged commit 836b9b9 into main May 15, 2026
9 checks passed
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README.md to use a Git repository URL instead of a raw file URL for the codex plugin marketplace add command, incorporating flags for sparse checkouts and adding a warning about raw URL failures. A suggestion was made to wrap the repository URL in double quotes for better shell compatibility.

Comment thread README.md
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.

Install instructions don't work

1 participant