Skip to content

Scope the Stellar skill to the browser; mark React Native WIP - #54

Merged
adrianvrj merged 1 commit into
mainfrom
skill/stellar-browser-scope
Nov 12, 2026
Merged

adrianvrj merged 1 commit into
mainfrom
skill/stellar-browser-scope

Conversation

@adrianvrj

Copy link
Copy Markdown
Contributor

Raised on stellar/stellar-dev-skill#185.

  • Cavos.connect passes deviceKey to CavosStellar but no factor, and CavosStellar does not use deviceKey. The React Native provider connects without vault, socialRecovery or passkey. So on React Native and Node, Stellar is not on the MasterDEK path.
  • The skill now covers the browser only and says so. The React Native keystore and Node lines are gone.
  • The vault is described as the default in the React provider with appId, opt-in with Cavos.connect.

On React Native and Node the Stellar adapter does not use the device
unwrap key, and the React Native provider connects without vault or
recovery, so Stellar there is not on the MasterDEK path. Drop the React
Native keystore and Node claims, and say the vault is the default only in
the React provider with an appId.
@adrianvrj
adrianvrj merged commit 6cdc91f into main Nov 12, 2026
2 checks passed
@adrianvrj
adrianvrj deleted the skill/stellar-browser-scope branch November 12, 2026 16:08
@adrianvrj adrianvrj changed the title Scope the Stellar skill to the browser Scope the Stellar skill to the browser; mark React Native WIP Nov 12, 2026
@adrianvrj

Copy link
Copy Markdown
Contributor Author

Also marks React Native as a work in progress in the README (intro and the React Native section). The npm description in package.json still says React Native; it changes on the next release.

@adrianvrj

Copy link
Copy Markdown
Contributor Author

Update: the npm description now says React Native is a work in progress too. It reaches npm with the next version bump.

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.

1 participant