Skip to content

Conversation

@shoom3301
Copy link
Contributor

@shoom3301 shoom3301 commented Nov 5, 2025

Added hook dappIds for AAVE flash-loans.
Actually copied and fixed https://github.com/cowprotocol/cow-sdk/pull/640/files

Summary by CodeRabbit

  • Infrastructure Improvements
    • Enhanced dApp identification for AAVE flash loan transactions, enabling better tracking and categorization of CollateralSwap, DebtSwap, and RepayCollateral operations.

@shoom3301 shoom3301 requested a review from a team November 5, 2025 10:21
@shoom3301 shoom3301 self-assigned this Nov 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

📦 GitHub Packages Published

Last updated: Nov 5, 2025, 10:24:07 AM UTC

The following packages have been published to GitHub Packages with pre-release version pr-645-233de7f1:


Installation

These packages require authentication to install from GitHub Packages. First, create a .npmrc file:

# Create .npmrc file in your project root
echo "@cowprotocol:registry=https://npm.pkg.github.com" > .npmrc
echo "//npm.pkg.github.com/:_authToken=YOUR_GITHUB_TOKEN" >> .npmrc

To get your GitHub token:

  1. Go to https://github.com/settings/tokens
  2. Click "Generate new token (classic)"
  3. Check only the "read:packages" scope
  4. Copy the token and replace YOUR_GITHUB_TOKEN in the .npmrc file

Then install any of the packages above, either by exact version (i.e. @cowprotocol/[email protected]) or more conveniently by using the tag (@cowprotocol/cow-sdk@pr-645):

# Yarn
yarn add npm:@cowprotocol/cow-sdk@pr-645

# pnpm
pnpm install npm:@cowprotocol/cow-sdk@pr-645

# NPM
npm install npm:@cowprotocol/cow-sdk@pr-645

Update to the latest version (only if you used the tag)

Every commit will publish a new package. To upgrade to the latest version, run:

# Yarn
yarn upgrade @cowprotocol/cow-sdk

# pnpm
pnpm update @cowprotocol/cow-sdk

# NPM
npm update @cowprotocol/cow-sdk

View Packages

You can view the published packages at: https://github.com/cowprotocol/cow-sdk/packages

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 5, 2025

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

📥 Commits

Reviewing files that changed from the base of the PR and between 203dbc6 and 020228c.

📒 Files selected for processing (2)
  • packages/flash-loans/src/aave/AaveCollateralSwapSdk.ts (2 hunks)
  • packages/flash-loans/src/aave/const.ts (1 hunks)
 _____________________________________________________________________________________________________________________________________________________________________________________________________
< Don't think outside the box - find the box. When faced with an impossible problem, identify the real constraints. Ask yourself: 'Does it have to be done this way? Does it have to be done at all?' >
 -----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/aave-flash-loan-hooks-dappId

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@shoom3301 shoom3301 merged commit b5312af into main Nov 5, 2025
8 of 9 checks passed
@shoom3301 shoom3301 deleted the feat/aave-flash-loan-hooks-dappId branch November 5, 2025 10:25
@github-actions github-actions bot locked and limited conversation to collaborators Nov 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants