@quiltt/core 3.6.9 → 3.6.10

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @quiltt/core
 
+## 3.6.10
+
+### Patch Changes
+
+- [#268](https://github.com/quiltt/quiltt-js/pull/268) [`8a82094`](https://github.com/quiltt/quiltt-js/commit/8a82094a709d0d7e1478ec32142be33825323708) Thanks [@zubairaziz](https://github.com/zubairaziz)! - Migrate linter to Biome
+
 ## 3.6.9
 
 ### Patch Changes

package/dist/{SubscriptionLink-client-CO7h3iAK.js → SubscriptionLink-client-bQ_XR09W.js}

@@ -1,7 +1,7 @@
 'use client';
 import { ApolloLink, Observable as Observable$1 } from '@apollo/client/core/index.js';
 import { print } from 'graphql';
-import { c as createConsumer } from './index-client-Bp2EphG9.js';
+import { c as createConsumer } from './index-client-B02K2JNP.js';
 
 /**
  * An error and type safe wrapper for localStorage.
@@ -13,8 +13,8 @@ import { c as createConsumer } from './index-client-Bp2EphG9.js';
         this.observers = {};
         this.isEnabled = ()=>{
             try {
-                localStorage.setItem(`quiltt.ping`, 'pong');
-                localStorage.removeItem(`quiltt.ping`);
+                localStorage.setItem('quiltt.ping', 'pong');
+                localStorage.removeItem('quiltt.ping');
                 return true;
             } catch (error) {
                 return false;
@@ -61,7 +61,7 @@ import { c as createConsumer } from './index-client-Bp2EphG9.js';
         // if there is a key, then trigger the related updates. If there is not key
         // it means that a record has been removed and everything needs to be rechecked.
         this.handleStorageEvent = (event)=>{
-            if (event.key && event.key.includes('quiltt.')) {
+            if (event.key?.includes('quiltt.')) {
                 const newState = event.newValue ? JSON.parse(event.newValue) : null;
                 if (this.observers[event.key]) {
                     this.observers[event.key].forEach((update)=>update(newState));
@@ -105,7 +105,7 @@ import { c as createConsumer } from './index-client-Bp2EphG9.js';
 
 /**
  * This is designed to support effectively an in memory key value store singleton,
- * similar to localstorage, but allows you to subscribe to changes within the current
+ * similar to localStorage, but allows you to subscribe to changes within the current
  * window.
  */ class MemoryStorage {
     constructor(){
@@ -113,9 +113,8 @@ import { c as createConsumer } from './index-client-Bp2EphG9.js';
         this.get = (key)=>{
             if (this.observables[key]) {
                 return this.observables[key].get();
-            } else {
-                return undefined;
             }
+            return undefined;
         };
         this.set = (key, state)=>{
             if (!this.observables[key]) {
@@ -204,7 +203,7 @@ import { c as createConsumer } from './index-client-Bp2EphG9.js';
  */ const GlobalStorage = new Storage();
 
 var name = "@quiltt/core";
-var version$1 = "3.6.9";
+var version$1 = "3.6.10";
 
 const QUILTT_API_INSECURE = (()=>{
     try {
@@ -253,7 +252,7 @@ class ActionCableLink extends ApolloLink {
     request(operation, _next) {
         const token = GlobalStorage.get('session');
         if (!token) {
-            console.warn(`QuilttClient attempted to send an unauthenticated Subscription`);
+            console.warn('QuilttClient attempted to send an unauthenticated Subscription');
             return null;
         }
         if (!this.cables[token]) {

package/dist/{index-client-Bp2EphG9.js → index-client-B02K2JNP.js}

@@ -1,5 +1,5 @@
 'use client';
-import { d as debugging } from './SubscriptionLink-client-CO7h3iAK.js';
+import { d as debugging } from './SubscriptionLink-client-bQ_XR09W.js';
 
 var adapters = {
     logger: typeof globalThis !== 'undefined' ? globalThis.console : undefined,
@@ -33,6 +33,7 @@ class ConnectionMonitor {
     start() {
         if (!this.isRunning()) {
             this.startedAt = now();
+            // biome-ignore lint/performance/noDelete: <explanation>
             delete this.stoppedAt;
             this.startPolling();
             addEventListener('visibilitychange', this.visibilityDidChange);
@@ -56,6 +57,7 @@ class ConnectionMonitor {
     recordConnect() {
         this.reconnectAttempts = 0;
         this.recordPing();
+        // biome-ignore lint/performance/noDelete: <explanation>
         delete this.disconnectedAt;
         logger.log('ConnectionMonitor recorded connect');
     }
@@ -79,7 +81,7 @@ class ConnectionMonitor {
     }
     getPollInterval() {
         const { staleThreshold, reconnectionBackoffRate } = this.constructor;
-        const backoff = Math.pow(1 + reconnectionBackoffRate, Math.min(this.reconnectAttempts, 10));
+        const backoff = (1 + reconnectionBackoffRate) ** Math.min(this.reconnectAttempts, 10);
         const jitterMax = this.reconnectAttempts === 0 ? 1.0 : reconnectionBackoffRate;
         const jitter = jitterMax * Math.random();
         return staleThreshold * 1000 * backoff * (1 + jitter);
@@ -158,28 +160,26 @@ class Connection {
         if (this.isOpen()) {
             this.webSocket.send(JSON.stringify(data));
             return true;
-        } else {
-            return false;
         }
+        return false;
     }
     open() {
         if (this.isActive()) {
             logger.log(`Attempted to open WebSocket, but existing socket is ${this.getState()}`);
             return false;
-        } else {
-            const socketProtocols = [
-                ...protocols,
-                ...this.consumer.subprotocols || []
-            ];
-            logger.log(`Opening WebSocket, current state is ${this.getState()}, subprotocols: ${socketProtocols}`);
-            if (this.webSocket) {
-                this.uninstallEventHandlers();
-            }
-            this.webSocket = new adapters.WebSocket(this.consumer.url, socketProtocols);
-            this.installEventHandlers();
-            this.monitor.start();
-            return true;
         }
+        const socketProtocols = [
+            ...protocols,
+            ...this.consumer.subprotocols || []
+        ];
+        logger.log(`Opening WebSocket, current state is ${this.getState()}, subprotocols: ${socketProtocols}`);
+        if (this.webSocket) {
+            this.uninstallEventHandlers();
+        }
+        this.webSocket = new adapters.WebSocket(this.consumer.url, socketProtocols);
+        this.installEventHandlers();
+        this.monitor.start();
+        return true;
     }
     close({ allowReconnect } = {
         allowReconnect: true
@@ -246,7 +246,7 @@ class Connection {
     }
     uninstallEventHandlers() {
         for(const eventName in this.events){
-            this.webSocket[`on${eventName}`] = function() {};
+            this.webSocket[`on${eventName}`] = ()=>{};
         }
     }
 }
@@ -278,11 +278,10 @@ Connection.prototype.events = {
                     return this.subscriptions.notify(identifier, 'connected', {
                         reconnected: true
                     });
-                } else {
-                    return this.subscriptions.notify(identifier, 'connected', {
-                        reconnected: false
-                    });
                 }
+                return this.subscriptions.notify(identifier, 'connected', {
+                    reconnected: false
+                });
             case message_types.rejection:
                 return this.subscriptions.reject(identifier);
             default:
@@ -315,7 +314,7 @@ Connection.prototype.events = {
     }
 };
 
-const extend = function(object, properties) {
+const extend = (object, properties)=>{
     if (properties !== null) {
         for(const key in properties){
             const value = properties[key];
@@ -325,6 +324,7 @@ const extend = function(object, properties) {
     return object;
 };
 class Subscription {
+    // biome-ignore lint/style/useDefaultParameterLast: <explanation>
     constructor(consumer, params = {}, mixin){
         this.consumer = consumer;
         this.identifier = JSON.stringify(params);
@@ -539,13 +539,12 @@ function createWebSocketURL(url) {
     if (url && !/^wss?:/i.test(url)) {
         const a = document.createElement('a');
         a.href = url;
-        // eslint-disable-next-line no-self-assign
+        // biome-ignore lint/correctness/noSelfAssign: <explanation>
         a.href = a.href;
         a.protocol = a.protocol.replace('http', 'ws');
         return a.href;
-    } else {
-        return url;
     }
+    return url;
 }
 
 function createConsumer(url = getConfig('url') || INTERNAL.default_mount_path) {

package/dist/index.d.ts

@@ -339,7 +339,7 @@ declare class LocalStorage<T> {
 
 /**
  * This is designed to support effectively an in memory key value store singleton,
- * similar to localstorage, but allows you to subscribe to changes within the current
+ * similar to localStorage, but allows you to subscribe to changes within the current
  * window.
  */
 declare class MemoryStorage<T> {

package/dist/index.js

@@ -1,7 +1,7 @@
 import { ApolloLink, ApolloClient } from '@apollo/client/index.js';
 export { InMemoryCache, gql, useMutation, useQuery, useSubscription } from '@apollo/client/index.js';
-import { G as GlobalStorage, e as endpointGraphQL, v as version, d as debugging, S as SubscriptionLink, a as endpointAuth } from './SubscriptionLink-client-CO7h3iAK.js';
-export { L as LocalStorage, M as MemoryStorage, O as Observable, b as Storage, c as cdnBase, f as endpointWebsockets } from './SubscriptionLink-client-CO7h3iAK.js';
+import { G as GlobalStorage, e as endpointGraphQL, v as version, d as debugging, S as SubscriptionLink, a as endpointAuth } from './SubscriptionLink-client-bQ_XR09W.js';
+export { L as LocalStorage, M as MemoryStorage, O as Observable, b as Storage, c as cdnBase, f as endpointWebsockets } from './SubscriptionLink-client-bQ_XR09W.js';
 import { BatchHttpLink as BatchHttpLink$1 } from '@apollo/client/link/batch-http/index.js';
 import crossfetch from 'cross-fetch';
 import { onError } from '@apollo/client/link/error/index.js';
@@ -25,7 +25,7 @@ var ConnectorSDKEventType;
     request(operation, forward) {
         const token = GlobalStorage.get('session');
         if (!token) {
-            console.warn(`QuilttLink attempted to send an unauthenticated Query`);
+            console.warn('QuilttLink attempted to send an unauthenticated Query');
             return null;
         }
         operation.setContext(({ headers = {} })=>({
@@ -142,7 +142,7 @@ const RETRIES = 10 // 150, 300, 450, 600, 750, 900, 1050, 1200, 1350, 1500 = 8.2
         if (retry && (response.status >= 500 || response.status === 429)) {
             throw new Error('Retryable failure');
         }
-        throw new Error('HTTP error with status ' + response.status);
+        throw new Error(`HTTP error with status ${response.status}`);
     } catch (error) {
         if (retry) {
             const currentRetriesRemaining = retriesRemaining !== undefined ? retriesRemaining : RETRIES;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quiltt/core",
-  "version": "3.6.9",
+  "version": "3.6.10",
   "description": "Javascript API client and utilities for Quiltt",
   "keywords": [
     "quiltt",
@@ -39,16 +39,10 @@
     "graphql-ruby-client": "^1.13.3"
   },
   "devDependencies": {
-    "@trivago/prettier-plugin-sort-imports": "4.1.1",
-    "@types/node": "20.12.7",
+    "@biomejs/biome": "1.8.1",
+    "@types/node": "20.14.5",
     "@types/react": "18.2.73",
-    "@typescript-eslint/eslint-plugin": "5.60.1",
-    "@typescript-eslint/parser": "5.60.1",
     "bunchee": "4.4.8",
-    "eslint": "8.43.0",
-    "eslint-config-prettier": "8.8.0",
-    "eslint-plugin-prettier": "4.2.1",
-    "prettier": "2.8.8",
     "rimraf": "5.0.5",
     "typescript": "5.4.3"
   },
@@ -59,7 +53,7 @@
     "build": "bunchee",
     "clean": "rimraf .turbo dist",
     "dev": "bunchee --watch",
-    "lint": "TIMING=1 eslint --ext .js,.jsx,.ts,.tsx src/ --fix",
+    "lint": "TIMING=1 biome check src/ --fix",
     "typecheck": "tsc --project tsconfig.json --noEmit"
   }
 }

package/src/api/graphql/links/ActionCableLink.ts

@@ -37,7 +37,7 @@ class ActionCableLink extends ApolloLink {
     const token = GlobalStorage.get('session')
 
     if (!token) {
-      console.warn(`QuilttClient attempted to send an unauthenticated Subscription`)
+      console.warn('QuilttClient attempted to send an unauthenticated Subscription')
       return null
     }
 

package/src/api/graphql/links/AuthLink.ts

@@ -14,7 +14,7 @@ export class AuthLink extends ApolloLink {
     const token = GlobalStorage.get('session')
 
     if (!token) {
-      console.warn(`QuilttLink attempted to send an unauthenticated Query`)
+      console.warn('QuilttLink attempted to send an unauthenticated Query')
       return null
     }
 

package/src/api/graphql/links/actioncable/connection.ts

@@ -24,28 +24,26 @@ class Connection {
     if (this.isOpen()) {
       this.webSocket.send(JSON.stringify(data))
       return true
-    } else {
-      return false
     }
+    return false
   }
 
   open() {
     if (this.isActive()) {
       logger.log(`Attempted to open WebSocket, but existing socket is ${this.getState()}`)
       return false
-    } else {
-      const socketProtocols = [...protocols, ...(this.consumer.subprotocols || [])]
-      logger.log(
-        `Opening WebSocket, current state is ${this.getState()}, subprotocols: ${socketProtocols}`
-      )
-      if (this.webSocket) {
-        this.uninstallEventHandlers()
-      }
-      this.webSocket = new adapters.WebSocket(this.consumer.url, socketProtocols)
-      this.installEventHandlers()
-      this.monitor.start()
-      return true
     }
+    const socketProtocols = [...protocols, ...(this.consumer.subprotocols || [])]
+    logger.log(
+      `Opening WebSocket, current state is ${this.getState()}, subprotocols: ${socketProtocols}`
+    )
+    if (this.webSocket) {
+      this.uninstallEventHandlers()
+    }
+    this.webSocket = new adapters.WebSocket(this.consumer.url, socketProtocols)
+    this.installEventHandlers()
+    this.monitor.start()
+    return true
   }
 
   close({ allowReconnect } = { allowReconnect: true }) {
@@ -122,7 +120,7 @@ class Connection {
 
   uninstallEventHandlers() {
     for (const eventName in this.events) {
-      this.webSocket[`on${eventName}`] = function () {}
+      this.webSocket[`on${eventName}`] = () => {}
     }
   }
 }
@@ -152,9 +150,8 @@ Connection.prototype.events = {
         if (this.reconnectAttempted) {
           this.reconnectAttempted = false
           return this.subscriptions.notify(identifier, 'connected', { reconnected: true })
-        } else {
-          return this.subscriptions.notify(identifier, 'connected', { reconnected: false })
         }
+        return this.subscriptions.notify(identifier, 'connected', { reconnected: false })
       case message_types.rejection:
         return this.subscriptions.reject(identifier)
       default:

package/src/api/graphql/links/actioncable/connection_monitor.ts

@@ -18,6 +18,7 @@ class ConnectionMonitor {
   start() {
     if (!this.isRunning()) {
       this.startedAt = now()
+      // biome-ignore lint/performance/noDelete: <explanation>
       delete this.stoppedAt
       this.startPolling()
       addEventListener('visibilitychange', this.visibilityDidChange)
@@ -47,6 +48,7 @@ class ConnectionMonitor {
   recordConnect() {
     this.reconnectAttempts = 0
     this.recordPing()
+    // biome-ignore lint/performance/noDelete: <explanation>
     delete this.disconnectedAt
     logger.log('ConnectionMonitor recorded connect')
   }
@@ -76,7 +78,7 @@ class ConnectionMonitor {
 
   getPollInterval() {
     const { staleThreshold, reconnectionBackoffRate } = this.constructor
-    const backoff = Math.pow(1 + reconnectionBackoffRate, Math.min(this.reconnectAttempts, 10))
+    const backoff = (1 + reconnectionBackoffRate) ** Math.min(this.reconnectAttempts, 10)
     const jitterMax = this.reconnectAttempts === 0 ? 1.0 : reconnectionBackoffRate
     const jitter = jitterMax * Math.random()
     return staleThreshold * 1000 * backoff * (1 + jitter)

package/src/api/graphql/links/actioncable/consumer.ts

@@ -75,13 +75,12 @@ export function createWebSocketURL(url: string | (() => string)): string {
   if (url && !/^wss?:/i.test(url)) {
     const a = document.createElement('a')
     a.href = url
-    // eslint-disable-next-line no-self-assign
+    // biome-ignore lint/correctness/noSelfAssign: <explanation>
     a.href = a.href
     a.protocol = a.protocol.replace('http', 'ws')
     return a.href
-  } else {
-    return url
   }
+  return url
 }
 
 export default Consumer

package/src/api/graphql/links/actioncable/subscription.ts

@@ -2,7 +2,7 @@ import type { Consumer } from './consumer'
 
 export type Data = { [id: string]: string | object | null | undefined }
 
-const extend = function (object: Data, properties: Data) {
+const extend = (object: Data, properties: Data) => {
   if (properties !== null) {
     for (const key in properties) {
       const value = properties[key]
@@ -16,6 +16,7 @@ export class Subscription {
   consumer: Consumer
   identifier: string
 
+  // biome-ignore lint/style/useDefaultParameterLast: <explanation>
   constructor(consumer: Consumer, params: Data = {}, mixin: Data) {
     this.consumer = consumer
     this.identifier = JSON.stringify(params)

package/src/api/graphql/links/actioncable/subscriptions.ts

@@ -79,7 +79,7 @@ export class Subscriptions {
   }
 
   notify(subscription: Subscription, callbackName: string, ...args: any[]) {
-    let subscriptions
+    let subscriptions: Array<Subscription>
     if (typeof subscription === 'string') {
       subscriptions = this.findAll(subscription)
     } else {

package/src/api/rest/fetchWithRetry.ts

@@ -55,7 +55,7 @@ export const fetchWithRetry = async <T>(
       throw new Error('Retryable failure')
     }
 
-    throw new Error('HTTP error with status ' + response.status)
+    throw new Error(`HTTP error with status ${response.status}`)
   } catch (error) {
     if (retry) {
       const currentRetriesRemaining = retriesRemaining !== undefined ? retriesRemaining : RETRIES

package/src/storage/Local.ts

@@ -1,5 +1,5 @@
-import type { Maybe } from '../types'
 import type { Observer } from '../Observable'
+import type { Maybe } from '../types'
 
 /**
  * An error and type safe wrapper for localStorage.
@@ -18,8 +18,8 @@ export class LocalStorage<T> {
 
   isEnabled = (): boolean => {
     try {
-      localStorage.setItem(`quiltt.ping`, 'pong')
-      localStorage.removeItem(`quiltt.ping`)
+      localStorage.setItem('quiltt.ping', 'pong')
+      localStorage.removeItem('quiltt.ping')
       return true
     } catch (error) {
       return false
@@ -78,7 +78,7 @@ export class LocalStorage<T> {
   // if there is a key, then trigger the related updates. If there is not key
   // it means that a record has been removed and everything needs to be rechecked.
   private handleStorageEvent = (event: StorageEvent) => {
-    if (event.key && event.key.includes('quiltt.')) {
+    if (event.key?.includes('quiltt.')) {
       const newState = event.newValue ? JSON.parse(event.newValue) : null
 
       if (this.observers[event.key]) {

package/src/storage/Memory.ts

@@ -5,7 +5,7 @@ import type { Observer } from '../Observable'
 
 /**
  * This is designed to support effectively an in memory key value store singleton,
- * similar to localstorage, but allows you to subscribe to changes within the current
+ * similar to localStorage, but allows you to subscribe to changes within the current
  * window.
  */
 export class MemoryStorage<T> {
@@ -14,9 +14,8 @@ export class MemoryStorage<T> {
   get = (key: string) => {
     if (this.observables[key]) {
       return this.observables[key].get()
-    } else {
-      return undefined
     }
+    return undefined
   }
 
   set = (key: string, state: Maybe<T> | undefined): void => {