Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Nov 21, 2025

🚀 Weekly Release to Staging

Release Date: November 21, 2025
Release Branch: release/staging-2025-11-21

This automated PR promotes a snapshot of dev to staging for testing.

What's Included

All commits merged to dev up to the branch creation time.

Note: This PR uses a dedicated release branch, so new commits to dev will NOT automatically appear here.

Review Checklist

  • All CI checks pass
  • Code review completed
  • QA team notified
  • Ready to merge to staging environment

Next Steps

After merging, the staging environment will be updated. A production release PR will be created on Sunday.


This PR was automatically created by the Release Calendar workflow on November 21, 2025


Note

Derives private key directly from mnemonic and updates recovery flows to load/parse passport data separately, adjust CSCA handling, and improve analytics.

  • Account Recovery:
    • Replace loadPassportDataAndSecret with loadPassportData and derive secret via getPrivateKeyFromMnemonic(...).
    • Parse passportData JSON before use and pass parsed data to isUserRegisteredWithAlternativeCSCA(...) and reStorePassportDataWithRightCSCA(...).
    • Improve analytics: add detailed failure reasons (e.g., no_passport_data_or_secret, document_not_registered) and include hasCSCA flag.
  • Auth Provider:
    • Add getPrivateKeyFromMnemonic(mnemonic) utility using ethers.HDNodeWallet.fromPhrase.
    • Update unsafe_getPrivateKey to use getPrivateKeyFromMnemonic(...).

Written by Cursor Bugbot for commit fb4cd92. This will update automatically on new commits. Configure here.

* Recovery: reduce fingerprint popup count

* add await to reStorePassportDataWithRightCSCA
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 21, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 17

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

const passportDataAndSecret = await loadPassportDataAndSecret();
if (!passportDataAndSecret) {
const passportData = await loadPassportData();
const secret = getPrivateKeyFromMnemonic(slimMnemonic);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Recovery derivation uses parameter instead of stored mnemonic

The new code derives the private secret directly from the user-provided mnemonic parameter instead of loading the stored mnemonic from keychain like the old code did. If any discrepancy exists between the parameter and what was actually stored by restoreAccountFromMnemonic (due to encoding, normalization, or storage issues), the derived secret will be incorrect and cause authentication to fail. The old approach validated consistency by loading from the authoritative stored source.

Fix in Cursor Fix in Web

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the goal here is to reduce the amount of mnemonic verifications that the user needs to perform

#1434

@transphorm
Copy link
Member

this pull request is stale. closing in favor of the newer pr to update staging

#1451

@transphorm transphorm closed this Dec 1, 2025
@transphorm transphorm deleted the release/staging-2025-11-21 branch December 6, 2025 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants