feat(engine): add a stable, versioned getTrackRecordSummary read API#6316
Conversation
Wraps the existing computeTrackRecordSummary in a single documented, versioned envelope (TRACK_RECORD_SUMMARY_READ_VERSION + the computed summary) so a future consumer (JSONbored#6208's reputation bridge) can depend on one fixed read contract regardless of how TrackRecordSummary evolves. Read-only, adds no score/ranking fields — preserves the summary's existing public-safe boundary. Closes JSONbored#6246.
|
🚨 Contributor flagged. Click here for more info: Superagent Dashboard |
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning ⏸️ LoopOver review result - manual review recommendedReview updated: 2026-07-16 04:27:16 UTC
Review summary Nits — 4 non-blocking
Flagged checks (non-blocking)
Decision drivers
Context & advisory signals — never blocks the verdict
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://gittensory.aethereal.dev/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.
|
Summary
Groundwork for #6208 (reputation-bridge spec), part of the #6206 AMS+ORB synergy epic. Today AMS's calibration/track-record data is read ad-hoc with no single documented contract a future consumer could rely on. This adds one.
getTrackRecordSummary(input): TrackRecordSummaryReadResultin@loopover/engine— a thin, versioned wrapper over the existingcomputeTrackRecordSummary, returning{ version, summary }under a fixed envelope (TRACK_RECORD_SUMMARY_READ_VERSION) so a consumer can depend on one shape regardless of howTrackRecordSummaryevolves.Tests
packages/loopover-engine/test/track-record-summary.test.tsadds a populated case (asserts the envelope carries exactlycomputeTrackRecordSummary's output, with the right version and merge/close counts) and an empty/no-history case (default-disabled summary, zeroed counts). Verified passing locally vianode --test(30/30).Closes #6246.