@simplysf/simply-cicd-core 0.10.0 → 0.10.1

package/lib/build/devHubs.d.ts

@@ -8,5 +8,13 @@ export type ResolvedDevHub = {
     /** Only present when the Dev Hub was JWT-authenticated. */
     privateKeyFile?: string;
 };
-/** Resolves multiple Dev Hub aliases in parallel. Each must already be authenticated. */
+/**
+ * Resolves multiple Dev Hub aliases in parallel. Each must already be authenticated.
+ *
+ * A blank entry is skipped with a warning rather than failing the run: it's what an unset CI
+ * variable produces (`--dev-hub "$BACKUP_DEVHUB"`), and the remaining Dev Hubs can still be used.
+ *
+ * @throws {SfError} `NoDevHubGivenError` if every entry is blank; `NamedOrgNotFoundError` if no
+ * auth exists for an alias or username.
+ */
 export declare function resolveDevHubs(aliases: string[]): Promise<ResolvedDevHub[]>;

package/lib/build/devHubs.js

@@ -13,6 +13,8 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+import { SfError } from '@salesforce/core';
+import { logger } from '../logger.js';
 import { resolveAuthInfo } from '../resolveOrg.js';
 /**
  * Resolves a Dev Hub alias or username to its already-authenticated config. `simply-cicd` never
@@ -35,8 +37,24 @@ async function resolveDevHub(alias) {
         privateKeyFile: fields.privateKey,
     };
 }
-/** Resolves multiple Dev Hub aliases in parallel. Each must already be authenticated. */
+/**
+ * Resolves multiple Dev Hub aliases in parallel. Each must already be authenticated.
+ *
+ * A blank entry is skipped with a warning rather than failing the run: it's what an unset CI
+ * variable produces (`--dev-hub "$BACKUP_DEVHUB"`), and the remaining Dev Hubs can still be used.
+ *
+ * @throws {SfError} `NoDevHubGivenError` if every entry is blank; `NamedOrgNotFoundError` if no
+ * auth exists for an alias or username.
+ */
 export async function resolveDevHubs(aliases) {
-    return Promise.all(aliases.map(resolveDevHub));
+    const given = aliases.map((alias) => alias.trim()).filter((alias) => alias.length > 0);
+    const blanks = aliases.length - given.length;
+    if (blanks > 0) {
+        logger.warn(`Ignoring ${blanks} empty --dev-hub value${blanks === 1 ? '' : 's'} (an unset CI variable passed as --dev-hub?).`);
+        if (given.length === 0) {
+            throw new SfError('Every --dev-hub value is empty. Pass the alias or username of an authenticated Dev Hub, and check that the CI variable it comes from is set in this job.', 'NoDevHubGivenError');
+        }
+    }
+    return Promise.all(given.map(resolveDevHub));
 }
 //# sourceMappingURL=devHubs.js.map

package/lib/build/devHubs.js.map

