feat: support keplr wallet

.changeset/chilled-spies-bathe.md

@@ -0,0 +1,5 @@
+---
+"@starknet-io/get-starknet-core": patch
+---
+
+Add keplr wallet

README.md

@@ -10,7 +10,6 @@
 - ⚙️ Customizable and extensible
 - 🌍 Open source and controlled by the community
 
-
 ## Installation
 
 ```
@@ -24,7 +23,8 @@ yarn add @starknet-io/get-starknet starknet@next
 pnpm add @starknet-io/get-starknet starknet@next
 ```
 
-Read more about the new Starkent Dapp<>Wallet API in the [post](https://community.starknet.io/t/new-starknet-wallet-dapp-api/114295)
+Read more about the new Starkent Dapp<>Wallet API in the
+[post](https://community.starknet.io/t/new-starknet-wallet-dapp-api/114295)
 
 ## Usage for dApp developers
 

packages/core/src/__test__/main.test.ts

@@ -3,6 +3,8 @@ import { mockStorageFunction } from "./storage.mock"
 import {
  ArgentXMock,
  BraavosMock,
+ KeplrMock,
+ OKXMock,
  UnknownWalletAMock,
  UnknownWalletBMock,
  makeAuthorized,
@@ -31,11 +33,15 @@ describe("getAvailableWallets()", () => {
  },
  starknet: ArgentXMock,
  "starknet-braavos": BraavosMock,
+ starknet_okxwallet: OKXMock,
+ starknet_keplr: KeplrMock,
  })
  const availableWallets = await sn.getAvailableWallets()
- expect(availableWallets.length).toBe(2)
+ expect(availableWallets.length).toBe(4)
  expect(availableWallets).toContainEqual(ArgentXMock)
  expect(availableWallets).toContainEqual(BraavosMock)
+ expect(availableWallets).toContainEqual(OKXMock)
+ expect(availableWallets).toContainEqual(KeplrMock)
  })
  it("should return one injected wallet", async () => {
  const sn = getWallet({
@@ -110,16 +116,22 @@ describe("getAuthorizedWallets()", () => {
  const sn = getWallet({
  "starknet-argent": makeAuthorized(true)(ArgentXMock),
  "starknet-braavos": makeAuthorized(true)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(true)(OKXMock),
+ starknet_keplr: makeAuthorized(true)(KeplrMock),
  })
- const authorizedWallets = await sn.getAuthorizedWallets()
- expect(authorizedWallets.length).toBe(2)
- expect(authorizedWallets.map((w) => w.id)).contains(ArgentXMock.id)
- expect(authorizedWallets.map((w) => w.id)).contains(BraavosMock.id)
+ const preauthorizedWallets = await sn.getAuthorizedWallets()
+ expect(preauthorizedWallets.length).toBe(4)
+ expect(preauthorizedWallets.map((w) => w.id)).contains(ArgentXMock.id)
+ expect(preauthorizedWallets.map((w) => w.id)).contains(BraavosMock.id)
+ expect(preauthorizedWallets.map((w) => w.id)).contains(OKXMock.id)
+ expect(preauthorizedWallets.map((w) => w.id)).contains(KeplrMock.id)
  })
  it("should return one authorized wallet", async () => {
  const sn = getWallet({
  "starknet-argent": makeAuthorized(true)(ArgentXMock),
  "starknet-braavos": makeAuthorized(false)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(false)(OKXMock),
+ starknet_keplr: makeAuthorized(false)(KeplrMock),
  })
  const authorizedWallets = await sn.getAuthorizedWallets()
  expect(authorizedWallets.length).toBe(1)
@@ -131,8 +143,10 @@ describe("getDiscoveryWallets()", () => {
  it("should return all discovery wallets", async () => {
  const sn = getWallet({})
  const discoveryWallets = await sn.getDiscoveryWallets()
- expect(discoveryWallets.length).toBe(3)
+ expect(discoveryWallets.length).toBe(5)
  expect(discoveryWallets.map((w) => w.id)).contains(ArgentXMock.id)
  expect(discoveryWallets.map((w) => w.id)).contains(BraavosMock.id)
+ expect(discoveryWallets.map((w) => w.id)).contains(OKXMock.id)
+ expect(discoveryWallets.map((w) => w.id)).contains(KeplrMock.id)
  })
 })

packages/core/src/__test__/storage.test.ts

@@ -3,6 +3,8 @@ import { mockStorageFunction } from "./storage.mock"
 import {
  ArgentXMock,
  BraavosMock,
+ KeplrMock,
+ OKXMock,
  UnknownWalletAMock,
  UnknownWalletBMock,
  makeAuthorized,
@@ -48,6 +50,8 @@ describe("getLastConnectedWallet()", () => {
  {
  "starknet-argentX": makeAuthorized(false)(ArgentXMock),
  "starknet-braavos": makeAuthorized(false)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(false)(OKXMock),
+ starknet_keplr: makeAuthorized(false)(KeplrMock),
  },
  mockStorageFunction({
  "gsw-last": "braavos",
@@ -61,6 +65,8 @@ describe("getLastConnectedWallet()", () => {
  {
  "starknet-argentX": makeAuthorized(false)(ArgentXMock),
  "starknet-braavos": makeAuthorized(true)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(false)(OKXMock),
+ starknet_keplr: makeAuthorized(false)(KeplrMock),
  },
  mockStorageFunction({
  "gsw-last": "braavos",
@@ -74,6 +80,8 @@ describe("getLastConnectedWallet()", () => {
  {
  "starknet-argentX": makeAuthorized(true)(ArgentXMock),
  "starknet-braavos": makeAuthorized(true)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(true)(OKXMock),
+ starknet_keplr: makeAuthorized(true)(KeplrMock),
  },
  mockStorageFunction({ "gsw-last": "braavos" }),
  )
@@ -84,6 +92,8 @@ describe("getLastConnectedWallet()", () => {
  const sn = getWallet({
  "starknet-argentX": makeConnected(true)(ArgentXMock),
  "starknet-braavos": makeConnected(true)(BraavosMock),
+ starknet_okxwallet: makeConnected(true)(OKXMock),
+ starknet_keplr: makeAuthorized(true)(KeplrMock),
  })
  const lastConnectedWallet = await sn.getLastConnectedWallet()
  expect(lastConnectedWallet).toBe(null)
@@ -108,6 +118,8 @@ describe("getLastConnectedWallet()", () => {
  {
  "starknet-argentX": makeAuthorized(true)(ArgentXMock),
  "starknet-braavos": makeAuthorized(true)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(true)(OKXMock),
+ starknet_keplr: makeAuthorized(true)(KeplrMock),
  },
  mockStorageFunction({
  "gsw-last": "braavos",
@@ -125,6 +137,8 @@ describe("getLastConnectedWallet()", () => {
  {
  "starknet-argentX": makeAuthorized(true)(ArgentXMock),
  "starknet-braavos": makeAuthorized(true)(BraavosMock),
+ starknet_okxwallet: makeAuthorized(true)(OKXMock),
+ starknet_keplr: makeAuthorized(true)(KeplrMock),
  },
  mockStorageFunction({
  "gsw-last": "braavos",

packages/core/src/__test__/wallet.mock.ts

@@ -50,6 +50,36 @@ export const BraavosMock: StarknetWindowObject = {
  off: () => {},
 }
 
+export const OKXMock: StarknetWindowObject = {
+ ...wallets.find((w) => w.id === "okxwallet")!,
+ version: "0.0.0",
+ request: async (request) => {
+ switch (request.type) {
+ case "wallet_getPermissions":
+ return []
+ default:
+ return undefined as any
+ }
+ },
+ on: () => {},
+ off: () => {},
+}
+
+export const KeplrMock: StarknetWindowObject = {
+ ...wallets.find((w) => w.id === "keplr")!,
+ version: "0.0.0",
+ request: async (request) => {
+ switch (request.type) {
+ case "wallet_getPermissions":
+ return []
+ default:
+ return undefined as any
+ }
+ },
+ on: () => {},
+ off: () => {},
+}
+
 export function makeAuthorized(authorized: boolean) {
  return (wallet: StarknetWindowObject) =>
  ({