@salesforce/packaging 0.0.23 → 0.0.26

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.26](https://github.com/forcedotcom/packaging/compare/v0.0.25...v0.0.26) (2022-08-25)
+
+### Bug Fixes
+
+- remove hard coded string ([8c6d2dd](https://github.com/forcedotcom/packaging/commit/8c6d2dd769c2a91d33c8afafca02f3489c296060))
+
+### [0.0.25](https://github.com/forcedotcom/packaging/compare/v0.0.24...v0.0.25) (2022-08-25)
+
+### Bug Fixes
+
+- improve packge property maintenance ([c62acf8](https://github.com/forcedotcom/packaging/commit/c62acf84ad84deec6d77079200290ab09064f6d7))
+
+### [0.0.24](https://github.com/forcedotcom/packaging/compare/v0.0.23...v0.0.24) (2022-08-24)
+
+### Bug Fixes
+
+- correct package alias generation ([c4617bd](https://github.com/forcedotcom/packaging/commit/c4617bd16eac1f2bfe5efadafc1062ea78ee352c))
+
 ### [0.0.23](https://github.com/forcedotcom/packaging/compare/v0.0.22...v0.0.23) (2022-08-22)
 
 ### Bug Fixes

package/lib/package/packageVersion.d.ts

@@ -62,5 +62,4 @@ export declare class PackageVersion {
     update(): Promise<void>;
     private updateDeprecation;
     private updateProjectWithPackageVersion;
-    private generatePackageDirectory;
 }

package/lib/package/packageVersion.js

@@ -13,7 +13,6 @@ const utils_1 = require("../utils");
 const packageVersionCreate_1 = require("./packageVersionCreate");
 const packageVersionReport_1 = require("./packageVersionReport");
 const packageVersionCreateRequestReport_1 = require("./packageVersionCreateRequestReport");
-const package_1 = require("./package");
 core_1.Messages.importMessagesDirectory(__dirname);
 class PackageVersion {
     constructor(options) {
@@ -129,19 +128,8 @@ class PackageVersion {
                             payload: report,
                         };
                     case 'Success':
-                        await this.updateProjectWithPackageVersion(this.project, report);
                         await core_1.Lifecycle.getInstance().emit('success', report);
-                        if (!process.env.SFDX_PROJECT_AUTOUPDATE_DISABLE_FOR_PACKAGE_CREATE) {
-                            // get the newly created package version from the server
-                            const versionResult = (await this.connection.tooling.query(`SELECT Branch, MajorVersion, MinorVersion, PatchVersion, BuildNumber FROM Package2Version WHERE SubscriberPackageVersionId='${report.SubscriberPackageVersionId}'`)).records[0];
-                            const version = `${(0, utils_1.getPackageAliasesFromId)(report.Package2Id, this.project).join()}@${versionResult.MajorVersion ?? 0}.${versionResult.MinorVersion ?? 0}.${versionResult.PatchVersion ?? 0}`;
-                            const build = versionResult.BuildNumber ? `-${versionResult.BuildNumber}` : '';
-                            const branch = versionResult.Branch ? `-${versionResult.Branch}` : '';
-                            // set packageAliases entry '<package>@<major>.<minor>.<patch>-<build>-<branch>: <result.subscriberPackageVersionId>'
-                            this.project.getSfProjectJson().getContents().packageAliases[`${version}${build}${branch}`] =
-                                report.SubscriberPackageVersionId;
-                            await this.project.getSfProjectJson().write();
-                        }
+                        await this.updateProjectWithPackageVersion(this.project, report);
                         return { completed: true, payload: report };
                     case 'Error':
                         await core_1.Lifecycle.getInstance().emit('error', report);
@@ -201,25 +189,17 @@ class PackageVersion {
     }
     async updateProjectWithPackageVersion(withProject, results) {
         if (withProject && !process.env.SFDX_PROJECT_AUTOUPDATE_DISABLE_FOR_PACKAGE_VERSION_CREATE) {
-            const query = `SELECT Name, Package2Id, MajorVersion, MinorVersion, PatchVersion, BuildNumber, Description, Branch FROM Package2Version WHERE Id = '${results.Package2VersionId}'`;
-            const packageVersion = await this.connection.singleRecordQuery(query, {
-                tooling: true,
-            });
-            const packageVersionVersionString = `${packageVersion.MajorVersion}.${packageVersion.MinorVersion}.${packageVersion.PatchVersion}.${packageVersion.BuildNumber}`;
-            await this.generatePackageDirectory(packageVersion, withProject, packageVersionVersionString);
-            const newConfig = await (0, utils_1.generatePackageAliasEntry)(this.connection, withProject, packageVersion.SubscriberPackageVersionId, packageVersionVersionString, packageVersion.Branch, packageVersion.Package2Id);
-            withProject.getSfProjectJson().set('packageAliases', newConfig);
-            await withProject.getSfProjectJson().write();
+            // get the newly created package version from the server
+            const versionResult = (await this.connection.tooling.query(`SELECT Branch, MajorVersion, MinorVersion, PatchVersion, BuildNumber FROM Package2Version WHERE SubscriberPackageVersionId='${results.SubscriberPackageVersionId}'`)).records[0];
+            const version = `${(0, utils_1.getPackageAliasesFromId)(results.Package2Id, this.project).join()}@${versionResult.MajorVersion ?? 0}.${versionResult.MinorVersion ?? 0}.${versionResult.PatchVersion ?? 0}`;
+            const build = versionResult.BuildNumber ? `-${versionResult.BuildNumber}` : '';
+            const branch = versionResult.Branch ? `-${versionResult.Branch}` : '';
+            // set packageAliases entry '<package>@<major>.<minor>.<patch>-<build>-<branch>: <result.subscriberPackageVersionId>'
+            this.project.getSfProjectJson().getContents().packageAliases[`${version}${build}${branch}`] =
+                results.SubscriberPackageVersionId;
+            await this.project.getSfProjectJson().write();
         }
     }
-    async generatePackageDirectory(packageVersion, withProject, packageVersionVersionString) {
-        const pkg = await (await package_1.Package.create({ connection: this.connection })).getPackage(packageVersion.Package2Id);
-        const pkgDir = (0, utils_1.getConfigPackageDirectory)(withProject.getPackageDirectories(), 'id', pkg.Id) ?? {};
-        pkgDir.versionNumber = packageVersionVersionString;
-        pkgDir.versionDescription = packageVersion.Description;
-        const packageDirs = withProject.getPackageDirectories().map((pd) => (pkgDir['id'] === pd['id'] ? pkgDir : pd));
-        withProject.getSfProjectJson().set('packageDirectories', packageDirs);
-    }
 }
 exports.PackageVersion = PackageVersion;
 //# sourceMappingURL=packageVersion.js.map

package/lib/package/packageVersionCreate.js

@@ -415,10 +415,12 @@ class PackageVersionCreate {
         }
         // from the packageDirectories in sfdx-project.json, find the correct package entry either by finding a matching package (name) or path
         this.packageAlias = (0, utils_1.getPackageAliasesFromId)(this.options.packageId, this.options.project).join();
-        // set on the class so we can access them in other methods without redoing this logic
-        this.packageObject = this.project.getPackageDirectories().find((pkg) => pkg.package === this.packageAlias);
+        this.packageId = this.options.packageId;
+        // set on the class, so we can access them in other methods without redoing this logic
+        this.packageObject = this.project
+            .getPackageDirectories()
+            .find((pkg) => pkg.package === this.packageAlias || pkg['id'] === this.options.packageId);
         this.options.profileApi = await this.resolveUserLicenses(this.packageObject.includeProfileUserLicenses);
-        this.packageId = (0, utils_1.getPackageIdFromAlias)(this.packageObject.package, this.project);
         // At this point, the packageIdFromAlias should have been resolved to an Id.  Now, we
         // need to validate that the Id is correct.
         pkgUtils.validateId(pkgUtils.BY_LABEL.PACKAGE_ID, this.packageId);

package/lib/utils/packageUtils.d.ts

@@ -148,9 +148,7 @@ export declare function pollForStatusWithInterval(id: string, retries: number, p
  * @private
  */
 export declare function generatePackageAliasEntry(connection: Connection, project: SfProject, packageVersionId: string, packageVersionNumber: string, branch: string, packageId: string): Promise<{
-    packageAliases: {
-        [p: string]: string;
-    };
+    [p: string]: string;
 }>;
 export declare function formatDate(date: Date): string;
 export declare function combineSaveErrors(sObject: string, crudOperation: string, errors: SaveError[]): SfError;

package/lib/utils/packageUtils.js

@@ -72,7 +72,7 @@ function validateIdNoThrow(idObj, value) {
 }
 exports.validateIdNoThrow = validateIdNoThrow;
 function validateVersionNumber(versionNumberString, supportedBuildNumberToken, supportedBuildNumberToken2) {
-    const versionNumber = versionNumber_1.VersionNumber.from('1.1.0.NEXT');
+    const versionNumber = versionNumber_1.VersionNumber.from(versionNumberString);
     // build number can be a number or valid token
     if (Number.isNaN(parseInt(versionNumber.build, 10)) &&
         versionNumber.build !== supportedBuildNumberToken &&
@@ -653,7 +653,7 @@ async function generatePackageAliasEntry(connection, project, packageVersionId,
         ? `${packageName}@${packageVersionNumber}-${branch}`
         : `${packageName}@${packageVersionNumber}`;
     packageAliases[packageAlias] = packageVersionId;
-    return { packageAliases };
+    return packageAliases;
 }
 exports.generatePackageAliasEntry = generatePackageAliasEntry;
 /**

package/package.json

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