@tywalk/pcf-helper-run 1.3.0 → 1.3.2

package/dist/__tests__/argumentUtil.test.js

@@ -0,0 +1,112 @@
+"use strict";
+/// <reference types="jest" />
+/// <reference types="node" />
+Object.defineProperty(exports, "__esModule", { value: true });
+const argumentUtil_1 = require("../util/argumentUtil");
+describe('pcf-helper-run argumentUtil', () => {
+    describe('getArg', () => {
+        it('should return the argument value if found', () => {
+            const args = ['-p', '/path/to/project', '-v'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('/path/to/project');
+        });
+        it('should return undefined if argument not found', () => {
+            const args = ['-p', '/path/to/project'];
+            expect((0, argumentUtil_1.getArg)(args, '-v')).toBeUndefined();
+        });
+        it('should return undefined if argument is at the end with no value', () => {
+            const args = ['-p', '/path/to/project', '-v'];
+            expect((0, argumentUtil_1.getArg)(args, '-v')).toBeUndefined();
+        });
+        it('should handle empty args', () => {
+            const args = [];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBeUndefined();
+        });
+        it('should find value after flag', () => {
+            const args = ['build', '-p', '/project', 'deploy'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('/project');
+        });
+        it('should return undefined for flag at end of array', () => {
+            const args = ['build', '-v', '-p'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBeUndefined();
+        });
+    });
+    describe('getArgValue', () => {
+        it('should return value for matching option', () => {
+            const args = ['-n', 'controlName'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-n'])).toBe('controlName');
+        });
+        it('should return value for long form option', () => {
+            const args = ['--name', 'controlName'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['--name'])).toBe('controlName');
+        });
+        it('should find first matching option from list', () => {
+            const args = ['--publisher-name', 'MyPub'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-pn', '--publisher-name'])).toBe('MyPub');
+        });
+        it('should return default if not found', () => {
+            const args = ['-p', '/path'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-n', '--name'], 'default')).toBe('default');
+        });
+        it('should return undefined if not found and no default', () => {
+            const args = ['-p', '/path'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-n'])).toBeUndefined();
+        });
+        it('should prioritize first matching option', () => {
+            const args = ['-e', 'env1', '-e', 'env2'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-e'])).toBe('env1');
+        });
+        it('should handle value at end bounds', () => {
+            const args = ['command', '-t', 'value'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-t'])).toBe('value');
+        });
+        it('should return undefined if flag has no value', () => {
+            const args = ['-n'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-n'])).toBeUndefined();
+        });
+        it('should handle multiple aliases', () => {
+            const args = ['--environment', 'prod'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-e', '--environment'])).toBe('prod');
+        });
+        it('should find short option when long form not present', () => {
+            const args = ['-e', 'test'];
+            expect((0, argumentUtil_1.getArgValue)(args, ['-e', '--environment'])).toBe('test');
+        });
+    });
+    describe('complex argument patterns', () => {
+        it('should parse build command arguments', () => {
+            const args = ['build', '-p', '/path/to/project', '-t', '30000', '-v'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('/path/to/project');
+            expect((0, argumentUtil_1.getArg)(args, '-t')).toBe('30000');
+        });
+        it('should parse deploy command with environment', () => {
+            const args = ['deploy', '-p', '/path', '-e', 'prod'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('/path');
+            expect((0, argumentUtil_1.getArg)(args, '-e')).toBe('prod');
+        });
+        it('should parse init command with multiple options', () => {
+            const args = ['init', '-p', '/path', '-n', 'MyControl', '--publisher-name', 'Publisher'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('/path');
+            expect((0, argumentUtil_1.getArg)(args, '-n')).toBe('MyControl');
+            expect((0, argumentUtil_1.getArg)(args, '--publisher-name')).toBe('Publisher');
+        });
+        it('should handle boolean flags (no value)', () => {
+            const args = ['build', '--watch'];
+            expect((0, argumentUtil_1.getArg)(args, '-v')).toBeUndefined(); // -v flag has no value
+            expect((0, argumentUtil_1.getArg)(args, '--watch')).toBeUndefined();
+        });
+        it('should parse deprecated -env flag', () => {
+            const args = ['import', '-p', '/path', '-env', 'prod'];
+            expect((0, argumentUtil_1.getArg)(args, '-env')).toBe('prod');
+        });
+        it('should handle paths with special characters', () => {
+            const args = ['-p', 'C:\\Users\\test\\project', '-n', 'My-Control.v2'];
+            expect((0, argumentUtil_1.getArg)(args, '-p')).toBe('C:\\Users\\test\\project');
+            expect((0, argumentUtil_1.getArg)(args, '-n')).toBe('My-Control.v2');
+        });
+        it('should handle URLs as argument values', () => {
+            const args = ['-u', 'https://tenant.crm.dynamics.com', '-s', 'https://cdn.example.com/bundle.js'];
+            expect((0, argumentUtil_1.getArg)(args, '-u')).toBe('https://tenant.crm.dynamics.com');
+            expect((0, argumentUtil_1.getArg)(args, '-s')).toBe('https://cdn.example.com/bundle.js');
+        });
+    });
+});

package/dist/__tests__/pcf-helper-run.test.js

@@ -36,6 +36,20 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const child_process_1 = require("child_process");
 const package_json_1 = require("../package.json");
 const tasks = __importStar(require("@tywalk/pcf-helper"));
+beforeAll(() => {
+    var _a;
+    const npmExecPath = process.env.npm_execpath;
+    if (!npmExecPath) {
+        throw new Error('Unable to locate npm executable path from npm_execpath.');
+    }
+    const build = (0, child_process_1.spawnSync)(process.execPath, [npmExecPath, 'run', 'build'], {
+        cwd: process.cwd(),
+        encoding: 'utf-8'
+    });
+    if (build.status !== 0) {
+        throw new Error(`Failed to build pcf-helper-run before CLI tests.\n${((_a = build.error) === null || _a === void 0 ? void 0 : _a.message) || build.stderr || build.stdout}`);
+    }
+}, 60000);
 test('displays version', (done) => {
     const task = (0, child_process_1.spawn)('node', ['./dist/index.js', '--version']);
     let output = '';

package/dist/__tests__/performanceUtil.test.js

@@ -0,0 +1,90 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const performanceUtil_1 = require("../util/performanceUtil");
+describe('pcf-helper-run performanceUtil', () => {
+    describe('formatMsToSec', () => {
+        it('should format milliseconds to seconds', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Duration: %is', 5000);
+            expect(result).toBe('Duration: 5s');
+        });
+        it('should handle fractional seconds', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Time: %is', 1500);
+            expect(result).toBe('Time: 1.5s');
+        });
+        it('should handle small durations', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Quick: %is', 250);
+            expect(result).toBe('Quick: 0.25s');
+        });
+        it('should handle zero ms', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Instant: %is', 0);
+            expect(result).toBe('Instant: 0s');
+        });
+        it('should handle large durations', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Long: %is', 300000);
+            expect(result).toBe('Long: 300s');
+        });
+        it('should work with different format strings', () => {
+            const result = (0, performanceUtil_1.formatMsToSec)('Task %s took %is', 1000);
+            expect(result).toContain('1');
+        });
+    });
+    describe('formatTime', () => {
+        it('should format Date to time string', () => {
+            const date = new Date('2024-01-15T14:30:45Z');
+            const result = (0, performanceUtil_1.formatTime)(date);
+            expect(result).toMatch(/\d{2}:\d{2}:\d{2}/);
+        });
+        it('should use 24-hour format', () => {
+            // Create a date at specific time
+            const date = new Date('2024-01-15T23:45:30Z');
+            const result = (0, performanceUtil_1.formatTime)(date);
+            expect(result).toMatch(/\d{2}:\d{2}:\d{2}/);
+        });
+        it('should handle midnight', () => {
+            const date = new Date('2024-01-15T00:00:00Z');
+            const result = (0, performanceUtil_1.formatTime)(date);
+            expect(result).toMatch(/\d{2}:\d{2}:\d{2}/);
+        });
+        it('should format consistent for same date', () => {
+            const date = new Date('2024-06-21T12:30:45Z');
+            const result1 = (0, performanceUtil_1.formatTime)(date);
+            const result2 = (0, performanceUtil_1.formatTime)(date);
+            expect(result1).toBe(result2);
+        });
+        it('should format current time', () => {
+            const now = new Date();
+            const result = (0, performanceUtil_1.formatTime)(now);
+            expect(result).toMatch(/\d{2}:\d{2}:\d{2}/);
+        });
+        it('should include leading zeros for single digit times', () => {
+            const date = new Date('2024-01-15T01:02:03Z');
+            const result = (0, performanceUtil_1.formatTime)(date);
+            expect(result).toMatch(/\d{2}:\d{2}:\d{2}/); // Always 2 digits
+        });
+    });
+    describe('time formatting consistency', () => {
+        it('should format times consistently across calls', () => {
+            const times = [];
+            for (let i = 0; i < 5; i++) {
+                const date = new Date('2024-06-21T15:30:45Z');
+                times.push((0, performanceUtil_1.formatTime)(date));
+            }
+            // All should be identical
+            expect(new Set(times).size).toBe(1);
+        });
+        it('should format duration calculations correctly', () => {
+            const start = 1000;
+            const end = 6000;
+            const duration = end - start;
+            const result = (0, performanceUtil_1.formatMsToSec)('Duration: %is', duration);
+            expect(result).toBe('Duration: 5s');
+        });
+        it('should handle combined time and duration formatting', () => {
+            const date = new Date('2024-06-21T12:30:45Z');
+            const timeStr = (0, performanceUtil_1.formatTime)(date);
+            const durationStr = (0, performanceUtil_1.formatMsToSec)('in %is', 2500);
+            expect(timeStr).toMatch(/\d{2}:\d{2}:\d{2}/);
+            expect(durationStr).toBe('in 2.5s');
+        });
+    });
+});

package/dist/index.js

@@ -287,6 +287,7 @@ withCommonOptions(commander_1.program.command('session'))
         logger.log('[PCF Helper Run] ' + (0, performanceUtil_1.formatTime)(new Date()) + ' session started.\n');
         if (!options.url || options.config) {
             const config = tasks.loadConfig(options.config || 'session.config.json');
+            // Use the config values from the config file, falling back to the CLI options if the config values are not set
             yield tasks.runSession((_a = config.remoteEnvironmentUrl) !== null && _a !== void 0 ? _a : options.url, (_b = config.remoteScriptToIntercept) !== null && _b !== void 0 ? _b : options.script, (_c = config.remoteStylesheetToIntercept) !== null && _c !== void 0 ? _c : options.stylesheet, (_d = config.localBundlePath) !== null && _d !== void 0 ? _d : options.bundle, (_e = config.localCssPath) !== null && _e !== void 0 ? _e : options.css, config.startWatch || options.watch);
         }
         else {

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@tywalk/pcf-helper-run",
-    "version": "1.2.16",
+    "version": "1.3.1",
     "description": "Unified CLI interface for Power Platform Component Framework (PCF) development — init, build, import, deploy, and manage PCF controls in Dataverse.",
     "main": "dist/index.js",
     "types": "./types/",
@@ -52,7 +52,7 @@
     },
     "dependencies": {
         "@tywalk/color-logger": "^1.0.3",
-        "@tywalk/pcf-helper": "^1.9.0"
+        "@tywalk/pcf-helper": "^1.9.2"
     },
     "devDependencies": {
         "@semantic-release/git": "^10.0.1",

package/dist/util/argumentUtil.js

@@ -13,7 +13,7 @@ function getArgValue(args, argOpts, defaultIfExists) {
     var _a;
     const arg = args.find(a => argOpts.includes(a));
     if (typeof arg === 'undefined') {
-        return undefined;
+        return defaultIfExists;
     }
     const argIndex = args.indexOf(arg) + 1;
     return (_a = args.at(argIndex)) !== null && _a !== void 0 ? _a : defaultIfExists;

package/dist/util/performanceUtil.js

@@ -22,7 +22,9 @@ const formatter = new Intl.DateTimeFormat('en-US', {
  */
 function formatMsToSec(format, ms) {
     const seconds = ms / 1000;
-    return util_1.default.format(format, seconds);
+    // %i truncates floats in util.format, so normalize to %f to preserve precision.
+    const normalizedFormat = format.replace(/%i/g, '%f');
+    return util_1.default.format(normalizedFormat, seconds);
 }
 /**
  * Formats a Date object into a human-readable string.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tywalk/pcf-helper-run",
-  "version": "1.3.0",
+  "version": "1.3.2",
   "description": "Unified CLI interface for Power Platform Component Framework (PCF) development — init, build, import, deploy, and manage PCF controls in Dataverse.",
   "main": "dist/index.js",
   "types": "./types/",
@@ -52,7 +52,7 @@
   },
   "dependencies": {
     "@tywalk/color-logger": "^1.0.3",
-    "@tywalk/pcf-helper": "^1.9.0"
+    "@tywalk/pcf-helper": "^1.9.2"
   },
   "devDependencies": {
     "@semantic-release/git": "^10.0.1",

package/types/__tests__/argumentUtil.test.d.ts

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

package/types/__tests__/performanceUtil.test.d.ts

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