chore: cleanup types

Author: maxakuru

src/index.js

@@ -15,7 +15,7 @@ import { resolveConfig } from './utils/config.js';
 import handlers from './routes/index.js';
 
 /**
- * @param {import("@cloudflare/workers-types/experimental").Request} req
+ * @param {import("@cloudflare/workers-types").Request} req
  */
 async function parseData(req) {
   if (['GET', 'HEAD', 'OPTIONS'].includes(req.method)) {
@@ -41,7 +41,7 @@ async function parseData(req) {
 
 /**
  * @param {import("@cloudflare/workers-types/experimental").ExecutionContext} eCtx
- * @param {import("@cloudflare/workers-types/experimental").Request} req
+ * @param {import("@cloudflare/workers-types").Request} req
  * @param {Env} env
  * @returns {Promise<Context>}
  */
@@ -72,7 +72,7 @@ export async function makeContext(eCtx, req, env) {
 
 export default {
   /**
-   * @param {import("@cloudflare/workers-types/experimental").Request} request
+   * @param {import("@cloudflare/workers-types").Request} request
    * @param {Env} env
    * @param {import("@cloudflare/workers-types/experimental").ExecutionContext} eCtx
    * @returns {Promise<Response>}

src/routes/auth/fetch.js

@@ -14,7 +14,7 @@ import { assertAuthorization } from '../../utils/auth.js';
 import { errorResponse } from '../../utils/http.js';
 
 /**
- * @param {Context} ctx
+ * @type {RouteHandler}
  */
 export default async function fetch(ctx) {
   const { config } = ctx;

src/routes/auth/handler.js

@@ -16,16 +16,7 @@ import update from './update.js';
 import rotate from './rotate.js';
 
 /**
- * @type {Record<
- *  string,
- *  Record<
- *    string,
- *    (
- *      ctx: Context,
- *      req: import("@cloudflare/workers-types/experimental").Request
- *    ) => Promise<Response>
- *  >
- * >}
+ * @type {Record<string, Record<string, RouteHandler>>}
  */
 const handlers = {
   token: {
@@ -36,9 +27,7 @@ const handlers = {
 };
 
 /**
- * @param {Context} ctx
- * @param {import("@cloudflare/workers-types/experimental").Request} req
- * @returns {Promise<Response>}
+ * @type {RouteHandler}
  */
 export default async function handler(ctx, req) {
   const {

src/routes/auth/rotate.js

@@ -15,7 +15,7 @@ import { errorResponse } from '../../utils/http.js';
 import { updateToken } from './update.js';
 
 /**
- * @param {Context} ctx
+ * @type {RouteHandler}
  */
 export default async function rotate(ctx) {
   const { data } = ctx;

src/routes/auth/update.js

@@ -33,7 +33,7 @@ export async function updateToken(ctx, token = generateToken()) {
 }
 
 /**
- * @param {Context} ctx
+ * @type {RouteHandler}
  */
 export default async function update(ctx) {
   const { data } = ctx;

src/routes/catalog/StorageClient.js

@@ -115,7 +115,7 @@ export default class StorageClient {
         }
 
         // Attempt to save the product
-        const putResponse = await env.CATALOG_BUCKET.put(key, body, {
+        await env.CATALOG_BUCKET.put(key, body, {
           httpMetadata: { contentType: 'application/json' },
           customMetadata,
         });
@@ -136,7 +136,6 @@ export default class StorageClient {
          */
         const result = {
           sku,
-          status: putResponse.status,
           message: 'Product saved successfully.',
           ...adminResponse.paths,
         };
@@ -203,7 +202,7 @@ export default class StorageClient {
           };
         }
         const { customMetadata } = productHead;
-        const deleteProductResponse = await env.CATALOG_BUCKET.delete(productKey);
+        await env.CATALOG_BUCKET.delete(productKey);
 
         const { urlKey } = customMetadata;
         if (urlKey) {
@@ -217,7 +216,6 @@ export default class StorageClient {
          */
         const result = {
           sku,
-          status: deleteProductResponse?.status,
           message: 'Product deleted successfully.',
           ...adminResponse.paths,
         };

src/routes/catalog/fetch.js

@@ -13,9 +13,7 @@
 import StorageClient from './StorageClient.js';
 
 /**
- * Handles a GET request for a product.
- * @param {Context} ctx - The context object containing request information and utilities.
- * @returns {Promise<Response>} - A promise that resolves to the product response.
+ * @type {RouteHandler}
  */
 export default async function fetch(ctx) {
   const { sku } = ctx.config;

src/routes/catalog/handler.js

@@ -17,17 +17,7 @@ import update from './update.js';
 import remove from './remove.js';
 
 /**
- * @type {Record<
- *    string,
- *    Record<
- *      string,
- *      (
- *        ctx: Context,
- *        req: import("@cloudflare/workers-types/experimental").Request
- *      ) => Promise<Response>
- *    >
- *  >
- * }
+ * @type {Record<string, Record<string, RouteHandler>>}
  */
 const handlers = {
   lookup: {
@@ -47,10 +37,7 @@ const handlers = {
 };
 
 /**
- * Handles productbus requests.
- * @param {Context} ctx - The context object containing request information and utilities.
- * @param {import("@cloudflare/workers-types/experimental").Request} request - The request object.
- * @returns {Promise<Response>} - A promise that resolves to the catalog response.
+ * @type {RouteHandler}
  */
 export default async function handler(ctx, request) {
   const {

src/routes/catalog/lookup.js

@@ -13,9 +13,7 @@
 import StorageClient from './StorageClient.js';
 
 /**
- * Handles a product lookup request.
- * @param {Context} ctx - The context object.
- * @returns {Promise<Response>} - A promise that resolves to the product response.
+ * @type {RouteHandler}
  */
 export default async function lookup(ctx) {
   const {

src/routes/catalog/remove.js

@@ -15,9 +15,7 @@ import { errorResponse, errorWithResponse } from '../../utils/http.js';
 import StorageClient from './StorageClient.js';
 
 /**
- * Handles a DELETE request for a product.
- * @param {Context} ctx - The context object containing request information and utilities.
- * @returns {Promise<Response>} - A promise that resolves to the product response.
+ * @type {RouteHandler}
  */
 export default async function remove(ctx) {
   const { log, config } = ctx;

src/routes/catalog/update.js

@@ -15,10 +15,9 @@ import { errorResponse } from '../../utils/http.js';
 import StorageClient from './StorageClient.js';
 import { assertAuthorization } from '../../utils/auth.js';
 import { extractAndReplaceImages } from '../../utils/media.js';
+
 /**
- * Handles a PUT request to update a product.
- * @param {Context} ctx - The context object containing request information and utilities.
- * @returns {Promise<Response>} - A promise that resolves to the product response.
+ * @type {RouteHandler}
  */
 export default async function update(ctx) {
   const { config, log, data } = ctx;

src/routes/config/handler.js

@@ -17,8 +17,7 @@ import { updateToken } from '../auth/update.js';
 import ConfigSchema from '../../schemas/Config.js';
 
 /**
- * @param {Context} ctx
- * @returns {Promise<Response>}
+ * @type {RouteHandler}
  */
 export default async function configHandler(ctx) {
   const { method } = ctx.info;

src/routes/content/adobe-commerce/index.js

@@ -212,8 +212,7 @@ function lookupProductSKU(urlkey, config) {
 }
 
 /**
- * @param {Context} ctx
- * @returns {Promise<Response>}
+ * @type {RouteHandler}
  */
 export default async function handler(ctx) {
   const { config } = ctx;

src/routes/content/handler.js

@@ -18,10 +18,9 @@ import media from './media.js';
 const ALLOWED_METHODS = ['GET'];
 
 /**
- * @param {Context} ctx
- * @returns {Promise<Response>}
+ * @type {RouteHandler}
  */
-export default async function contentHandler(ctx) {
+export default async function contentHandler(ctx, req) {
   const {
     log,
     config,
@@ -33,7 +32,7 @@ export default async function contentHandler(ctx) {
   }
 
   if (['png', 'jpg', 'jpeg', 'gif', 'svg', 'webp'].includes(info.extension)) {
-    return media(ctx);
+    return media(ctx, req);
   }
 
   if (!config.pageType) {
@@ -42,8 +41,8 @@ export default async function contentHandler(ctx) {
   log.debug('config: ', JSON.stringify(config, null, 2));
 
   if (config.catalogSource === 'adobe-commerce') {
-    return adobeCommerce(ctx);
+    return adobeCommerce(ctx, req);
   }
 
-  return helixCommerce(ctx);
+  return helixCommerce(ctx, req);
 }

src/routes/content/helix-commerce.js

@@ -145,8 +145,7 @@ ${indent(product.images?.map(pictureTemplate).join('\n'), 10)}
 </html>`;
 
 /**
- * @param {Context} ctx
- * @returns {Promise<Response>}
+ * @type {RouteHandler}
  */
 export default async function handler(ctx) {
   const { config: { params } } = ctx;

src/routes/content/media.js

@@ -13,7 +13,7 @@
 import { errorResponse } from '../../utils/http.js';
 
 /**
- * @param {Context} ctx
+ * @type {RouteHandler}
  */
 export default async function handler(ctx) {
   const {

src/routes/index.js

@@ -16,13 +16,7 @@ import config from './config/handler.js';
 import auth from './auth/handler.js';
 
 /**
- * @type {Record<
- *  string,
- *  (
- *    ctx: Context,
- *    request: import("@cloudflare/workers-types/experimental").Request
- *  ) => Promise<Response>
- * >}
+ * @type {Record<string, RouteHandler>}
  */
 export default {
   content,

src/types.d.ts

@@ -414,6 +414,8 @@ declare global {
     status: number;
     message?: string;
   }
+
+  export type RouteHandler = (ctx: Context, request: import("@cloudflare/workers-types").Request) => Promise<Response>;
 }
 
 export { };

src/utils/admin.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-disable no-await-in-loop */
-
 import { getPreviewPublishPaths } from './product.js';
 
 /**
@@ -125,7 +123,9 @@ export async function callPreviewPublish(config, method, sku, urlKey) {
 
     // Conditional sequencing or parallel execution based on the method
     const operations = method === 'POST'
+      // eslint-disable-next-line no-await-in-loop
       ? [await callAdminWithOp('preview'), await callAdminWithOp('live')]
+      // eslint-disable-next-line no-await-in-loop
       : await Promise.all(['preview', 'live'].map(callAdminWithOp));
 
     operations.forEach(({ op, status, message }) => {