@salesforce/packaging 0.0.26 → 0.0.29

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 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.
 
+### [0.0.29](https://github.com/forcedotcom/packaging/compare/v0.0.28...v0.0.29) (2022-08-26)
+
+### Bug Fixes
+
+- add PackageVersion.update, NUTs ([af8a816](https://github.com/forcedotcom/packaging/commit/af8a8167d5fbcf7056f7b72db49e4e9f6ed0363f))
+
+### [0.0.28](https://github.com/forcedotcom/packaging/compare/v0.0.27...v0.0.28) (2022-08-26)
+
+### Bug Fixes
+
+- finish pvl, implement a few missing things ([9bb825c](https://github.com/forcedotcom/packaging/commit/9bb825c97b6cb8c8b9522ddcb5e5fb32e5653995))
+
+### [0.0.27](https://github.com/forcedotcom/packaging/compare/v0.0.26...v0.0.27) (2022-08-26)
+
+### Bug Fixes
+
+- fix validateDependencyValues ([e37ac1e](https://github.com/forcedotcom/packaging/commit/e37ac1e08f5d764aefa8cf04271c6e72a30d78ea))
+
 ### [0.0.26](https://github.com/forcedotcom/packaging/compare/v0.0.25...v0.0.26) (2022-08-25)
 
 ### Bug Fixes

package/lib/interfaces/packagingInterfacesAndType.d.ts

@@ -147,17 +147,25 @@ export declare type PackageVersionCreateRequest = {
     CalculateCodeCoverage: boolean;
     SkipValidation: boolean;
 };
-export declare type PackageVersionQueryOptions = {
-    project: SfProject;
+export declare type PackageVersionListOptions = {
     orderBy: string;
     modifiedLastDays: number;
     createdLastDays: number;
     packages: string[];
-    connection: Connection;
     verbose: boolean;
     concise: boolean;
     isReleased: boolean;
 };
+export declare type PackageVersionUpdateOptions = {
+    InstallKey?: string;
+    VersionName?: string;
+    VersionDescription?: string;
+    Branch?: string;
+    Tag?: string;
+};
+export declare type ListPackageVersionOptions = PackageVersionListOptions & {
+    connection: Connection;
+};
 export declare type PackageSaveResult = SaveResult;
 export declare type PackageVersionCreateRequestOptions = {
     path: string;

package/lib/package/packageVersion.d.ts

@@ -1,5 +1,5 @@
 import { Duration } from '@salesforce/kit';
-import { PackageSaveResult, PackageVersionCreateOptions, PackageVersionCreateRequestResult, PackageVersionOptions, PackageVersionReportResult } from '../interfaces';
+import { PackageSaveResult, PackageVersionCreateOptions, PackageVersionCreateRequestResult, PackageVersionListOptions, PackageVersionListResult, PackageVersionOptions, PackageVersionReportResult, PackageVersionUpdateOptions } from '../interfaces';
 export declare class PackageVersion {
     private options;
     private readonly project;
@@ -56,10 +56,10 @@ export declare class PackageVersion {
     }): Promise<PackageVersionCreateRequestResult>;
     convert(): Promise<void>;
     install(): Promise<void>;
-    list(): Promise<void>;
+    list(options: PackageVersionListOptions): Promise<PackageVersionListResult[]>;
     uninstall(): Promise<void>;
     promote(id: string): Promise<PackageSaveResult>;
-    update(): Promise<void>;
+    update(id: string, options: PackageVersionUpdateOptions): Promise<PackageSaveResult>;
     private updateDeprecation;
     private updateProjectWithPackageVersion;
 }

package/lib/package/packageVersion.js

@@ -13,6 +13,7 @@ const utils_1 = require("../utils");
 const packageVersionCreate_1 = require("./packageVersionCreate");
 const packageVersionReport_1 = require("./packageVersionReport");
 const packageVersionCreateRequestReport_1 = require("./packageVersionCreateRequestReport");
+const packageVersionList_1 = require("./packageVersionList");
 core_1.Messages.importMessagesDirectory(__dirname);
 class PackageVersion {
     constructor(options) {
@@ -153,8 +154,8 @@ class PackageVersion {
     install() {
         return Promise.resolve(undefined);
     }
-    list() {
-        return Promise.resolve(undefined);
+    async list(options) {
+        return (await (0, packageVersionList_1.listPackageVersions)({ ...options, ...{ connection: this.connection } })).records;
     }
     uninstall() {
         return Promise.resolve(undefined);
@@ -166,8 +167,28 @@ class PackageVersion {
         }
         return await this.options.connection.tooling.update('Package2Version', { IsReleased: true, Id: id });
     }
-    update() {
-        return Promise.resolve(undefined);
+    async update(id, options) {
+        // ID can be an 04t or 05i
+        (0, utils_1.validateId)([utils_1.BY_LABEL.SUBSCRIBER_PACKAGE_VERSION_ID, utils_1.BY_LABEL.PACKAGE_VERSION_ID], id);
+        // lookup the 05i ID, if needed
+        id = await (0, utils_1.getPackageVersionId)(id, this.connection);
+        const request = {
+            Id: id,
+            InstallKey: options.InstallKey,
+            Name: options.VersionName,
+            Description: options.VersionDescription,
+            Branch: options.Branch,
+            Tag: options.Tag,
+        };
+        // filter out any undefined values and their keys
+        Object.keys(request).forEach((key) => request[key] === undefined && delete request[key]);
+        const result = await this.connection.tooling.update('Package2Version', request);
+        if (!result.success) {
+            throw new Error(result.errors.join(', '));
+        }
+        // Use the 04t ID for the success message
+        result.id = await (0, utils_1.getSubscriberPackageVersionId)(id, this.connection);
+        return result;
     }
     async updateDeprecation(idOrAlias, IsDeprecated) {
         const packageVersionId = (0, utils_1.getPackageIdFromAlias)(idOrAlias, this.project);

package/lib/package/packageVersionCreate.js

@@ -14,7 +14,6 @@ const core_1 = require("@salesforce/core");
 const source_deploy_retrieve_1 = require("@salesforce/source-deploy-retrieve");
 const scratchOrgSettingsGenerator_1 = require("@salesforce/core/lib/org/scratchOrgSettingsGenerator");
 const xml2js = require("xml2js");
-const scratchOrgInfoGenerator_1 = require("@salesforce/core/lib/org/scratchOrgInfoGenerator");
 const uniqid_1 = require("../utils/uniqid");
 const pkgUtils = require("../utils/packageUtils");
 const versionNumber_1 = require("../utils/versionNumber");
@@ -75,19 +74,22 @@ class PackageVersionCreate {
         // If valid 04t package, just return it to be used straight away.
         if (dependency.subscriberPackageVersionId) {
             pkgUtils.validateId(pkgUtils.BY_LABEL.SUBSCRIBER_PACKAGE_VERSION_ID, dependency.subscriberPackageVersionId);
+            return;
         }
         if (dependency.packageId && dependency.package) {
             throw messages.createError('errorPackageAndPackageIdCollision', []);
         }
+        const packageIdFromAlias = pkgUtils.getPackageIdFromAlias(dependency.packageId || dependency.package, this.project);
         // If valid 04t package, just return it to be used straight away.
-        if (pkgUtils.validateIdNoThrow(pkgUtils.BY_LABEL.SUBSCRIBER_PACKAGE_VERSION_ID, this.packageId)) {
-            dependency.subscriberPackageVersionId = this.packageId;
+        if (pkgUtils.validateIdNoThrow(pkgUtils.BY_LABEL.SUBSCRIBER_PACKAGE_VERSION_ID, packageIdFromAlias)) {
+            dependency.subscriberPackageVersionId = packageIdFromAlias;
+            return;
         }
-        if (!this.packageId || !dependency.versionNumber) {
+        if (!packageIdFromAlias || !dependency.versionNumber) {
             throw messages.createError('errorDependencyPair', [JSON.stringify(dependency)]);
         }
         // Just override dependency.packageId value to the resolved alias.
-        dependency.packageId = this.packageId;
+        dependency.packageId = packageIdFromAlias;
         pkgUtils.validateId(pkgUtils.BY_LABEL.PACKAGE_ID, dependency.packageId);
         pkgUtils.validateVersionNumber(dependency.versionNumber, versionNumber_1.BuildNumberToken.LATEST_BUILD_NUMBER_TOKEN, versionNumber_1.BuildNumberToken.RELEASED_BUILD_NUMBER_TOKEN);
         // Validate that the Package2 id exists on the server
@@ -96,7 +98,6 @@ class PackageVersionCreate {
         if (!result.records || result.records.length !== 1) {
             throw messages.createError('errorNoIdInHub', [dependency.packageId]);
         }
-        return result;
     }
     /**
      * A dependency in the workspace config file may be specified using either a subscriber package version id (04t)
@@ -206,7 +207,12 @@ class PackageVersionCreate {
             SkipValidation: this.options.skipvalidation || false,
         };
         if (preserveFiles) {
-            logger.info(messages.getMessage('tempFileLocation', [packageVersTmpRoot]));
+            const message = messages.getMessage('tempFileLocation', [packageVersTmpRoot]);
+            await core_1.Lifecycle.getInstance().emit('packageVersionCreate:preserveFiles', {
+                location: packageVersTmpRoot,
+                message,
+            });
+            logger.info(message);
             return requestObject;
         }
         else {
@@ -279,9 +285,7 @@ class PackageVersionCreate {
         // branch can be set via options or descriptor; option takes precedence
         this.options.branch = this.options.branch ?? packageDescriptorJson.branch;
         const resultValues = await Promise.all(!dependencies ? [] : dependencies.map((dependency) => this.retrieveSubscriberPackageVersionId(dependency)));
-        const ancestorId = await (0, scratchOrgInfoGenerator_1.getAncestorIds)(
-        // TODO: investigate if it's ok to convert to ScratchOggInfoPayload
-        this.packageObject, this.options.project.getSfProjectJson(), await core_1.Org.create({ aliasOrUsername: this.options.connection.getUsername() }));
+        const ancestorId = await (0, utils_1.getAncestorId)(packageDescriptorJson, this.options.project, this.options.connection, this.options.versionnumber ?? packageDescriptorJson.versionNumber, this.options.skipancestorcheck);
         // If dependencies exist, the resultValues array will contain the dependencies populated with a resolved
         // subscriber pkg version id.
         if (resultValues.length > 0) {

package/lib/package/packageVersionList.d.ts

@@ -1,9 +1,7 @@
-import { SfProject } from '@salesforce/core';
 import { QueryResult } from 'jsforce';
-import { PackageVersionListResult, PackageVersionQueryOptions } from '../interfaces';
+import { PackageVersionListResult, ListPackageVersionOptions } from '../interfaces';
 export declare const DEFAULT_ORDER_BY_FIELDS = "Package2Id, Branch, MajorVersion, MinorVersion, PatchVersion, BuildNumber";
-export declare function listPackageVersions(options: PackageVersionQueryOptions): Promise<QueryResult<PackageVersionListResult>>;
-export declare function _constructQuery(options: PackageVersionQueryOptions): string;
-export declare function _assembleQueryParts(select: string, where: string[], orderBy?: string): string;
-export declare function _constructWhere(idsOrAliases: string[], createdLastDays: number, lastModLastDays: number, project: SfProject): string[];
-export declare function _getLastDays(paramName: string, lastDays: number): number;
+export declare function listPackageVersions(options: ListPackageVersionOptions): Promise<QueryResult<PackageVersionListResult>>;
+export declare function assembleQueryParts(select: string, where: string[], orderBy?: string): string;
+export declare function constructWhere(packageIds: string[], createdLastDays: number, lastModLastDays: number, isReleased: boolean): string[];
+export declare function validateDays(paramName: string, lastDays: number): number;

package/lib/package/packageVersionList.js

@@ -6,7 +6,7 @@
  * For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports._getLastDays = exports._constructWhere = exports._assembleQueryParts = exports._constructQuery = exports.listPackageVersions = exports.DEFAULT_ORDER_BY_FIELDS = void 0;
+exports.validateDays = exports.constructWhere = exports.assembleQueryParts = exports.listPackageVersions = exports.DEFAULT_ORDER_BY_FIELDS = void 0;
 const core_1 = require("@salesforce/core");
 const ts_types_1 = require("@salesforce/ts-types");
 const utils_1 = require("../utils");
@@ -25,67 +25,60 @@ const VERBOSE_SELECT = 'SELECT Id, Package2Id, SubscriberPackageVersionId, Name,
 exports.DEFAULT_ORDER_BY_FIELDS = 'Package2Id, Branch, MajorVersion, MinorVersion, PatchVersion, BuildNumber';
 const logger = core_1.Logger.childFromRoot('packageVersionList');
 async function listPackageVersions(options) {
-    return options.connection.tooling.query(_constructQuery(options));
+    return options.connection.tooling.query(constructQuery(options));
 }
 exports.listPackageVersions = listPackageVersions;
-function _constructQuery(options) {
+function constructQuery(options) {
     // construct custom WHERE clause, if applicable
-    const where = _constructWhere(options.packages, options.createdLastDays, options.modifiedLastDays, options.project);
-    if (options.isReleased) {
-        where.push('IsReleased = true');
-    }
-    return _assembleQueryParts(options.verbose === true ? VERBOSE_SELECT : DEFAULT_SELECT, where, options.orderBy);
+    const where = constructWhere(options.packages, options.createdLastDays, options.modifiedLastDays, options.isReleased);
+    return assembleQueryParts(options.verbose === true ? VERBOSE_SELECT : DEFAULT_SELECT, where, options.orderBy);
 }
-exports._constructQuery = _constructQuery;
-function _assembleQueryParts(select, where, orderBy = exports.DEFAULT_ORDER_BY_FIELDS) {
+function assembleQueryParts(select, where, orderBy) {
     // construct ORDER BY clause
-    // TODO: validate given fields
     const orderByPart = `ORDER BY ${orderBy ? orderBy : exports.DEFAULT_ORDER_BY_FIELDS}`;
     const wherePart = where.length > 0 ? `WHERE ${where.join(' AND ')}` : '';
     const query = `${select} ${wherePart} ${orderByPart}`;
     logger.debug(query);
     return query;
 }
-exports._assembleQueryParts = _assembleQueryParts;
+exports.assembleQueryParts = assembleQueryParts;
 // construct custom WHERE clause parts
-function _constructWhere(idsOrAliases, createdLastDays, lastModLastDays, project) {
+function constructWhere(packageIds, createdLastDays, lastModLastDays, isReleased) {
     const where = [];
     // filter on given package ids
-    if (idsOrAliases?.length > 0) {
+    if (packageIds?.length > 0) {
         // remove dups
-        const aliasesOrIds = [...new Set(idsOrAliases)];
-        // resolve any aliases
-        const packageIds = aliasesOrIds.map((idOrAlias) => (0, utils_1.getPackageIdFromAlias)(idOrAlias, project));
+        const uniquePackageIds = [...new Set(packageIds)];
         // validate ids
-        packageIds.forEach((packageId) => {
+        uniquePackageIds.forEach((packageId) => {
             (0, utils_1.validateId)(utils_1.BY_LABEL.PACKAGE_ID, packageId);
         });
         // stash where part
-        where.push(`Package2Id IN ('${packageIds.join("','")}')`);
+        where.push(`Package2Id IN ('${uniquePackageIds.join("','")}')`);
     }
     // filter on created date, days ago: 0 for today, etc
     if ((0, ts_types_1.isNumber)(createdLastDays)) {
-        createdLastDays = _getLastDays('createdlastdays', createdLastDays);
+        createdLastDays = validateDays('createdlastdays', createdLastDays);
         where.push(`CreatedDate = LAST_N_DAYS:${createdLastDays}`);
     }
     // filter on last mod date, days ago: 0 for today, etc
     if ((0, ts_types_1.isNumber)(lastModLastDays)) {
-        lastModLastDays = _getLastDays('modifiedlastdays', lastModLastDays);
+        lastModLastDays = validateDays('modifiedlastdays', lastModLastDays);
         where.push(`LastModifiedDate = LAST_N_DAYS:${lastModLastDays}`);
     }
+    if (isReleased) {
+        where.push('IsReleased = true');
+    }
     // exclude deleted
     where.push('IsDeprecated = false');
     return where;
 }
-exports._constructWhere = _constructWhere;
-function _getLastDays(paramName, lastDays) {
-    if (isNaN(lastDays)) {
-        return 0;
-    }
+exports.constructWhere = constructWhere;
+function validateDays(paramName, lastDays) {
     if (lastDays < 0) {
         throw messages.createError('invalidDaysNumber', [paramName, `${lastDays}`]);
     }
     return lastDays;
 }
-exports._getLastDays = _getLastDays;
+exports.validateDays = validateDays;
 //# sourceMappingURL=packageVersionList.js.map

package/lib/utils/packageUtils.d.ts

@@ -2,7 +2,7 @@ import { Connection, NamedPackageDir, PackageDir, SfdcUrl, SfError, SfProject }
 import { Duration } from '@salesforce/kit';
 import { Many, Nullable } from '@salesforce/ts-types';
 import { SaveError } from 'jsforce';
-import { PackageType, PackageVersionCreateRequestResult, PackagingSObjects } from '../interfaces';
+import { PackageDescriptorJson, PackageType, PackageVersionCreateRequestResult, PackagingSObjects } from '../interfaces';
 export declare const VERSION_NUMBER_SEP = ".";
 export declare type IdRegistryValue = {
     prefix: string;
@@ -102,6 +102,7 @@ export declare function queryWithInConditionChunking<T = Record<string, unknown>
  */
 export declare function getInClauseItemsCount(items: string[], startIndex: number, maxLength: number): number;
 export declare function validateAncestorId(ancestorId: string, highestReleasedVersion: PackagingSObjects.Package2Version, explicitUseNoAncestor: boolean, isPatch: boolean, skipAncestorCheck: boolean, origSpecifiedAncestor: string): string;
+export declare function getAncestorId(packageDescriptorJson: PackageDescriptorJson, project: SfProject, connection: Connection, versionNumberString: string, skipAncestorCheck: boolean): Promise<string>;
 export declare function getAncestorIdHighestRelease(connection: Connection, packageId: string, versionNumberString: string, explicitUseHighestRelease: boolean, skipAncestorCheck: boolean): Promise<{
     finalAncestorId: string;
     highestReleasedVersion: PackagingSObjects.Package2Version;

package/lib/utils/packageUtils.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.combineSaveErrors = exports.formatDate = exports.generatePackageAliasEntry = exports.pollForStatusWithInterval = exports.findOrCreatePackage = exports.getPackageAliasesFromId = exports.convertCamelCaseStringToSentence = exports.getPackageIdFromAlias = exports.getConfigPackageDirectory = exports.getConfigPackageDirectories = exports.getPackageVersionNumber = exports.concatVersion = exports.getAncestorIdHighestRelease = exports.validateAncestorId = exports.getInClauseItemsCount = exports.queryWithInConditionChunking = exports.getPackageVersionStrings = exports.getHasMetadataRemoved = exports.getContainerOptions = exports.getSubscriberPackageVersionId = exports.getPackageTypeBy04t = exports.getPackageType = exports.escapeInstallationKey = exports.getPackageVersionId = exports.applyErrorAction = exports.massageErrorMessage = exports.isErrorPackageNotAvailable = exports.isErrorFromSPVQueryRestriction = exports.validatePatchVersion = exports.validateVersionNumber = exports.validateIdNoThrow = exports.validateId = exports.BY_LABEL = exports.BY_PREFIX = exports.DEFAULT_PACKAGE_DIR = exports.POLL_INTERVAL_SECONDS = exports.SOQL_WHERE_CLAUSE_MAX_LENGTH = exports.INSTALL_URL_BASE = exports.VERSION_NUMBER_SEP = void 0;
+exports.combineSaveErrors = exports.formatDate = exports.generatePackageAliasEntry = exports.pollForStatusWithInterval = exports.findOrCreatePackage = exports.getPackageAliasesFromId = exports.convertCamelCaseStringToSentence = exports.getPackageIdFromAlias = exports.getConfigPackageDirectory = exports.getConfigPackageDirectories = exports.getPackageVersionNumber = exports.concatVersion = exports.getAncestorIdHighestRelease = exports.getAncestorId = exports.validateAncestorId = exports.getInClauseItemsCount = exports.queryWithInConditionChunking = exports.getPackageVersionStrings = exports.getHasMetadataRemoved = exports.getContainerOptions = exports.getSubscriberPackageVersionId = exports.getPackageTypeBy04t = exports.getPackageType = exports.escapeInstallationKey = exports.getPackageVersionId = exports.applyErrorAction = exports.massageErrorMessage = exports.isErrorPackageNotAvailable = exports.isErrorFromSPVQueryRestriction = exports.validatePatchVersion = exports.validateVersionNumber = exports.validateIdNoThrow = exports.validateId = exports.BY_LABEL = exports.BY_PREFIX = exports.DEFAULT_PACKAGE_DIR = exports.POLL_INTERVAL_SECONDS = exports.SOQL_WHERE_CLAUSE_MAX_LENGTH = exports.INSTALL_URL_BASE = exports.VERSION_NUMBER_SEP = void 0;
 /*
  * Copyright (c) 2022, salesforce.com, inc.
  * All rights reserved.
@@ -324,7 +324,9 @@ async function getPackageVersionStrings(subscriberPackageVersionIds, connection)
     }
     // remove any duplicate Ids
     const ids = [...new Set(subscriberPackageVersionIds)];
-    const query = 'SELECT SubscriberPackageVersionId, MajorVersion, MinorVersion, PatchVersion, BuildNumber FROM Package2Version WHERE SubscriberPackageVersionId IN (%IDS%)';
+    const query = `SELECT SubscriberPackageVersionId, MajorVersion, MinorVersion, PatchVersion, BuildNumber FROM Package2Version WHERE SubscriberPackageVersionId IN (${ids
+        .map((id) => `'${id}'`)
+        .join(',')})`;
     const records = await queryWithInConditionChunking(query, ids, '%IDS%', connection);
     if (records && records.length > 0) {
         results = new Map(records.map((record) => {
@@ -423,6 +425,92 @@ function validateAncestorId(ancestorId, highestReleasedVersion, explicitUseNoAnc
     return ancestorId;
 }
 exports.validateAncestorId = validateAncestorId;
+// eslint-disable-next-line complexity
+async function getAncestorId(packageDescriptorJson, project, connection, versionNumberString, skipAncestorCheck) {
+    // If an id property is present, use it.  Otherwise, look up the package id from the package property.
+    const packageId = packageDescriptorJson.id ?? getPackageIdFromAlias(packageDescriptorJson.package, project);
+    // No need to proceed if Unlocked
+    const packageType = await getPackageType(packageId, connection);
+    if (packageType === 'Unlocked') {
+        return '';
+    }
+    let ancestorId = '';
+    // ancestorID can be alias, 05i, or 04t;
+    // validate and convert to 05i, as needed
+    const versionNumber = versionNumber_1.VersionNumber.from(versionNumberString);
+    let origSpecifiedAncestor = packageDescriptorJson.ancestorId;
+    let highestReleasedVersion = null;
+    const explicitUseHighestRelease = packageDescriptorJson.ancestorId === versionNumber_1.BuildNumberToken.HIGHEST_VERSION_NUMBER_TOKEN ||
+        packageDescriptorJson.ancestorVersion === versionNumber_1.BuildNumberToken.HIGHEST_VERSION_NUMBER_TOKEN;
+    const explicitUseNoAncestor = packageDescriptorJson.ancestorId === versionNumber_1.BuildNumberToken.NONE_VERSION_NUMBER_TOKEN ||
+        packageDescriptorJson.ancestorVersion === versionNumber_1.BuildNumberToken.NONE_VERSION_NUMBER_TOKEN;
+    if ((explicitUseHighestRelease || explicitUseNoAncestor) &&
+        packageDescriptorJson.ancestorId &&
+        packageDescriptorJson.ancestorVersion) {
+        if (packageDescriptorJson.ancestorId !== packageDescriptorJson.ancestorVersion) {
+            // both ancestorId and ancestorVersion specified, HIGHEST and/or NONE are used, the values disagree
+            throw messages.createError('errorAncestorIdVersionHighestOrNoneMismatch', [
+                packageDescriptorJson.ancestorId,
+                packageDescriptorJson.ancestorVersion,
+            ]);
+        }
+    }
+    if (explicitUseNoAncestor && skipAncestorCheck) {
+        return '';
+    }
+    else {
+        const result = await getAncestorIdHighestRelease(connection, packageId, versionNumberString, explicitUseHighestRelease, skipAncestorCheck);
+        if (result.finalAncestorId) {
+            return result.finalAncestorId;
+        }
+        highestReleasedVersion = result.highestReleasedVersion;
+    }
+    // at this point if explicitUseHighestRelease=true, we have returned the ancestorId or thrown an error
+    // highestReleasedVersion should be null only if skipAncestorCheck or if there is no existing released package version
+    if (!explicitUseNoAncestor && packageDescriptorJson.ancestorId) {
+        ancestorId = getPackageIdFromAlias(packageDescriptorJson.ancestorId, project);
+        validateId([exports.BY_LABEL.SUBSCRIBER_PACKAGE_VERSION_ID, exports.BY_LABEL.PACKAGE_VERSION_ID], ancestorId);
+        ancestorId = await getPackageVersionId(ancestorId, connection);
+    }
+    if (!explicitUseNoAncestor && packageDescriptorJson.ancestorVersion) {
+        const regNumbers = new RegExp('^[0-9]+$');
+        const versionNumber = packageDescriptorJson.ancestorVersion.split(exports.VERSION_NUMBER_SEP);
+        if (versionNumber.length < 3 ||
+            versionNumber.length > 4 ||
+            !versionNumber[0].match(regNumbers) ||
+            !versionNumber[1].match(regNumbers) ||
+            !versionNumber[2].match(regNumbers)) {
+            throw new Error(messages.getMessage('errorInvalidAncestorVersionFormat', [packageDescriptorJson.ancestorVersion]));
+        }
+        const query = 'SELECT Id, IsReleased FROM Package2Version ' +
+            `WHERE Package2Id = '${packageId}' AND MajorVersion = ${versionNumber[0]} AND MinorVersion = ${versionNumber[1]} AND PatchVersion = ${versionNumber[2]}`;
+        let queriedAncestorId;
+        const ancestorVersionResult = await connection.tooling.query(query);
+        if (!ancestorVersionResult || !ancestorVersionResult.totalSize) {
+            throw messages.createError('errorNoMatchingAncestor', [packageDescriptorJson.ancestorVersion, packageId]);
+        }
+        else {
+            const releasedAncestor = ancestorVersionResult.records.find((rec) => rec.IsReleased === true);
+            if (!releasedAncestor) {
+                throw messages.createError('errorAncestorNotReleased', [packageDescriptorJson.ancestorVersion]);
+            }
+            else {
+                queriedAncestorId = releasedAncestor.Id;
+            }
+        }
+        // check for discrepancy between queried ancestorId and descriptor's ancestorId
+        if (Object.prototype.hasOwnProperty.call(packageDescriptorJson, 'ancestorId') && ancestorId !== queriedAncestorId) {
+            throw messages.createError('errorAncestorIdVersionMismatch', [
+                packageDescriptorJson.ancestorVersion,
+                packageDescriptorJson.ancestorId,
+            ]);
+        }
+        ancestorId = queriedAncestorId;
+        origSpecifiedAncestor = packageDescriptorJson.ancestorVersion;
+    }
+    return validateAncestorId(ancestorId, highestReleasedVersion, explicitUseNoAncestor, versionNumber.patch !== '0', skipAncestorCheck, origSpecifiedAncestor);
+}
+exports.getAncestorId = getAncestorId;
 async function getAncestorIdHighestRelease(connection, packageId, versionNumberString, explicitUseHighestRelease, skipAncestorCheck) {
     const versionNumber = versionNumberString.split(exports.VERSION_NUMBER_SEP);
     const isPatch = versionNumber[2] !== '0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/packaging",
-  "version": "0.0.26",
+  "version": "0.0.29",
   "description": "packing libraries to Salesforce packaging platform",
   "main": "lib/exported",
   "types": "lib/exported.d.ts",