Skip to content

fix(mcp): Surface MCP OAuth login diagnostics#468

Merged
CSRessel merged 4 commits into
mainfrom
fix-mcp-login-url
May 13, 2026
Merged

fix(mcp): Surface MCP OAuth login diagnostics#468
CSRessel merged 4 commits into
mainfrom
fix-mcp-login-url

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

Summary

🤖 Generated with Nori

  • Show richer MCP OAuth failure details by preserving the anyhow error chain.
  • Add regression coverage for Linear-style percent-encoded OAuth callback codes.
  • Ignore invalid empty .git directories when detecting TUI git roots, keeping temp-dir tests and instruction discovery stable.

Test Plan

  • env -u RUSTC_WRAPPER cargo test -p codex-rmcp-client
  • env -u RUSTC_WRAPPER cargo test -p nori-tui
  • cargo build --bin nori
  • env -u RUSTC_WRAPPER cargo test -p tui-pty-e2e
  • just fmt
  • env -u RUSTC_WRAPPER just fix -p codex-rmcp-client
  • env -u RUSTC_WRAPPER just fix -p nori-tui
  • TUI smoke test with elizacp via isolated tmux

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

@CSRessel CSRessel changed the title Fix MCP OAuth login diagnostics fix(mcp): Surface MCP OAuth login diagnostics May 12, 2026
# Conflicts:
#	nori-rs/tui/src/app/config_persistence.rs
#	nori-rs/tui/src/effective_cwd_tracker.rs
#	nori-rs/tui/src/nori/session_header/mod.rs
@CSRessel CSRessel merged commit aa168c4 into main May 13, 2026
3 checks passed
@CSRessel CSRessel deleted the fix-mcp-login-url branch May 13, 2026 20:27
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