fix(project): adjust things

Author: olavoparno

package.json

@@ -15,6 +15,11 @@
     "node": ">=8",
     "npm": ">=5"
   },
+  "files": [
+    "LICENSE",
+    "README.md",
+    "dist/"
+  ],
   "husky": {
     "hooks": {
       "pre-commit": "npm run lint && npm run test && npm run make-badges && git add 'README.md'"

rollup.config.js

@@ -1,10 +1,9 @@
-import typescript from 'rollup-plugin-typescript2'
-import commonjs from 'rollup-plugin-commonjs'
-import external from 'rollup-plugin-peer-deps-external'
-import resolve from 'rollup-plugin-node-resolve'
-import url from 'rollup-plugin-url'
+import typescript from 'rollup-plugin-typescript2';
+import commonjs from 'rollup-plugin-commonjs';
+import external from 'rollup-plugin-peer-deps-external';
+import resolve from 'rollup-plugin-node-resolve';
 
-import pkg from './package.json'
+import pkg from './package.json';
 
 export default {
   input: 'src/index.tsx',
@@ -13,23 +12,22 @@ export default {
       file: pkg.main,
       format: 'cjs',
       exports: 'named',
-      sourcemap: true
+      sourcemap: true,
     },
     {
       file: pkg.module,
       format: 'es',
       exports: 'named',
-      sourcemap: true
-    }
+      sourcemap: true,
+    },
   ],
   plugins: [
     external(),
-    url({ exclude: ['**/*.svg'] }),
     resolve(),
     typescript({
       rollupCommonJSResolveHack: true,
-      clean: true
+      clean: true,
     }),
-    commonjs()
-  ]
-}
+    commonjs(),
+  ],
+};

src/index.tsx

@@ -6,25 +6,21 @@ type TUseHotjar = {
     hotjarId: string,
     hotjarVersion: string,
     shouldLog?: boolean
-  ) => Promise<boolean>;
+  ) => boolean;
   identityHotjar: (
     userId: string,
     userInfo: string,
     shouldLog?: boolean
-  ) => Promise<boolean>;
+  ) => boolean;
 };
 
 export function useHotjar(): TUseHotjar {
   const { appendHeadScript } = useAppendHeadScript();
 
   const initHotjar = React.useCallback(
-    async (
-      hotjarId: string,
-      hotjarVersion: string,
-      shouldLog = true
-    ): Promise<boolean> => {
+    (hotjarId: string, hotjarVersion: string, shouldLog = true): boolean => {
       const hotjarScript = `(function(h,o,t,j,a,r){h.hj=h.hj||function(){(h.hj.q=h.hj.q||[]).push(arguments)};h._hjSettings={hjid:${hotjarId},hjsv:${hotjarVersion}};a=o.getElementsByTagName('head')[0];r=o.createElement('script');r.async=1;r.src=t+h._hjSettings.hjid+j+h._hjSettings.hjsv;a.appendChild(r);})(window,document,'https://static.hotjar.com/c/hotjar-','.js?sv=');`;
-      const isHotjarAppended = await appendHeadScript({
+      const isHotjarAppended = appendHeadScript({
         scriptText: hotjarScript,
         scriptId: 'hotjar-script',
       });
@@ -37,14 +33,10 @@ export function useHotjar(): TUseHotjar {
   );
 
   const identityHotjar = React.useCallback(
-    async (
-      userId: string,
-      userInfo: string,
-      shouldLog = true
-    ): Promise<boolean> => {
+    (userId: string, userInfo: string, shouldLog = true): boolean => {
       try {
         const hotjarIdentityScript = `var userId="${userId}" || null;window.hj("identify",userId,{${userInfo}}});`;
-        const isIdentified = await appendHeadScript({
+        const isIdentified = appendHeadScript({
           scriptText: hotjarIdentityScript,
           scriptId: 'identity-script',
         });

src/useAppendHeadScript.ts

@@ -6,34 +6,29 @@ type TAppendHeadScript = {
 };
 
 type TUseAppendHeadScript = {
-  appendHeadScript: ({
-    scriptText,
-    scriptId,
-  }: TAppendHeadScript) => Promise<boolean>;
+  appendHeadScript: ({ scriptText, scriptId }: TAppendHeadScript) => boolean;
 };
 
 export function useAppendHeadScript(): TUseAppendHeadScript {
   const appendHeadScript = ({
     scriptText,
     scriptId,
-  }: TAppendHeadScript): Promise<boolean> => {
-    return new Promise((resolve, reject) => {
-      try {
-        const existentScript = document.getElementById(
-          scriptId
-        ) as HTMLScriptElement;
-        const script = existentScript || document.createElement('script');
-        script.id = scriptId;
-        script.innerText = scriptText;
-        script.crossOrigin = 'anonymous';
+  }: TAppendHeadScript): boolean => {
+    try {
+      const existentScript = document.getElementById(
+        scriptId
+      ) as HTMLScriptElement;
+      const script = existentScript || document.createElement('script');
+      script.id = scriptId;
+      script.innerText = scriptText;
+      script.crossOrigin = 'anonymous';
 
-        document.head.appendChild(script);
+      document.head.appendChild(script);
 
-        resolve(true);
-      } catch (error) {
-        reject(error);
-      }
-    });
+      return true;
+    } catch (error) {
+      return error;
+    }
   };
 
   return React.useMemo(() => ({ appendHeadScript }), []);