Print node, npm and yarn versions after installation (#368)

__tests__/installer.test.ts

@@ -249,6 +249,21 @@ describe('setup-node', () => {
 
     let expPath = path.join(toolPath, 'bin');
 
+    expect(getExecOutputSpy).toHaveBeenCalledWith(
+      'node',
+      ['--version'],
+      expect.anything()
+    );
+    expect(getExecOutputSpy).toHaveBeenCalledWith(
+      'npm',
+      ['--version'],
+      expect.anything()
+    );
+    expect(getExecOutputSpy).toHaveBeenCalledWith(
+      'yarn',
+      ['--version'],
+      expect.anything()
+    );
     expect(dlSpy).toHaveBeenCalled();
     expect(exSpy).toHaveBeenCalled();
     expect(logSpy).toHaveBeenCalledWith(

dist/cache-save/index.js

@@ -60979,74 +60979,74 @@ exports.fromPromise = function (fn) {
 /***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
 
 "use strict";
-
+
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
+    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        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 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); }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
+    });
-};
+};
-var __importStar = (this && this.__importStar) || function (mod) {
+var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
+    if (mod && mod.__esModule) return mod;
-    var result = {};
+    var result = {};
-    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
+    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
-    result["default"] = mod;
+    result["default"] = mod;
-    return result;
+    return result;
-};
+};
-var __importDefault = (this && this.__importDefault) || function (mod) {
+var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
+    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
+};
-Object.defineProperty(exports, "__esModule", ({ value: true }));
+Object.defineProperty(exports, "__esModule", ({ value: true }));
-const core = __importStar(__nccwpck_require__(2186));
+const core = __importStar(__nccwpck_require__(2186));
-const cache = __importStar(__nccwpck_require__(7799));
+const cache = __importStar(__nccwpck_require__(7799));
-const fs_1 = __importDefault(__nccwpck_require__(7147));
+const fs_1 = __importDefault(__nccwpck_require__(7147));
-const constants_1 = __nccwpck_require__(9042);
+const constants_1 = __nccwpck_require__(9042);
-const cache_utils_1 = __nccwpck_require__(1678);
+const cache_utils_1 = __nccwpck_require__(1678);
-// Catch and log any unhandled exceptions.  These exceptions can leak out of the uploadChunk method in
+// Catch and log any unhandled exceptions.  These exceptions can leak out of the uploadChunk method in
-// @actions/toolkit when a failed upload closes the file descriptor causing any in-process reads to
+// @actions/toolkit when a failed upload closes the file descriptor causing any in-process reads to
-// throw an uncaught exception.  Instead of failing this action, just warn.
+// throw an uncaught exception.  Instead of failing this action, just warn.
-process.on('uncaughtException', e => {
+process.on('uncaughtException', e => {
-    const warningPrefix = '[warning]';
+    const warningPrefix = '[warning]';
-    core.info(`${warningPrefix}${e.message}`);
+    core.info(`${warningPrefix}${e.message}`);
-});
+});
-function run() {
+function run() {
-    return __awaiter(this, void 0, void 0, function* () {
+    return __awaiter(this, void 0, void 0, function* () {
-        try {
+        try {
-            const cacheLock = core.getInput('cache');
+            const cacheLock = core.getInput('cache');
-            yield cachePackages(cacheLock);
+            yield cachePackages(cacheLock);
-        }
+        }
-        catch (error) {
+        catch (error) {
-            core.setFailed(error.message);
+            core.setFailed(error.message);
-        }
+        }
-    });
+    });
-}
+}
-exports.run = run;
+exports.run = run;
-const cachePackages = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
+const cachePackages = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
-    const state = core.getState(constants_1.State.CacheMatchedKey);
+    const state = core.getState(constants_1.State.CacheMatchedKey);
-    const primaryKey = core.getState(constants_1.State.CachePrimaryKey);
+    const primaryKey = core.getState(constants_1.State.CachePrimaryKey);
-    const packageManagerInfo = yield cache_utils_1.getPackageManagerInfo(packageManager);
+    const packageManagerInfo = yield cache_utils_1.getPackageManagerInfo(packageManager);
-    if (!packageManagerInfo) {
+    if (!packageManagerInfo) {
-        core.debug(`Caching for '${packageManager}' is not supported`);
+        core.debug(`Caching for '${packageManager}' is not supported`);
-        return;
+        return;
-    }
+    }
-    const cachePath = yield cache_utils_1.getCacheDirectoryPath(packageManagerInfo, packageManager);
+    const cachePath = yield cache_utils_1.getCacheDirectoryPath(packageManagerInfo, packageManager);
-    if (!fs_1.default.existsSync(cachePath)) {
+    if (!fs_1.default.existsSync(cachePath)) {
-        throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePath}`);
+        throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePath}`);
-    }
+    }
-    if (primaryKey === state) {
+    if (primaryKey === state) {
-        core.info(`Cache hit occurred on the primary key ${primaryKey}, not saving cache.`);
+        core.info(`Cache hit occurred on the primary key ${primaryKey}, not saving cache.`);
-        return;
+        return;
-    }
+    }
-    const cacheId = yield cache.saveCache([cachePath], primaryKey);
+    const cacheId = yield cache.saveCache([cachePath], primaryKey);
-    if (cacheId == -1) {
+    if (cacheId == -1) {
-        return;
+        return;
-    }
+    }
-    core.info(`Cache saved with the key: ${primaryKey}`);
+    core.info(`Cache saved with the key: ${primaryKey}`);
-});
+});
-run();
+run();
 
 
 /***/ }),
@@ -61055,109 +61055,109 @@ run();
 /***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
 
 "use strict";
-
+
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
+    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        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 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); }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
+    });
-};
+};
-var __importStar = (this && this.__importStar) || function (mod) {
+var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
+    if (mod && mod.__esModule) return mod;
-    var result = {};
+    var result = {};
-    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
+    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
-    result["default"] = mod;
+    result["default"] = mod;
-    return result;
+    return result;
-};
+};
-Object.defineProperty(exports, "__esModule", ({ value: true }));
+Object.defineProperty(exports, "__esModule", ({ value: true }));
-const core = __importStar(__nccwpck_require__(2186));
+const core = __importStar(__nccwpck_require__(2186));
-const exec = __importStar(__nccwpck_require__(1514));
+const exec = __importStar(__nccwpck_require__(1514));
-const cache = __importStar(__nccwpck_require__(7799));
+const cache = __importStar(__nccwpck_require__(7799));
-exports.supportedPackageManagers = {
+exports.supportedPackageManagers = {
-    npm: {
+    npm: {
-        lockFilePatterns: ['package-lock.json', 'npm-shrinkwrap.json', 'yarn.lock'],
+        lockFilePatterns: ['package-lock.json', 'npm-shrinkwrap.json', 'yarn.lock'],
-        getCacheFolderCommand: 'npm config get cache'
+        getCacheFolderCommand: 'npm config get cache'
-    },
+    },
-    pnpm: {
+    pnpm: {
-        lockFilePatterns: ['pnpm-lock.yaml'],
+        lockFilePatterns: ['pnpm-lock.yaml'],
-        getCacheFolderCommand: 'pnpm store path --silent'
+        getCacheFolderCommand: 'pnpm store path --silent'
-    },
+    },
-    yarn1: {
+    yarn1: {
-        lockFilePatterns: ['yarn.lock'],
+        lockFilePatterns: ['yarn.lock'],
-        getCacheFolderCommand: 'yarn cache dir'
+        getCacheFolderCommand: 'yarn cache dir'
-    },
+    },
-    yarn2: {
+    yarn2: {
-        lockFilePatterns: ['yarn.lock'],
+        lockFilePatterns: ['yarn.lock'],
-        getCacheFolderCommand: 'yarn config get cacheFolder'
+        getCacheFolderCommand: 'yarn config get cacheFolder'
-    }
+    }
-};
+};
-exports.getCommandOutput = (toolCommand) => __awaiter(void 0, void 0, void 0, function* () {
+exports.getCommandOutput = (toolCommand) => __awaiter(void 0, void 0, void 0, function* () {
-    let { stdout, stderr, exitCode } = yield exec.getExecOutput(toolCommand, undefined, { ignoreReturnCode: true });
+    let { stdout, stderr, exitCode } = yield exec.getExecOutput(toolCommand, undefined, { ignoreReturnCode: true });
-    if (exitCode) {
+    if (exitCode) {
-        stderr = !stderr.trim()
+        stderr = !stderr.trim()
-            ? `The '${toolCommand}' command failed with exit code: ${exitCode}`
+            ? `The '${toolCommand}' command failed with exit code: ${exitCode}`
-            : stderr;
+            : stderr;
-        throw new Error(stderr);
+        throw new Error(stderr);
-    }
+    }
-    return stdout.trim();
+    return stdout.trim();
-});
+});
-const getPackageManagerVersion = (packageManager, command) => __awaiter(void 0, void 0, void 0, function* () {
+const getPackageManagerVersion = (packageManager, command) => __awaiter(void 0, void 0, void 0, function* () {
-    const stdOut = yield exports.getCommandOutput(`${packageManager} ${command}`);
+    const stdOut = yield exports.getCommandOutput(`${packageManager} ${command}`);
-    if (!stdOut) {
+    if (!stdOut) {
-        throw new Error(`Could not retrieve version of ${packageManager}`);
+        throw new Error(`Could not retrieve version of ${packageManager}`);
-    }
+    }
-    return stdOut;
+    return stdOut;
-});
+});
-exports.getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
+exports.getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
-    if (packageManager === 'npm') {
+    if (packageManager === 'npm') {
-        return exports.supportedPackageManagers.npm;
+        return exports.supportedPackageManagers.npm;
-    }
+    }
-    else if (packageManager === 'pnpm') {
+    else if (packageManager === 'pnpm') {
-        return exports.supportedPackageManagers.pnpm;
+        return exports.supportedPackageManagers.pnpm;
-    }
+    }
-    else if (packageManager === 'yarn') {
+    else if (packageManager === 'yarn') {
-        const yarnVersion = yield getPackageManagerVersion('yarn', '--version');
+        const yarnVersion = yield getPackageManagerVersion('yarn', '--version');
-        core.debug(`Consumed yarn version is ${yarnVersion}`);
+        core.debug(`Consumed yarn version is ${yarnVersion}`);
-        if (yarnVersion.startsWith('1.')) {
+        if (yarnVersion.startsWith('1.')) {
-            return exports.supportedPackageManagers.yarn1;
+            return exports.supportedPackageManagers.yarn1;
-        }
+        }
-        else {
+        else {
-            return exports.supportedPackageManagers.yarn2;
+            return exports.supportedPackageManagers.yarn2;
-        }
+        }
-    }
+    }
-    else {
+    else {
-        return null;
+        return null;
-    }
+    }
-});
+});
-exports.getCacheDirectoryPath = (packageManagerInfo, packageManager) => __awaiter(void 0, void 0, void 0, function* () {
+exports.getCacheDirectoryPath = (packageManagerInfo, packageManager) => __awaiter(void 0, void 0, void 0, function* () {
-    const stdOut = yield exports.getCommandOutput(packageManagerInfo.getCacheFolderCommand);
+    const stdOut = yield exports.getCommandOutput(packageManagerInfo.getCacheFolderCommand);
-    if (!stdOut) {
+    if (!stdOut) {
-        throw new Error(`Could not get cache folder path for ${packageManager}`);
+        throw new Error(`Could not get cache folder path for ${packageManager}`);
-    }
+    }
-    core.debug(`${packageManager} path is ${stdOut}`);
+    core.debug(`${packageManager} path is ${stdOut}`);
-    return stdOut.trim();
+    return stdOut.trim();
-});
+});
-function isGhes() {
+function isGhes() {
-    const ghUrl = new URL(process.env['GITHUB_SERVER_URL'] || 'https://github.com');
+    const ghUrl = new URL(process.env['GITHUB_SERVER_URL'] || 'https://github.com');
-    return ghUrl.hostname.toUpperCase() !== 'GITHUB.COM';
+    return ghUrl.hostname.toUpperCase() !== 'GITHUB.COM';
-}
+}
-exports.isGhes = isGhes;
+exports.isGhes = isGhes;
-function isCacheFeatureAvailable() {
+function isCacheFeatureAvailable() {
-    if (!cache.isFeatureAvailable()) {
+    if (!cache.isFeatureAvailable()) {
-        if (isGhes()) {
+        if (isGhes()) {
-            throw new Error('Cache action is only supported on GHES version >= 3.5. If you are on version >=3.5 Please check with GHES admin if Actions cache service is enabled or not.');
+            throw new Error('Cache action is only supported on GHES version >= 3.5. If you are on version >=3.5 Please check with GHES admin if Actions cache service is enabled or not.');
-        }
+        }
-        else {
+        else {
-            core.warning('The runner was not able to contact the cache service. Caching will be skipped');
+            core.warning('The runner was not able to contact the cache service. Caching will be skipped');
-        }
+        }
-        return false;
+        return false;
-    }
+    }
-    return true;
+    return true;
-}
+}
-exports.isCacheFeatureAvailable = isCacheFeatureAvailable;
+exports.isCacheFeatureAvailable = isCacheFeatureAvailable;
 
 
 /***/ }),
@@ -61166,23 +61166,23 @@ exports.isCacheFeatureAvailable = isCacheFeatureAvailable;
 /***/ ((__unused_webpack_module, exports) => {
 
 "use strict";
-
+
-Object.defineProperty(exports, "__esModule", ({ value: true }));
+Object.defineProperty(exports, "__esModule", ({ value: true }));
-var LockType;
+var LockType;
-(function (LockType) {
+(function (LockType) {
-    LockType["Npm"] = "npm";
+    LockType["Npm"] = "npm";
-    LockType["Pnpm"] = "pnpm";
+    LockType["Pnpm"] = "pnpm";
-    LockType["Yarn"] = "yarn";
+    LockType["Yarn"] = "yarn";
-})(LockType = exports.LockType || (exports.LockType = {}));
+})(LockType = exports.LockType || (exports.LockType = {}));
-var State;
+var State;
-(function (State) {
+(function (State) {
-    State["CachePrimaryKey"] = "CACHE_KEY";
+    State["CachePrimaryKey"] = "CACHE_KEY";
-    State["CacheMatchedKey"] = "CACHE_RESULT";
+    State["CacheMatchedKey"] = "CACHE_RESULT";
-})(State = exports.State || (exports.State = {}));
+})(State = exports.State || (exports.State = {}));
-var Outputs;
+var Outputs;
-(function (Outputs) {
+(function (Outputs) {
-    Outputs["CacheHit"] = "cache-hit";
+    Outputs["CacheHit"] = "cache-hit";
-})(Outputs = exports.Outputs || (exports.Outputs = {}));
+})(Outputs = exports.Outputs || (exports.Outputs = {}));
 
 
 /***/ }),

docs/advanced-usage.md

@@ -283,3 +283,6 @@ steps:
   run: yarn install --immutable
 ```
 NOTE: As per https://github.com/actions/setup-node/issues/49 you cannot use `secrets.GITHUB_TOKEN` to access private GitHub Packages within the same organisation but in a different repository.
+
+### always-auth input
+The always-auth input sets `always-auth=true` in .npmrc file. With this option set [npm](https://docs.npmjs.com/cli/v6/using-npm/config#always-auth)/yarn sends the authentication credentials when making a request to the registries.

src/main.ts

@@ -40,17 +40,7 @@ export async function run() {
       await installer.getNode(version, stable, checkLatest, auth, arch);
     }
 
-    // Output version of node is being used
+    await printEnvDetailsAndSetOutput();
-    try {
-      const {stdout: installedVersion} = await exec.getExecOutput(
-        'node',
-        ['--version'],
-        {ignoreReturnCode: true, silent: true}
-      );
-      core.setOutput('node-version', installedVersion.trim());
-    } catch (err) {
-      core.setOutput('node-version', '');
-    }
 
     const registryUrl: string = core.getInput('registry-url');
     const alwaysAuth: string = core.getInput('always-auth');
@@ -111,3 +101,39 @@ function resolveVersionInput(): string {
 
   return version;
 }
+
+export async function printEnvDetailsAndSetOutput() {
+  core.startGroup('Environment details');
+
+  const promises = ['node', 'npm', 'yarn'].map(async tool => {
+    const output = await getToolVersion(tool, ['--version']);
+
+    if (tool === 'node') {
+      core.setOutput(`${tool}-version`, output);
+    }
+
+    core.info(`${tool}: ${output}`);
+  });
+
+  await Promise.all(promises);
+
+  core.endGroup();
+}
+
+async function getToolVersion(tool: string, options: string[]) {
+  try {
+    const {stdout, stderr, exitCode} = await exec.getExecOutput(tool, options, {
+      ignoreReturnCode: true,
+      silent: true
+    });
+
+    if (exitCode > 0) {
+      core.warning(`[warning]${stderr}`);
+      return '';
+    }
+
+    return stdout;
+  } catch (err) {
+    return '';
+  }
+}