Skip to content

Wire a real AMS-side consumer of ORB's hosted GET /v1/contributors/:login/pr-outcomes #7658

Description

@JSONbored

Problem

src/signals/contributor-pr-outcomes.ts backs GET /v1/contributors/:login/pr-outcomes — a genuinely hosted, multi-tenant-safe, per-GitHub-login data source (public-safe attribution only, no reward/wallet fields) that is exactly the server-side ground truth a hosted miner dashboard's "past attempts (merged)" view needs, since it doesn't depend on any per-tenant local sqlite store. Nothing on the AMS side calls it today.

Area

packages/loopover-miner/.

Proposal

Deliverable is a CLI command specifically — not MCP, not miner-ui (both are separate, already-tracked surfaces; see #7656 and #7675 for the miner-ui/dashboard-side work). Add a real CLI command that calls GET /v1/contributors/:login/pr-outcomes for the current miner's own configured GitHub login and prints the result, mirroring this package's existing CLI command patterns (check packages/loopover-miner/lib/*-cli.js/*-cli.ts for the house shape before starting — flag parsing, output formatting, error handling should all match an existing command, not invent a new CLI convention).

Deliverables

  • A CLI command calling this endpoint for the current miner's login, following an existing *-cli.ts command's exact structure.
  • Tests mocking the HTTP call — no live network calls in the test suite.

Test Coverage Requirements

packages/loopover-miner/** — match this package's existing CLI test conventions (see any existing *-cli.test.ts for the pattern).

Resources

  • src/signals/contributor-pr-outcomes.ts, src/api/routes.ts (the existing endpoint)
  • packages/loopover-miner/lib/ — pick an existing *-cli.ts file as the structural template

Boundaries

Merged-PR outcomes only, per the endpoint's current scope (no closed/rejected outcomes, no in-flight attempts) — those are separate, local-only data (see #7656) unless a future issue extends this endpoint itself. CLI only — MCP and dashboard exposure are separate, already-filed issues, not this one's job.

Contributor-eligible. CLI command only, per the Proposal above — a PR that builds an MCP tool or a miner-ui route instead does not satisfy this issue.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.help wantedExtra attention is neededroadmapOn the Wave-2 agent-layer roadmap board (project 9)

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions