Merge pull request #6 from MFB-Technologies-Inc/bugfix/5-uncaught-exception

5: improve error handling and logging

Author: ethan-mfb

changes/bugfix-5-uncaught-exception_2025-3-18-20-22-30.json

@@ -0,0 +1,9 @@
+{
+  "changes": [
+    {
+      "packageName": "@mfbtech/changelog-generator",
+      "comment": "Improve error messages and logging",
+      "type": "PATCH"
+    }
+  ]
+}

src/index.ts

@@ -15,18 +15,34 @@ import {
 } from "./constants.js"
 import { publish } from "./commands/publish.js"
 
+// Add global handler for unhandled promise rejections
+process.on("unhandledRejection", (reason, promise) => {
+  console.error("Unhandled promise rejection:", promise, "reason: ", reason)
+  process.exit(1)
+})
+
 const ccgProgram = new Command("ccg")
 
 ccgProgram
   .command(ChangeCommandName)
   .description(ChangeCommandDescription)
   .option(ChangeCommandOptionFlag.verify, ChangeCommandOptionDescription.verify)
-  .action(change)
+  .action(args => {
+    change(args).catch(e => {
+      console.error(e)
+      process.exit(1)
+    })
+  })
 
 ccgProgram
   .command(PublishCommandName)
   .description(PublishCommandDescription)
   .option(PublishCommandOptionFlag.apply, PublishCommandOptionDescription.apply)
-  .action(publish)
+  .action(args => {
+    publish(args).catch(e => {
+      console.error(e)
+      process.exit(1)
+    })
+  })
 
 ccgProgram.parse(process.argv)

src/services/processes.ts

@@ -12,25 +12,60 @@ export async function runCommand(args: {
   return new Promise((resolve, reject) => {
     let result = ""
     command.on("error", err => {
-      reject("command failed: failed to start the subprocess: " + err.message)
+      reject(
+        new Error(
+          "command failed: failed to start the subprocess: " +
+            err.message +
+            "\ncommand: " +
+            args.command +
+            " " +
+            args.args?.join(" ")
+        )
+      )
     })
     command.on("close", code => {
       if (code !== 0) {
-        reject("command failed with exit code " + code)
+        reject(
+          new Error(
+            "command failed with exit code " +
+              code +
+              "\ncommand: " +
+              args.command +
+              " " +
+              args.args?.join(" ")
+          )
+        )
       } else {
         resolve(result)
       }
     })
     command.stdout.on("data", (data: unknown) => {
       if (!hasToStringMethod(data)) {
-        reject("unknown response data: " + JSON.stringify(data))
+        reject(
+          new Error(
+            "unknown response data: " +
+              JSON.stringify(data) +
+              "\ncommand: " +
+              args.command +
+              " " +
+              args.args?.join(" ")
+          )
+        )
         command.kill(1)
         return
       }
       result += data.toString().trim()
     })
     command.stderr.on("data", data => {
-      reject(`command failed: ${data}`)
+      reject(
+        new Error(
+          `command failed: ${data}` +
+            "\ncommand: " +
+            args.command +
+            " " +
+            args.args?.join(" ")
+        )
+      )
       command.kill(1)
     })
   })