Skip to content

fix(tui): show MCP tool selectors in the policy viewer - #5345

Merged
shiju-nv merged 2 commits into
NVIDIA:mainfrom
shiju-nv:fix/term-mcp-policy-tools/shiju-nv
Nov 6, 2026
Merged

shiju-nv merged 2 commits into
NVIDIA:mainfrom
shiju-nv:fix/term-mcp-policy-tools/shiju-nv

Conversation

@shiju-nv

@shiju-nv shiju-nv commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Show each allowed or denied MCP tool name or pattern in the Policy pane instead of the misleading tools/call * display.

Related Issue

Closes #5343

Changes

  • Render tool selectors on separate scrollable lines.
  • Add coverage for tool lists, patterns and omitted methods.

Release Notes

Testing

  • Focused tests (2), Clippy and live TUI verification (12 tools) pass.
  • Unit tests added; faulty-base failure and fixed-renderer success verified.
  • Full E2E suite (not run).

Checklist

  • Follows Conventional Commits.
  • Commits signed off (DCO).
  • Architecture docs updated (not applicable).

Signed-off-by: Shiju <shiju@nvidia.com>
Signed-off-by: Shiju <shiju@nvidia.com>
@shiju-nv shiju-nv added area:tui Terminal UI work topic:l7 Application-layer policy and inspection work rust Pull requests that update rust code area:platform and removed topic:l7 Application-layer policy and inspection work area:tui Terminal UI work labels Nov 6, 2026
@shiju-nv
shiju-nv enabled auto-merge November 6, 2026 20:25
@shiju-nv
shiju-nv added this pull request to the merge queue Nov 6, 2026
Merged via the queue into NVIDIA:main with commit 00f7ba7 Nov 6, 2026
95 of 99 checks passed
@shiju-nv
shiju-nv deleted the fix/term-mcp-policy-tools/shiju-nv branch November 6, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:platform rust Pull requests that update rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

openshell term hides MCP tool selectors in the policy pane

2 participants