Merge pull request #458 from kkeirstead/copilot/update-install-and-require-packages

Pin @actions/core@2 and @actions/github@8; support versioned packages in installAndRequirePackages

Author: kkeirstead

.github/actions/action-utils.js

@@ -2,21 +2,22 @@ const util = require("util");
 const fs = require("fs");
 const jsExec = util.promisify(require("child_process").exec);
 
-module.exports.installAndRequirePackages = async function(...newPackages)
-{
+module.exports.installAndRequirePackages = async function (...packages) {
+    // Back-compat: allow strings ("@actions/core") as well as objects ({ name, version })
+    const normalized = packages.map(p => typeof p === "string" ? { name: p } : p);
+
+    const installSpecs = normalized.map(p =>
+        p.version ? `${p.name}@${p.version}` : p.name
+    );
+
     console.log("Installing npm dependency");
-    const { stdout, stderr } = await jsExec(`npm install ${newPackages.join(' ')}`);
+    const { stdout, stderr } = await jsExec(`npm install ${installSpecs.join(" ")}`);
     console.log("npm-install stderr:\n\n" + stderr);
     console.log("npm-install stdout:\n\n" + stdout);
     console.log("Finished installing npm dependencies");
 
-    let requiredPackages = [];
-    for (const packageName of newPackages) {
-        requiredPackages.push(require(packageName));
-    }
-
-    return requiredPackages;
-}
+    return normalized.map(p => require(p.name));
+};
 
 function splitVersionTag(tag) {
     const regex = /v(?<major>\d+)\.(?<minor>\d+)\.(?<patch>\d+)(-(?<versionLabel>[a-zA-Z]+)\.(?<iteration>\d+))?/;

.github/actions/generate-release-notes/index.js

@@ -5,7 +5,7 @@ const UpdateReleaseNotesLabel = "update-release-notes";
 const BackportLabel = "backport";
 
 async function run() {
-    const [core, github] = await actionUtils.installAndRequirePackages("@actions/core", "@actions/github");
+    const [core, github] = await actionUtils.installAndRequirePackages({ name: "@actions/core", version: "2" }, { name: "@actions/github", version: "8" });
 
     const octokit = github.getOctokit(core.getInput("auth_token", { required: true }));
 

.github/actions/learning-path-staleness-check/index.js

@@ -202,7 +202,7 @@ function ValidateLinks(learningPathContents, repoURLToSearch, modifiedPRFiles, l
 
 const main = async () => {
 
-  const [core] = await actionUtils.installAndRequirePackages("@actions/core");
+  const [core] = await actionUtils.installAndRequirePackages({ name: "@actions/core", version: "2" });
 
   try {
     const learningPathDirectory = core.getInput('learningPathsDirectory', { required: true });

.github/actions/update-releases-json/index.js

@@ -2,7 +2,7 @@ const actionUtils = require('../action-utils.js');
 const path = require('path');
 
 async function run() {
-    const [core, github] = await actionUtils.installAndRequirePackages("@actions/core", "@actions/github");
+    const [core, github] = await actionUtils.installAndRequirePackages({ name: "@actions/core", version: "2" }, { name: "@actions/github", version: "8" });
 
     const releasesDataFile = core.getInput("releases_json_file", { required: true });
     let octokit = undefined;

.github/actions/update-releases-md/index.js

@@ -1,7 +1,7 @@
 const actionUtils = require('../action-utils.js');
 
 async function run() {
-    const [core, github] = await actionUtils.installAndRequirePackages("@actions/core", "@actions/github");
+    const [core, github] = await actionUtils.installAndRequirePackages({ name: "@actions/core", version: "2" }, { name: "@actions/github", version: "8" });
 
     const releasesDataFile = core.getInput("releases_json_file", { required: true });
     const outputFile = core.getInput("releases_md_file", { required: true });