Skip to content

AX-1644 - add jfrog mcp and fix list available#25

Open
yanivt-jfrog wants to merge 2 commits into
mainfrom
AX-1644-fix-list-available
Open

AX-1644 - add jfrog mcp and fix list available#25
yanivt-jfrog wants to merge 2 commits into
mainfrom
AX-1644-fix-list-available

Conversation

@yanivt-jfrog

Copy link
Copy Markdown
Collaborator

No description provided.

yanivt-jfrog and others added 2 commits June 21, 2026 11:30
Two changes that mirror the same fix shipped on claude-plugin
main as PR #18:

1. plugin/.mcp.json: drop the trailing slash on the bundled
   --registry URL
   (.../coding-agents-npm/ -> .../coding-agents-npm). VS Code's
   MCP loader (and Claude Code's) dedupes server entries by
   (command, registry-URL); when the user already has an
   AI-Catalog-installed jfrog-mcp / chrome-devtools-mcp / etc.
   under the same npx + same registry URL, the bundled jfrog is
   silently skipped. The trailing-slash difference is enough to
   make the URL string distinct so dedup no longer collapses them.
   Functionally identical for npm, the agent-guard hook allowlist,
   and the agent guard itself.

2. plugin/templates/copilot-instructions.md: drop the
   top-of-document "# Built-in jfrog MCP" priming section. That
   section's "Hard rules" were strong enough to make Copilot bias
   listing answers toward the installed view and skip the
   AI-Catalog rows.

   Distribute the rules as small per-subsection notes that only
   fire when the LLM is actually doing that operation:
   - ## Adding an MCP: bundled jfrog is never installed via
     catalog; coexist as jfrog-catalog if catalog tool policy
     wanted.
   - ## Removing an MCP: removal is by uninstalling the JFrog
     plugin from VS Code; never delete the bundled .mcp.json.
   - ### Installed MCPs: report bundled jfrog as scope:
     plugin (jfrog), package: jfrog-mcp (bundled).

   The AX-1644 troubleshooting bullet ("Built-in jfrog MCP
   missing") is preserved exactly.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant