chore: enable wrapcheck for new code

This has been disabled because manually wrapping all errors is very
tedious when iterating quickly, but it's come to the point where we need
to do this in order to provide useful errors. Recent examples of this
are #1885 and
#1879.

.github/workflows/ci.yml

@@ -63,7 +63,7 @@ jobs:
       - name: Build Cache
         uses: ./.github/actions/build-cache
       - name: golangci-lint
-        run: golangci-lint run --out-format github-actions ./...
+        run: golangci-lint run --new-from-rev=HEAD~ --out-format github-actions ./...
       - name: go-check-sumtype
         shell: bash
         run: go-check-sumtype ./... 2>&1 | to-annotation

.golangci.yml

@@ -57,7 +57,6 @@ linters:
     - interfacebloat
     - tagalign
     - nolintlint
-    - wrapcheck # We might want to re-enable this if we manually wrap all the existing errors with fmt.Errorf
     - protogetter
 
 linters-settings:

Justfile

@@ -139,7 +139,7 @@ lint-frontend: build-frontend
 
 # Lint the backend
 lint-backend:
-  @golangci-lint run ./...
+  @golangci-lint run --new-from-rev=HEAD~ ./...
 
 # Run live docs server
 docs: