fix(server): fix bug that visibility become empty when project import (#1777)

soneda-yuya · web-flow · commit 4d6085299487 · 2025-08-22T18:53:12.000+09:00
diff --git a/server/internal/app/file_uploader.go b/server/internal/app/file_uploader.go
@@ -126,7 +126,7 @@ func serveUploadFiles(
 
 func CreateProcessingProject(ctx context.Context, usecases *interfaces.Container, op *usecase.Operator, wsId string) (*project.Project, error) {
 
-	visibility := "private"
+	visibility := string(project.VisibilityPrivate)
 	coreSupport := true
 	unknown := "It's importing now..."
 	workspaceID, err := accountdomain.WorkspaceIDFrom(wsId)
diff --git a/server/internal/usecase/interactor/project.go b/server/internal/usecase/interactor/project.go
@@ -343,8 +343,12 @@ func (i *Project) Create(ctx context.Context, input interfaces.CreateProjectPara
 		visibility = project.Visibility(*input.Visibility)
 	}
 
-	if i.policyChecker != nil {
+	// override visibility to public if it's an import
+	if isImport {
+		visibility = project.VisibilityPublic
+	}
 
+	if i.policyChecker != nil {
 		if isImport {
 			// Try checking if user can create private project
 			errPrivate := i.checkGeneralPolicy(ctx, input.WorkspaceID, project.VisibilityPrivate)
@@ -362,7 +366,6 @@ func (i *Project) Create(ctx context.Context, input interfaces.CreateProjectPara
 
 	}
 
-	// project.Visibility(input.Visibility),
 	return i.createProject(ctx, createProjectInput{
 		WorkspaceID:  input.WorkspaceID,
 		Visualizer:   input.Visualizer,
@@ -1095,6 +1098,12 @@ func (i *Project) ImportProjectData(ctx context.Context, workspace string, proje
 		topics = &ret
 	}
 
+	visibility := project.VisibilityPublic
+	if ret, ok := projectData["visibility"].(string); ok {
+		vis := project.Visibility(ret)
+		visibility = vis
+	}
+
 	result, err := i.createProject(ctx, createProjectInput{
 		WorkspaceID:  workspaceId,
 		ProjectID:    projectId,
@@ -1109,7 +1118,7 @@ func (i *Project) ImportProjectData(ctx context.Context, workspace string, proje
 		Readme:       readme,
 		License:      license,
 		Topics:       topics,
-		// skip Visibility
+		Visibility:   &visibility,
 	}, op)
 	if err != nil {
 		return nil, err
