Skip to content

Show skill MCP scopes per agent in status - #523

Draft
xsh310 wants to merge 1 commit into
xshen/skill-per-agent-addfrom
xshen/skill-per-agent-status
Draft

Show skill MCP scopes per agent in status#523
xsh310 wants to merge 1 commit into
xshen/skill-per-agent-addfrom
xshen/skill-per-agent-status

Conversation

@xsh310

@xsh310 xsh310 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

What

ug status rendered a single Skill MCP Locations line from the flat scope mirror. Once agents carry different skill scopes (which per-agent add now makes possible), that single line hides the divergence. This makes status report each agent's real scope.

How

  • The Skills section reads each configured client's scope with skill_locations_for_client.
  • When every agent shares the same scope, it collapses to the original single Skill MCP Locations + Configured lines.
  • When scopes diverge, it prints a per-agent line (Claude Code skill MCP locations: ..., Codex skill MCP locations: ...), matching how _print_skills_summary already reports scopes.

Tests

  • test_cli.py: a state whose per-client scopes diverge renders one line per agent with that agent's own locations. The existing uniform-scope and no-locations status tests still pass unchanged.

uv run pytest tests/test_mcp.py tests/test_cli.py tests/test_lint.py is green.

Stacking

Second in the stacked per-agent skills series, based on xshen/skill-per-agent-add (#522). Reviewing the diff against that base shows just this change. It rebuilds behavior originally designed by Arthur Jenoudet on the current per-client-map state model.

This pull request and its description were written by Isaac.

`ug status` rendered a single `Skill MCP Locations` line off the flat mirror,
which hides divergence once agents carry different skill scopes. Read each
configured client's scope with `skill_locations_for_client` and collapse to one
line when every agent shares a scope, splitting into a per-agent line when they
diverge, matching how the skills summary already reports scopes.

Co-authored-by: Arthur Jenoudet <arthur.jenoudet@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@xsh310
xsh310 force-pushed the xshen/skill-per-agent-status branch from 189150f to a1669b2 Compare September 8, 2026 05:43
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