chore: Update Makefile and GitHub Actions for improved CI/CD

Author: hashkode

.github/workflows/go.yaml

@@ -14,16 +14,23 @@ jobs:
   build:
     name: Build & Test
     runs-on: ubuntu-latest
+
+    # Set GOPRIVATE to prevent fetching internal packages externally
     env:
       GOPRIVATE: github.com/co-browser/agent-browser
+      
     steps:
       - name: Check out code
         uses: actions/checkout@v4
+        with:
+          fetch-depth: 1  # Shallow clone for faster checkout
 
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
           go-version: '1.24'
+          check-latest: true
+          cache: true  # Enable Go module cache
 
       - name: Setup tools
         run: make setup
@@ -34,9 +41,11 @@ jobs:
       - name: Check formatting
         run: |
           make fmt
-          # Check if git status is clean after formatting
-          if ! git diff --exit-code; then
+          FORMAT_DIFF=$(git diff)
+          if [ -n "$FORMAT_DIFF" ]; then
             echo "Code is not formatted correctly. Run 'make fmt' locally."
+            echo "The following files need formatting:"
+            git diff --name-only
             exit 1
           fi
 
@@ -45,6 +54,17 @@ jobs:
 
       - name: Run tests
         run: make test
+        # Show test coverage if any tests exist
+        if: success()
+        continue-on-error: true
 
       - name: Build application
-        run: make build 
+        run: make build
+        
+      - name: Archive build artifacts
+        if: success()
+        uses: actions/upload-artifact@v4
+        with:
+          name: agent-browser-binary
+          path: out/agent-browser
+          retention-days: 7 

Makefile

@@ -50,7 +50,7 @@ fmt:
 
 tidy:
 	@echo "Tidying modules..."
-	@go mod tidy
+	@GOPRIVATE=github.com/co-browser/agent-browser go mod tidy
 
 lint:
 	@echo "Linting code... (requires golangci-lint)"

internal/web/server.go

@@ -33,7 +33,7 @@ func NewServer(mux *http.ServeMux) *http.Server {
 // Renamed to be exported.
 func RegisterWebServerHooks(lifecycle fx.Lifecycle, server *http.Server, logger log.Logger) {
 	lifecycle.Append(fx.Hook{
-		OnStart: func(ctx context.Context) error {
+		OnStart: func(_ context.Context) error {
 			go func() {
 				logger.Info().Str("addr", server.Addr).Msg("Starting HTTP server")
 				if err := server.ListenAndServe(); err != nil && !errors.Is(err, http.ErrServerClosed) {