fix(cli): fix CI command hanging due to process.exit calls (#1149)

* fix(cli): remove posthog shutdown to prevent hanging

- Extract createPosthogClient() as a reusable function
- Remove posthog.shutdown() call that was causing process to hang
- PostHog with flushAt:1 sends events immediately, shutdown not needed

* chore(cli): upgrade posthog-node to 5.8.1

- Upgrade from 5.5.1 to 5.8.1 for better stability
- Latest version may have improved handling of process exits

* fix(cli): replace process.exit with exceptions in commands

- Replace process.exit(1) with throw new Error() in i18n command
- Replace process.exit(1) with throw error in run command
- Remove exitGracefully() calls as no longer needed
- This fixes CI command integration where process.exit() was terminating
  the parent process instead of returning control

* style: apply prettier formatting

* docs: add changeset for CI command fix

* chore: prettier

Author: maxprilutskiy

.changeset/fix-ci-command-hanging.md

@@ -0,0 +1,10 @@
+---
+"lingo.dev": patch
+---
+
+Fix CI command hanging due to process.exit calls
+
+- Remove PostHog shutdown() call that was causing process to hang
+- Replace process.exit() with proper exception throwing in i18n and run commands
+- Upgrade posthog-node from 5.5.1 to 5.8.1 for better stability
+- This fixes the CI command integration where process.exit() was terminating the parent process instead of returning control

packages/cli/README.md

@@ -35,7 +35,6 @@
   </a>
 </p>
 
-
 ---
 
 ## Meet the Compiler 🆕

packages/cli/package.json

@@ -182,7 +182,7 @@
     "p-limit": "^6.2.0",
     "php-array-reader": "^2.1.2",
     "plist": "^3.1.0",
-    "posthog-node": "^5.5.1",
+    "posthog-node": "^5.8.1",
     "prettier": "^3.4.2",
     "react": "^18.3.1",
     "rehype-stringify": "^10.0.1",

packages/cli/src/cli/cmd/i18n.ts

@@ -24,7 +24,6 @@ import createProcessor from "../processor";
 import { withExponentialBackoff } from "../utils/exp-backoff";
 import trackEvent from "../utils/observability";
 import { createDeltaProcessor } from "../utils/delta";
-import { exitGracefully } from "../utils/exit-gracefully";
 
 export default new Command()
   .command("i18n")
@@ -117,7 +116,7 @@ export default new Command()
         ora.succeed(`Authenticated as ${auth.email}`);
       }
 
-      trackEvent(authId, "cmd.i18n.start", {
+      await trackEvent(authId, "cmd.i18n.start", {
         i18nConfig,
         flags,
       });
@@ -143,7 +142,9 @@ export default new Command()
           ora.fail(
             "No buckets found. All buckets were filtered out by --file option.",
           );
-          process.exit(1);
+          throw new Error(
+            "No buckets found. All buckets were filtered out by --file option.",
+          );
         } else {
           ora.info(`\x1b[36mProcessing only filtered buckets:\x1b[0m`);
           buckets.map((bucket: any) => {
@@ -296,7 +297,9 @@ export default new Command()
             `Localization data has changed; please update i18n.lock or run without --frozen.`,
           );
           ora.fail(`  Details: ${message}`);
-          process.exit(1);
+          throw new Error(
+            `Localization data has changed; please update i18n.lock or run without --frozen. Details: ${message}`,
+          );
         } else {
           ora.succeed("No lockfile updates required.");
         }
@@ -499,25 +502,23 @@ export default new Command()
       console.log();
       if (!hasErrors) {
         ora.succeed("Localization completed.");
-        trackEvent(authId, "cmd.i18n.success", {
+        await trackEvent(authId, "cmd.i18n.success", {
           i18nConfig,
           flags,
         });
       } else {
         ora.warn("Localization completed with errors.");
-        trackEvent(authId || "unknown", "cmd.i18n.error", {
+        await trackEvent(authId || "unknown", "cmd.i18n.error", {
           flags,
         });
       }
-      exitGracefully();
     } catch (error: any) {
       ora.fail(error.message);
 
-      trackEvent(authId || "unknown", "cmd.i18n.error", {
+      await trackEvent(authId || "unknown", "cmd.i18n.error", {
         flags,
         error,
       });
-      process.exit(1);
     }
   });
 

packages/cli/src/cli/cmd/run/index.ts

@@ -14,7 +14,6 @@ import {
 } from "../../utils/ui";
 import trackEvent from "../../utils/observability";
 import { determineAuthId } from "./_utils";
-import { exitGracefully } from "../../utils/exit-gracefully";
 
 export default new Command()
   .command("run")
@@ -92,7 +91,7 @@ export default new Command()
 
       authId = await determineAuthId(ctx);
 
-      trackEvent(authId, "cmd.run.start", {
+      await trackEvent(authId, "cmd.run.start", {
         config: ctx.config,
         flags: ctx.flags,
       });
@@ -113,13 +112,12 @@ export default new Command()
         await watch(ctx);
       }
 
-      trackEvent(authId, "cmd.run.success", {
+      await trackEvent(authId, "cmd.run.success", {
         config: ctx.config,
         flags: ctx.flags,
       });
-      exitGracefully();
     } catch (error: any) {
-      trackEvent(authId || "unknown", "cmd.run.error", {});
-      process.exit(1);
+      await trackEvent(authId || "unknown", "cmd.run.error", {});
+      throw error;
     }
   });

packages/cli/src/cli/utils/observability.ts

@@ -1,6 +1,20 @@
 import pkg from "node-machine-id";
 const { machineIdSync } = pkg;
 
+export async function createPosthogClient() {
+  const { PostHog } = await import("posthog-node");
+  const posthog = new PostHog(
+    "phc_eR0iSoQufBxNY36k0f0T15UvHJdTfHlh8rJcxsfhfXk",
+    {
+      host: "https://eu.i.posthog.com",
+      flushAt: 1,
+      flushInterval: 0,
+    },
+  );
+
+  return posthog;
+}
+
 export default async function trackEvent(
   distinctId: string | null | undefined,
   event: string,
@@ -13,15 +27,7 @@ export default async function trackEvent(
   try {
     const actualId = distinctId || `device-${machineIdSync()}`;
 
-    const { PostHog } = await import("posthog-node");
-    const posthog = new PostHog(
-      "phc_eR0iSoQufBxNY36k0f0T15UvHJdTfHlh8rJcxsfhfXk",
-      {
-        host: "https://eu.i.posthog.com",
-        flushAt: 1,
-        flushInterval: 0,
-      },
-    );
+    const posthog = await createPosthogClient();
 
     await posthog.capture({
       distinctId: actualId,
@@ -34,8 +40,6 @@ export default async function trackEvent(
         },
       },
     });
-
-    await posthog.shutdown();
   } catch (error) {
     if (process.env.DEBUG) {
       console.error(error);