chore: lint fixes part 10

[pawbana]

• nilnil (4 → 0): added nolint:nilnil to intentional nil, nil returns in test mocks (MockProvider.CreateInterceptor, mockServerProxier.CallTool) and lastUserPrompt (no user prompt found is not an error)
• gocritic FullResponseWriter (1 → 0): added Hijack() method to statusCapturingWriter delegating to underlying http.Hijacker
• revive defer-in-loop (1 → 0): replaced defer shutdownCancel() with explicit shutdownCancel() call after events.Shutdown
• staticcheck SA1026 (1 → 0): replaced responses.Error (which embeds *http.Request with unmarshalable GetBody func) with a plain struct matching the same JSON shape

All lint errors are now resolved — 0 remaining.

[pawbana]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• chore: add typo check to linter #264
• feat: add lint check to CI cheks #263
• chore: lint fixes part 10 #262 👈 (View in Graphite)
• chore: lint fixes part 9 #261
• chore: lint fixes part 8 #260
• chore: lint fixes part 7 #259
• chore: lint fixes part 6 #258
• chore: lint fixes part 5 #257
• chore: lint fixes part 4 #256
• chore: lint fixes part 3 #255
• chore: lint fixes part 2 #254
• chore: lint fixes part 1 (automatic fixes) #253
• feat: extend golang lint check #252
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[dannykopping]

"to satisfy the FullResponseWriter lint rule"
I don't see any details about this; can you elaborate pls?

[pawbana]

circuitbreaker/circuitbreaker.go:106:6: ruleguard: ResponseWriter "statusCapturingWriter" must implement http.Flusher and http.Hijacker (gocritic)

I believe rule is simply that anythink that linter believes is ResponeWriter must implement Flusherand Hijacker