Merge pull request #2 from thedadams/add-ci

chore: add CI for merges and PRs

Author: thedadams

.github/workflows/test.yaml

@@ -0,0 +1,54 @@
+name: test
+on:
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - README.md
+  pull_request:
+    branches:
+      - main
+    paths-ignore:
+      - README.md
+
+jobs:
+  test-linux:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+      - uses: actions/setup-go@v5
+        with:
+          cache: false
+          go-version: "1.22"
+      - name: Validate
+        run: make validate
+      - name: Install gptscript
+        run: |
+          curl https://get.gptscript.ai/releases/default_linux_amd64_v1/gptscript -o gptscript
+          chmod +x gptscript
+      - name: Run Tests
+        env:
+          GPTSCRIPT_BIN: ./gptscript
+          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+        run: make test
+
+  test-windows:
+    runs-on: windows-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+      - uses: actions/setup-go@v5
+        with:
+          cache: false
+          go-version: "1.22"
+      - name: Install gptscript
+        run: |
+          curl https://get.gptscript.ai/releases/default_windows_amd64_v1/gptscript.exe -o gptscript.exe
+      - name: Run Tests
+        env:
+          GPTSCRIPT_BIN: .\gptscript.exe
+          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+        run: make test

.golangci.yaml

@@ -0,0 +1,23 @@
+run:
+  timeout: 5m
+
+output:
+  format: github-actions
+
+linters:
+  disable-all: true
+  enable:
+    - errcheck
+    - gofmt
+    - gosimple
+    - govet
+    - ineffassign
+    - staticcheck
+    - typecheck
+    - thelper
+    - unused
+    - goimports
+    - whitespace
+    - revive
+  fast: false
+  max-same-issues: 50

Makefile

@@ -0,0 +1,24 @@
+.PHONY: tidy lint test validate setup-ci
+
+tidy:
+	go mod tidy
+
+test:
+	go test -v ./...
+
+GOLANGCI_LINT_VERSION ?= v1.56.1
+lint:
+	if ! command -v golangci-lint &> /dev/null; then \
+  		echo "Could not find golangci-lint, installing version $(GOLANGCI_LINT_VERSION)."; \
+		curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $$(go env GOPATH)/bin $(GOLANGCI_LINT_VERSION); \
+	fi
+
+	golangci-lint run
+
+validate: tidy lint
+	if [ -n "$$(git status --porcelain)" ]; then \
+		git status --porcelain; \
+		echo "Encountered dirty repo!"; \
+		git diff; \
+		exit 1 \
+	;fi

exec_test.go

@@ -67,8 +67,7 @@ func TestExecFileChdir(t *testing.T) {
 
 func TestExecComplexTool(t *testing.T) {
 	tool := &Tool{
-		Tools:        []string{"sys.write"},
-		JsonResponse: true,
+		JSONResponse: true,
 		Instructions: `
 Create three short graphic artist descriptions and their muses.
 These should be descriptive and explain their point of view.

tool.go

@@ -17,7 +17,7 @@ type Tool struct {
 	Args           map[string]string
 	InternalPrompt bool
 	Instructions   string
-	JsonResponse   bool
+	JSONResponse   bool
 }
 
 // NewTool is a constructor for Tool struct.
@@ -33,7 +33,7 @@ func NewTool(name, description string, tools []string, maxTokens *int, model str
 		Args:           args,
 		InternalPrompt: internalPrompt,
 		Instructions:   instructions,
-		JsonResponse:   jsonResponse,
+		JSONResponse:   jsonResponse,
 	}
 }
 
@@ -62,7 +62,7 @@ func (t *Tool) String() string {
 	if t.Temperature != nil {
 		sb.WriteString(fmt.Sprintf("Temperature: %f\n", *t.Temperature))
 	}
-	if t.JsonResponse {
+	if t.JSONResponse {
 		sb.WriteString("JSON Response: true\n")
 	}
 	if len(t.Args) > 0 {