fix: suppress nolintlint for platform-specific contextcheck

salmonumbrella · claude · salmonumbrella · commit a6309506f06f · 2026-01-02T10:50:32.000-08:00
The contextcheck linter only triggers on macOS (where EnsureKeychainAccess has a real implementation), but on Linux the stub doesn't trigger it, causing nolintlint to complain the directive is unused. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/internal/googleauth/accounts_server.go b/internal/googleauth/accounts_server.go
@@ -294,7 +294,7 @@ func (ms *ManageServer) handleOAuthCallback(w http.ResponseWriter, r *http.Reque
 	}
 
 	// Pre-flight: ensure keychain is accessible before storing token
-	if err := secrets.EnsureKeychainAccess(); err != nil { //nolint:contextcheck // keychain ops don't use context
+	if err := secrets.EnsureKeychainAccess(); err != nil { //nolint:contextcheck,nolintlint // keychain ops don't use context; nolint unused on non-Darwin
 		w.WriteHeader(http.StatusInternalServerError)
 		renderErrorPage(w, "Keychain is locked: "+err.Error())
 

Original file line number	Diff line number	Diff line change
`@@ -294,7 +294,7 @@ func (ms ManageServer) handleOAuthCallback(w http.ResponseWriter, r http.Reque`
`294`	`294`	`}`
`295`	`295`
`296`	`296`	`// Pre-flight: ensure keychain is accessible before storing token`
`297`		`- if err := secrets.EnsureKeychainAccess(); err != nil { //nolint:contextcheck // keychain ops don't use context`
	`297`	`+ if err := secrets.EnsureKeychainAccess(); err != nil { //nolint:contextcheck,nolintlint // keychain ops don't use context; nolint unused on non-Darwin`
`298`	`298`	`w.WriteHeader(http.StatusInternalServerError)`
`299`	`299`	`renderErrorPage(w, "Keychain is locked: "+err.Error())`
`300`	`300`