Skip to content

feat(provider-inventory): screen bids by verification - #3718

Draft
chalabi2 wants to merge 5 commits into
akash-network:mainfrom
chalabi2:aep-86/console-verification-bid-screening
Draft

chalabi2 wants to merge 5 commits into
akash-network:mainfrom
chalabi2:aep-86/console-verification-bid-screening

Conversation

@chalabi2

Copy link
Copy Markdown

Why

Marketplace preflight must enforce AEP-86 placement requirements before Console presents providers as eligible. This is the provider-inventory application slice of CON-800.

What

  • Preserve SDL verification requirements in bid-screening requests
  • Poll and persist normalized verification facts with provider inventory
  • Hard-filter providers that fail tier, capability, auditor-count, or named-auditor policy
  • Keep incomplete chain facts visible as not_evaluated instead of treating them as passed
  • Return both the first actionable failure and the complete exclusion set
  • Add the provider-inventory verification column and generated migration

Dependencies

The dependency commits are included temporarily so this branch can build and will disappear from the diff as the foundation PRs merge.

@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
+ node_modules/@akashnetwork/provider-verification                                         link:packages/provider-verification
+ packages/provider-verification                                                           0.0.0   

@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
Added@​akashnetwork/​provider-verification@​0.0.0N/AN/AN/AN/AN/A

View full report

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.44961% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.86%. Comparing base (37e3cc2) to head (f7b6ce8).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...trollers/bid-screening/bid-screening.controller.ts 0.00% 1 Missing ⚠️
...n-provider-poller/chain-provider-poller.service.ts 97.14% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3718      +/-   ##
==========================================
+ Coverage   79.76%   79.86%   +0.10%     
==========================================
  Files        1186     1188       +2     
  Lines       31867    31973     +106     
  Branches     7848     7868      +20     
==========================================
+ Hits        25418    25535     +117     
+ Misses       5688     5682       -6     
+ Partials      761      756       -5     
Flag Coverage Δ
api 90.33% <ø> (-0.04%) ⬇️
deploy-web 71.02% <ø> (ø)
log-collector 85.85% <ø> (ø)
notifications 94.33% <100.00%> (+0.01%) ⬆️
provider-console 81.38% <ø> (ø)
provider-inventory 87.06% <98.37%> (+2.08%) ⬆️
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%> (ø)
apps/provider-inventory/src/config/env.config.ts 100.00% <ø> (ø)
...inventory/src/http-schemas/bid-screening.schema.ts 60.86% <100.00%> (+60.86%) ⬆️
...y/src/mappers/groupspec-mapper/groupspec-mapper.ts 100.00% <100.00%> (ø)
...erification-mapper/provider-verification-mapper.ts 100.00% <100.00%> (ø)
...as/provider-inventory/provider-inventory.schema.ts 100.00% <ø> (ø)
...ers/provider-verification-query-client.provider.ts 100.00% <100.00%> (ø)
...sitories/bid-screening/bid-screening.repository.ts 100.00% <100.00%> (ø)
...rovider-inventory/provider-inventory.repository.ts 100.00% <100.00%> (ø)
...rc/services/bid-screening/bid-screening.service.ts 100.00% <100.00%> (ø)
... and 3 more

... and 4 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.

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
@chalabi2
chalabi2 force-pushed the aep-86/console-verification-bid-screening branch from 9cfb5e2 to f027a96 Compare August 27, 2026 16:49
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>
Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
@chalabi2
chalabi2 force-pushed the aep-86/console-verification-bid-screening branch from f027a96 to f7b6ce8 Compare August 27, 2026 17:03
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