Merge pull request #223 from authzed/add-prettier-to-repo

Add prettier to repo

Author: tstirrat15

.github/workflows/test.yaml

@@ -47,9 +47,11 @@ jobs:
         with:
           useLockFile: false
       - name: Run lint
-        run: CI=true yarn lint
+        run: "CI=true yarn lint"
+      - name: Run prettier
+        run: "CI=true yarn run prettier src -c"
       - name: Run Yarn tests
-        run: CI=true yarn only-run-tests
+        run: "CI=true yarn only-run-tests"
   build-js-client:
     name: Build and Test JS client
     runs-on: "depot-ubuntu-24.04-small"

.prettierignore

@@ -0,0 +1 @@
+src/authzedapi

package.json

@@ -56,6 +56,7 @@
     "@typescript-eslint/parser": "^8.29.1",
     "eslint": "^9.24.0",
     "grpc-tools": "^1.13.0",
+    "prettier": "^3.5.3",
     "rollup": "^4.40.0",
     "tsc-esm-fix": "^3.1.2",
     "typescript": "^5.8",

src/__utils__/helpers.ts

@@ -1,9 +1,9 @@
 /**
  * Generates a random token with a prefix to support
  * unique, idempotent local testing.
- * @param prefix 
- * @returns 
+ * @param prefix
+ * @returns
  */
 export function generateTestToken(prefix: string): string {
-  return `${prefix}-${Date.now().toString()}`
-}
+  return `${prefix}-${Date.now().toString()}`;
+}

src/full-promises.test.ts

@@ -1,14 +1,18 @@
 import { ClientSecurity } from "./util.js";
 import * as v1 from "./v1.js";
 import { Consistency } from "./v1.js";
-import { generateTestToken } from './__utils__/helpers.js' 
-import { describe, it, expect } from 'vitest'
+import { generateTestToken } from "./__utils__/helpers.js";
+import { describe, it, expect } from "vitest";
 
 describe("a check following a write of schema and relationships", () => {
   it("should succeed", async () => {
     // Write the schema.
-    const token = generateTestToken('full-promises')
-    const { promises: v1client } = v1.NewClient(token, "localhost:50051", ClientSecurity.INSECURE_LOCALHOST_ALLOWED);
+    const token = generateTestToken("full-promises");
+    const { promises: v1client } = v1.NewClient(
+      token,
+      "localhost:50051",
+      ClientSecurity.INSECURE_LOCALHOST_ALLOWED,
+    );
 
     const writeSchemaRequest = v1.WriteSchemaRequest.create({
       schema: `
@@ -21,7 +25,7 @@ describe("a check following a write of schema and relationships", () => {
   `,
     });
 
-    await v1client.writeSchema(writeSchemaRequest)
+    await v1client.writeSchema(writeSchemaRequest);
     // Create the relationship between the resource and the user.
     const resource = v1.ObjectReference.create({
       objectType: "test/resource",
@@ -54,7 +58,7 @@ describe("a check following a write of schema and relationships", () => {
       updates: [update],
     });
 
-    const response = await v1client.writeRelationships(writeRequest)
+    const response = await v1client.writeRelationships(writeRequest);
     expect(response).toBeTruthy();
 
     const checkPermissionRequest = v1.CheckPermissionRequest.create({
@@ -66,12 +70,14 @@ describe("a check following a write of schema and relationships", () => {
           oneofKind: "fullyConsistent",
           fullyConsistent: true,
         },
-      })
+      }),
     });
 
-    const permissionResponse = await v1client.checkPermission(checkPermissionRequest)
+    const permissionResponse = await v1client.checkPermission(
+      checkPermissionRequest,
+    );
     expect(permissionResponse?.permissionship).toBe(
-      v1.CheckPermissionResponse_Permissionship.HAS_PERMISSION
+      v1.CheckPermissionResponse_Permissionship.HAS_PERMISSION,
     );
   });
 });

src/full.test.ts

@@ -1,85 +1,90 @@
 import { ClientSecurity } from "./util.js";
 import * as v1 from "./v1.js";
 import { Consistency } from "./v1.js";
