unexclude amnis-finance, the apy fix it was waiting on is in

[0xshubhs]

amnis-finance was excluded in e5d514d with the note "temp only, until fix is in", and the fix landed the same day (826131c, amnis finance fix apy) — but the exclusion entry was never lifted, so the adapter still exports 0 pools while the protocol sits at ~$4.1M TVL on Aptos.

No adapter changes needed, just removes the exclusion entry.

Local run: 13/13 tests, 1 pool — stAPT $3.46M @ 2.51% apyBase, which lines up with api.amnis.finance/api/v1/stake/info (apr 2.51) and the on-chain stAPT supply × pricePerShare.

Summary by CodeRabbit

• Bug Fixes
  • Removed a hardcoded protocol exclusion so the affected protocol is no longer blocked by the default exclusion list.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2d66ef1d-3f50-4b1e-a801-e894eb58364e

📥 Commits

Reviewing files that changed from the base of the PR and between 6719a9c and 8c82e48.

📒 Files selected for processing (1)

• src/utils/exclude.js

💤 Files with no reviewable changes (1)

• src/utils/exclude.js

---

📝 Walkthrough

Walkthrough

Removed the hardcoded exclusion entry for the amnis-finance protocol (id 3667) from the excludedProtocols array in src/utils/exclude.js, which affects the derived excludeAdaptors list.

Changes

Exclusion list update

Layer / File(s)	Summary
Remove amnis-finance from excludedProtocols src/utils/exclude.js	Deleted the amnis-finance (id 3667) entry from the excludedProtocols array, removing it from the derived excludeAdaptors list.

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

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly states that amnis-finance is being re-enabled because the APY fix has landed.
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

• X
• Mastodon
• Reddit
• LinkedIn

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