feat(mcp-guardrails): ui changes

[Madhuvod]

Summary

Briefly explain the purpose of this PR and the problem it solves.

Changes

• What was changed and why
• Any notable design decisions or trade-offs

Type of change

• Bug fix
• Feature
• Refactor
• Documentation
• Chore/CI

Affected areas

• Core (Go)
• Transports (HTTP)
• Providers/Integrations
• Plugins
• UI (React)
• Docs

How to test

Describe the steps to validate this change. Include commands and expected outcomes.

# Core/Transports
go version
go test ./...

# UI
cd ui
pnpm i || npm i
pnpm test || npm test
pnpm build || npm run build

If adding new configs or environment variables, document them here.

Screenshots/Recordings

If UI changes, add before/after screenshots or short clips.

Breaking changes

• Yes
• No

If yes, describe impact and migration instructions.

Related issues

Link related issues and discussions. Example: Closes #123

Security considerations

Note any security implications (auth, secrets, PII, sandboxing, etc.).

Checklist

• I read docs/contributing/README.md and followed the guidelines
• I added/updated tests where appropriate
• I updated documentation where needed
• I verified builds succeed (Go and UI)
• I verified the CI pipeline passes locally if applicable

[Madhuvod]

• added plugin logs in mcp logs #5746
• feat(mcp-guardrails): ui changes #5745 👈 (View in Graphite)
• feat(mcp-guardrails): add MCP log redaction changes #5744
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• New Features

  • MCP log details now support permission-controlled reveal of original values.
  • Redacted inputs, outputs, results, and error details are displayed consistently with available mappings.
  • Reveal controls reset automatically when viewing a different log or when access changes.
  • Redaction mappings are now consistently supported across log and MCP tool log data.

• Bug Fixes

  • Improved handling of overlapping and nested redaction replacements while preserving displayed data integrity.

Walkthrough

The change centralizes redaction utilities, adds shared redaction mapping types, and enables RBAC-controlled reveal behavior for MCP log details. Redacted arguments, results, and errors use transformed values unless reveal access is available and enabled.

Changes

MCP log redaction and reveal

Layer / File(s)	Summary
Redaction contracts and transformation utilities ui/lib/types/logs.ts, ui/lib/utils/redaction.ts, ui/lib/utils/redaction.test.ts	Adds the shared RedactionMapping type and utilities for detecting, merging, and recursively applying redaction mappings. Tests cover replacement, conflicts, immutability, and identical mappings.
RBAC-controlled MCP log reveal ui/app/workspace/mcp-logs/page.tsx, ui/app/workspace/mcp-logs/views/mcpLogDetailsSheet.tsx	Checks Logs reveal access, passes canReveal, adds reveal state and a switch, resets state when log availability changes, and renders transformed arguments, results, and errors.
Shared utility adoption in log details ui/app/workspace/logs/sheets/logDetailView.tsx	Imports redaction helpers from the shared utility module.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant MCPLogsPage
  participant MCPLogDetailSheet
  participant RedactionUtilities
  MCPLogsPage->>MCPLogsPage: Check Logs Reveal access
  MCPLogsPage->>MCPLogDetailSheet: Pass canReveal
  MCPLogDetailSheet->>RedactionUtilities: Apply redaction mappings
  RedactionUtilities-->>MCPLogDetailSheet: Return transformed values
  MCPLogDetailSheet->>MCPLogDetailSheet: Toggle original values when permitted

Loading

Possibly related PRs

• maximhq/bifrost#5359: Adds the MCP RedactionMapping data consumed by this UI change.
• maximhq/bifrost#5366: Contains related redaction utilities, MCP log components, RBAC reveal flow, and log types.
• maximhq/bifrost#5738: Extends MCP log redaction mappings into the UI.

Suggested reviewers: akshaydeo, impoiler, pratham-mishra04

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (3 warnings)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The PR changes MCP log redaction and reveal access, but linked issue #123 requires File API support for provider file uploads.	Implement the File API support described in issue #123, or link the issue that defines the MCP guardrails UI requirements.
Out of Scope Changes check	⚠️ Warning	The changes concern MCP log UI redaction and reveal access, which are unrelated to the File API support required by issue #123.	Remove the unrelated MCP UI changes from this PR or update the linked issues to match its intended scope.
Description check	⚠️ Warning	The description contains only the unfilled template and omits implementation details, testing steps, screenshots, and security information.	Replace the template placeholders with a summary, concrete changes, affected areas, test steps, UI screenshots, breaking-change status, related issues, and security considerations.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Title check	✅ Passed	The title identifies UI changes for MCP guardrails and matches the pull request's primary scope.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 07-20-feat_mcp-guardrails_ui_changes

---

_{Comment @coderabbitai help to get the list of available commands.}

[akshaydeo]

Merge activity

• Jul 31, 12:04 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jul 31, 12:05 PM UTC: @akshaydeo merged this pull request with Graphite.

The base branch was changed.