chore: bootstrap biome and reformat code

Author: taeold

biome.json

@@ -0,0 +1,82 @@
+{
+  "$schema": "https://biomejs.dev/schemas/2.2.5/schema.json",
+  "formatter": {
+    "enabled": true,
+    "indentStyle": "space",
+    "indentWidth": 2,
+    "lineWidth": 100
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": false,
+      "correctness": {
+        "noUnusedVariables": "error",
+        "noUndeclaredVariables": "off"
+      },
+      "style": {
+        "useConst": "off",
+        "useBlockStatements": "off",
+        "useSelfClosingElements": "off"
+      },
+      "suspicious": {
+        "noDebugger": "error",
+        "noExtraNonNullAssertion": "warn",
+        "noTsIgnore": "warn",
+        "useAdjacentOverloadSignatures": "warn"
+      }
+    }
+  },
+  "javascript": {
+    "formatter": {
+      "quoteStyle": "double",
+      "jsxQuoteStyle": "double",
+      "quoteProperties": "asNeeded",
+      "trailingCommas": "es5",
+      "semicolons": "always",
+      "arrowParentheses": "always"
+    },
+    "globals": ["console", "process", "module", "require", "exports", "__dirname", "__filename"],
+    "parser": {
+      "unsafeParameterDecoratorsEnabled": true
+    }
+  },
+  "overrides": [
+    {
+      "includes": ["**/*.spec.ts", "**/*.spec.js"],
+      "javascript": {
+        "globals": ["describe", "it", "before", "after", "beforeEach", "afterEach"]
+      }
+    },
+    {
+      "includes": ["docgen/**/*.ts"],
+      "linter": {
+        "enabled": false
+      }
+    },
+    {
+      "includes": ["spec/fixtures/credential/unparsable.key.json"],
+      "formatter": {
+        "enabled": false
+      },
+      "linter": {
+        "enabled": false
+      }
+    },
+    {
+      "includes": ["spec/fixtures/**"],
+      "formatter": {
+        "enabled": false
+      },
+      "linter": {
+        "enabled": false
+      }
+    }
+  ],
+  "assist": {
+    "enabled": false
+  },
+  "files": {
+    "includes": ["src/**/*.{ts,js}", "spec/**/*.{ts,js}"]
+  }
+}

spec/common/metaprogramming.ts

@@ -22,4 +22,4 @@
 // This method will fail to compile if value is not of the explicit parameter type.
 /* eslint-disable @typescript-eslint/no-unused-vars,@typescript-eslint/no-empty-function */
 export function expectType<Type>(value: Type) {}
-export function expectNever<Type extends never>() {}
+export function expectNever<_Type extends never>() {}

src/common/change.ts

@@ -92,5 +92,8 @@ export class Change<T> {
 
     return Change.fromObjects(customizer(before || {}), customizer(json.after || {}));
   }
-  constructor(public before: T, public after: T) {}
+  constructor(
+    public before: T,
+    public after: T
+  ) {}
 }

src/common/debug.ts

@@ -38,7 +38,7 @@ function loadDebugFeatures(): DebugFeatures {
       return {};
     }
     return obj as DebugFeatures;
-  } catch (e) {
+  } catch (_e) {
     return {};
   }
 }

src/common/params.ts

@@ -27,34 +27,32 @@ import { Expression } from "../params";
  *
  * For example Split<"a/b/c", "/"> is ['a' | "b" | "c"]
  */
-export type Split<S extends string, D extends string> =
-  // A non-literal string splits into a string[]
-  string extends S
-    ? string[]
-    : // A literal empty string turns into a zero-tuple
+export type Split<S extends string, D extends string> = string extends S // A non-literal string splits into a string[]
+  ? string[]
+  : // A literal empty string turns into a zero-tuple
     S extends ""
     ? []
     : // Split the string; Head may be the empty string
-    S extends `${D}${infer Tail}`
-    ? [...Split<Tail, D>]
-    : S extends `${infer Head}${D}${infer Tail}`
-    ? // Drop types that are exactly string; they'll eat up literal string types
-      string extends Head
+      S extends `${D}${infer Tail}`
       ? [...Split<Tail, D>]
-      : [Head, ...Split<Tail, D>]
-    : // A string without delimiters splits into an array of itself
-      [S];
+      : S extends `${infer Head}${D}${infer Tail}`
+        ? // Drop types that are exactly string; they'll eat up literal string types
+          string extends Head
+          ? [...Split<Tail, D>]
+          : [Head, ...Split<Tail, D>]
+        : // A string without delimiters splits into an array of itself
+          [S];
 
 /**
  * A type that ensure that type S is not null or undefined.
  */
 export type NullSafe<S extends null | undefined | string> = S extends null
   ? never
   : S extends undefined
