@output.ai/core 0.0.8 → 0.0.9

package/src/worker/webpack_loaders/workflow_rewriter/collect_target_imports.js

@@ -42,7 +42,9 @@ export default function collectTargetImports( ast, fileDir, { stepsNameCache, wo
     ImportDeclaration: path => {
       const src = path.node.source.value;
       // Ignore other imports
-      if ( !isStepsPath( src ) && !isWorkflowPath( src ) ) { return; }
+      if ( !isStepsPath( src ) && !isWorkflowPath( src ) ) {
+        return;
+      }
 
       const absolutePath = toAbsolutePath( fileDir, src );
       if ( isStepsPath( src ) ) {
@@ -51,7 +53,9 @@ export default function collectTargetImports( ast, fileDir, { stepsNameCache, wo
           const importedName = s.imported.name;
           const localName = s.local.name;
           const stepName = nameMap.get( importedName );
-          if ( stepName ) { stepImports.push( { localName, stepName } ); }
+          if ( stepName ) {
+            stepImports.push( { localName, stepName } );
+          }
         }
       }
       if ( isWorkflowPath( src ) ) {
@@ -64,7 +68,9 @@ export default function collectTargetImports( ast, fileDir, { stepsNameCache, wo
             const importedName = s.imported.name;
             const localName = s.local.name;
             const workflowName = named.get( importedName );
-            if ( workflowName ) { flowImports.push( { localName, workflowName } ); }
+            if ( workflowName ) {
+              flowImports.push( { localName, workflowName } );
+            }
           }
         }
       }
@@ -73,16 +79,24 @@ export default function collectTargetImports( ast, fileDir, { stepsNameCache, wo
     VariableDeclarator: path => {
       const init = path.node.init;
       // Not a require call
-      if ( !isCallExpression( init ) ) { return; }
+      if ( !isCallExpression( init ) ) {
+        return;
+      }
       // Different callee
-      if ( !isIdentifier( init.callee, { name: 'require' } ) ) { return; }
+      if ( !isIdentifier( init.callee, { name: 'require' } ) ) {
+        return;
+      }
       const firstArgument = init.arguments[0];
       // Dynamic require is not supported
-      if ( !isStringLiteral( firstArgument ) ) { return; }
+      if ( !isStringLiteral( firstArgument ) ) {
+        return;
+      }
 
       const req = firstArgument.value;
       // Must be steps/workflows module
-      if ( !isStepsPath( req ) && !isWorkflowPath( req ) ) { return; }
+      if ( !isStepsPath( req ) && !isWorkflowPath( req ) ) {
+        return;
+      }
 
       const absolutePath = toAbsolutePath( fileDir, req );
       if ( isStepsPath( req ) && isObjectPattern( path.node.id ) ) {
@@ -92,7 +106,9 @@ export default function collectTargetImports( ast, fileDir, { stepsNameCache, wo
           const localName = getLocalNameFromDestructuredProperty( prop );
           if ( localName ) {
             const stepName = nameMap.get( importedName );
-            if ( stepName ) { stepImports.push( { localName, stepName } ); }
+            if ( stepName ) {
+              stepImports.push( { localName, stepName } );
+            }
           }
         }
         if ( isVariableDeclaration( path.parent ) && path.parent.declarations.length === 1 ) {

package/src/worker/webpack_loaders/workflow_rewriter/index.mjs

@@ -40,7 +40,9 @@ export default function stepImportRewriterAstLoader( source, inputMap ) {
 
     const rewrote = rewriteFnBodies( ast, stepImports, flowImports );
     // No edits performed
-    if ( !rewrote ) { return callback( null, source, inputMap ); }
+    if ( !rewrote ) {
+      return callback( null, source, inputMap );
+    }
 
     const { code, map } = generate( ast, {
       sourceMaps: true,

package/src/worker/webpack_loaders/workflow_rewriter/rewrite_fn_bodies.js

@@ -25,12 +25,16 @@ export default function rewriteFnBodies( ast, stepImports, flowImports ) {
   traverse( ast, {
     ObjectProperty: path => {
       // If it is not fn property: skip
-      if ( !isIdentifier( path.node.key, { name: 'fn' } ) ) { return; }
+      if ( !isIdentifier( path.node.key, { name: 'fn' } ) ) {
+        return;
+      }
 
       const val = path.node.value;
 
       // if it is not function, return
-      if ( !isFunctionExpression( val ) && !isArrowFunctionExpression( val ) ) { return; }
+      if ( !isFunctionExpression( val ) && !isArrowFunctionExpression( val ) ) {
+        return;
+      }
 
       // replace arrow fn in favor of a function
       if ( isArrowFunctionExpression( val ) ) {
@@ -42,7 +46,9 @@ export default function rewriteFnBodies( ast, stepImports, flowImports ) {
       path.get( 'value.body' ).traverse( {
         CallExpression: cPath => {
           const callee = cPath.node.callee;
-          if ( !isIdentifier( callee ) ) { return; } // Skip: complex callee not supported
+          if ( !isIdentifier( callee ) ) {
+            return;
+          } // Skip: complex callee not supported
           const step = stepImports.find( x => x.localName === callee.name );
           if ( step ) {
             const args = cPath.node.arguments;

package/src/worker/webpack_loaders/workflow_rewriter/tools.js

@@ -63,8 +63,12 @@ export const extractTopLevelStringConsts = ast =>
  * @returns {string|null} Key name or null when unsupported.
  */
 export const getObjectKeyName = node => {
-  if ( isIdentifier( node ) ) { return node.name; }
-  if ( isStringLiteral( node ) ) { return node.value; }
+  if ( isIdentifier( node ) ) {
+    return node.name;
+  }
+  if ( isStringLiteral( node ) ) {
+    return node.value;
+  }
   return null;
 };
 
@@ -75,8 +79,12 @@ export const getObjectKeyName = node => {
  * @returns {string|null} Local identifier name or null.
  */
 export const getLocalNameFromDestructuredProperty = prop => {
-  if ( isIdentifier( prop.value ) ) { return prop.value.name; }
-  if ( isAssignmentPattern( prop.value ) && isIdentifier( prop.value.left ) ) { return prop.value.left.name; }
+  if ( isIdentifier( prop.value ) ) {
+    return prop.value.name;
+  }
+  if ( isAssignmentPattern( prop.value ) && isIdentifier( prop.value.left ) ) {
+    return prop.value.left.name;
+  }
   return null;
 };
 
@@ -132,15 +140,21 @@ export const resolveNameFromOptions = ( optionsNode, consts, errorMessagePrefix
 
   // Look specifically for the 'name' property
   for ( const prop of optionsNode.properties ) {
-    if ( getObjectKeyName( prop.key ) !== 'name' ) { continue; }
+    if ( getObjectKeyName( prop.key ) !== 'name' ) {
+      continue;
+    }
 
     const val = prop.value;
     // if it is a string literal: jackpot
-    if ( isStringLiteral( val ) ) { return val.value; }
+    if ( isStringLiteral( val ) ) {
+      return val.value;
+    }
 
     // if it is an identifier, it needs to be deterministic (top-level const)
     if ( isIdentifier( val ) ) {
-      if ( consts.has( val.name ) ) { return consts.get( val.name ); }
+      if ( consts.has( val.name ) ) {
+        return consts.get( val.name );
+      }
       throw new Error( `${errorMessagePrefix}: Name identifier "${val.name}" is not a top-level const string` );
     }
 
@@ -159,7 +173,9 @@ export const resolveNameFromOptions = ( optionsNode, consts, errorMessagePrefix
  * @throws {Error} When a step name is invalid (non-static or missing).
  */
 export const buildStepsNameMap = ( path, cache ) => {
-  if ( cache.has( path ) ) { return cache.get( path ); }
+  if ( cache.has( path ) ) {
+    return cache.get( path );
+  }
   const text = readFileSync( path, 'utf8' );
   const ast = parse( text, path );
   const consts = extractTopLevelStringConsts( ast );
@@ -191,7 +207,9 @@ export const buildStepsNameMap = ( path, cache ) => {
  * @throws {Error} When a workflow name is invalid (non-static or missing).
  */
 export const buildWorkflowNameMap = ( path, cache ) => {
-  if ( cache.has( path ) ) { return cache.get( path ); }
+  if ( cache.has( path ) ) {
+    return cache.get( path );
+  }
   const text = readFileSync( path, 'utf8' );
   const ast = parse( text, path );
   const consts = extractTopLevelStringConsts( ast );
@@ -206,7 +224,9 @@ export const buildWorkflowNameMap = ( path, cache ) => {
       for ( const d of node.declaration.declarations ) {
         if ( isIdentifier( d.id ) && isCallExpression( d.init ) && isIdentifier( d.init.callee, { name: 'workflow' } ) ) {
           const name = resolveNameFromOptions( d.init.arguments[0], consts, `Invalid workflow name in ${path} for '${d.id.name}` );
-          if ( name ) { result.named.set( d.id.name, name ); }
+          if ( name ) {
+            result.named.set( d.id.name, name );
+          }
         }
       }