Skip to content

resolve remark call IDs from metadata - #6

Merged
mcjkula merged 4 commits into
mainfrom
taolk-runtime-call-ids
Jun 7, 2026
Merged

resolve remark call IDs from metadata#6
mcjkula merged 4 commits into
mainfrom
taolk-runtime-call-ids

Conversation

@mcjkula

@mcjkula mcjkula commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

Resolve System remark call IDs from runtime metadata instead of hardcoding Bittensor call indices.

What changed

  • Runtime call IDs: Adds typed remark call IDs to chain state and resolves System.remark / System.remark_with_event from metadata.
  • Submission path: Uses the metadata-resolved remark_with_event call ID when signing outbound remarks.
  • Reader path: Uses the resolved call IDs when parsing incoming chain remarks.
  • Mirror path: Threads the same call ID state through mirror-backed reads.
  • Explicit failure: Fails with a clear runtime error when System.remark_with_event is unavailable.
  • Coverage: Adds runtime metadata fixtures and tests for non-default call IDs, missing calls, and refresh/submission paths.

Test plan

  • cargo test
  • GitHub Actions green: fmt, check, clippy, test, coverage
  • Codecov reports all modified coverable lines covered

@codecov-commenter

codecov-commenter commented May 20, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mcjkula
mcjkula merged commit 89971f7 into main Jun 7, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants