Fix import of ES module open (#491)

Author: nglgzz

.changeset/large-donuts-sniff.md

@@ -0,0 +1,5 @@
+---
+'@relate/cli': patch
+---
+
+Fix ES import of open package

packages/cli/src/modules/dbms/open.module.ts

@@ -1,6 +1,5 @@
 import {Inject, Module, OnApplicationBootstrap} from '@nestjs/common';
 import {SystemModule, SystemProvider} from '@relate/common';
-import {openApp} from 'open';
 
 import OpenCommand from '../../commands/dbms/open';
 import {isInteractive} from '../../stdin';
@@ -18,10 +17,11 @@ export class OpenModule implements OnApplicationBootstrap {
         @Inject(SystemProvider) protected readonly systemProvider: SystemProvider,
     ) {}
 
-    logOrOpen(path: string): void {
+    async logOrOpen(path: string): Promise<void> {
         if (this.parsed.flags.log) {
             this.utils.log(path);
         } else {
+            const {openApp} = await import('open');
             openApp(path);
         }
     }
@@ -43,6 +43,6 @@ export class OpenModule implements OnApplicationBootstrap {
             throw new Error(`DBMS ${dbmsId} could not be opened`);
         }
 
-        this.logOrOpen(dbms.rootPath);
+        await this.logOrOpen(dbms.rootPath);
     }
 }

packages/cli/src/modules/environment/open.module.ts

@@ -1,6 +1,5 @@
 import {Inject, Module, OnApplicationBootstrap} from '@nestjs/common';
 import {SystemModule, SystemProvider} from '@relate/common';
-import {openApp} from 'open';
 
 import OpenCommand from '../../commands/environment/open';
 
@@ -16,10 +15,11 @@ export class OpenModule implements OnApplicationBootstrap {
         @Inject(SystemProvider) protected readonly systemProvider: SystemProvider,
     ) {}
 
-    logOrOpen(path: string): void {
+    async logOrOpen(path: string): Promise<void> {
         if (this.parsed.flags.log) {
             this.utils.log(path);
         } else {
+            const {openApp} = await import('open');
             openApp(path);
         }
     }
@@ -29,6 +29,6 @@ export class OpenModule implements OnApplicationBootstrap {
         const {environment: environmentId} = args;
         const environment = await this.systemProvider.getEnvironment(environmentId);
 
-        this.logOrOpen(environment.configPath);
+        await this.logOrOpen(environment.configPath);
     }
 }

packages/cli/src/modules/project/open.module.ts

@@ -1,6 +1,5 @@
 import {OnApplicationBootstrap, Module, Inject} from '@nestjs/common';
 import {SystemModule, SystemProvider} from '@relate/common';
-import {openApp} from 'open';
 
 import {selectProjectPrompt} from '../../prompts';
 import OpenCommand from '../../commands/project/open';
@@ -17,10 +16,11 @@ export class OpenModule implements OnApplicationBootstrap {
         @Inject(SystemProvider) protected readonly systemProvider: SystemProvider,
     ) {}
 
-    logOrOpen(path: string): void {
+    async logOrOpen(path: string): Promise<void> {
         if (this.parsed.flags.log) {
             this.utils.log(path);
         } else {
+            const {openApp} = await import('open');
             openApp(path);
         }
     }
@@ -33,6 +33,6 @@ export class OpenModule implements OnApplicationBootstrap {
 
         const project = await environment.projects.get(projectId);
 
-        this.logOrOpen(project.root);
+        await this.logOrOpen(project.root);
     }
 }