Merge pull request #1 from vllm-project/main

Author: yuluo-yx

.github/workflows/auto-merge.yml

@@ -137,6 +137,10 @@ jobs:
             }
 
             commentBody += '---\n';
+            commentBody += '<div align="center">\n';
+            commentBody += '<img src="https://raw.githubusercontent.com/vllm-project/semantic-router/main/website/static/img/repo.png" alt="vLLM" width="80%"/>';
+            commentBody += '</div>\n\n';
+            commentBody += '## 🎉 Thanks for your contributions!\n\n';
             commentBody += '*This comment was automatically generated based on the OWNER files in the repository.*';
 
             // Get existing comments

.github/workflows/owner-notification.yml

@@ -51,6 +51,9 @@ jobs:
           ~/go/pkg/mod
         key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
 
+    - name: Check go mod tidy
+      run: make check-go-mod-tidy
+
     - name: Build Rust library
       run: make rust
 

.github/workflows/test-and-build.yml

@@ -222,6 +222,10 @@ pre-commit run --all-files
 - Use meaningful variable and function names
 - Add comments for exported functions and types
 - Write unit tests for new functionality
+- **Keep Go modules tidy:** Run `go mod tidy` in the appropriate directory after adding or removing dependencies
+  - For `candle-binding`: `cd candle-binding && go mod tidy`
+  - For `src/semantic-router`: `cd src/semantic-router && go mod tidy`
+  - The CI will automatically check that `go.mod` and `go.sum` files are tidy using `make check-go-mod-tidy`
 
 ### Rust Code
 - Follow Rust formatting (`cargo fmt`)
@@ -242,6 +246,11 @@ pre-commit run --all-files
    python e2e-tests/run_all_tests.py
    ```
 
+   The `make test` command includes:
+   - `go vet` for static analysis
+   - `check-go-mod-tidy` for Go module dependency verification
+   - Unit tests for all components
+
 2. **Create a pull request** with:
    - Clear description of changes
    - Reference to any related issues

CONTRIBUTING.md

@@ -55,6 +55,30 @@ run-envoy:
 	@echo "Starting Envoy..."
 	func-e run --config-path config/envoy.yaml --component-log-level "ext_proc:trace,router:trace,http:trace"
 
+# Run go vet for all Go modules
+vet:
+	@echo "Running go vet..."
+	@cd candle-binding && go vet ./...
+	@cd src/semantic-router && go vet ./...
+
+# Check go mod tidy for all Go modules
+check-go-mod-tidy:
+	@echo "Checking go mod tidy for all Go modules..."
+	@echo "Checking candle-binding..."
+	@cd candle-binding && go mod tidy && \
+		(git diff --exit-code go.mod 2>/dev/null || (echo "ERROR: go.mod file is not tidy in candle-binding. Please run 'go mod tidy' in candle-binding directory and commit the changes." && git diff go.mod && exit 1)) && \
+		(test ! -f go.sum || git diff --exit-code go.sum 2>/dev/null || (echo "ERROR: go.sum file is not tidy in candle-binding. Please run 'go mod tidy' in candle-binding directory and commit the changes." && git diff go.sum && exit 1))
+	@echo "✅ candle-binding go mod tidy check passed"
+	@echo "Checking src/semantic-router..."
+	@cd src/semantic-router && go mod tidy && \
+		if ! git diff --exit-code go.mod go.sum; then \
+			echo "ERROR: go.mod or go.sum files are not tidy in src/semantic-router. Please run 'go mod tidy' in src/semantic-router directory and commit the changes."; \
+			git diff go.mod go.sum; \
+			exit 1; \
+		fi
+	@echo "✅ src/semantic-router go mod tidy check passed"
+	@echo "✅ All go mod tidy checks passed"
+
 # Test the Rust library
 test-binding: rust
 	@echo "Running Go tests with static library..."
@@ -86,7 +110,7 @@ test-semantic-router: build-router
 		cd src/semantic-router && CGO_ENABLED=1 go test -v ./...
 
 # Test the Rust library and the Go binding
-test: download-models test-binding test-semantic-router
+test: vet check-go-mod-tidy download-models test-binding test-semantic-router
 
 # Clean built artifacts
 clean: