refactor(cloud): decouple GCP-specific naming, add cloud-neutral API and AWS provider

Phase 1 — Internal naming unification:
- Rename gcp_project_id → project_id, gcs_prefix_uri → cloud_prefix_uri
- Rename gcs_uri → cloud_uri, _gcs_* → _cloud_*, _parse_gcs_uri → _parse_cloud_uri
- All public APIs keep backward-compat aliases with DeprecationWarning
- Env vars: new CLOUD_* names with fallback to old GCS_* names
- URI parsers now accept both gs:// and s3://

Phase 2 — AWS provider:
- Add aws_provider.py implementing all 6 Protocol types
- SecretStore → AWS Secrets Manager, ObjectStore → S3
- DocumentStore → DynamoDB, ComputeDiscovery → EC2
- Register aws in all factory functions (QSL_CLOUD_PROVIDER=aws)

Phase 3 — Downstream repos:
- BinancePlatform: remove hardcoded google.cloud.firestore import
- CryptoLivePoolPipelines: rename gcp_project_id/gcs_bucket/gcs_root_prefix
- All platform repos: update kwarg names to cloud-neutral variants

Tests: 366 passed, 0 failed, +7 new AWS provider registration tests.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Pigbibi

.github/codex_auto_merge_policy.json

@@ -0,0 +1,49 @@
+{
+  "version": 1,
+  "auto_merge_label": "auto-merge-ok",
+  "human_review_label": "human-review-required",
+  "monthly_marker_prefix": "<!-- codex-monthly-remediation:issue-",
+  "blocked_path_patterns": [
+    "(^|/)(\\.env|.*secret.*|.*credential.*|.*token.*|.*private.*|.*\\.pem|.*\\.key)$"
+  ],
+  "risk_policy": {
+    "low": {
+      "prefixes": [
+        "docs/",
+        "tests/"
+      ],
+      "exact": [
+        "README.md",
+        "README.zh-CN.md"
+      ],
+      "reason": "docs/tests/readme-only changes"
+    },
+    "high": {
+      "reason": "source code changes require review"
+    }
+  },
+  "max_changed_files": 30,
+  "max_changed_lines": 2000,
+  "pr_review": {
+    "enabled": true,
+    "block_severities": [
+      "critical",
+      "high"
+    ],
+    "skip_paths": [
+      "docs/**",
+      "**.md",
+      "**.txt",
+      "**.json",
+      "**.csv",
+      "LICENSE",
+      ".github/dependabot*"
+    ],
+    "skip_risk_levels": [
+      "low"
+    ],
+    "timeout_minutes": 20,
+    "max_diff_lines": 2400,
+    "block_on_review_failure": false
+  }
+}

.github/workflows/codex_pr_review.yml

@@ -0,0 +1,53 @@
+name: Codex PR Review
+
+# Always run on PRs to guarantee a check run exists for branch protection.
+# The Python script handles skip logic internally for trivial changes.
+on:
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+permissions:
+  contents: read
+  id-token: write
+  issues: write
+  pull-requests: write
+
+concurrency:
+  group: codex-pr-review-${{ github.event.pull_request.number }}
+  cancel-in-progress: true
+
+jobs:
+  review:
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+    env:
+      FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+
+      - name: Set up Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: "3.11"
+
+      - name: Run Codex PR Review
+        id: review
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
+          ANTHROPIC_MODEL: ${{ vars.ANTHROPIC_MODEL || 'claude-sonnet-4-6' }}
+          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+          OPENAI_MODEL: ${{ vars.OPENAI_MODEL || 'gpt-5.4-mini' }}
+          CODEX_AUDIT_SERVICE_URL: ${{ secrets.CODEX_AUDIT_SERVICE_URL }}
+          CODEX_AUDIT_SERVICE_AUDIENCE: ${{ vars.CODEX_AUDIT_SERVICE_AUDIENCE || 'quant-codex-audit' }}
+        run: python scripts/run_codex_pr_review.py
+
+      - name: Upload review diagnostics
+        if: always()
+        uses: actions/upload-artifact@v7
+        with:
+          name: codex-pr-review-${{ github.event.pull_request.number }}-${{ github.run_id }}
+          path: data/output/codex_pr_review/
+          if-no-files-found: warn

prompts/pr_review.md

@@ -0,0 +1,50 @@
+You are reviewing a pull request for a **production quantitative trading and data pipeline codebase**.
+
+## Review priorities (in order)
+
+1. **Security**: credential leaks, injection vectors, unauthorized data access
+2. **Correctness**: logic errors, wrong calculations, data corruption
+3. **Crash risks**: unhandled exceptions, null pointer dereferences, resource exhaustion
+4. **Data integrity**: silent data loss, incorrect transformations, schema violations
+5. **API compatibility**: breaking changes to function signatures, configuration formats
+6. **Race conditions**: concurrent access to shared state, inconsistent reads
+
+## What NOT to flag
+
+- Code style or formatting preferences
+- Variable/function naming suggestions  
+- Missing type annotations
+- Documentation quality
+- Minor refactoring opportunities
+- Test coverage suggestions
+
+## Severity definitions
+
+| Severity | Definition | Example |
+|----------|-----------|---------|
+| critical | Causes data loss, security breach, or production crash | SQL injection, credential in plaintext, deletion without backup |
+| high | Produces wrong results or breaks downstream systems | Wrong formula, API signature change, resource leak |
+| medium | Degrades reliability or performance under load | Missing error handling, N+1 query, unbounded growth |
+| low | Misleading or confusing but not dangerous | Stale comment, redundant code, unclear intent |
+
+## Output format
+
+Return exactly one JSON object (do not wrap in markdown fences):
+
+```json
+{
+  "summary": "Brief assessment of the PR (1-3 sentences)",
+  "findings": [
+    {
+      "severity": "critical",
+      "category": "security",
+      "file": "path/to/file.py",
+      "line": 42,
+      "description": "What's wrong",
+      "suggestion": "How to fix it"
+    }
+  ]
+}
+```
+
+If no issues found, return `"findings": []`.