vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-07-09 17:19:02 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

fix(cli): accept uppercase URL schemes in mcp add transport detection (#5426)
Some checks are pending
Qwen Code CI / Classify PR (push) Waiting to run
Details
Qwen Code CI / Lint (push) Blocked by required conditions
Details
Qwen Code CI / Test (macos-latest, Node 22.x) (push) Blocked by required conditions
Details
Qwen Code CI / Test (ubuntu-latest, Node 22.x) (push) Blocked by required conditions
Details
Qwen Code CI / Test (windows-latest, Node 22.x) (push) Blocked by required conditions
Details
Qwen Code CI / Post Coverage Comment (push) Blocked by required conditions
Details
Qwen Code CI / CodeQL (push) Blocked by required conditions
Details
Qwen Code CI / Integration Tests (CLI, No Sandbox) (push) Waiting to run
Details
E2E Tests / E2E Test (Linux) - sandbox:docker (push) Waiting to run
Details
E2E Tests / E2E Test (Linux) - sandbox:none (push) Waiting to run
Details
E2E Tests / E2E Test - macOS (push) Waiting to run
Details

Browse source
This commit is contained in:
Yufeng He 2026-06-20 09:36:44 +08:00 • committed by GitHub
parent 18cc73ce05
commit 196670a356
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 11 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
packages/cli/src/commands/mcp/add.test.ts
View file

@ -115,6 +115,16 @@ describe('mcp add command', () => {
});
});
it('should auto-detect http transport when commandOrUrl uses an uppercase URL scheme', async () => {
await parser.parseAsync('add http-server HTTPS://example.com/mcp');
expect(mockSetValue).toHaveBeenCalledWith(SettingScope.User, 'mcpServers', {
'http-server': {
httpUrl: 'HTTPS://example.com/mcp',
},
});
});
it('should respect explicit transport even when commandOrUrl is a URL', async () => {
await parser.parseAsync(
'add --transport sse sse-server https://example.com/sse-endpoint',

6
packages/cli/src/commands/mcp/add.ts
View file

@ -302,11 +302,7 @@ export const addCommand: CommandModule = {
// Auto-detect transport from URL if not explicitly specified
if (!argv['transport']) {
const commandOrUrl = argv['commandOrUrl'] as string;
if (
commandOrUrl &&
(commandOrUrl.startsWith('http://') ||
commandOrUrl.startsWith('https://'))
) {
if (commandOrUrl && /^https?:\/\//i.test(commandOrUrl)) {
argv['transport'] = 'http';
} else {
argv['transport'] = 'stdio';