From b5c7f683b0dd07de2a29daaf5fbabcf08d8fa774 Mon Sep 17 00:00:00 2001
From: William Mantly <wmantly@gmail.com>
Date: Wed, 31 Dec 2025 16:54:49 -0500
Subject: [PATCH] Set up github

---
 .github/workflows/README.md    | 83 ++++++++++++++++++++++++++++++++++
 .github/workflows/pr-tests.yml | 61 +++++++++++++++++++++++++
 2 files changed, 144 insertions(+)
 create mode 100644 .github/workflows/README.md
 create mode 100644 .github/workflows/pr-tests.yml

diff --git a/.github/workflows/README.md b/.github/workflows/README.md
new file mode 100644
index 0000000..72036cf
--- /dev/null
+++ b/.github/workflows/README.md
@@ -0,0 +1,83 @@
+# CI/CD Workflows
+
+## Pull Request Testing
+
+The `pr-tests.yml` workflow automatically runs on every pull request to the `master` branch.
+
+### What it does:
+
+1. **Multi-version testing**: Tests run on Node.js 18.x, 20.x, and 22.x
+2. **Comprehensive coverage**: Runs unit tests, integration tests, and full test suite
+3. **Blocks merging**: PRs cannot be merged until all tests pass on all Node versions
+
+### Workflow triggers:
+
+- Opening a pull request to `master`
+- Pushing new commits to an existing PR
+- Updates to PR branches
+
+### Test jobs:
+
+1. **Unit tests** - Tests isolated components (callback queue, host lookup, unix socket)
+2. **Integration tests** - Tests DNS provider contracts
+3. **Full test suite** - Complete test coverage
+
+### Branch protection:
+
+The `master` branch is protected and requires:
+- All tests must pass before merging
+- Status check: `test` job must succeed
+- Applies to all contributors (admins can override)
+
+## Running tests locally:
+
+Before creating a PR, run tests locally to catch issues early:
+
+```bash
+cd nodejs
+npm run test:unit         # Run unit tests only
+npm run test:integration  # Run integration tests only
+npm test                  # Run all tests
+npm run test:watch        # Watch mode for development
+```
+
+## Adding new workflows:
+
+To add new CI/CD workflows:
+
+1. Create a new `.yml` file in `.github/workflows/`
+2. Define triggers, jobs, and steps
+3. Test the workflow by creating a PR
+4. Add status check to branch protection if required for merging
+
+## Troubleshooting:
+
+**Tests pass locally but fail in CI:**
+- Check Node.js version compatibility (workflow tests 18.x, 20.x, 22.x)
+- Verify all dependencies are in package.json (not installed globally)
+- Check for environment-specific issues (paths, permissions)
+
+**Branch protection preventing merge:**
+- Ensure all required status checks pass
+- Check workflow logs for detailed error messages
+- Re-run failed jobs if transient failures occurred
+
+**Modifying branch protection:**
+
+```bash
+# View current protection settings
+gh api repos/theta42/proxy/branches/master/protection
+
+# Update required status checks
+gh api repos/theta42/proxy/branches/master/protection \
+  -X PUT \
+  -F required_status_checks[contexts][]=test \
+  -F required_status_checks[contexts][]=your-new-check
+```
+
+## Workflow status:
+
+View workflow runs and status:
+- GitHub UI: https://github.com/theta42/proxy/actions
+- CLI: `gh run list`
+- PR checks: Automatically shown on PR page
diff --git a/.github/workflows/pr-tests.yml b/.github/workflows/pr-tests.yml
new file mode 100644
index 0000000..976adf2
--- /dev/null
+++ b/.github/workflows/pr-tests.yml
@@ -0,0 +1,61 @@
+name: Pull Request Tests
+
+# Run tests on pull requests to master and when pushing to PRs
+on:
+  pull_request:
+    branches:
+      - master
+  push:
+    branches-ignore:
+      - master
+
+jobs:
+  test:
+    name: Run Tests
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [18.x, 20.x, 22.x]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'npm'
+          cache-dependency-path: nodejs/package-lock.json
+
+      - name: Install dependencies
+        working-directory: ./nodejs
+        run: npm ci
+
+      - name: Run unit tests
+        working-directory: ./nodejs
+        run: npm run test:unit
+
+      - name: Run integration tests
+        working-directory: ./nodejs
+        run: npm run test:integration
+
+      - name: Run all tests
+        working-directory: ./nodejs
+        run: npm test
+
+  test-summary:
+    name: Test Summary
+    runs-on: ubuntu-latest
+    needs: test
+    if: always()
+
+    steps:
+      - name: Check test results
+        run: |
+          if [ "${{ needs.test.result }}" != "success" ]; then
+            echo "Tests failed. PR cannot be merged."
+            exit 1
+          fi
+          echo "All tests passed successfully!"