@contrail/cli 3.5.5-alpha.960e382 → 3.5.5-alpha.a4b6430

package/README.md

@@ -17,7 +17,7 @@ $ npm install -g @contrail/cli
 $ contrail COMMAND
 running command...
 $ contrail (--version)
-@contrail/cli/3.5.5-alpha.960e382 linux-x64 node-v24.21.0
+@contrail/cli/3.5.5-alpha.a4b6430 linux-x64 node-v24.21.0
 $ contrail --help [COMMAND]
 USAGE
   $ contrail COMMAND

package/dist/api/apps.js

@@ -26,6 +26,8 @@ async function upgradeAppVersion(appOrg, appVersionId) {
         id: appOrg.id,
         object: {
             appVersionId: appVersionId,
+            // The API then updates installed workflows to the new version's configuration, keeping any value the org changed.
+            syncEventWorkflowTemplates: true,
         },
     });
 }

package/dist/api/apps.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/api/apps.spec.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const update = jest.fn();
+jest.mock('@contrail/sdk', () => ({
+    Auth: jest.fn(),
+    Entities: jest.fn(() => ({
+        update: (...args) => update(...args),
+    })),
+}));
+const apps_1 = require("./apps");
+describe('upgradeAppVersion', () => {
+    it('should ask the API to apply the new version workflow configuration along with the version change', async () => {
+        update.mockResolvedValue({ id: 'app-org-1' });
+        await (0, apps_1.upgradeAppVersion)({ id: 'app-org-1' }, 'version-2');
+        expect(update).toHaveBeenCalledWith({
+            entityName: 'app-org',
+            id: 'app-org-1',
+            object: { appVersionId: 'version-2', syncEventWorkflowTemplates: true },
+        });
+    });
+});

package/dist/commands/feature-flags/import.d.ts

@@ -34,5 +34,9 @@ export declare function uploadFeatureFlagsIfPermitted(dir: string): Promise<void
  *
  * Failures are collected and rethrown so an import never reports success while the org is
  * missing flags.
+ *
+ * A file that lists no flags at all is skipped rather than applied: reconciling against an
+ * empty list would disable every flag in the target org, which is almost never what a copy
+ * is meant to do. Flags that should be off in the target have to be disabled by hand.
  */
 export declare function uploadFeatureFlags(dir: string): Promise<void>;

package/dist/commands/feature-flags/import.js

@@ -107,6 +107,10 @@ exports.uploadFeatureFlagsIfPermitted = uploadFeatureFlagsIfPermitted;
  *
  * Failures are collected and rethrown so an import never reports success while the org is
  * missing flags.
+ *
+ * A file that lists no flags at all is skipped rather than applied: reconciling against an
+ * empty list would disable every flag in the target org, which is almost never what a copy
+ * is meant to do. Flags that should be off in the target have to be disabled by hand.
  */
 async function uploadFeatureFlags(dir) {
     const filePath = path.join(dir, export_1.FEATURE_FLAGS_FILE);
@@ -114,11 +118,16 @@ async function uploadFeatureFlags(dir) {
         logger_1.default.info(`${chalk_1.default.yellow.bold('FEATURE FLAGS')} No feature flags found in ${dir}. Skipping.`);
         return;
     }
-    const data = (0, file_helpers_1.loadData)(filePath);
+    const data = (0, file_helpers_1.loadData)(filePath) ?? [];
     if (!Array.isArray(data)) {
         throw new Error(`${filePath} must contain a YAML list of feature flags`);
     }
     const featureNamesToEnable = new Set(data.map(toFeatureName).filter(Boolean));
+    if (featureNamesToEnable.size === 0) {
+        logger_1.default.info(`${chalk_1.default.yellow.bold('FEATURE FLAGS')} ${filePath} lists no feature flags. Skipping, so ` +
+            'the flags currently enabled in this org are left as they are.');
+        return;
+    }
     const existingFlags = (await (0, feature_flags_1.getFeatureFlags)()) ?? [];
     const enabledFeatureNames = new Set(existingFlags.map((flag) => flag?.featureName));
     const toCreate = [...featureNamesToEnable].filter((name) => !enabledFeatureNames.has(name));

package/dist/commands/feature-flags/import.spec.js

@@ -58,6 +58,26 @@ describe('uploadFeatureFlags', () => {
         expect(mocked(feature_flags_1.getFeatureFlags)).not.toHaveBeenCalled();
         expect(mocked(feature_flags_1.createFeatureFlag)).not.toHaveBeenCalled();
     });
+    it('skips an export that lists no flags instead of disabling everything in the target org', async () => {
+        mocked(file_helpers_1.loadData).mockReturnValue([]);
+        mocked(feature_flags_1.getFeatureFlags).mockResolvedValue([
+            { id: '1', featureName: 'SHOWROOM' },
+            { id: '2', featureName: 'DEV_TABS' },
+        ]);
+        const infoSpy = jest.spyOn(logger_1.default, 'info').mockImplementation(() => undefined);
+        await (0, import_1.uploadFeatureFlags)('export-dir');
+        expect(mocked(feature_flags_1.deleteFeatureFlag)).not.toHaveBeenCalled();
+        expect(mocked(feature_flags_1.createFeatureFlag)).not.toHaveBeenCalled();
+        expect(infoSpy).toHaveBeenCalledWith(expect.stringContaining('lists no feature flags'));
+        infoSpy.mockRestore();
+    });
+    it('treats a blank feature-flags.yml the same as an empty list', async () => {
+        mocked(file_helpers_1.loadData).mockReturnValue(undefined);
+        mocked(feature_flags_1.getFeatureFlags).mockResolvedValue([{ id: '1', featureName: 'SHOWROOM' }]);
+        await (0, import_1.uploadFeatureFlags)('export-dir');
+        expect(mocked(feature_flags_1.getFeatureFlags)).not.toHaveBeenCalled();
+        expect(mocked(feature_flags_1.deleteFeatureFlag)).not.toHaveBeenCalled();
+    });
     it('enables every flag from the export that the target org is missing', async () => {
         mocked(file_helpers_1.loadData).mockReturnValue([{ featureName: 'SHOWROOM' }, { featureName: 'PROMOTE' }]);
         mocked(feature_flags_1.getFeatureFlags).mockResolvedValue([{ id: '1', featureName: 'SHOWROOM' }]);

package/dist/interfaces.d.ts

@@ -168,6 +168,7 @@ export interface AppOrg {
     appVersion?: AppVersion;
     apiKey?: AppApiKey;
     appConfig?: any;
+    syncEventWorkflowTemplates?: boolean;
 }
 export interface Webhook {
     id?: string;

package/oclif.manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "3.5.5-alpha.960e382",
+  "version": "3.5.5-alpha.a4b6430",
   "commands": {
     "app:create": {
       "id": "app:create",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/cli",
-  "version": "3.5.5-alpha.960e382",
+  "version": "3.5.5-alpha.a4b6430",
   "description": "VibeIQ's Contrail Platform CLI.",
   "bin": {
     "contrail-cli": "./bin/run",