ABN-298: Extended net terms, surcharge config, and dev tooling #184
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Claude Code Review | |
| on: | |
| pull_request: | |
| types: [opened, synchronize] | |
| issue_comment: | |
| types: [created] | |
| pull_request_review_comment: | |
| types: [created] | |
| pull_request_review: | |
| types: [submitted] | |
| permissions: | |
| contents: write | |
| actions: read | |
| issues: write | |
| id-token: write | |
| pull-requests: write | |
| jobs: | |
| code-review: | |
| runs-on: ${{ vars.RUNNER_STANDARD }} | |
| if: | | |
| ( | |
| github.event_name == 'pull_request' && | |
| vars.CLAUDE_REVIEW_CONFIG != '' && | |
| fromJSON(vars.CLAUDE_REVIEW_CONFIG)[format('{0}:{1}', github.base_ref, github.event.action)] == true | |
| ) || | |
| (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) || | |
| (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) || | |
| (github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) | |
| steps: | |
| - uses: two-inc/claude-code-reviewer@main | |
| env: | |
| LINEAR_API_KEY: ${{ secrets.LINEAR_API_KEY }} | |
| with: | |
| claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} | |
| github_app_id: ${{ vars.TWO_INC_APP_ID }} | |
| github_app_private_key: ${{ secrets.TWO_INC_APP_PRIVATE_KEY }} | |
| extra_prompt: | | |
| ## Magento Plugin Specific Guidelines: | |
| For this PHP Magento 2 plugin, pay special attention to: | |
| - **Magento conventions**: Follow Magento 2 coding standards and module structure | |
| - **Dependency injection**: Use Magento DI patterns, avoid direct instantiation | |
| - **Database schema**: Review db_schema.xml changes for proper declarative schema | |
| - **API contracts**: Verify API interfaces follow backward compatibility | |
| - **Plugin/Observer pattern**: Check for proper plugin and event observer implementation | |
| - **Security**: Validate input sanitization and ACL permissions | |
| - **Performance**: Review query efficiency and caching strategies | |
| - **Composer**: Check composer.json dependencies and version constraints |