Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughTop-level Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsTimed out fetching pipeline failures after 30000ms 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/public/NearSolana.json`:
- Around line 18-25: There are two token entries with chainId 1000000001 and
symbol "SOL" (addresses "So11111111111111111111111111111111111111112" wrapped
SOL and "11111111111111111111111111111111" native SOL) causing symbol
collisions; update the wrapped SOL entry by changing its symbol from "SOL" to
"WSOL" (and optionally update its name to "Wrapped SOL") so consumers keyed by
(chainId, symbol) can disambiguate between the native and wrapped token.
- Around line 72-73: Replace fragile external logoURI values with stable
canonical sources: update the three problematic "logoURI" entries in
NearSolana.json to use project-controlled CDN or immutable storage (e.g., GitHub
raw URLs following the existing USDC pattern or Arweave/IPFS immutable CID
gateway URLs like the Arweave example already present). Locate the entries by
searching for the "logoURI" keys that reference coinall.ltd, imgur.com, and the
Pinata gateway and swap them for equivalent immutable URLs
(raw.githubusercontent.com or arweave/ipfs CID-based gateway URLs) so the assets
are hosted under a stable, project-controlled or content-addressed location.
🪄 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: 1e49dcd0-c1eb-4322-8089-22bcdd2353fe
📒 Files selected for processing (1)
src/public/NearSolana.json
| "logoURI": "https://static.coinall.ltd/cdn/oksupport/common/20250512-095503.72e1f41d9b9a06.png" | ||
| }, |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Read-only URL health check for newly added logo URIs.
set -euo pipefail
urls=(
"https://static.coinall.ltd/cdn/oksupport/common/20250512-095503.72e1f41d9b9a06.png"
"https://i.imgur.com/fLpAyY4.png"
"https://gateway.pinata.cloud/ipfs/bafkreiduq5c43xdbcbhzx3plcdtz2jwkjpzlm2hwkoxhjgmnmqbl6otyfu"
)
for u in "${urls[@]}"; do
echo "=== $u"
curl -sSI --max-time 15 "$u" | sed -n '1,10p'
echo
doneRepository: cowprotocol/token-lists
Length of output: 1320
🏁 Script executed:
cat -n src/public/NearSolana.json | sed -n '65,130p'Repository: cowprotocol/token-lists
Length of output: 2715
Logo URIs use non-canonical hosts that may break over time.
The new logoURI values on lines 72, 103, and 119 rely on coinall.ltd, imgur.com, and a Pinata IPFS gateway. These are more fragile than immutable/canonical sources and can cause intermittent broken icons.
Use a stable canonical source (project-controlled repo/CDN or immutable IPFS CID URL) for these logos, consistent with the existing GitHub raw pattern used for USDC (line 80) or Arweave (line 95).
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/public/NearSolana.json` around lines 72 - 73, Replace fragile external
logoURI values with stable canonical sources: update the three problematic
"logoURI" entries in NearSolana.json to use project-controlled CDN or immutable
storage (e.g., GitHub raw URLs following the existing USDC pattern or
Arweave/IPFS immutable CID gateway URLs like the Arweave example already
present). Locate the entries by searching for the "logoURI" keys that reference
coinall.ltd, imgur.com, and the Pinata gateway and swap them for equivalent
immutable URLs (raw.githubusercontent.com or arweave/ipfs CID-based gateway
URLs) so the assets are hosted under a stable, project-controlled or
content-addressed location.
2 participants
Summary by CodeRabbit