@canva/cli 1.9.0 → 1.10.0

package/templates/dam/utils/backend/jwt_middleware/jwt_middleware.ts

@@ -1,8 +1,6 @@
 /* eslint-disable no-console */
 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";
 import jwt from "jsonwebtoken";
 import { JwksClient, SigningKeyNotFoundError } from "jwks-rsa";
 
@@ -115,7 +113,7 @@ export function createJwtMiddleware(
         userId: payload.userId,
       };
 
-      next();
+      return next();
     } catch (e) {
       if (e instanceof JWTAuthorizationError) {
         return sendUnauthorizedResponse(res, e.message);
@@ -136,7 +134,7 @@ export function createJwtMiddleware(
         return sendUnauthorizedResponse(res, "Token expired");
       }
 
-      next(e);
+      return next(e);
     }
   };
 }

package/templates/dam/webpack.config.ts

@@ -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/data_connector/package.json

@@ -42,8 +42,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/data_connector/src/utils/data_table.ts

@@ -12,7 +12,8 @@ import type { APIResponseItem } from "src/api";
 export interface DataTableColumn<T extends APIResponseItem> {
   label: string;
   getValue: keyof T | ((result: T) => boolean | string | number | Date);
-  toCell: (value) => DataTableCell;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  toCell: (value: any) => DataTableCell;
 }
 
 export function toDataTable<T extends APIResponseItem>(

package/templates/data_connector/src/utils/tests/data_table.test.ts

@@ -92,8 +92,8 @@ describe("data table utils", () => {
     const result = toDataTable(testItems, columns, 10);
 
     // Check the custom column values
-    expect(result.rows[0].cells[5].value).toBe("Test Item 1 (42)");
-    expect(result.rows[1].cells[5].value).toBe("Test Item 2 (0)");
+    expect(result.rows[0]?.cells[5]?.value).toBe("Test Item 1 (42)");
+    expect(result.rows[1]?.cells[5]?.value).toBe("Test Item 2 (0)");
   });
 
   test("cell formatter functions create correct cell structures", () => {

package/templates/data_connector/tsconfig.json

@@ -28,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,
@@ -44,8 +47,7 @@
     "./utils/**/*",
     "./scripts/**/*",
     "./declarations/declarations.d.ts",
-    "./styles/**/*",
-    "./node_modules/@types/**/*"
+    "./styles/**/*"
   ],
   "ts-node": {
     "compilerOptions": {

package/templates/data_connector/webpack.config.ts

@@ -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/gen_ai/backend/routers/image.ts

@@ -151,7 +151,7 @@ export const createImageRouter = () => {
     // Add the job to the jobQueue along with the timeoutId
     jobQueue.push({ jobId, prompt, timeoutId });
 
-    res.status(200).send({
+    return res.status(200).send({
       jobId,
     });
   });
@@ -208,15 +208,13 @@ export const createImageRouter = () => {
     }
 
     const index = jobQueue.findIndex((job) => job.jobId === jobId);
-    if (index !== -1) {
+    const job = jobQueue[index];
+    if (index !== -1 && job) {
       cancelledJobs.push({ jobId });
       // If the job is found, remove it from the jobQueue
-      const { timeoutId } = jobQueue[index];
       jobQueue.splice(index, 1);
       // Also clear the timeout associated with this job if it exists
-      if (timeoutId) {
-        clearTimeout(timeoutId);
-      }
+      clearTimeout(job.timeoutId);
       return res.status(200).send("Job successfully cancelled.");
     }
 

package/templates/gen_ai/package.json

@@ -44,9 +44,9 @@
     "@pmmmwh/react-refresh-webpack-plugin": "0.6.1",
     "@svgr/webpack": "8.1.0",
     "@testing-library/react": "16.3.0",
+    "@types/cors": "2.8.19",
     "@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/gen_ai/src/components/image_grid.tsx

@@ -34,7 +34,9 @@ export const ImageGrid = () => {
   ) => {
     const parentNode = event.currentTarget.parentElement;
     try {
-      parentNode?.classList.add(styles.hidden);
+      if (styles.hidden) {
+        parentNode?.classList.add(styles.hidden);
+      }
 
       await ui.startDragToPoint(event, {
         type: "image",
@@ -50,7 +52,9 @@ export const ImageGrid = () => {
         },
       });
     } finally {
-      parentNode?.classList.remove(styles.hidden);
+      if (styles.hidden) {
+        parentNode?.classList.remove(styles.hidden);
+      }
     }
   };
 

package/templates/gen_ai/src/components/prompt_input.tsx

@@ -59,8 +59,11 @@ const generateExamplePrompt = (currentPrompt: string): string => {
   const MAX_ATTEMPTS = 3;
 
   while (currentPrompt === newPrompt && attempts < MAX_ATTEMPTS) {
-    newPrompt =
+    const randomPrompt =
       examplePrompts[Math.floor(Math.random() * examplePrompts.length)];
+    if (randomPrompt) {
+      newPrompt = randomPrompt;
+    }
     attempts++;
   }
 

package/templates/gen_ai/tsconfig.json

@@ -28,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,
@@ -44,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

@@ -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 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

@@ -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

@@ -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/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

@@ -28,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,
@@ -44,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

@@ -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";