perf(bundler): remove redundant scope walks

Author: harlan-zw

packages/bundler/src/unplugin/TreeshakeServerComposables.ts

@@ -100,10 +100,7 @@ export const TreeshakeServerComposables = createUnplugin<TreeshakeServerComposab
         const ast = parseAndWalkSource(code, id, { scopeTracker })
         const s = new MagicString(code)
 
-        // The parse walk collects all declarations first so hoisted locals
-        // (`function useServerHead() {}` below a call site) are visible when
-        // the removal walk visits earlier statements.
-        walk(ast.program, { scopeTracker })
+        // The parse walk collected all declarations, including hoisted locals.
         scopeTracker.freeze()
 
         walk(ast.program, {

packages/bundler/src/unplugin/UseSeoMetaTransform.ts

@@ -109,10 +109,7 @@ export const UseSeoMetaTransform = createUnplugin<UseSeoMetaTransformOptions, fa
         const ast = parseAndWalkSource(code, id, { scopeTracker })
         const s = new MagicString(code)
 
-        // The parse walk collects all declarations first so hoisted locals
-        // (`function useSeoMeta() {}` below a call site) are visible when
-        // the rewrite walk visits earlier statements.
-        walk(ast.program, { scopeTracker })
+        // The parse walk collected all declarations, including hoisted locals.
         scopeTracker.freeze()
 
         // Track which ImportDeclarations need specifier rewrites