chore: lint

Author: pi0

eslint.config.mjs

@@ -8,5 +8,6 @@ export default unjs({
     "@typescript-eslint/no-unused-vars": 0,
     "unicorn/filename-case": 0,
     "unicorn/consistent-function-scoping": 0,
+    "@typescript-eslint/no-empty-object-type": 0,
   },
 });

src/config/index.ts

@@ -3,6 +3,5 @@ import type { NitroConfig } from "nitropack/types";
 export type { NitroConfig } from "nitropack/types";
 
 export function defineNitroConfig(config: NitroConfig): NitroConfig {
-  config.cloudflare?.wrangler?.compatibility_flags;
   return config;
 }

src/types/_utils.ts

@@ -12,7 +12,7 @@ export type IntRange<F extends number, T extends number> = Exclude<
 
 export type ExcludeFunctions<G extends Record<string, any>> = Pick<
   G,
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   { [P in keyof G]: NonNullable<G[P]> extends Function ? never : P }[keyof G]
 >;
 

src/types/config.ts

@@ -323,7 +323,6 @@ export interface StorageMounts {
 }
 
 // Database
-// eslint-disable-next-line @typescript-eslint/ban-types
 export type DatabaseConnectionName = "default" | (string & {});
 export type DatabaseConnectionConfig = {
   connector: ConnectorName;

src/types/fetch/_serialize.ts

@@ -5,14 +5,11 @@ type JsonPrimitive =
   | string
   | number
   | boolean
-  // eslint-disable-next-line @typescript-eslint/ban-types
-  | String
-  // eslint-disable-next-line @typescript-eslint/ban-types
-  | Number
-  // eslint-disable-next-line @typescript-eslint/ban-types
-  | Boolean
+  | string
+  | number
+  | boolean
   | null;
-// eslint-disable-next-line @typescript-eslint/ban-types
+// eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
 type NonJsonPrimitive = undefined | Function | symbol;
 
 /*

src/types/fetch/fetch.ts

@@ -8,7 +8,6 @@ export interface InternalApi {}
 export type NitroFetchRequest =
   | Exclude<keyof InternalApi, `/_${string}` | `/api/_${string}`>
   | Exclude<FetchRequest, string>
-  // eslint-disable-next-line @typescript-eslint/ban-types
   | (string & {});
 
 export type MiddlewareOf<

src/types/handler.ts

@@ -47,10 +47,7 @@ export interface NitroEventHandler {
   /*
    * Environments to include this handler
    */
-  env?: MaybeArray<
-    // eslint-disable-next-line @typescript-eslint/ban-types
-    "dev" | "prod" | "prerender" | PresetName | (string & {})
-  >;
+  env?: MaybeArray<"dev" | "prod" | "prerender" | PresetName | (string & {})>;
 }
 
 export interface NitroDevEventHandler {

src/types/nitro.ts

@@ -35,7 +35,6 @@ export type NitroTypes = {
 };
 
 export interface NitroFrameworkInfo {
-  // eslint-disable-next-line @typescript-eslint/ban-types
   name?: "nitro" | (string & {});
   version?: string;
 }

test/fixture/routes/assets/[id].ts

@@ -1,5 +1,6 @@
 export default eventHandler(async (event) => {
   const serverAssets = useStorage("assets/server");
+
   const id = event.context.params.id;
 
   if (!(await serverAssets.hasItem(id))) {
@@ -9,9 +10,18 @@ export default eventHandler(async (event) => {
   const meta = (await serverAssets.getMeta(
     event.context.params.id
   )) as unknown as { type: string; etag: string; mtime: string };
-  meta.type && setResponseHeader(event, "content-type", meta.type);
-  meta.etag && setResponseHeader(event, "etag", meta.etag);
-  meta.mtime && setResponseHeader(event, "last-modified", meta.mtime);
+
+  if (meta.type) {
+    setResponseHeader(event, "content-type", meta.type);
+  }
+
+  if (meta.etag) {
+    setResponseHeader(event, "etag", meta.etag);
+  }
+
+  if (meta.mtime) {
+    setResponseHeader(event, "last-modified", meta.mtime);
+  }
 
   return serverAssets.getItemRaw(event.context.params.id);
 });

test/fixture/types.ts

@@ -230,7 +230,6 @@ describe("API routes", () => {
     expectTypeOf($fetch("/api/serialized/null")).toEqualTypeOf<Promise<any>>();
 
     expectTypeOf($fetch("/api/serialized/function")).toEqualTypeOf<
-      // eslint-disable-next-line @typescript-eslint/ban-types
       Promise<{}>
     >();
 
@@ -297,7 +296,6 @@ describe("defineCachedEventHandler", () => {
       Promise<{ message: string }>
     >(fixture);
     expectTypeOf(b).toEqualTypeOf<
-      // eslint-disable-next-line @typescript-eslint/ban-types
       EventHandler<{}, Promise<{ message: string }>>
     >();
   });