Merge pull request #10 from getappmap/fix/handle-exit-via-signal

fix: Handle exit via signal
getappmap · Apr 14, 2023 · e55a192 · e55a192
2 parents 9860cfa + a90533d
commit e55a192
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 5 deletions.
diff --git a/.github/workflows/appmap-archive.yml b/.github/workflows/appmap-archive.yml
@@ -19,6 +19,9 @@ jobs:
         run: yarn install
       - name: Install and configure AppMap
         uses: getappmap/[email protected]
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          verbose: true
       - name: Build AppMaps
         run: yarn appmap
       - name: Archive AppMaps

diff --git a/.github/workflows/preflight.yml b/.github/workflows/preflight.yml
@@ -21,6 +21,14 @@ jobs:
         run: yarn install
       - name: Install and configure AppMap
         uses: getappmap/[email protected]
+          with:
+            github-token: ${{ secrets.GITHUB_TOKEN }}
+            verbose: true
+      - name: Commit AppMap configuration changes
+        uses: EndBug/add-and-commit@v7
+        with:
+          add: .
+          message: 'chore: Update AppMap installation'
       - name: Build
         run: yarn build
       - name: Make AppMaps

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/executeCommand.ts b/src/executeCommand.ts
@@ -38,8 +38,9 @@ export function executeCommand(
     });
   }
   return new Promise<string>((resolve, reject) => {
-    command.addListener('exit', code => {
-      if (code === 0) {
+    command.addListener('exit', (code, signal) => {
+      if (signal || code === 0) {
+        if (signal) log(LogLevel.Info, `Command killed by signal ${signal}`);
         resolve(result.join(''));
       } else {
         if (!printCommand) log(LogLevel.Warn, commandString);