Merge pull request #176 from benthecarman/from-scratch-grpc

Replace REST and RabbitMQ with gRPC

Author: tnull

.github/workflows/check-protos.yml

@@ -3,10 +3,10 @@ name: Check Generated Protos
 on:
   push:
     paths:
-      - 'ldk-server-protos/**'
+      - 'ldk-server-grpc/**'
   pull_request:
     paths:
-      - 'ldk-server-protos/**'
+      - 'ldk-server-grpc/**'
   workflow_dispatch:
 
 permissions:
@@ -28,12 +28,12 @@ jobs:
       - name: Install protoc
         run: sudo apt-get install -y protobuf-compiler
       - name: Generate protos
-        run: RUSTFLAGS="--cfg genproto" cargo build -p ldk-server-protos
+        run: RUSTFLAGS="--cfg genproto" cargo build -p ldk-server-grpc
       - name: Format generated code
         run: rustup component add rustfmt && cargo fmt --all
       - name: Check for differences
         run: |
           if ! git diff --exit-code; then
-            echo "error: Generated protobuf files are out of date. Run: RUSTFLAGS=\"--cfg genproto\" cargo build -p ldk-server-protos && cargo fmt --all"
+            echo "error: Generated protobuf files are out of date. Run: RUSTFLAGS=\"--cfg genproto\" cargo build -p ldk-server-grpc && cargo fmt --all"
             exit 1
           fi

…ws/integration-tests-events-rabbitmq.yml .github/workflows/e2e-tests.yml.github/workflows/integration-tests-events-rabbitmq.yml renamed to .github/workflows/e2e-tests.yml .github/workflows/integration-tests-events-rabbitmq.yml renamed to .github/workflows/e2e-tests.yml

@@ -1,4 +1,4 @@
-name: Integration Tests
+name: End-to-End Tests
 
 on: [ push, pull_request ]
 
@@ -10,23 +10,9 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  integration-tests:
+  e2e-tests:
     runs-on: ubuntu-latest
 
-    services:
-      rabbitmq:
-        image: rabbitmq:3
-        env:
-          RABBITMQ_DEFAULT_USER: guest
-          RABBITMQ_DEFAULT_PASS: guest
-        ports:
-          - 5672:5672
-        options: >-
-          --health-cmd "rabbitmqctl node_health_check"
-          --health-interval 10s
-          --health-timeout 5s
-          --health-retries 5
-
     steps:
       - name: Checkout code
         uses: actions/checkout@v6
@@ -48,11 +34,6 @@ jobs:
       - name: Set bitcoind environment variable
         run: echo "BITCOIND_EXE=$( pwd )/bin/bitcoind-${{ runner.os }}-${{ runner.arch }}" >> "$GITHUB_ENV"
 
-      - name: Run RabbitMQ integration tests
-        run: cargo test --features integration-tests-events-rabbitmq --verbose --color=always -- --nocapture
-        env:
-          RUST_BACKTRACE: 1
-
       - name: Run end-to-end tests
         run: cargo test --manifest-path e2e-tests/Cargo.toml --verbose --color=always -- --test-threads=4 --nocapture
         env:

CLAUDE.md

@@ -9,7 +9,7 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for build commands, testing, code style,
 - **ldk-server** - Main daemon server (entry point: `src/main.rs`)
 - **ldk-server-cli** - CLI client using clap
 - **ldk-server-client** - Reqwest-based client library
-- **ldk-server-protos** - Protocol buffer definitions and generated Rust code
+- **ldk-server-grpc** - Protocol buffer definitions, generated Rust code, and shared gRPC primitives
 
 ## Development Rules
 

CONTRIBUTING.md

@@ -40,13 +40,13 @@ cargo clippy --all-features -- -D warnings -A clippy::drop_non_drop  # Lint (CI
 ## Protocol Buffer Generation
 
 ```bash
-RUSTFLAGS="--cfg genproto" cargo build -p ldk-server-protos
+RUSTFLAGS="--cfg genproto" cargo build -p ldk-server-grpc
 cargo fmt --all
 ```
 
 ## Adding a New API Endpoint
 
-1. Define request/response messages in `ldk-server-protos/src/proto/api.proto`
+1. Define request/response messages in `ldk-server-grpc/src/proto/api.proto`
 2. Regenerate protos (see above)
 3. Create handler in `ldk-server/src/api/` (follow existing patterns)
 4. Add route in `ldk-server/src/service.rs`