@wular/pnext 0.0.4 → 0.0.6

package/src/compat/bundler/modularize-imports.ts

@@ -13,17 +13,17 @@
 // resolution. Namespace, default and mixed-default imports of the specifier are left untouched - Next
 // bails on those too.
 
-import { rewriteFacts, type ImportBinding } from '../../resolve/scan-facts';
-import { spliceSource } from '../../dev/module-transform';
-import { withSniff, type ServerSourceTransform } from '../../extensions';
-import { specifierSniffTokens } from '../../utils/source';
+import { rewriteFacts, type ImportBinding } from '../../resolve/scan-facts'
+import { spliceSource } from '../../runtime/module-transform'
+import { withSniff, type ServerSourceTransform } from '../../extensions'
+import { specifierSniffTokens } from '../../utils/code'
 
 /** A normalized `modularizeImports[specifier]` entry (string-template form). */
 export interface ModularizeRule {
   /** The output-path template, e.g. `'#/design-system/icons/{{ kebabCase member }}'`. */
-  transform: string;
+  transform: string
   /** `true` => named import of the member from the target; otherwise a default import. */
-  skipDefaultConversion?: boolean;
+  skipDefaultConversion?: boolean
 }
 
 /** Split an identifier into words on camelCase / digit / `_` / `-` boundaries. */
@@ -34,7 +34,7 @@ function splitWords(value: string): string[] {
     .replace(/[_-]+/g, ' ')
     .trim()
     .split(/\s+/)
-    .filter(Boolean);
+    .filter(Boolean)
 }
 
 /** Apply a template helper (`kebabCase`/`camelCase`/`lowerCase`/`upperCase`) to a member name. */
@@ -43,19 +43,19 @@ function applyHelper(helper: string, member: string): string {
     case 'kebabCase':
       return splitWords(member)
         .map(word => word.toLowerCase())
-        .join('-');
+        .join('-')
     case 'camelCase':
       return splitWords(member)
         .map((word, index) =>
           index === 0 ? word.toLowerCase() : word[0]!.toUpperCase() + word.slice(1).toLowerCase(),
         )
-        .join('');
+        .join('')
     case 'lowerCase':
-      return member.toLowerCase();
+      return member.toLowerCase()
     case 'upperCase':
-      return member.toUpperCase();
+      return member.toUpperCase()
     default:
-      return member;
+      return member
   }
 }
 
@@ -63,7 +63,7 @@ function applyHelper(helper: string, member: string): string {
 function applyTemplate(template: string, member: string): string {
   return template.replace(/\{\{\s*(?:(\w+)\s+)?member\s*\}\}/g, (_full, helper?: string) =>
     helper ? applyHelper(helper, member) : member,
-  );
+  )
 }
 
 /**
@@ -71,23 +71,26 @@ function applyTemplate(template: string, member: string): string {
  * statements. Returns `undefined` (leaving the original import untouched) for a
  * clause the rewrite does not own: no members, or a `type` import.
  */
