Merge branch 'main' into fix/standalone-build-and-test-hangs

Resolve conflict by keeping e2e.yml from main (PR #108 added functional e2e.yml).

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>

Author: hyperpolymath

.github/workflows/e2e.yml

@@ -0,0 +1,109 @@
+# SPDX-License-Identifier: MPL-2.0
+# E2E + Point-to-Point + Unit tests for echidnabot.
+#
+# Taxonomy: standards/testing-and-benchmarking/TESTING-TAXONOMY.adoc
+#   Part I §1 Unit · §2 Point-to-Point · §3 End-to-End
+# Provenance rule (TESTING-TAXONOMY.adoc, "draw from proven-tests-and-benches
+# FIRST"): the categories and their meanings come from that suite's Taxonomy
+# module. The assertions here are echidnabot's own, because the proven suite has
+# no echidnabot subject to reuse -- see the gap note at the foot of this file.
+#
+# ⚠ WHY THIS WORKFLOW CHECKS OUT TWO REPOSITORIES
+# echidnabot CANNOT BUILD STANDALONE. Its Cargo.toml declares
+#   gitbot-shared-context = { path = "../../shared-context" }
+# which lives in hyperpolymath/gitbot-fleet. A fresh clone of echidnabot alone
+# fails dependency resolution outright:
+#   "failed to load source for dependency gitbot-shared-context"
+# Verified 2026-07-29. The layout below (gitbot-fleet/bots/echidnabot) is the one
+# that resolves; it is also how the fleet vendors this bot.
+#
+# ⚠ WHY `cargo test --lib` IS NOT RUN HERE
+# The unit tests inside src/lib.rs HANG -- measured: no completion in 20 minutes,
+# exit 124. Every INTEGRATION binary passes in about two seconds (91 tests). This
+# workflow therefore runs the integration suites explicitly, by name, rather than
+# a bare `cargo test` that would hang the runner. The hang is recorded as debt,
+# not hidden: the job below states it in its summary on every run.
+name: E2E + P2P + Unit
+on:
+  push:
+    branches: [main, master]
+    paths: ['src/**', 'tests/**', 'Cargo.toml', 'Cargo.lock', '.github/workflows/e2e.yml']
+  pull_request:
+    branches: [main, master]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
+jobs:
+  tests:
+    name: E2E — Unit, P2P and End-to-End
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+    permissions:
+      contents: read
+    steps:
+      # gitbot-fleet supplies shared-context; echidnabot must sit at bots/echidnabot.
+      - name: Checkout gitbot-fleet (provides ../../shared-context)
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        with:
+          repository: hyperpolymath/gitbot-fleet
+          path: fleet
+
+      - name: Checkout echidnabot into fleet/bots/echidnabot
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        with:
+          path: fleet/bots/echidnabot
+
+      - name: Prove the layout resolves before building
+        working-directory: fleet/bots/echidnabot
+        run: |
+          set -euo pipefail
+          # Fails loudly if the two-repo layout is ever broken, rather than
+          # producing a confusing compile error thousands of lines later.
+          cargo metadata --format-version 1 >/dev/null
+          echo "✅ dependency graph resolves (gitbot-shared-context found)"
+
+      - uses: Swatinem/rust-cache@f0deed1e0edfc6a9be95417288c0e1099b1eeec3 # v2.7.7
+        with:
+          workspaces: fleet/bots/echidnabot
+
+      - name: Unit + Point-to-Point + End-to-End
+        working-directory: fleet/bots/echidnabot
+        run: |
+          set -euo pipefail
+          # Named explicitly. A bare `cargo test` also runs --lib, which hangs
+          # (see the header). Each binary maps to a taxonomy category:
+          #   smoke             -> Part I §8  Smoke
+          #   property_tests    -> Part I §9  Property-based
+          #   lifecycle         -> Part I §7  Lifecycle
+          #   integration_tests -> Part I §2  Point-to-Point
+          #   seam_test         -> Part I §3  End-to-End (webhook -> corpus)
+          for t in smoke property_tests lifecycle integration_tests seam_test; do
+            echo "::group::$t"
+            cargo test --test "$t" -- --test-threads=4
+            echo "::endgroup::"
+          done
+
+      - name: Summary
+        if: always()
+        run: |
+          {
+            echo "### E2E + P2P + Unit"
+            echo
+            echo "| Suite | Taxonomy category |"
+            echo "|---|---|"
+            echo "| \`smoke\` | Part I §8 Smoke |"
+            echo "| \`property_tests\` | Part I §9 Property-based |"
+            echo "| \`lifecycle\` | Part I §7 Lifecycle |"
+            echo "| \`integration_tests\` | Part I §2 Point-to-Point |"
+            echo "| \`seam_test\` | Part I §3 End-to-End |"
+            echo
+            echo "**Known debt:** \`cargo test --lib\` hangs (>20 min, exit 124) and is"
+            echo "deliberately NOT run here. The integration suites above complete in ~2s."
+            echo "This is stated rather than silently excluded."
+          } >> "$GITHUB_STEP_SUMMARY"