@verdaccio/store 6.0.0-6-next.28 → 6.0.0-6-next.30

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @verdaccio/store
 
+## 6.0.0-6-next.30
+
+### Patch Changes
+
+- @verdaccio/core@6.0.0-6-next.50
+- @verdaccio/config@6.0.0-6-next.50
+- @verdaccio/tarball@11.0.0-6-next.19
+- @verdaccio/url@11.0.0-6-next.16
+- @verdaccio/hooks@6.0.0-6-next.20
+- @verdaccio/loaders@6.0.0-6-next.19
+- @verdaccio/logger@6.0.0-6-next.18
+- @verdaccio/local-storage@11.0.0-6-next.20
+- @verdaccio/proxy@6.0.0-6-next.28
+- @verdaccio/utils@6.0.0-6-next.18
+
+## 6.0.0-6-next.29
+
+### Minor Changes
+
+- ce013d2f: refactor: npm star command support reimplemented
+
+### Patch Changes
+
+- Updated dependencies [ce013d2f]
+  - @verdaccio/url@11.0.0-6-next.15
+  - @verdaccio/tarball@11.0.0-6-next.18
+  - @verdaccio/local-storage@11.0.0-6-next.19
+  - @verdaccio/core@6.0.0-6-next.49
+  - @verdaccio/config@6.0.0-6-next.49
+  - @verdaccio/hooks@6.0.0-6-next.19
+  - @verdaccio/loaders@6.0.0-6-next.18
+  - @verdaccio/logger@6.0.0-6-next.17
+  - @verdaccio/proxy@6.0.0-6-next.27
+  - @verdaccio/utils@6.0.0-6-next.17
+
 ## 6.0.0-6-next.28
 
 ### Major Changes

package/build/lib/star-utils.d.ts

@@ -1,14 +1,16 @@
-import { Manifest } from '@verdaccio/types';
-import { Users } from '../type';
+import { Manifest, PackageUsers } from '@verdaccio/types';
 /**
- * Check whether the package metadta has enough data to be published
+ * Check whether the package metadata has enough data to be published
  * @param pkg metadata
  */
+export declare function isPublishablePackage(pkg: Manifest): boolean;
 /**
- * Check whether the package metadta has enough data to be published
- * @param pkg metadata
+ * Verify if the user is actually executing an action, to avoid unnecessary calls
+ * to the storage.
+ * @param localUsers current state at cache
+ * @param username user is executing the action
+ * @param userIsAddingStar whether user is removing or adding star
+ * @returns boolean
  */
-export declare function isPublishablePackage(pkg: Manifest): boolean;
-export declare function isRelatedToDeprecation(pkgInfo: Manifest): boolean;
-export declare function validateInputs(localUsers: Users, username: string, isStar: boolean): boolean;
+export declare function isExecutingStarCommand(localUsers: PackageUsers, username: string, userIsAddingStar: boolean): boolean;
 export declare function isStarManifest(manifest: Manifest): boolean;

package/build/lib/star-utils.js

@@ -3,59 +3,43 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
+exports.isExecutingStarCommand = isExecutingStarCommand;
 exports.isPublishablePackage = isPublishablePackage;
-exports.isRelatedToDeprecation = isRelatedToDeprecation;
 exports.isStarManifest = isStarManifest;
-exports.validateInputs = validateInputs;
-
-var _lodash = _interopRequireDefault(require("lodash"));
 
 var _core = require("@verdaccio/core");
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-
 /**
- * Check whether the package metadta has enough data to be published
- * @param pkg metadata
- */
-
-/**
- * Check whether the package metadta has enough data to be published
+ * Check whether the package metadata has enough data to be published
  * @param pkg metadata
  */
 function isPublishablePackage(pkg) {
   // TODO: we can do better, no need get keys
   const keys = Object.keys(pkg);
   return keys.includes('versions');
-} // @deprecated don't think this is used anymore (REMOVE)
-
-
-function isRelatedToDeprecation(pkgInfo) {
-  const {
-    versions
-  } = pkgInfo;
-
-  for (const version in versions) {
-    if (Object.prototype.hasOwnProperty.call(versions[version], 'deprecated')) {
-      return true;
-    }
-  }
-
-  return false;
 }
+/**
+ * Verify if the user is actually executing an action, to avoid unnecessary calls
+ * to the storage.
+ * @param localUsers current state at cache
+ * @param username user is executing the action
+ * @param userIsAddingStar whether user is removing or adding star
+ * @returns boolean
+ */
+
 
