Skip to content

Add Claude GitHub workflows for security-focused PR reviews and summaries - #236

Merged
matjazv merged 5 commits into
developmentfrom
prot-81-add-claude-code-review-to-lisk-contracts-repository
Feb 3, 2026
Merged

matjazv merged 5 commits into
developmentfrom
prot-81-add-claude-code-review-to-lisk-contracts-repository

Conversation

@matjazv

@matjazv matjazv commented Feb 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This PR adds three GitHub Actions workflows for Claude AI integration, providing automated security-focused code reviews, PR summaries, and interactive assistance tailored for this smart contracts repository.

GitHub Actions Automation:

  • Claude PR Code Review Workflow
    Adds .github/workflows/claude-pr-review-caller.yml to enable security-focused code review when /review is commented or the needs-review label is applied. The review prompt is specifically tailored for Solidity smart contracts that hold real funds, covering reentrancy, access control, fund theft vectors, flash loan attacks, and oracle manipulation risks.

  • Claude PR Summary Workflow
    Adds .github/workflows/claude-pr-summary-caller.yml to automatically generate and update a concise technical summary in the PR description upon PR open/sync/reopen events. Uses the cost-efficient haiku model.

  • Claude Code Interactive Workflow
    Adds .github/workflows/claude-interactive-caller.yml to allow interactive Claude responses when @claude is mentioned in PRs, issues, or comments. Includes Foundry and Node.js tooling for running tests and TypeScript checks.

@matjazv matjazv self-assigned this Feb 3, 2026
@matjazv
matjazv force-pushed the prot-81-add-claude-code-review-to-lisk-contracts-repository branch from 00672e0 to cfe2788 Compare February 3, 2026 10:52
matjazv and others added 4 commits February 3, 2026 12:01
Fix CI failure due to deleted commit reference. The previous commit
(143ac6885) no longer exists in the remote repository. Updated to
aa1a0a46877b (2025-10-20) which is before the contracts were moved
to evm-adapters package.

Also removed submodules: recursive from claude.yml as it's not needed.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…atibility

The redstone-oracles-monorepo imports AggregatorV3Interface from
@chainlink/contracts/src/v0.8/shared/interfaces/, but the previous
submodule commit (3cf93c9f84) predates the shared/interfaces directory
reorganization. Update to commit 811fe603ae67 (2024-05-13) which has
the required interface files in the expected path.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@matjazv
matjazv requested review from ricott1 and shuse2 February 3, 2026 12:27
@matjazv
matjazv marked this pull request as draft February 3, 2026 12:30
@matjazv
matjazv marked this pull request as ready for review February 3, 2026 12:30
@matjazv

matjazv commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

/review

@matjazv
matjazv merged commit c5c680d into development Feb 3, 2026
4 checks passed
@matjazv
matjazv deleted the prot-81-add-claude-code-review-to-lisk-contracts-repository branch February 3, 2026 14:17
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.

3 participants