feat: Add new apps tranche3#232
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
More reviews will be available in 36 minutes and 27 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThree new projects and apps are registered with icons, metadata, and benchmark definitions. ChangesNew project/app registrations and benchmark data
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Adds tranche-3 app/project data to the ENSAwards dataset so the new apps can appear on the site and participate in benchmark display/aggregation.
Changes:
- Added new
ProjectIdsand project definitions for Exodus, Taho, and Pendle (including icons + socials). - Added new app definitions for Exodus Wallet, Taho Wallet, and Pendle DeFi, plus initial benchmark entries for each app.
- Extended Coinbase Exchange app socials to include an ENS name.
Reviewed changes
Copilot reviewed 15 out of 21 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| ensawards.org/data/projects/types.ts | Registers new project IDs for Exodus, Taho, and Pendle. |
| ensawards.org/data/projects/taho/index.ts | Defines the Taho project entry (metadata + icon + socials). |
| ensawards.org/data/projects/pendle/index.ts | Defines the Pendle project entry (metadata + icon + socials). |
| ensawards.org/data/projects/pendle/icon.tsx | Adds the Pendle project SVG icon component. |
| ensawards.org/data/projects/exodus/index.ts | Defines the Exodus project entry (metadata + icon + socials). |
| ensawards.org/data/projects/exodus/icon.tsx | Adds the Exodus project SVG icon component (gradient-based). |
| ensawards.org/data/apps/taho-wallet/index.ts | Adds the Taho Wallet app entry linked to the Taho project. |
| ensawards.org/data/apps/taho-wallet/benchmarks/index.tsx | Adds initial benchmark results/notes for Taho Wallet. |
| ensawards.org/data/apps/pendle-defi/index.ts | Adds the Pendle DeFi app entry linked to the Pendle project. |
| ensawards.org/data/apps/pendle-defi/icon.tsx | Adds the Pendle DeFi SVG icon component. |
| ensawards.org/data/apps/pendle-defi/benchmarks/index.tsx | Adds initial benchmark results/notes for Pendle DeFi. |
| ensawards.org/data/apps/exodus-wallet/index.ts | Adds the Exodus Wallet app entry linked to the Exodus project. |
| ensawards.org/data/apps/exodus-wallet/icon.tsx | Adds the Exodus Wallet SVG icon component (gradient-based). |
| ensawards.org/data/apps/exodus-wallet/benchmarks/index.tsx | Adds initial benchmark results/notes for Exodus Wallet. |
| ensawards.org/data/apps/coinbase-exchange/index.ts | Adds an ens entry to Coinbase Exchange socials. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
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
`@ensawards.org/data/apps/pendle-defi/benchmarks/resolution/deposit-addresses/index.tsx`:
- Line 39: Fix the punctuation typo in the user-facing benchmark note copy by
removing the extra space before the comma in the text rendered from the
deposit-addresses benchmark component. Update the string in the JSX around the
note content so it reads naturally, and verify the surrounding copy in the same
markup block still concatenates correctly.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 6bd877d9-ca5d-468f-b6dd-43b267d40d95
⛔ Files ignored due to path filters (12)
ensawards.org/data/apps/exodus-wallet/benchmarks/resolution/deposit-addresses/at-1.pngis excluded by!**/*.pngensawards.org/data/apps/exodus-wallet/benchmarks/resolution/ensv2-ready-resolution/at-1.pngis excluded by!**/*.pngensawards.org/data/apps/pendle-defi/benchmarks/resolution/deposit-addresses/at-1-boros.gifis excluded by!**/*.gifensawards.org/data/apps/pendle-defi/benchmarks/resolution/deposit-addresses/at-1-v2.gifis excluded by!**/*.gifensawards.org/data/apps/pendle-defi/benchmarks/resolution/ensv2-ready-resolution/at-1-boros.gifis excluded by!**/*.gifensawards.org/data/apps/pendle-defi/benchmarks/resolution/ensv2-ready-resolution/at-1-v2.gifis excluded by!**/*.gifensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/at-1.pngis excluded by!**/*.pngensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/at-2.pngis excluded by!**/*.pngensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/at-3.pngis excluded by!**/*.pngensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/at-4.pngis excluded by!**/*.pngensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/at-5.gifis excluded by!**/*.gifensawards.org/data/apps/taho-wallet/benchmarks/resolution/ensv2-ready-resolution/at-1.pngis excluded by!**/*.png
📒 Files selected for processing (9)
ensawards.org/data/apps/exodus-wallet/benchmarks/index.tsxensawards.org/data/apps/exodus-wallet/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/exodus-wallet/benchmarks/resolution/ensv2-ready-resolution/index.tsxensawards.org/data/apps/pendle-defi/benchmarks/index.tsxensawards.org/data/apps/pendle-defi/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/pendle-defi/benchmarks/resolution/ensv2-ready-resolution/index.tsxensawards.org/data/apps/taho-wallet/benchmarks/index.tsxensawards.org/data/apps/taho-wallet/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/taho-wallet/benchmarks/resolution/ensv2-ready-resolution/index.tsx
Greptile SummaryAdds five new apps in tranche 3 — Exodus (wallet), Taho (wallet), Pendle (DeFi), Hyperliquid (DeFi), and Polymarket (DeFi) — along with their project definitions, icons, and benchmark data. Also patches
Confidence Score: 5/5Safe to merge; all new files follow the established registration and benchmark pattern with no structural or logic errors. All five new apps, their project counterparts, and their benchmark data are structurally correct and consistent with existing entries. The AT08 → AT05 dependency for Taho is accurate (zissou.eth is tested in the Base chain context). The only item flagged is a minor JSX whitespace cosmetic in one Polymarket note. The acknowledged limitation around Polymarket's geographic testing restrictions is clearly documented in the PR description and is a process matter for the team rather than a code defect. No files require special attention; the Polymarket deposit-addresses benchmark file has a trivial cosmetic fix noted above. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Projects ["New Project Definitions (data/projects/)"]
P1[exodus/index.ts]
P2[taho/index.ts]
P3[pendle/index.ts]
P4[hyperliquid/index.ts]
P5[polymarket/index.ts]
end
subgraph Apps ["New App Definitions (data/apps/)"]
A1[exodus-wallet/index.ts]
A2[taho-wallet/index.ts]
A3[pendle-defi/index.ts]
A4[hyperliquid-defi/index.ts]
A5[polymarket-defi/index.ts]
end
subgraph Benchmarks ["Benchmark Data per App"]
B1["deposit-addresses\n(AT01-AT08)"]
B2["ensv2-ready-resolution\n(1 AT)"]
B3["Contract Naming\n(stubbed undefined)"]
B4["Contract Naming\n(Polymarket: filled)"]
end
P1 --> A1
P2 --> A2
P3 --> A3
P4 --> A4
P5 --> A5
A1 --> B1 & B2 & B3
A2 --> B1 & B2 & B3
A3 --> B1 & B2 & B3
A4 --> B1 & B2 & B3
A5 --> B1 & B2 & B4
B1 -->|"AT01 Fail: AT02-AT08 N/A"| R1[All N/A cascade]
B1 -->|"AT01-AT03 Pass, AT04 Fail, AT05 N/A"| R2[Taho partial pass]
B2 -->|"N/A no ENS integration"| R3[N/A for 4 apps]
B2 -->|"Fail ENSv2 not ready"| R4[Taho Fail]
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
subgraph Projects ["New Project Definitions (data/projects/)"]
P1[exodus/index.ts]
P2[taho/index.ts]
P3[pendle/index.ts]
P4[hyperliquid/index.ts]
P5[polymarket/index.ts]
end
subgraph Apps ["New App Definitions (data/apps/)"]
A1[exodus-wallet/index.ts]
A2[taho-wallet/index.ts]
A3[pendle-defi/index.ts]
A4[hyperliquid-defi/index.ts]
A5[polymarket-defi/index.ts]
end
subgraph Benchmarks ["Benchmark Data per App"]
B1["deposit-addresses\n(AT01-AT08)"]
B2["ensv2-ready-resolution\n(1 AT)"]
B3["Contract Naming\n(stubbed undefined)"]
B4["Contract Naming\n(Polymarket: filled)"]
end
P1 --> A1
P2 --> A2
P3 --> A3
P4 --> A4
P5 --> A5
A1 --> B1 & B2 & B3
A2 --> B1 & B2 & B3
A3 --> B1 & B2 & B3
A4 --> B1 & B2 & B3
A5 --> B1 & B2 & B4
B1 -->|"AT01 Fail: AT02-AT08 N/A"| R1[All N/A cascade]
B1 -->|"AT01-AT03 Pass, AT04 Fail, AT05 N/A"| R2[Taho partial pass]
B2 -->|"N/A no ENS integration"| R3[N/A for 4 apps]
B2 -->|"Fail ENSv2 not ready"| R4[Taho Fail]
Reviews (2): Last reviewed commit: "Apply AI agents' suggestions, pt.2" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@ensawards.org/data/apps/polymarket-defi/icon.tsx`:
- Around line 5-6: The icon definition is embedding a large base64 payload
directly in executable JS, which increases bundle size and prevents separate
asset caching. Replace the polymarketIconDataUri constant usage in the icon
module with a static image asset import or referenced file so the icon is served
as an external resource instead of inline bytes.
In `@ensawards.org/data/projects/polymarket/icon.tsx`:
- Around line 5-6: Move the inlined PNG payload out of polymarketIconDataUri in
the icon module and replace it with a reference to a static image asset. Load
the icon through the existing TSX component using an imported file or asset URL
so the binary bytes are not embedded in JavaScript, and update any consumers of
polymarketIconDataUri to use the new asset-backed symbol.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 1e358a17-ff56-4e2c-b1d3-2210c86306e0
⛔ Files ignored due to path filters (6)
ensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/deposit-addresses/at-1-explorer.gifis excluded by!**/*.gifensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/deposit-addresses/at-1-withdraw.pngis excluded by!**/*.pngensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/ensv2-ready-resolution/at-1-explorer.gifis excluded by!**/*.gifensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/ensv2-ready-resolution/at-1-withdraw.pngis excluded by!**/*.pngensawards.org/data/apps/polymarket-defi/benchmarks/resolution/deposit-addresses/at-1.gifis excluded by!**/*.gifensawards.org/data/apps/polymarket-defi/benchmarks/resolution/ensv2-ready-resolution/at-1.gifis excluded by!**/*.gif
📒 Files selected for processing (20)
ensawards.org/data/apps/exodus-wallet/icon.tsxensawards.org/data/apps/hyperliquid-defi/benchmarks/index.tsxensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/hyperliquid-defi/benchmarks/resolution/ensv2-ready-resolution/index.tsxensawards.org/data/apps/hyperliquid-defi/icon.tsxensawards.org/data/apps/hyperliquid-defi/index.tsensawards.org/data/apps/pendle-defi/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/polymarket-defi/benchmarks/contract-naming/display-named-smart-contracts-l2-chains/index.tsxensawards.org/data/apps/polymarket-defi/benchmarks/contract-naming/display-named-smart-contracts-mainnet/index.tsxensawards.org/data/apps/polymarket-defi/benchmarks/index.tsxensawards.org/data/apps/polymarket-defi/benchmarks/resolution/deposit-addresses/index.tsxensawards.org/data/apps/polymarket-defi/benchmarks/resolution/ensv2-ready-resolution/index.tsxensawards.org/data/apps/polymarket-defi/icon.tsxensawards.org/data/apps/polymarket-defi/index.tsensawards.org/data/projects/exodus/icon.tsxensawards.org/data/projects/hyperliquid/icon.tsxensawards.org/data/projects/hyperliquid/index.tsensawards.org/data/projects/polymarket/icon.tsxensawards.org/data/projects/polymarket/index.tsensawards.org/data/projects/types.ts
Member
Author
lightwalker-eth
approved these changes
Jun 29, 2026
lightwalker-eth
left a comment
lightwalker-eth
left a comment
Member
There was a problem hiding this comment.
@Y3drk Great updates! Thank you! 🫡
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lite PR → Add new apps tranche3 [WIP]
Summary
Why
Notes for reviewer
I couldn't find a legitimate testing surface for the following apps:
More details in the Google sheet spec. Appreciate the advice on that
Testing
typecheck,lint, andtestcommands locally to ensure that the migration didn't break anything, and later confirmed that in our CI workflowPre-Review Checklist (Blocking)