Prevent CI from requiring Atlas on fork PRs (#250)

# Problem

CI was failing on fork PRs because lockfile-only changes (like adding a dependency in one package) would trigger all test targets, which forced `verify-schemas-synced` to run, requiring Atlas Cloud Token that forks don't have access to.

**Example**: Commit f11c32d only changed CLI code and `pnpm-lock.yaml`, but triggered:

1. `core:test:pgtap` (no inputs defined)
2. → `verify-migrations` (dependency)
3. → `verify-schemas-synced` (dependency)
4. → ❌ Atlas required → CI fails on forks

## Solution

Added explicit `inputs` to test targets so they only run when relevant files change, not on lockfile-only changes:

- **Core SQL tests** (`test:pgtap`): Narrow inputs - only schemas, migrations, and test files
- **TypeScript tests** (`test:vitest`, `test:integration`, `test:unit`): Broader inputs - source code and dependency production files

## How It Works

When `pnpm-lock.yaml` changes without source code changes:

1. Test targets check their inputs
2. No matching files changed → use cached results
3. `dependsOn` chain stops → `verify-schemas-synced` never called
4. Atlas not needed → ✅ CI succeeds on forks

## Trade-offs

**Benefit**: Fork contributors can run CI without Atlas/Supabase secrets

**Small risk**: Lockfile-only changes that re-resolve transitive dependencies (e.g., `postgres@3.0.5` → `3.0.6`) won't trigger tests. This is:

- Standard NX practice
- Usually safe (semver patch/minor changes)
- Mitigated by manual testing
- Can force re-run with `--skip-nx-cache`

## Changes

### Modified Files

- `pkgs/core/project.json` - Added narrow inputs to `test:pgtap`, `test:pgtap:watch`
- `pkgs/client/project.json` - Added inputs to `test:vitest`, `test:integration`, `test:unit`, `benchmark`
- `pkgs/edge-worker/project.json` - Added inputs to `test:unit`, `test:integration`, `test:e2e`

### New Documentation

- `.claude/nx_inputs_strategy.md` - Explains input strategy and trade-offs
- `TODO.md` - Future optimization for `dump-realtime-schema` caching

## Testing

Verified with commit f11c32d (CLI + lockfile changes):

- ✅ Core SQL tests would skip (no schemas/migrations changed)
- ✅ Client tests would skip (no client/dependency code changed)
- ✅ Edge-worker tests would skip (no edge-worker/dependency code changed)
- ✅ Atlas never called

---

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Author: jumski

CONTRIBUTING.md

@@ -0,0 +1,42 @@
+# Contributing to pgflow
+
+Thanks for your interest in pgflow! 🎉
+
+## Best Ways to Contribute
+
+The most valuable contributions right now are:
+
+- **Use pgflow and spread the word** 📢 - Share on social media, tell colleagues, write blog posts
+- **Report bugs** via [GitHub Issues](https://github.com/pgflow-dev/pgflow/issues)
+- **Share feedback** on [Discord](https://pgflow.dev/discord/) or GitHub Discussions
+- **Improve docs** - typos, clarity, examples
+
+Helping people discover pgflow is incredibly valuable!
+
+## Before Contributing Code
+
+pgflow has ongoing development and planned features. To avoid duplicate or conflicting work:
+
+1. **Join [Discord](https://pgflow.dev/discord/)** - Discuss your idea first
+2. **Open an issue** - Get feedback before investing time
+3. **Start a discussion** - For bigger ideas or questions
+
+This is especially important for SQL/schema changes, which may conflict with upcoming work and require complex setup.
+
+## Fork PR Limitations
+
+Forks don't have access to repository secrets, so CI may fail for database-related changes. Test locally and mention it in your PR.
+
+## Code Style
+
+- Follow existing patterns in the codebase
+- Run `pnpm nx lint <package>` before committing
+- See `.claude/` directory for detailed guidelines
+
+## Questions?
+
+- **Quick questions**: [Discord](https://pgflow.dev/discord/)
+- **Bug reports**: [GitHub Issues](https://github.com/pgflow-dev/pgflow/issues)
+- **Feature ideas**: [GitHub Discussions](https://github.com/pgflow-dev/pgflow/discussions)
+
+Thanks for helping make pgflow better! 🚀

pkgs/client/project.json

@@ -144,6 +144,7 @@
       "executor": "nx:run-commands",
       "local": true,
       "dependsOn": ["db:ensure", "build"],
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "{projectRoot}",
         "commands": ["vitest run __tests__/integration/"],
@@ -153,6 +154,7 @@
     "test:unit": {
       "executor": "nx:run-commands",
       "dependsOn": ["build"],
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "{projectRoot}",
         "commands": ["vitest run __tests__/"],
@@ -163,6 +165,7 @@
       "executor": "nx:run-commands",
       "local": true,
       "dependsOn": ["db:ensure", "build"],
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "{projectRoot}",
         "commands": ["vitest run __tests__/"],
@@ -177,6 +180,7 @@
       "executor": "nx:run-commands",
       "local": true,
       "dependsOn": ["db:ensure", "build"],
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "{projectRoot}",
         "commands": ["node scripts/performance-benchmark.mjs"],

pkgs/core/project.json

@@ -19,7 +19,11 @@
     "migrationVerificationCache": [
       "{projectRoot}/.nx-inputs/verify-migrations.txt"
     ],
-    "databaseTypes": ["{projectRoot}/src/database-types.ts"]
+    "databaseTypes": ["{projectRoot}/src/database-types.ts"],
+    "pgtapTests": [
+      "{projectRoot}/supabase/tests/**/*.sql",
+      "{projectRoot}/scripts/run-test-with-colors"
+    ]
   },
   "targets": {
     "dump-realtime-schema": {
@@ -200,6 +204,8 @@
       "executor": "nx:run-commands",
       "local": true,
       "dependsOn": ["verify-migrations", "supabase:ensure-started"],
+      "inputs": ["schemas", "migrations", "pgtapTests"],
+      "cache": true,
       "options": {
         "cwd": "{projectRoot}",
         "commands": ["scripts/run-test-with-colors"],
@@ -220,11 +226,12 @@
       "executor": "nx:run-commands",
       "local": true,
       "dependsOn": ["verify-migrations", "supabase:ensure-started"],
+      "inputs": ["schemas", "migrations", "pgtapTests"],
+      "cache": false,
       "options": {
         "cwd": "{projectRoot}",
         "command": "scripts/watch-test"
-      },
-      "cache": false
+      }
     },
     "gen-types": {
       "executor": "nx:run-commands",

pkgs/edge-worker/project.json

@@ -132,6 +132,7 @@
       "dependsOn": ["db:ensure", "^build"],
       "executor": "nx:run-commands",
       "local": true,
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "pkgs/edge-worker",
         "commands": [
@@ -144,6 +145,7 @@
       "dependsOn": ["db:ensure", "^build"],
       "executor": "nx:run-commands",
       "local": true,
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "pkgs/edge-worker",
         "commands": [
@@ -156,6 +158,7 @@
       "executor": "nx:run-commands",
       "dependsOn": ["db:ensure", "^build"],
       "local": true,
+      "inputs": ["default", "^production"],
       "options": {
         "cwd": "pkgs/edge-worker",
         "commands": [