Skip to content

feat: Add BONK token support to backend and SDK - #12489

Merged
faridsalau merged 3 commits into
mainfrom
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged

feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom
pr-2-bonk-backend-sdk-support

Conversation

@faridsalau

@faridsalaufaridsalau commented Jul 9, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Description

  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support

This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.

How Has This Been Tested?

npm run web:prod and ran #12454 PR which holds these changes

@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot

changeset-botBot commented Jul 9, 2025•
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: cef2b6f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@gitguardian

gitguardianBot commented Jul 9, 2025•
edited
Loading

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider

🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@faridsalaufaridsalau mentioned this pull request Jul 9, 2025
Merged
@faridsalau
faridsalau marked this pull request as ready for review July 10, 2025 15:27
cursor[bot]
cursorBot reviewed Jul 10, 2025
View reviewed changes

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch

The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.

packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64

https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64

packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64

https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64

Fix in Cursor • Fix in Web

BugBot free trial expires on July 22, 2025
You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

dylanjeffers
dylanjeffers approved these changes Jul 10, 2025
View reviewed changes

@dylanjeffersdylanjeffers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good

Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
!destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||

@dylanjeffersdylanjeffersJul 10, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice refactor

Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
mint: configWithDefaults.mints.USDC
})
}
this.authorities = Object.fromEntries(

@dylanjeffersdylanjeffersJul 10, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cool

@faridsalau
faridsalau merged commit a1b4336 into mainJul 10, 2025
@faridsalau
faridsalau deleted the pr-2-bonk-backend-sdk-support branch July 10, 2025 18:31
audius-infra pushed a commit that referenced this pull request Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit that referenced this pull request Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit that referenced this pull request Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit that referenced this pull request Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit that referenced this pull request Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit that referenced this pull request Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees

No one assigned

Labels

size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
feat: Add BONK token support to backend and SDK by faridsalau · Pull Request #12489 · AudiusProject/apps · GitHub
Skip to content
feat: Add BONK token support to backend and SDK - #12489
Merged
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Jul 10, 2025
Merged
feat: Add BONK token support to backend and SDK#12489
faridsalau merged 3 commits into
mainfrom 
pr-2-bonk-backend-sdk-support
Conversation
@faridsalau
@faridsalaufaridsalau
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

Contributor
Description
  • Add bonkTokenMint to SDK configs (dev/staging/prod)
  • Update ClaimableTokensClient to dynamically handle BONK authorities
  • Add BONK support to PaymentRouterClient configuration
  • Add BONK to TokenNameSchema and mintFixedDecimalMap
  • Add BONK currency to fixed-decimal package
  • Add comprehensive token configuration system to solana-relay
  • Add token utilities for dynamic token address resolution
  • Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services and establishes the foundation for multi-token operations.
How Has This Been Tested?
npm run web:prod and ran #12454 PR which holds these changes
@faridsalau
feat: Add BONK token support to backend and SDK
9370492
- Add bonkTokenMint to SDK configs (dev/staging/prod)
- Update ClaimableTokensClient to dynamically handle BONK authorities
- Add BONK support to PaymentRouterClient configuration
- Add BONK to TokenNameSchema and mintFixedDecimalMap
- Add BONK currency to fixed-decimal package
- Add comprehensive token configuration system to solana-relay
- Add token utilities for dynamic token address resolution
- Refactor relay instruction validation for multi-token support
This enables BONK token transactions across all backend services
and establishes the foundation for multi-token operations.
@changeset-bot
changeset-botBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: cef2b6f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@gitguardian
gitguardianBot
commented
Jul 9, 2025•

edited

Loading

Copy link


Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
18250942TriggeredGeneric High Entropy Secret9370492packages/sdk/src/sdk/config/staging.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@faridsalaufaridsalau
mentioned this pull request
Jul 9, 2025
 Merged

@faridsalau
faridsalau
marked this pull request as ready for review
July 10, 2025 15:27
cursor[bot]
cursorBot
reviewed
Jul 10, 2025
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Bug: Token Symbol Case Mismatch
The getPaymentRouterUsdcTokenAccount() function attempts to access config.tokens.usdc.address using a lowercase symbol. However, the config.tokens object stores token configurations with uppercase symbols (e.g., USDC) as keys, leading to a runtime error due to a case mismatch.
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts#L56-L64
packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
https://github.com/AudiusProject/audius-protocol/blob/cef2b6f29ce34f1a98cf204e245a4140280f785d/packages/discovery-provider/plugins/pedalboard/apps/solana-relay/src/routes/relay/tokenUtils.ts#L56-L64
Fix in Cursor • Fix in Web
BugBot free trial expires on July 22, 2025

You have used $0.00 of your $0.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
dylanjeffers
dylanjeffers
approved these changes
Jul 10, 2025
View reviewed changes
@dylanjeffersdylanjeffers
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

looks good
Comment thread...ider/plugins/pedalboard/apps/solana-relay/src/routes/relay/assertRelayAllowedInstructions.ts
 !destination.equals(PAYMENT_ROUTER_USDC_TOKEN_ACCOUNT)
 ) {
const isValidDestination =
userbankAddresses.some((userbank) => destination.equals(userbank)) ||
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

nice refactor
Comment threadpackages/sdk/src/sdk/services/Solana/programs/ClaimableTokensClient/ClaimableTokensClient.ts
 mint: configWithDefaults.mints.USDC
 })
 }
 this.authorities = Object.fromEntries(
@dylanjeffersdylanjeffersJul 10, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

cool
@faridsalau
faridsalau
merged commit a1b4336
into
mainJul 10, 2025
@faridsalau
faridsalau
deleted the

pr-2-bonk-backend-sdk-support

branch
July 10, 2025 18:31
audius-infra pushed a commit
that referenced
this pull request
Jul 11, 2025
audius-infra
Audius Protocol v0.7.196
5ac2de3
[46abcb5] Replace packages/discovery-provider/ddl with new api version (#12512) Ray Jacobson
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[90ae6ce] Remove foreign key cascade delete for blocks/shares (#12505) Randy Schott
audius-infra pushed a commit
that referenced
this pull request
Jul 12, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.143
e5e6832
[1cd89dc] [PE-6512] Hex USDC Logos (#12522) Reed
[82780e3] Drop smart collections, old explore (#12516) Dylan Jeffers
[73dd687] Fix follower count formatting (#12520) JD Francis
[5c2a3f9] [PE-6503] Add highlighting for comment replies (#12519) KJ
[371ab90] [PE-6506] Artist coin message blast UI (#12518) Reed
[afd0b16] Drop coming soon offline mode features (#12514) Dylan Jeffers
[9bd6f7d] Fix purchase complete x share button size (#12513) Dylan Jeffers
[9d3a7df] Migrate to UserBadgesV2 on mobile (#12511) Reed
[d479052] Migrate to UserBadgesV2 on web (#12507) Reed
[442ce6d] [PE-6477] BONK badges (#12506) Reed
[99a5a43] [QA-2205] Fix remix section scrolling to top (properly this time) (#12509) JD Francis
[464bdd0] Fix useHighlightComment hook (#12508) KJ
[2c8aa30] [API-261] Fire async share action when opening the share modal/drawer for content (#12492) Randy Schott
[5908ced] [QA-2207] Fix double scrollbars (#12504) JD Francis
[a1b4336] feat: Add BONK token support to backend and SDK (#12489) Farid Salau
[5ab9772] [PE-6460] Add top of section display and highlighting on linked comments (#12501) KJ
[3a223ce] [QA-2205] Fix remix section scroll to top bug (#12503) JD Francis
[543ee7e] feat: Add core token registry infrastructure (#12488) Farid Salau
[0b788ef] Fix scrolling in recent searches (#12500) Isaac Solo
[2d5426c] Fix guest account issues (#12496) JD Francis
[8e17858] [QA-2209] Fix track replace crashing app (#12497) JD Francis
[b066b17] Fix wrong amounts for USDC purchase buyer notifs (#12494) Reed
[d34e011] [PE-6458] Update SDK and getComment hook (#12487) KJ
[eddb083] [QA-2185] Fix user link flow (#12476) Dylan Jeffers
[bccc9b7] [PE-6492] Mobile hex tokens (#12491) Reed
[fa34cca] Don't use call with queryClient in sagas (#12486) Reed
[15bba9a] Fix mobile profile navving to reposts tab for artists sometimes (#12478) JD Francis
[45b2bde] Fix search result scroll cutoff (#12484) Isaac Solo
[1a08b32] [PE-6476] Web hexagonal token icons (#12480) Reed
[f878d8f] Fix podcast playback rate on mobile (#12475) Dylan Jeffers
[0f78928] [PE-6466] Fix /user network spam on app start (#12450) JD Francis
[19208ad] [QA-2192] Fix social link spacing (#12474) Dylan Jeffers
[0dfd7d0] Fix X icon color on mobile (#12479) Reed
[bad4d5f] Search explore milestone 2: for you, recent premium, best selling (#12467) Isaac Solo
[f780db4] [PE-6421] Improve ProfileScreen render time by ~50% (#12470) JD Francis
[1d20005] Remove ToS banner update (#12473) Ray Jacobson
[c5f8442] [HOTFIX] Small ui bugs - UserLink margin & trending playlist indices (#12472) JD Francis
[27aadc3] Add mobile web search explore (#12456) Isaac Solo
[8d2627f] [PE-6485] Add track tile integration tests (#12466) Dylan Jeffers
[bdc3982] [PE-6489] Make buy/sell work on stage (#12471) Farid Salau
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)"
411b2ed
This reverts commit a1b4336.
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
c220118
…2553)
rickyrombo added a commit
that referenced
this pull request
Jul 17, 2025
@rickyrombo
Revert "feat: Add BONK token support to backend and SDK (#12489)" (#1…
ffc53e2
…2553)
audius-infra pushed a commit
that referenced
this pull request
Jul 18, 2025
audius-infra
Audius Protocol v0.7.201
98e6c5e
[179b457] Update eth rpc urls (#12552) KJ
[33d6c1a] Create bonk artist coin account on stage (#12558) Reed
[c220118] Revert "feat: Add BONK token support to backend and SDK (#12489)" (#12553) Marcus Pasell
[4c09e50] [PE-6537] Add comment_id to notification action data (#12548) Dylan Jeffers
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@dylanjeffersdylanjeffersdylanjeffers approved these changes
@dharit-tandharit-tanAwaiting requested review from dharit-tan
@Kyle-ShanksKyle-ShanksAwaiting requested review from Kyle-Shanks

Assignees


No one assigned


Labels

size/L

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@faridsalau@dylanjeffers