chore: lint fixes part 4

[pawbana]

• import-shadowing (11): renamed recorder parameter to rec in Setup() interface + all 9 implementations to avoid shadowing the recorder package import; renamed provider parameter to prov in
  newPassthroughRouter to avoid shadowing the provider package
• unused-receiver (1): omitted receiver name on AsyncRecorder.RecordInterception panic method
• context-as-argument (2): moved context.Context to first parameter in newBridgeTestServer and newMockUpstream, updated ~60 call sites
• G306/gosec (2): tightened WriteFile permissions from 0o644 to 0o600 in API dump files
• forcetypeassert (2): added nolint directives for sync.Map type assertions in circuit breaker (always stores the same type)

[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
• chore: lint fixes part 5 #257
• chore: lint fixes part 4 #256 👈 (View in Graphite)
• 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]

Was this picked up by a linter? Seems like an unneccesary change.

[pawbana]

I believe it this regarding this point:

context-as-argument (2): moved context.Context to first parameter in newBridgeTestServer and newMockUpstream, updated ~60 call sites

linter rule is that context needs to be first argument IIRC.

[dannykopping]

OK 👍