View table name on tablet page for datashard tablet and topic name on tablet page for PersQueue and PersQueueReadBalancer tablets #371
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: | |
| workflow_dispatch: | |
| issue_comment: | |
| types: [created] | |
| jobs: | |
| claude-review: | |
| if: | | |
| github.event_name == 'workflow_dispatch' || | |
| (github.event_name == 'issue_comment' && | |
| github.event.issue.pull_request && | |
| contains(github.event.comment.body, '/claude_review')) | |
| runs-on: ubuntu-latest | |
| permissions: | |
| contents: read | |
| pull-requests: read | |
| issues: read | |
| id-token: write | |
| steps: | |
| - name: Get PR details | |
| id: pr-details | |
| if: github.event_name == 'issue_comment' | |
| env: | |
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |
| run: | | |
| PR_NUMBER="${{ github.event.issue.number }}" | |
| PR_DATA=$(curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \ | |
| "https://api.github.com/repos/${{ github.repository }}/pulls/${PR_NUMBER}") | |
| PR_HEAD_SHA=$(echo "$PR_DATA" | jq -r '.head.sha') | |
| echo "pr_head_sha=$PR_HEAD_SHA" >> $GITHUB_OUTPUT | |
| echo "pr_number=$PR_NUMBER" >> $GITHUB_OUTPUT | |
| - name: Checkout repository | |
| uses: actions/checkout@v4 | |
| with: | |
| # For workflow_dispatch, checkout default branch | |
| # For issue_comment on PR, checkout the PR head commit | |
| ref: ${{ github.event_name == 'issue_comment' && steps.pr-details.outputs.pr_head_sha || github.ref }} | |
| fetch-depth: 0 | |
| - name: Run Claude Code Review | |
| id: claude-review | |
| uses: anthropics/claude-code-action@beta | |
| with: | |
| claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} | |
| # Specify model for consistency with main workflow | |
| model: 'claude-opus-4-20250514' | |
| # Direct prompt for automated review (no @claude mention needed) | |
| direct_prompt: | | |
| Review this YDB Embedded UI pull request focusing on: | |
| 1. **Code Standards**: | |
| - TypeScript type safety and proper interface usage (prefixed with 'T') | |
| - React best practices and Gravity UI component usage | |
| - Redux Toolkit patterns with RTK Query | |
| - BEM naming convention with cn() utility | |
| 2. **Critical Requirements**: | |
| - All user-facing text must use i18n (no hardcoded strings) | |
| - API calls must go through window.api, never direct calls | |
| - No state mutations in RTK Query | |
| - Route params validation (React Router v5) | |
| 3. **Performance & Security**: | |
| - Virtual scrolling for large tables (PaginatedTable) | |
| - React.memo for expensive renders | |
| - No exposed secrets or keys | |
| - Proper error handling and loading states | |
| 4. **Testing**: | |
| - Unit tests in __test__ directories | |
| - E2E tests follow page object pattern | |
| Be constructive and reference specific files with file_path:line_number format. | |
| # Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR | |
| use_sticky_comment: true | |
| # Optional: Customize review based on file types | |
| # direct_prompt: | | |
| # Review this PR focusing on: | |
| # - For TypeScript files: Type safety and proper interface usage | |
| # - For API endpoints: Security, input validation, and error handling | |
| # - For React components: Performance, accessibility, and best practices | |
| # - For tests: Coverage, edge cases, and test quality | |
| # Optional: Different prompts for different authors | |
| # direct_prompt: | | |
| # ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' && | |
| # 'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' || | |
| # 'Please provide a thorough code review focusing on our coding standards and best practices.' }} | |
| # Add specific tools for running tests or linting | |
| allowed_tools: | | |
| Bash(npm run lint) | |
| Bash(npm run typecheck) | |
| Bash(npm test -- --no-watch) | |
| Bash(npm run unused) | |
| Bash(git diff) | |
| Bash(git log) | |
| # Optional: Skip review for certain conditions | |
| # if: | | |
| # !contains(github.event.pull_request.title, '[skip-review]') && | |
| # !contains(github.event.pull_request.title, '[WIP]') |