js: Enforce uniform formatting (#1574)

We're going to commit more generated code soon that needs
auto-formatting to look good. Seems like we didn't have auto-formatting
set up correctly so we had some overly long lines and a mix of two-space
and four-space indentation. This PR fixes that, and updates CI to error
on unformatted files.

Author: svix-jplatte

javascript/.prettierrc.cjs

@@ -0,0 +1,2 @@
+printWidth = 90
+trailingComma = "es5"

javascript/.prettierrc.toml

@@ -24,8 +24,8 @@
     "test": "jest",
     "prepublishOnly": "yarn lint",
     "lint:eslint": "eslint --ignore-path .lintignore --ext .js,.jsx,.ts,.tsx src",
-    "lint:prettier": "prettier --ignore-path .lintignore --write src/**.ts",
-    "lint": "yarn run lint:prettier && yarn run lint:eslint --max-warnings=0",
+    "lint:prettier": "prettier --ignore-path .lintignore src",
+    "lint": "yarn run lint:prettier --check && yarn run lint:eslint --max-warnings=0",
     "lint:fix": "yarn run lint:prettier --write && yarn run lint:eslint --fix"
   },
   "dependencies": {
@@ -49,4 +49,4 @@
     "ts-jest": "^29.2.5",
     "typescript": "^4.0"
   }
-}
+}

javascript/package.json

@@ -1,61 +1,69 @@
-import { ApplicationApi, ApplicationIn, ApplicationOut, ApplicationPatch, Configuration, ListResponseApplicationOut, Ordering } from "../openapi";
+import {
+  ApplicationApi,
+  ApplicationIn,
+  ApplicationOut,
+  ApplicationPatch,
+  Configuration,
+  ListResponseApplicationOut,
+  Ordering,
+} from "../openapi";
 import { PostOptions } from "../util";
 
 export interface ApplicationListOptions {
-    /// Limit the number of returned items
-    limit?: number;
-    /// The iterator returned from a prior invocation
-    iterator?: string | null;
-    /// The sorting order of the returned items
-    order?: Ordering;
+  /// Limit the number of returned items
+  limit?: number;
+  /// The iterator returned from a prior invocation
+  iterator?: string | null;
+  /// The sorting order of the returned items
+  order?: Ordering;
 }
 
 export class Application {
-    private readonly api: ApplicationApi;
-
-    public constructor(config: Configuration) {
-        this.api = new ApplicationApi(config);
-    }
-
-    public list(options?: ApplicationListOptions): Promise<ListResponseApplicationOut> {
-        const iterator = options?.iterator ?? undefined;
-        return this.api.v1ApplicationList({ ...options, iterator });
-    }
-
-    public create(
-        applicationIn: ApplicationIn,
-        options?: PostOptions
-    ): Promise<ApplicationOut> {
-        return this.api.v1ApplicationCreate({ applicationIn, ...options });
-    }
-
-    public getOrCreate(
-        applicationIn: ApplicationIn,
-        options?: PostOptions
-    ): Promise<ApplicationOut> {
-        return this.api.v1ApplicationCreate({
-            applicationIn,
-            ...options,
-            getIfExists: true,
-        });
-    }
-
-    public get(appId: string): Promise<ApplicationOut> {
-        return this.api.v1ApplicationGet({ appId });
-    }
-
-    public update(appId: string, applicationIn: ApplicationIn): Promise<ApplicationOut> {
-        return this.api.v1ApplicationUpdate({ appId, applicationIn });
-    }
-
-    public patch(
-        appId: string,
-        applicationPatch: ApplicationPatch
-    ): Promise<ApplicationOut> {
-        return this.api.v1ApplicationPatch({ appId, applicationPatch });
-    }
-
-    public delete(appId: string): Promise<void> {
-        return this.api.v1ApplicationDelete({ appId });
-    }
+  private readonly api: ApplicationApi;
+
+  public constructor(config: Configuration) {
+    this.api = new ApplicationApi(config);
+  }
+
+  public list(options?: ApplicationListOptions): Promise<ListResponseApplicationOut> {
+    const iterator = options?.iterator ?? undefined;
+    return this.api.v1ApplicationList({ ...options, iterator });
+  }
+
+  public create(
+    applicationIn: ApplicationIn,
+    options?: PostOptions
+  ): Promise<ApplicationOut> {
+    return this.api.v1ApplicationCreate({ applicationIn, ...options });
+  }
+
+  public getOrCreate(
+    applicationIn: ApplicationIn,
+    options?: PostOptions
+  ): Promise<ApplicationOut> {
+    return this.api.v1ApplicationCreate({
+      applicationIn,
+      ...options,
+      getIfExists: true,
+    });
+  }
+
+  public get(appId: string): Promise<ApplicationOut> {
+    return this.api.v1ApplicationGet({ appId });
+  }
+
+  public update(appId: string, applicationIn: ApplicationIn): Promise<ApplicationOut> {
+    return this.api.v1ApplicationUpdate({ appId, applicationIn });
+  }
+
+  public patch(
+    appId: string,
+    applicationPatch: ApplicationPatch
+  ): Promise<ApplicationOut> {
+    return this.api.v1ApplicationPatch({ appId, applicationPatch });
+  }
+
+  public delete(appId: string): Promise<void> {
+    return this.api.v1ApplicationDelete({ appId });
+  }
 }

