From 21630fdcc22bcc3a8a93b0c6463c273e8aff9163 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com> Date: Tue, 13 Jan 2026 08:18:29 +0000 Subject: [PATCH 1/3] docs: create comprehensive README Co-authored-by: Ben Kremer --- README.md | 148 +++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 146 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 0b10fb3..7d7f9db 100644 --- a/README.md +++ b/README.md @@ -1,2 +1,146 @@ -# github-actions -Shared actions and workflows +# WalletConnect GitHub Actions + +Shared GitHub Actions and reusable workflows for infrastructure, deployments, and AI-powered code review. + +## Directory Structure + +``` +actions/ +├── actions/ # High-level composite actions +│ ├── deploy-terraform/ # Complete Terraform deployment +│ ├── deploy-ecs/ # ECS service deployment +│ ├── plan-terraform/ # Terraform planning +│ └── fmt-check-terraform/ # Terraform formatting check +├── terraform/ # Terraform primitives +│ ├── init/ # Initialize Terraform +│ ├── plan/ # Generate execution plan +│ ├── apply/ # Apply changes +│ ├── validate/ # Validate configuration +│ ├── select-workspace/ # Switch workspace +│ └── check-fmt/ # Format checking +├── aws/ # AWS-specific actions +│ ├── ecs/ # ECS operations (deploy-image, get-task-image) +│ └── grafana/ # Grafana key management +├── github/ # GitHub utilities +│ ├── paths-filter/ # Path-based filtering +│ ├── branch-name/ # Branch name utilities +│ ├── latest-release-version/ # Release versioning +│ ├── update-release-version/ # Update release versions +│ └── cta-assistant/ # CTA assistant +├── claude/ # AI-powered review +│ ├── agent/ # Generic Claude agent handling +│ ├── auto-review/ # PR code review with inline comments +│ └── terraform-plan-review/ # Terraform plan analysis +└── deploy-window/ # Deploy timing controls +``` + +## Quick Start + +### Deploy Terraform Infrastructure + +```yaml +- uses: WalletConnect/actions/actions/deploy-terraform@master + with: + environment: staging + aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }} + aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} + aws-region: us-east-1 +``` + +### Deploy ECS Service + +```yaml +- uses: WalletConnect/actions/actions/deploy-ecs@master + with: + aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }} + aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} + aws-region: us-east-1 + cluster-name: my-cluster + service-name: my-service + task-definition-name: my-task + image-name: my-image:v1.0.0 +``` + +### AI Code Review + +```yaml +- uses: WalletConnect/actions/claude/auto-review@master + with: + anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} + model: claude-sonnet-4-5-20250929 + project_context: | + This is a payment processing service using Node.js and TypeScript. +``` + +## Key Conventions + +### Composite Actions +- All actions use `using: composite` with bash steps +- Inputs use kebab-case naming +- Outputs written via `>> $GITHUB_OUTPUT` +- Use `working-directory` field over `cd` in scripts + +### Terraform Usage +- Set environment via `TF_WORKSPACE` env var +- Run non-interactively with `-no-color` and `TF_INPUT=0` +- Apply with `-auto-approve` +- Var files located at `vars/{environment}.tfvars` + +### AWS Usage +- Configure credentials with `aws-actions/configure-aws-credentials@v4` +- Always require explicit region input +- Grafana keys have short TTL, cleanup via `always()` condition + +### Security +- Mask sensitive values with `::add-mask::` +- External domains limited to: reown.com, walletconnect.com, walletconnect.org +- Cleanup temporary credentials/keys in `always()` steps + +### Action References +- Consumers should pin to full 40-char commit SHA, not branch names +- Internal action refs use `@master` for this repo + +## Development + +### Auto-Review Tests + +```bash +cd claude/auto-review +pnpm install # Install dependencies +pnpm test # Run tests +pnpm test:watch # Watch mode +pnpm test:coverage # With coverage +``` + +### Pre-commit Hooks + +```bash +pre-commit install # Install hooks +pre-commit run --all-files # Run all checks +``` + +## Claude Auto-Review + +The `claude/auto-review` action provides automated PR reviews with inline findings comments. + +**Key components:** +- `action.yml` - Main composite action with dynamic prompt generation +- `scripts/extract-findings-from-comment.js` - Parses AI comment to extract structured findings +- `scripts/comment-pr-findings.js` - Posts inline PR review comments from findings.json +- `scripts/lib/github-utils.js` - Shared utilities (gh CLI wrapper, context loading) + +**Flow:** AI agent reviews PR → posts comment → extract-findings parses issues → comment-pr-findings posts inline comments + +## Workflows + +- `.github/workflows/claude.yml` - General AI agent invocation (@claude mentions) +- `.github/workflows/claude-review.yml` - Auto-review on PR open or @claude review comment +- `.github/workflows/test-claude-auto-review.yml` - CI for auto-review scripts + +## Contributing + +See [CLAUDE.md](CLAUDE.md) for detailed architecture documentation and AI agent guidance. + +## License + +[Add license information] From 5370ec36dace48c2287bb303bafc90a753572c14 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com> Date: Tue, 13 Jan 2026 08:22:42 +0000 Subject: [PATCH 2/3] docs: remove Contributing and License sections Co-authored-by: Ben Kremer --- README.md | 8 -------- 1 file changed, 8 deletions(-) diff --git a/README.md b/README.md index 7d7f9db..290b7aa 100644 --- a/README.md +++ b/README.md @@ -136,11 +136,3 @@ The `claude/auto-review` action provides automated PR reviews with inline findin - `.github/workflows/claude.yml` - General AI agent invocation (@claude mentions) - `.github/workflows/claude-review.yml` - Auto-review on PR open or @claude review comment - `.github/workflows/test-claude-auto-review.yml` - CI for auto-review scripts - -## Contributing - -See [CLAUDE.md](CLAUDE.md) for detailed architecture documentation and AI agent guidance. - -## License - -[Add license information] From 18b408cd8833f50d549dcfe734a426d92a4467f0 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com> Date: Tue, 13 Jan 2026 08:24:51 +0000 Subject: [PATCH 3/3] docs: fix directory structure diagram Co-authored-by: Ben Kremer --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 290b7aa..3f20e12 100644 --- a/README.md +++ b/README.md @@ -5,7 +5,7 @@ Shared GitHub Actions and reusable workflows for infrastructure, deployments, an ## Directory Structure ``` -actions/ +. ├── actions/ # High-level composite actions │ ├── deploy-terraform/ # Complete Terraform deployment │ ├── deploy-ecs/ # ECS service deployment