@kumori/aurora-backend-handler 1.0.62 → 1.0.63

package/helpers/revision-helper.ts

@@ -217,12 +217,16 @@ const updateServiceWithRevision = (
   const existingRevisionIndex = existingService.revisions.findIndex(
     (rev) => String(rev.id) === String(entityId),
   );
-  const existingRevision = existingRevisionIndex !== -1
-    ? existingService.revisions[existingRevisionIndex]
-    : null;
-  const hasExistingSchema = existingRevision?.schema &&
-    ("parameters" in existingRevision.schema || "resources" in existingRevision.schema);
-  const hasNewSchema = newRevision.schema &&
+  const existingRevision =
+    existingRevisionIndex !== -1
+      ? existingService.revisions[existingRevisionIndex]
+      : null;
+  const hasExistingSchema =
+    existingRevision?.schema &&
+    ("parameters" in existingRevision.schema ||
+      "resources" in existingRevision.schema);
+  const hasNewSchema =
+    newRevision.schema &&
     ("parameters" in newRevision.schema || "resources" in newRevision.schema);
   if (hasExistingSchema && !hasNewSchema) {
     newRevision.schema = existingRevision!.schema;
@@ -238,9 +242,10 @@ const updateServiceWithRevision = (
   const updatedService: Service = {
     ...existingService,
     revisions: updatedRevisions,
-    currentRevision: Number(entityId) >= Number(existingService.currentRevision ?? 0)
-      ? entityId
-      : existingService.currentRevision,
+    currentRevision:
+      Number(entityId) >= Number(existingService.currentRevision ?? 0)
+        ? entityId
+        : existingService.currentRevision,
     role: roles.length > 0 ? roles : existingService.role,
     usage: newRevision.usage,
     startedAt: newRevision.createdAt || existingService.startedAt,
@@ -276,10 +281,14 @@ const updateServiceWithRevision = (
       deploymentErrorEvent = updatedService;
     }
   }
+  const incomingRevisionId = Number(entityId);
+  const currentRevisionId = Number(existingService.currentRevision ?? 0);
+  if (incomingRevisionId >= currentRevisionId && !eventData.status.error) {
+    updatedService.error = undefined;
+  }
 
   return { updatedService, deploymentErrorEvent };
 };
-
 const updateEnvironmentConsumption = (
   environment: Environment,
   servicesMap: Map<string, Service>,
@@ -535,7 +544,8 @@ export const extractStructuredSchema = (
   if (!configSchema) return {};
   const deployedConfig = revisionData?.config?.config ?? {};
   const parameterValues: Record<string, any> = deployedConfig.parameter ?? {};
-  const configResourceValues: Record<string, any> = deployedConfig.resource ?? {};
+  const configResourceValues: Record<string, any> =
+    deployedConfig.resource ?? {};
 
   const requiredParams: string[] =
     configSchema?.properties?.config?.required ?? [];
@@ -566,14 +576,22 @@ export const extractStructuredSchema = (
 
   const resolveResourceValue = (configValue: any): string => {
     if (!configValue || typeof configValue !== "object") return "";
-    if (typeof configValue.secret === "string") return extractResourceName(configValue.secret);
-    if (typeof configValue.domain === "string") return extractResourceName(configValue.domain);
-    if (typeof configValue.port === "string") return extractResourceName(configValue.port);
-    if (typeof configValue.certificate === "string") return extractResourceName(configValue.certificate);
-    if (typeof configValue.ca === "string") return extractResourceName(configValue.ca);
+    if (typeof configValue.secret === "string")
+      return extractResourceName(configValue.secret);
+    if (typeof configValue.domain === "string")
+      return extractResourceName(configValue.domain);
+    if (typeof configValue.port === "string")
+      return extractResourceName(configValue.port);
+    if (typeof configValue.certificate === "string")
+      return extractResourceName(configValue.certificate);
+    if (typeof configValue.ca === "string")
+      return extractResourceName(configValue.ca);
     if (configValue.volume) {
       const vol = configValue.volume;
-      return convertToGigabytes(vol.size, vol.unit)?.toString() ?? String(vol.size ?? "");
+      return (
+        convertToGigabytes(vol.size, vol.unit)?.toString() ??
+        String(vol.size ?? "")
+      );
     }
     return "";
   };
@@ -656,7 +674,9 @@ export const processRevisionData = (
 ): Service => {
   const { solution } = revisionData;
   const topLevelConfig = revisionData?.config?.config ?? {};
-  const topLevelParameters = extractParametersFromConfig(topLevelConfig.parameter || {});
+  const topLevelParameters = extractParametersFromConfig(
+    topLevelConfig.parameter || {},
+  );
   const topLevelResources = extractResources(topLevelConfig.resource || {});
 
   const schemaResult = revisionData.revision
@@ -736,11 +756,21 @@ export const processRevisionData = (
             id: revisionId,
             schema,
             usage: svc.usage || {
-              current: { cpu: 0, memory: 0, storage: 0, volatileStorage: 0, nonReplicatedStorage: 0, persistentStorage: 0 },
+              current: {
+                cpu: 0,
+                memory: 0,
+                storage: 0,
+                volatileStorage: 0,
+                nonReplicatedStorage: 0,
+                persistentStorage: 0,
+              },
               limit: {
-                cpu: { max: 0, min: 0 }, memory: { max: 0, min: 0 },
-                storage: { max: 0, min: 0 }, volatileStorage: { max: 0, min: 0 },
-                nonReplicatedStorage: { max: 0, min: 0 }, persistentStorage: { max: 0, min: 0 },
+                cpu: { max: 0, min: 0 },
+                memory: { max: 0, min: 0 },
+                storage: { max: 0, min: 0 },
+                volatileStorage: { max: 0, min: 0 },
+                nonReplicatedStorage: { max: 0, min: 0 },
+                persistentStorage: { max: 0, min: 0 },
               },
               cost: 0,
             },
@@ -754,7 +784,10 @@ export const processRevisionData = (
     return {
       ...svc,
       revisions: updatedRevisions,
-      currentRevision: Number(revisionId) >= Number(currentRevId) ? revisionId : svc.currentRevision,
+      currentRevision:
+        Number(revisionId) >= Number(currentRevId)
+          ? revisionId
+          : svc.currentRevision,
     };
   };
 
@@ -771,11 +804,21 @@ export const processRevisionData = (
         ),
       );
     }
-    return applySchemaToService({ ...service, parameters: topLevelParameters, resources: topLevelResources });
+    return applySchemaToService({
+      ...service,
+      parameters: topLevelParameters,
+      resources: topLevelResources,
+    });
   }
 
   return applySchemaToService(
-    processDeployment(service, deployment, revisionData, topLevelParameters, topLevelResources),
+    processDeployment(
+      service,
+      deployment,
+      revisionData,
+      topLevelParameters,
+      topLevelResources,
+    ),
   );
 };
 
@@ -796,13 +839,14 @@ export const processDeployment = (
     Object.entries(rolesDefinition).forEach(
       ([roleName, roleData]: [string, any]) => {
         const existingRole = service.role.find((r) => r.name === roleName);
-        const roleArtifactConfig = roleData?.artifact?.description?.config ?? {};
+        const roleArtifactConfig =
+          roleData?.artifact?.description?.config ?? {};
 
         const roleParameters = extractParametersFromConfig(
-          roleArtifactConfig.parameter ?? {}
+          roleArtifactConfig.parameter ?? {},
         );
         const roleResources = extractResources(
-          roleArtifactConfig.resource ?? {}
+          roleArtifactConfig.resource ?? {},
         );
         const parameters = roleParameters;
         const resources = roleResources;
@@ -819,8 +863,7 @@ export const processDeployment = (
           roleArtifactConfig.resource || {},
         ).some((resourceData: any) => resourceData.volume);
 
-        const artifactName: string =
-          roleData.artifact?.ref?.name || "";
+        const artifactName: string = roleData.artifact?.ref?.name || "";
         const role: Role = {
           name: roleName,
           artifactName: artifactName || undefined,
@@ -868,7 +911,12 @@ export const processDeployment = (
       },
     ];
   }
-  return { ...service, resources: fallbackResources, parameters: fallbackParameters, role: updatedRoles };
+  return {
+    ...service,
+    resources: fallbackResources,
+    parameters: fallbackParameters,
+    role: updatedRoles,
+  };
 };
 
 export const extractParametersFromConfig = (
@@ -1081,4 +1129,4 @@ const extractResourceName = (resourcePath: string): string => {
     return parts[1];
   }
   return resourcePath;
-};
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kumori/aurora-backend-handler",
-  "version": "1.0.62",
+  "version": "1.0.63",
   "description": "backend handler",
   "main": "backend-handler.ts",
   "scripts": {