chore: use new repository for chain data #9
Conversation
✅ Deploy Preview for sovryn-layer ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull Request Overview
Switches token data sourcing from the old token-list repository to the new slayer-data repository and updates expected JSON structure. Key changes:
- Introduces ENV.DATA_BASE_URL and removes hardcoded GIT_TOKEN_LIST_URL constant.
- Adjusts token fetching logic to use new URL pattern and expects a wrapped items array in the JSON.
- Adds bob-sepolia chain config and alters the list of chain IDs fetched (Rootstock testnet 31 commented out, bob-sepolia 808813 added).
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| apps/indexer/src/workers/ingest/sources/tokens/token-fetcher.ts | Points to new data base URL, changes JSON parsing, modifies fetched chain IDs. |
| apps/indexer/src/env.ts | Adds DATA_BASE_URL environment variable (with default) using envalid url validator. |
| apps/indexer/src/configs/chains.ts | Adds bob-sepolia chain configuration entry. |
Comments suppressed due to low confidence (1)
apps/indexer/src/workers/ingest/sources/tokens/token-fetcher.ts:1
- ChainId 31 (Rootstock Testnet) remains defined in configs/chains.ts but is now excluded from token fetching by commenting it out here, creating an inconsistency that will prevent tokens for that configured chain from being ingested. Either restore 31 in this array or remove/flag the chain configuration entry if intentional.
import { and, eq, inArray, sql } from 'drizzle-orm';
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| return await response.json().then((data) => | ||
| uniqBy( | ||
| (data as TokenData[]).map((item) => ({ | ||
| (data as { items: TokenData[] }).items.map((item) => ({ |
There was a problem hiding this comment.
Assumes the fetched JSON always has an items array; if the structure differs or items is missing/null, .items.map will throw. Safely access with a runtime check (e.g., const items = Array.isArray((data as any).items) ? (data as any).items : [];).
| (data as { items: TokenData[] }).items.map((item) => ({ | |
| (Array.isArray((data as any).items) ? (data as any).items : []).map((item: TokenData) => ({ |
Copilot uses AI. Check for mistakes.
2 participants
https://sovryn.atlassian.net/browse/SOV-5195