fix: send Insights docs link to issue-discovery doc in Discovery mode…

[edwin-rivera-dev]

Summary

Fixes #152. On a miner's detail page, the "Learn more about scoring in the docs" link in the Insights & Next Actions card always pointed to docs.gittensor.io/oss-contributions.html, even when the user had toggled the page to Issue Discovery mode. The link now matches the active view.

Root cause

MinerDetailsPage already derives viewMode: 'prs' | 'issues' from the URL toggle at the top of the page and passes it to MinerScoreCard. It did not pass the same prop to MinerInsightsCard, so the insights card had no way to know which docs page to link to and used the OSS one as a hardcoded fallback.

Fix

• MinerInsightsCard accepts a new optional viewMode?: 'prs' | 'issues' prop (default 'prs' for backward compatibility).
• A single docsUrl constant inside the component picks between issue-discovery.html and oss-contributions.html based on viewMode.
• MinerDetailsPage forwards its existing viewMode to MinerInsightsCard, matching the pattern it already uses for MinerScoreCard.

No other callers exist (verified via grep).

Type of Change

• Bug fix

Testing

• npm run build
• npm run lint:fix
• Manual (per issue repro):
  1. Open /miners/details?githubId=152555741 - toggle is on OSS Contributions --> "docs" link opens docs.gittensor.io/oss-contributions.html ✅ (unchanged)
  2. Click Issue Discovery toggle in the top-right --> URL gains &viewMode=issues, card re-renders --> "docs" link now opens docs.gittensor.io/issue-discovery.html ✅
  3. Refresh on the Issue Discovery URL --> link still points to issue-discovery.html

Checklist

• Backwards-compatible (prop is optional, defaults to 'prs')
• No new dependencies
• Two files touched
• Follows the pattern already used by MinerScoreCard for the same viewMode prop

Media to Upload

14.04.2026_22.08.25_REC_152.mp4

[anderdc]

Closing — CI is failing (build + format) and the diff includes unrelated changes (removing the viewMode guard on MinerScoreBreakdown). The bug is valid but the fix is a one-liner — no need for the restructuring that broke the build.

[anderdc]

The bug is real but the fix is a one-liner — the component already has isIssueMode, just use it in the href. Drop the assembleIssueInsights restructuring (it's what broke the build) and the unrelated MinerScoreBreakdown guard removal.

[anderdc]

Docs link fix looks good now. Still has the unrelated MinerScoreBreakdown guard removal in MinerDetailsPage.tsx — drop that change, it's not part of this fix.

[edwin-rivera-dev]

Hi, @anderdc Thanks for your feedback. I have just updated. Please review again and let me know. 👍

[anderdc]

MinerScoreBreakdown revert done. Clean one-file fix. Looks good.

Bug is already fixed on test — viewMode is already passed and the conditional href already works.

[anderdc]

Closing — the bug from #152 is already fixed on test. MinerDetailsPage already passes viewMode to MinerInsightsCard, and the conditional href already picks the right docs URL. This PR just extracts the inline conditional to a variable, which isn't worth a standalone PR.

[anderdc]

Reopening — you addressed all the feedback. Sorry for the back and forth.

[anderdc]

All feedback addressed. Clean fix.