Skip to content

Prepare direct clients for Summit descriptors#409

Merged
UtkarshBhardwaj007 merged 4 commits into
mainfrom
chore/summit-descriptor-support
Jun 17, 2026
Merged

Prepare direct clients for Summit descriptors#409
UtkarshBhardwaj007 merged 4 commits into
mainfrom
chore/summit-descriptor-support

Conversation

@charlesHetterich

Copy link
Copy Markdown
Collaborator

Summary

  • Select Summit product-sdk descriptors for direct PAPI clients when the active/requested env is summit.
  • Thread env-aware descriptor selection through registry, contract deploy, Bulletin auth checks, and Playground metadata upload.
  • Keep tests config-default agnostic so the Summit switch can remain a separate change.

Testing

  • pnpm format:check
  • pnpm lint:license
  • pnpm typecheck
  • pnpm test

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Dev build ready — try this branch:

curl -fsSL https://raw.githubusercontent.com/paritytech/playground-cli/main/install.sh | VERSION=dev/chore/summit-descriptor-support bash

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

E2E Test Pass · ❌ FAIL

Tag: e2e-ci-pr · Branch: chore/summit-descriptor-support · Commit: a5389e8 · Run logs

Cell Result Time
pr-deploy-cdm ✅ PASS 4m07s
pr-mod ❌ FAIL 2m40s
pr-deploy-foundry ✅ PASS 1m13s
pr-login-session ❌ FAIL 5m06s
pr-install ✅ PASS 1m06s
pr-preflight ❌ FAIL 2m36s
pr-deploy-frontend ❌ FAIL 4m47s
${{ matrix.cell }} ⏭️ SKIP 0m-1s
${{ matrix.cell }} ⏭️ SKIP 0m-9s
❌ Failed tests (2)
  • e2e/cli/deploy.test.ts › dot deploy — preflight and validation > reports mainnet not yet supported
    expected non-zero exit for --env mainnet, got 0 ╭────────────────────────────────────────────────────────────────────────────╮ │ Join the competition first │ │ Playground commands are for builders who've joined the competition, so you │ │ need to be signed in first. │ │ │ │ Run playground login and scan the QR code with your Polkadot mobile app, │ │ then become a builder and join the competition at playground.dot in your │ │ desktop app. │ ╰────────────────────────────────────────────────────────────────────────────╯ : expected +0 not to be +0 // Object.is equality
  • e2e/cli/deploy.test.ts › dot deploy — preflight and validation > domain availability check runs before build/upload
    availability banner not found: ╭────────────────────────────────────────────────────────────────────────────╮ │ Join the competition first │ │ Playground commands are for builders who've joined the competition, so you │ │ need to be signed in first. │ │ │ │ Run playground login and scan the QR code with your Polkadot mobile app, │ │ then become a builder and join the competition at playground.dot in your │ │ desktop app. │ ╰────────────────────────────────────────────────────────────────────────────╯ : expected -1 to be greater than -1

Sentry traces: view spans for this run

charlesHetterich and others added 3 commits June 17, 2026 18:54
The consumer tests (connection / registry / bulletinAuthContext) only
exercise whichever env is the active DEFAULT_ENV, so with paseo as the
default the summit arms of the descriptor selectors never ran. Add a
direct test for descriptors.ts that asserts both branches explicitly,
independent of ACTIVE_TESTNET_ENV, so summit selection is tested today
and the one-line network switch stays a no-surprise flip.
@UtkarshBhardwaj007 UtkarshBhardwaj007 merged commit faaffe4 into main Jun 17, 2026
15 of 20 checks passed
@UtkarshBhardwaj007 UtkarshBhardwaj007 deleted the chore/summit-descriptor-support branch June 17, 2026 17:43
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