Upgrade golangci-lint and fix linting issues (#544)

## Summary
Upgrade golangci-lint and fix linting issues

## How was it tested?
devbox run lint (and CICD)

## Community Contribution License

All community contributions in this pull request are licensed to the
project maintainers under the terms of the [Apache 2
License](https://www.apache.org/licenses/LICENSE-2.0).

By creating this pull request I represent that I have the right to
license the contributions to the project maintainers under the Apache 2
License as stated in the [Community Contribution
License](https://github.com/jetify-com/opensource/blob/main/CONTRIBUTING.md#community-contribution-license).

Author: loreto

constructors.go

@@ -120,7 +120,7 @@ func newTypeID(prefix string, suffixBuf [26]byte) TypeID {
 
 // FromUUID encodes the given UUID (in hex string form) as a TypeID with the given prefix.
 // If you want to create an id without a prefix, pass an empty string for the prefix.
-func FromUUID(prefix string, uidStr string) (TypeID, error) {
+func FromUUID(prefix, uidStr string) (TypeID, error) {
 	// Validate prefix early
 	if err := validatePrefix(prefix); err != nil {
 		return zeroID, err

typeid_test.go

@@ -235,7 +235,7 @@ func testInvalidExample(t *testing.T, example InvalidExample) {
 }
 
 // testInvalidPrefix tests that all constructors properly reject invalid prefixes
-func testInvalidPrefix(t *testing.T, prefix string, desc string) {
+func testInvalidPrefix(t *testing.T, prefix, desc string) {
 	t.Helper()
 
 	// Test Generate constructor with invalid prefix