AX-1849 Moving from session start hook to skills (skills sync in from jfrog-skills)#36
Open
MatanEden1 wants to merge 1 commit into
Open
AX-1849 Moving from session start hook to skills (skills sync in from jfrog-skills)#36MatanEden1 wants to merge 1 commit into
MatanEden1 wants to merge 1 commit into
Conversation
… jfrog-skills) Retires the legacy SessionStart hook that force-injected the ~480-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: - plugin/hooks/hooks.json — the SessionStart hook definition - plugin/scripts/inject-instructions.mjs — the injector - plugin/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 README (Agent Guard is now delivered through the synced skills). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.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
Retires the legacy
SessionStarthook that force-injected a ~480-line MCP-management template into every session. JFrog MCP management now lives in discoverable skills that are vendored in via sync fromjfrog/jfrog-skillsrather than driven by a hand-maintained hook + template — so this branch removes the hook machinery to avoid a second source of truth.Mirrors the equivalent change in
jfrog/claude-plugin#33.Net change vs
main(pure removal)plugin/hooks/hooks.json— theSessionStarthook definitionplugin/scripts/inject-instructions.mjs— the injector that force-fed the templateplugin/templates/jfrog-mcp-management.md— the always-on templatescripts/validate-hook-injector.mjs— its validator.github/workflows/validate-inject-instructions.yml— CI for the injectorplugin/.claude-plugin/plugin.json— dropped the now-unused"hooks"keyREADME.md— removed dangling hook references (the Agent Guard is now delivered through the synced skills); Node bumped to ≥ 18Where the skills live now
The JFrog MCP skills + their shared references arrive in this plugin through the vendored sync from
jfrog/jfrog-skills, not by hand-committing them here.Test plan
🤖 Generated with Claude Code