Add unique

actions · Jun 6, 2023 · 001e87e · 001e87e
1 parent 885f6b5
commit 001e87e
Show file tree

Hide file tree

Showing 4 changed files with 146 additions and 22 deletions.
diff --git a/dist/cache-save/index.js b/dist/cache-save/index.js
@@ -60441,6 +60441,7 @@ const cache = __importStar(__nccwpck_require__(7799));
 const glob = __importStar(__nccwpck_require__(8090));
 const path_1 = __importDefault(__nccwpck_require__(1017));
 const fs_1 = __importDefault(__nccwpck_require__(7147));
+const util_1 = __nccwpck_require__(2629);
 exports.supportedPackageManagers = {
     npm: {
         name: 'npm',
@@ -60526,11 +60527,7 @@ const getProjectDirectoriesFromCacheDependencyPath = (cacheDependencyPath) => __
         cacheDependenciesPaths = (yield globber.glob()) || [''];
         exports.memoizedCacheDependencies[cacheDependencyPath] = cacheDependenciesPaths;
     }
-    const existingDirectories = cacheDependenciesPaths
-        .map(path_1.default.dirname)
-        // uniq in order to do not traverse the same directories during the further processing
-        .filter((item, i, src) => item != null && src.indexOf(item) === i)
-        .filter(directory => fs_1.default.existsSync(directory) && fs_1.default.lstatSync(directory).isDirectory());
+    const existingDirectories = util_1.arrayUniqNotNull(cacheDependenciesPaths.map(path_1.default.dirname)).filter(directory => fs_1.default.existsSync(directory) && fs_1.default.lstatSync(directory).isDirectory());
     // if user explicitly pointed out some file, but it does not exist it is definitely
     // not he wanted, thus we should throw an error not trying to workaround with unexpected
     // result to the whole build
@@ -60554,7 +60551,7 @@ const getCacheDirectoriesFromCacheDependencyPath = (packageManagerInfo, cacheDep
         return cacheFolderPath;
     })));
     // uniq in order to do not cache the same directories twice
-    return cacheFoldersPaths.filter((item, i, src) => src.indexOf(item) === i);
+    return util_1.arrayUniq(cacheFoldersPaths);
 });
 /**
  * Finds the cache directories configured for the repo ignoring cache-dependency-path
@@ -60627,6 +60624,117 @@ var Outputs;
 })(Outputs = exports.Outputs || (exports.Outputs = {}));
 
 
+/***/ }),
+
+/***/ 2629:
+/***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.arrayUniqNotNull = exports.arrayUniq = exports.printEnvDetailsAndSetOutput = exports.parseNodeVersionFile = void 0;
+const core = __importStar(__nccwpck_require__(2186));
+const exec = __importStar(__nccwpck_require__(1514));
+function parseNodeVersionFile(contents) {
+    var _a, _b, _c;
+    let nodeVersion;
+    // Try parsing the file as an NPM `package.json` file.
+    try {
+        nodeVersion = (_a = JSON.parse(contents).volta) === null || _a === void 0 ? void 0 : _a.node;
+        if (!nodeVersion)
+            nodeVersion = (_b = JSON.parse(contents).engines) === null || _b === void 0 ? void 0 : _b.node;
+    }
+    catch (_d) {
+        core.info('Node version file is not JSON file');
+    }
+    if (!nodeVersion) {
+        const found = contents.match(/^(?:nodejs\s+)?v?(?<version>[^\s]+)$/m);
+        nodeVersion = (_c = found === null || found === void 0 ? void 0 : found.groups) === null || _c === void 0 ? void 0 : _c.version;
+    }
+    // In the case of an unknown format,
+    // return as is and evaluate the version separately.
+    if (!nodeVersion)
+        nodeVersion = contents.trim();
+    return nodeVersion;
+}
+exports.parseNodeVersionFile = parseNodeVersionFile;
+function printEnvDetailsAndSetOutput() {
+    return __awaiter(this, void 0, void 0, function* () {
+        core.startGroup('Environment details');
+        const promises = ['node', 'npm', 'yarn'].map((tool) => __awaiter(this, void 0, void 0, function* () {
+            const output = yield getToolVersion(tool, ['--version']);
+            return { tool, output };
+        }));
+        const tools = yield Promise.all(promises);
+        tools.forEach(({ tool, output }) => {
+            if (tool === 'node') {
+                core.setOutput(`${tool}-version`, output);
+            }
+            core.info(`${tool}: ${output}`);
+        });
+        core.endGroup();
+    });
+}
+exports.printEnvDetailsAndSetOutput = printEnvDetailsAndSetOutput;
+function getToolVersion(tool, options) {
+    return __awaiter(this, void 0, void 0, function* () {
+        try {
+            const { stdout, stderr, exitCode } = yield exec.getExecOutput(tool, options, {
+                ignoreReturnCode: true,
+                silent: true
+            });
+            if (exitCode > 0) {
+                core.info(`[warning]${stderr}`);
+                return '';
+            }
+            return stdout.trim();
+        }
+        catch (err) {
+            return '';
+        }
+    });
+}
+const arrayUniq = (src) => [...new Set(src)];
+exports.arrayUniq = arrayUniq;
+const arrayUniqNotNull = (src) => {
+    const set = new Set();
+    src.forEach(item => {
+        if (item !== null)
+            set.add(item);
+    });
+    return [...set];
+};
+exports.arrayUniqNotNull = arrayUniqNotNull;
+
+
 /***/ }),
 
 /***/ 2877:

