Add Claude PR review caller workflow (pilot) #1
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 PR Review | ||
| on: | ||
| pull_request: | ||
| types: [opened, ready_for_review, reopened] | ||
| jobs: | ||
| review: | ||
| uses: openmrs/openmrs-contrib-gha-workflows/.github/workflows/claude-pr-review.yml@main | ||
|
Check failure on line 9 in .github/workflows/claude-review.yml
|
||
| secrets: | ||
| ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }} | ||
| with: | ||
| extra_instructions: | | ||
| This is a TypeScript/React OpenMRS frontend repository (part of OpenMRS 3.x microfrontend architecture). Pay particular attention to: | ||
| - i18n: user-facing strings must use the t() function from useTranslation, not hard-coded text. Flag hard-coded English strings in JSX, alerts, and toasts. | ||
| - Type safety: avoid `any` in new code; prefer specific types or `unknown` with type guards. Flag new uses of `any`. | ||
| - Data fetching: use SWR/React Query patterns consistent with the rest of the codebase; don't introduce ad-hoc fetch wrappers. | ||
| - Public exports: flag breaking changes to anything exported from index.ts or package entry points. | ||
| - Testing: use @testing-library/react idioms; avoid testing implementation details. Flag tests that mock OpenMRS framework primitives instead of using the official testing utilities. | ||
| - Performance: flag obvious re-render hazards (new object/array literals as hook deps, missing useMemo/useCallback on hot paths) but skip nits. | ||