-import { generateTestToken } from './__utils__/helpers.js'
-import { describe, it, expect } from 'vitest'
+import { generateTestToken } from "./__utils__/helpers.js";
+import { describe, it, expect } from "vitest";
 
 describe("a check following a write of schema and relationships", () => {
-  it("should succeed", () => new Promise<void>((done) => {
-    // Write the schema.
-    const token = generateTestToken('full-test')
-    const v1client = v1.NewClient(token, "localhost:50051", ClientSecurity.INSECURE_LOCALHOST_ALLOWED);
+  it("should succeed", () =>
+    new Promise<void>((done) => {
+      // Write the schema.
+      const token = generateTestToken("full-test");
+      const v1client = v1.NewClient(
+        token,
+        "localhost:50051",
+        ClientSecurity.INSECURE_LOCALHOST_ALLOWED,
+      );
 
-    const writeSchemaRequest = v1.WriteSchemaRequest.create({
-      schema: `
+      const writeSchemaRequest = v1.WriteSchemaRequest.create({
+        schema: `
   definition test/user {}
   
   definition test/resource {
       relation viewer: test/user
       permission view = viewer
   }
   `,
-    });
-
-    v1client.writeSchema(writeSchemaRequest, function (err) {
-      expect(err).toBe(null);
-
-      // Create the relationship between the resource and the user.
-      const resource = v1.ObjectReference.create({
-        objectType: "test/resource",
-        objectId: "someresource",
-      });
-
-      // Create the user reference.
-      const userref = v1.ObjectReference.create({
-        objectType: "test/user",
-        objectId: "someuser",
-      });
-
-      const user = v1.SubjectReference.create({
-        object: userref,
       });
 
-      const relationship = v1.Relationship.create({
-        resource,
-        relation: "viewer",
-        subject: user,
-      });
+      v1client.writeSchema(writeSchemaRequest, function (err) {
+        expect(err).toBe(null);
 
-      const update = v1.RelationshipUpdate.create({
-        operation: v1.RelationshipUpdate_Operation.CREATE,
-        relationship: relationship,
-      });
+        // Create the relationship between the resource and the user.
+        const resource = v1.ObjectReference.create({
+          objectType: "test/resource",
+          objectId: "someresource",
+        });
 
-      // Write the relationship.
-      const writeRequest = v1.WriteRelationshipsRequest.create({
-        updates: [update],
-      });
+        // Create the user reference.
+        const userref = v1.ObjectReference.create({
+          objectType: "test/user",
+          objectId: "someuser",
+        });
 
-      v1client.writeRelationships(writeRequest, function (err, response) {
-        expect(err).toBe(null);
-        expect(response).toBeTruthy();
+        const user = v1.SubjectReference.create({
+          object: userref,
+        });
 
-        const checkPermissionRequest = v1.CheckPermissionRequest.create({
+        const relationship = v1.Relationship.create({
           resource,
-          permission: "view",
+          relation: "viewer",
           subject: user,
-          consistency: Consistency.create({
-            requirement: {
-              oneofKind: "fullyConsistent",
-              fullyConsistent: true,
-            },
-          })
         });
 
-        v1client.checkPermission(checkPermissionRequest, (err, response) => {
+        const update = v1.RelationshipUpdate.create({
+          operation: v1.RelationshipUpdate_Operation.CREATE,
+          relationship: relationship,
+        });
+
+        // Write the relationship.
+        const writeRequest = v1.WriteRelationshipsRequest.create({
+          updates: [update],
+        });
+
+        v1client.writeRelationships(writeRequest, function (err, response) {
           expect(err).toBe(null);
-          expect(response?.permissionship).toBe(
-            v1.CheckPermissionResponse_Permissionship.HAS_PERMISSION
-          );
-          done();
+          expect(response).toBeTruthy();
+
+          const checkPermissionRequest = v1.CheckPermissionRequest.create({
+            resource,
+            permission: "view",
+            subject: user,
+            consistency: Consistency.create({
+              requirement: {
+                oneofKind: "fullyConsistent",
+                fullyConsistent: true,
+              },
+            }),
+          });
+
+          v1client.checkPermission(checkPermissionRequest, (err, response) => {
+            expect(err).toBe(null);
+            expect(response?.permissionship).toBe(
+              v1.CheckPermissionResponse_Permissionship.HAS_PERMISSION,
+            );
+            done();
+          });
         });
       });
-    });
-  }));
+    }));
 });

src/index.ts

@@ -1,4 +1,4 @@
 "use strict";
 
-export * as protobuf from './protobuf.js';
-export * as v1 from './v1.js';
+export * as protobuf from "./protobuf.js";
+export * as v1 from "./v1.js";

src/protobuf.ts

@@ -1,4 +1,4 @@
-export * from './authzedapi/google/protobuf/descriptor.js';
-export * from './authzedapi/google/protobuf/duration.js';
-export * from './authzedapi/google/protobuf/struct.js';
-export * from './authzedapi/google/protobuf/timestamp.js';
+export * from "./authzedapi/google/protobuf/descriptor.js";
+export * from "./authzedapi/google/protobuf/duration.js";
+export * from "./authzedapi/google/protobuf/struct.js";
+export * from "./authzedapi/google/protobuf/timestamp.js";

src/types.ts

@@ -6,7 +6,7 @@ import {
   ClientWritableStream,
   Metadata,
   ServiceError,
-} from '@grpc/grpc-js';
+} from "@grpc/grpc-js";
 
 /**
  * Picks all function properties of a type that except for those with a specific return type
@@ -37,7 +37,7 @@ export type OmitBaseMethods<C extends B, B> = {
 export type StreamCall<T, U> = (
   request: T,
   metadata?: Metadata | CallOptions,
-  options?: CallOptions
+  options?: CallOptions,
 ) => ClientReadableStream<U>;
 
 /**
@@ -54,8 +54,8 @@ export type WritableStreamCall<T, U> = (
     | undefined,
   callback?: (
     err: ServiceError | null,
-    value?: U | undefined
-  ) => void | undefined
+    value?: U | undefined,
+  ) => void | undefined,
 ) => ClientWritableStream<T>;
 
 /**
@@ -65,7 +65,7 @@ export type UnaryCall<T, U> = (
   request: T,
   metadata: Metadata | CallOptions,
   options: Partial<CallOptions>,
-  callback: (err: ServiceError | null, res?: U) => void
+  callback: (err: ServiceError | null, res?: U) => void,
 ) => ClientUnaryCall;
 
 /**
@@ -74,7 +74,7 @@ export type UnaryCall<T, U> = (
 export type PromisifiedCall<T, U> = (
   request: T,
   metadata?: Metadata,
-  options?: Partial<CallOptions>
+  options?: Partial<CallOptions>,
 ) => Promise<U>;
 
 /**