chore: lint fixes part 6

[pawbana]

• unhandled-error (26): explicitly handled return values from in-memory buffer writes (bytes.Buffer, strings.Builder — never fail), os.File.Close (best-effort cleanup), io.PipeWriter.Write/Close (error
  handled via pipe read side); properly checked errors from circuitbreaker.Execute and responseCopier.forwardResp in tests
• exported stutter (3): messages.MessagesRequestPayload → messages.RequestPayload (+ constructor NewRequestPayload); responses.ResponsesRequestPayload → responses.RequestPayload (+ constructor
  NewRequestPayload); recorder.RecorderWrapper → recorder.WrappedRecorder (+ constructor NewWrappedRecorder)

[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
• chore: lint fixes part 9 #261
• chore: lint fixes part 8 #260
• chore: lint fixes part 7 #259
• chore: lint fixes part 6 #258 👈 (View in Graphite)
• 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]

This is kind of at odds with #257 where errorResponse became chatCompletionResponseError.

Let's pick one approach and use it consistently.

[pawbana]

Thanks, good catch. There are 2 different rules:
- errname: error types need to have Error suffix
- exported stutter: exported symbols cannot start with package name

RequestPayload types are exported so it is enforced while error types from #257 are internal.
I'll update #257 to match stutter rule even thou error types are not exported.