[codex] fix bundled skill installation and cleanup#29
Merged
Conversation
- record npm publish and registry consumer smoke proof - record GitHub release workflow repair and release URL - document the Chrome Web Store auth blocker on this machine Co-authored-by: Codex <noreply@openai.com>
- add a package postinstall runner that redistributes canonical bundled skills into managed global targets - ship the postinstall helper in npm artifacts and cover the path with focused tests - document install-time skill redistribution for package installs Co-authored-by: Codex <noreply@openai.com>
- remove repo-owned research/shopping alias installs by fingerprint - preserve user-modified alias directories during sync and uninstall - add historical alias fixtures to installer tests Co-authored-by: Codex <noreply@openai.com>
- delete the remaining research/shopping alias retirement branch - keep installer sync and uninstall focused on canonical opendevbrowser packs - update focused installer tests to prove legacy alias dirs are ignored Co-authored-by: Codex <noreply@openai.com>
- describe uninstall as canonical pack cleanup only - remove stale wording that implied legacy alias pruning Co-authored-by: Codex <noreply@openai.com>
- drop inert legacy alias fields from installer sync/remove outputs - remove dead alias preservation types from installer result shape - update focused installer tests to assert the fields are absent Co-authored-by: Codex <noreply@openai.com>
- restore explicit update scope precedence for managed skill refreshes - harden packaged postinstall validation docs and coverage - keep scoped lifecycle cleanup from mutating the other managed install mode Co-authored-by: Codex <noreply@openai.com>
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
1 participant
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
origin/mainRoot Cause
npm install -g opendevbrowserdid not run bundled skill redistribution, so canonical skills could be missing after package installValidation
npm run lintnpm run typechecknpm run buildnpm run test