fix: move the fig complete import to an optional require

jmcdo29 · Dec 4, 2023 · 72b2a00 · 72b2a00
1 parent 9317680
commit 72b2a00
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/.changeset/stale-pillows-talk.md b/.changeset/stale-pillows-talk.md
@@ -0,0 +1,6 @@
+---
+'nest-commander': patch
+---
+
+Move the fig completion package to an optional import to get around jest
+throwing an error about esm packages
diff --git a/packages/nest-commander/src/completion.factory.ts b/packages/nest-commander/src/completion.factory.ts
@@ -1,5 +1,3 @@
-/* eslint-disable @typescript-eslint/ban-ts-comment */
-import { addCompletionSpecCommand } from '@fig/complete-commander';
 import type { INestApplicationContext } from '@nestjs/common';
 import { Command } from 'commander';
 import type { CompletionFactoryOptions } from './completion.factory.interface';
@@ -50,7 +48,15 @@ export class CompletionFactory {
     }
 
     if (parsedOptions.fig) {
-      addCompletionSpecCommand(commander);
+      try {
+        // eslint-disable-next-line @typescript-eslint/no-var-requires
+        const { addCompletionSpecCommand } = require('@fig/complete-commander');
+        addCompletionSpecCommand(commander);
+      } catch {
+        throw new Error(
+          `There was a problem creating the fig completion. Did you make sure to install "@fig/complete-commander"?`,
+        );
+      }
     }
   }