diff --git a/dist/setup/index.js b/dist/setup/index.js
@@ -71223,6 +71223,7 @@ const cache = __importStar(__nccwpck_require__(7799));
 const glob = __importStar(__nccwpck_require__(8090));
 const path_1 = __importDefault(__nccwpck_require__(1017));
 const fs_1 = __importDefault(__nccwpck_require__(7147));
+const util_1 = __nccwpck_require__(2629);
 exports.supportedPackageManagers = {
     npm: {
         name: 'npm',
@@ -71308,11 +71309,7 @@ const getProjectDirectoriesFromCacheDependencyPath = (cacheDependencyPath) => __
         cacheDependenciesPaths = (yield globber.glob()) || [''];
         exports.memoizedCacheDependencies[cacheDependencyPath] = cacheDependenciesPaths;
     }
-    const existingDirectories = cacheDependenciesPaths
-        .map(path_1.default.dirname)
-        // uniq in order to do not traverse the same directories during the further processing
-        .filter((item, i, src) => item != null && src.indexOf(item) === i)
-        .filter(directory => fs_1.default.existsSync(directory) && fs_1.default.lstatSync(directory).isDirectory());
+    const existingDirectories = util_1.arrayUniqNotNull(cacheDependenciesPaths.map(path_1.default.dirname)).filter(directory => fs_1.default.existsSync(directory) && fs_1.default.lstatSync(directory).isDirectory());
     // if user explicitly pointed out some file, but it does not exist it is definitely
     // not he wanted, thus we should throw an error not trying to workaround with unexpected
     // result to the whole build
@@ -71336,7 +71333,7 @@ const getCacheDirectoriesFromCacheDependencyPath = (packageManagerInfo, cacheDep
         return cacheFolderPath;
     })));
     // uniq in order to do not cache the same directories twice
-    return cacheFoldersPaths.filter((item, i, src) => src.indexOf(item) === i);
+    return util_1.arrayUniq(cacheFoldersPaths);
 });
 /**
  * Finds the cache directories configured for the repo ignoring cache-dependency-path
@@ -72240,7 +72237,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.printEnvDetailsAndSetOutput = exports.parseNodeVersionFile = void 0;
+exports.arrayUniqNotNull = exports.arrayUniq = exports.printEnvDetailsAndSetOutput = exports.parseNodeVersionFile = void 0;
 const core = __importStar(__nccwpck_require__(2186));
 const exec = __importStar(__nccwpck_require__(1514));
 function parseNodeVersionFile(contents) {
@@ -72302,6 +72299,17 @@ function getToolVersion(tool, options) {
         }
     });
 }
+const arrayUniq = (src) => [...new Set(src)];
+exports.arrayUniq = arrayUniq;
+const arrayUniqNotNull = (src) => {
+    const set = new Set();
+    src.forEach(item => {
+        if (item !== null)
+            set.add(item);
+    });
+    return [...set];
+};
+exports.arrayUniqNotNull = arrayUniqNotNull;
 
 
 /***/ }),

diff --git a/src/cache-utils.ts b/src/cache-utils.ts
@@ -4,6 +4,7 @@ import * as cache from '@actions/cache';
 import * as glob from '@actions/glob';
 import path from 'path';
 import fs from 'fs';
+import {arrayUniq, arrayUniqNotNull} from './util';
 
 export interface PackageManagerInfo {
   name: string;
@@ -136,14 +137,12 @@ const getProjectDirectoriesFromCacheDependencyPath = async (
     memoizedCacheDependencies[cacheDependencyPath] = cacheDependenciesPaths;
   }
 
-  const existingDirectories: string[] = cacheDependenciesPaths
-    .map(path.dirname)
-    // uniq in order to do not traverse the same directories during the further processing
-    .filter((item, i, src) => item != null && src.indexOf(item) === i)
-    .filter(
-      directory =>
-        fs.existsSync(directory) && fs.lstatSync(directory).isDirectory()
-    ) as string[];
+  const existingDirectories: string[] = arrayUniqNotNull(
+    cacheDependenciesPaths.map(path.dirname)
+  ).filter(
+    directory =>
+      fs.existsSync(directory) && fs.lstatSync(directory).isDirectory()
+  ) as string[];
 
   // if user explicitly pointed out some file, but it does not exist it is definitely
   // not he wanted, thus we should throw an error not trying to workaround with unexpected
@@ -183,7 +182,7 @@ const getCacheDirectoriesFromCacheDependencyPath = async (
     )
   );
   // uniq in order to do not cache the same directories twice
-  return cacheFoldersPaths.filter((item, i, src) => src.indexOf(item) === i);
+  return arrayUniq(cacheFoldersPaths);
 };
 
 /**

diff --git a/src/util.ts b/src/util.ts
@@ -61,3 +61,12 @@ async function getToolVersion(tool: string, options: string[]) {
     return '';
   }
 }
+
+export const arrayUniq = <T>(src: Array<T>): Array<T> => [...new Set(src)];
+export const arrayUniqNotNull = <T>(src: Array<T | null>): Array<T> => {
+  const set = new Set<T>();
+  src.forEach(item => {
+    if (item !== null) set.add(item);
+  });
+  return [...set];
+};