feat(COW-163): Update cow-sdk version, add default RPC and run script for Ink#1307
feat(COW-163): Update cow-sdk version, add default RPC and run script for Ink#1307alfetopito merged 8 commits intomainfrom
Conversation
📝 WalkthroughWalkthroughAdd INK (chainId 57073) across configs, scripts, CI, token lists, and auxiliary mappings: RPC URL, NETWORK_CONFIG, CoinGecko/display name, new npm scripts for INK permit fetching, new public token and permit JSONs, ts-node options, .env example and gitignore entries, and an SDK version bump. Changes
Sequence Diagram(s)sequenceDiagram
participant CI as CI (GitHub Actions)
participant Repo as Repository
participant Script as fetchPermitInfo:ink
participant RPC as INK RPC\n(rpc-ten.inkonchain.com)
participant CoinGecko as CoinGecko API
participant FS as File System
CI->>Repo: workflow triggers (matrix includes 57073)
CI->>Script: run `fetchPermitInfo:ink`
Script->>RPC: query chain data (57073)
Script->>CoinGecko: request token metadata for 'ink'
Script->>FS: write `src/public/CoinGecko.57073.json` & `src/public/PermitInfo.57073.json`
FS-->>Repo: updated files staged/committed
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
All contributors have signed the CLA ✍️ ✅ |
![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
🤖 Fix all issues with AI agents
In `@package.json`:
- Around line 52-54: The package.json references an unpublished pre-release
"@cowprotocol/cow-sdk": "7.3.0"; change that dependency to the latest published
release "7.2.13" (or the correct published version you intend) in package.json,
then run your package manager to update lockfiles (npm/yarn/pnpm) and verify
imports/usages of `@cowprotocol/cow-sdk` in your scripts compile and tests pass;
if 7.3.0 was intentional, replace the version with a published tag (or add a
clear comment and registry configuration) and ensure the registry contains that
pre-release before merging.
| "dependencies": { | ||
| "@cowprotocol/cow-sdk": "7.1.0", | ||
| "@cowprotocol/cow-sdk": "7.3.0", | ||
| "@cowprotocol/permit-utils": "^0.7.0-RC.1", |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
@cowprotocol/cow-sdk 7.3.0 release notes changelog breaking changes
💡 Result:
As of January 28, 2026, there is no published GitHub Release/Tag for @cowprotocol/cow-sdk version 7.3.0 in the cowprotocol/cow-sdk repository (so there are no official 7.3.0 release notes / changelog / breaking-changes entry to reference). The latest published cow-sdk release visible on GitHub is:
cow-sdk: v7.2.13(2026-01-22) — release notes list dependency bumps only and do not mention breaking changes. [1]
If you meant a different artifact (e.g., an npm prerelease tag like 7.3.0-beta.x), tell me the exact version string and where you saw it (npm, GitHub, etc.), and I can look up that changelog.
Sources: [1]
🏁 Script executed:
# Examine the full package.json to understand current dependencies and peer deps
cat package.jsonRepository: cowprotocol/token-lists
Length of output: 4094
🏁 Script executed:
# Check if there are any scripts that depend on cow-sdk
rg "cow-sdk" --type json --type ts --type js -A 2 -B 2 | head -50Repository: cowprotocol/token-lists
Length of output: 3174
@cowprotocol/cow-sdk 7.3.0 is not a published release. The latest available version is v7.2.13 (2026-01-22). Verify whether 7.3.0 is an intentional pre-release version or a version mismatch, as release notes and peer dependency requirements cannot be confirmed for an unpublished version. Multiple scripts depend on this package (src/scripts/auxLists/, src/scripts/lp-tokens/, src/permitInfo/*), so the intended version must be clarified and available in the package registry.
🤖 Prompt for AI Agents
In `@package.json` around lines 52 - 54, The package.json references an
unpublished pre-release "@cowprotocol/cow-sdk": "7.3.0"; change that dependency
to the latest published release "7.2.13" (or the correct published version you
intend) in package.json, then run your package manager to update lockfiles
(npm/yarn/pnpm) and verify imports/usages of `@cowprotocol/cow-sdk` in your
scripts compile and tests pass; if 7.3.0 was intentional, replace the version
with a published tag (or add a clear comment and registry configuration) and
ensure the registry contains that pre-release before merging.
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @.env.example:
- Line 1: The file ends without a trailing newline; add a single newline
character after the final line so the file ends with a blank line (i.e., ensure
the COINGECKO_API_KEY= line is followed by a newline/linefeed at EOF).
| @@ -0,0 +1 @@ | |||
| COINGECKO_API_KEY= No newline at end of file | |||
There was a problem hiding this comment.
Add a trailing newline.
The file is missing a trailing newline at the end. Text files should end with a newline character per POSIX standards to ensure consistent behavior across tools and editors.
📝 Proposed fix
-COINGECKO_API_KEY=
+COINGECKO_API_KEY=
+(Add a blank line after the last line)
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| COINGECKO_API_KEY= | |
| COINGECKO_API_KEY= | |
🧰 Tools
🪛 dotenv-linter (4.0.0)
[warning] 1-1: [EndingBlankLine] No blank line at the end of the file
(EndingBlankLine)
🤖 Prompt for AI Agents
In @.env.example at line 1, The file ends without a trailing newline; add a
single newline character after the final line so the file ends with a blank line
(i.e., ensure the COINGECKO_API_KEY= line is followed by a newline/linefeed at
EOF).
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/ISSUE_TEMPLATE/3-removeTokenForm.yml (1)
24-35: Fix network naming mismatch: template usesGNOSIS_CHAIN, code expectsGNOSIS.The issue templates list
GNOSIS_CHAINas an option, butNETWORK_CONFIGinprocessRequest.mjsuses the keyGNOSIS. When an issue is submitted withGNOSIS_CHAIN,applyNetworkConfigwill fail because the lookupNETWORK_CONFIG[values.network]returns undefined, triggering the error "No valid network found."Align the template options with the
NETWORK_CONFIGkeys or add a mapping layer to handle the mismatch.
|
hey @Danziger , could you please describe testing steps here? |
No QA needed/applicable here. |
alfetopito
left a comment
alfetopito
left a comment
There was a problem hiding this comment.
Looks good, just 2 comments.
Remember to test the issue templates after merging them by adding some ink tokens.
For example, the wrapped native token for the chain.
| bridgeContractAddress: OMNIBRIDGE_ADDRESS, | ||
| bridgeContractAbi: OMNIBRIDGE_CONTRACT_ABI, | ||
| methodName: 'calculateL2TokenAddress', | ||
| methodName: 'bridgedTokenAddress', |
There was a problem hiding this comment.
Is this change needed?
There was a problem hiding this comment.
Not really, but I just noticed OMNIBRIDGE_CONTRACT_ABI doesn't define a calculateL2TokenAddress method, only bridgedTokenAddress.
Co-authored-by: Leandro <alfetopito@users.noreply.github.com>
3 participants
Summary by CodeRabbit
New Features
Chores
✏️ Tip: You can customize this high-level summary in your review settings.