Skip to content

misc: revert additional path pattern for kusto graph plugin #3108

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JasonYeMSFT (JasonYeMSFT) merged 1 commit into from
Aug 24, 2026
Merged

misc: revert additional path pattern for kusto graph plugin #3108

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 3 additions & 27 deletions hooks/scripts/track-telemetry.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,20 +74,14 @@
# - toolArgs.path / toolArgs.filePath (Copilot CLI)
# - tool_input.filePath / tool_input.file_path / tool_input.path (Claude Code / VS Code)
#
# Recognized install paths (one set per plugin, see $pathPatterns below):
# azure-skills:
# Recognized azure-skills install paths:
# - .copilot/installed-plugins/<catalog-name>/azure/skills/...
# (<catalog-name> is the marketplace/catalog folder the plugin was
# installed under, e.g. "awesome-copilot" — it does not necessarily
# match the plugin's own name, "azure")
# - .claude/plugins/cache/azure-skills/azure/<version>/skills/...
# - .claude/plugins/cache/claude-plugins-official/azure/<version>/skills/...
# - .vscode/agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-skills/skills/...
# azure-kusto-graph-skills:
# - .copilot/installed-plugins/<catalog-name>/azure-kusto-graph-skills/skills/...
# - .claude/plugins/cache/azure-skills/azure-kusto-graph-skills/<version>/skills/...
# - .vscode/agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-kusto-graph-skills/skills/...
# shared:
# - .agents/skills/...
#
# If the path matches AND is not a SKILL.md file, the relative path after
Expand Down Expand Up @@ -295,32 +289,14 @@ function Get-ToolInputPath {

# === STEP 2: Determine what to track for azmcp ===

# Path patterns per client, one block per plugin (used for SKILL.md and
# file-reference matching). Add a new block (with the "azure-skills"
# segments swapped for the new plugin's name) when onboarding another plugin.

# --- azure-skills plugin ---
# The Copilot CLI pattern wildcards the catalog/marketplace folder name
# (e.g. "awesome-copilot") since it does not necessarily match the plugin's
# own name ("azure").
# Azure-skills path patterns per client (used for SKILL.md and file-reference matching)
$pathPatternCopilot = '\.copilot/installed-plugins/[^/]+/azure/skills/'
$pathPatternClaude = '\.claude/plugins/cache/(azure-skills|claude-plugins-official)/azure/[0-9.]+/skills/'
$pathPatternVscodeAgentPlugins = 'agent-plugins/github\.com/microsoft/azure-skills/\.github/plugins/azure-skills/skills/'

# --- azure-kusto-graph-skills plugin ---
$pathPatternCopilotKustoGraph = '\.copilot/installed-plugins/[^/]+/azure-kusto-graph-skills/skills/'
$pathPatternClaudeKustoGraph = '\.claude/plugins/cache/azure-skills/azure-kusto-graph-skills/[0-9.]+/skills/'
$pathPatternVscodeAgentPluginsKustoGraph = 'agent-plugins/github\.com/microsoft/azure-skills/\.github/plugins/azure-kusto-graph-skills/skills/'

# --- shared across all plugins ---
$pathPatternAgentsSkills = '\.agents/skills/'

# Put the path patterns into an array for easier iteration
$pathPatterns = @(
$pathPatternCopilot, $pathPatternClaude, $pathPatternVscodeAgentPlugins,
$pathPatternCopilotKustoGraph, $pathPatternClaudeKustoGraph, $pathPatternVscodeAgentPluginsKustoGraph,
$pathPatternAgentsSkills
)
$pathPatterns = @($pathPatternCopilot, $pathPatternClaude, $pathPatternVscodeAgentPlugins, $pathPatternAgentsSkills)

# If $env:AZURE_SKILLS_PLUGIN_ROOT is set, add it to the path patterns for local skill development
if ($env:AZURE_SKILLS_PLUGIN_ROOT) {
Expand Down
28 changes: 3 additions & 25 deletions hooks/scripts/track-telemetry.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,20 +76,14 @@
# - toolArgs.path / toolArgs.filePath (Copilot CLI)
# - tool_input.filePath / tool_input.file_path / tool_input.path (Claude Code / VS Code)
#
# Recognized install paths (one set per plugin, see is_azure_skills_path):
# azure-skills:
# Recognized azure-skills install paths:
# - .copilot/installed-plugins/<catalog-name>/azure/skills/...
# (<catalog-name> is the marketplace/catalog folder the plugin was
# installed under, e.g. "awesome-copilot" — it does not necessarily
# match the plugin's own name, "azure")
# - .claude/plugins/cache/azure-skills/azure/<version>/skills/...
# - .claude/plugins/cache/claude-plugins-official/azure/<version>/skills/...
# - .vscode/agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-skills/skills/...
# azure-kusto-graph-skills:
# - .copilot/installed-plugins/<catalog-name>/azure-kusto-graph-skills/skills/...
# - .claude/plugins/cache/azure-skills/azure-kusto-graph-skills/<version>/skills/...
# - .vscode/agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-kusto-graph-skills/skills/...
# shared:
# - .agents/skills/...
#
# If the path matches AND is not a SKILL.md file, the relative path after
Expand Down Expand Up @@ -302,30 +296,15 @@ fi

# === STEP 2: Determine what to track for azmcp ===

# Check if a path matches any known plugin skills folder structure.
# Each plugin has its own block below — add a new block (with the
# "azure-skills" segments swapped for the new plugin's name) when onboarding
# another plugin. Returns 0 (true) if matched, 1 (false) otherwise.
# Check if a path matches any known azure-skills folder structure
# Returns 0 (true) if matched, 1 (false) otherwise
is_azure_skills_path() {
local p="$1"

# --- azure-skills plugin ---
# The Copilot CLI pattern wildcards the catalog/marketplace folder name
# (e.g. "awesome-copilot") since it does not necessarily match the
# plugin's own name ("azure").
[[ "$p" == *".copilot/installed-plugins/"*"/azure/skills/"* ]] && return 0
[[ "$p" == *".claude/plugins/cache/azure-skills/azure/"*"/skills/"* ]] && return 0
[[ "$p" == *".claude/plugins/cache/claude-plugins-official/azure/"*"/skills/"* ]] && return 0
[[ "$p" == *"agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-skills/skills/"* ]] && return 0

# --- azure-kusto-graph-skills plugin ---
[[ "$p" == *".copilot/installed-plugins/"*"/azure-kusto-graph-skills/skills/"* ]] && return 0
[[ "$p" == *".claude/plugins/cache/azure-skills/azure-kusto-graph-skills/"*"/skills/"* ]] && return 0
[[ "$p" == *"agent-plugins/github.com/microsoft/azure-skills/.github/plugins/azure-kusto-graph-skills/skills/"* ]] && return 0

# --- shared across all plugins ---
[[ "$p" == *".agents/skills/"* ]] && return 0

# Local plugin development: match paths under AZURE_SKILLS_PLUGIN_ROOT/skills/
# (e.g. when loading a local plugin via `--plugin-dir`)
if [ -n "$AZURE_SKILLS_PLUGIN_ROOT" ]; then
Expand Down Expand Up @@ -452,4 +431,3 @@ fi

# Output success to stdout (required by hooks)
return_success

Loading