Skip to content

feat(indexer): add provider verification schema - #3716

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

feat(indexer): add provider verification schema#3716
chalabi2 wants to merge 4 commits into
akash-network:mainfrom
chalabi2:aep-86/console-verification-schema

Conversation

@chalabi2

Copy link
Copy Markdown

Why

Part of CON-800.

Console needs durable, queryable AEP-86 state before the indexer and API can
consume verification events.

Depends on #3713.

What

  • add shared database definitions for attestations, auditors, bonds, snapshots,
    escrows, discrepancies, grace state, maintenance, and reconciliation targets
  • register the new Akash chain definitions
  • add the corresponding additive indexer migration
  • keep generated Drizzle SQL, schema, and relations in a separate commit

@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.

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

@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.74%. Comparing base (37e3cc2) to head (7d207ec).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3716      +/-   ##
==========================================
- Coverage   79.76%   79.74%   -0.02%     
==========================================
  Files        1186     1186              
  Lines       31867    31870       +3     
  Branches     7848     7849       +1     
==========================================
- Hits        25418    25416       -2     
- Misses       5688     5692       +4     
- Partials      761      762       +1     
Flag Coverage Δ
api 90.31% <ø> (-0.06%) ⬇️
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 2 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.

@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

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
@chalabi2
chalabi2 force-pushed the aep-86/console-verification-schema branch from e849ce0 to d839660 Compare August 27, 2026 16:48
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>
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