-function expandMembers(bindings: readonly ImportBinding[], rule: ModularizeRule): string | undefined {
-  if (bindings.length === 0) return undefined;
+function expandMembers(
+  bindings: readonly ImportBinding[],
+  rule: ModularizeRule,
+): string | undefined {
+  if (bindings.length === 0) return undefined
 
-  const statements: string[] = [];
+  const statements: string[] = []
   for (const binding of bindings) {
-    if (binding.type || binding.imported === 'default' || binding.imported === '*') return undefined;
-    const original = binding.imported;
-    const local = binding.local;
-    const target = JSON.stringify(applyTemplate(rule.transform, original));
+    if (binding.type || binding.imported === 'default' || binding.imported === '*') return undefined
+    const original = binding.imported
+    const local = binding.local
+    const target = JSON.stringify(applyTemplate(rule.transform, original))
     if (rule.skipDefaultConversion) {
-      const named = original === local ? original : `${original} as ${local}`;
-      statements.push(`import { ${named} } from ${target};`);
+      const named = original === local ? original : `${original} as ${local}`
+      statements.push(`import { ${named} } from ${target};`)
     } else {
-      statements.push(`import ${local} from ${target};`);
+      statements.push(`import ${local} from ${target};`)
     }
   }
-  return statements.join(' ');
+  return statements.join(' ')
 }
 
 /**
@@ -98,22 +101,22 @@ function expandMembers(bindings: readonly ImportBinding[], rule: ModularizeRule)
 export function createModularizeImportsTransform(
   rules: Record<string, ModularizeRule>,
 ): ServerSourceTransform {
-  if (Object.keys(rules).length === 0) return withSniff([], source => source);
+  if (Object.keys(rules).length === 0) return withSniff([], source => source)
 
   // Folded onto the module record: one memoized parse instead
   // of a per-rule import regex, and a configured specifier quoted in a string or
   // a comment no longer expands.
   return withSniff(specifierSniffTokens(Object.keys(rules)), (source, file) => {
-    if (!source.includes('import')) return source;
-    const edits: { start: number; end: number; value: string }[] = [];
+    if (!source.includes('import')) return source
+    const edits: { start: number; end: number; value: string }[] = []
     for (const statement of rewriteFacts(file, source).imports) {
-      const rule = rules[statement.specifier];
-      if (!rule) continue;
-      const expanded = expandMembers(statement.bindings, rule);
+      const rule = rules[statement.specifier]
+      if (!rule) continue
+      const expanded = expandMembers(statement.bindings, rule)
       if (expanded !== undefined) {
-        edits.push({ start: statement.start, end: statement.end, value: expanded });
+        edits.push({ start: statement.start, end: statement.end, value: expanded })
       }
     }
-    return edits.length > 0 ? spliceSource(source, edits) : source;
-  });
+    return edits.length > 0 ? spliceSource(source, edits) : source
+  })
 }

package/src/compat/bundler/new-url-asset.ts

@@ -12,22 +12,22 @@
 //
 // Applied to both graphs as a registered source transform.
 
-import { rewriteReactProfilerServerSource } from './react-compiler';
+import { rewriteReactProfilerServerSource } from './react-compiler'
 
 // A `new URL('<image>', import.meta.url)` call whose first argument is a string
 // literal ending in a static-image extension (optional ?query/#hash) and whose
 // second argument is exactly `import.meta.url`. Non-asset `new URL(...)` calls
 // (e.g. `new URL(request.url)`) never match.
-const IMAGE_EXT = 'png|jpe?g|gif|webp|avif|svg|ico|bmp';
+const IMAGE_EXT = 'png|jpe?g|gif|webp|avif|svg|ico|bmp'
 const newUrlAssetPattern = () =>
   new RegExp(
     `new\\s+URL\\(\\s*(['"])([^'"]+?\\.(?:${IMAGE_EXT})(?:[?#][^'"]*)?)\\1\\s*,\\s*import\\.meta\\.url\\s*\\)`,
     'g',
-  );
+  )
 
 /** Cheap gate: does the source contain any candidate `new URL(...import.meta.url)`? */
 export function sourceHasNewUrlAsset(source: string): boolean {
-  return source.includes('import.meta.url') && newUrlAssetPattern().test(source);
+  return source.includes('import.meta.url') && newUrlAssetPattern().test(source)
 }
 
 /**
@@ -36,11 +36,11 @@ export function sourceHasNewUrlAsset(source: string): boolean {
  * Returns the source unchanged when there is nothing to rewrite.
  */
 export function rewriteNewUrlAssetSource(source: string, filePath?: string): string {
-  const rewritten = rewriteReactProfilerServerSource(source, filePath);
+  const rewritten = rewriteReactProfilerServerSource(source, filePath)
   return rewriteNewUrlAssets(
     rewritten,
     typeof filePath === 'string' && filePath.includes('middleware'),
-  );
+  )
 }
 
 /**
@@ -49,28 +49,28 @@ export function rewriteNewUrlAssetSource(source: string, filePath?: string): str
  * - that one is registered as its own client transform.
  */
 export function rewriteNewUrlAssetClientSource(source: string): string {
-  return rewriteNewUrlAssets(source, false);
+  return rewriteNewUrlAssets(source, false)
 }
 
 function rewriteNewUrlAssets(source: string, isMiddleware: boolean): string {
-  if (!source.includes('import.meta.url')) return source;
-  const bindings = new Map<string, string>();
+  if (!source.includes('import.meta.url')) return source
+  const bindings = new Map<string, string>()
   const replaced = source.replace(newUrlAssetPattern(), (_match, _quote, spec: string) => {
-    let binding = bindings.get(spec);
+    let binding = bindings.get(spec)
     if (!binding) {
-      binding = `__pnext_new_url_${bindings.size}`;
-      bindings.set(spec, binding);
+      binding = `__pnext_new_url_${bindings.size}`
+      bindings.set(spec, binding)
     }
     if (isMiddleware) {
-      return `('blob:' + ${binding}.src.split('/').pop())`;
+      return `('blob:' + ${binding}.src.split('/').pop())`
     }
-    return `(${binding}.src)`;
-  });
-  if (bindings.size === 0) return source;
+    return `(${binding}.src)`
+  })
+  if (bindings.size === 0) return source
   const imports = [...bindings]
     .map(([spec, binding]) => `import ${binding} from ${JSON.stringify(spec)};`)
