Skip to content

feat(deploy-web): add provider verification workflows - #3722

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

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

Conversation

@chalabi2

Copy link
Copy Markdown

Why

Part of CON-800.

Console users need to define AEP-86 placement requirements, understand why a
provider is excluded, and inspect provider verification facts without querying
the chain directly. This keeps the complete user workflow in one application
PR even though the resulting change is necessarily XL.

Feature surface

AEP-86 provider verification feature surface

What

  • add flag-gated verification controls to SDL authoring and preserve import and
    generation round trips
  • preflight placement requirements and explain when no provider can satisfy them
  • hard-filter marketplace results and display verification-based exclusions
  • show effective tier, auditors, capabilities, snapshots, maintenance, escrows,
    and discrepancies on provider views
  • show the placement policy beside current provider facts on active deployments
  • preserve legacy signedBy requirements when both policies are present

Dependencies

The implementation remains behind the provider_verification feature flag.

@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

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.90076% with 112 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.82%. Comparing base (37e3cc2) to head (356ba38).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...tplaceProvidersTable/MarketplaceProvidersTable.tsx 60.00% 25 Missing and 1 partial ⚠️
...mponents/providers/ProviderVerificationDetails.tsx 84.05% 22 Missing ⚠️
...ketplaceProvidersTable/MarketplaceProviderCell.tsx 51.42% 17 Missing ⚠️
...pps/deploy-web/src/queries/useScreenedProviders.ts 74.00% 13 Missing ⚠️
...ploy-web/src/components/sdl/PlacementFormModal.tsx 0.00% 7 Missing and 1 partial ⚠️
...ponents/providers/ProviderVerificationListCell.tsx 84.37% 5 Missing ⚠️
...loy-web/src/components/providers/ProviderTable.tsx 0.00% 4 Missing ⚠️
...eDeploymentHeader/verificationProviderPreflight.ts 81.25% 3 Missing ⚠️
...eploymentPlacements/PlacementVerificationPanel.tsx 96.55% 3 Missing ⚠️
...oy-web/src/components/providers/ProviderDetail.tsx 0.00% 3 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3722      +/-   ##
==========================================
+ Coverage   79.76%   79.82%   +0.06%     
==========================================
  Files        1186     1194       +8     
  Lines       31867    32494     +627     
  Branches     7848     8149     +301     
==========================================
+ Hits        25418    25938     +520     
- Misses       5688     5793     +105     
- Partials      761      763       +2     
Flag Coverage Δ
api 90.33% <ø> (-0.04%) ⬇️
deploy-web 71.45% <82.74%> (+0.43%) ⬆️
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 Δ
...gureDeploymentHeader/ConfigureDeploymentHeader.tsx 100.00% <100.00%> (ø)
...ent/DeploymentPane/PlacementCard/PlacementCard.tsx 96.77% <100.00%> (+0.22%) ⬆️
...Pane/PlacementCard/PlacementVerificationEditor.tsx 100.00% <100.00%> (ø)
...gureDeployment/MarketplacePane/MarketplacePane.tsx 100.00% <100.00%> (ø)
...ymentDetail/DeploymentPlacements/PlacementCard.tsx 88.33% <100.00%> (+1.29%) ⬆️
...DeploymentPlacements/placementVerificationModel.ts 100.00% <100.00%> (ø)
...eb/src/components/providers/providerListFilters.ts 100.00% <100.00%> (ø)
...omponents/sdl/PlacementVerificationFormControl.tsx 100.00% <100.00%> (ø)
apps/deploy-web/src/queries/usePlacementOffers.ts 100.00% <100.00%> (ø)
apps/deploy-web/src/types/sdlBuilder/sdlBuilder.ts 83.42% <100.00%> (+0.47%) ⬆️
... and 17 more

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

Signed-off-by: Joseph Chalabi <chalabi.joseph@gmail.com>
@chalabi2
chalabi2 force-pushed the aep-86/console-verification-deploy-web branch from 06f2dd5 to 0d63201 Compare August 27, 2026 16:50
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>
@chalabi2
chalabi2 force-pushed the aep-86/console-verification-deploy-web branch from 0d63201 to 356ba38 Compare August 27, 2026 17:04
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