slicejs-cli 3.2.0 → 3.4.0

package/tests/client-update-flow-contract.test.js

@@ -1,272 +1,272 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { parse } from '@babel/parser';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const clientPath = path.join(__dirname, '..', 'client.js');
-const clientSource = fs.readFileSync(clientPath, 'utf-8');
-const ast = parse(clientSource, {
-  sourceType: 'module',
-  errorRecovery: true,
-  loc: true,
-  ranges: true
-});
-
-function lineOf(node) {
-  return node && node.loc && node.loc.start ? node.loc.start.line : null;
-}
-
-function walk(node, visit) {
-  if (!node || typeof node !== 'object') {
-    return;
-  }
-
-  visit(node);
-
-  for (const value of Object.values(node)) {
-    if (Array.isArray(value)) {
-      for (const item of value) {
-        walk(item, visit);
-      }
-      continue;
-    }
-
-    walk(value, visit);
-  }
-}
-
-function isMethodCall(node, methodName) {
-  return (
-    node &&
-    node.type === 'CallExpression' &&
-    node.callee &&
-    node.callee.type === 'MemberExpression' &&
-    node.callee.property &&
-    node.callee.property.type === 'Identifier' &&
-    node.callee.property.name === methodName
-  );
-}
-
-function isCommandUpdateExpression(node) {
-  if (!node || node.type !== 'CallExpression') {
-    return false;
-  }
-
-  if (
-    node.callee &&
-    node.callee.type === 'MemberExpression' &&
-    node.callee.property &&
-    node.callee.property.type === 'Identifier' &&
-    node.callee.property.name === 'command' &&
-    node.arguments[0] &&
-    node.arguments[0].type === 'StringLiteral' &&
-    node.arguments[0].value === 'update'
-  ) {
-    return true;
-  }
-
-  if (node.callee && node.callee.type === 'MemberExpression') {
-    return isCommandUpdateExpression(node.callee.object);
-  }
-
-  return false;
-}
-
-function getRunWithVersionCheckNode() {
-  let foundNode = null;
-
-  walk(ast, (node) => {
-    if (
-      node.type === 'FunctionDeclaration' &&
-      node.id &&
-      node.id.type === 'Identifier' &&
-      node.id.name === 'runWithVersionCheck'
-    ) {
-      foundNode = node;
-      return;
-    }
-
-    if (node.type !== 'VariableDeclarator') {
-      return;
-    }
-
-    if (!node.id || node.id.type !== 'Identifier' || node.id.name !== 'runWithVersionCheck') {
-      return;
-    }
-
-    if (
-      node.init &&
-      (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression')
-    ) {
-      foundNode = node.init;
-    }
-  });
-
-  return foundNode;
-}
-
-function getFunctionBodyNode(fnNode) {
-  if (!fnNode) {
-    return null;
-  }
-
-  if (fnNode.type === 'FunctionDeclaration' || fnNode.type === 'FunctionExpression' || fnNode.type === 'ArrowFunctionExpression') {
-    return fnNode.body;
-  }
-
-  return null;
-}
-
-function getFunctionLikeByName(name) {
-  let foundNode = null;
-
-  walk(ast, (node) => {
-    if (
-      node.type === 'FunctionDeclaration' &&
-      node.id &&
-      node.id.type === 'Identifier' &&
-      node.id.name === name
-    ) {
-      foundNode = node;
-      return;
-    }
-
-    if (node.type !== 'VariableDeclarator') {
-      return;
-    }
-
-    if (!node.id || node.id.type !== 'Identifier' || node.id.name !== name) {
-      return;
-    }
-
-    if (
-      node.init &&
-      (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression')
-    ) {
-      foundNode = node.init;
-    }
-  });
-
-  return foundNode;
-}
-
-test('runWithVersionCheck uses non-blocking update notifications', () => {
-  const runWithVersionCheckNode = getRunWithVersionCheckNode();
-  const runWithVersionCheckBody = getFunctionBodyNode(runWithVersionCheckNode);
-
-  assert.ok(
-    runWithVersionCheckNode,
-    'runWithVersionCheck function should exist as a declaration or function-valued variable'
-  );
-  assert.ok(runWithVersionCheckBody, 'runWithVersionCheck should have a traversable function body');
-
-  let hasNotifyCall = false;
-  let hasAwaitedNotifyCall = false;
-  let notifyCallLine = null;
-  let hasPromptCall = false;
-  let promptCallLine = null;
-
-  walk(runWithVersionCheckBody, (node) => {
-    if (isMethodCall(node, 'notifyAvailableUpdates')) {
-      hasNotifyCall = true;
-      notifyCallLine = notifyCallLine ?? lineOf(node);
-    }
-    if (
-      node.type === 'AwaitExpression' &&
-      isMethodCall(node.argument, 'notifyAvailableUpdates')
-    ) {
-      hasAwaitedNotifyCall = true;
-    }
-    if (isMethodCall(node, 'checkAndPromptUpdates')) {
-      hasPromptCall = true;
-      promptCallLine = promptCallLine ?? lineOf(node);
-    }
-  });
-
-  assert.equal(
-    hasNotifyCall,
-    true,
-    `runWithVersionCheck must call updateManager.notifyAvailableUpdates() (found at line ${notifyCallLine ?? 'unknown'})`
-  );
-  assert.equal(
-    hasAwaitedNotifyCall,
-    false,
-    `runWithVersionCheck should fire-and-forget notifyAvailableUpdates() without await (notify call line ${notifyCallLine ?? 'unknown'})`
-  );
-  assert.equal(
-    hasPromptCall,
-    false,
-    `runWithVersionCheck must not call updateManager.checkAndPromptUpdates() (found at line ${promptCallLine ?? 'unknown'})`
-  );
-});
-
-test('update command remains explicitly interactive', () => {
-  let foundAwaitedInteractiveUpdateAction = false;
-  let updateActionHandlerLine = null;
-  let relatedPromptCallLine = null;
-  let handlerReferenceName = null;
-
-  walk(ast, (node) => {
-    if (
-      node.type === 'CallExpression' &&
-      node.callee &&
-      node.callee.type === 'MemberExpression' &&
-      node.callee.property &&
-      node.callee.property.type === 'Identifier' &&
-      node.callee.property.name === 'action' &&
-      isCommandUpdateExpression(node.callee.object)
-    ) {
-      const actionHandler = node.arguments[0];
-      if (!actionHandler) {
-        return;
-      }
-
-      let resolvedHandler = null;
-
-      if (actionHandler.type === 'Identifier') {
-        handlerReferenceName = actionHandler.name;
-        resolvedHandler = getFunctionLikeByName(actionHandler.name);
-      } else if (
-        actionHandler.type === 'ArrowFunctionExpression' ||
-        actionHandler.type === 'FunctionExpression'
-      ) {
-        resolvedHandler = actionHandler;
-      }
-
-      if (!resolvedHandler || resolvedHandler.async !== true) {
-        return;
-      }
-
-      const resolvedBody = getFunctionBodyNode(resolvedHandler);
-      if (!resolvedBody) {
-        return;
-      }
-
-      updateActionHandlerLine = updateActionHandlerLine ?? lineOf(resolvedHandler);
-
-      walk(resolvedBody, (actionNode) => {
-        if (isMethodCall(actionNode, 'checkAndPromptUpdates')) {
-          relatedPromptCallLine = relatedPromptCallLine ?? lineOf(actionNode);
-        }
-
-        if (
-          actionNode.type !== 'AwaitExpression' ||
-          !isMethodCall(actionNode.argument, 'checkAndPromptUpdates')
-        ) {
-          return;
-        }
-
-        foundAwaitedInteractiveUpdateAction = true;
-      });
-    }
-  });
-
-  assert.equal(
-    foundAwaitedInteractiveUpdateAction,
-    true,
-    `update command action must await checkAndPromptUpdates(...) (action line ${updateActionHandlerLine ?? 'unknown'}, related call line ${relatedPromptCallLine ?? 'unknown'}${handlerReferenceName ? `, handler reference ${handlerReferenceName}` : ''})`
-  );
-});
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { parse } from '@babel/parser';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const clientPath = path.join(__dirname, '..', 'client.js');
+const clientSource = fs.readFileSync(clientPath, 'utf-8');
+const ast = parse(clientSource, {
+  sourceType: 'module',
+  errorRecovery: true,
+  loc: true,
+  ranges: true
+});
+
+function lineOf(node) {
+  return node && node.loc && node.loc.start ? node.loc.start.line : null;
+}
+
+function walk(node, visit) {
+  if (!node || typeof node !== 'object') {
+    return;
+  }
+
+  visit(node);
+
+  for (const value of Object.values(node)) {
+    if (Array.isArray(value)) {
+      for (const item of value) {
+        walk(item, visit);
+      }
+      continue;
+    }
+
+    walk(value, visit);
+  }
+}
+
+function isMethodCall(node, methodName) {
+  return (
+    node &&
+    node.type === 'CallExpression' &&
+    node.callee &&
+    node.callee.type === 'MemberExpression' &&
+    node.callee.property &&
+    node.callee.property.type === 'Identifier' &&
+    node.callee.property.name === methodName
+  );
+}
+
+function isCommandUpdateExpression(node) {
+  if (!node || node.type !== 'CallExpression') {
+    return false;
+  }
+
+  if (
+    node.callee &&
+    node.callee.type === 'MemberExpression' &&
+    node.callee.property &&
+    node.callee.property.type === 'Identifier' &&
+    node.callee.property.name === 'command' &&
+    node.arguments[0] &&
+    node.arguments[0].type === 'StringLiteral' &&
+    node.arguments[0].value === 'update'
+  ) {
+    return true;
+  }
+
+  if (node.callee && node.callee.type === 'MemberExpression') {
+    return isCommandUpdateExpression(node.callee.object);
+  }
+
+  return false;
+}
+
+function getRunWithVersionCheckNode() {
+  let foundNode = null;
+
+  walk(ast, (node) => {
+    if (
+      node.type === 'FunctionDeclaration' &&
+      node.id &&
+      node.id.type === 'Identifier' &&
+      node.id.name === 'runWithVersionCheck'
+    ) {
+      foundNode = node;
+      return;
+    }
+
+    if (node.type !== 'VariableDeclarator') {
+      return;
+    }
+
+    if (!node.id || node.id.type !== 'Identifier' || node.id.name !== 'runWithVersionCheck') {
+      return;
+    }
+
+    if (
+      node.init &&
+      (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression')
+    ) {
+      foundNode = node.init;
+    }
+  });
+
+  return foundNode;
+}
+
+function getFunctionBodyNode(fnNode) {
+  if (!fnNode) {
+    return null;
+  }
+
+  if (fnNode.type === 'FunctionDeclaration' || fnNode.type === 'FunctionExpression' || fnNode.type === 'ArrowFunctionExpression') {
+    return fnNode.body;
+  }
+
+  return null;
+}
+
+function getFunctionLikeByName(name) {
+  let foundNode = null;
+
+  walk(ast, (node) => {
+    if (
+      node.type === 'FunctionDeclaration' &&
+      node.id &&
+      node.id.type === 'Identifier' &&
+      node.id.name === name
+    ) {
+      foundNode = node;
+      return;
+    }
+
+    if (node.type !== 'VariableDeclarator') {
+      return;
+    }
+
+    if (!node.id || node.id.type !== 'Identifier' || node.id.name !== name) {
+      return;
+    }
+
+    if (
+      node.init &&
+      (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression')
+    ) {
+      foundNode = node.init;
+    }
+  });
+
+  return foundNode;
+}
+
+test('runWithVersionCheck uses non-blocking update notifications', () => {
+  const runWithVersionCheckNode = getRunWithVersionCheckNode();
+  const runWithVersionCheckBody = getFunctionBodyNode(runWithVersionCheckNode);
+
+  assert.ok(
+    runWithVersionCheckNode,
+    'runWithVersionCheck function should exist as a declaration or function-valued variable'
+  );
+  assert.ok(runWithVersionCheckBody, 'runWithVersionCheck should have a traversable function body');
+
+  let hasNotifyCall = false;
+  let hasAwaitedNotifyCall = false;
+  let notifyCallLine = null;
+  let hasPromptCall = false;
+  let promptCallLine = null;
+
+  walk(runWithVersionCheckBody, (node) => {
+    if (isMethodCall(node, 'notifyAvailableUpdates')) {
+      hasNotifyCall = true;
+      notifyCallLine = notifyCallLine ?? lineOf(node);
+    }
+    if (
+      node.type === 'AwaitExpression' &&
+      isMethodCall(node.argument, 'notifyAvailableUpdates')
+    ) {
+      hasAwaitedNotifyCall = true;
+    }
+    if (isMethodCall(node, 'checkAndPromptUpdates')) {
+      hasPromptCall = true;
+      promptCallLine = promptCallLine ?? lineOf(node);
+    }
+  });
+
+  assert.equal(
+    hasNotifyCall,
+    true,
+    `runWithVersionCheck must call updateManager.notifyAvailableUpdates() (found at line ${notifyCallLine ?? 'unknown'})`
+  );
+  assert.equal(
+    hasAwaitedNotifyCall,
+    false,
+    `runWithVersionCheck should fire-and-forget notifyAvailableUpdates() without await (notify call line ${notifyCallLine ?? 'unknown'})`
+  );
+  assert.equal(
+    hasPromptCall,
+    false,
+    `runWithVersionCheck must not call updateManager.checkAndPromptUpdates() (found at line ${promptCallLine ?? 'unknown'})`
+  );
+});
+
+test('update command remains explicitly interactive', () => {
+  let foundAwaitedInteractiveUpdateAction = false;
+  let updateActionHandlerLine = null;
+  let relatedPromptCallLine = null;
+  let handlerReferenceName = null;
+
+  walk(ast, (node) => {
+    if (
+      node.type === 'CallExpression' &&
+      node.callee &&
+      node.callee.type === 'MemberExpression' &&
+      node.callee.property &&
+      node.callee.property.type === 'Identifier' &&
+      node.callee.property.name === 'action' &&
+      isCommandUpdateExpression(node.callee.object)
+    ) {
+      const actionHandler = node.arguments[0];
+      if (!actionHandler) {
+        return;
+      }
+
+      let resolvedHandler = null;
+
+      if (actionHandler.type === 'Identifier') {
+        handlerReferenceName = actionHandler.name;
+        resolvedHandler = getFunctionLikeByName(actionHandler.name);
+      } else if (
+        actionHandler.type === 'ArrowFunctionExpression' ||
+        actionHandler.type === 'FunctionExpression'
+      ) {
+        resolvedHandler = actionHandler;
+      }
+
+      if (!resolvedHandler || resolvedHandler.async !== true) {
+        return;
+      }
+
+      const resolvedBody = getFunctionBodyNode(resolvedHandler);
+      if (!resolvedBody) {
+        return;
+      }
+
+      updateActionHandlerLine = updateActionHandlerLine ?? lineOf(resolvedHandler);
+
+      walk(resolvedBody, (actionNode) => {
+        if (isMethodCall(actionNode, 'checkAndPromptUpdates')) {
+          relatedPromptCallLine = relatedPromptCallLine ?? lineOf(actionNode);
+        }
+
+        if (
+          actionNode.type !== 'AwaitExpression' ||
+          !isMethodCall(actionNode.argument, 'checkAndPromptUpdates')
+        ) {
+          return;
+        }
+
+        foundAwaitedInteractiveUpdateAction = true;
+      });
+    }
+  });
+
+  assert.equal(
+    foundAwaitedInteractiveUpdateAction,
+    true,
+    `update command action must await checkAndPromptUpdates(...) (action line ${updateActionHandlerLine ?? 'unknown'}, related call line ${relatedPromptCallLine ?? 'unknown'}${handlerReferenceName ? `, handler reference ${handlerReferenceName}` : ''})`
+  );
+});

package/tests/dependency-analyzer.test.js

@@ -1,24 +1,24 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import DependencyAnalyzer from '../commands/utils/bundling/DependencyAnalyzer.js';
-
-test('parseComponentsConfig returns components map', () => {
-  // Arrange
-  const analyzer = new DependencyAnalyzer(import.meta.url);
-  const content = `const components = {"Button": "Visual", "FetchManager": "Service"};\nexport default components;`;
-
-  // Act
-  const result = analyzer.parseComponentsConfig(content);
-
-  // Assert
-  assert.deepEqual(result, { Button: 'Visual', FetchManager: 'Service' });
-});
-
-test('parseComponentsConfig throws for invalid config', () => {
-  // Arrange
-  const analyzer = new DependencyAnalyzer(import.meta.url);
-  const content = `const notComponents = {"Button": "Visual"};`;
-
-  // Act / Assert
-  assert.throws(() => analyzer.parseComponentsConfig(content), /components object not found/);
-});
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import DependencyAnalyzer from '../commands/utils/bundling/DependencyAnalyzer.js';
+
+test('parseComponentsConfig returns components map', () => {
+  // Arrange
+  const analyzer = new DependencyAnalyzer(import.meta.url);
+  const content = `const components = {"Button": "Visual", "FetchManager": "Service"};\nexport default components;`;
+
+  // Act
+  const result = analyzer.parseComponentsConfig(content);
+
+  // Assert
+  assert.deepEqual(result, { Button: 'Visual', FetchManager: 'Service' });
+});
+
+test('parseComponentsConfig throws for invalid config', () => {
+  // Arrange
+  const analyzer = new DependencyAnalyzer(import.meta.url);
+  const content = `const notComponents = {"Button": "Visual"};`;
+
+  // Act / Assert
+  assert.throws(() => analyzer.parseComponentsConfig(content), /components object not found/);
+});