Publish application

Author: Thomasr

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationApiService.java

@@ -3,6 +3,7 @@
 import jakarta.annotation.Nonnull;
 import org.lowcoder.api.application.view.ApplicationInfoView;
 import org.lowcoder.api.application.view.ApplicationPermissionView;
+import org.lowcoder.api.application.view.ApplicationPublishRequest;
 import org.lowcoder.api.application.view.ApplicationView;
 import org.lowcoder.domain.application.model.Application;
 import org.lowcoder.domain.application.model.ApplicationRequestType;
@@ -33,7 +34,7 @@ public interface ApplicationApiService {
 
     Mono<ApplicationView> update(String applicationId, Application application);
 
-    Mono<ApplicationView> publish(String applicationId);
+    Mono<ApplicationView> publish(String applicationId, ApplicationPublishRequest applicationPublishRequest);
 
     Mono<Boolean> updateEditState(String applicationId, ApplicationEndpoints.UpdateEditStateRequest updateEditStateRequest);
 

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationApiServiceImpl.java

@@ -13,6 +13,7 @@
 import org.lowcoder.api.application.ApplicationEndpoints.CreateApplicationRequest;
 import org.lowcoder.api.application.view.ApplicationInfoView;
 import org.lowcoder.api.application.view.ApplicationPermissionView;
+import org.lowcoder.api.application.view.ApplicationPublishRequest;
 import org.lowcoder.api.application.view.ApplicationView;
 import org.lowcoder.api.bizthreshold.AbstractBizThresholdChecker;
 import org.lowcoder.api.home.FolderApiService;
@@ -23,6 +24,7 @@
 import org.lowcoder.api.usermanagement.OrgDevChecker;
 import org.lowcoder.domain.application.model.*;
 import org.lowcoder.domain.application.service.ApplicationHistorySnapshotService;
+import org.lowcoder.domain.application.service.ApplicationRecordService;
 import org.lowcoder.domain.application.service.ApplicationService;
 import org.lowcoder.domain.datasource.model.Datasource;
 import org.lowcoder.domain.datasource.service.DatasourceService;
@@ -91,6 +93,7 @@ public class ApplicationApiServiceImpl implements ApplicationApiService {
     private final PermissionHelper permissionHelper;
     private final DatasourceService datasourceService;
     private final ApplicationHistorySnapshotService applicationHistorySnapshotService;
+    private final ApplicationRecordService applicationRecordService;
 
     @Override
     public Mono<ApplicationView> create(CreateApplicationRequest createApplicationRequest) {
@@ -352,11 +355,19 @@ private Mono<Application> doUpdateApplication(String applicationId, Application
     }
 
     @Override
-    public Mono<ApplicationView> publish(String applicationId) {
+    public Mono<ApplicationView> publish(String applicationId, ApplicationPublishRequest applicationPublishRequest) {
         return checkApplicationStatus(applicationId, NORMAL)
                 .then(sessionUserService.getVisitorId())
                 .flatMap(userId -> resourcePermissionService.checkAndReturnMaxPermission(userId,
                         applicationId, PUBLISH_APPLICATIONS))
+                .delayUntil(__ -> applicationService.findById(applicationId)
+                        .map(application -> ApplicationRecord.builder()
+                                .tag(applicationPublishRequest.tag())
+                                .commitMessage(applicationPublishRequest.commitMessage())
+                                .applicationId(application.getId())
+                                .applicationDSL(application.getEditingApplicationDSL())
+                                .build())
+                        .map(applicationRecordService::insert))
                 .flatMap(permission -> applicationService.publish(applicationId)
                         .map(applicationUpdated -> ApplicationView.builder()
                                 .applicationInfoView(buildView(applicationUpdated, permission.getResourceRole().getValue()))

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationController.java

@@ -1,10 +1,7 @@
 package org.lowcoder.api.application;
 
 import lombok.RequiredArgsConstructor;
-import org.lowcoder.api.application.view.ApplicationInfoView;
-import org.lowcoder.api.application.view.ApplicationPermissionView;
-import org.lowcoder.api.application.view.ApplicationView;
-import org.lowcoder.api.application.view.MarketplaceApplicationInfoView;
+import org.lowcoder.api.application.view.*;
 import org.lowcoder.api.framework.view.PageResponseView;
 import org.lowcoder.api.framework.view.ResponseView;
 import org.lowcoder.api.home.SessionUserService;
@@ -132,9 +129,10 @@ public Mono<ResponseView<ApplicationView>> update(@PathVariable String applicati
     }
 
     @Override
-    public Mono<ResponseView<ApplicationView>> publish(@PathVariable String applicationId) {
+    public Mono<ResponseView<ApplicationView>> publish(@PathVariable String applicationId,
+                                                       @RequestBody ApplicationPublishRequest applicationPublishRequest) {
         return gidService.convertApplicationIdToObjectId(applicationId).flatMap(appId ->
-            applicationApiService.publish(appId)
+            applicationApiService.publish(appId, applicationPublishRequest)
                 .map(ResponseView::success));
     }
 

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationEndpoints.java

@@ -4,12 +4,10 @@
 import io.swagger.v3.oas.annotations.Operation;
 import jakarta.annotation.Nullable;
 import org.apache.commons.lang3.BooleanUtils;
-import org.lowcoder.api.application.view.ApplicationInfoView;
-import org.lowcoder.api.application.view.ApplicationPermissionView;
-import org.lowcoder.api.application.view.ApplicationView;
-import org.lowcoder.api.application.view.MarketplaceApplicationInfoView;
+import org.lowcoder.api.application.view.*;
 import org.lowcoder.api.framework.view.ResponseView;
 import org.lowcoder.api.home.UserHomepageView;
+import org.lowcoder.api.query.view.LibraryQueryPublishRequest;
 import org.lowcoder.domain.application.model.Application;
 import org.lowcoder.domain.application.model.ApplicationStatus;
 import org.lowcoder.infra.constant.NewUrl;
@@ -135,7 +133,8 @@ public Mono<ResponseView<ApplicationView>> update(@PathVariable String applicati
 		    description = "Set a Lowcoder Application identified by its ID as available to all selected Users or User-Groups. This is similar to the classic deployment. The Lowcoder Apps gets published in production mode."
 	)
     @PostMapping("/{applicationId}/publish")
-    public Mono<ResponseView<ApplicationView>> publish(@PathVariable String applicationId);
+	public Mono<ResponseView<ApplicationView>> publish(@PathVariable String applicationId,
+												@RequestBody ApplicationPublishRequest applicationPublishRequest);
 
 	@Operation(
 			tags = TAG_APPLICATION_MANAGEMENT,

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/view/ApplicationPublishRequest.java

@@ -0,0 +1,5 @@
+package org.lowcoder.api.application.view;
+
+public record ApplicationPublishRequest(String commitMessage, String tag) {
+
+}

server/api-service/lowcoder-server/src/test/java/org/lowcoder/api/application/ApplicationApiServiceTest.java

@@ -8,6 +8,7 @@
 import org.junit.jupiter.api.TestInstance;
 import org.lowcoder.api.application.ApplicationEndpoints.CreateApplicationRequest;
 import org.lowcoder.api.application.view.ApplicationPermissionView;
+import org.lowcoder.api.application.view.ApplicationPublishRequest;
 import org.lowcoder.api.application.view.ApplicationView;
 import org.lowcoder.api.common.InitData;
 import org.lowcoder.api.common.mockuser.WithMockUser;
@@ -152,7 +153,7 @@ public void testPublishApplication() {
 
         // publish
         applicationIdMono = applicationIdMono
-                .delayUntil(id -> applicationApiService.publish(id));
+                .delayUntil(id -> applicationApiService.publish(id, new ApplicationPublishRequest("Test Publish", "1.0.0")));
 
         // edit dsl after publish
         StepVerifier.create(applicationIdMono.flatMap(id -> applicationApiService.getEditingApplication(id)))