Skip to content

feat(mcp): register loopover_get_repo_onboarding_pack as a local #7976

Closed
andriypolanski wants to merge 3 commits into
JSONbored:mainfrom
andriypolanski:fix/7756
Closed

feat(mcp): register loopover_get_repo_onboarding_pack as a local #7976
andriypolanski wants to merge 3 commits into
JSONbored:mainfrom
andriypolanski:fix/7756

Conversation

@andriypolanski

Copy link
Copy Markdown
Contributor

Summary

  • loopover_get_repo_onboarding_pack was a remote MCP tool (src/mcp/server.ts) with a maintain onboarding-pack CLI mirror (CLI mirror for loopover_get_repo_onboarding_pack #6738), but had no local stdio MCP tool registration.
  • This registers the missing stdio tool next to its get-repo siblings, calling the same GET .../onboarding-pack/preview endpoint the CLI already uses (refresh: true?refresh=true).

Closes #7756

Change

  • Input schema (packages/loopover-mcp/bin/loopover-mcp.ts): added repoOnboardingPackShape (owner/repo required, refresh optional boolean).
  • Local stdio MCP tool: registered loopover_get_repo_onboarding_pack with a matching STDIO_TOOL_DESCRIPTORS entry (maintainer category). Thin GET proxy of {repoBase}/onboarding-pack/preview; refresh === true forwards ?refresh=true exactly as the CLI does.
  • Tool count pin: 96 → 97 in mcp-tool-rename-aliases.test.ts (current main already includes Register loopover_get_agent_audit_feed as a local stdio MCP tool #7757 agent-audit-feed).

Tests

  • New test/unit/mcp-cli-get-repo-onboarding-pack.test.ts: in-process InMemoryTransport coverage — both arms of the refresh === true ternary (omitted → no query; refresh: true?refresh=true). Fixture already served by mcp-cli-harness.

Validation

  • git diff --check clean; npm run build:mcp clean.
  • Targeted vitest suites pass (onboarding-pack in-process, tools list, rename-aliases).
  • Codecov patch = 100%. Verified from coverage-final.json: all added statements/functions hit; refresh ternary arms are [1,1].
  • Replaces closed conflicted feat(mcp): register loopover_get_repo_onboarding_pack as a local stdio tool #7968 (that PR targeted a stale ~94→95 pin; this branch is based on current main at 96 tools).

UI Evidence

N/A — local stdio MCP tool only; no UI surface changed.

@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@loopover-orb loopover-orb Bot closed this Jul 22, 2026
@loopover-orb

loopover-orb Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

This repository reviews pull requests one-shot: the PR must be correct as originally opened. Pushing an additional commit closes it automatically instead of restarting review — open a fresh pull request with every fix included.

@loopover-orb loopover-orb Bot added the review-evasion Gittensor contributor context label Jul 22, 2026
@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.92%. Comparing base (4dccaa6) to head (69ebfd3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7976      +/-   ##
==========================================
- Coverage   89.11%   81.92%   -7.19%     
==========================================
  Files          93       94       +1     
  Lines       21723    23950    +2227     
  Branches     3755     4603     +848     
==========================================
+ Hits        19358    19621     +263     
- Misses       2187     4133    +1946     
- Partials      178      196      +18     
Flag Coverage Δ
shard-1 0.00% <0.00%> (?)
shard-2 11.80% <100.00%> (?)
shard-3 0.00% <0.00%> (?)

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 11.80% <100.00%> (ø)

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

Labels

review-evasion Gittensor contributor context

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Register loopover_get_repo_onboarding_pack as a local stdio MCP tool

2 participants