chore(cli): migrate @fern-api/cli-logger to use CliError

packages/cli/cli-logger/src/logErrorMessage.ts

@@ -1,7 +1,9 @@
 import { Logger, LogLevel } from "@fern-api/logger";
-import { FernCliError } from "@fern-api/task-context";
+import { TaskAbortSignal } from "@fern-api/task-context";
 import chalk from "chalk";
 
+const USE_NODE_18_OR_ABOVE_MESSAGE = "The Fern CLI requires Node 18+ or above.";
+
 export function logErrorMessage({
     message,
     error,
@@ -20,7 +22,7 @@ export function logErrorMessage({
     }
 
     // thrower is responsible for logging, so we don't need to log the error's message too
-    if (error instanceof FernCliError) {
+    if (error instanceof TaskAbortSignal) {
         return;
     }
 
@@ -42,6 +44,10 @@ function convertErrorToString(error: unknown): string | undefined {
         return error;
     }
     if (error instanceof Error) {
+        if ((error as Error)?.message?.includes("globalThis")) {
+            return USE_NODE_18_OR_ABOVE_MESSAGE;
+        }
+
         return error.message;
     }
     return undefined;

packages/cli/cli-v2/src/__test__/ApiChecker.test.ts

@@ -26,7 +26,7 @@ describe("ApiChecker", () => {
             }
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -53,7 +53,7 @@ describe("ApiChecker", () => {
             }
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -71,7 +71,7 @@ describe("ApiChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -89,7 +89,7 @@ describe("ApiChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -110,7 +110,7 @@ describe("ApiChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -128,7 +128,7 @@ describe("ApiChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 
@@ -146,7 +146,7 @@ describe("ApiChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new ApiChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 cliVersion: "0.0.0"
             });
 

packages/cli/cli-v2/src/__test__/DocsChecker.test.ts

@@ -29,7 +29,7 @@ describe("DocsChecker", () => {
             const cwd = AbsoluteFilePath.of(join(__dirname, "fixtures/simple-api"));
 
             const checker = new DocsChecker({
-                context: createTestContext({ cwd })
+                context: await createTestContext({ cwd })
             });
 
             const result = await checker.check({ workspace });
@@ -49,7 +49,7 @@ describe("DocsChecker", () => {
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new DocsChecker({
-                context: createTestContext({ cwd: testDir })
+                context: await createTestContext({ cwd: testDir })
             });
 
             const result = await checker.check({ workspace });
@@ -69,7 +69,7 @@ describe("DocsChecker", () => {
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new DocsChecker({
-                context: createTestContext({ cwd: testDir })
+                context: await createTestContext({ cwd: testDir })
             });
 
             const result = await checker.check({ workspace });
@@ -84,7 +84,7 @@ describe("DocsChecker", () => {
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new DocsChecker({
-                context: createTestContext({ cwd: testDir })
+                context: await createTestContext({ cwd: testDir })
             });
 
             const resultStrict = await checker.check({ workspace, strict: true });
@@ -133,7 +133,7 @@ docs:
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new DocsChecker({
-                context: createTestContext({ cwd: testDir })
+                context: await createTestContext({ cwd: testDir })
             });
 
             const result = await checker.check({ workspace, strict: true });

packages/cli/cli-v2/src/__test__/LegacyApiSpecAdapter.test.ts

@@ -1,17 +1,23 @@
 import type { OpenAPISpec, OpenRPCSpec } from "@fern-api/api-workspace-commons";
 import { generatorsYml } from "@fern-api/configuration";
 import { AbsoluteFilePath } from "@fern-api/fs-utils";
-import { describe, expect, it } from "vitest";
+import { beforeEach, describe, expect, it } from "vitest";
 import { LegacyApiSpecAdapter } from "../api/adapter/LegacyApiSpecAdapter.js";
 import type { AsyncApiSpec } from "../api/config/AsyncApiSpec.js";
 import type { OpenApiSpec } from "../api/config/OpenApiSpec.js";
 import type { OpenRpcSpec } from "../api/config/OpenRpcSpec.js";
+import type { Context } from "../context/Context.js";
 import { createTestContext } from "./utils/createTestContext.js";
 
 describe("LegacyApiSpecAdapter", () => {
     const cwd = AbsoluteFilePath.of("/test/path");
-    const context = createTestContext({ cwd });
-    const adapter = new LegacyApiSpecAdapter({ context });
+    let context: Context;
+    let adapter: LegacyApiSpecAdapter;
+
+    beforeEach(async () => {
+        context = await createTestContext({ cwd });
+        adapter = new LegacyApiSpecAdapter({ context });
+    });
 
     describe("convertOpenApiSettings", () => {
         it("returns undefined when settings are not provided", () => {

packages/cli/cli-v2/src/__test__/LegacyFernWorkspaceAdapter.test.ts

@@ -27,7 +27,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
 
         it("adapts Fern spec to FernWorkspace using LazyFernWorkspace", async () => {
             const { cwd, apiDefinition } = await loadApiDefinition("fern-definition");
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
@@ -44,7 +44,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
 
         it("loads types and services from Fern definition files", async () => {
             const { cwd, apiDefinition } = await loadApiDefinition("fern-definition");
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const fernWorkspace = await adapter.adapt(apiDefinition);
             const definitionFiles = fernWorkspace.definition.namedDefinitionFiles;
@@ -82,7 +82,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
                 expect(isOpenApiSpec(firstSpec)).toBe(true);
             }
 
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
             expect(fernWorkspace.definition).toBeDefined();
@@ -103,7 +103,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
 
         it("adapts Conjure spec to FernWorkspace using ConjureWorkspace", async () => {
             const { cwd, apiDefinition } = await loadApiDefinition("conjure-definition");
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
@@ -120,7 +120,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
                 expect(spec.conjure.toString()).toContain("conjure-definition/conjure");
             }
 
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
             expect(fernWorkspace.absoluteFilePath.toString()).toBe(cwd.toString());
@@ -130,7 +130,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
     describe("Multiple OpenAPI Specs", () => {
         it("allows multiple OpenAPI specs to be mixed together", async () => {
             const cwd = AbsoluteFilePath.of(join(FIXTURES_DIR, "simple-api"));
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const openApiSpec1: OpenApiSpec = {
                 openapi: AbsoluteFilePath.of(join(FIXTURES_DIR, "simple-api", "openapi.yml"))
@@ -173,7 +173,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
 
         it("override auth scheme", async () => {
             const { cwd, apiDefinition } = await loadApiDefinition("api-config-override");
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
@@ -186,7 +186,7 @@ describe("LegacyFernWorkspaceAdapter", () => {
 
         it("override environments", async () => {
             const { cwd, apiDefinition } = await loadApiDefinition("api-config-override");
-            const adapter = createAdapter(cwd);
+            const adapter = await createAdapter(cwd);
 
             const fernWorkspace = await adapter.adapt(apiDefinition);
             expect(fernWorkspace).toBeDefined();
@@ -207,8 +207,8 @@ describe("LegacyFernWorkspaceAdapter", () => {
 /**
  * Creates an adapter for testing with the given cwd.
  */
-function createAdapter(cwd: AbsoluteFilePath): LegacyFernWorkspaceAdapter {
-    const context = createTestContext({ cwd });
+async function createAdapter(cwd: AbsoluteFilePath): Promise<LegacyFernWorkspaceAdapter> {
+    const context = await createTestContext({ cwd });
     const task = createMockTask();
     return new LegacyFernWorkspaceAdapter({
         context,

packages/cli/cli-v2/src/__test__/SdkChecker.test.ts

@@ -34,7 +34,7 @@ describe("SdkChecker", () => {
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -61,7 +61,7 @@ api:
             const workspace = await loadTempWorkspace(testDir);
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd: testDir }),
+                context: await createTestContext({ cwd: testDir }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -98,7 +98,7 @@ sdks:
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new SdkChecker({
-                context: createTestContext({ cwd: testDir }),
+                context: await createTestContext({ cwd: testDir }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -113,7 +113,7 @@ sdks:
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -146,7 +146,7 @@ sdks:
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new SdkChecker({
-                context: createTestContext({ cwd: testDir }),
+                context: await createTestContext({ cwd: testDir }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -163,7 +163,7 @@ sdks:
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -198,7 +198,7 @@ sdks:
 
             const workspace = await loadTempWorkspace(testDir);
             const checker = new SdkChecker({
-                context: createTestContext({ cwd: testDir }),
+                context: await createTestContext({ cwd: testDir }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -213,7 +213,7 @@ sdks:
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 versionChecker: NOOP_VERSION_CHECKER
             });
 
@@ -229,7 +229,7 @@ sdks:
             const workspace = await loadWorkspace("simple-api");
 
             const checker = new SdkChecker({
-                context: createTestContext({ cwd }),
+                context: await createTestContext({ cwd }),
                 versionChecker: async ({ target }) => ({
                     violation: {
                         severity: "warning",

packages/cli/cli-v2/src/__test__/check.test.ts

@@ -51,7 +51,7 @@ paths: {}
 `
         );
 
-        const { context, getStdout, getStderr } = createTestContextWithCapture({ cwd: testDir });
+        const { context, getStdout, getStderr } = await createTestContextWithCapture({ cwd: testDir });
         const cmd = new CheckCommand();
 
         // The sdk checker will find the defaultGroup violation, which causes an error exit.
@@ -129,7 +129,7 @@ paths: {}
 `
         );
 
-        const { context, getStdout, getStderr } = createTestContextWithCapture({ cwd: testDir });
+        const { context, getStdout, getStderr } = await createTestContextWithCapture({ cwd: testDir });
         const cmd = new SdkCheckCommand();
 
         try {