feat: print transform errors and warnings

lib/formatMessages.ts

@@ -0,0 +1,69 @@
+import {
+ BuildFailure,
+ BuildResult,
+ formatMessagesSync,
+ TransformFailure,
+ TransformResult,
+} from "esbuild";
+import { WriteStream } from "tty";
+
+export function printBuildErrors(
+ failure: TransformFailure | BuildFailure,
+ {
+ dest = process.stdout,
+ color = true,
+ prefix = "",
+ }: {
+ dest?: WriteStream;
+ color?: boolean;
+ prefix?: string;
+ } = {}
+): void {
+ const errors = formatMessagesSync(failure.errors, {
+ kind: "error",
+ color,
+ });
+
+ if (errors.length > 0) {
+ dest.write(`${prefix}Errors:\n\n`);
+ dest.write(errors.join("\n\n"));
+ }
+}
+
+export function printBuildWarnings(
+ failure: TransformFailure | BuildFailure | TransformResult | BuildResult,
+ {
+ dest = process.stdout,
+ color = true,
+ prefix = "",
+ }: {
+ dest?: WriteStream;
+ color?: boolean;
+ prefix?: string;
+ } = {}
+): void {
+ const warnings = formatMessagesSync(failure.warnings, {
+ kind: "warning",
+ color,
+ });
+
+ if (warnings.length > 0) {
+ dest.write(`${prefix}Warning:\n\n`);
+ dest.write(warnings.join("\n\n"));
+ }
+}
+
+export function printBuildMessages(
+ failure: TransformFailure | BuildFailure | TransformResult | BuildResult,
+ props: {
+ dest?: WriteStream;
+ color?: boolean;
+ prefix?: string;
+ } = {}
+): void {
+ printBuildWarnings(failure, props);
+
+ if ((failure as BuildFailure).errors) {
+ printBuildErrors(failure as BuildFailure, props);
+ }
+}

lib/inline-code.ts

@@ -1,18 +1,26 @@
 import { InlineCode } from "@aws-cdk/aws-lambda";
 import { transformSync, TransformOptions, Loader } from "esbuild";
+import { printBuildMessages } from "./formatMessages";
 
 abstract class BaseInlineCode extends InlineCode {
  public constructor(
  code: string,
  loader: Loader,
  transformOptions: TransformOptions = {}
  ) {
- const transformedCode = transformSync(code, {
- loader,
- ...transformOptions,
- });
+ try {
+ const transformedCode = transformSync(code, {
+ loader,
+ ...transformOptions,
+ });
+ printBuildMessages(transformedCode, { prefix: "Transform " });
 
- super(transformedCode.code);
+ super(transformedCode.code);
+ } catch (error) {
+ printBuildMessages(error, { prefix: "Transform " });
+
+ throw new Error("Failed to transform InlineCode");
+ }
  }
 }
 

test/inline-code.test.ts

@@ -1,12 +1,22 @@
 import { Stack } from "@aws-cdk/core";
+import { mocked } from "ts-jest/utils";
 import {
  InlineJavaScriptCode,
  InlineJsxCode,
  InlineTsxCode,
  InlineTypeScriptCode,
 } from "../lib";
+import { printBuildMessages } from "../lib/formatMessages";
+
+jest.mock("../lib/formatMessages", () => ({
+ printBuildMessages: jest.fn(),
+}));
 
 describe("inline-code", () => {
+ beforeEach(() => {
+ mocked(printBuildMessages).mockReset();
+ });
+
  describe("given some js code", () => {
  it("should transform the code", () => {
  const code = new InlineJavaScriptCode(
@@ -56,4 +66,15 @@ describe("inline-code", () => {
  );
  });
  });
+
+ describe("given some broken ts code", () => {
+ it("should display errors and warnings", () => {
+ expect(() => {
+ const code = new InlineTypeScriptCode("let : d ===== 1");
+ code.bind(new Stack());
+ }).toThrowError("Failed to transform InlineCode");
+
+ expect(mocked(printBuildMessages)).toHaveBeenCalledTimes(1);
+ });
+ });
 });