javascript/src/api/application.ts

@@ -1,42 +1,42 @@
 import {
-    Configuration,
-    AuthenticationApi,
-    DashboardAccessOut,
-    AppPortalAccessOut,
-    AppPortalAccessIn,
+  Configuration,
+  AuthenticationApi,
+  DashboardAccessOut,
+  AppPortalAccessOut,
+  AppPortalAccessIn,
 } from "../openapi";
 import { PostOptions } from "../util";
 
 export class Authentication {
-    private readonly api: AuthenticationApi;
+  private readonly api: AuthenticationApi;
 
-    public constructor(config: Configuration) {
-        this.api = new AuthenticationApi(config);
-    }
+  public constructor(config: Configuration) {
+    this.api = new AuthenticationApi(config);
+  }
 
-    public appPortalAccess(
-        appId: string,
-        appPortalAccessIn: AppPortalAccessIn,
-        options?: PostOptions
-    ): Promise<AppPortalAccessOut> {
-        return this.api.v1AuthenticationAppPortalAccess({
-            appId,
-            appPortalAccessIn,
-            ...options,
-        });
-    }
+  public appPortalAccess(
+    appId: string,
+    appPortalAccessIn: AppPortalAccessIn,
+    options?: PostOptions
+  ): Promise<AppPortalAccessOut> {
+    return this.api.v1AuthenticationAppPortalAccess({
+      appId,
+      appPortalAccessIn,
+      ...options,
+    });
+  }
 
-    public dashboardAccess(
-        appId: string,
-        options?: PostOptions
-    ): Promise<DashboardAccessOut> {
-        return this.api.v1AuthenticationDashboardAccess({
-            appId,
-            ...options,
-        });
-    }
+  public dashboardAccess(
+    appId: string,
+    options?: PostOptions
+  ): Promise<DashboardAccessOut> {
+    return this.api.v1AuthenticationDashboardAccess({
+      appId,
+      ...options,
+    });
+  }
 
-    public logout(options?: PostOptions): Promise<void> {
-        return this.api.v1AuthenticationLogout({ ...options });
-    }
+  public logout(options?: PostOptions): Promise<void> {
+    return this.api.v1AuthenticationLogout({ ...options });
+  }
 }

javascript/src/api/authentication.ts

@@ -1,36 +1,36 @@
 import {
-    Configuration,
-    BackgroundTaskStatus,
-    BackgroundTaskType,
-    BackgroundTaskOut,
-    ListResponseBackgroundTaskOut,
-    BackgroundTasksApi,
+  Configuration,
+  BackgroundTaskStatus,
+  BackgroundTaskType,
+  BackgroundTaskOut,
+  ListResponseBackgroundTaskOut,
+  BackgroundTasksApi,
 } from "../openapi";
 
 export interface BackgroundTaskListOptions {
-    iterator?: string | null;
-    limit?: number;
-    status?: BackgroundTaskStatus;
-    task?: BackgroundTaskType;
+  iterator?: string | null;
+  limit?: number;
+  status?: BackgroundTaskStatus;
+  task?: BackgroundTaskType;
 }
 
 export class BackgroundTask {
-    private readonly api: BackgroundTasksApi;
+  private readonly api: BackgroundTasksApi;
 
-    public constructor(config: Configuration) {
-        this.api = new BackgroundTasksApi(config);
-    }
+  public constructor(config: Configuration) {
+    this.api = new BackgroundTasksApi(config);
+  }
 
-    public listByEndpoint(
-        options?: BackgroundTaskListOptions
-    ): Promise<ListResponseBackgroundTaskOut> {
-        const iterator = options?.iterator ?? undefined;
-        return this.api.listBackgroundTasks({ ...options, iterator });
-    }
+  public listByEndpoint(
+    options?: BackgroundTaskListOptions
+  ): Promise<ListResponseBackgroundTaskOut> {
+    const iterator = options?.iterator ?? undefined;
+    return this.api.listBackgroundTasks({ ...options, iterator });
+  }
 
-    public get(taskId: string): Promise<BackgroundTaskOut> {
-        return this.api.getBackgroundTask({
-            taskId,
-        });
-    }
+  public get(taskId: string): Promise<BackgroundTaskOut> {
+    return this.api.getBackgroundTask({
+      taskId,
+    });
+  }
 }