-    .join('\n');
-  return injectAfterDirectives(replaced, imports);
+    .join('\n')
+  return injectAfterDirectives(replaced, imports)
 }
 
 // Insert injected imports after any leading directive prologue ('use client' /
@@ -78,10 +78,8 @@ function rewriteNewUrlAssets(source: string, isMiddleware: boolean): string {
 // `'use client'` stays the first statement (otherwise the directive is demoted
 // to a plain expression and the module is treated as server-only).
 function injectAfterDirectives(source: string, injection: string): string {
-  const prologue = /^(\uFEFF?(?:#![^\n]*\n)?(?:[ \t]*(['"])use [\w-]+\2[ \t]*;?[ \t]*(?:\r?\n|$))*)/.exec(
-    source,
-  );
-  const index = prologue ? prologue[0].length : 0;
-  return `${source.slice(0, index)}${injection}\n${source.slice(index)}`;
+  const prologue =
+    /^(\uFEFF?(?:#![^\n]*\n)?(?:[ \t]*(['"])use [\w-]+\2[ \t]*;?[ \t]*(?:\r?\n|$))*)/.exec(source)
+  const index = prologue ? prologue[0].length : 0
+  return `${source.slice(0, index)}${injection}\n${source.slice(index)}`
 }
-

package/src/compat/bundler/optimize-package-imports.ts

@@ -12,18 +12,18 @@
 // the entry must be re-exports and nothing else - no declarations, no side effects - or the package
 // is left alone, which is why an over-broad list cannot produce wrong code.
 
-import path from 'node:path';
-import { readFileSync } from 'node:fs';
+import path from 'node:path'
+import { readFileSync } from 'node:fs'
 // Lazy: the oxc-parser native binding costs ~12.6 MB RSS; load it only when a parse happens.
 const parseSync: typeof import('oxc-parser').parseSync = (...args) =>
   // eslint-disable-next-line @typescript-eslint/no-require-imports
-  loadNative(() => require('oxc-parser') as typeof import('oxc-parser')).parseSync(...args);
-import { loadNative } from '../../utils/native-require';
-import { rewriteFacts, type ImportBinding } from '../../resolve/scan-facts';
-import { spliceSource } from '../../dev/module-transform';
-import { resolvePackageSpecifier, resolveVendorPackageSpecifier } from '../../resolve/imports';
-import { withSniff, type ServerSourceTransform } from '../../extensions';
-import { specifierSniffTokens } from '../../utils/source';
+  loadNative(() => require('oxc-parser') as typeof import('oxc-parser')).parseSync(...args)
+import { loadNative } from '../../utils/native-require'
+import { rewriteFacts, type ImportBinding } from '../../resolve/scan-facts'
+import { spliceSource } from '../../runtime/module-transform'
+import { resolvePackageSpecifier, resolveVendorPackageSpecifier } from '../../resolve/imports'
+import { withSniff, type ServerSourceTransform } from '../../extensions'
+import { specifierSniffTokens } from '../../utils/code'
 
 /**
  * Next's default `optimizePackageImports` set - the barrels big enough that compiling the entry is the
@@ -55,29 +55,29 @@ const DEFAULT_OPTIMIZED_PACKAGES = [
   'react-use',
   'recharts',
   'rxjs',
-];
+]
 
 export function optimizedPackageSet(configured: readonly string[]): string[] {
   // Rewriting a barrel changes what the vendor pipeline compiles, so it is also
   // what a slow or wrong artifact has to be bisected against — and how the
   // bench A/Bs it (same seam shape as `PNEXT_VENDOR_GROUP=0`).
   // eslint-disable-next-line turbo/no-undeclared-env-vars
-  if (process.env.PNEXT_OPTIMIZE_PACKAGE_IMPORTS === '0') return [];
-  return [...new Set([...DEFAULT_OPTIMIZED_PACKAGES, ...configured])];
+  if (process.env.PNEXT_OPTIMIZE_PACKAGE_IMPORTS === '0') return []
+  return [...new Set([...DEFAULT_OPTIMIZED_PACKAGES, ...configured])]
 }
 
 /** Where one exported name really lives: the barrel's re-export target. */
 interface BarrelMember {
   /** The leaf module, relative to the barrel entry. */
-  source: string;
+  source: string
   /** The name to take from it: `default`, `*`, or an export name. */
-  imported: string;
+  imported: string
 }
 
 /** The exported names a pure re-export barrel publishes, and their leaves. */
-type BarrelMap = ReadonlyMap<string, BarrelMember>;
+type BarrelMap = ReadonlyMap<string, BarrelMember>
 
-const barrelMaps = new Map<string, BarrelMap | undefined>();
+const barrelMaps = new Map<string, BarrelMap | undefined>()
 
 /**
  * The leaf map of a barrel entry, or undefined when the entry is not a pure
@@ -85,67 +85,67 @@ const barrelMaps = new Map<string, BarrelMap | undefined>();
  * however many modules import it.
  */
 function barrelMap(entry: string): BarrelMap | undefined {
-  if (barrelMaps.has(entry)) return barrelMaps.get(entry);
-  let map: BarrelMap | undefined;
+  if (barrelMaps.has(entry)) return barrelMaps.get(entry)
+  let map: BarrelMap | undefined
   try {
-    map = parseBarrel(entry);
+    map = parseBarrel(entry)
   } catch {
-    map = undefined;
+    map = undefined
   }
-  barrelMaps.set(entry, map);
-  return map;
+  barrelMaps.set(entry, map)
+  return map
 }
 
 function parseBarrel(entry: string): BarrelMap | undefined {
-  const source = readFileSync(entry, 'utf8');
-  const parsed = parseSync(entry, source, { lang: 'js' });
-  if (parsed.errors.length > 0) return undefined;
+  const source = readFileSync(entry, 'utf8')
+  const parsed = parseSync(entry, source, { lang: 'js' })
+  if (parsed.errors.length > 0) return undefined
 
   // A namespace re-export (`import * as icons from './icons/index.js'; export
   // { icons }`) is two statements; the local binding is only allowed to leave
   // the module through that export, or the barrel is doing something else.
-  const namespaces = new Map<string, string>();
+  const namespaces = new Map<string, string>()
   for (const node of parsed.program.body) {
     if (node.type === 'ImportDeclaration') {
-      const specifiers = node.specifiers ?? [];
-      const only = specifiers[0];
-      if (specifiers.length !== 1 || only?.type !== 'ImportNamespaceSpecifier') return undefined;
-      namespaces.set(only.local.name, node.source.value);
-      continue;
+      const specifiers = node.specifiers ?? []
+      const only = specifiers[0]
+      if (specifiers.length !== 1 || only?.type !== 'ImportNamespaceSpecifier') return undefined
+      namespaces.set(only.local.name, node.source.value)
+      continue
     }
     if (node.type === 'ExportNamedDeclaration') {
       // `export const x = …` is a definition, not a re-export: not a barrel.
-      if (node.declaration) return undefined;
-      continue;
+      if (node.declaration) return undefined
+      continue
     }
     // `export * from` hides its names in the target, so a demanded name cannot
     // be attributed to a leaf; anything else is code with a side effect.
-    return undefined;
+    return undefined
   }
 
-  const members = new Map<string, BarrelMember>();
+  const members = new Map<string, BarrelMember>()
   for (const statement of parsed.module.staticExports) {
     for (const record of statement.entries) {
-      if (record.isType) continue;
-      const exported = record.exportName.name;
-      if (!exported) return undefined;
-      const request = record.moduleRequest?.value;
+      if (record.isType) continue
+      const exported = record.exportName.name
+      if (!exported) return undefined
+      const request = record.moduleRequest?.value
       if (request) {
-        const kind: string = record.importName.kind;
-        if (kind === 'AllButDefault') return undefined;
+        const kind: string = record.importName.kind
+        if (kind === 'AllButDefault') return undefined
         const imported =
-          kind === 'Default' ? 'default' : kind === 'All' ? '*' : record.importName.name;
-        if (!imported) return undefined;
-        members.set(exported, { source: request, imported });
-        continue;
+          kind === 'Default' ? 'default' : kind === 'All' ? '*' : record.importName.name
+        if (!imported) return undefined
+        members.set(exported, { source: request, imported })
+        continue
       }
-      const local = record.localName.name;
-      const namespace = local === null ? undefined : namespaces.get(local);
-      if (!namespace) return undefined;
-      members.set(exported, { source: namespace, imported: '*' });
+      const local = record.localName.name
+      const namespace = local === null ? undefined : namespaces.get(local)
+      if (!namespace) return undefined
+      members.set(exported, { source: namespace, imported: '*' })
     }
   }
-  return members.size > 0 ? members : undefined;
+  return members.size > 0 ? members : undefined
 }
 
 /**
@@ -154,11 +154,11 @@ function parseBarrel(entry: string): BarrelMap | undefined {
  * catalogue per layer, and one rewritten source is compiled for both.
  */
 function barrelEntry(root: string, file: string, specifier: string) {
-  const client = resolveVendorPackageSpecifier(root, file, specifier, CLIENT_CONDITIONS);
-  if (!client) return undefined;
+  const client = resolveVendorPackageSpecifier(root, file, specifier, CLIENT_CONDITIONS)
+  if (!client) return undefined
   return resolveVendorPackageSpecifier(root, file, specifier, SERVER_CONDITIONS) === client
     ? client
-    : undefined;
+    : undefined
 }
 
 /**
@@ -169,48 +169,51 @@ function barrelEntry(root: string, file: string, specifier: string) {
  * because one rewritten source is compiled for both.
  */
 function leafSpecifier(root: string, entry: string, member: BarrelMember): string | undefined {
-  const key = `${entry}\0${member.source}`;
-  if (leafSpecifiers.has(key)) return leafSpecifiers.get(key);
-  const specifier = resolveLeafSpecifier(root, entry, member);
-  leafSpecifiers.set(key, specifier);
-  return specifier;
+  const key = `${entry}\0${member.source}`
+  if (leafSpecifiers.has(key)) return leafSpecifiers.get(key)
+  const specifier = resolveLeafSpecifier(root, entry, member)
+  leafSpecifiers.set(key, specifier)
+  return specifier
 }
 
-const leafSpecifiers = new Map<string, string | undefined>();
+const leafSpecifiers = new Map<string, string | undefined>()
 
-const CLIENT_CONDITIONS = ['module', 'import'];
-const SERVER_CONDITIONS = ['react-server', 'node', 'module', 'import'];
+const CLIENT_CONDITIONS = ['module', 'import']
+const SERVER_CONDITIONS = ['react-server', 'node', 'module', 'import']
 
 function resolveLeafSpecifier(root: string, entry: string, member: BarrelMember) {
-  if (!member.source.startsWith('.')) return undefined;
-  const file = path.resolve(path.dirname(entry), member.source);
-  const packageDir = packageDirOf(entry);
-  if (!packageDir) return undefined;
-  const name = packageNameOfDir(packageDir);
-  const relative = path.relative(packageDir, file);
-  if (!name || relative.startsWith('..') || path.isAbsolute(relative)) return undefined;
-  const specifier = `${name}/${relative.split(path.sep).join('/')}`;
+  if (!member.source.startsWith('.')) return undefined
+  const file = path.resolve(path.dirname(entry), member.source)
+  const packageDir = packageDirOf(entry)
+  if (!packageDir) return undefined
+  const name = packageNameOfDir(packageDir)
+  const relative = path.relative(packageDir, file)
+  if (!name || relative.startsWith('..') || path.isAbsolute(relative)) return undefined
+  const specifier = `${name}/${relative.split(path.sep).join('/')}`
   for (const conditions of [CLIENT_CONDITIONS, SERVER_CONDITIONS]) {
-    if (resolvePackageSpecifier(root, entry, specifier, conditions) !== file) return undefined;
+    if (resolvePackageSpecifier(root, entry, specifier, conditions) !== file) return undefined
   }
-  return specifier;
+  return specifier
 }
 
 /** The installed package a barrel entry belongs to, by its `node_modules` segment. */
 function packageDirOf(entry: string) {
-  const marker = `${path.sep}node_modules${path.sep}`;
-  const index = entry.lastIndexOf(marker);
-  if (index < 0) return undefined;
-  const rest = entry.slice(index + marker.length).split(path.sep);
-  const segments = rest[0]?.startsWith('@') ? rest.slice(0, 2) : rest.slice(0, 1);
-  return path.join(entry.slice(0, index + marker.length), ...segments);
+  const marker = `${path.sep}node_modules${path.sep}`
+  const index = entry.lastIndexOf(marker)
+  if (index < 0) return undefined
+  const rest = entry.slice(index + marker.length).split(path.sep)
+  const segments = rest[0]?.startsWith('@') ? rest.slice(0, 2) : rest.slice(0, 1)
+  return path.join(entry.slice(0, index + marker.length), ...segments)
 }
 
 function packageNameOfDir(packageDir: string) {
-  const marker = `${path.sep}node_modules${path.sep}`;
-  const index = packageDir.lastIndexOf(marker);
-  if (index < 0) return undefined;
-  return packageDir.slice(index + marker.length).split(path.sep).join('/');
+  const marker = `${path.sep}node_modules${path.sep}`
+  const index = packageDir.lastIndexOf(marker)
+  if (index < 0) return undefined
+  return packageDir
+    .slice(index + marker.length)
+    .split(path.sep)
+    .join('/')
 }
 
 /**
@@ -224,28 +227,29 @@ function expandBarrelImport(
   root: string,
   entry: string,
 ): string | undefined {
-  if (bindings.length === 0) return undefined;
-  const statements: string[] = [];
+  if (bindings.length === 0) return undefined
+  const statements: string[] = []
   for (const binding of bindings) {
     // A barrel publishes its types off the same entry, so `{ Icon, type Props }`
     // is the common shape. The type binding is erased before anything resolves
     // it, so dropping it is what lets the value bindings split at all.
-    if (binding.type) continue;
-    if (binding.imported === 'default' || binding.imported === '*') return undefined;
-    const member = map.get(binding.imported);
-    if (!member) return undefined;
-    const specifier = leafSpecifier(root, entry, member);
-    if (!specifier) return undefined;
-    const target = JSON.stringify(specifier);
-    if (member.imported === 'default') statements.push(`import ${binding.local} from ${target};`);
-    else if (member.imported === '*') statements.push(`import * as ${binding.local} from ${target};`);
+    if (binding.type) continue
+    if (binding.imported === 'default' || binding.imported === '*') return undefined
+    const member = map.get(binding.imported)
+    if (!member) return undefined
+    const specifier = leafSpecifier(root, entry, member)
+    if (!specifier) return undefined
+    const target = JSON.stringify(specifier)
+    if (member.imported === 'default') statements.push(`import ${binding.local} from ${target};`)
+    else if (member.imported === '*')
+      statements.push(`import * as ${binding.local} from ${target};`)
     else {
       const named =
-        member.imported === binding.local ? binding.local : `${member.imported} as ${binding.local}`;
-      statements.push(`import { ${named} } from ${target};`);
+        member.imported === binding.local ? binding.local : `${member.imported} as ${binding.local}`
+      statements.push(`import { ${named} } from ${target};`)
     }
   }
-  return statements.length > 0 ? statements.join(' ') : undefined;
+  return statements.length > 0 ? statements.join(' ') : undefined
 }
 
 /**
@@ -257,21 +261,21 @@ export function createOptimizePackageImportsTransform(
   packages: readonly string[],
   root: string,
 ): ServerSourceTransform {
-  if (packages.length === 0) return withSniff([], source => source);
-  const optimized = new Set(packages);
+  if (packages.length === 0) return withSniff([], source => source)
+  const optimized = new Set(packages)
   return withSniff(specifierSniffTokens(packages), (source, file) => {
-    if (!source.includes('import')) return source;
-    const edits: { start: number; end: number; value: string }[] = [];
+    if (!source.includes('import')) return source
+    const edits: { start: number; end: number; value: string }[] = []
     for (const statement of rewriteFacts(file, source).imports) {
-      if (!optimized.has(statement.specifier)) continue;
-      const entry = barrelEntry(root, file, statement.specifier);
-      const map = entry && barrelMap(entry);
-      if (!map || !entry) continue;
-      const expanded = expandBarrelImport(statement.bindings, map, root, entry);
+      if (!optimized.has(statement.specifier)) continue
+      const entry = barrelEntry(root, file, statement.specifier)
+      const map = entry && barrelMap(entry)
+      if (!map || !entry) continue
+      const expanded = expandBarrelImport(statement.bindings, map, root, entry)
       if (expanded !== undefined) {
-        edits.push({ start: statement.start, end: statement.end, value: expanded });
+        edits.push({ start: statement.start, end: statement.end, value: expanded })
       }
     }
-    return edits.length > 0 ? spliceSource(source, edits) : source;
-  });
+    return edits.length > 0 ? spliceSource(source, edits) : source
+  })
 }