Skip to content

Add Qoder IDE as a supported MCP client - #5870

Open
haaaashimi wants to merge 3 commits into
stacklok:mainfrom
haaaashimi:add-qoder-client
Open

Add Qoder IDE as a supported MCP client#5870
haaaashimi wants to merge 3 commits into
stacklok:mainfrom
haaaashimi:add-qoder-client

Conversation

@haaaashimi

Copy link
Copy Markdown

Summary

  • Add Qoder IDE to the list of supported MCP clients, enabling automatic MCP server registration for Qoder users
  • Qoder uses platform-specific config paths: ~/.qoder/mcp.json on macOS/Linux, ~/AppData/Roaming/Qoder/SharedClientCache/mcp.json on Windows
  • Fix a pre-existing bug in IsClientInstalled where clients with empty RelPath but non-empty PlatformPrefix were incorrectly detected

Type of change

  • New feature

Test plan

  • Unit tests (task test)
  • Linting (task lint-fix)

Changes

File Change
pkg/client/config.go Add Qoder constant and supportedClientIntegrations entry with platform-specific paths
pkg/client/config_test.go Add Qoder mock config, update client count assertions from 24 to 25
pkg/client/discovery.go Fix IsClientInstalled to check both RelPath and PlatformPrefix before falling back to home-dir path
docs/cli/thv_client_register.md Regenerate CLI docs to include qoder
docs/cli/thv_client_remove.md Regenerate CLI docs to include qoder
docs/server/swagger.yaml Regenerate Swagger to include qoder in ClientApp enum
docs/server/swagger.json Regenerate Swagger to include qoder in ClientApp enum
docs/server/docs.go Regenerate Swagger docs

Does this introduce a user-facing change?

Yes — users can now run thv client register qoder to configure Qoder for MCP server integration.

Generated with Claude Code

@haaaashimi

Copy link
Copy Markdown
Author

mcp.json path is verified with all platforms.

Add Qoder to the list of supported MCP clients with platform-specific
config paths:
- macOS/Linux: ~/.qoder/mcp.json
- Windows: ~/AppData/Roaming/Qoder/SharedClientCache/mcp.json

Also fix IsClientInstalled to correctly handle clients with empty RelPath
but non-empty PlatformPrefix (e.g. Qoder on all platforms).

Signed-off-by: haashim-mac <haashimaag@gmail.com>
@haaaashimi

Copy link
Copy Markdown
Author

force pushed to add sign off

@github-actions github-actions Bot added the size/XS Extra small PR: < 100 lines changed label Jul 20, 2026
@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.28%. Comparing base (1c60f0f) to head (511c87b).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5870      +/-   ##
==========================================
- Coverage   71.32%   71.28%   -0.04%     
==========================================
  Files         693      693              
  Lines       70587    70587              
==========================================
- Hits        50347    50321      -26     
- Misses      16605    16628      +23     
- Partials     3635     3638       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

haaaashimi and others added 2 commits July 23, 2026 12:06
- Fix exported const comment: QoderIDE -> Qoder
- Add missing Qoder cases in exhaustive switches in config_test
- Add Qoder to the catch-all case list in switch statement

Signed-off-by: haashim-mac <haashimaag@gmail.com>
@haaaashimi

Copy link
Copy Markdown
Author

fixed linting issues.

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

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant