Skip to content

Add K9 Finance (K9) token metadata#1818

Open
simplefarmer69 wants to merge 2 commits into
MetaMask:masterfrom
simplefarmer69:add-k9-token
Open

Add K9 Finance (K9) token metadata#1818
simplefarmer69 wants to merge 2 commits into
MetaMask:masterfrom
simplefarmer69:add-k9-token

Conversation

@simplefarmer69
Copy link
Copy Markdown

@simplefarmer69 simplefarmer69 commented May 18, 2026

Summary

Adds token metadata for K9 Finance ($K9) on Ethereum mainnet.

Field Value
Name K9 Finance
Symbol K9
Contract 0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930
Chain Ethereum mainnet (eip155:1)
Decimals 18
Standard ERC-20

Files Added

  • icons/eip155:1/erc20:0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930.png — 200×200 px PNG logo
  • metadata/eip155:1/erc20:0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930.json — CAIP-19 metadata

Verification

Made with Cursor


Note

Low Risk
Adds a single new ERC-20 metadata JSON entry; no executable code or security-sensitive logic changes.

Overview
Adds new token metadata for Ethereum mainnet ERC-20 0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930 (K9), including name, symbol, decimals, erc20 flag, and a logo path reference.

Reviewed by Cursor Bugbot for commit 10800e7. Bugbot is set up for automated code reviews on this repo. Configure here.

- Token: K9 Finance
- Symbol: K9
- Contract: 0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930 (Ethereum mainnet)
- Decimals: 18
- Adds icon and CAIP-19 metadata under eip155:1

Co-authored-by: Cursor <cursoragent@cursor.com>
@simplefarmer69 simplefarmer69 requested review from a team and MRabenda as code owners May 18, 2026 18:42
Comment thread metadata/eip155:1/erc20:0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930.json Outdated
Copy link
Copy Markdown

@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.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 10800e7. Configure here.

@@ -0,0 +1,7 @@
{
"logo": "./icons/eip155:1/erc20:0xbc920c7Fa25D3Cd5606394aa4C6751d71DCb7930.png",
"name": "K9",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Token name field contradicts PR description

Low Severity

The name field is set to "K9", which is identical to the symbol field. The PR description explicitly states the token Name is "K9 Finance" and the Symbol is "K9", suggesting name was intended to be "K9 Finance". In this repo, name typically represents the full human-readable token name (e.g., "Uniswap" not "UNI", "Basic Attention Token" not "BAT"), so having name equal symbol loses descriptive information.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 10800e7. Configure here.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The name field is correctly set to "K9" — this is the token's actual on-chain name as returned by token.name(). The initial PR description was inaccurate in calling it "K9 Finance"; the on-chain name is simply "K9" (same as the symbol). Using the on-chain name is the most accurate representation and avoids confusion with the unrelated K9 Finance DAO project.

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