AX-1847 Moving from session start hook to skills (skills sync in from jfrog-skills)#43
Open
MatanEden1 wants to merge 1 commit into
Open
AX-1847 Moving from session start hook to skills (skills sync in from jfrog-skills)#43MatanEden1 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 ~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>
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 ~440-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)plugins/jfrog/hooks/hooks.json— thesessionStarthook definitionplugins/jfrog/scripts/inject-instructions.mjs— the injector that force-fed the templateplugins/jfrog/templates/jfrog-mcp-management.md— the always-on templatescripts/validate-hook-injector.mjs— its validator.github/workflows/validate-inject-instructions.yml— CI for the injectorplugins/jfrog/.cursor-plugin/plugin.json— dropped the now-unused"hooks"keyREADME.md/plugins/jfrog/README.md— removed dangling hook references; Node bumped to ≥ 18The
validate-template.mjsgeneral manifest validator and itsvalidate-template.ymlworkflow are kept — they are not hook-specific.Where 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
node scripts/validate-template.mjspasses.🤖 Generated with Claude Code