Merge pull request #25 from GrayCodeAI/chore/sync-quality-hardening-and-codegen

chore(sync): quality-hardening + SDK codegen tools, resolution logic, deps and tests

Author: Patel230

Makefile

@@ -23,11 +23,12 @@ GOLANGCI    := $(GOBIN_DIR)/golangci-lint
 GOFUMPT     := $(GOBIN_DIR)/gofumpt
 GOIMPORTS   := $(GOBIN_DIR)/goimports
 GOVULNCHECK := $(GOBIN_DIR)/govulncheck
+OAPICODEGEN := $(GOBIN_DIR)/oapi-codegen
 
 # ---------------------------------------------------------------------------
 # Phony declarations (alphabetical).
 # ---------------------------------------------------------------------------
-.PHONY: all bench boundary-guard build ci clean cover fmt help lint lint-fix \
+.PHONY: all bench boundary-guard build ci clean cover fmt gen help lint lint-fix \
         security test test-10x test-race tidy version vet
 
 # ---------------------------------------------------------------------------
@@ -96,7 +97,7 @@ tidy: ## Tidy go.mod / go.sum.
 # ---------------------------------------------------------------------------
 # Composite gate used by CI and pre-push.
 # ---------------------------------------------------------------------------
-ci: tidy fmt vet boundary-guard lint test-race security ## Run everything CI runs.
+ci: tidy gen fmt vet boundary-guard lint test-race security ## Run everything CI runs.
 	@echo "All CI checks passed."
 
 # ---------------------------------------------------------------------------
@@ -107,6 +108,10 @@ version: ## Print the version that will be embedded.
 	@echo "Commit:  $(COMMIT)"
 	@echo "Date:    $(DATE)"
 
+gen: ## Generate code from the OpenAPI spec (requires oapi-codegen).
+	go generate ./internal/spec/
+	@echo "Code generation complete."
+
 clean: ## Remove build artefacts.
 	rm -rf coverage.out coverage.html
 	go clean -testcache

agent_test.go

@@ -55,7 +55,7 @@ func TestAgent_ChatWithTools(t *testing.T) {
 			json.NewEncoder(w).Encode(ChatWithToolsResponse{
 				ChatResponse: ChatResponse{SessionID: "s-tools"},
 				ToolCalls: []ToolCall{
-					{ID: "tc-1", Name: "greet", Arguments: `{"name":"world"}`},
+					{ID: "tc-1", Name: "greet", Arguments: map[string]interface{}{"name": "world"}},
 				},
 			})
 		} else {
@@ -80,9 +80,10 @@ func TestAgent_ChatWithTools(t *testing.T) {
 					Description: "Greets someone",
 					Parameters:  json.RawMessage(`{"type":"object","properties":{"name":{"type":"string"}}}`),
 				},
-				Run: func(ctx context.Context, args string) (string, error) {
+				Run: func(ctx context.Context, args map[string]interface{}) (string, error) {
 					var p struct{ Name string }
-					json.Unmarshal([]byte(args), &p)
+					b, _ := json.Marshal(args)
+					json.Unmarshal(b, &p)
 					return "Hello, " + p.Name + "!", nil
 				},
 			},

benchmark_test.go

@@ -258,7 +258,7 @@ func BenchmarkChatWithTools(b *testing.B) {
 		if round%2 == 1 {
 			json.NewEncoder(w).Encode(ChatWithToolsResponse{
 				ChatResponse: ChatResponse{SessionID: "s-1", Response: ""},
-				ToolCalls:    []ToolCall{{ID: "tc-1", Name: "echo", Arguments: `{"msg":"hi"}`}},
+				ToolCalls:    []ToolCall{{ID: "tc-1", Name: "echo", Arguments: map[string]interface{}{"msg": "hi"}}},
 				FinishReason: "tool_calls",
 			})
 		} else {
@@ -274,7 +274,7 @@ func BenchmarkChatWithTools(b *testing.B) {
 	tools := []Tool{
 		{
 			Schema: ToolSchema{Name: "echo", Description: "echo", Parameters: json.RawMessage(`{"type":"object"}`)},
-			Run:    func(_ context.Context, args string) (string, error) { return "echoed", nil },
+			Run:    func(_ context.Context, args map[string]interface{}) (string, error) { return "echoed", nil },
 		},
 	}
 	ctx := context.Background()

codegen_tools.go

@@ -0,0 +1,7 @@
+//go:build tools
+
+package hawksdk
+
+import (
+	_ "github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen"
+)