@canva/cli 1.8.0 → 1.10.0

package/templates/gen_ai/tsconfig.json

@@ -18,7 +18,8 @@
     "experimentalDecorators": true,
     "importHelpers": true,
     "noImplicitOverride": true,
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
+    "esModuleInterop": true,
     "rootDir": ".",
     "outDir": "dist",
     "strict": true,
@@ -27,7 +28,10 @@
     "sourceMap": true,
     "inlineSources": true,
     "module": "ESNext",
-    "noImplicitAny": false,
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "noFallthroughCasesInSwitch": true,
+    "noUncheckedIndexedAccess": true,
     "removeComments": true,
     "preserveConstEnums": true,
     "allowSyntheticDefaultImports": true,
@@ -43,8 +47,7 @@
     "./utils/**/*",
     "./scripts/**/*",
     "./declarations/declarations.d.ts",
-    "./styles/**/*",
-    "./node_modules/@types/**/*"
+    "./styles/**/*"
   ],
   "ts-node": {
     "compilerOptions": {

package/templates/gen_ai/utils/backend/base_backend/create.ts

@@ -1,10 +1,10 @@
 /* eslint-disable no-console */
 import debug from "debug";
-import * as express from "express";
+import express from "express";
 import type { NextFunction, Request, Response } from "express";
-import * as fs from "fs";
-import * as http from "http";
-import * as https from "https";
+import fs from "fs";
+import http from "http";
+import https from "https";
 
 const serverDebug = debug("server");
 
@@ -41,12 +41,12 @@ export function createBaseServer(router: express.Router): BaseServer {
   app.disable("x-powered-by");
 
   // Health check endpoint
-  app.get("/healthz", (req, res: Response) => {
+  app.get("/healthz", (req, res) => {
     res.sendStatus(200);
   });
 
   // logging middleware
-  app.use((req: Request, res: Response, next: NextFunction) => {
+  app.use((req, _res, next) => {
     serverDebug(`${new Date().toISOString()}: ${req.method} ${req.url}`);
     next();
   });
@@ -62,7 +62,7 @@ export function createBaseServer(router: express.Router): BaseServer {
   });
 
   // default error handler
-  app.use((err, req, res, next) => {
+  app.use((err: Error, _req: Request, res: Response, _next: NextFunction) => {
     console.error(err.stack);
     res.status(500).send({
       error: "something went wrong",

package/templates/gen_ai/utils/backend/bearer_middleware/bearer_middleware.ts

@@ -1,8 +1,6 @@
 /* eslint-disable no-console */
-import * as debug from "debug";
+import debug from "debug";
 import type { NextFunction, Request, Response } from "express";
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
-import Express from "express-serve-static-core";
 
 /**
  * Prefix your start command with `DEBUG=express:middleware:bearer` to enable debug logging
@@ -47,13 +45,13 @@ export function createBearerMiddleware(
 
       req["user_id"] = user;
 
-      next();
+      return next();
     } catch (e) {
       if (e instanceof AuthorizationError) {
         return sendUnauthorizedResponse(res, e.message);
       }
 
-      next(e);
+      return next(e);
     }
   };
 }

package/templates/gen_ai/webpack.config.ts

@@ -1,9 +1,9 @@
 import type { Configuration } from "webpack";
 import { DefinePlugin, optimize } from "webpack";
-import * as path from "path";
-import * as TerserPlugin from "terser-webpack-plugin";
+import path from "path";
+import TerserPlugin from "terser-webpack-plugin";
 import { transform } from "@formatjs/ts-transformer";
-import * as chalk from "chalk";
+import chalk from "chalk";
 import { config } from "dotenv";
 import { Configuration as DevServerConfiguration } from "webpack-dev-server";
 
@@ -196,7 +196,7 @@ function buildDevConfig(options?: DevConfig): {
     return {};
   }
 
-  const { port, enableHmr, appOrigin, appId, enableHttps, certFile, keyFile } =
+  const { port, enableHmr, appOrigin, enableHttps, certFile, keyFile } =
     options;
   const host = "localhost";
 

package/templates/hello_world/package.json

@@ -20,7 +20,7 @@
   },
   "dependencies": {
     "@canva/app-i18n-kit": "^1.1.1",
-    "@canva/app-ui-kit": "^5.2.0",
+    "@canva/app-ui-kit": "^5.2.1",
     "@canva/asset": "^2.2.1",
     "@canva/design": "^2.7.3",
     "@canva/error": "^2.1.0",
@@ -39,8 +39,8 @@
     "@pmmmwh/react-refresh-webpack-plugin": "0.6.1",
     "@svgr/webpack": "8.1.0",
     "@testing-library/react": "16.3.0",
+    "@types/debug": "4.1.12",
     "@types/express": "4.17.21",
-    "@types/express-serve-static-core": "5.0.7",
     "@types/jest": "29.5.14",
     "@types/jsonwebtoken": "9.0.10",
     "@types/node": "20.19.2",

package/templates/hello_world/scripts/copy_env.ts

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 /* eslint-disable no-console */
-import * as fs from "fs";
-import * as path from "path";
+import fs from "fs";
+import path from "path";
 
 const envPath = path.resolve(__dirname, "..", ".env");
 const templatePath = path.resolve(__dirname, "..", ".env.template");

package/templates/hello_world/scripts/ssl/ssl.ts

@@ -1,7 +1,7 @@
-import * as crypto from "crypto";
-import * as fs from "fs/promises";
+import crypto from "crypto";
+import fs from "fs/promises";
 import { pki } from "node-forge";
-import * as path from "path";
+import path from "path";
 
 const SSL_CERT_DIR = path.resolve(process.cwd(), "..", "..", ".ssl");
 const CERT_FILE = path.resolve(SSL_CERT_DIR, "certificate.pem");

package/templates/hello_world/scripts/start/app_runner.ts

@@ -1,12 +1,13 @@
 /* eslint-disable no-console */
 import { generatePreviewUrl } from "@canva/cli";
-import * as ngrok from "@ngrok/ngrok";
-import * as chalk from "chalk";
-import * as Table from "cli-table3";
-import * as nodemon from "nodemon";
+import ngrok from "@ngrok/ngrok";
+import chalk from "chalk";
+import Table from "cli-table3";
+import nodemon from "nodemon";
 import open from "open";
-import * as webpack from "webpack";
-import * as WebpackDevServer from "webpack-dev-server";
+import os from "os";
+import webpack from "webpack";
+import WebpackDevServer from "webpack-dev-server";
 import { buildConfig } from "../../webpack.config";
 import type { Certificate } from "../ssl/ssl";
 import { createOrRetrieveCertificate } from "../ssl/ssl";
@@ -18,6 +19,22 @@ export const errorChalk = chalk.bgRed.bold;
 export const highlightChalk = chalk.greenBright.bold;
 export const linkChalk = chalk.cyan;
 
+/**
+ * Returns the appropriate modifier key text based on the user's operating system.
+ * @returns "cmd" for macOS, "ctrl" for Windows and Linux
+ */
+export function getModifierKey(): string {
+  const platform = os.platform();
+  switch (platform) {
+    case "darwin": // macOS
+      return "cmd";
+    case "win32": // Windows
+      return "ctrl";
+    default: // Linux and others
+      return "ctrl";
+  }
+}
+
 export class AppRunner {
   async run(ctx: Context) {
     console.log(
@@ -189,9 +206,14 @@ export class AppRunner {
       return;
     }
 
+    const modifierKey = getModifierKey();
+
     table.push([
       previewCellHeader,
-      { content: "Preview URL", href: generatePreviewResult.data },
+      {
+        content: `Preview URL (${modifierKey} + click)`,
+        href: generatePreviewResult.data,
+      },
     ]);
 
     if (openPreview) {

package/templates/hello_world/scripts/start/context.ts

@@ -1,5 +1,5 @@
-import * as fs from "fs";
-import * as path from "path";
+import fs from "fs";
+import path from "path";
 
 interface CliArgs {
   example?: string;

package/templates/hello_world/scripts/start/start.ts

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import * as yargs from "yargs";
+import yargs from "yargs";
 import { hideBin } from "yargs/helpers";
 import { AppRunner } from "./app_runner";
 import { Context } from "./context";

package/templates/hello_world/src/tests/app.tests.tsx

@@ -73,7 +73,7 @@ describe("Hello World Tests", () => {
     expect(mockRequestOpenExternalUrl).toHaveBeenCalled();
 
     // assert that the requestOpenExternalUrl function was called with the expected arguments
-    expect(mockRequestOpenExternalUrl.mock.calls[0][0]).toEqual({
+    expect(mockRequestOpenExternalUrl.mock.calls[0]?.[0]).toEqual({
       url: DOCS_URL,
     });
   });

package/templates/hello_world/tsconfig.json

@@ -18,7 +18,8 @@
     "experimentalDecorators": true,
     "importHelpers": true,
     "noImplicitOverride": true,
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
+    "esModuleInterop": true,
     "rootDir": ".",
     "outDir": "dist",
     "strict": true,
@@ -27,7 +28,10 @@
     "sourceMap": true,
     "inlineSources": true,
     "module": "ESNext",
-    "noImplicitAny": false,
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "noFallthroughCasesInSwitch": true,
+    "noUncheckedIndexedAccess": true,
     "removeComments": true,
     "preserveConstEnums": true,
     "allowSyntheticDefaultImports": true,
@@ -43,8 +47,7 @@
     "./utils/**/*",
     "./scripts/**/*",
     "./declarations/declarations.d.ts",
-    "./styles/**/*",
-    "./node_modules/@types/**/*"
+    "./styles/**/*"
   ],
   "ts-node": {
     "compilerOptions": {

package/templates/hello_world/utils/use_add_element.ts

@@ -30,6 +30,11 @@ export const useAddElement = () => {
       } else if (features.isSupported(addElementAtCursor)) {
         return addElementAtCursor(element);
       }
+      // eslint-disable-next-line no-console
+      console.warn(
+        "Neither addElementAtPoint nor addElementAtCursor are supported",
+      );
+      return Promise.resolve();
     };
   });
 
@@ -40,6 +45,11 @@ export const useAddElement = () => {
       } else if (isSupported(addElementAtCursor)) {
         return addElementAtCursor(element);
       }
+      // eslint-disable-next-line no-console
+      console.warn(
+        "Neither addElementAtPoint nor addElementAtCursor are supported",
+      );
+      return Promise.resolve();
     };
     setAddElement(() => addElement);
   }, [isSupported]);

package/templates/hello_world/webpack.config.ts

@@ -1,9 +1,9 @@
 import type { Configuration } from "webpack";
 import { DefinePlugin, optimize } from "webpack";
-import * as path from "path";
-import * as TerserPlugin from "terser-webpack-plugin";
+import path from "path";
+import TerserPlugin from "terser-webpack-plugin";
 import { transform } from "@formatjs/ts-transformer";
-import * as chalk from "chalk";
+import chalk from "chalk";
 import { config } from "dotenv";
 import { Configuration as DevServerConfiguration } from "webpack-dev-server";
 
@@ -196,7 +196,7 @@ function buildDevConfig(options?: DevConfig): {
     return {};
   }
 
-  const { port, enableHmr, appOrigin, appId, enableHttps, certFile, keyFile } =
+  const { port, enableHmr, appOrigin, enableHttps, certFile, keyFile } =
     options;
   const host = "localhost";