Skip to content

fix(mcp): add OAuth authentication status column to mcp list output#119

Closed
echobt wants to merge 1 commit intomasterfrom
fix/bounty-issue-1311
Closed

fix(mcp): add OAuth authentication status column to mcp list output#119
echobt wants to merge 1 commit intomasterfrom
fix/bounty-issue-1311

Conversation

@echobt
Copy link
Contributor

@echobt echobt commented Jan 26, 2026

Summary

This PR fixes an issue where users could not see the OAuth authentication status of MCP servers when running cortex mcp list.

Changes

  • Added an "Auth" column to the mcp list command output
  • HTTP servers display their authentication status (Authenticated/Not Authenticated/Unknown)
  • Stdio servers display "N/A" since OAuth is only applicable to HTTP transport
  • Adjusted column widths to accommodate the new Auth column

Before

Name                 Status       Tools    Transport
-----------------------------------------------------------------------------
my-api               enabled      -        http: https://api.example.com
my-tool              enabled      -        stdio: npx my-tool

After

Name                 Status       Tools    Auth               Transport
------------------------------------------------------------------------------------------
my-api               enabled      -        Authenticated      http: https://api.example.com
my-tool              enabled      -        N/A                stdio: npx my-tool

Related

Fixes PlatformNetwork/bounty-challenge#1311

Fixes bounty issue #1311

The mcp list command now displays an Auth column showing authentication status
for HTTP MCP servers (Authenticated/Not Authenticated/Unknown) and N/A for
stdio servers since OAuth only applies to HTTP transport.
echobt pushed a commit that referenced this pull request Jan 27, 2026
This commit implements the following open PRs for cortex-cli:

1. PR #117 - Validate --days parameter range (1-3650)
   - Added value_parser to validate days is within 1-3650 range
   - Provides clear error messages for invalid values

2. PR #120 - Add pricing information to models --json output
   - Added input_cost_per_million and output_cost_per_million fields
   - Populated pricing for all cloud-hosted models
   - Local Ollama models show null (no API cost)

3. PR #121 - Add --compact flag for minified JSON config output
   - Added --compact flag to config command (requires --json)
   - Outputs single-line JSON instead of pretty-printed

4. PR #123 - Add explanation for Native field in agent show output
   - Changed from 'Native: true/false' to 'Native: yes/no (explanation)'
   - Shows 'built-in agent bundled with Cortex' or 'user-defined agent'

5. PR #125 - Add --diff flag to debug config command
   - Compares local project config with global config
   - Shows lines unique to each config and unified diff
   - Supports JSON output format

6. PR #128 - Display helpful error when running in non-TTY context
   - Checks for terminal support before starting TUI
   - Provides guidance to use 'cortex run <prompt>' for non-interactive use

7. PR #131 - Add help text explaining -- separator for mcp add command
   - Added after_help with examples showing correct usage
   - Explains the -- separator requirement for stdio servers

8. PR #132 - Defer success message until config write succeeds
   - Success messages now only print after std::fs::write() succeeds
   - Prevents misleading output when config write fails

9. PR #119 - Add OAuth authentication status column to mcp list output
   - Added 'Auth' column showing authentication status for HTTP servers
   - Stdio servers display 'N/A' (OAuth only applies to HTTP)

10. PR #116 - Return non-zero exit code when github status detects missing workflow
    - Now exits with code 1 if workflow is not installed
    - Works for both human-readable and JSON output modes

Co-authored-by: Droid Agent <droid@factory.ai>
echobt added a commit that referenced this pull request Jan 27, 2026
This commit implements the following open PRs for cortex-cli:

1. PR #117 - Validate --days parameter range (1-3650)
   - Added value_parser to validate days is within 1-3650 range
   - Provides clear error messages for invalid values

2. PR #120 - Add pricing information to models --json output
   - Added input_cost_per_million and output_cost_per_million fields
   - Populated pricing for all cloud-hosted models
   - Local Ollama models show null (no API cost)

3. PR #121 - Add --compact flag for minified JSON config output
   - Added --compact flag to config command (requires --json)
   - Outputs single-line JSON instead of pretty-printed

4. PR #123 - Add explanation for Native field in agent show output
   - Changed from 'Native: true/false' to 'Native: yes/no (explanation)'
   - Shows 'built-in agent bundled with Cortex' or 'user-defined agent'

5. PR #125 - Add --diff flag to debug config command
   - Compares local project config with global config
   - Shows lines unique to each config and unified diff
   - Supports JSON output format

6. PR #128 - Display helpful error when running in non-TTY context
   - Checks for terminal support before starting TUI
   - Provides guidance to use 'cortex run <prompt>' for non-interactive use

7. PR #131 - Add help text explaining -- separator for mcp add command
   - Added after_help with examples showing correct usage
   - Explains the -- separator requirement for stdio servers

8. PR #132 - Defer success message until config write succeeds
   - Success messages now only print after std::fs::write() succeeds
   - Prevents misleading output when config write fails

9. PR #119 - Add OAuth authentication status column to mcp list output
   - Added 'Auth' column showing authentication status for HTTP servers
   - Stdio servers display 'N/A' (OAuth only applies to HTTP)

10. PR #116 - Return non-zero exit code when github status detects missing workflow
    - Now exits with code 1 if workflow is not installed
    - Works for both human-readable and JSON output modes

Co-authored-by: Droid Agent <droid@factory.ai>
@echobt
Copy link
Contributor Author

echobt commented Jan 27, 2026

Closing as this feature has already been implemented in a batch commit (see commit history). The changes are merged into master.

@echobt echobt closed this Jan 27, 2026
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.

[BUG] MCP list missing OAuth status column

2 participants