Skip to content

feat(mcp): add remote + stdio surfaces for loopover_get_gate_config_effective#7929

Closed
kai392 wants to merge 1 commit into
JSONbored:mainfrom
kai392:feat/7800-mcp-gate-config-effective-v3
Closed

feat(mcp): add remote + stdio surfaces for loopover_get_gate_config_effective#7929
kai392 wants to merge 1 commit into
JSONbored:mainfrom
kai392:feat/7800-mcp-gate-config-effective-v3

Conversation

@kai392

@kai392 kai392 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Scope

  • The PR title follows type(scope): short summary Conventional Commit format.
  • This PR is focused and does not mix unrelated backend, UI, MCP, docs, dependency, and deploy changes.
  • This follows CONTRIBUTING.md and does not reintroduce GitHub Pages, VitePress, site/, or CNAME.
  • I linked a currently open issue this PR resolves (Closes #7800).

Validation

  • git diff --check
  • npm run branding-drift:check
  • npm run typecheck
  • npm run build:mcp
  • vitest --coverage on mcp-output-schemas + mcp-gate-config-effective + mcp-cli-gate-config-effective + mcp-tool-rename-aliases (63/63)
  • Local patch coverage on changed src/mcp/server.ts / bin executable lines: 100% (13/13)
  • Full unsharded npm run test:coverage deferred to CI
  • New or changed behavior has unit/integration tests for new branches

If any required check was skipped, explain why:

Safety

  • No secrets/wallet/hotkey/coldkey/PAT exposure.
  • Auth negative-path covered (forbidden outside allowlist).
  • API/OpenAPI/MCP behavior updated and tested where needed.
  • No visible UI changes (UI Evidence N/A).

UI Evidence

N/A — MCP/API only.

Notes

Made with Cursor

@kai392
kai392 requested a review from JSONbored as a code owner July 21, 2026 18:50
@kai392

kai392 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

Closing as duplicate of earlier open #7928 for the same issue (#7800). Local validation on this branch passed; prefer the first open PR to avoid competing heads.

@kai392 kai392 closed this Jul 21, 2026
…ffective

Closes JSONbored#7800: mirror REST gate-config/effective on remote MCP and local stdio with canAccessRepo/allowlist auth, declare outputSchema (JSONbored#550), and pin the stdio tool count 84→85.

Co-authored-by: Cursor <cursoragent@cursor.com>
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent did not find any vulnerabilities or security issues in this PR.

@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.50%. Comparing base (1257c20) to head (97e2b84).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7929      +/-   ##
==========================================
- Coverage   91.77%   82.50%   -9.27%     
==========================================
  Files         731       90     -641     
  Lines       74913    24437   -50476     
  Branches    22850     4701   -18149     
==========================================
- Hits        68752    20162   -48590     
+ Misses       5083     4039    -1044     
+ Partials     1078      236     -842     
Flag Coverage Δ
shard-1 22.77% <29.41%> (-37.17%) ⬇️
shard-2 22.18% <17.64%> (-24.31%) ⬇️
shard-3 33.04% <88.23%> (-21.74%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
packages/loopover-mcp/bin/loopover-mcp.ts 13.95% <100.00%> (+0.11%) ⬆️
src/mcp/server.ts 96.64% <100.00%> (+0.04%) ⬆️

... and 641 files with indirect coverage changes

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.

loopover_get_gate_config_effective has a REST route but no remote MCP tool or local stdio MCP tool

2 participants