@@ -1 +1 @@
-{"version":3,"file":"devHubs.js","sourceRoot":"","sources":["../../src/build/devHubs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAanD;;;;;;;;;GASG;AACH,KAAK,UAAU,aAAa,CAAC,KAAa;IACxC,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,SAAS,EAAE,CAAC;IAEpC,OAAO;QACL,KAAK;QACL,QAAQ,EAAE,QAAQ,CAAC,WAAW,EAAE;QAChC,WAAW,EAAE,MAAM,CAAC,WAAW,IAAI,EAAE;QACrC,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,cAAc,EAAE,MAAM,CAAC,UAAU;KAClC,CAAC;AACJ,CAAC;AAED,yFAAyF;AACzF,MAAM,CAAC,KAAK,UAAU,cAAc,CAAC,OAAiB;IACpD,OAAO,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC;AACjD,CAAC"}
+{"version":3,"file":"devHubs.js","sourceRoot":"","sources":["../../src/build/devHubs.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AAC3C,OAAO,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AACtC,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAanD;;;;;;;;;GASG;AACH,KAAK,UAAU,aAAa,CAAC,KAAa;IACxC,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,QAAQ,CAAC,SAAS,EAAE,CAAC;IAEpC,OAAO;QACL,KAAK;QACL,QAAQ,EAAE,QAAQ,CAAC,WAAW,EAAE;QAChC,WAAW,EAAE,MAAM,CAAC,WAAW,IAAI,EAAE;QACrC,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,cAAc,EAAE,MAAM,CAAC,UAAU;KAClC,CAAC;AACJ,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAAC,OAAiB;IACpD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IACvF,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC;IAC7C,IAAI,MAAM,GAAG,CAAC,EAAE,CAAC;QACf,MAAM,CAAC,IAAI,CACT,YAAY,MAAM,yBAAyB,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,+CAA+C,CAClH,CAAC;QACF,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACvB,MAAM,IAAI,OAAO,CACf,0JAA0J,EAC1J,oBAAoB,CACrB,CAAC;QACJ,CAAC;IACH,CAAC;IACD,OAAO,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC;AAC/C,CAAC"}

package/lib/resolveOrg.d.ts

@@ -20,6 +20,8 @@ export declare function resolveOrg(aliasOrUsername?: string, options?: ResolveOr
  * The saved auth for an alias or username, without connecting to the org. `AuthInfo.create` only
  * accepts a username, so an alias is resolved to its username first.
  *
- * @throws {SfError} `NamedOrgNotFoundError` if no auth exists for the alias/username.
+ * @throws {SfError} `NamedOrgNotFoundError` if no auth exists for the alias/username;
+ * `NoUsernameFoundError` if it's blank (rather than `AuthInfo.create`'s unhelpful
+ * `AuthInfoCreationError`).
  */
 export declare function resolveAuthInfo(aliasOrUsername: string): Promise<AuthInfo>;

package/lib/resolveOrg.js

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { AuthInfo, ConfigAggregator, Org, StateAggregator } from '@salesforce/core';
+import { AuthInfo, ConfigAggregator, Org, SfError, StateAggregator } from '@salesforce/core';
 /**
  * The authenticated org that an `--alias`/`--target-org`-style value names, resolved the way `sf`
  * resolves `--target-org`: an alias or the username of an existing auth, with a blank value meaning
@@ -39,10 +39,16 @@ export async function resolveOrg(aliasOrUsername, options = {}) {
  * The saved auth for an alias or username, without connecting to the org. `AuthInfo.create` only
  * accepts a username, so an alias is resolved to its username first.
  *
- * @throws {SfError} `NamedOrgNotFoundError` if no auth exists for the alias/username.
+ * @throws {SfError} `NamedOrgNotFoundError` if no auth exists for the alias/username;
+ * `NoUsernameFoundError` if it's blank (rather than `AuthInfo.create`'s unhelpful
+ * `AuthInfoCreationError`).
  */
 export async function resolveAuthInfo(aliasOrUsername) {
+    const name = aliasOrUsername.trim();
+    if (!name) {
+        throw new SfError('No org alias or username was given (it was empty).', 'NoUsernameFoundError');
+    }
     const stateAggregator = await StateAggregator.getInstance();
-    return AuthInfo.create({ username: stateAggregator.aliases.resolveUsername(aliasOrUsername.trim()) });
+    return AuthInfo.create({ username: stateAggregator.aliases.resolveUsername(name) });
 }
 //# sourceMappingURL=resolveOrg.js.map

package/lib/resolveOrg.js.map

@@ -1 +1 @@
-{"version":3,"file":"resolveOrg.js","sourceRoot":"","sources":["../src/resolveOrg.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,QAAQ,EAAE,gBAAgB,EAAE,GAAG,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAOpF;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,eAAwB,EAAE,UAA6B,EAAE;IACxF,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,MAAM,IAAI,GAAG,eAAe,EAAE,IAAI,EAAE,CAAC;IACrC,IAAI,IAAI,EAAE,CAAC;QACT,OAAO,GAAG,CAAC,MAAM,CAAC,EAAE,eAAe,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;IACzD,CAAC;IACD,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,gBAAgB,CAAC,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC;IACpE,OAAO,GAAG,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,QAAQ,EAAE,CAAC,CAAC;AAC9C,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,eAAuB;IAC3D,MAAM,eAAe,GAAG,MAAM,eAAe,CAAC,WAAW,EAAE,CAAC;IAC5D,OAAO,QAAQ,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,eAAe,CAAC,OAAO,CAAC,eAAe,CAAC,eAAe,CAAC,IAAI,EAAE,CAAC,EAAE,CAAC,CAAC;AACxG,CAAC"}
+{"version":3,"file":"resolveOrg.js","sourceRoot":"","sources":["../src/resolveOrg.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,QAAQ,EAAE,gBAAgB,EAAE,GAAG,EAAE,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAO7F;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,eAAwB,EAAE,UAA6B,EAAE;IACxF,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,MAAM,IAAI,GAAG,eAAe,EAAE,IAAI,EAAE,CAAC;IACrC,IAAI,IAAI,EAAE,CAAC;QACT,OAAO,GAAG,CAAC,MAAM,CAAC,EAAE,eAAe,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;IACzD,CAAC;IACD,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,gBAAgB,CAAC,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC;IACpE,OAAO,GAAG,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,QAAQ,EAAE,CAAC,CAAC;AAC9C,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,eAAuB;IAC3D,MAAM,IAAI,GAAG,eAAe,CAAC,IAAI,EAAE,CAAC;IACpC,IAAI,CAAC,IAAI,EAAE,CAAC;QACV,MAAM,IAAI,OAAO,CAAC,oDAAoD,EAAE,sBAAsB,CAAC,CAAC;IAClG,CAAC;IACD,MAAM,eAAe,GAAG,MAAM,eAAe,CAAC,WAAW,EAAE,CAAC;IAC5D,OAAO,QAAQ,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,eAAe,CAAC,OAAO,CAAC,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;AACtF,CAAC"}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@simplysf/simply-cicd-core",
   "description": "The library layer behind @simplysf/simply-cicd: build/deploy/notify pipeline stages, sfdx-dependabot, ALM issue linking (Jira, GitLab Issues) and VCS API clients (GitHub, GitLab) — published separately for direct consumption (e.g. a bot or script) without the CLI framework",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "author": "@ClayChipps",
   "homepage": "https://github.com/SimplySF/simply-plugins-core",
   "bugs": "https://github.com/SimplySF/simply-plugins-core/issues",
@@ -142,5 +142,5 @@
       "output": []
     }
   },
-  "gitHead": "3767326fb9448f67ba8224accc6c74c5d0ef04ee"
+  "gitHead": "6628627030fcc8994b4ec95272a0a824d3f3bc58"
 }