feat(mcp): register loopover_get_agent_audit_feed as a local stdio tool - #7967
feat(mcp): register loopover_get_agent_audit_feed as a local stdio tool#7967real-venus wants to merge 1 commit into
Conversation
loopover_get_agent_audit_feed has a remote MCP tool (src/mcp/server.ts) and a `maintain audit-feed` CLI command, but no local stdio MCP tool registration. JSONbored#6733 added the REST route + CLI but never the matching stdio tool, so a self-host operator using the local MCP server couldn't call it. Adds the registerStdioTool block following the existing sibling pattern -- a thin GET proxy of the same {repoBase}/agent/audit-feed route the CLI hits, forwarding the optional since/limit query filters verbatim (the route validates + defaults). Input mirrors the remote auditFeedShape (owner/repo/since/limit); description via stdioToolDescription; category "agent". test/unit/mcp-cli-agent-audit-feed.test.ts drives it in-process (JSONbored#7764 entrypoint guard) so the registration + handler get real Codecov coverage, covering both the with-filters (since+limit) and no-filters (empty query) branches. Count 94 -> 95. Closes JSONbored#7757
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning ⏸️ LoopOver review result - manual review recommendedReview updated: 2026-07-22 00:41:00 UTC
Review summary Nits — 6 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
|
@JSONbored |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7967 +/- ##
===========================================
- Coverage 91.87% 81.78% -10.09%
===========================================
Files 735 91 -644
Lines 75477 23720 -51757
Branches 22960 4554 -18406
===========================================
- Hits 69344 19400 -49944
+ Misses 5041 4124 -917
+ Partials 1092 196 -896
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
LoopOver is closing this pull request on the maintainer's behalf (conflicts with the base branch — resolve and open a fresh PR). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed. |
Summary
Closes #7757 —
loopover_get_agent_audit_feedhas a remote MCP tool (src/mcp/server.ts) and amaintain audit-feedCLI command, but no local stdio MCP tool registration. #6733 added the REST route + CLI but never the matching stdio tool, so a self-host operator using the local MCP server (not the CLI or remote MCP) couldn't call it.What changed (
packages/loopover-mcp/bin/loopover-mcp.ts)registerStdioTool("loopover_get_agent_audit_feed", …)block, placed next toloopover_get_maintainer_noise, mirroring the sibling proxy pattern exactly — a thin GET of the same{repoBase}/agent/audit-feedroute themaintain audit-feedCLI already calls, forwarding the optionalsince/limitquery filters verbatim (the route validates them and applies defaults).auditFeedShapemirrors the remoteauditFeedShape(owner/repo/since/limit). Description centralized viastdioToolDescription(...)+ aSTDIO_TOOL_DESCRIPTORSentry (category: "agent", matching the remote tool). No hardcoded inline description.Testing / coverage
test/unit/mcp-cli-agent-audit-feed.test.tsdrives the tool in-process (the loopover_plan_repo_issues has no REST route, CLI command, or local stdio MCP tool #7764isProcessEntrypointguard +InMemoryTransport, mirroringmcp-cli-repo-focus-manifest.test.ts) so the registration + handler get real Codecov-measured coverage — a subprocess spawn can't be v8-instrumented. Covers both the with-filters path (since+limitforwarded, verified against the harness's echoed query) and the no-filters path (empty query string — thequery.size > 0false branch).test/unit/mcp-tool-rename-aliases.test.tsbumped to 95.No REST/OpenAPI/CLI-surface change (the route and
maintain audit-feedCLI already exist) —ui:openapi:check,command-reference:check,docs/manifestdrift all clean;build:mcpclean.