-  ? never
-  : S extends string
-  ? S
-  : never;
+    ? never
+    : S extends string
+      ? S
+      : never;
 
 /**
  * A type that extracts parameter name enclosed in bracket as string.
@@ -67,10 +65,10 @@ export type NullSafe<S extends null | undefined | string> = S extends null
 export type Extract<Part extends string> = Part extends `{${infer Param}=**}`
   ? Param
   : Part extends `{${infer Param}=*}`
-  ? Param
-  : Part extends `{${infer Param}}`
-  ? Param
-  : never;
+    ? Param
+    : Part extends `{${infer Param}}`
+      ? Param
+      : never;
 
 /**
  * A type that maps all parameter capture gropus into keys of a record.
@@ -85,12 +83,12 @@ export type ParamsOf<PathPattern extends string | Expression<string>> =
   PathPattern extends Expression<string>
     ? Record<string, string>
     : string extends PathPattern
-    ? Record<string, string>
-    : {
-        // N.B. I'm not sure why PathPattern isn't detected to not be an
-        // Expression<string> per the check above. Since we have the check above
-        // The Exclude call should be safe.
-        [Key in Extract<
-          Split<NullSafe<Exclude<PathPattern, Expression<string>>>, "/">[number]
-        >]: string;
-      };
+      ? Record<string, string>
+      : {
+          // N.B. I'm not sure why PathPattern isn't detected to not be an
+          // Expression<string> per the check above. Since we have the check above
+          // The Exclude call should be safe.
+          [Key in Extract<
+            Split<NullSafe<Exclude<PathPattern, Expression<string>>>, "/">[number]
+          >]: string;
+        };

src/common/providers/https.ts

@@ -539,7 +539,7 @@ export function unsafeDecodeToken(token: string): unknown {
       if (typeof obj === "object") {
         payload = obj;
       }
-    } catch (e) {
+    } catch (_e) {
       // ignore error
     }
   }

src/common/providers/identity.ts

@@ -84,7 +84,10 @@ export type UserInfo = auth.UserInfo;
  * Helper class to create the user metadata in a `UserRecord` object.
  */
 export class UserRecordMetadata implements auth.UserMetadata {
-  constructor(public creationTime: string, public lastSignInTime: string) {}
+  constructor(
+    public creationTime: string,
+    public lastSignInTime: string
+  ) {}
 
   /** Returns a plain JavaScript object with the properties of UserRecordMetadata. */
   toJSON(): AuthUserMetadata {
@@ -897,8 +900,8 @@ export function wrapHandler(eventType: AuthBlockingEventType, handler: AuthBlock
       const decodedPayload: DecodedPayload = isDebugFeatureEnabled("skipTokenVerification")
         ? unsafeDecodeAuthBlockingToken(req.body.data.jwt)
         : handler.platform === "gcfv1"
-        ? await auth.getAuth(getApp())._verifyAuthBlockingToken(req.body.data.jwt)
-        : await auth.getAuth(getApp())._verifyAuthBlockingToken(req.body.data.jwt, "run.app");
+          ? await auth.getAuth(getApp())._verifyAuthBlockingToken(req.body.data.jwt)
+          : await auth.getAuth(getApp())._verifyAuthBlockingToken(req.body.data.jwt, "run.app");
       let authUserRecord: AuthUserRecord | undefined;
       if (
         decodedPayload.event_type === "beforeCreate" ||

src/logger/index.ts

@@ -90,9 +90,8 @@ function removeCircular(obj: any, refs: any[] = []): any {
 export function write(entry: LogEntry) {
   const ctx = traceContext.getStore();
   if (ctx?.traceId) {
-    entry[
-      "logging.googleapis.com/trace"
-    ] = `projects/${process.env.GCLOUD_PROJECT}/traces/${ctx.traceId}`;
+    entry["logging.googleapis.com/trace"] =
+      `projects/${process.env.GCLOUD_PROJECT}/traces/${ctx.traceId}`;
   }
 
   UNPATCHED_CONSOLE[CONSOLE_SEVERITY[entry.severity]](JSON.stringify(removeCircular(entry)));

src/params/types.ts

@@ -84,7 +84,7 @@ function refOf<T extends string | number | boolean | string[]>(arg: T | Expressi
  * A CEL expression corresponding to a ternary operator, e.g {{ cond ? ifTrue : ifFalse }}
  */
 export class TernaryExpression<
-  T extends string | number | boolean | string[]
+  T extends string | number | boolean | string[],
 > extends Expression<T> {
   constructor(
     private readonly test: Expression<boolean>,
@@ -111,7 +111,7 @@ export class TernaryExpression<
  * between the value of another expression and a literal of that same type.
  */
 export class CompareExpression<
-  T extends string | number | boolean | string[]
+  T extends string | number | boolean | string[],
 > extends Expression<boolean> {
   cmp: "==" | "!=" | ">" | ">=" | "<" | "<=";
   lhs: Expression<T>;
@@ -226,7 +226,7 @@ type ParamInput<T> =
  * provided validationRegex, if present, will be retried.
  */
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
-export interface TextInput<T = unknown> {
+export interface TextInput<_T = unknown> {
   text: {
     example?: string;
     /**
@@ -340,7 +340,10 @@ export type ParamOptions<T extends string | number | boolean | string[]> = Omit<
 export abstract class Param<T extends string | number | boolean | string[]> extends Expression<T> {
   static type: ParamValueType = "string";
 
-  constructor(readonly name: string, readonly options: ParamOptions<T> = {}) {
+  constructor(
+    readonly name: string,
+    readonly options: ParamOptions<T> = {}
+  ) {
     super();
   }
 
@@ -483,7 +486,10 @@ export class StringParam extends Param<string> {
  * @internal
  */
 export class InternalExpression extends Param<string> {
-  constructor(name: string, private readonly getter: (env: NodeJS.ProcessEnv) => string) {
+  constructor(
+    name: string,
+    private readonly getter: (env: NodeJS.ProcessEnv) => string
+  ) {
     super(name);
   }
 

src/runtime/manifest.ts

@@ -196,7 +196,7 @@ function initEndpoint(
  */
 export function initV1Endpoint(...opts: ManifestOptions[]): ManifestEndpoint {
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  const { concurrency, ...resetOpts } = RESETTABLE_OPTIONS;
+  const { _concurrency, ...resetOpts } = RESETTABLE_OPTIONS;
   return initEndpoint({ ...resetOpts }, ...opts);
 }