fix: onExtract option fix

Anidetrix · May 13, 2020 · 65d74cf · 65d74cf
1 parent 3408f29
commit 65d74cf
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 17 deletions.
diff --git a/package.json b/package.json
@@ -83,7 +83,7 @@
     "@types/mime-types": "^2.1.0",
     "@types/node": "*",
     "@types/postcss-load-config": "^2.0.1",
-    "@types/resolve": "^1.17.0",
+    "@types/resolve": "^1.17.1",
     "@types/uglifycss": "^0.0.5",
     "@typescript-eslint/eslint-plugin": "^2.33.0",
     "@typescript-eslint/parser": "^2.33.0",

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/index.ts b/src/index.ts
@@ -248,7 +248,7 @@ export default (options: Options = {}): Plugin => {
         const res = getExtractedData(name, ids);
 
         if (options.onExtract) {
-          const shouldExtract = options.onExtract(getExtractedData);
+          const shouldExtract = options.onExtract(res);
           if (!shouldExtract) continue;
         }
 

diff --git a/src/types.ts b/src/types.ts
@@ -345,14 +345,15 @@ export interface Options {
    * */
   loaders?: Loader[];
   /**
-   * Function which is invoked on CSS file import
+   * Function which is invoked on CSS file import,
+   * before any transformations are applied
    * @default undefined
    * */
   onImport?: (code: string, id: string) => void;
   /**
-   * Function which is invoked on CSS file import.
+   * Function which is invoked on CSS file export.
    * Return `boolean` to decide if you want to extract the file or not.
    * @default undefined
    * */
-  onExtract?: (fn: (name: string, ids: string[]) => ExtractedData) => boolean;
+  onExtract?: (data: ExtractedData) => boolean;
 }