chore: turbo lint

[aljo242]

Linter rules have been expanded a lot to ensure a more uniform codebase:

enable:
    # Check for pass []any as any in variadic func(...any).
      # Rare case but saved me from debugging a few times.
    - asasalint
    # I prefer plane ASCII identifiers.
    # Symbol `∆` instead of `delta` looks cool but no thanks.
    - asciicheck
    # Checks for dangerous unicode character sequences.
    # Super rare but why not to be a bit paranoid?
    - bidichk
    - bodyclose
    # Check whether the function uses a non-inherited context.
    - contextcheck
    # Check for two durations multiplied together.
    - durationcheck
    - depguard
    - dogsled
    - errcheck
    # Checks `Err-` prefix for var and `-Error` suffix for error type.
    - errname
    # Suggests to use `%w` for error-wrapping.
    - errorlint
    # Checks for pointers to enclosing loop variables.
    - exportloopref
    - goconst
    - gocritic
    # Forces to put `.` at the end of the comment. Code is poetry.
    - godot
    # Might not be that important, but I prefer to keep all of them.
    # `gofumpt` is amazing, kudos to Daniel Marti https://github.com/mvdan/gofumpt
    - gofmt
    - gofumpt
    - goimports
    - gosec
    - gosimple
    - govet
    - ineffassign
    - misspell
    - nakedret
    # Finds sending HTTP request without context.Context.
    - noctx
    # Finds slices that could potentially be pre-allocated.
    # Small performance win + cleaner code.
    - prealloc
    # Finds shadowing of Go's predeclared identifiers.
    # I hear a lot of complaints from junior developers.
    # But after some time they find it very useful.
    - predeclared
    # Lint your Prometheus metrics name.
    - promlinter
    # Checks that package variables are not reassigned.
    # Super rare case but can catch bad things (like `io.EOF = nil`)
    - reassign
    - revive
    - staticcheck
    - stylecheck
    # Checks that package variables are not reassigned.
    # Super rare case but can catch bad things (like `io.EOF = nil`)
    - reassign
    - typecheck
    # Test-related checks. All of them are good.
    - tenv
    - testableexamples
    - thelper
    - tparallel
    - unconvert
    - unused
    - unparam
    # Detect the possibility to use variables/constants from stdlib.
    - usestdlibvars
    # Finds wasted assignment statements.
    - wastedassign

Also linting tests to ensure better quality.

NO functionality is changed here. Most of the changes are just adding periods to comments

[aljo242]

Will regenerate all of the proto - idk how that got in

[ajansari95]

looks good but- should revert the changes to generated files and some other comments also conflicts with #357 merged

[joe-bowman]

Lgtm now

[joe-bowman]

@ajansari95

[ajansari95]

lgtm