-function validateInputs(localUsers, username, isStar) {
-  const isExistlocalUsers = _lodash.default.isNil(localUsers[username]) === false;
+function isExecutingStarCommand(localUsers, username, userIsAddingStar) {
+  const isExist = typeof localUsers[username] !== 'undefined'; // fails if user already exist and us trying to add star.
 
-  if (isStar && isExistlocalUsers && localUsers[username]) {
-    return true;
-  } else if (!isStar && isExistlocalUsers) {
+  if (userIsAddingStar && isExist && localUsers[username]) {
+    return false; // if is not adding a start but user exists (removing star)
+  } else if (!userIsAddingStar && isExist) {
+    return true; // fails if user does not exist and is not adding any star
+  } else if (!userIsAddingStar && !isExist) {
     return false;
-  } else if (!isStar && !isExistlocalUsers) {
-    return true;
   }
 
-  return false;
+  return true;
 }
 
 function isStarManifest(manifest) {

package/build/lib/star-utils.js.map

@@ -1 +1 @@
-{"version":3,"file":"star-utils.js","names":["isPublishablePackage","pkg","keys","Object","includes","isRelatedToDeprecation","pkgInfo","versions","version","prototype","hasOwnProperty","call","validateInputs","localUsers","username","isStar","isExistlocalUsers","_","isNil","isStarManifest","manifest","validatioUtils","isObject","users"],"sources":["../../src/lib/star-utils.ts"],"sourcesContent":["import _ from 'lodash';\n\nimport { validatioUtils } from '@verdaccio/core';\nimport { Manifest } from '@verdaccio/types';\n\nimport { Users } from '../type';\n\n/**\n * Check whether the package metadta has enough data to be published\n * @param pkg metadata\n */\n\n/**\n * Check whether the package metadta has enough data to be published\n * @param pkg metadata\n */\nexport function isPublishablePackage(pkg: Manifest): boolean {\n  // TODO: we can do better, no need get keys\n  const keys: string[] = Object.keys(pkg);\n\n  return keys.includes('versions');\n}\n\n// @deprecated don't think this is used anymore (REMOVE)\nexport function isRelatedToDeprecation(pkgInfo: Manifest): boolean {\n  const { versions } = pkgInfo;\n  for (const version in versions) {\n    if (Object.prototype.hasOwnProperty.call(versions[version], 'deprecated')) {\n      return true;\n    }\n  }\n  return false;\n}\n\nexport function validateInputs(localUsers: Users, username: string, isStar: boolean): boolean {\n  const isExistlocalUsers = _.isNil(localUsers[username]) === false;\n  if (isStar && isExistlocalUsers && localUsers[username]) {\n    return true;\n  } else if (!isStar && isExistlocalUsers) {\n    return false;\n  } else if (!isStar && !isExistlocalUsers) {\n    return true;\n  }\n  return false;\n}\n\nexport function isStarManifest(manifest: Manifest): boolean {\n  return isPublishablePackage(manifest) === false && validatioUtils.isObject(manifest.users);\n}\n"],"mappings":";;;;;;;;;;AAAA;;AAEA;;;;AAKA;AACA;AACA;AACA;;AAEA;AACA;AACA;AACA;AACO,SAASA,oBAAT,CAA8BC,GAA9B,EAAsD;EAC3D;EACA,MAAMC,IAAc,GAAGC,MAAM,CAACD,IAAP,CAAYD,GAAZ,CAAvB;EAEA,OAAOC,IAAI,CAACE,QAAL,CAAc,UAAd,CAAP;AACD,C,CAED;;;AACO,SAASC,sBAAT,CAAgCC,OAAhC,EAA4D;EACjE,MAAM;IAAEC;EAAF,IAAeD,OAArB;;EACA,KAAK,MAAME,OAAX,IAAsBD,QAAtB,EAAgC;IAC9B,IAAIJ,MAAM,CAACM,SAAP,CAAiBC,cAAjB,CAAgCC,IAAhC,CAAqCJ,QAAQ,CAACC,OAAD,CAA7C,EAAwD,YAAxD,CAAJ,EAA2E;MACzE,OAAO,IAAP;IACD;EACF;;EACD,OAAO,KAAP;AACD;;AAEM,SAASI,cAAT,CAAwBC,UAAxB,EAA2CC,QAA3C,EAA6DC,MAA7D,EAAuF;EAC5F,MAAMC,iBAAiB,GAAGC,eAAA,CAAEC,KAAF,CAAQL,UAAU,CAACC,QAAD,CAAlB,MAAkC,KAA5D;;EACA,IAAIC,MAAM,IAAIC,iBAAV,IAA+BH,UAAU,CAACC,QAAD,CAA7C,EAAyD;IACvD,OAAO,IAAP;EACD,CAFD,MAEO,IAAI,CAACC,MAAD,IAAWC,iBAAf,EAAkC;IACvC,OAAO,KAAP;EACD,CAFM,MAEA,IAAI,CAACD,MAAD,IAAW,CAACC,iBAAhB,EAAmC;IACxC,OAAO,IAAP;EACD;;EACD,OAAO,KAAP;AACD;;AAEM,SAASG,cAAT,CAAwBC,QAAxB,EAAqD;EAC1D,OAAOpB,oBAAoB,CAACoB,QAAD,CAApB,KAAmC,KAAnC,IAA4CC,oBAAA,CAAeC,QAAf,CAAwBF,QAAQ,CAACG,KAAjC,CAAnD;AACD"}
+{"version":3,"file":"star-utils.js","names":["isPublishablePackage","pkg","keys","Object","includes","isExecutingStarCommand","localUsers","username","userIsAddingStar","isExist","isStarManifest","manifest","validatioUtils","isObject","users"],"sources":["../../src/lib/star-utils.ts"],"sourcesContent":["import { validatioUtils } from '@verdaccio/core';\nimport { Manifest, PackageUsers } from '@verdaccio/types';\n\n/**\n * Check whether the package metadata has enough data to be published\n * @param pkg metadata\n */\nexport function isPublishablePackage(pkg: Manifest): boolean {\n  // TODO: we can do better, no need get keys\n  const keys: string[] = Object.keys(pkg);\n\n  return keys.includes('versions');\n}\n\n/**\n * Verify if the user is actually executing an action, to avoid unnecessary calls\n * to the storage.\n * @param localUsers current state at cache\n * @param username user is executing the action\n * @param userIsAddingStar whether user is removing or adding star\n * @returns boolean\n */\nexport function isExecutingStarCommand(\n  localUsers: PackageUsers,\n  username: string,\n  userIsAddingStar: boolean\n): boolean {\n  const isExist = typeof localUsers[username] !== 'undefined';\n  // fails if user already exist and us trying to add star.\n  if (userIsAddingStar && isExist && localUsers[username]) {\n    return false;\n    // if is not adding a start but user exists (removing star)\n  } else if (!userIsAddingStar && isExist) {\n    return true;\n    // fails if user does not exist and is not adding any star\n  } else if (!userIsAddingStar && !isExist) {\n    return false;\n  }\n  return true;\n}\n\nexport function isStarManifest(manifest: Manifest): boolean {\n  return isPublishablePackage(manifest) === false && validatioUtils.isObject(manifest.users);\n}\n"],"mappings":";;;;;;;;;AAAA;;AAGA;AACA;AACA;AACA;AACO,SAASA,oBAAT,CAA8BC,GAA9B,EAAsD;EAC3D;EACA,MAAMC,IAAc,GAAGC,MAAM,CAACD,IAAP,CAAYD,GAAZ,CAAvB;EAEA,OAAOC,IAAI,CAACE,QAAL,CAAc,UAAd,CAAP;AACD;AAED;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;;AACO,SAASC,sBAAT,CACLC,UADK,EAELC,QAFK,EAGLC,gBAHK,EAII;EACT,MAAMC,OAAO,GAAG,OAAOH,UAAU,CAACC,QAAD,CAAjB,KAAgC,WAAhD,CADS,CAET;;EACA,IAAIC,gBAAgB,IAAIC,OAApB,IAA+BH,UAAU,CAACC,QAAD,CAA7C,EAAyD;IACvD,OAAO,KAAP,CADuD,CAEvD;EACD,CAHD,MAGO,IAAI,CAACC,gBAAD,IAAqBC,OAAzB,EAAkC;IACvC,OAAO,IAAP,CADuC,CAEvC;EACD,CAHM,MAGA,IAAI,CAACD,gBAAD,IAAqB,CAACC,OAA1B,EAAmC;IACxC,OAAO,KAAP;EACD;;EACD,OAAO,IAAP;AACD;;AAEM,SAASC,cAAT,CAAwBC,QAAxB,EAAqD;EAC1D,OAAOX,oBAAoB,CAACW,QAAD,CAApB,KAAmC,KAAnC,IAA4CC,oBAAA,CAAeC,QAAf,CAAwBF,QAAQ,CAACG,KAAjC,CAAnD;AACD"}

package/build/local-storage.d.ts

@@ -2,17 +2,17 @@ import { pluginUtils } from '@verdaccio/core';
 import { Config, Logger } from '@verdaccio/types';
 export declare const noSuchFile = "ENOENT";
 export declare const resourceNotAvailable = "EAGAIN";
-export declare type IPluginStorage = pluginUtils.IPluginStorage<Config>;
+export declare type PluginStorage = pluginUtils.Storage<Config>;
 /**
  * Implements Storage interface (same for storage.js, local-storage.js, up-storage.js).
  */
 declare class LocalStorage {
     config: Config;
-    storagePlugin: IPluginStorage;
+    storagePlugin: PluginStorage;
     logger: Logger;
     constructor(config: Config, logger: Logger);
     init(): Promise<void>;
-    getStoragePlugin(): IPluginStorage;
+    getStoragePlugin(): PluginStorage;
     getSecret(config: Config): Promise<void>;
     private loadStorage;
     private loadStorePlugin;

package/build/local-storage.js

@@ -84,7 +84,7 @@ class LocalStorage {
       config: this.config,
       logger: this.logger
     }, plugin => {
-      return plugin.getPackageStorage;
+      return typeof plugin.getPackageStorage !== 'undefined';
     }, (_this$config = this.config) === null || _this$config === void 0 ? void 0 : (_this$config$serverSe = _this$config.serverSettings) === null || _this$config$serverSe === void 0 ? void 0 : _this$config$serverSe.pluginPrefix);
 
     if (plugins.length > 1) {

package/build/local-storage.js.map

@@ -1 +1 @@
-{"version":3,"file":"local-storage.js","names":["debug","buildDebug","noSuchFile","resourceNotAvailable","LocalStorage","constructor","config","logger","child","sub","storagePlugin","init","loadStorage","warn","getStoragePlugin","errorUtils","getInternalError","getSecret","secretKey","setSecret","checkSecretKey","Storage","loadStorePlugin","_","isNil","assert","storage","LocalDatabase","plugins","asyncLoadPlugin","store","plugin","getPackageStorage","serverSettings","pluginPrefix","length","head"],"sources":["../src/local-storage.ts"],"sourcesContent":["import assert from 'assert';\nimport buildDebug from 'debug';\nimport _ from 'lodash';\n\nimport { errorUtils, pluginUtils } from '@verdaccio/core';\nimport { asyncLoadPlugin } from '@verdaccio/loaders';\nimport LocalDatabase from '@verdaccio/local-storage';\nimport { Config, Logger } from '@verdaccio/types';\n\nconst debug = buildDebug('verdaccio:storage:local');\n\nexport const noSuchFile = 'ENOENT';\nexport const resourceNotAvailable = 'EAGAIN';\nexport type IPluginStorage = pluginUtils.IPluginStorage<Config>;\n\n/**\n * Implements Storage interface (same for storage.js, local-storage.js, up-storage.js).\n */\nclass LocalStorage {\n  public config: Config;\n  public storagePlugin: IPluginStorage;\n  public logger: Logger;\n\n  public constructor(config: Config, logger: Logger) {\n    debug('local storage created');\n    this.logger = logger.child({ sub: 'fs' });\n    this.config = config;\n    // @ts-expect-error\n    this.storagePlugin = null;\n  }\n\n  public async init() {\n    if (this.storagePlugin === null) {\n      this.storagePlugin = await this.loadStorage(this.config, this.logger);\n      debug('storage plugin init');\n      await this.storagePlugin.init();\n      debug('storage plugin initialized');\n    } else {\n      this.logger.warn('storage plugin has been already initialized');\n    }\n    return;\n  }\n\n  public getStoragePlugin(): IPluginStorage {\n    if (this.storagePlugin === null) {\n      throw errorUtils.getInternalError('storage plugin is not initialized');\n    }\n\n    return this.storagePlugin;\n  }\n\n  public async getSecret(config: Config): Promise<void> {\n    const secretKey = await this.storagePlugin.getSecret();\n\n    return this.storagePlugin.setSecret(config.checkSecretKey(secretKey));\n  }\n\n  private async loadStorage(config: Config, logger: Logger): Promise<IPluginStorage> {\n    const Storage = await this.loadStorePlugin();\n    if (_.isNil(Storage)) {\n      assert(this.config.storage, 'CONFIG: storage path not defined');\n      debug('no custom storage found, loading default storage @verdaccio/local-storage');\n      return new LocalDatabase(config, logger);\n    }\n    return Storage as IPluginStorage;\n  }\n\n  private async loadStorePlugin(): Promise<IPluginStorage | undefined> {\n    const plugins: IPluginStorage[] = await asyncLoadPlugin<IPluginStorage>(\n      this.config.store,\n      {\n        config: this.config,\n        logger: this.logger,\n      },\n      (plugin): IPluginStorage => {\n        return plugin.getPackageStorage;\n      },\n      this.config?.serverSettings?.pluginPrefix\n    );\n\n    if (plugins.length > 1) {\n      this.logger.warn(\n        'more than one storage plugins has been detected, multiple storage are not supported, one will be selected automatically'\n      );\n    }\n\n    return _.head(plugins);\n  }\n}\n\nexport { LocalStorage };\n"],"mappings":";;;;;;;AAAA;;AACA;;AACA;;AAEA;;AACA;;AACA;;;;AAGA,MAAMA,KAAK,GAAG,IAAAC,cAAA,EAAW,yBAAX,CAAd;AAEO,MAAMC,UAAU,GAAG,QAAnB;;AACA,MAAMC,oBAAoB,GAAG,QAA7B;;;AAGP;AACA;AACA;AACA,MAAMC,YAAN,CAAmB;EAKVC,WAAW,CAACC,MAAD,EAAiBC,MAAjB,EAAiC;IACjDP,KAAK,CAAC,uBAAD,CAAL;IACA,KAAKO,MAAL,GAAcA,MAAM,CAACC,KAAP,CAAa;MAAEC,GAAG,EAAE;IAAP,CAAb,CAAd;IACA,KAAKH,MAAL,GAAcA,MAAd,CAHiD,CAIjD;;IACA,KAAKI,aAAL,GAAqB,IAArB;EACD;;EAEgB,MAAJC,IAAI,GAAG;IAClB,IAAI,KAAKD,aAAL,KAAuB,IAA3B,EAAiC;MAC/B,KAAKA,aAAL,GAAqB,MAAM,KAAKE,WAAL,CAAiB,KAAKN,MAAtB,EAA8B,KAAKC,MAAnC,CAA3B;MACAP,KAAK,CAAC,qBAAD,CAAL;MACA,MAAM,KAAKU,aAAL,CAAmBC,IAAnB,EAAN;MACAX,KAAK,CAAC,4BAAD,CAAL;IACD,CALD,MAKO;MACL,KAAKO,MAAL,CAAYM,IAAZ,CAAiB,6CAAjB;IACD;;IACD;EACD;;EAEMC,gBAAgB,GAAmB;IACxC,IAAI,KAAKJ,aAAL,KAAuB,IAA3B,EAAiC;MAC/B,MAAMK,gBAAA,CAAWC,gBAAX,CAA4B,mCAA5B,CAAN;IACD;;IAED,OAAO,KAAKN,aAAZ;EACD;;EAEqB,MAATO,SAAS,CAACX,MAAD,EAAgC;IACpD,MAAMY,SAAS,GAAG,MAAM,KAAKR,aAAL,CAAmBO,SAAnB,EAAxB;IAEA,OAAO,KAAKP,aAAL,CAAmBS,SAAnB,CAA6Bb,MAAM,CAACc,cAAP,CAAsBF,SAAtB,CAA7B,CAAP;EACD;;EAEwB,MAAXN,WAAW,CAACN,MAAD,EAAiBC,MAAjB,EAA0D;IACjF,MAAMc,OAAO,GAAG,MAAM,KAAKC,eAAL,EAAtB;;IACA,IAAIC,eAAA,CAAEC,KAAF,CAAQH,OAAR,CAAJ,EAAsB;MACpB,IAAAI,eAAA,EAAO,KAAKnB,MAAL,CAAYoB,OAAnB,EAA4B,kCAA5B;MACA1B,KAAK,CAAC,2EAAD,CAAL;MACA,OAAO,IAAI2B,qBAAJ,CAAkBrB,MAAlB,EAA0BC,MAA1B,CAAP;IACD;;IACD,OAAOc,OAAP;EACD;;EAE4B,MAAfC,eAAe,GAAwC;IAAA;;IACnE,MAAMM,OAAyB,GAAG,MAAM,IAAAC,wBAAA,EACtC,KAAKvB,MAAL,CAAYwB,KAD0B,EAEtC;MACExB,MAAM,EAAE,KAAKA,MADf;MAEEC,MAAM,EAAE,KAAKA;IAFf,CAFsC,EAMrCwB,MAAD,IAA4B;MAC1B,OAAOA,MAAM,CAACC,iBAAd;IACD,CARqC,kBAStC,KAAK1B,MATiC,0EAStC,aAAa2B,cATyB,0DAStC,sBAA6BC,YATS,CAAxC;;IAYA,IAAIN,OAAO,CAACO,MAAR,GAAiB,CAArB,EAAwB;MACtB,KAAK5B,MAAL,CAAYM,IAAZ,CACE,yHADF;IAGD;;IAED,OAAOU,eAAA,CAAEa,IAAF,CAAOR,OAAP,CAAP;EACD;;AArEgB"}
+{"version":3,"file":"local-storage.js","names":["debug","buildDebug","noSuchFile","resourceNotAvailable","LocalStorage","constructor","config","logger","child","sub","storagePlugin","init","loadStorage","warn","getStoragePlugin","errorUtils","getInternalError","getSecret","secretKey","setSecret","checkSecretKey","Storage","loadStorePlugin","_","isNil","assert","storage","LocalDatabase","plugins","asyncLoadPlugin","store","plugin","getPackageStorage","serverSettings","pluginPrefix","length","head"],"sources":["../src/local-storage.ts"],"sourcesContent":["import assert from 'assert';\nimport buildDebug from 'debug';\nimport _ from 'lodash';\n\nimport { errorUtils, pluginUtils } from '@verdaccio/core';\nimport { asyncLoadPlugin } from '@verdaccio/loaders';\nimport LocalDatabase from '@verdaccio/local-storage';\nimport { Config, Logger } from '@verdaccio/types';\n\nconst debug = buildDebug('verdaccio:storage:local');\n\nexport const noSuchFile = 'ENOENT';\nexport const resourceNotAvailable = 'EAGAIN';\nexport type PluginStorage = pluginUtils.Storage<Config>;\n\n/**\n * Implements Storage interface (same for storage.js, local-storage.js, up-storage.js).\n */\nclass LocalStorage {\n  public config: Config;\n  public storagePlugin: PluginStorage;\n  public logger: Logger;\n\n  public constructor(config: Config, logger: Logger) {\n    debug('local storage created');\n    this.logger = logger.child({ sub: 'fs' });\n    this.config = config;\n    // @ts-expect-error\n    this.storagePlugin = null;\n  }\n\n  public async init() {\n    if (this.storagePlugin === null) {\n      this.storagePlugin = await this.loadStorage(this.config, this.logger);\n      debug('storage plugin init');\n      await this.storagePlugin.init();\n      debug('storage plugin initialized');\n    } else {\n      this.logger.warn('storage plugin has been already initialized');\n    }\n    return;\n  }\n\n  public getStoragePlugin(): PluginStorage {\n    if (this.storagePlugin === null) {\n      throw errorUtils.getInternalError('storage plugin is not initialized');\n    }\n\n    return this.storagePlugin;\n  }\n\n  public async getSecret(config: Config): Promise<void> {\n    const secretKey = await this.storagePlugin.getSecret();\n\n    return this.storagePlugin.setSecret(config.checkSecretKey(secretKey));\n  }\n\n  private async loadStorage(config: Config, logger: Logger): Promise<PluginStorage> {\n    const Storage = await this.loadStorePlugin();\n    if (_.isNil(Storage)) {\n      assert(this.config.storage, 'CONFIG: storage path not defined');\n      debug('no custom storage found, loading default storage @verdaccio/local-storage');\n      return new LocalDatabase(config, logger);\n    }\n    return Storage as PluginStorage;\n  }\n\n  private async loadStorePlugin(): Promise<PluginStorage | undefined> {\n    const plugins: PluginStorage[] = await asyncLoadPlugin<pluginUtils.Storage<unknown>>(\n      this.config.store,\n      {\n        config: this.config,\n        logger: this.logger,\n      },\n      (plugin) => {\n        return typeof plugin.getPackageStorage !== 'undefined';\n      },\n      this.config?.serverSettings?.pluginPrefix\n    );\n\n    if (plugins.length > 1) {\n      this.logger.warn(\n        'more than one storage plugins has been detected, multiple storage are not supported, one will be selected automatically'\n      );\n    }\n\n    return _.head(plugins);\n  }\n}\n\nexport { LocalStorage };\n"],"mappings":";;;;;;;AAAA;;AACA;;AACA;;AAEA;;AACA;;AACA;;;;AAGA,MAAMA,KAAK,GAAG,IAAAC,cAAA,EAAW,yBAAX,CAAd;AAEO,MAAMC,UAAU,GAAG,QAAnB;;AACA,MAAMC,oBAAoB,GAAG,QAA7B;;;AAGP;AACA;AACA;AACA,MAAMC,YAAN,CAAmB;EAKVC,WAAW,CAACC,MAAD,EAAiBC,MAAjB,EAAiC;IACjDP,KAAK,CAAC,uBAAD,CAAL;IACA,KAAKO,MAAL,GAAcA,MAAM,CAACC,KAAP,CAAa;MAAEC,GAAG,EAAE;IAAP,CAAb,CAAd;IACA,KAAKH,MAAL,GAAcA,MAAd,CAHiD,CAIjD;;IACA,KAAKI,aAAL,GAAqB,IAArB;EACD;;EAEgB,MAAJC,IAAI,GAAG;IAClB,IAAI,KAAKD,aAAL,KAAuB,IAA3B,EAAiC;MAC/B,KAAKA,aAAL,GAAqB,MAAM,KAAKE,WAAL,CAAiB,KAAKN,MAAtB,EAA8B,KAAKC,MAAnC,CAA3B;MACAP,KAAK,CAAC,qBAAD,CAAL;MACA,MAAM,KAAKU,aAAL,CAAmBC,IAAnB,EAAN;MACAX,KAAK,CAAC,4BAAD,CAAL;IACD,CALD,MAKO;MACL,KAAKO,MAAL,CAAYM,IAAZ,CAAiB,6CAAjB;IACD;;IACD;EACD;;EAEMC,gBAAgB,GAAkB;IACvC,IAAI,KAAKJ,aAAL,KAAuB,IAA3B,EAAiC;MAC/B,MAAMK,gBAAA,CAAWC,gBAAX,CAA4B,mCAA5B,CAAN;IACD;;IAED,OAAO,KAAKN,aAAZ;EACD;;EAEqB,MAATO,SAAS,CAACX,MAAD,EAAgC;IACpD,MAAMY,SAAS,GAAG,MAAM,KAAKR,aAAL,CAAmBO,SAAnB,EAAxB;IAEA,OAAO,KAAKP,aAAL,CAAmBS,SAAnB,CAA6Bb,MAAM,CAACc,cAAP,CAAsBF,SAAtB,CAA7B,CAAP;EACD;;EAEwB,MAAXN,WAAW,CAACN,MAAD,EAAiBC,MAAjB,EAAyD;IAChF,MAAMc,OAAO,GAAG,MAAM,KAAKC,eAAL,EAAtB;;IACA,IAAIC,eAAA,CAAEC,KAAF,CAAQH,OAAR,CAAJ,EAAsB;MACpB,IAAAI,eAAA,EAAO,KAAKnB,MAAL,CAAYoB,OAAnB,EAA4B,kCAA5B;MACA1B,KAAK,CAAC,2EAAD,CAAL;MACA,OAAO,IAAI2B,qBAAJ,CAAkBrB,MAAlB,EAA0BC,MAA1B,CAAP;IACD;;IACD,OAAOc,OAAP;EACD;;EAE4B,MAAfC,eAAe,GAAuC;IAAA;;IAClE,MAAMM,OAAwB,GAAG,MAAM,IAAAC,wBAAA,EACrC,KAAKvB,MAAL,CAAYwB,KADyB,EAErC;MACExB,MAAM,EAAE,KAAKA,MADf;MAEEC,MAAM,EAAE,KAAKA;IAFf,CAFqC,EAMpCwB,MAAD,IAAY;MACV,OAAO,OAAOA,MAAM,CAACC,iBAAd,KAAoC,WAA3C;IACD,CARoC,kBASrC,KAAK1B,MATgC,0EASrC,aAAa2B,cATwB,0DASrC,sBAA6BC,YATQ,CAAvC;;IAYA,IAAIN,OAAO,CAACO,MAAR,GAAiB,CAArB,EAAwB;MACtB,KAAK5B,MAAL,CAAYM,IAAZ,CACE,yHADF;IAGD;;IAED,OAAOU,eAAA,CAAEa,IAAF,CAAOR,OAAP,CAAP;EACD;;AArEgB"}

package/build/storage.d.ts

@@ -1,18 +1,19 @@
 /// <reference types="node" />
 import { PassThrough, Readable } from 'stream';
-import { searchUtils } from '@verdaccio/core';
+import { pluginUtils, searchUtils } from '@verdaccio/core';
 import { ISyncUplinksOptions, ProxySearchParams } from '@verdaccio/proxy';
 import { AbbreviatedManifest, Config, Logger, Manifest, MergeTags, StringValue, Token, TokenFilter, Version } from '@verdaccio/types';
 import { UpdateManifestOptions } from '.';
 import { ProxyInstanceList } from './lib/uplink-util';
 import { LocalStorage } from './local-storage';
-import { IGetPackageOptionsNext, IPluginFilters } from './type';
+import { IGetPackageOptionsNext, StarManifestBody } from './type';
+export declare type Filters = pluginUtils.ManifestFilter<Config>[];
 export declare const noSuchFile = "ENOENT";
 export declare const resourceNotAvailable = "EAGAIN";
 export declare const PROTO_NAME = "__proto__";
 declare class Storage {
     localStorage: LocalStorage;
-    filters: IPluginFilters | null;
+    filters: Filters | null;
     readonly config: Config;
     readonly logger: Logger;
     readonly uplinks: ProxyInstanceList;
@@ -67,7 +68,7 @@ declare class Storage {
     /**
      * Initialize the storage asynchronously.
      * @param config Config
-     * @param filters IPluginFilters
+     * @param filters Filters
      * @returns Storage instance
      */
     init(config: Config): Promise<void>;
@@ -114,7 +115,7 @@ declare class Storage {
      */
     mergeTagsNext(name: string, tags: MergeTags): Promise<Manifest>;
     private getUpLinkForDistFile;
-    updateManifest(manifest: Manifest, options: UpdateManifestOptions): Promise<void>;
+    updateManifest(manifest: Manifest | StarManifestBody, options: UpdateManifestOptions): Promise<string | undefined>;
     private deprecate;
     private star;
     /**

package/build/storage.js

@@ -631,7 +631,7 @@ class Storage {
   /**
    * Initialize the storage asynchronously.
    * @param config Config
-   * @param filters IPluginFilters
+   * @param filters Filters
    * @returns Storage instance
    */
 
@@ -654,7 +654,7 @@ class Storage {
         config: this.config,
         logger: this.logger
       }, plugin => {
-        return plugin.filterMetadata;
+        return typeof plugin.filterMetadata !== 'undefined';
       }, (_this$config = this.config) === null || _this$config === void 0 ? void 0 : (_this$config$serverSe = _this$config.serverSettings) === null || _this$config$serverSe === void 0 ? void 0 : _this$config$serverSe.pluginPrefix);
       debug('filters available %o', this.filters);
     }
@@ -941,10 +941,11 @@ class Storage {
       // if user request to apply a star to the manifest
       await this.star(manifest, { ...options
       });
+      return _core.API_MESSAGE.PKG_CHANGED;
     } else if (_core.validatioUtils.validatePublishSingleVersion(manifest)) {
       // if continue, the version to be published does not exist
       // we create a new package
-      const [mergedManifest, version] = await this.publishANewVersion(manifest, { ...options
+      const [mergedManifest, version, message] = await this.publishANewVersion(manifest, { ...options
       }); // send notification of publication (notification step, non transactional)
 
       try {
@@ -962,6 +963,8 @@ class Storage {
           error: error.message
         }, 'notify batch service has failed: @{error}');
       }
+
+      return message;
     } else {
       debug('invalid body format');
 
@@ -979,15 +982,52 @@ class Storage {
     } = manifest;
     debug('deprecating %s', name);
     return this.changePackage(name, manifest, options.revision);
-  } // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  }
+
+  async star(manifest, options) {
+    const {
+      users
+    } = manifest;
+    const {
+      requestOptions,
+      name
+    } = options;
+    debug('star %s', name);
+    const {
+      username
+    } = requestOptions;
+
+    if (!username) {
+      throw _core.errorUtils.getBadRequest('update users only allowed to logged users');
+    }
+
+    const localPackage = await this.getPackageManifest({
+      name,
+      requestOptions,
+      uplinksLook: false
+    }); // backward compatible in case users are not in the storage.
+
+    const localStarUsers = localPackage.users || {}; // if trying to add a star
+
+    const userIsAddingStar = Object.keys(users).includes(username);
 
+    if (!(0, _starUtils.isExecutingStarCommand)(localPackage.users, username, userIsAddingStar)) {
+      return _core.API_MESSAGE.PKG_CHANGED;
+    }
+
+    const newUsers = userIsAddingStar ? { ...localStarUsers,
+      [username]: true
+    } : _lodash.default.reduce(localStarUsers, (users, value, key) => {
+      if (key !== username) {
+        users[key] = value;
+      }
 
-  async star(_body, _options) {
-    // // const storage: IPackageStorage = this.getPrivatePackageStorage(opname);
-    // if (typeof storage === 'undefined') {
-    //   throw errorUtils.getNotFound();
-    // }
-    throw _core.errorUtils.getInternalError('no implementation ready for npm star');
+      return users;
+    }, {});
+    await this.changePackage(name, { ...localPackage,
+      users: newUsers
+    }, options.revision);
+    return _core.API_MESSAGE.PKG_CHANGED;
   }
   /**
    * Get local package, on fails return null.
@@ -1055,6 +1095,7 @@ class Storage {
       name
     } = options;
     debug('publishing a new package for %o', name);
+    let successResponseMessage;
     const manifest = { ..._core.validatioUtils.normalizeMetadata(body, name)
     };
     const {
@@ -1099,6 +1140,9 @@ class Storage {
 
       if (!hasPackageInStorage) {
         await this.createNewLocalCachePackage(name, versionToPublish);
+        successResponseMessage = _core.API_MESSAGE.PKG_CREATED;
+      } else {
+        successResponseMessage = _core.API_MESSAGE.PKG_CHANGED;
       }
     } catch (err) {
       debug('error on change or update a package with %o', err.message);
@@ -1167,7 +1211,7 @@ class Storage {
       version: versionToPublish
     }, 'package @{name}@@{version} has been published');
 
-    return [mergedManifest, versionToPublish];
+    return [mergedManifest, versionToPublish, successResponseMessage];
   } // TODO: pending implementation
   // eslint-disable-next-line @typescript-eslint/no-unused-vars