@salesforce/core 2.35.0 → 2.35.3

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [2.35.3](https://github.com/forcedotcom/sfdx-core/compare/v2.35.2...v2.35.3) (2022-02-23)
+
+### Bug Fixes
+
+- always try to clean up aliases and configs ([cd0adc4](https://github.com/forcedotcom/sfdx-core/commit/cd0adc456978f27fc2d44cbcb61632925edcccd9))
+
+### [2.35.2](https://github.com/forcedotcom/sfdx-core/compare/v2.35.1...v2.35.2) (2022-02-16)
+
+### Bug Fixes
+
+- network error tolerance ([#517](https://github.com/forcedotcom/sfdx-core/issues/517)) ([676ebfe](https://github.com/forcedotcom/sfdx-core/commit/676ebfe58b13826b53f461b2fef321c21f583004))
+- remove redundant warnings about no listerners on warnings ([7a5bd23](https://github.com/forcedotcom/sfdx-core/commit/7a5bd2390713da929e886f41d9dcbc811104f99a))
+
+### [2.35.1](https://github.com/forcedotcom/sfdx-core/compare/v2.35.0...v2.35.1) (2022-02-10)
+
+### Bug Fixes
+
+- add error message on failed settings deploy ([e218b1e](https://github.com/forcedotcom/sfdx-core/commit/e218b1e40414ea9e4fec256b74224626d96eff4a))
+- prefer instanceUrl for loginUrl on user:create ([#518](https://github.com/forcedotcom/sfdx-core/issues/518)) ([c070a87](https://github.com/forcedotcom/sfdx-core/commit/c070a871861409dea722e85845eeb91f262ab66f))
+
 ## [2.35.0](https://github.com/forcedotcom/sfdx-core/compare/v2.34.2...v2.35.0) (2022-01-25)
 
 ### Bug Fixes

package/lib/lifecycleEvents.js

@@ -170,7 +170,7 @@ class Lifecycle {
      */
     async emit(eventName, data) {
         const listeners = this.getListeners(eventName);
-        if (listeners.length === 0) {
+        if (listeners.length === 0 && eventName !== Lifecycle.warningEventName) {
             this.debug(`A lifecycle event with the name ${eventName} does not exist. An event must be registered before it can be emitted.`);
         }
         else {

package/lib/org.js

@@ -644,35 +644,33 @@ class Org extends kit_1.AsyncCreatable {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     async removeUsers(throwWhenRemoveFails) {
         this.logger.debug(`Removing users associate with org: ${this.getOrgId()}`);
-        const config = await this.retrieveOrgUsersConfig();
+        const [config, authInfos, aliases] = await Promise.all([
+            this.retrieveOrgUsersConfig(),
+            this.readUserAuthFiles(),
+            aliases_1.Aliases.create(aliases_1.Aliases.getDefaultOptions()),
+        ]);
         this.logger.debug(`using path for org users: ${config.getPath()}`);
-        if (await config.exists()) {
-            const authInfos = await this.readUserAuthFiles();
-            const aliases = await aliases_1.Aliases.create(aliases_1.Aliases.getDefaultOptions());
-            this.logger.info(`Cleaning up usernames in org: ${this.getOrgId()}`);
-            for (const auth of authInfos) {
-                const username = auth.getFields().username;
-                const aliasKeys = (username && aliases.getKeysByValue(username)) || [];
-                aliases.unsetAll(aliasKeys);
-                let orgForUser;
-                if (username === this.getUsername()) {
-                    orgForUser = this;
-                }
-                else {
-                    const info = await authInfo_1.AuthInfo.create({ username });
-                    const connection = await connection_1.Connection.create({ authInfo: info });
-                    orgForUser = await Org.create({ connection });
-                }
-                const orgType = this.isDevHubOrg() ? config_1.Config.DEFAULT_DEV_HUB_USERNAME : config_1.Config.DEFAULT_USERNAME;
-                const configInfo = orgForUser.configAggregator.getInfo(orgType);
-                if ((configInfo.value === username || aliasKeys.includes(configInfo.value)) &&
-                    (configInfo.isGlobal() || configInfo.isLocal())) {
-                    await config_1.Config.update(configInfo.isGlobal(), orgType, undefined);
-                }
-                await orgForUser.removeAuth();
-            }
-            await aliases.write();
-        }
+        this.logger.info(`Cleaning up usernames in org: ${this.getOrgId()}`);
+        await Promise.all(authInfos
+            .map((auth) => auth.getFields().username)
+            .map(async (username) => {
+            const aliasKeys = (username && aliases.getKeysByValue(username)) || [];
+            aliases.unsetAll(aliasKeys);
+            const orgForUser = username === this.getUsername()
+                ? this
+                : await Org.create({
+                    connection: await connection_1.Connection.create({ authInfo: await authInfo_1.AuthInfo.create({ username }) }),
+                });
+            const orgType = this.isDevHubOrg() ? config_1.Config.DEFAULT_DEV_HUB_USERNAME : config_1.Config.DEFAULT_USERNAME;
+            const configInfo = orgForUser.configAggregator.getInfo(orgType);
+            const needsConfigUpdate = (configInfo.isGlobal() || configInfo.isLocal()) &&
+                (configInfo.value === username || aliasKeys.includes(configInfo.value));
+            return [
+                orgForUser.removeAuth(),
+                needsConfigUpdate ? config_1.Config.update(configInfo.isGlobal(), orgType, undefined) : undefined,
+            ].filter(Boolean);
+        }));
+        await aliases.write();
     }
     /**
      * Remove an associate sandbox config.

package/lib/scratchOrgErrorCodes.d.ts

@@ -1,4 +1,4 @@
 import { Optional } from '@salesforce/ts-types';
 import { Logger } from './logger';
 import { ScratchOrgInfo } from './scratchOrgInfoApi';
-export declare const checkScratchOrgInfoForErrors: (orgInfo: ScratchOrgInfo, hubUsername: Optional<string>, logger: Logger) => ScratchOrgInfo;
+export declare const checkScratchOrgInfoForErrors: (orgInfo: Optional<ScratchOrgInfo>, hubUsername: Optional<string>, logger: Logger) => ScratchOrgInfo;

package/lib/scratchOrgErrorCodes.js

@@ -29,6 +29,9 @@ const optionalErrorCodeMessage = (errorCode, args) => {
     }
 };
 const checkScratchOrgInfoForErrors = (orgInfo, hubUsername, logger) => {
+    if (!orgInfo) {
+        throw new sfdxError_1.SfdxError('No scratch org info found.', 'ScratchOrgInfoNotFound');
+    }
     if (orgInfo.Status === 'Active') {
         return orgInfo;
     }

package/lib/scratchOrgInfoApi.d.ts

@@ -1,7 +1,6 @@
 import { Optional } from '@salesforce/ts-types';
 import { Duration } from '@salesforce/kit';
 import { RecordResult } from 'jsforce';
-import { retry } from 'ts-retry-promise';
 import { Org } from './org';
 import { AuthInfo } from './authInfo';
 import SettingsGenerator, { ObjectSetting } from './scratchOrgSettingsGenerator';

package/lib/scratchOrgInfoApi.js

@@ -18,6 +18,7 @@ const authInfo_1 = require("./authInfo");
 const messages_1 = require("./messages");
 const sfdxError_1 = require("./sfdxError");
 const sfdcUrl_1 = require("./util/sfdcUrl");
+const pollingClient_1 = require("./status/pollingClient");
 const myDomainResolver_1 = require("./status/myDomainResolver");
 const scratchOrgErrorCodes_1 = require("./scratchOrgErrorCodes");
 messages_1.Messages.importMessagesDirectory(__dirname);
@@ -251,30 +252,53 @@ const pollForScratchOrgInfo = async (hubOrg, scratchOrgInfoId,
 timeout = kit_1.Duration.minutes(15)) => {
     const logger = await logger_1.Logger.child('scratchOrgInfoApi-pollForScratchOrgInfo');
     logger.debug(`PollingTimeout in minutes: ${timeout.minutes}`);
-    const response = await ts_retry_promise_1.retry(async () => {
-        const resultInProgress = await hubOrg
-            .getConnection()
-            .sobject('ScratchOrgInfo')
-            .retrieve(scratchOrgInfoId);
-        logger.debug(`polling client result: ${JSON.stringify(resultInProgress, null, 4)}`);
-        // Once it's "done" we can return it
-        if (resultInProgress.Status === 'Active' || resultInProgress.Status === 'Error') {
-            return resultInProgress;
+    const pollingOptions = {
+        async poll() {
+            try {
+                const resultInProgress = await hubOrg
+                    .getConnection()
+                    .sobject('ScratchOrgInfo')
+                    .retrieve(scratchOrgInfoId);
+                logger.debug(`polling client result: ${JSON.stringify(resultInProgress, null, 4)}`);
+                // Once it's "done" we can return it
+                if (resultInProgress.Status === 'Active' || resultInProgress.Status === 'Error') {
+                    return {
+                        completed: true,
+                        payload: resultInProgress,
+                    };
+                }
+                logger.debug(`Scratch org status is ${resultInProgress.Status}`);
+                return {
+                    completed: false,
+                };
+            }
+            catch (error) {
+                logger.debug(`An error occurred trying to retrieve scratchOrgInfo for ${scratchOrgInfoId}`);
+                logger.debug(`Error: ${error.message}`);
+                logger.debug('Re-trying deploy check again....');
+                return {
+                    completed: false,
+                };
+            }
+        },
+        timeout,
+        frequency: kit_1.Duration.seconds(1),
+        timeoutErrorName: 'ScratchOrgInfoTimeoutError',
+    };
+    const client = await pollingClient_1.PollingClient.create(pollingOptions);
+    try {
+        const resultInProgress = await client.subscribe();
+        return scratchOrgErrorCodes_1.checkScratchOrgInfoForErrors(resultInProgress, hubOrg.getUsername(), logger);
+    }
+    catch (error) {
+        const err = error;
+        if (err.message) {
+            throw sfdxError_1.SfdxError.wrap(err);
         }
-        // all other statuses, OR lack of status (e.g. network errors) will cause a retry
-        throw new sfdxError_1.SfdxError(`Scratch org status is ${resultInProgress.Status}`);
-    }, {
-        retries: 'INFINITELY',
-        timeout: timeout.milliseconds,
-        delay: kit_1.Duration.seconds(2).milliseconds,
-        backoff: 'LINEAR',
-        maxBackOff: kit_1.Duration.seconds(30).milliseconds,
-    }).catch(() => {
         throw new sfdxError_1.SfdxError(`The scratch org did not complete within ${timeout.minutes} minutes`, 'orgCreationTimeout', [
             'Try your force:org:create command again with a longer --wait value',
         ]);
-    });
-    return scratchOrgErrorCodes_1.checkScratchOrgInfoForErrors(response, hubOrg.getUsername(), logger);
+    }
 };
 exports.pollForScratchOrgInfo = pollForScratchOrgInfo;
 /**

package/lib/scratchOrgSettingsGenerator.js

@@ -7,9 +7,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.RequestStatus = void 0;
-// Node
 const path = require("path");
-// @salesforce
 const kit_1 = require("@salesforce/kit");
 const ts_types_1 = require("@salesforce/ts-types");
 const js2xmlparser = require("js2xmlparser");
@@ -77,7 +75,7 @@ class SettingsGenerator {
         const pollingOptions = {
             async poll() {
                 try {
-                    result = await connection.metadata.checkDeployStatus(id);
+                    result = await connection.metadata.checkDeployStatus(id, true);
                     logger.debug(`Deploy id: ${id} status: ${result.status}`);
                     if (breakPolling.includes(result.status)) {
                         return {
@@ -103,9 +101,13 @@ class SettingsGenerator {
             timeoutErrorName: 'DeployingSettingsTimeoutError',
         };
         const client = await pollingClient_1.PollingClient.create(pollingOptions);
-        const status = (await client.subscribe());
+        const status = await client.subscribe();
         if (status !== RequestStatus.Succeeded) {
-            const error = new sfdxError_1.SfdxError(`A scratch org was created with username ${username}, but the settings failed to deploy`, 'ProblemDeployingSettings');
+            const componentFailures = ts_types_1.ensureObject(result.details).componentFailures;
+            const failures = (Array.isArray(componentFailures) ? componentFailures : [componentFailures])
+                .map((failure) => failure.problem)
+                .join('\n');
+            const error = new sfdxError_1.SfdxError(`A scratch org was created with username ${username}, but the settings failed to deploy due to: \n${failures}`, 'ProblemDeployingSettings');
             error.setData(result);
             throw error;
         }

package/lib/status/pollingClient.d.ts

@@ -38,7 +38,7 @@ export declare class PollingClient extends AsyncOptionalCreatable<PollingClient.
      * Returns a promise to call the specified polling function using the interval and timeout specified
      * in the polling options.
      */
-    subscribe(): Promise<AnyJson | undefined>;
+    subscribe<T = AnyJson>(): Promise<T | undefined>;
 }
 export declare namespace PollingClient {
     /**

package/lib/status/pollingClient.js

@@ -12,6 +12,7 @@ const ts_types_1 = require("@salesforce/ts-types");
 const ts_retry_promise_1 = require("ts-retry-promise");
 const logger_1 = require("../logger");
 const sfdxError_1 = require("../sfdxError");
+const lifecycleEvents_1 = require("../lifecycleEvents");
 /**
  * This is a polling client that can be used to poll the status of long running tasks. It can be used as a replacement
  * for Streaming when streaming topics are not available or when streaming handshakes are failing. Why wouldn't you
@@ -51,6 +52,7 @@ class PollingClient extends kit_1.AsyncOptionalCreatable {
      * Returns a promise to call the specified polling function using the interval and timeout specified
      * in the polling options.
      */
+    // TODO v3.0 remove undefined as this method ensures that the payload is always returned
     async subscribe() {
         var _a;
         let errorInPollingFunction; // keep this around for returning in the catch block
@@ -60,11 +62,18 @@ class PollingClient extends kit_1.AsyncOptionalCreatable {
                 result = await this.options.poll();
             }
             catch (error) {
-                errorInPollingFunction = error;
+                const err = (errorInPollingFunction = error);
+                if (['ETIMEDOUT', 'ENOTFOUND', 'ECONNRESET', 'socket hang up'].some((retryableNetworkError) => err.message.includes(retryableNetworkError))) {
+                    this.logger.debug('Network error on the request', err);
+                    await lifecycleEvents_1.Lifecycle.getInstance().emitWarning('Network error occurred. Continuing to poll.');
+                    throw sfdxError_1.SfdxError.wrap(err);
+                }
                 // there was an actual error thrown, so we don't want to keep retrying
-                throw new ts_retry_promise_1.NotRetryableError(error.name);
+                throw new ts_retry_promise_1.NotRetryableError(err.name);
             }
             if (result.completed) {
+                // TODO v3.0: payload should be of type T always so that
+                // consumers get the same type in return.
                 return result.payload;
             }
             throw new Error('Operation did not complete.  Retrying...'); // triggers a retry

package/lib/user.js

@@ -280,6 +280,7 @@ class User extends kit_1.AsyncCreatable {
      * ```
      */
     async createUser(fields) {
+        var _a;
         // Create a user and get a refresh token
         const refreshTokenSecret = await this.createUserInternal(fields);
         // Create the initial auth info
@@ -287,7 +288,8 @@ class User extends kit_1.AsyncCreatable {
         const adminUserAuthFields = authInfo.getFields(true);
         // Setup oauth options for the new user
         const oauthOptions = {
-            loginUrl: adminUserAuthFields.loginUrl,
+            // Communities users require the instance for auth
+            loginUrl: (_a = adminUserAuthFields.instanceUrl) !== null && _a !== void 0 ? _a : adminUserAuthFields.loginUrl,
             refreshToken: refreshTokenSecret.buffer.value((buffer) => buffer.toString('utf8')),
             clientId: adminUserAuthFields.clientId,
             clientSecret: adminUserAuthFields.clientSecret,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/core",
-  "version": "2.35.0",
+  "version": "2.35.3",
   "description": "Core libraries to interact with SFDX projects, orgs, and APIs.",
   "main": "lib/exported",
   "types": "lib/exported.d.ts",
@@ -82,7 +82,7 @@
     "nyc": "^15.1.0",
     "prettier": "^2.0.5",
     "pretty-quick": "^3.1.0",
-    "shelljs": "0.8.1",
+    "shelljs": "0.8.5",
     "sinon": "10.0.0",
     "ts-node": "^10.0.0",
     "typescript": "^4.1.3"