fix(cli): register works with the implicit dev account and survives store-sync failures - #279
Merged
Merged
Conversation
…tore-sync failures Two register-flow fixes found during the previewnet end-to-end test: - The default auth branch was the only one returning no credential, so `register` with the implicit shared dev account aborted on the retry-cache check after resolving a usable signer. It now carries the (public) dev phrase as its credential like every other branch. Fixes paritytech/dotns#266 - A post-registration label-store sync failure escaped from the initial reads and aborted the whole command with the unmapped-origin hint, even though the name was registered, verified, and the account mapped. The sync is now best-effort: failures and unsynced results surface as warnings carrying the real error. Fixes paritytech/dotns#267 Bumps the CLI to 0.8.5 for the release guard.
CI Summary
Release - PassedTest this PR Download artifact (GitHub CLI required): gh run download 33736673714 -n cli-release-0.0.0-pr.279 -R paritytech/dotns-sdkInstall globally: npm install -g ./parity-dotns-cli-0.0.0-pr.279.tgzVerify: dotns --helpDeploy Example — Passed
Labelspkg: cli, type: test, dependencies Test - Passed305 passed, 0 failed across 305 tests. |
filip-parity
requested review from
corey-hathaway,
filvecchiato and
sphamjoli
September 2, 2026 15:15
bun install had injected the papi descriptors file: dependency into packages/cli/package.json and it rode into the previous commit; the committed lockfile does not record it, so CI's frozen install failed. Restores both files to main plus the 0.8.5 version bump only.
re-gius
reviewed
Sep 2, 2026
- drop issue references from code comments - isolate the default-credential test behind DOTNS_KEYSTORE_PATH - carry the last claim error on LabelStoreSyncResult and include it in the store-sync warning - only print the step success line when the store actually synced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Two register bugs found during the previewnet e2e test:
registerwith no account configured aborted with "Could not resolve a credential for the registration retry cache". The default auth branch was the only one without acredential; it now uses the public dev phrase like the other branches.Fixes
Fixes paritytech/dotns#266
Fixes paritytech/dotns#267
Testing
bun test packages/cli/tests/unit/authcovers the credential fallback; full suite passes. Verified on previewnet: register with the implicit account now gets past the credential check.Version bumped to 0.8.5 for the release guard.