Merge pull request #16 from farm-fe/chore/version

chore: bump version
farm-fe · May 26, 2024 · 0870a5d · 0870a5d
2 parents dd682dc + c593603
commit 0870a5d
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 3 deletions.
diff --git a/.changeset/fresh-eagles-check.md b/.changeset/fresh-eagles-check.md
@@ -0,0 +1,5 @@
+---
+'farmup': patch
+---
+
+optimize build performance
diff --git a/src/core/proxyCompiler/index.ts b/src/core/proxyCompiler/index.ts
@@ -10,8 +10,10 @@ export class ProxyCompiler {
 
     private lastResources: string[] = [];
     private _preProxyFnList: (keyof Compiler)[] = [];
+    private alreadyProxyFnList: Set<keyof Compiler> = new Set();
 
     start(compiler: Compiler) {
+        const isRestart = !!this.compiler;
         this.compiler = compiler;
 
         if (this._preProxyFnList.length) {
@@ -21,6 +23,15 @@ export class ProxyCompiler {
 
             this._preProxyFnList = [];
         }
+
+        if (isRestart) {
+            const proxyFnList = this.alreadyProxyFnList;
+            this.alreadyProxyFnList = new Set();
+            for (const fnName of proxyFnList) {
+                this.proxyCompiler(fnName);
+            }
+        }
+
         this.on('resources', (r) => {
             this.lastResources = Object.keys(r.result);
         });
@@ -30,11 +41,18 @@ export class ProxyCompiler {
         return this.lastResources;
     }
 
-    proxyCompiler<K extends keyof Compiler>(fnName: K) {
+    private proxyCompiler<K extends keyof Compiler>(fnName: K) {
         if (!this.compiler) {
             this._preProxyFnList.push(fnName);
             return;
         }
+
+        if (this.alreadyProxyFnList.has(fnName)) {
+            return;
+        }
+
+        this.alreadyProxyFnList.add(fnName);
+
         // biome-ignore lint/suspicious/noExplicitAny: <explanation>
         proxyCompilerFn(this.compiler, fnName, (...args: any[]) => this.event.emit(fnName, ...args));
     }
@@ -46,7 +64,7 @@ export class ProxyCompiler {
         OFT extends Record<keyof T, (...args: any) => any> = {
             // biome-ignore lint/suspicious/noExplicitAny: <explanation>
             [KK in keyof T]: T[KK] extends (...args: any[]) => any ? T[KK] : never;
-        },
+        }
     >(fnName: K, fn: (context: FnContext<Parameters<OFT[K]>, ReturnType<OFT[K]>>) => void) {
         this.proxyCompiler(fnName as keyof Compiler);
         this.event.on(fnName.toString(), fn);

diff --git a/src/core/proxyCompiler/util.ts b/src/core/proxyCompiler/util.ts
@@ -3,8 +3,10 @@ import type { Compiler } from '@farmfe/core';
 export function defineProperty<O, K extends keyof O, V extends O[K]>(obj: O, key: K, value: V): V {
     const origin = obj[key];
 
+    const descriptor = Object.getOwnPropertyDescriptor(obj, key);
     Object.defineProperty(obj, key, {
         value,
+        ...descriptor,
     });
 
     return origin as V;
@@ -19,7 +21,7 @@ export function proxyCompilerFn<
     },
     K extends keyof OFT = keyof OFT,
     // biome-ignore lint/suspicious/noExplicitAny: <explanation>
-    F extends (...args: any) => any = OFT[K],
+    F extends (...args: any) => any = OFT[K]
 >(compiler: T, fnName: K, callback: F) {
     const handler = ((...args: Parameters<OFT[K]>) => {
         const r = origin.bind(compiler)(...args);