chore: migrate fmt to biome (#936)

* configure

* migrate ci

* apply format

* weird cf stuff

* format then lint

Author: viktormarinho

.github/workflows/test.yml

@@ -16,27 +16,26 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v4
 
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+
+      # Remove when lint is migrated to Biome
       - name: Set up Deno
         uses: denoland/setup-deno@v2
         with:
           deno-version: v2.x
 
       - name: Install dependencies
-        run: deno install
-
-      - name: Run lint
-        run: deno run lint
+        run: bun install
 
       - name: Run format
-        run: deno run fmt --check
+        run: bun run fmt:check
 
-      - name: Print TSC version
-        run: deno run version
+      - name: Run lint
+        run: bun run lint
 
       - name: Run check
-        env:
-          DENO_V8_FLAGS: --max-old-space-size=16384
-        run: deno run check
+        run: bun run check
 
       - name: Run test
-        run: deno run test
+        run: bun run test

apps/api/main.ts

@@ -17,7 +17,7 @@ const instrumentedApp = getRuntimeKey() === "deno" ? app : instrument(app);
 const SELF_DOMAINS: string[] = [
   Hosts.API,
   // @ts-expect-error env is not typed
-  ...env.VITE_USE_LOCAL_BACKEND ? [] : [Hosts.APPS],
+  ...(env.VITE_USE_LOCAL_BACKEND ? [] : [Hosts.APPS]),
   // @ts-expect-error env is not typed
   `localhost:${env.PORT || 8000}`,
 ];
@@ -71,11 +71,7 @@ globalThis.fetch = async function patchedFetch(
 // Default export that wraps app with per-request context initializer
 export default {
   email,
-  fetch(
-    request: Request,
-    env: any,
-    ctx: ExecutionContext,
-  ): Promise<Response> {
+  fetch(request: Request, env: any, ctx: ExecutionContext): Promise<Response> {
     return contextStorage.run({ env, ctx }, async () => {
       return await instrumentedApp.fetch!(
         request as Request<unknown, IncomingRequestCfProperties<unknown>>,

apps/api/src/api.ts

@@ -53,13 +53,14 @@ export const honoCtxToAppCtx = (c: Context<AppEnv>): AppContext => {
     token: c.req.header("Authorization")?.replace("Bearer ", ""),
     kbFileProcessor: c.env.KB_FILE_PROCESSOR,
     workspaceDO: c.env.WORKSPACE_DB,
-    workspace: slug && root
-      ? {
-        root,
-        slug,
-        value: workspace,
-      }
-      : undefined,
+    workspace:
+      slug && root
+        ? {
+            root,
+            slug,
+            value: workspace,
+          }
+        : undefined,
   };
 };
 
@@ -101,12 +102,14 @@ const createMCPHandlerFor = (
         {
           annotations: tool.annotations,
           description: tool.description,
-          inputSchema: "shape" in tool.inputSchema
-            ? (tool.inputSchema.shape as z.ZodRawShape)
-            : z.object({}).shape,
+          inputSchema:
+            "shape" in tool.inputSchema
+              ? (tool.inputSchema.shape as z.ZodRawShape)
+              : z.object({}).shape,
           outputSchema:
-            tool.outputSchema && typeof tool.outputSchema === "object" &&
-              "shape" in tool.outputSchema
+            tool.outputSchema &&
+            typeof tool.outputSchema === "object" &&
+            "shape" in tool.outputSchema
               ? (tool.outputSchema.shape as z.ZodRawShape)
               : z.object({}).shape,
         },
@@ -182,28 +185,30 @@ const createToolCallHandlerFor = <
 app.use(logger());
 
 // Enable CORS for all routes on api.deco.chat and localhost
-app.use(cors({
-  origin: (origin) => origin,
-  maxAge: 86400, // one day
-  allowMethods: ["HEAD", "GET", "POST", "PUT", "DELETE", "PATCH", "OPTIONS"],
-  allowHeaders: [
-    "Content-Type",
-    "Authorization",
-    "Cookie",
-    "Accept",
-    "cache-control",
-    "pragma",
-    "x-trace-debug-id",
-    "x-deno-isolate-instance-id",
-  ],
-  exposeHeaders: [
-    "Content-Type",
-    "Authorization",
-    "Set-Cookie",
-    "x-trace-debug-id",
-  ],
-  credentials: true,
-}));
+app.use(
+  cors({
+    origin: (origin) => origin,
+    maxAge: 86400, // one day
+    allowMethods: ["HEAD", "GET", "POST", "PUT", "DELETE", "PATCH", "OPTIONS"],
+    allowHeaders: [
+      "Content-Type",
+      "Authorization",
+      "Cookie",
+      "Accept",
+      "cache-control",
+      "pragma",
+      "x-trace-debug-id",
+      "x-deno-isolate-instance-id",
+    ],
+    exposeHeaders: [
+      "Content-Type",
+      "Authorization",
+      "Set-Cookie",
+      "x-trace-debug-id",
+    ],
+    credentials: true,
+  }),
+);
 
 app.use(withContextMiddleware);
 app.use(setUserMiddleware);
@@ -221,24 +226,12 @@ app.use(async (c, next) => {
 app.use(withActorsMiddleware);
 
 // MCP endpoint handlers
-app.all(
-  "/mcp",
-  createMCPHandlerFor(GLOBAL_TOOLS),
-);
-app.all(
-  "/:root/:slug/mcp",
-  createMCPHandlerFor(WORKSPACE_TOOLS),
-);
-app.all(
-  "/:root/:slug/agents/:agentId/mcp",
-  createMCPHandlerFor(AGENT_TOOLS),
-);
+app.all("/mcp", createMCPHandlerFor(GLOBAL_TOOLS));
+app.all("/:root/:slug/mcp", createMCPHandlerFor(WORKSPACE_TOOLS));
+app.all("/:root/:slug/agents/:agentId/mcp", createMCPHandlerFor(AGENT_TOOLS));
 
 // Tool call endpoint handlers
-app.post(
-  "/tools/call/:tool",
-  createToolCallHandlerFor(GLOBAL_TOOLS),
-);
+app.post("/tools/call/:tool", createToolCallHandlerFor(GLOBAL_TOOLS));
 app.post(
   "/:root/:slug/tools/call/:tool",
   createToolCallHandlerFor(WORKSPACE_TOOLS),
@@ -302,8 +295,8 @@ app.get("/files/:root/:slug/:path{.+}", async (c) => {
   }
 
   return c.body(response.body, 200, {
-    "Content-Type": response.headers.get("content-type") ||
-      "application/octet-stream",
+    "Content-Type":
+      response.headers.get("content-type") || "application/octet-stream",
   });
 });
 
@@ -313,7 +306,7 @@ const getPublicKey = async (): Promise<JsonWebKey> => {
 };
 app.get("/.well-known/jwks.json", async () => {
   return Response.json({
-    keys: [{ ...await getPublicKey(), kid: DECO_CHAT_KEY_ID }],
+    keys: [{ ...(await getPublicKey()), kid: DECO_CHAT_KEY_ID }],
   });
 });
 // External webhooks

apps/api/src/app.ts

@@ -11,24 +11,28 @@ export const appsDomainOf = (req: Request, url?: URL) => {
   url ??= new URL(req.url);
   const referer = req.headers.get("referer");
 
-  return url.searchParams.get(APPS_DOMAIN_QS) ||
-    (referer && new URL(referer).searchParams.get(APPS_DOMAIN_QS));
+  return (
+    url.searchParams.get(APPS_DOMAIN_QS) ||
+    (referer && new URL(referer).searchParams.get(APPS_DOMAIN_QS))
+  );
 };
 
 const normalizeHost = (req: Request) => {
-  const host = req.headers.get("host") ?? new URL(req.url).hostname ??
-    "localhost";
+  const host =
+    req.headers.get("host") ?? new URL(req.url).hostname ?? "localhost";
 
   const appsHost = appsDomainOf(req);
   if (appsHost) {
     return Hosts.APPS;
   }
-  return {
-    [Hosts.API]: Hosts.API,
-    localhost: Hosts.API,
-    "localhost:3001": Hosts.API,
-    "localhost:8000": Hosts.API,
-  }[host] ?? Hosts.APPS;
+  return (
+    {
+      [Hosts.API]: Hosts.API,
+      localhost: Hosts.API,
+      "localhost:3001": Hosts.API,
+      "localhost:8000": Hosts.API,
+    }[host] ?? Hosts.APPS
+  );
 };
 
 export const app = new Hono<AppEnv>({

apps/api/src/apps.ts

@@ -27,11 +27,15 @@ export const fetchScript = async (
   }
   const scriptFetcher = dispatcher.get<{
     DECO_CHAT_APP_ORIGIN: string;
-  }>(script, {}, {
-    outbound: {
-      DECO_CHAT_APP_ORIGIN: script,
+  }>(
+    script,
+    {},
+    {
+      outbound: {
+        DECO_CHAT_APP_ORIGIN: script,
+      },
     },
-  });
+  );
   const response = await scriptFetcher.fetch(req).catch((err) => {
     if ("message" in err && err.message.startsWith("Worker not found")) {
       // we tried to get a worker that doesn't exist in our dispatch namespace
@@ -46,26 +50,25 @@ export const fetchScript = async (
 app.use(withContextMiddleware);
 app.all("/*", async (c: Context<AppEnv>) => {
   const url = new URL(c.req.url);
-  let host = appsDomainOf(c.req.raw) ?? c.req.header("host") ??
-    url.host;
+  let host = appsDomainOf(c.req.raw) ?? c.req.header("host") ?? url.host;
   if (!host) {
     return new Response("No host", { status: 400 });
   }
   const locator = Entrypoint.script(host);
   // if it has a deployment ID, we can use the script ID directly
   let script = locator?.isCanonical ? locator.slug : null;
   const getScriptFn = async (): Promise<string | null> => {
-    const { data, error } = await c.var.db.from("deco_chat_hosting_routes")
+    const { data, error } = await c.var.db
+      .from("deco_chat_hosting_routes")
       .select(`
             *,
             deco_chat_hosting_apps_deployments!deployment_id(
               id,
               deco_chat_hosting_apps!hosting_app_id(slug)
             )
-          `).eq(
-        "route_pattern",
-        host,
-      ).maybeSingle();
+          `)
+      .eq("route_pattern", host)
+      .maybeSingle();
 
     if ((error || !data) && locator) {
       return locator.slug;
@@ -83,10 +86,9 @@ app.all("/*", async (c: Context<AppEnv>) => {
   };
   const isNoCache = c.req.header("x-domain-swr-ignore-cache") === "true";
   if (!script) {
-    script = isNoCache ? await getScriptFn() : await domainSWRCache.cache(
-      getScriptFn,
-      host,
-    ).catch(() => null);
+    script = isNoCache
+      ? await getScriptFn()
+      : await domainSWRCache.cache(getScriptFn, host).catch(() => null);
   }
 
   if (!script) {

apps/api/src/auth/index.ts

@@ -71,9 +71,9 @@ export const getUser = async (
     supabase,
     DECO_CHAT_API_JWT_PRIVATE_KEY && DECO_CHAT_API_JWT_PUBLIC_KEY
       ? {
-        public: DECO_CHAT_API_JWT_PUBLIC_KEY,
-        private: DECO_CHAT_API_JWT_PRIVATE_KEY,
-      }
+          public: DECO_CHAT_API_JWT_PUBLIC_KEY,
+          private: DECO_CHAT_API_JWT_PRIVATE_KEY,
+        }
       : undefined,
   );
 
@@ -123,7 +123,8 @@ appLogin.all("/oauth", async (ctx: AppContext) => {
 
   // user already logged in, set by userMiddleware
   if (user && !user.is_anonymous) {
-    const origin = ctx.req.header("referer") ||
+    const origin =
+      ctx.req.header("referer") ||
       ctx.req.header("origin") ||
       "https://deco.chat";
     return ctx.redirect(origin);
@@ -177,8 +178,8 @@ appLogin.all("/magiclink", async (ctx: AppContext) => {
     const redirectTo = cli
       ? AUTH_URL_CLI
       : url.host.includes("localhost")
-      ? "http://localhost:3001/"
-      : "https://api.deco.chat/";
+        ? "http://localhost:3001/"
+        : "https://api.deco.chat/";
 
     await db.auth.signInWithOtp({
       email,
@@ -229,7 +230,8 @@ appAuth.all("/callback/magiclink", async (ctx: AppContext) => {
     const request = ctx.req.raw;
     const { db, headers } = createDbAndHeadersForRequest(ctx);
     const url = new URL(request.url);
-    const next = url.searchParams.get("next") ||
+    const next =
+      url.searchParams.get("next") ||
       (url.host.includes("localhost")
         ? "http://localhost:3000"
         : "https://deco.chat");

apps/api/src/durable-objects/workspace-database.ts

@@ -15,10 +15,12 @@ export class WorkspaceDatabase extends DurableObject implements IWorkspaceDB {
 
   exec({ sql, params }: DatatabasesRunSqlInput) {
     return {
-      result: [{
-        results: this.sql.exec(sql, ...(params ?? [])).toArray(),
-        success: true,
-      }],
+      result: [
+        {
+          results: this.sql.exec(sql, ...(params ?? [])).toArray(),
+          success: true,
+        },
+      ],
       [Symbol.dispose]: () => {},
     };
   }