Skip to content

data: add SPL token standard to remaining Solana-supporting wallet offers - #3681

Open
foxxx009 wants to merge 1 commit into
Chain-Love:mainfrom
foxxx009:spl-coolwallet-dcent-gate
Open

data: add SPL token standard to remaining Solana-supporting wallet offers#3681
foxxx009 wants to merge 1 commit into
Chain-Love:mainfrom
foxxx009:spl-coolwallet-dcent-gate

Conversation

@foxxx009

@foxxx009 foxxx009 commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Backfills the missing SPL token standard in the canonical tokensSupport cell for the three Solana-supporting wallet offers not covered by #3669:

  • coolwallet
  • dcent
  • gate

3 cells in references/offers/wallets.csv. No other columns, rows, or files touched.

Evidence (primary provider docs)

Wallet Source Quote
coolwallet help.coolwallet.io — Coins/Tokens Supported By CoolWallet Pro Mainnet list includes "Solana (SOL)"; "Supported Tokens: All ERC-20 / ERC-20(Arbitrum, Polygon) / BEP-20 / TRC-20 / SPL / FLR / TT customized tokens, and All ERC-721, ERC-1155 NFT tokens!"
dcent store.dcentwallet.com blog + product page "D'CENT Wallet is integrated with Solana SPL Tokens" ("Support for Solana chain and native currency (SOL asset); Support for Solana chain based token assets (SPL token assets)"); product page: "Supports all types of token standards, ERC-20, ERC-721, ERC-1155, SPL, Trustlines, and more"
gate gate.com Learn — Gate Wallet analysis "supports over 100 mainstream public chains, including Ethereum, BSC, Polygon, Solana, and Arbitrum, covering the mainstream ERC-20, BEP-20, and SPL token types"

Same root cause as the Phantom correction in #3660 and the #3669 backfill: these canonical rows are referenced by listings/specific-networks/solana/wallets.csv via !offer:<slug>, so the missing SPL standard propagates as inherited data to every network listing referencing these wallets.

Validation

  • validate_csv.py: All checks passed.
  • csv_to_json.py + validate.py (json-tools @ origin/json-tools, schema 2.0.0): exit 0.
  • Surgical edits verified programmatically: 3 changed cells, 0 off-target cells.
  • Only pre-existing baseline warnings remain.

Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 4
🔗 Unique 4
✅ Successful 4
⏳ Timeouts 0
🔀 Redirected 1
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

USS-Supervisor
USS-Supervisor previously approved these changes Sep 6, 2026

@USS-Supervisor USS-Supervisor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Verdict: APPROVE
Risk: LOW
Summary: Required validation is passing on the current head and the reviewer-owned current-cycle link-check run 34060772131 completed successfully. I checked the changed CSV surface and confirmed references/offers/wallets.csv still parses as 106 rows with 22 columns, with unchanged header, row count, and slug set; only the tokensSupport cells for coolwallet, dcent, and gate changed, and all three now include SPL without duplicate slugs or row-width shifts.
Findings: None.
Confidence: HIGH

The source check also supports the change: the touched canonical offers are referenced from listings/specific-networks/solana/wallets.csv, and the linked provider/Gate documentation supports SPL/Solana coverage for the three wallets. The current-cycle link-check summary reported 4 total/unique links, 4 successful, 0 errors, 0 timeouts, and 0 unsupported URLs.

eugene17kotov pushed a commit that referenced this pull request Sep 8, 2026
…3692)

Third batch of the SPL backfill for Solana-supporting canonical wallet
rows not covered by #3669/#3681, based on primary provider documentation:

- okxwallet: OKX Learn pages document OKX Wallet SPL Token Support
  ('OKX Wallet handles both Ethereum and Solana'; support table lists
  'SPL Token Support: Yes'); Solana Compass: 'hold SOL and SPL tokens'
- tangem: tangem.com blog 'Token compatibility: ERC-20, BEP-20, SPL
  (Solana), TRC-20'; help center lists Solana among custom-token networks
