Skip to content

Update PROGRAM_ADDRESS to new value#20071

Closed
williamspatrick3353-droid wants to merge 1 commit into
DefiLlama:mainfrom
williamspatrick3353-droid:patch-56
Closed

Update PROGRAM_ADDRESS to new value#20071
williamspatrick3353-droid wants to merge 1 commit into
DefiLlama:mainfrom
williamspatrick3353-droid:patch-56

Conversation

@williamspatrick3353-droid

@williamspatrick3353-droid williamspatrick3353-droid commented Jul 16, 2026

Copy link
Copy Markdown

NOTE

Please enable "Allow edits by maintainers" while putting up the PR.


  1. If you would like to add a volume/fees/revenue adapter please submit the PR here.

  2. Once your adapter has been merged, it takes time to show on the UI. If more than 24 hours have passed, please let us know in Discord.

  3. Sorry, We no longer accept fetch adapter for new projects, we prefer the tvl to computed from blockchain data, if you have trouble with creating a the adapter, please hop onto our discord, we are happy to assist you.

  4. For updating listing info Please send a mail to metadata@defillama.com

  5. Please do not add new npm dependencies, do not edit/push pnpm-lock.yaml file as part of your changes


(Needs to be filled only for new listings)

Name (to be shown on DefiLlama):
Twitter Link:
List of audit links if any:
Website Link:
Logo (High resolution, will be shown with rounded borders):
Current TVL:
Treasury Addresses (if the protocol has treasury)
Chain:
Coingecko ID (so your TVL can appear on Coingecko, leave empty if not listed): (https://api.coingecko.com/api/v3/coins/list)
Coinmarketcap ID (so your TVL can appear on Coinmarketcap, leave empty if not listed): (https://api.coinmarketcap.com/data-api/v3/map/all?listing_status=active,inactive,untracked&start=1&limit=10000)
Short Description (to be shown on DefiLlama):
Token address and ticker if any:
Category (full list at https://defillama.com/categories) *Please choose only one:
Oracle Provider(s): Specify the oracle(s) used (e.g., Chainlink, Band, API3, TWAP, etc.):
Implementation Details: Briefly describe how the oracle is integrated into your project:
Documentation/Proof: Provide links to documentation or any other resources that verify the oracle's usage:
forkedFrom (Does your project originate from another project):
methodology (what is being counted as tvl, how is tvl being calculated):
Github org/user (Optional, if your code is open source, we can track activity):
Does this project have a referral program?

Summary by CodeRabbit

  • Bug Fixes
    • Updated the manifest’s program identifier to ensure TVL calculations and related address lookups use the correct program address.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The manifest’s Solana program address is replaced with a new hex address, changing the derived program ID and PDA calculations. The module export terminator is also adjusted.

Changes

Manifest program update

Layer / File(s) Summary
Update program address and export
projects/manifest-trade/index.js
The manifest now derives its program ID and PDAs from the new program address, and its module export closing line is adjusted.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is just boilerplate and leaves all required fields empty, so it doesn't describe the actual change. Replace the template text with a real PR summary covering the change, affected file, and any testing or migration notes.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the main change: updating PROGRAM_ADDRESS to a new value.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@projects/manifest-trade/index.js`:
- Line 4: Update the PROGRAM_ADDRESS constant to the program’s valid Solana
public key in base58 format, replacing the current 0x-prefixed hexadecimal value
so new PublicKey(PROGRAM_ADDRESS) initializes successfully.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 65b480f0-fae9-459b-a202-05e519fd0956

📥 Commits

Reviewing files that changed from the base of the PR and between e031d7f and 5d14742.

📒 Files selected for processing (1)
  • projects/manifest-trade/index.js

Comment thread projects/manifest-trade/index.js
@RohanNero RohanNero closed this Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants