fix: add missing newline at end of package.json and standardize quotes in pnpm-workspace.yaml

refactor: update formatting in sessionManager.test.ts for consistency

Author: dtoxvanilla1991

package.json

@@ -90,4 +90,4 @@
     "express-session": "^1.18.1",
     "qs": "^6.14.0"
   }
-}
+}

pnpm-workspace.yaml

@@ -1,2 +1,2 @@
 overrides:
-  '@kinde/js-utils': link:../js-utils
+  "@kinde/js-utils": link:../js-utils

src/index.ts

@@ -21,7 +21,7 @@ export { jwtVerify };
  */
 export const setupKinde = <G extends GrantType>(
   config: SetupConfig<G>,
-  app: Express
+  app: Express,
 ) => {
   // setting up expressSession layer first for Kinde
   setupKindeSession(app);

src/setup/sessionManager.test.ts

@@ -1,103 +1,96 @@
-import {
-  describe,
-  it,
-  expect,
-  beforeEach,
-} from 'vitest';
-import express, {
-  type Express,
-} from 'express';
-import request from 'supertest';
-import session from 'express-session';
-import { setupKindeSession } from './sessionManager.js';
+import { describe, it, expect, beforeEach } from "vitest";
+import express, { type Express } from "express";
+import request from "supertest";
+import session from "express-session";
+import { setupKindeSession } from "./sessionManager.js";
 
-describe('sessionManager', () => {
+describe("sessionManager", () => {
   let app: Express;
 
   beforeEach(() => {
     app = express();
     app.use(express.json());
   });
 
-  describe('setupKindeSession()', () => {
-    it('should add session middleware if it has not been added', () => {
+  describe("setupKindeSession()", () => {
+    it("should add session middleware if it has not been added", () => {
       setupKindeSession(app);
       const sessionMiddleware = app._router.stack.find(
-        (layer) => layer.name === 'session'
+        (layer) => layer.name === "session",
       );
       expect(sessionMiddleware).toBeDefined();
     });
 
-    it('should not add session middleware if it has already been added', () => {
+    it("should not add session middleware if it has already been added", () => {
       app.use(
         session({
           secret:
             process.env.TEST_SESSION_SECRET || "test-secret-" + Date.now(),
           resave: false,
           saveUninitialized: true,
-        })
+        }),
       );
       const sessionMiddlewareCount = app._router.stack.filter(
-        (layer) => layer.name === 'session'
+        (layer) => layer.name === "session",
       ).length;
       expect(sessionMiddlewareCount).toBe(1);
       setupKindeSession(app);
       const sessionMiddlewareCountAfter = app._router.stack.filter(
-        (layer) => layer.name === 'session'
+        (layer) => layer.name === "session",
       ).length;
       expect(sessionMiddlewareCountAfter).toBe(1);
     });
 
-    it('should add getSessionManager middleware', () => {
+    it("should add getSessionManager middleware", () => {
       const initialStackLength = app._router.stack.length;
       setupKindeSession(app);
       const finalStackLength = app._router.stack.length;
       expect(finalStackLength).toBeGreaterThan(initialStackLength);
     });
   });
 
-  describe('ExpressSessionManager functionality', () => {
+  describe("ExpressSessionManager functionality", () => {
     beforeEach(() => {
       setupKindeSession(app);
-      app.get('/session-functions', (req, res) => {
+      app.get("/session-functions", (req, res) => {
         res.json({
-          setSessionItem: typeof req.setSessionItem === 'function',
-          getSessionItem: typeof req.getSessionItem === 'function',
-          removeSessionItem: typeof req.removeSessionItem === 'function',
-          destroySession: typeof req.destroySession === 'function',
+          setSessionItem: typeof req.setSessionItem === "function",
+          getSessionItem: typeof req.getSessionItem === "function",
+          removeSessionItem: typeof req.removeSessionItem === "function",
+          destroySession: typeof req.destroySession === "function",
         });
       });
 
-      app.post('/set-item', async (req, res) => {
+      app.post("/set-item", async (req, res) => {
         const { key, value } = req.body;
         await req.setSessionItem(key, value);
         res.sendStatus(200);
       });
 
-      app.get('/get-item', async (req, res) => {
+      app.get("/get-item", async (req, res) => {
         const { key } = req.query;
         const value = await req.getSessionItem(key as string);
         res.json({ value });
       });
 
-      app.post('/remove-item', async (req, res) => {
+      app.post("/remove-item", async (req, res) => {
         const { key } = req.body;
         await req.removeSessionItem(key);
         res.sendStatus(200);
       });
 
-      app.post('/destroy', async (req, res) => {
+      app.post("/destroy", async (req, res) => {
         try {
           await req.destroySession();
           res.sendStatus(200);
         } catch {
-          res.status(500).send('Error destroying session');
+          res.status(500).send("Error destroying session");
         }
       });
     });
 
-    it('adds ExpressSessionManager methods to the request object', async () => {
-      const res = await request(app).get('/session-functions');
+    it("adds ExpressSessionManager methods to the request object", async () => {
+      const res = await request(app).get("/session-functions");
       expect(res.status).toBe(200);
       expect(res.body).toEqual({
         setSessionItem: true,
@@ -107,32 +100,32 @@ describe('sessionManager', () => {
       });
     });
 
-    it('sets and gets a session item via ExpressSessionManager', async () => {
+    it("sets and gets a session item via ExpressSessionManager", async () => {
       const agent = request.agent(app);
       await agent
-        .post('/set-item')
-        .send({ key: 'testKey', value: 'testValue' });
-      const res = await agent.get('/get-item').query({ key: 'testKey' });
-      expect(res.body.value).toBe('testValue');
+        .post("/set-item")
+        .send({ key: "testKey", value: "testValue" });
+      const res = await agent.get("/get-item").query({ key: "testKey" });
+      expect(res.body.value).toBe("testValue");
     });
 
-    it('removes a session item via ExpressSessionManager', async () => {
+    it("removes a session item via ExpressSessionManager", async () => {
       const agent = request.agent(app);
       await agent
-        .post('/set-item')
-        .send({ key: 'testKey', value: 'testValue' });
-      await agent.post('/remove-item').send({ key: 'testKey' });
-      const res = await agent.get('/get-item').query({ key: 'testKey' });
+        .post("/set-item")
+        .send({ key: "testKey", value: "testValue" });
+      await agent.post("/remove-item").send({ key: "testKey" });
+      const res = await agent.get("/get-item").query({ key: "testKey" });
       expect(res.body.value).toBe(null);
     });
 
-    it('destroys the session via ExpressSessionManager', async () => {
+    it("destroys the session via ExpressSessionManager", async () => {
       const agent = request.agent(app);
       await agent
-        .post('/set-item')
-        .send({ key: 'testKey', value: 'testValue' });
-      await agent.post('/destroy');
-      const res = await agent.get('/get-item').query({ key: 'testKey' });
+        .post("/set-item")
+        .send({ key: "testKey", value: "testValue" });
+      await agent.post("/destroy");
+      const res = await agent.get("/get-item").query({ key: "testKey" });
       expect(res.body.value).toBe(null);
     });
   });