chore: sync skills to v0.20.0 - #48
Open
yanivt-jfrog wants to merge 7 commits into
Open
Conversation
… jfrog-skills) Retires the legacy sessionStart hook that force-injected the ~440-line MCP-management template into every session. JFrog MCP management now lives in discoverable skills that sync in from jfrog/jfrog-skills, so this removes the hook machinery and the hand-maintained template to avoid a second source of truth. Removed: - plugins/jfrog/hooks/hooks.json — the sessionStart hook definition - plugins/jfrog/scripts/inject-instructions.mjs — the injector - plugins/jfrog/templates/jfrog-mcp-management.md — the always-on template - scripts/validate-hook-injector.mjs — the injector validator - .github/workflows/validate-inject-instructions.yml — CI for the injector Also dropped the now-unused "hooks" key from the plugin manifest and cleaned up the residual hook references in the READMEs. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The sessionStart hook was retired in favor of skills, but three README references still described Agent Guard as a "Hook": - README.md capabilities table: relabel the Agent Guard row Hook -> Skill - plugins/jfrog/README.md: drop "hook" from the JFrog CLI prerequisite Completes the README cleanup listed in the PR description / test plan. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Cursor skips re-installing/updating a plugin whose resolved version hasn't changed (see VENDOR.md). The hook->skills change alters plugin.json's manifest and hook wiring, so it needs a version bump for existing installs to pick it up.
Ports the harness-agnostic jfrog-mcp-management skill from jfrog-skills (feature/AX-1777-jfrog-mcp-skills) plus fixes found during the AX-1777 bug hunt (see Confluence "BUG hunt - Jfrog mcp skill"), including the Cursor "multiple/no JFrog server configured" case: the troubleshooting entry now points at the existing isDefault-based resolution chain in agent-guard-common.md instead of just saying "pass --server <ID>" with no guidance on picking one. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This skill is vendored from jfrog/jfrog-skills via the sync process and should not be hand-committed here.
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
2 participants
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.
Vendored from
v0.20.0of jfrog/jfrog-skills.