feat: option to not copy debug dir

adamdbradley · adamdbradley · commit b888df2ca108 · 2022-02-04T06:09:58.000-06:00
diff --git a/bin/partytown.cjs b/bin/partytown.cjs
@@ -1,40 +1,53 @@
 #!/usr/bin/env node
 
 async function run() {
-  const args = process.argv.slice(2);
-  const task = args[0];
+  const task = process.argv.slice(2).filter((a) => !a.startsWith('-'))[0];
+  const args = process.argv.slice(2).filter((a) => a !== task);
 
-  if (!task) {
-    panic('Missing partytown task argument. Example command: partytown copylib dest/directory');
-  }
-
-  switch (task) {
-    case 'copylib': {
-      await copyLibTask(args);
-      break;
-    }
-    default: {
-      panic('Unknown partytown task: ' + task);
-    }
+  if (task === 'help' || args.includes('--help') || args.includes('-h')) {
+    help();
+  } else if (task === 'version' || args.includes('--version') || args.includes('-v')) {
+    console.log(version());
+  } else if (task === 'copylib') {
+    await copyLibTask(args);
+  } else {
+    panic('Unknown partytown task: ' + task);
   }
 }
 
 async function copyLibTask(args) {
   try {
     const utils = require('../utils/index.cjs');
-    const destDir = args[1];
-    const result = await utils.copyLibFiles(destDir);
+    const destDir = args.filter((a) => !a.startsWith('-'))[0];
+    const logResult = !args.includes('--silent');
+    const includeDebugDir = !args.includes('--no-debug');
+    const result = await utils.copyLibFiles(destDir, {
+      debugDir: includeDebugDir,
+    });
 
-    if (!args.includes('--silent')) {
+    if (logResult) {
       console.log('Partytown lib copied to: ' + result.dest);
     }
   } catch (e) {
     panic(String(e.message || e));
   }
 }
 
+function help() {
+  console.log(``);
+  console.log(`Partytown (${version()}):`);
+  console.log(``);
+  console.log(`  copylib <destDir> [--no-debug | --silent]`);
+  console.log(``);
+}
+
+function version() {
+  return require('../package.json').version;
+}
+
 function panic(msg) {
-  console.error('❌ ' + msg);
+  console.error('\n❌ ' + msg);
+  help();
   process.exit(1);
 }
 
diff --git a/src/react/snippet.tsx b/src/react/snippet.tsx
@@ -13,7 +13,7 @@ export interface PartytownProps extends PartytownConfig {}
 
 /**
  * The React `<Partytown/>` component should be placed within the `<head>`
- * of the document`.
+ * of the document.
  *
  * @public
  */
diff --git a/src/utils/api.md b/src/utils/api.md
@@ -5,11 +5,16 @@
 ```ts
 
 // @public
-export function copyLibFiles(destDir: string): Promise<{
+export function copyLibFiles(destDir: string, opts?: CopyLibFilesOptions): Promise<{
     src: string;
     dest: string;
 }>;
 
+// @public (undocumented)
+export interface CopyLibFilesOptions {
+    debugDir?: boolean;
+}
+
 // @public
 export function libDirPath(): string;
 
diff --git a/src/utils/copy-lib-files.ts b/src/utils/copy-lib-files.ts
@@ -29,23 +29,27 @@ export function libDirPath() {
  * This utility function is to make it easier to locate the source library files
  * and copy them to your server's correct location, for example: `./public/~partytown/`.
  *
+ * By default, both the minified and debug builds are copied to the destination.
+ * However, by setting the `debugDir` option to `false`, the debug directory will
+ * not be copied.
+ *
  * @public
  */
-export async function copyLibFiles(destDir: string) {
+export async function copyLibFiles(destDir: string, opts: CopyLibFilesOptions = {}) {
   if (typeof destDir !== 'string' || destDir.length === 0) {
     throw new Error('Missing destDir');
   }
   if (!isAbsolute(destDir)) {
     destDir = resolve(process.cwd(), destDir);
   }
-  await copyLibDir(libDirPath(), destDir);
+  await copyLibDir(libDirPath(), destDir, opts);
   return {
     src: libDirPath(),
     dest: destDir,
   };
 }
 
-async function copyLibDir(srcDir: string, destDir: string) {
+async function copyLibDir(srcDir: string, destDir: string, opts: CopyLibFilesOptions) {
   try {
     await mkdir(destDir, { recursive: true });
   } catch (e) {}
@@ -61,8 +65,21 @@ async function copyLibDir(srcDir: string, destDir: string) {
       if (s.isFile()) {
         await copyFile(srcPath, destPath);
       } else if (s.isDirectory()) {
-        await copyLibDir(srcPath, destPath);
+        if (srcName === 'debug' && opts.debugDir === false) {
+          return;
+        }
+        await copyLibDir(srcPath, destPath, opts);
       }
     })
   );
 }
+
+/**
+ * @public
+ */
+export interface CopyLibFilesOptions {
+  /**
+   * When set to `false` the debug directory will not be copied.
+   */
+  debugDir?: boolean;
+}
diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -1 +1,2 @@
 export { copyLibFiles, libDirPath } from './copy-lib-files';
+export type { CopyLibFilesOptions } from './copy-lib-files';

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ export interface PartytownProps extends PartytownConfig {}`
`13`	`13`
`14`	`14`	`/**`
`15`	`15`	* The React `<Partytown/>` component should be placed within the `<head>`
`16`		- * of the document`.
	`16`	`+ * of the document.`
`17`	`17`	`*`
`18`	`18`	`* @public`
`19`	`19`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -1 +1,2 @@`
`1`	`1`	`export { copyLibFiles, libDirPath } from './copy-lib-files';`
	`2`	`+export type { CopyLibFilesOptions } from './copy-lib-files';`