fix: hasTarget and check target for output in the target (#12)

Author: Willem Wyndham

dist/cli.js

@@ -110,7 +110,7 @@ function safeRequire(path: string): any {
 }
 
 function hasTarget(config: any, target: string): boolean {
-  return config.target && config.target[target][target];
+  return config.targets && config.targets[target];
 }
 
 function compileProject(
@@ -189,7 +189,7 @@ function compileProject(
   }
   if (
     args.outDir ||
-    !(hasOutput(ascArgv, ".wasm") || config.options?.binaryFile)
+    !containsOutput(config, target, ascArgv)
   ) {
     ascArgv.push("--binaryFile", wasmFile);
   }
@@ -203,3 +203,11 @@ function compileProject(
 function hasOutput(ascArgv: string[], suffix: string): boolean {
   return ascArgv.some((s) => s.endsWith(suffix));
 }
+
+
+function containsOutput(config: any, target: string, ascArgv: string[]): boolean {
+  if (hasOutput(ascArgv, ".wasm")) return true;
+  if (config.options?.binaryFile) return true;
+  if (config.targets && config.targets[target]?.binaryFile) return true;
+  return false;
+}

src/cli.ts

@@ -0,0 +1,10 @@
+{
+  "targets": {
+    "debug": {
+      "debug": false,
+      "noAssert": true,
+      "binaryFile": "out/main.wasm",
+      "optimize": true
+    }
+  }
+}

test/targets/asconfig.json

@@ -0,0 +1,3 @@
+if (!ASC_NO_ASSERT) {
+  assert(false, "noAssert should be true");
+}

test/targets/assembly/index.ts

@@ -0,0 +1,6 @@
+{
+  "extends": "assemblyscript/std/assembly.json",
+  "include": [
+    "./**/*.ts"
+  ]
+}

test/targets/assembly/tsconfig.json

@@ -0,0 +1,5 @@
+{
+  "debug": false,
+  "optimize": true,
+  "binaryFile": "out/main.wasm"
+}

test/targets/expected.json

@@ -0,0 +1,7 @@
+{
+  "name": "main",
+  "license": "MIT",
+  "scripts": {
+    "test": "ts-node .. --target debug -- --showConfig"
+  }
+}