fix: Tact CLI returns non-zero exit code if compilation fails (#278)

skip Node.js's exit codes
https://nodejs.org/docs/v20.12.1/api/process.html#exit-codes;

test non-zero exit code in CI

.github/workflows/tact.yml

@@ -56,6 +56,11 @@ jobs:
           then false
           fi
 
+      - name: Tact CLI non-zero exit code
+        if: runner.os != 'Windows'
+        run: |
+          ! ./bin/tact --config test/tact-cli/tact.config.json
+
       - name: Link Tact compiler
         run: |
           yarn link

CHANGELOG.md

@@ -41,6 +41,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Operation precendence for bitwise operators, equality and comparisons now matches common languages, like JavaScript: PR [#265](https://github.com/tact-lang/tact/pull/265)
 - Incorrect variable scoping in `repeat`, `while` and `until` loops: PR [#269](https://github.com/tact-lang/tact/pull/269)
 - FunC compilation errors when trying to `dump()` such types as `Cell`, `Slice`, `Builder` and `StringBuilder`: PR [#271](https://github.com/tact-lang/tact/pull/271)
+- Tact's CLI returns a non-zero exit code if compilation fails: PR [#278](https://github.com/tact-lang/tact/pull/278)
 
 ## [1.2.0] - 2024-02-29
 

bin/tact

@@ -22,14 +22,17 @@ if (!args["--config"]) {
 // Perform compilation
 (async () => {
   try {
-    await main.run({
+    const success = await main.run({
       configPath: args["--config"],
       projectNames: args["--project"] ? args["--project"] : [],
     });
+    // https://nodejs.org/docs/v20.12.1/api/process.html#exit-codes
+    if (!success) process.exit(30);
   } catch (e) {
     console.warn(
       "Internal compiler error. Please, report it to https://github.com/tact-lang/tact/issues.",
     );
     console.log(e);
+    process.exit(30);
   }
 })();

test/tact-cli/fail.tact

@@ -0,0 +1,3 @@
+contract Fail {
+    syntax_error
+}

test/tact-cli/tact.config.json

@@ -0,0 +1,10 @@
+{
+  "$schema": "http://raw.githubusercontent.com/tact-lang/tact/main/grammar/configSchema.json",
+  "projects": [
+    {
+      "name": "fail",
+      "path": "./fail.tact",
+      "output": "./output"
+    }
+  ]
+}