From e93f446ad36e18269e3f3687a16297587219e73d Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 10 Feb 2026 05:10:39 +0000
Subject: [PATCH 1/5] Initial plan


From 43b6d44eb97310a6d840d54291d647c9720dac65 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 10 Feb 2026 05:16:18 +0000
Subject: [PATCH 2/5] fix(build): allow Makefile to work without GOBIN in PATH

Co-authored-by: PavanRaghavendraKulkarni <66769833+PavanRaghavendraKulkarni@users.noreply.github.com>
---
 Makefile | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index 7aea39d..4ac8839 100644
--- a/Makefile
+++ b/Makefile
@@ -101,7 +101,7 @@ ${SQLC_STAMP_FILE}: ${SQLC} ${SQLC_SOURCES} ${SQLC_SOURCE_CONFIG}
 ${PROTO_GO_STAMP_FILE}: ${PROTOC} ${PROTOC_GEN_GO} ${PROTOC_GEN_GRPC} ${PROTO_SOURCES}
 	@echo "Generating internal protobuf code..."
 	@rm -rf internal/gen && mkdir -p internal/gen
-	@${PROTOC} \
+	@PATH="$(GOBIN):$$PATH" ${PROTOC} \
 		${PROTO_SOURCES} \
 		-I=proto \
 		--go_out=internal/gen \
@@ -116,8 +116,7 @@ ${PROTO_GO_STAMP_FILE}: ${PROTOC} ${PROTOC_GEN_GO} ${PROTOC_GEN_GRPC} ${PROTO_SO
 .PHONY: path
 path:
 	@if ! echo "$$PATH" | tr ':' '\n' | grep -qx "$(GOBIN)"; then \
-		echo "Add 'export PATH=$$PATH:$(GOBIN)' to your shell profile to use installed tools."; \
-		exit 1; \
+		echo "Warning: $(GOBIN) is not in PATH. Add 'export PATH=$$PATH:$(GOBIN)' to your shell profile to use installed tools."; \
 	fi
 
 .PHONY: tools

From 87d1bc6c3add3fcacce69e748ef9403487911206 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 10 Feb 2026 05:18:06 +0000
Subject: [PATCH 3/5] fix(build): make uvx optional for lint and provide better
 error messages

Co-authored-by: PavanRaghavendraKulkarni <66769833+PavanRaghavendraKulkarni@users.noreply.github.com>
---
 Makefile | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index 4ac8839..8f85869 100644
--- a/Makefile
+++ b/Makefile
@@ -53,10 +53,14 @@ lint:
 		--show-stats=false --fix
 	@gofmt -l . # Lists files that are likely to be changed by the next command
 	@go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.4.0 fmt
-	@uvx -q sqlfluff fix -q \
-		--disable-progress-bar \
-		--config=internal/server/postgres/sql/.sqlfluff.toml \
-		internal/server/postgres/sql/queries
+	@if command -v uvx > /dev/null 2>&1; then \
+		uvx -q sqlfluff fix -q \
+			--disable-progress-bar \
+			--config=internal/server/postgres/sql/.sqlfluff.toml \
+			internal/server/postgres/sql/queries; \
+	else \
+		echo "Warning: uvx not found, skipping SQL linting"; \
+	fi
 
 .PHONY: bench
 bench: gen
@@ -178,6 +182,11 @@ endef
 export GEN_PYPROJ
 .PHONY: gen.proto.python
 gen.proto.python: ${PROTOC}
+	@if ! command -v uvx > /dev/null 2>&1; then \
+		echo "Error: uvx is required for Python bindings generation but not found."; \
+		echo "Please install uv: https://docs.astral.sh/uv/getting-started/installation/"; \
+		exit 1; \
+	fi
 	@echo "Generating Python client bindings..."
 	@rm -rf gen/python && mkdir -p gen/python/src/dp_sdk
 	@uvx --from 'betterproto[compiler]==2.0.0b7' ${PROTOC} \

From 4b3ab3e49e7e312bbb78e834638f44b6f4ad720d Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 10 Feb 2026 05:18:46 +0000
Subject: [PATCH 4/5] fix(build): redirect uvx warning to stderr in lint target

---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 8f85869..30f7b1a 100644
--- a/Makefile
+++ b/Makefile
@@ -59,7 +59,7 @@ lint:
 			--config=internal/server/postgres/sql/.sqlfluff.toml \
 			internal/server/postgres/sql/queries; \
 	else \
-		echo "Warning: uvx not found, skipping SQL linting"; \
+		echo "Warning: uvx not found, skipping SQL linting" >&2; \
 	fi
 
 .PHONY: bench

From dc91e5b69138c4ee703f7eccd8e8e5a7bb10ceab Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 10 Feb 2026 05:20:30 +0000
Subject: [PATCH 5/5] fix(build): make PATH suggestion consistent with actual
 usage

Co-authored-by: PavanRaghavendraKulkarni <66769833+PavanRaghavendraKulkarni@users.noreply.github.com>
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 30f7b1a..f270fe0 100644
--- a/Makefile
+++ b/Makefile
@@ -120,7 +120,7 @@ ${PROTO_GO_STAMP_FILE}: ${PROTOC} ${PROTOC_GEN_GO} ${PROTOC_GEN_GRPC} ${PROTO_SO
 .PHONY: path
 path:
 	@if ! echo "$$PATH" | tr ':' '\n' | grep -qx "$(GOBIN)"; then \
-		echo "Warning: $(GOBIN) is not in PATH. Add 'export PATH=$$PATH:$(GOBIN)' to your shell profile to use installed tools."; \
+		echo "Warning: $(GOBIN) is not in PATH. Add 'export PATH=$(GOBIN):$$PATH' to your shell profile to use installed tools."; \
 	fi
 
 .PHONY: tools