Skip to content

feat: add advisory Claude review workflow - #221

Draft
tobal-g wants to merge 1 commit into
devfrom
feat/claude-review
Draft

feat: add advisory Claude review workflow#221
tobal-g wants to merge 1 commit into
devfrom
feat/claude-review

Conversation

@tobal-g

@tobal-g tobal-g commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds a Claude review workflow for pull requests targeting dev.

  • Automatically reviews same-repository and external-fork PRs.
  • Uses the protected base workflow and a validated synthetic merge without executing PR-controlled code.
  • Keeps Claude jobs read-only and uses least-privilege publishers for comments and guidance proposals.
  • Restricts comment-triggered Claude interactions to maintainers.
  • Adds deterministic review context, advisory scanners, workflow contract tests, and setup guidance.

@tobal-g
tobal-g marked this pull request as draft August 21, 2026 19:37
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6914522a-ad2e-4986-b9c2-897c391e858d

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

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.

@github-actions

Copy link
Copy Markdown

Coverage after merging feat/claude-review into dev will be

100.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   ALMProxy.sol100%100%100%100%
   ALMProxyFreezable.sol100%100%100%100%
   AccessControls.sol100%100%100%100%
   Beacon.sol100%100%100%100%
   Controller.sol98.53%93.10%100%100%105–106
   ControllerSharedStorage.sol100%100%100%100%
   PAUFactory.sol100%100%100%100%
   RateLimits.sol100%100%100%100%
src/facets
   Facet.sol100%100%100%100%
src/facets/aave
   AaveFacet.sol100%100%100%100%
src/facets/aave-v4
   AaveV4Facet.sol100%100%100%100%
src/facets/basin
   BasinFacet.sol100%100%100%100%
src/facets/cctp
   CCTPFacet.sol100%100%100%100%
src/facets/centrifuge
   CentrifugeFacet.sol100%100%100%100%
src/facets/curve
   CurveFacet.sol99%93.94%100%100%434, 471
src/facets/dai-usds
   DAIUSDSFacet.sol100%100%100%100%
src/facets/erc4626
   ERC4626Facet.sol100%100%100%100%
src/facets/erc7540
   ERC7540Facet.sol100%100%100%100%
src/facets/ethena
   EthenaFacet.sol100%100%100%100%
src/facets/farm
   FarmFacet.sol100%100%100%100%
src/facets/layer-zero
   LayerZeroFacet.sol100%100%100%100%
src/facets/maple
   MapleFacet.sol100%100%100%100%
src/facets/merkl
   MerklFacet.sol100%100%100%100%
src/facets/nfat-halo
   NFATHaloFacet.sol100%100%100%100%
src/facets/nfat-prime
   NFATPrimeFacet.sol100%100%100%100%
src/facets/otc
   OTCBuffer.sol100%100%100%100%
   OTCFacet.sol100%100%100%100%
src/facets/pendle
   PendleFacet.sol100%100%100%100%
src/facets/psm
   PSMFacet.sol100%100%100%100%
src/facets/psm3
   PSM3Facet.sol100%100%100%100%
src/facets/spark-vault
   SparkVaultFacet.sol100%100%100%100%
src/facets/superstate
   SuperstateFacet.sol100%100%100%100%
src/facets/transfer-asset
   TransferAssetFacet.sol100%100%100%100%
src/facets/uniswap-v3
   UniswapV3Facet.sol99.43%97.47%100%100%988–989
   UniswapV3Utils.sol96.36%77.78%100%100%155, 56
src/facets/uniswap-v4
   UniswapV4Facet.sol100%100%100%100%
src/facets/usds
   USDSFacet.sol100%100%100%100%
src/facets/weeth
   WEETHFacet.sol100%100%100%100%
   WEETHModule.sol100%100%100%100%
src/facets/wrap-proxy-eth
   WrapProxyETHFacet.sol100%100%100%100%
src/facets/wsteth
   WSTETHFacet.sol100%100%100%100%
src/libraries
   ApproveLib.sol100%100%100%100%

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants