@abtnode/core 1.16.0-beta-ad6df3ae → 1.16.0-beta-58020de5

package/lib/blocklet/manager/disk.js

@@ -150,10 +150,19 @@ const pm2StatusMap = {
  */
 const getBlockletEngineNameByPlatform = (blockletMeta) => getBlockletEngine(blockletMeta).interpreter;
 
-const getSkippedProcessIds = ({ newBlocklet, oldBlocklet, context = {} }) => {
+/**
+ * @param {{
+ *   newBlocklet,
+ *   oldBlocklet,
+ *   context: { forceStartProcessIds?: string[] },
+ * }}
+ * @returns {{ installedComponentNames: string[], skippedProcessIds: string[] }}
+ */
+const getComponentChangedInfoForUpgrade = ({ newBlocklet, oldBlocklet, context = {} }) => {
   const { forceStartProcessIds = [] } = context;
   const idMap = {};
-  const res = [];
+  const skippedProcessIds = [];
+  const installedComponentNames = [];
 
   forEachBlockletSync(oldBlocklet, (b, { ancestors }) => {
     if (b.meta.dist?.integrity) {
@@ -164,15 +173,18 @@ const getSkippedProcessIds = ({ newBlocklet, oldBlocklet, context = {} }) => {
   forEachBlockletSync(newBlocklet, (b, { ancestors }) => {
     const id = getComponentProcessId(b, ancestors);
     if (forceStartProcessIds.includes(id)) {
+      installedComponentNames.push(b.meta.title);
       return;
     }
 
     if (!b.meta.dist?.integrity || b.meta.dist.integrity === idMap[id]) {
-      res.push(id);
+      skippedProcessIds.push(id);
+    } else {
+      installedComponentNames.push(b.meta.title);
     }
   });
 
-  return res;
+  return { skippedProcessIds, installedComponentNames };
 };
 
 // 10s 上报统计一次
@@ -506,7 +518,7 @@ class BlockletManager extends BaseBlockletManager {
 
       const error = Array.isArray(err) ? err[0] : err;
       logger.error('Failed to start blocklet', { error, did, name: blocklet.meta.name });
-      const description = `Start blocklet ${blocklet.meta.name} failed with error: ${error.message}`;
+      const description = `Start blocklet ${blocklet.meta.title} failed with error: ${error.message}`;
       this._createNotification(did, {
         title: 'Start Blocklet Failed',
         description,
@@ -635,7 +647,7 @@ class BlockletManager extends BaseBlockletManager {
 
       this._createNotification(did, {
         title: 'Blocklet Restart Failed',
-        description: `Blocklet ${did} restart failed with error: ${err.message || 'queue exception'}`,
+        description: `Blocklet ${result.meta.title} restart failed with error: ${err.message || 'queue exception'}`,
         entityType: 'blocklet',
         entityId: did,
         severity: 'error',
@@ -686,7 +698,7 @@ class BlockletManager extends BaseBlockletManager {
       const doc = await this._deleteBlocklet({ did, keepData, keepLogsDir, keepConfigs }, context);
       this._createNotification(doc.meta.did, {
         title: 'Blocklet Deleted',
-        description: `Blocklet ${doc.meta.name}@${doc.meta.version} is deleted.`,
+        description: `Blocklet ${doc.meta.title} is deleted.`,
         entityType: 'blocklet',
         entityId: doc.meta.did,
         severity: 'success',
@@ -699,7 +711,7 @@ class BlockletManager extends BaseBlockletManager {
 
       this._createNotification(doc.meta.did, {
         title: 'Blocklet Deleted',
-        description: `Blocklet ${doc.meta.name}@${doc.meta.version} is deleted.`,
+        description: `Blocklet ${doc.meta.title} is deleted.`,
         entityType: 'blocklet',
         entityId: doc.meta.did,
         severity: 'success',
@@ -817,7 +829,7 @@ class BlockletManager extends BaseBlockletManager {
 
     this._createNotification(newBlocklet.meta.did, {
       title: 'Component Deleted',
-      description: `Component ${child.meta.name} of ${newBlocklet.meta.name} is successfully deleted.`,
+      description: `Component ${child.meta.title} of ${newBlocklet.meta.title} is successfully deleted.`,
       entityType: 'blocklet',
       entityId: newBlocklet.meta.did,
       severity: 'success',
@@ -1069,6 +1081,24 @@ class BlockletManager extends BaseBlockletManager {
       await this._updateDidDocument(blocklet);
     }
 
+    // update blocklet meta
+    if (blocklet.structVersion && !childDids.length) {
+      const changedTitle = newConfigs.find((x) => x.key === BLOCKLET_CONFIGURABLE_KEY.BLOCKLET_APP_NAME)?.value;
+      const changedDescription = newConfigs.find(
+        (x) => x.key === BLOCKLET_CONFIGURABLE_KEY.BLOCKLET_APP_DESCRIPTION
+      )?.value;
+
+      if (changedTitle || changedDescription) {
+        await states.blocklet.updateBlocklet(rootDid, {
+          meta: {
+            ...blocklet.meta,
+            title: changedTitle || blocklet.meta.title,
+            description: changedDescription || blocklet.meta.description,
+          },
+        });
+      }
+    }
+
     await this._updateBlockletEnvironment(rootDid);
 
     // response
@@ -1418,7 +1448,7 @@ class BlockletManager extends BaseBlockletManager {
       skipCheckIntegrity,
     } = params;
     const { meta } = blocklet;
-    const { name, did, version } = meta;
+    const { title, name, did, version } = meta;
 
     // check status
     if (!skipCheckStatusBeforeDownload) {
@@ -1489,7 +1519,7 @@ class BlockletManager extends BaseBlockletManager {
       });
       this._createNotification(did, {
         title: 'Blocklet Download Failed',
-        description: `Blocklet ${name}@${version} download failed with error: ${err.message}`,
+        description: `Blocklet ${title} download failed with error: ${err.message}`,
         entityType: 'blocklet',
         entityId: did,
         severity: 'error',
@@ -1610,7 +1640,7 @@ class BlockletManager extends BaseBlockletManager {
     const blocklet = await this.getBlocklet(did);
 
     const { meta } = blocklet;
-    const { name } = meta;
+    const { name, title } = meta;
 
     try {
       // healthy check
@@ -1633,7 +1663,7 @@ class BlockletManager extends BaseBlockletManager {
 
       this._createNotification(did, {
         title: 'Blocklet Start Failed',
-        description: `Blocklet ${name} start failed: ${error.message}`,
+        description: `Blocklet ${title} start failed: ${error.message}`,
         entityType: 'blocklet',
         entityId: did,
         severity: 'error',
@@ -1947,7 +1977,7 @@ class BlockletManager extends BaseBlockletManager {
         });
         this._createNotification(did, {
           title: 'Blocklet Install Failed',
-          description: `Blocklet ${meta.name}@${meta.version} install failed with error: ${err.message}`,
+          description: `Blocklet ${meta.title} install failed with error: ${err.message}`,
           entityType: 'blocklet',
           entityId: did,
           severity: 'error',
@@ -1962,10 +1992,15 @@ class BlockletManager extends BaseBlockletManager {
 
   async _upgradeBlocklet({ newBlocklet, oldBlocklet, context = {} }) {
     const { meta, source, deployedFrom, children } = newBlocklet;
-    const { did, version, name } = meta;
+    const { did, version, name, title } = meta;
 
     // ids
-    context.skippedProcessIds = getSkippedProcessIds({ newBlocklet, oldBlocklet, context });
+    const { skippedProcessIds, installedComponentNames } = getComponentChangedInfoForUpgrade({
+      newBlocklet,
+      oldBlocklet,
+      context,
+    });
+    context.skippedProcessIds = skippedProcessIds;
 
     try {
       // delete old process
@@ -2040,9 +2075,7 @@ class BlockletManager extends BaseBlockletManager {
         this.emit(BlockletEvents.upgraded, { blocklet, context });
         this._createNotification(did, {
           title: 'Blocklet Upgrade Success',
-          description: `Blocklet ${name}@${version} upgrade successfully. (Source: ${
-            deployedFrom || fromBlockletSource(source)
-          })`,
+          description: `Blocklet ${title} upgrade successfully. (Component: ${installedComponentNames.join(', ')})`,
           action: `/blocklets/${did}/overview`,
           entityType: 'blocklet',
           entityId: did,
@@ -2071,7 +2104,9 @@ class BlockletManager extends BaseBlockletManager {
 
       this._createNotification(did, {
         title: 'Blocklet Upgrade Failed',
-        description: `Blocklet ${name}@${version} upgrade failed with error: ${err.message}`,
+        description: `Blocklet ${title} upgrade failed with error: ${
+          err.message
+        }. (Component: ${installedComponentNames.join(', ')})`,
         entityType: 'blocklet',
         entityId: did,
         severity: 'error',

package/lib/blocklet/manager/helper/install-component-from-url.js

@@ -66,7 +66,11 @@ const installComponentFromUrl = async ({
   }
 
   if (!isComponentBlocklet(meta)) {
-    throw new Error('The blocklet cannot be a component');
+    throw new Error(
+      `The blocklet cannot be a component : ${
+        meta.title || meta.name
+      }. The reason may be that the developer set capabilities.component to false in blocklet.yml`
+    );
   }
 
   if (title) {

package/lib/blocklet/storage/utils/disk.js

@@ -9,7 +9,7 @@ const restoreDirName = 'tmp/restore-disk';
 const getBackupList = (dataDir) => {
   const baseBackupDir = path.join(dataDir, backupDirName);
   const backupList = [];
-  const appDidList = fs.readdirSync(baseBackupDir);
+  const appDidList = fs.existsSync(baseBackupDir) ? fs.readdirSync(baseBackupDir) : [];
 
   appDidList.forEach((appDid) => {
     const metaFile = path.join(baseBackupDir, appDid, 'meta.json');

package/lib/util/blocklet.js

@@ -814,7 +814,9 @@ const parseComponents = async (component, context = {}) => {
     validateBlockletMeta(rawMeta, { ensureDist: true });
 
     if (!isComponentBlocklet(rawMeta)) {
-      throw new Error(`The blocklet cannot be a component: ${rawMeta.title}`);
+      throw new Error(
+        `The blocklet cannot be a component: ${rawMeta.title}. The reason may be that the developer set capabilities.component to false in blocklet.yml`
+      );
     }
 
     const webInterface = findWebInterface(rawMeta);

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.16.0-beta-ad6df3ae",
+  "version": "1.16.0-beta-58020de5",
   "description": "",
   "main": "lib/index.js",
   "files": [
@@ -19,18 +19,18 @@
   "author": "wangshijun <wangshijun2010@gmail.com> (http://github.com/wangshijun)",
   "license": "MIT",
   "dependencies": {
-    "@abtnode/auth": "1.16.0-beta-ad6df3ae",
-    "@abtnode/certificate-manager": "1.16.0-beta-ad6df3ae",
-    "@abtnode/constant": "1.16.0-beta-ad6df3ae",
-    "@abtnode/cron": "1.16.0-beta-ad6df3ae",
-    "@abtnode/db": "1.16.0-beta-ad6df3ae",
-    "@abtnode/logger": "1.16.0-beta-ad6df3ae",
-    "@abtnode/queue": "1.16.0-beta-ad6df3ae",
-    "@abtnode/rbac": "1.16.0-beta-ad6df3ae",
-    "@abtnode/router-provider": "1.16.0-beta-ad6df3ae",
-    "@abtnode/static-server": "1.16.0-beta-ad6df3ae",
-    "@abtnode/timemachine": "1.16.0-beta-ad6df3ae",
-    "@abtnode/util": "1.16.0-beta-ad6df3ae",
+    "@abtnode/auth": "1.16.0-beta-58020de5",
+    "@abtnode/certificate-manager": "1.16.0-beta-58020de5",
+    "@abtnode/constant": "1.16.0-beta-58020de5",
+    "@abtnode/cron": "1.16.0-beta-58020de5",
+    "@abtnode/db": "1.16.0-beta-58020de5",
+    "@abtnode/logger": "1.16.0-beta-58020de5",
+    "@abtnode/queue": "1.16.0-beta-58020de5",
+    "@abtnode/rbac": "1.16.0-beta-58020de5",
+    "@abtnode/router-provider": "1.16.0-beta-58020de5",
+    "@abtnode/static-server": "1.16.0-beta-58020de5",
+    "@abtnode/timemachine": "1.16.0-beta-58020de5",
+    "@abtnode/util": "1.16.0-beta-58020de5",
     "@arcblock/did": "1.18.64",
     "@arcblock/did-motif": "^1.1.10",
     "@arcblock/did-util": "1.18.64",
@@ -38,9 +38,9 @@
     "@arcblock/jwt": "^1.18.64",
     "@arcblock/pm2-events": "^0.0.5",
     "@arcblock/vc": "1.18.64",
-    "@blocklet/constant": "1.16.0-beta-ad6df3ae",
-    "@blocklet/meta": "1.16.0-beta-ad6df3ae",
-    "@blocklet/sdk": "1.16.0-beta-ad6df3ae",
+    "@blocklet/constant": "1.16.0-beta-58020de5",
+    "@blocklet/meta": "1.16.0-beta-58020de5",
+    "@blocklet/sdk": "1.16.0-beta-58020de5",
     "@did-space/client": "^0.2.45",
     "@fidm/x509": "^1.2.1",
     "@ocap/client": "1.18.64",
@@ -91,5 +91,5 @@
     "express": "^4.18.2",
     "jest": "^27.5.1"
   },
-  "gitHead": "8fa60176a00400cd412b308df3c7d6dcd9af2db1"
+  "gitHead": "9d3f47f9827acf13e9efea38cd605d3b36f9f523"
 }