Skip to content

Conversation

@continue
Copy link
Contributor

@continue continue bot commented Oct 7, 2025

Description

This PR implements support for URLs in the --mcp flag. When a URL starting with http:// or https:// is passed to the --mcp flag, it will now automatically be configured as a streamable-http MCP server connection.

Changes

  • Modified configEnhancer.ts to detect URLs in MCP specs and create streamable-http configurations
  • Added comprehensive tests for URL handling in the --mcp flag

Usage

Users can now run:

cn --mcp https://docs.continue.dev/mcp

This works alongside existing hub slug support:

cn --mcp https://example.com/mcp --mcp owner/package

Fixes CON-4285


This agent session was co-authored by nate and Continue.


Summary by cubic

Allows the --mcp flag to accept http/https URLs and auto-configures them as streamable-http MCP servers. Addresses CON-4285 and works alongside existing hub slug support.

  • New Features
    • Detects http/https in --mcp and creates a streamable-http config (type, url, name from hostname).
    • Supports mixing URLs and hub slugs in the same command.
    • Adds tests for URL handling and mixed inputs.

When a URL starting with http:// or https:// is passed to the --mcp flag,
it will now automatically be configured as a streamable-http MCP server
connection instead of trying to load it from the hub.

This allows users to run:
  cn --mcp https://docs.continue.dev/mcp

The URL will be parsed and configured with:
- type: streamable-http
- url: the provided URL
- name: hostname from the URL

Fixes CON-4285

Generated with [Continue](https://continue.dev)

Co-authored-by: Username <[email protected]>
sestinj
sestinj previously approved these changes Oct 7, 2025
const config = await enhancer.enhanceConfig(mockConfig, options);

// MCPs from --mcp flag should be prepended (at the start)
expect(config.mcpServers).toHaveLength(2);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like these tests failed, you need to make sure they pass

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 7, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 7, 2025
@sestinj sestinj marked this pull request as ready for review October 7, 2025 18:24
@sestinj sestinj requested a review from a team as a code owner October 7, 2025 18:24
@sestinj sestinj requested review from Patrick-Erichsen and removed request for a team October 7, 2025 18:24
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 7, 2025
@github-actions
Copy link

github-actions bot commented Oct 7, 2025

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

const config = await enhancer.enhanceConfig(mockConfig, options);

// MCPs from --mcp flag should be prepended (at the start)
expect(config.mcpServers).toHaveLength(2);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix the failing tests

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Oct 8, 2025
@sestinj sestinj merged commit 7b17637 into main Oct 8, 2025
53 of 55 checks passed
@sestinj sestinj deleted the continuebot/con-4285 branch October 8, 2025 16:49
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 8, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2025
@sestinj
Copy link
Contributor

sestinj commented Oct 12, 2025

🎉 This PR is included in version 1.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Oct 14, 2025

🎉 This PR is included in version 1.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Oct 15, 2025

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants