Skip to content

Conversation

@nazihkalo
Copy link
Contributor

@nazihkalo nazihkalo commented Jan 18, 2026

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

This pull request adds a new stablecoin entry to the Solana stablecoins model. The change is minor and simply updates the list of recognized stablecoins.

  • Added the CASH stablecoin (symbol: CASH, address: CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH) as a fiat-backed stablecoin issued by CASH to the tokens_solana_stablecoins.sql model.

quick links for more information:

Copilot AI review requested due to automatic review settings January 18, 2026 23:41
@cursor
Copy link

cursor bot commented Jan 18, 2026

PR Summary

Adds a new Solana stablecoin entry.

  • Adds CASH (Fiat Stablecoin, issuer CASH, 6 decimals) to tokens_solana_stablecoins.sql with address CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH

Written by Cursor Bugbot for commit 3e8f064. Configure here.

@github-actions github-actions bot marked this pull request as draft January 18, 2026 23:41
@github-actions github-actions bot added the WIP work in progress label Jan 18, 2026
@github-actions
Copy link

github-actions bot commented Jan 18, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@github-actions github-actions bot added the dbt: solana covers the Solana dbt subproject label Jan 18, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the Phantom Cash (CASH) stablecoin to the Solana stablecoins token list. The PR title indicates this is "Phantom Cash," a new fiat-backed stablecoin on the Solana blockchain.

Changes:

  • Added CASH stablecoin entry to the tokens_solana_stablecoins.sql static table

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

, ('solana', 'ZUSD','FrBfWJ4qE5sCzKm3k3JaAtqZcXUh4LvJygDeketsrsH4', 6,'Dollar-Pegged','GMO-Z')
, ('solana', 'PAI', 'Ea5SjE2Y6yvCeW5dYTn7PYMuW5ikXkvbGdcmSnXeaLjS', 6,'Crypto Stablecoin', 'Parrot')
, ('solana', 'FDUSD', '9zNQRsGLjNKwCUU5Gq5LR8beUCPzQMVMqKAi3SSZh54u', 6,'Fiat Stablecoin', 'First Digital Labs')
, ('solana', 'CASH', 'CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH', 6,'Fiat Stablecoin', 'CASH')
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

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

The name field should contain the issuer or organization name rather than repeating the token symbol. Looking at other entries, the name field contains values like 'Circle' (for USDC), 'Tether' (for USDT), 'Paxos' (for PYUSD), etc. For Phantom Cash, this should be 'Phantom' or the actual issuer organization name instead of 'CASH'.

Suggested change
, ('solana', 'CASH', 'CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH', 6,'Fiat Stablecoin', 'CASH')
, ('solana', 'CASH', 'CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH', 6,'Fiat Stablecoin', 'Phantom')

Copilot uses AI. Check for mistakes.
@nazihkalo
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@nazihkalo
Copy link
Contributor Author

recheck

github-actions bot added a commit that referenced this pull request Jan 18, 2026
@nazihkalo nazihkalo marked this pull request as ready for review January 18, 2026 23:44
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jan 18, 2026
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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

, ('solana', 'ZUSD','FrBfWJ4qE5sCzKm3k3JaAtqZcXUh4LvJygDeketsrsH4', 6,'Dollar-Pegged','GMO-Z')
, ('solana', 'PAI', 'Ea5SjE2Y6yvCeW5dYTn7PYMuW5ikXkvbGdcmSnXeaLjS', 6,'Crypto Stablecoin', 'Parrot')
, ('solana', 'FDUSD', '9zNQRsGLjNKwCUU5Gq5LR8beUCPzQMVMqKAi3SSZh54u', 6,'Fiat Stablecoin', 'First Digital Labs')
, ('solana', 'CASH', 'CASHx9KJUStyftLFWGvEVf59SGeG9sh5FfcnZMVPCASH', 6,'Fiat Stablecoin', 'CASH')
Copy link

Choose a reason for hiding this comment

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

Name field uses token symbol instead of issuer

Low Severity

The name field for the CASH token entry uses 'CASH' (the token symbol) instead of the issuer name. According to the schema, the name field contains "The name of the issuer or protocol of the stablecoin". All other entries follow this convention using issuer names like 'Circle', 'Tether', 'Paxos'. For Phantom Cash (issued by Phantom wallet), the value likely should be 'Phantom' rather than the token symbol.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dbt: solana covers the Solana dbt subproject ready-for-review this PR development is complete, please review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant