Apply suggestions from code review

Co-authored-by: colin <[email protected]>

Author: yiweichi

coordinator/internal/logic/auth/login.go

@@ -112,7 +112,7 @@ func (l *LoginLogic) Check(login *types.LoginParameter) error {
 		if login.Message.ProverProviderType == types.ProverProviderTypeUndefined {
 			login.Message.ProverProviderType = types.ProverProviderTypeInternal
 		} else {
-			log.Error("invalid prover_provider_type", "value", login.Message.ProverProviderType, "prover name", login.Message.ProverName, "prover_version", login.Message.ProverVersion)
+			log.Error("invalid prover_provider_type", "value", login.Message.ProverProviderType, "prover name", login.Message.ProverName, "prover version", login.Message.ProverVersion)
 			return errors.New("invalid prover provider type.")
 		}
 	}

coordinator/internal/types/prover.go

@@ -51,7 +51,7 @@ func (r ProverProviderType) String() string {
 	case ProverProviderTypeExternal:
 		return "prover provider type external"
 	default:
-		return fmt.Sprintf("prover provider type type: %d", r)
+		return fmt.Sprintf("prover provider type: %d", r)
 	}
 }
 

coordinator/internal/utils/prover_name.go

@@ -10,6 +10,6 @@ func IsExternalProverNameMatch(localName, remoteName string) bool {
 		return false
 	}
 
-	// note the name of cloud prover is in fact in the format of "cloud_prover_{provider-name}_index"
+	// note the name of cloud prover is in the format of "cloud_prover_{provider-name}_index"
 	return local[0] == remote[0] && local[1] == remote[1] && local[2] == remote[2]
 }