Skip to content

feat(stats): render provider verification records - #3725

Draft
chalabi2 wants to merge 3 commits into
akash-network:mainfrom
chalabi2:aep-86/console-verification-stats
Draft

chalabi2 wants to merge 3 commits into
akash-network:mainfrom
chalabi2:aep-86/console-verification-stats

Conversation

@chalabi2

Copy link
Copy Markdown

Why

Part of CON-800.

AEP-86 transactions and typed events should be readable in Stats without asking
reviewers or operators to decode raw protobuf JSON.

What

  • recognize all 22 verification and maintenance transaction messages
  • recognize all 33 verification and maintenance typed events
  • render addresses, tiers, capabilities, coins, hashes, and nested records
  • format both message and event names consistently
  • preserve the existing generic JSON fallback for unrelated transaction types

Dependencies

The registry was checked against the current verification and provider proto
surfaces; response messages and unrelated provider CRUD are intentionally absent.

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@socket-security

socket-security Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​akashnetwork/​chain-sdk@​1.0.0-alpha.41 ⏵ 1.0.0-alpha.4385 +1100100 +198100

View full report

Comment thread package-lock.json
@@ -42,7 +42,7 @@
"license": "Apache-2.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔄 Carefully review the package-lock.json diff

Resolve the comment if everything is ok

* node_modules/@akashnetwork/chain-sdk                                                     1.0.0-alpha.41 -> 1.0.0-alpha.43

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.76%. Comparing base (37e3cc2) to head (7fa5b29).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3725   +/-   ##
=======================================
  Coverage   79.76%   79.76%           
=======================================
  Files        1186     1186           
  Lines       31867    31870    +3     
  Branches     7848     7849    +1     
=======================================
+ Hits        25418    25421    +3     
- Misses       5688     5694    +6     
+ Partials      761      755    -6     
Flag Coverage Δ
api 90.36% <ø> (ø)
deploy-web 71.02% <ø> (ø)
log-collector 85.85% <ø> (ø)
notifications 94.33% <100.00%> (+0.01%) ⬆️
provider-console 81.38% <ø> (ø)
provider-inventory 84.98% <ø> (ø)
provider-proxy 88.22% <ø> (ø)
tx-signer 90.19% <ø> (ø)
Files with missing lines Coverage Δ
...s/src/modules/chain/providers/registry.provider.ts 100.00% <100.00%> (ø)
...y/src/mappers/groupspec-mapper/groupspec-mapper.ts 100.00% <ø> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@chalabi2
chalabi2 force-pushed the aep-86/console-verification-stats branch from 8440511 to 491e2bd Compare August 27, 2026 16:52
Keep the optional placement verification field optional at the JSON boundary and avoid expanding the generated type union while building the notifications registry.

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant