chore(cleanup): types in lib and coverage threshold

Increases coverage threshold based on updated test suite
Updates the types used in lib/users to match updated API types
Generated updated Swagger

Author: bitbeckers

src/__generated__/routes/routes.ts

@@ -1,6 +1,6 @@
 import { verifyAuthSignedData } from "../../utils/verifyAuthSignedData.js";
 import { SupabaseDataService } from "../../services/SupabaseDataService.js";
-import type { AddOrUpdateUserResponse } from "../../types/api.js";
+import type { UserResponse } from "../../types/api.js";
 
 import type { EOAUpdateRequest } from "./schemas.js";
 import type { UserUpsertStrategy } from "./UserUpsertStrategy.js";
@@ -16,7 +16,7 @@ export default class EOAUpdateStrategy implements UserUpsertStrategy {
     this.dataService = new SupabaseDataService();
   }
 
-  async execute(): Promise<AddOrUpdateUserResponse> {
+  async execute(): Promise<UserResponse> {
     await this.throwIfInvalidSignature();
     const user = await this.upsertUser();
     return {

src/__generated__/swagger.json

@@ -3,7 +3,7 @@ import SafeApiKit, { type SafeApiKitConfig } from "@safe-global/api-kit";
 
 import { SignatureRequestPurpose } from "../../graphql/schemas/typeDefs/signatureRequestTypeDefs.js";
 import { SupabaseDataService } from "../../services/SupabaseDataService.js";
-import { AddOrUpdateUserResponse } from "../../types/api.js";
+import { UserResponse } from "../../types/api.js";
 import { isTypedMessage } from "../../utils/signatures.js";
 
 import type { UserUpsertStrategy } from "./UserUpsertStrategy.js";
@@ -45,7 +45,7 @@ export default class MultisigUpdateStrategy implements UserUpsertStrategy {
   }
 
   // We could check if it's a 1 of 1 and execute right away
-  async execute(): Promise<AddOrUpdateUserResponse> {
+  async execute(): Promise<UserResponse> {
     const { message } = await this.safeApiKit.getMessage(
       this.request.messageHash,
     );
@@ -76,7 +76,6 @@ export default class MultisigUpdateStrategy implements UserUpsertStrategy {
     return {
       success: true,
       message: "Signature request created successfully",
-      data: null,
     };
   }
 }

src/lib/users/EOAUpsertStrategy.ts

@@ -1,11 +1,11 @@
-import { AddOrUpdateUserResponse } from "../../types/api.js";
+import { UserResponse } from "../../types/api.js";
 
 import MultisigUpsertStrategy from "./MultisigUpsertStrategy.js";
 import EOAUpsertStrategy from "./EOAUpsertStrategy.js";
 import { EOAUpdateRequest, MultisigUpdateRequest } from "./schemas.js";
 
 export interface UserUpsertStrategy {
-  execute(): Promise<AddOrUpdateUserResponse>;
+  execute(): Promise<UserResponse>;
 }
 
 export function createStrategy(

src/lib/users/MultisigUpsertStrategy.ts

@@ -1,8 +1,8 @@
-import { AddOrUpdateUserResponse } from "../../types/api.js";
+import { UserResponse } from "../../types/api.js";
 
 export class UserUpsertError extends Error {
   code: number;
-  public errors: AddOrUpdateUserResponse["errors"];
+  public errors: UserResponse["errors"];
   constructor(code: number, message: string) {
     super(message);
     this.name = "UserUpdateError";

src/lib/users/UserUpsertStrategy.ts

@@ -11,10 +11,10 @@ export default defineConfig({
       // If you want a coverage reports even if your tests are failing, include the reportOnFailure option
       reportOnFailure: true,
       thresholds: {
-        lines: 15,
-        branches: 54,
-        functions: 49,
-        statements: 15,
+        lines: 17,
+        branches: 60,
+        functions: 54,
+        statements: 17,
       },
       include: ["src/**/*.ts"],
       exclude: [
@@ -24,6 +24,7 @@ export default defineConfig({
         "src/__generated__/**/*",
         "src/graphql/**/*",
         "src/types/**/*",
+        "src/abis/**/*",
         "./lib/**/*",
       ],
     },