fix(webserver): validation should always comes with message field

ee/tabby-webserver/src/schema/auth.rs

@@ -251,13 +251,13 @@ impl relay::NodeType for User {
 
 #[derive(Validate, GraphQLInputObject)]
 pub struct RequestInvitationInput {
-    #[validate(email(code = "email"))]
+    #[validate(email(code = "email", message="Invalid email address"))]
     pub email: String,
 }
 
 #[derive(Validate, GraphQLInputObject)]
 pub struct RequestPasswordResetEmailInput {
-    #[validate(email(code = "email"))]
+    #[validate(email(code = "email", message="Invalid email address"))]
     pub email: String,
 }
 

ee/tabby-webserver/src/schema/constants.rs

@@ -2,5 +2,5 @@ use lazy_static::lazy_static;
 use regex::Regex;
 
 lazy_static! {
-    pub static ref REPOSITORY_NAME_REGEX: Regex = Regex::new("^[\\w-]+$").unwrap();
+    pub static ref REPOSITORY_NAME_REGEX: Regex = Regex::new("^[a-zA-Z][\\w.-]+$").unwrap();
 }

ee/tabby-webserver/src/schema/email.rs

@@ -32,7 +32,7 @@ pub struct EmailSetting {
 #[derive(GraphQLInputObject, Validate)]
 pub struct EmailSettingInput {
     pub smtp_username: String,
-    #[validate(email(code = "fromAddress"))]
+    #[validate(email(code = "fromAddress", message="Invalid email address"))]
     pub from_address: String,
     pub smtp_server: String,
     pub smtp_port: i32,

ee/tabby-webserver/src/schema/types.rs

@@ -5,10 +5,11 @@ use validator::Validate;
 pub struct CreateRepositoryProviderInput {
     #[validate(regex(
         code = "displayName",
-        path = "crate::schema::constants::REPOSITORY_NAME_REGEX"
+        path = "crate::schema::constants::REPOSITORY_NAME_REGEX",
+        message = "Invalid repository provider name"
     ))]
     pub display_name: String,
-    #[validate(length(code = "access_token", min = 10))]
+    #[validate(length(code = "accessToken", min = 10, message="Invalid access token"))]
     pub access_token: String,
 }
 
@@ -17,9 +18,10 @@ pub struct UpdateRepositoryProviderInput {
     pub id: ID,
     #[validate(regex(
         code = "displayName",
-        path = "crate::schema::constants::REPOSITORY_NAME_REGEX"
+        path = "crate::schema::constants::REPOSITORY_NAME_REGEX",
+        message = "Invalid repository provider name"
     ))]
     pub display_name: String,
-    #[validate(length(code = "access_token", min = 10))]
+    #[validate(length(code = "accessToken", min = 10, message="Invalid access token"))]
     pub access_token: String,
 }

rules/validate-requires-code.yml

@@ -1,17 +1,21 @@
 id: validate-requires-code
-message: Validations requires code being set for frontend error display
+message: Validations requires code / message being set for frontend error display
 severity: error
 language: rust
 files:
-- ./ee/tabby-webserver/src/**
+  - ./ee/tabby-webserver/src/**
 rule:
   all:
-    - pattern: '#[validate]'
+    - pattern: "#[validate]"
     - not:
-        has:
-          stopBy: end
-          pattern: code
+        all:
+          - has:
+              stopBy: end
+              pattern: code
+          - has:
+              stopBy: end
+              pattern: message
     - not:
         has:
           stopBy: end
-          pattern: custom
+          pattern: custom