@travetto/pack 5.0.0-rc.8 → 5.0.0-rc.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/pack",
-  "version": "5.0.0-rc.8",
+  "version": "5.0.0-rc.9",
   "description": "Code packing utilities",
   "keywords": [
     "travetto",
@@ -29,12 +29,12 @@
     "@rollup/plugin-json": "^6.1.0",
     "@rollup/plugin-node-resolve": "^15.2.3",
     "@rollup/plugin-terser": "^0.4.4",
-    "@travetto/runtime": "^5.0.0-rc.8",
-    "@travetto/terminal": "^5.0.0-rc.8",
+    "@travetto/runtime": "^5.0.0-rc.9",
+    "@travetto/terminal": "^5.0.0-rc.9",
     "rollup": "^4.18.0"
   },
   "peerDependencies": {
-    "@travetto/cli": "^5.0.0-rc.8"
+    "@travetto/cli": "^5.0.0-rc.9"
   },
   "peerDependenciesMeta": {
     "@travetto/cli": {

package/src/config-util.ts

@@ -74,7 +74,7 @@ export class PackConfigUtil {
   }
 
   /**
-   * Docker app files copied and permissioned
+   * Docker app files copied with proper permissions
    */
   static dockerAppFiles(cfg: DockerPackConfig): string {
     const { user, group, folder } = cfg.dockerRuntime;

package/support/bin/docker-operation.ts

@@ -22,8 +22,13 @@ export class DockerPackOperation {
     // Read os before writing
     cfg.dockerRuntime.os = await PackUtil.runCommand(
       ['docker', 'run', '--entrypoint', '/bin/sh', cfg.dockerImage, '-c', 'cat /etc/*release*']
-      // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-    ).then(out => out.match(/\b(?:debian|alpine|centos)\b/i)?.[0].toLowerCase() as 'alpine' ?? 'unknown');
+    ).then(out => {
+      const found = out.match(/\b(?:debian|alpine|centos)\b/i)?.[0].toLowerCase();
+      switch (found) {
+        case 'debian': case 'alpine': case 'centos': return found;
+        default: return 'unknown';
+      }
+    });
     yield* PackOperation.title(cfg, cliTpl`${{ title: 'Detected Image OS' }} ${{ param: cfg.dockerImage }} as ${{ param: cfg.dockerRuntime.os }}`);
   }
 

package/support/bin/util.ts

@@ -29,7 +29,7 @@ export class PackUtil {
       }
       await fs.mkdir(final, { recursive: true });
       await fs.cp(src, final, { recursive: true });
-    } catch (err) {
+    } catch {
       if (!ignoreFailure) {
         throw new Error(`Failed to copy ${src} to ${dest}`);
       }

package/support/rollup/config.ts

@@ -21,9 +21,16 @@ function getFilesFromModule(m: ManifestModule): string[] {
     .map(([f]) => ManifestModuleUtil.withOutputExtension(path.resolve(m.outputFolder, f)));
 }
 
+function getFormat(value: string = 'commonjs'): NodeModuleType {
+  switch (value) {
+    case 'module':
+    case 'commonjs': return value;
+    default: return 'commonjs';
+  }
+}
+
 export function getOutput(): OutputOptions {
-  // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-  const format = (process.env.BUNDLE_FORMAT ?? 'commonjs') as NodeModuleType;
+  const format = getFormat(process.env.BUNDLE_FORMAT);
   const dir = process.env.BUNDLE_OUTPUT!;
   const mainFile = process.env.BUNDLE_MAIN_FILE!;
   return {