add --set-dependencies flag

Author: sw-joelmut

libraries/botbuilder-vendors/src/actions/install.ts

@@ -16,7 +16,7 @@ import { logger } from '../utils';
  * @param param0.pkgDir Directory of the package.
  * @param param0.directory Directory to install vendor packages.
  */
-export async function install({ vendors, dependencies, pkgDir, directory }: any) {
+export async function install({ vendors, dependencies, pkgDir, directory, shouldSetDependencies }: any) {
     for (let i = 0; i < vendors.length; i++) {
         const vendor = vendors[i];
 
@@ -42,12 +42,12 @@ export async function install({ vendors, dependencies, pkgDir, directory }: any)
         await copyFile(source, destination);
     }
 
-    logger.package.dependencies.header({ dependencies: dependencies.length });
+    logger.package.dependencies.header({ dependencies: dependencies.length, shouldSetDependencies });
     for (let i = 0; i < dependencies.length; i++) {
         const { name, version } = dependencies[i];
         logger.package.dependencies.dependency({ isLast: i === dependencies.length - 1, name, version });
-        if (process.env.GITHUB_ACTIONS === 'true') {
-            // Only modify package.json if running in GitHub Actions, preventing changes to local files and pushing them back to the repository.
+        if (shouldSetDependencies) {
+            // Only modify package.json if the flag is set, preventing changes to local files and pushing them back to the repository.
             execSync(`npm pkg set dependencies["${name}"]="${version}"`, { cwd: pkgDir });
         }
     }

libraries/botbuilder-vendors/src/index.ts

@@ -17,7 +17,10 @@ const VENDORS_DIR = 'libraries/botbuilder-vendors/vendors';
 
 export const command = (argv: string[]) => async () => {
     try {
-        const flags = minimist(argv);
+        const flags = minimist(argv, {
+            alias: { setDependencies: 'set-dependencies' },
+            default: { setDependencies: false },
+        });
         const action = flags._[0];
         if (!actions.valid(action)) {
             throw failures.actionNotFound(action);
@@ -55,7 +58,7 @@ export const command = (argv: string[]) => async () => {
 
             logger.package.vendors.header({ vendors: newVendors.length });
 
-            await install({ vendors: newVendors, dependencies, pkgDir, directory });
+            await install({ vendors: newVendors, dependencies, pkgDir, directory, shouldSetDependencies: flags.setDependencies });
             await build({ pkgDir, vendors, directory });
 
             logger.package.footer();

libraries/botbuilder-vendors/src/utils/logger.ts

@@ -61,8 +61,9 @@ export const logger = {
             },
         },
         dependencies: {
-            header({ dependencies }: any) {
-                const tags = dependencies > 0 ? [`${colors.green}[added]`] : [`${colors.red}[not found]`];
+            header({ dependencies, shouldSetDependencies }: any) {
+                const tags = shouldSetDependencies ? dependencies > 0 ? [`${colors.green}[added]`] : [`${colors.red}[not found]`] : [`${colors.blue}[skipped]`];
+                tags.push(`${colors.yellow}[--set-dependencies=${shouldSetDependencies}]`)
                 console.log(
                     `├─ dependencies: ${colors.magenta}${dependencies} ${plural(dependencies, 'package')} ${tags.join('')}${colors.reset}`,
                 );

package.json

@@ -65,7 +65,7 @@
     "test:runtime:min": "wsrun -m -p \"botbuilder-dialogs-adaptive-runtime*\" -t test:min",
     "test:schemas": "mocha libraries/botbuilder-dialogs-declarative/tests/schemaMergeTest.js --exit --bail",
     "update-versions": "yarn workspace botbuilder-repo-utils update-versions",
-    "postupdate-versions": "yarn workspace botbuilder-vendors connect"
+    "postupdate-versions": "yarn workspace botbuilder-vendors connect --set-dependencies"
   },
   "resolutions": {
     "**/botbuilder-dialogs-adaptive-runtime-integration-restify/restify/send": "^0.19.0",