- zerion: zerion.io homepage ('Transaction history across EVM and
  Solana'; 'integrated Solana'); Google Play listing documents Solana
  SPL token support (USDT, USDC, BONK, JUP, RAY, PYTH on Solana)

Same root cause as #3660/#3669/#3681: these rows are referenced by
listings/specific-networks/solana/wallets.csv via !offer:<slug>.

No other columns, rows, or files touched. Validation (json-tools
validate_csv, csv_to_json, validate) passes.
eugene17kotov added a commit that referenced this pull request Sep 8, 2026
…fers (#3694)

* data: add SPL token standard to remaining Solana-supporting wallet offers

Fourth batch of the SPL backfill for Solana-supporting canonical wallet offers, not covered by #3669, #3681 or #3692:

- `1inch`
- `atomic-wallet`
- `coin-wallet`
- `cypherock-x1`
- `ecoin-wallet`
- `frontier`
- `fxwallet`
- `gem-wallet`
- `holdstation`
- `keystone-3-pro`
- `talisman`
- `trezor`
- `zengo`
- `zengo-pro`
- `zypto`

15 cells in `references/offers/wallets.csv`. No other columns, rows, or files touched.

## Evidence (primary provider docs)

| Wallet | Source | Quote |
|---|---|---|
| `1inch` | 1inch.io/wallet | Solana added to 1inch Wallet (2025-06-30); supports SOL and SPL tokens |
| `atomic-wallet` | atomicwallet.io + Solana Compass | Browser extension ships native Solana support: SOL, SPL tokens and NFTs |
| `coin-wallet` | coin.space/solana-wallet | Official Solana wallet page lists SOL and SPL token support |
| `cypherock-x1` | Cypherock release notes | Firmware v0.6.1793 adds SPL token support |
| `ecoin-wallet` | ecoinwallet.org/solana-wallet | v1.7.1 Solana integration with SPL token support |
| `frontier` | frontier.xyz | Solana listed among supported networks (35+ chains) |
| `fxwallet` | fxwallet.com + Solana Compass | Native support for SOL and SPL tokens |
| `gem-wallet` | docs.gemwallet.com | Full SPL and SPL Token-2022 support |
| `holdstation` | docs.holdstation.com + App Store v5.7 | "Solana Support - manage Solana (SOL) and SPL tokens"; Solana listed as Non-EVM |
| `keystone-3-pro` | Solana Compass | Manage SOL, SPL tokens and NFTs; Solflare integration |
| `talisman` | docs.talisman.xyz | "Solana, Substrate, and Bittensor networks and token balances are automatically enabled" |
| `trezor` | trezor.io | Solana and SPL support; Suite adds Token-2022 (2025-01) and native staking (2025-03) |
| `zengo` | zengo.com | "Send and receive SOL and SPL tokens" |
| `zengo-pro` | zengo.com/pro | Pro tier shares the same engine; Solana/SPL support |
| `zypto` | zypto.com | Full SOL and SPL token support with Raydium DEX integration |

Same root cause as the Phantom correction in #3660 and the #3669/#3681/#3692 backfills: these canonical rows are referenced by `listings/specific-networks/solana/wallets.csv` via `!offer:<slug>`, so the missing SPL standard propagates as inherited data to every network listing referencing these wallets.

## Deliberately not included

- `haha-wallet`: vendor copy is contradictory (App Store states "Ethereum and Polygon networks supported" while also listing Solana), so it needs a clearer primary source before claiming SPL.
- `metamask`, `base-wallet`, `ctrl-wallet`, `wigwam`, `pillar`, `clave-wallet`: still under verification.
- `kuma-wallet`, `taho`: these do not support Solana (EVM / Polkadot-ecosystem only) and appear mis-listed on the Solana page; reported separately instead of being given SPL.

## Validation

- Surgical edits verified programmatically: 15 changed cells, 0 off-target cells.
- Row count unchanged at 106; only the `tokensSupport` column was touched.

Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2

* data: add SPL token standard to MetaMask offer

MetaMask ships native Solana support, so its canonical row needs the SPL
token standard alongside its EVM standards.

Evidence:
- MetaMask Support, "Navigate Solana in MetaMask": "MetaMask supports Solana,
  allowing you to manage your SOL and SPL tokens alongside your EVM-supported
  (Ethereum) assets."
- Consensys blog, "Native Solana support in MetaMask accelerates new era of
  multichain access": users can "seamlessly manage Solana and SPL tokens,
  including sending, receiving, swapping, bridging, and onramping assets".

1 cell in `references/offers/wallets.csv`. No other columns, rows, or files touched.

Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2

* data: add SPL token standard to Wigwam and Base (Coinbase Wallet) offers

Both wallets ship Solana support, so their canonical rows need the SPL token
standard alongside their EVM standards.

Evidence:
- Wigwam (official, https://wigwam.app): "Wigwam is a free crypto wallet that
  supports the most common EVM blockchains and Solana from the box"; FAQ: "The
  Wigwam app supports all tokens issued on EVM blockchains and Solana."
- Coinbase (official blog, Solana support announcement): "Coinbase Wallet
  extension users can store, send, and receive Solana (SOL) and all of its SPL
  tokens alongside all of their EVM-compatible assets" -- Coinbase Wallet is the
  product now shipped as Base App / Base Wallet.

2 cells in `references/offers/wallets.csv`. No other columns, rows, or files touched.

Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2

* remove spl from trezor

* remove duplicated language

---------

Co-authored-by: Yevhenii Kobets <eu.kobets@gmail.com>
…oolwallet, dcent, gate)

Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2
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