Skip to content

Improve MCP token user linking - #181

Merged
ecarreras merged 1 commit into
mainfrom
issue-180-mcp-user-selection
Aug 7, 2026
Merged

Improve MCP token user linking#181
ecarreras merged 1 commit into
mainfrom
issue-180-mcp-user-selection

Conversation

@pilipilisbot

Copy link
Copy Markdown
Owner

Summary

  • replace the admin MCP owner free-text field with a selector populated from known dashboard users
  • add admin-only MCP token owner reassignment so existing active tokens can be linked to a user
  • reject typo/unknown owners at the API layer and document the operational behavior

Fixes #180

Screenshots

github-agent-bridge-issue180-mcp-owner-selector.png

Verification

  • . ../github-agent-bridge/.venv/bin/activate && pytest -q tests/test_mcp.py tests/test_backend.py (72 passed)
  • . ../github-agent-bridge/.venv/bin/activate && pytest -q (350 passed)
  • npm test -- --run src/main.test.tsx (50 passed)
  • npm run build

Co-authored-by: ecarreras <294235+ecarreras@users.noreply.github.com>
@pilipilisbot pilipilisbot mentioned this pull request Aug 5, 2026
@ecarreras
ecarreras merged commit 2fb807e into main Aug 7, 2026
3 checks passed
@ecarreras
ecarreras deleted the issue-180-mcp-user-selection branch August 7, 2026 08:59
@pilipilisbot

Copy link
Copy Markdown
Owner Author

Post-merge review pass completed for #181. I did not find a follow-up blocker in the owner-linking changes, and I reran the focused checks locally on the PR checkout:

  • . ../github-agent-bridge/.venv/bin/activate && pytest -q tests/test_mcp.py tests/test_backend.py -> 72 passed
  • npm test -- --run src/main.test.tsx -> 50 passed

No code or PR metadata changes were made because this was a review-only run and the PR is already merged.

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.

Millora MCP

2 participants