refactor: remove unnecessary async usage

Author: conico974

biome.json

@@ -82,7 +82,8 @@
  "noControlCharactersInRegex": "warn",
  "noPrototypeBuiltins": "warn",
  "noAssignInExpressions": "warn",
- "noArrayIndexKey": "warn"
+ "noArrayIndexKey": "warn",
+ "useAwait": "error"
  },
  "a11y": {
  "useSemanticElements": "warn",
@@ -152,6 +153,13 @@
  "include": ["*.test.ts", "packages/gitbook/tests/**/*"],
  "javascript": {
  "globals": ["Bun"]
+ },
+ "linter": {
+ "rules": {
+ "suspicious": {
+ "useAwait": "off"
+ }
+ }
  }
  },
  {
@@ -170,6 +178,16 @@
  "ExportedHandler"
  ]
  }
+ },
+ {
+ "include": ["packages/gitbook/openNext/**/*"],
+ "linter": {
+ "rules": {
+ "suspicious": {
+ "useAwait": "off"
+ }
+ }
+ }
  }
  ]
 }

packages/gitbook/e2e/internal.spec.ts

@@ -425,7 +425,7 @@ const testCases: TestsCase[] = [
  name: 'Text page',
  url: 'text-page.md',
  screenshot: false,
- run: async (_page, response) => {
+ run: (_page, response) => {
  expect(response?.status()).toBe(200);
  expect(response?.headers()['content-type']).toContain('text/markdown');
  },
@@ -441,7 +441,7 @@ const testCases: TestsCase[] = [
  name: 'llms.txt',
  url: 'llms.txt',
  screenshot: false,
- run: async (_page, response) => {
+ run: (_page, response) => {
  expect(response?.status()).toBe(200);
  expect(response?.headers()['content-type']).toContain('text/markdown');
  },
@@ -457,7 +457,7 @@ const testCases: TestsCase[] = [
  name: 'llms-full.txt',
  url: 'llms-full.txt',
  screenshot: false,
- run: async (_page, response) => {
+ run: (_page, response) => {
  expect(response?.status()).toBe(200);
  expect(response?.headers()['content-type']).toContain('text/markdown');
  },
@@ -473,7 +473,7 @@ const testCases: TestsCase[] = [
  name: 'blocks.md',
  url: 'blocks.md',
  screenshot: false,
- run: async (_page, response) => {
+ run: (_page, response) => {
  expect(response?.status()).toBe(200);
  expect(response?.headers()['content-type']).toContain('text/markdown');
  },

packages/gitbook/e2e/util.ts

@@ -144,7 +144,7 @@ export async function waitForCookiesDialog(page: Page) {
  await expect(dialog).toBeVisible();
 }
 
-export async function waitForNotFound(_page: Page, response: Response | null) {
+export function waitForNotFound(_page: Page, response: Response | null) {
  expect(response).not.toBeNull();
  expect(response?.status()).toBe(404);
 }

packages/gitbook/next.config.mjs

@@ -54,7 +54,7 @@ const nextConfig = {
  ],
  },
 
- async headers() {
+ headers() {
  return [
  {
  source: '/~gitbook/static/:path*',

packages/gitbook/openNext/customWorkers/default.js

@@ -22,7 +22,7 @@ export class R2WriteBuffer extends DurableObject {
 }
 
 export default {
- async fetch(request, env, ctx) {
+ fetch(request, env, ctx) {
  return runWithCloudflareRequestContext(request, env, ctx, async () => {
  // We can't move the handler import to the top level, otherwise the runtime will not be properly initialized
  const { handler } = await import(

packages/gitbook/openNext/customWorkers/do.js

@@ -26,7 +26,7 @@ export { DOQueueHandler } from '../../.open-next/.build/durable-objects/queue.js
 export { DOShardedTagCache } from '../../.open-next/.build/durable-objects/sharded-tag-cache.js';
 
 export default {
- async fetch() {
+ fetch() {
  // This worker does not handle any requests, it only provides Durable Objects
  return new Response('This worker is not meant to handle requests directly', {
  status: 400,

packages/gitbook/openNext/customWorkers/middleware.js

@@ -8,7 +8,7 @@ export { DOQueueHandler } from '../../.open-next/.build/durable-objects/queue.js
 export { DOShardedTagCache } from '../../.open-next/.build/durable-objects/sharded-tag-cache.js';
 
 export default class extends WorkerEntrypoint {
- async fetch(request) {
+ fetch(request) {
  return runWithCloudflareRequestContext(request, this.env, this.ctx, async () => {
  // - `Request`s are handled by the Next server
  const reqOrResp = await middlewareHandler(request, this.env, this.ctx);

packages/gitbook/openNext/queue/middleware.ts

@@ -5,8 +5,8 @@ import doQueue from '@opennextjs/cloudflare/overrides/queue/do-queue';
 
 export default {
  name: 'GitbookISRQueue',
- send: async (msg) => {
- return trace({ operation: 'gitbookISRQueueSend', name: msg.MessageBody.url }, async () => {
+ send: (msg) => {
+ return trace({ operation: 'gitbookISRQueueSend', name: msg.MessageBody.url }, () => {
  const { ctx } = getCloudflareContext();
  ctx.waitUntil(doQueue.send(msg));
  });

packages/gitbook/openNext/queue/server.ts

@@ -2,7 +2,7 @@ import type { Queue } from '@opennextjs/aws/types/overrides.js';
 
 export default {
  name: 'GitbookISRQueue',
- send: async (msg) => {
+ send: (msg) => {
  // We should never reach this point in the server. If that's the case, we should log it.
  console.warn('GitbookISRQueue: send called on server side, this should not happen.', msg);
  },

packages/gitbook/openNext/tagCache/middleware.ts

@@ -20,7 +20,7 @@ const originalTagCache = doShardedTagCache({
 export default {
  name: 'GitbookTagCache',
  mode: 'nextMode',
- getLastRevalidated: async (tags: string[]) => {
+ getLastRevalidated: (tags: string[]) => {
  const tagsToCheck = tags.filter(softTagFilter);
  if (tagsToCheck.length === 0) {
  // If we reach here, it probably means that there is an issue that we'll need to address.
@@ -40,7 +40,7 @@ export default {
  }
  );
  },
- hasBeenRevalidated: async (tags: string[], lastModified?: number) => {
+ hasBeenRevalidated: (tags: string[], lastModified?: number) => {
  const tagsToCheck = tags.filter(softTagFilter);
  if (tagsToCheck.length === 0) {
  // If we reach here, it probably means that there is an issue that we'll need to address.
@@ -61,7 +61,7 @@ export default {
  }
  );
  },
- writeTags: async (tags: string[]) => {
+ writeTags: (tags: string[]) => {
  return trace(
  {
  operation: 'gitbookTagCacheWriteTags',