Skip to content

Add donation attack patch audit reports#667

Open
Debugger022 wants to merge 1 commit intodevelopfrom
feat/audit
Open

Add donation attack patch audit reports#667
Debugger022 wants to merge 1 commit intodevelopfrom
feat/audit

Conversation

@Debugger022
Copy link
Copy Markdown
Contributor

Summary

  • Rename and standardize existing donation attack patch audit PDFs (Quantstamp, CertiK) to follow the {number}_{Name}_{Date} convention
  • Add the Hashdit audit report for the donation attack patch

Changes

Audits

  • Renamed 177_Venus VBep20 Donation Patch Final.pdf157_DonationAttack_Patch_Quantstamp_20260320.pdf
  • Renamed 178_CertiK-REP-final-20260320T090021Z.pdf158_DonationAttack_Patch_Certik_20260320.pdf
  • Added 159_DonationAttack_Patch_Hashdit_20260320.pdf

Replace inconsistently named audit PDFs (177, 178) with standardized
filenames (157, 158, 159) and add the Hashdit audit report.
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • audits/157_DonationAttack_Patch_Quantstamp_20260320.pdf is excluded by !**/*.pdf
  • audits/158_DonationAttack_Patch_Certik_20260320.pdf is excluded by !**/*.pdf
  • audits/159_DonationAttack_Patch_Hashdit_20260320.pdf is excluded by !**/*.pdf

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1ce50cef-7f2c-4a81-8980-11f16a24c53c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/audit

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

@Debugger022 Debugger022 self-assigned this Mar 26, 2026
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Health
contracts 100% 100%
contracts.Admin 88% 41%
contracts.Comptroller 100% 90%
contracts.Comptroller.Diamond 95% 64%
contracts.Comptroller.Diamond.facets 82% 69%
contracts.Comptroller.Diamond.interfaces 100% 100%
contracts.Comptroller.Types 100% 100%
contracts.Comptroller.legacy 100% 100%
contracts.Comptroller.legacy.Diamond 0% 0%
contracts.Comptroller.legacy.Diamond.facets 0% 0%
contracts.Comptroller.legacy.Diamond.interfaces 100% 100%
contracts.DelegateBorrowers 100% 89%
contracts.FlashLoan.interfaces 100% 100%
contracts.Governance 68% 45%
contracts.InterestRateModels 74% 59%
contracts.Lens 41% 34%
contracts.Liquidator 83% 60%
contracts.Oracle 100% 100%
contracts.PegStability 88% 84%
contracts.Swap 88% 58%
contracts.Swap.interfaces 100% 100%
contracts.Swap.lib 81% 55%
contracts.Tokens 100% 100%
contracts.Tokens.Prime 96% 72%
contracts.Tokens.Prime.Interfaces 100% 100%
contracts.Tokens.Prime.libs 90% 76%
contracts.Tokens.VAI 82% 53%
contracts.Tokens.VRT 20% 9%
contracts.Tokens.VTokens 67% 51%
contracts.Tokens.VTokens.legacy 0% 0%
contracts.Tokens.VTokens.legacy.Utils 0% 0%
contracts.Tokens.XVS 19% 8%
contracts.Tokens.test 100% 100%
contracts.Utils 52% 31%
contracts.VAIVault 50% 45%
contracts.VRTVault 49% 36%
contracts.XVSVault 63% 50%
contracts.external 100% 100%
contracts.lib 89% 71%
Summary 56% (3644 / 6509) 42% (1377 / 3260)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant