Pmiddleton/fix project delete (#38)

## Summary

<!-- What does this PR do? Why? -->

## Changes

<!-- Bullet list of changes -->

## Test plan

<!-- How was this tested? -->

- [ ] `make test` passes
- [ ] `cd web && npx tsc --noEmit` passes

Author: philjestin

.github/workflows/ci.yml

@@ -26,6 +26,14 @@ jobs:
       - name: Install dependencies
         run: go mod download
 
+      - name: Create frontend stub for embed
+        run: mkdir -p web/dist && touch web/dist/.gitkeep
+
+      - name: Lint
+        uses: golangci/golangci-lint-action@v7
+        with:
+          version: v2.10
+
       # Build internal packages and cmd/server (excludes Wails app which needs web/dist)
       - name: Build
         run: |

.golangci.yml

@@ -0,0 +1,105 @@
+version: "2"
+
+run:
+  timeout: 5m
+
+linters:
+  enable:
+    - errcheck
+    - govet
+    - staticcheck
+    - unused
+    - ineffassign
+    - gocritic
+    - misspell
+    - bodyclose
+    - durationcheck
+    - errname
+    - nilerr
+    - sqlclosecheck
+    - unconvert
+    - unparam
+    - wastedassign
+
+  settings:
+    errcheck:
+      check-type-assertions: false
+      check-blank: false
+      exclude-functions:
+        # Network cleanup — errors not actionable
+        - (net.Conn).Close
+        - (net.Conn).SetDeadline
+        - (net.Conn).SetReadDeadline
+        - (*net.UDPConn).Close
+        - (*net.UDPConn).SetDeadline
+        - (*net.UDPConn).SetReadDeadline
+        - (*net.UDPConn).WriteToUDP
+        - (*database/sql.DB).Close
+        - (*os.File).Close
+        - (io.Closer).Close
+        - (*io.PipeWriter).Close
+        - (*mime/multipart.Writer).Close
+        - (*net/http.Response).Body.Close
+        - os.Remove
+        - os.MkdirAll
+        - encoding/json.Unmarshal
+        - (*encoding/json.Encoder).Encode
+        - (*encoding/json.Decoder).Decode
+    gocritic:
+      enabled-tags:
+        - diagnostic
+        - performance
+      disabled-checks:
+        - hugeParam
+        - rangeValCopy
+        - ifElseChain
+        - filepathJoin
+        - exitAfterDefer
+    misspell:
+      locale: US
+
+  exclusions:
+    paths:
+      - web
+      - site
+      - build
+      - migrations
+    rules:
+      # Test files: relax errcheck and unparam
+      - path: _test\.go
+        linters:
+          - unparam
+          - errcheck
+          - gocritic
+      # Defer cleanup — error is not actionable
+      - linters:
+          - errcheck
+        source: "defer "
+      # Deprecated websocket library — tracked separately
+      - linters:
+          - staticcheck
+        text: "SA1019:"
+      # IPv6 hostport in printer discovery
+      - linters:
+          - govet
+        text: "hostport"
+      # "cancelled" is used in JSON struct tags and API responses
+      - linters:
+          - misspell
+        text: "(?i)cancelled"
+      # Best-effort event logging / audit trail
+      - linters:
+          - errcheck
+        source: "AddEvent|AppendEvent"
+      # Best-effort cleanup of OAuth state
+      - linters:
+          - errcheck
+        source: "DeleteOAuthState|UpdateLastSync|UpdateWebhookEventProcessed"
+      # Background printer connections (fire-and-forget goroutines)
+      - linters:
+          - errcheck
+        source: "go s\\.(manager|printerMgr)\\.Connect"
+      # Migration ALTER TABLE — intentionally ignores "column already exists"
+      - linters:
+          - errcheck
+        source: "// Ignore error if"

Makefile

@@ -1,4 +1,4 @@
-.PHONY: help dev build run test clean frontend backend stop start restart show-version bump-patch bump-minor bump-major release site site-build
+.PHONY: help dev build run test clean frontend backend stop start restart show-version bump-patch bump-minor bump-major release site site-build lint lint-go lint-web
 
 # Version
 VERSION := $(shell cat VERSION | tr -d 'v\n')
@@ -23,6 +23,9 @@ help:
 	@echo "  make frontend   - Run React frontend only"
 	@echo "  make build      - Build production binaries"
 	@echo "  make test       - Run tests"
+	@echo "  make lint       - Run all linters"
+	@echo "  make lint-go    - Run Go linter (golangci-lint)"
+	@echo "  make lint-web   - Run frontend linter (ESLint)"
 	@echo "  make clean      - Clean build artifacts"
 	@echo ""
 	@echo "Site:"
@@ -68,6 +71,15 @@ test-coverage:
 	go test -coverprofile=coverage.out ./...
 	go tool cover -html=coverage.out
 
+# Linting
+lint: lint-go lint-web
+
+lint-go:
+	golangci-lint run ./...
+
+lint-web:
+	cd web && npm run lint
+
 # Cleanup
 clean:
 	rm -rf bin/

internal/api/dispatch_handler.go

@@ -162,8 +162,3 @@ func (h *DispatchHandler) UpdatePrinterSettings(w http.ResponseWriter, r *http.R
 
 	respondJSON(w, http.StatusOK, settings)
 }
-
-// PrintJobHandler extension for priority update
-type printJobPriorityRequest struct {
-	Priority int `json:"priority"`
-}

internal/api/handlers.go

@@ -519,7 +519,7 @@ func (h *DesignHandler) Download(w http.ResponseWriter, r *http.Request) {
 
 	w.Header().Set("Content-Disposition", "attachment; filename="+design.FileName)
 	w.Header().Set("Content-Type", "application/octet-stream")
-	io.Copy(w, reader)
+	io.Copy(w, reader) //nolint:errcheck // best-effort streaming to HTTP client
 }
 
 // OpenExternal opens a design file in an external application.
@@ -918,6 +918,22 @@ func (h *SpoolHandler) Get(w http.ResponseWriter, r *http.Request) {
 	respondJSON(w, http.StatusOK, spool)
 }
 
+// Delete deletes a spool by ID.
+func (h *SpoolHandler) Delete(w http.ResponseWriter, r *http.Request) {
+	id, err := parseUUID(r, "id")
+	if err != nil {
+		respondError(w, http.StatusBadRequest, "invalid spool ID")
+		return
+	}
+
+	if err := h.service.Delete(r.Context(), id); err != nil {
+		respondError(w, http.StatusInternalServerError, err.Error())
+		return
+	}
+
+	w.WriteHeader(http.StatusNoContent)
+}
+
 // PrintJobHandler handles print job endpoints.
 type PrintJobHandler struct {
 	service *service.PrintJobService
@@ -1396,7 +1412,7 @@ func (h *FileHandler) Get(w http.ResponseWriter, r *http.Request) {
 
 	w.Header().Set("Content-Disposition", "attachment; filename="+file.OriginalName)
 	w.Header().Set("Content-Type", file.ContentType)
-	io.Copy(w, reader)
+	io.Copy(w, reader) //nolint:errcheck // best-effort streaming to HTTP client
 }
 
 // ExpenseHandler handles expense endpoints.

internal/api/router.go

@@ -196,6 +196,7 @@ func NewRouter(services *service.Services, hub *realtime.Hub) http.Handler {
 			r.Get("/", spoolHandler.List)
 			r.Post("/", spoolHandler.Create)
 			r.Get("/{id}", spoolHandler.Get)
+			r.Delete("/{id}", spoolHandler.Delete)
 		})
 
 		// Print Jobs

internal/api/templates_test.go

@@ -109,14 +109,6 @@ func (m *MockTemplateService) CreateProjectFromTemplate(ctx context.Context, tem
 	return project, []model.PrintJob{}, nil
 }
 
-// Test helper to create a template handler with mock service
-func setupTemplateHandler() (*TemplateHandler, *MockTemplateService) {
-	mock := NewMockTemplateService()
-	// We need to wrap the mock in a real service struct for the handler
-	// For now, we'll test via HTTP handlers directly
-	return nil, mock
-}
-
 func TestTemplateHandler_Create(t *testing.T) {
 	// Create a test template
 	template := model.Template{

internal/database/schema.sql

@@ -115,7 +115,7 @@ CREATE INDEX IF NOT EXISTS idx_projects_sku ON projects(sku);
 -- Tasks table (Work Instances - created when processing orders)
 CREATE TABLE IF NOT EXISTS tasks (
     id TEXT PRIMARY KEY,
-    project_id TEXT NOT NULL REFERENCES projects(id),
+    project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
     order_id TEXT REFERENCES orders(id),
     order_item_id TEXT REFERENCES order_items(id),
     name TEXT NOT NULL,
@@ -816,7 +816,7 @@ CREATE TABLE IF NOT EXISTS quotes (
 CREATE INDEX IF NOT EXISTS idx_quotes_status ON quotes(status);
 CREATE INDEX IF NOT EXISTS idx_quotes_customer ON quotes(customer_id);
 CREATE INDEX IF NOT EXISTS idx_quotes_quote_number ON quotes(quote_number);
-CREATE UNIQUE INDEX IF NOT EXISTS idx_quotes_share_token ON quotes(share_token);
+-- idx_quotes_share_token is created in sqlite.go after ALTER TABLE adds the column for existing databases
 
 CREATE TABLE IF NOT EXISTS quote_options (
     id TEXT PRIMARY KEY,

internal/database/sqlite.go

@@ -132,7 +132,7 @@ func RunMigrations(db *sql.DB) error {
 	}
 
 	// Create indexes that may not exist
-	db.Exec(`CREATE UNIQUE INDEX IF NOT EXISTS idx_quotes_share_token ON quotes(share_token)`)
+	db.Exec(`CREATE UNIQUE INDEX IF NOT EXISTS idx_quotes_share_token ON quotes(share_token)`) //nolint:errcheck // best-effort index creation
 
 	return nil
 }

internal/printer/bambu.go

@@ -262,7 +262,7 @@ func (c *BambuClient) requestPushAll() {
 			Command:  "pushall",
 		},
 	}
-	c.sendCommand(cmd)
+	c.sendCommand(cmd) //nolint:errcheck // fire-and-forget status refresh
 }
 
 // sendCommand sends a command to the printer via MQTT.
@@ -415,7 +415,7 @@ func (c *BambuClient) uploadFile(localPath string, remoteName string) (string, e
 	remoteDir := "/cache"
 	if err := conn.ChangeDir(remoteDir); err != nil {
 		remoteDir = "/"
-		conn.ChangeDir(remoteDir)
+		conn.ChangeDir(remoteDir) //nolint:errcheck // fallback to root dir
 	}
 
 	// Upload file