refactor(mgmt): move full_name and metadata handling to EE only

Because

- CE does not have DB columns for full_name and metadata (migrations are in EE)
- Convertor should not reference fields that don't exist in CE database
- Integration tests should not test EE-only features

This commit

- Remove FullName handling from convertor.go (DB → PB and PB → DB)
- Remove fullName and metadata tests from integration tests
- Keep fields in datamodel for EE to use via shared struct

Author: xiaofei-du

integration-test/const.js

@@ -56,7 +56,6 @@ export const defaultUserUpdate = {
   profile: {
     displayName: "Instill AI",
     companyName: "Instill AI",
-    fullName: "",  // Reset to empty
   },
   role: "hobbyist",
   newsletterSubscription: true,
@@ -70,7 +69,6 @@ export const defaultUser = {
   profile: {
     displayName: "Instill AI",
     companyName: "Instill AI",
-    fullName: "",
   },
   role: "hobbyist",
   newsletterSubscription: true,

integration-test/rest-public-user.js

@@ -52,12 +52,6 @@ export function CheckPublicGetUser(header) {
           (r) => r.json().user.profile.displayName !== undefined,
         [`GET /${constant.mgmtVersion}/user response profile.companyName`]:
           (r) => r.json().user.profile.companyName !== undefined,
-        // fullName is optional, just check field exists in profile (can be empty string)
-        [`GET /${constant.mgmtVersion}/user response profile.fullName exists`]:
-          (r) => r.json().user.profile.hasOwnProperty('fullName') || r.json().user.profile.fullName === undefined,
-        // metadata is optional, check field exists in profile (can be null/undefined)
-        [`GET /${constant.mgmtVersion}/user response profile.metadata exists`]:
-          (r) => r.json().user.profile.hasOwnProperty('metadata') || r.json().user.profile.metadata === undefined,
         [`GET /${constant.mgmtVersion}/user response role`]:
           (r) => r.json().user.role !== undefined,
         [`GET /${constant.mgmtVersion}/user response newsletterSubscription`]:
@@ -78,8 +72,6 @@ export function CheckPublicPatchAuthenticatedUser(header) {
       profile: {
         displayName: "test",
         companyName: "company",
-        fullName: "Test User Full Name",
-        metadata: { usageType: "test", source: "integration-test" },
       },
       role: "ai-engineer",
       newsletterSubscription: true,
@@ -137,10 +129,6 @@ export function CheckPublicPatchAuthenticatedUser(header) {
           () => updatedUser.profile.displayName === userUpdate.profile.displayName,
         [`PATCH /${constant.mgmtVersion}/user response profile.companyName updated`]:
           () => updatedUser.profile.companyName === userUpdate.profile.companyName,
-        [`PATCH /${constant.mgmtVersion}/user response profile.fullName updated`]:
-          () => updatedUser.profile.fullName === userUpdate.profile.fullName,
-        [`PATCH /${constant.mgmtVersion}/user response profile.metadata updated`]:
-          () => updatedUser.profile.metadata !== undefined && updatedUser.profile.metadata.usageType === userUpdate.profile.metadata.usageType,
         [`PATCH /${constant.mgmtVersion}/user response role updated`]:
           () => updatedUser.role === userUpdate.role,
         [`PATCH /${constant.mgmtVersion}/user response newsletterSubscription updated`]:

pkg/service/convertor.go

@@ -160,7 +160,6 @@ func (s *service) DBUser2PBUser(ctx context.Context, dbUser *datamodel.Owner) (*
 			Avatar:             &avatar,
 			Bio:                &dbUser.Bio.String,
 			SocialProfileLinks: socialProfileLinks,
-			FullName:           &dbUser.FullName.String,
 		},
 		Email: dbUser.Email,
 	}, nil
@@ -211,7 +210,6 @@ func (s *service) DBUser2PBAuthenticatedUser(ctx context.Context, dbUser *datamo
 			Avatar:             &avatar,
 			Bio:                &dbUser.Bio.String,
 			SocialProfileLinks: socialProfileLinks,
-			FullName:           &dbUser.FullName.String,
 		},
 		OnboardingStatus: mgmtpb.OnboardingStatus(dbUser.OnboardingStatus),
 	}, nil
@@ -300,10 +298,6 @@ func (s *service) PBAuthenticatedUser2DBUser(ctx context.Context, pbUser *mgmtpb
 			return []byte{}
 		}(),
 		OnboardingStatus: datamodel.OnboardingStatus(pbUser.OnboardingStatus),
-		FullName: sql.NullString{
-			String: pbUser.GetProfile().GetFullName(),
-			Valid:  len(pbUser.GetProfile().GetFullName()) > 0,
-		},
 	}, nil
 }