feat(transformer): add sync transformer

.eslintrc.js

@@ -29,6 +29,7 @@ module.exports = {
       rules: {
         semi: ['error', 'always'],
         quotes: ['error', 'single'],
+        'new-cap': 'off',
         'object-curly-spacing': ['error', 'always'],
         'array-bracket-spacing': 'off',
         'unicorn/filename-case': 'off',

packages/core/lib/bundler/helpers/internal.ts

@@ -17,7 +17,7 @@ export const getTransformedPreludeScript = async (
    * Remove `"use strict";` added by sucrase.
    * @see {@link https://github.com/alangpierce/sucrase/issues/787#issuecomment-1483934492}
    */
-  const strippedScript = (await stripFlowWithSucrase(preludeScript, context))
+  const strippedScript = stripFlowWithSucrase(preludeScript, context)
     .replace(/"use strict";/, '')
     .trim();
 

packages/plugins/lib/reactNativeRuntimeTransformPlugin/reactNativeRuntimeTransformPlugin.ts

@@ -7,8 +7,8 @@ import {
 } from '@react-native-esbuild/core';
 import { getReactNativeInitializeCore } from '@react-native-esbuild/internal';
 import {
-  TransformPipeline,
-  type TransformStep,
+  AsyncTransformPipeline,
+  type AsyncTransformStep,
 } from '@react-native-esbuild/transformer';
 import { logger } from '../shared';
 import type { ReactNativeRuntimeTransformPluginConfig } from '../types';
@@ -42,7 +42,11 @@ export const createReactNativeRuntimeTransformPlugin: ReactNativeEsbuildPluginCr
       ...(config?.injectScriptPaths ?? []),
     ];
 
-    const onBeforeTransform: TransformStep = async (code, args, sharedData) => {
+    const onBeforeTransform: AsyncTransformStep = async (
+      code,
+      args,
+      sharedData,
+    ) => {
       const isChangedFile = bundlerSharedData.watcher.changed === args.path;
       const cacheConfig = await makeCacheConfig(
         cacheController,
@@ -87,7 +91,11 @@ export const createReactNativeRuntimeTransformPlugin: ReactNativeEsbuildPluginCr
       return { code: cachedCode ?? code, done: Boolean(cachedCode) };
     };
 
-    const onAfterTransform: TransformStep = async (code, _args, shared) => {
+    const onAfterTransform: AsyncTransformStep = async (
+      code,
+      _args,
+      shared,
+    ) => {
       if (!(shared.hash && shared.mtimeMs)) {
         logger.warn('unexpected cache config');
         return { code, done: true };
@@ -108,9 +116,8 @@ export const createReactNativeRuntimeTransformPlugin: ReactNativeEsbuildPluginCr
       return { code, done: true };
     };
 
-    let transformPipeline: TransformPipeline;
-    // eslint-disable-next-line new-cap -- builder
-    const transformPipelineBuilder = new TransformPipeline.builder(
+    let transformPipeline: AsyncTransformPipeline;
+    const transformPipelineBuilder = new AsyncTransformPipeline.builder(
       context.root,
       context.entry,
     )
@@ -127,8 +134,9 @@ export const createReactNativeRuntimeTransformPlugin: ReactNativeEsbuildPluginCr
     });
 
     build.onLoad({ filter: /\.(?:[mc]js|[tj]sx?)$/ }, async (args) => {
+      const rawCode = await fs.readFile(args.path, { encoding: 'utf-8' });
       return {
-        contents: await transformPipeline.transform(args),
+        contents: (await transformPipeline.transform(rawCode, args)).code,
         loader: 'js',
       } as OnLoadResult;
     });

packages/transformer/lib/helpers/index.ts

@@ -1 +1 @@
-export * from './transformByRule';
+export * from './transformer';

packages/transformer/lib/helpers/transformByRule.ts → packages/transformer/lib/helpers/transformer.ts

@@ -1,4 +1,9 @@
-import { transformWithBabel, transformWithSwc } from '../transform';
+import {
+  transformWithBabel,
+  transformSyncWithBabel,
+  transformWithSwc,
+  transformSyncWithSwc,
+} from '../transform';
 import type {
   TransformRuleBase,
   TransformerContext,
@@ -14,26 +19,50 @@ const getOptions = <T>(
   return options instanceof Function ? options(context.path, code) : options;
 };
 
-export const transformBySwcRule = (
-  rule: SwcTransformRule,
+export const transformByBabelRule = (
+  rule: BabelTransformRule,
   code: string,
   context: TransformerContext,
 ): Promise<string | null> => {
   return rule.test(context.path, code)
-    ? transformWithSwc(code, context, {
+    ? transformWithBabel(code, context, {
         customOptions: getOptions(rule.options, code, context),
       })
     : Promise.resolve(null);
 };
 
-export const transformByBabelRule = (
+export const transformSyncByBabelRule = (
   rule: BabelTransformRule,
   code: string,
   context: TransformerContext,
+): string | null => {
+  return rule.test(context.path, code)
+    ? transformSyncWithBabel(code, context, {
+        customOptions: getOptions(rule.options, code, context),
+      })
+    : null;
+};
+
+export const transformBySwcRule = (
+  rule: SwcTransformRule,
+  code: string,
+  context: TransformerContext,
 ): Promise<string | null> => {
   return rule.test(context.path, code)
-    ? transformWithBabel(code, context, {
+    ? transformWithSwc(code, context, {
         customOptions: getOptions(rule.options, code, context),
       })
     : Promise.resolve(null);
 };
+
+export const transformSyncBySwcRule = (
+  rule: SwcTransformRule,
+  code: string,
+  context: TransformerContext,
+): string | null => {
+  return rule.test(context.path, code)
+    ? transformSyncWithSwc(code, context, {
+        customOptions: getOptions(rule.options, code, context),
+      })
+    : null;
+};

packages/transformer/lib/index.ts

@@ -1,4 +1,4 @@
 export * from './transform';
+export * from './pipelines';
 export * from './helpers';
-export { TransformPipeline } from './TransformPipelineBuilder';
 export type * from './types';