@iebh/tera-fy 2.3.0 → 2.3.2

package/lib/terafy.server.ts

@@ -1,19 +1,36 @@
+/* eslint-disable @typescript-eslint/no-redundant-type-constituents */
 // Global 'app' declaration
-declare var app: any;
+declare let app: any;
 
 // Global window augmentation
 declare global {
     interface Window {
+        // eslint-disable-next-line no-unused-vars
         panic(text: any): void;
     }
 }
 
+interface UiWindowPermissions {
+	popup?: boolean;
+	location?: boolean;
+	menubar?: boolean;
+	status?: boolean;
+	scrollbars?: boolean;
+}
+
+interface UiWindowOptions {
+	width?: number;
+	height?: number;
+	center?: boolean;
+	permissions?: UiWindowPermissions;
+}
+
 import {cloneDeep} from 'lodash-es';
 import mixin from '#utils/mixin';
 import {nanoid} from 'nanoid';
 import pathTools from '#utils/pathTools';
 import promiseDefer from '#utils/pDefer';
-// @ts-ignore
+// @ts-expect-error TODO: Remove when reflib gets declaration file
 import Reflib from '@iebh/reflib';
 import {reactive} from 'vue';
 
@@ -23,7 +40,6 @@ import {reactive} from 'vue';
 * @class TeraFyServer
 */
 
-/* globals globalThis, app */
 export default class TeraFyServer {
 
 	/**
@@ -120,7 +136,7 @@ export default class TeraFyServer {
 			case TeraFyServer.SERVERMODE_TERA:
 			case TeraFyServer.SERVERMODE_FRAME: {
 				// Server is the top-level window so we need to send messages to an embedded iFrame
-				let iFrame = document.querySelector('iframe#external') as HTMLIFrameElement | null;
+				const iFrame = document.querySelector<HTMLIFrameElement>('iframe#external');
 				if (!iFrame) {
 					this.debug('INFO', 2, 'Cannot locate TERA-FY top-level->iFrame#external - maybe there is none');
 					return mixin(this, {
@@ -211,7 +227,7 @@ export default class TeraFyServer {
 	send(message: any): Promise<any> {
 		if (!this.messageEvent?.source) throw new Error('send() requires a messageEvent with a source');
 
-		let id = nanoid();
+		const id = nanoid();
 
 		this.acceptPostboxes[id] = {}; // Stub for the deferred promise
 		this.acceptPostboxes[id].promise = new Promise((resolve, reject) => {
@@ -289,7 +305,7 @@ export default class TeraFyServer {
 		// Ignore messages from the same origin (potential loops)
 		if (typeof window !== 'undefined' && rawMessage.origin === window.location.origin) return;
 
-		let message = rawMessage.data;
+		const message = rawMessage.data;
 		// Ensure message is an object and has TERA property
 		if (typeof message !== 'object' || message === null || !message.TERA) return;
 		this.debug('INFO', 3, 'Recieved message', message);
@@ -432,14 +448,14 @@ export default class TeraFyServer {
 	* @returns {Promise<User>} The current logged in user or null if none
 	*/
 	getUser(options?: any): Promise<any | null> {
-		let settings = {
+		const settings = {
 			forceRetry: false,
 			waitPromises: true,
 			...options,
 		};
 
-		let $auth = app.service('$auth');
-		let $subscriptions = app.service('$subscriptions');
+		const $auth = app.service('$auth');
+		const $subscriptions = app.service('$subscriptions');
 
 		return Promise.resolve()
 			.then(()=> settings.waitPromises && Promise.all([
@@ -584,7 +600,7 @@ export default class TeraFyServer {
 			try {
 				lsState = JSON.parse(lsState);
 
-				let $auth = app.service('$auth');
+				const $auth = app.service('$auth');
 				$auth.state = 'user';
 				$auth.ready = true;
 				$auth.isLoggedIn = true;
@@ -605,7 +621,7 @@ export default class TeraFyServer {
 
 		this.debug('INFO', 4, 'localStorage failed - using popup auth instead');
 
-		let focusContent = document.createElement('div');
+		const focusContent = document.createElement('div');
 		focusContent.innerHTML = '<div>Authenticate with <a href="https://tera-tools.com" target="_blank">TERA-tools.com</a></div>'
 			+ '<div class="mt-2"><a class="btn btn-light">Open Popup...</a></div>';
 
@@ -615,13 +631,13 @@ export default class TeraFyServer {
 		);
 
 		// Create a deferred promise which will (eventually) resolve when the downstream window signals its ready
-		let waitOnWindowAuth = promiseDefer();
+		const waitOnWindowAuth = promiseDefer();
 
 		// Create a listener for the message from the downstream window to resolve the promise
-		let listenMessages = ({data}: {data: any}) => {
+		const listenMessages = ({data}: {data: any}) => {
 			this.debug('INFO', 3, 'Recieved message from popup window', {data});
 			if (data.TERA && data.action == 'popupUserState' && data.user) { // Signal sent from landing page - we're logged in, yey!
-				let $auth = app.service('$auth');
+				const $auth = app.service('$auth');
 
 				// Accept user polyfill from opener
 				$auth.state = 'user';
@@ -686,7 +702,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Project|null>} The currently active project, if any
 	*/
 	getProject(): Promise<any | null> {
-		let $projects = app.service('$projects');
+		const $projects = app.service('$projects');
 
 		return $projects.promise()
 			.then(()=> $projects.active
@@ -707,7 +723,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Array<Project>>} Collection of projects the user has access to
 	*/
 	getProjects(): Promise<any[]> {
-		let $projects = app.service('$projects');
+		const $projects = app.service('$projects');
 
 		return $projects.promise()
 			.then(()=> $projects.list.map((project: any) => ({
@@ -744,7 +760,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Project>} The active project
 	*/
 	requireProject(options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			autoRequireUser: true,
 			autoSetActiveProject: true,
 			title: 'Select a project to work with',
@@ -760,7 +776,7 @@ export default class TeraFyServer {
 				if (active) return active; // Use active project
 
 				return new Promise((resolve, reject) => {
-					let askProject = (): Promise<any> => Promise.resolve()
+					const askProject = (): Promise<any> => Promise.resolve()
 						.then(()=> this.selectProject({
 							allowCancel: false,
 						}))
@@ -801,7 +817,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Project>} The active project
 	*/
 	selectProject(options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			title: 'Select a project to work with',
 			allowCancel: true,
 			setActive: false,
@@ -886,7 +902,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Object>} The current project state snapshot
 	*/
 	getProjectState(options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			autoRequire: true,
 			paths: null,
 			...options,
@@ -919,7 +935,7 @@ export default class TeraFyServer {
 	* @returns {Promise<*>} A promise which resolves to `value` when the operation has been dispatched to the server and saved
 	*/
 	setProjectState(path: string | string[], value: any, options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			strategy: 'set',
 			...options,
 		};
@@ -962,7 +978,7 @@ export default class TeraFyServer {
 		let settings = { ...options }; // Initialize settings from the third argument if present
 		if (!app.service('$projects').active) throw new Error('No active project');
 
-		let target = app.service('$projects').active;
+		const target = app.service('$projects').active;
 		let actualValue: any;
 
 		if (typeof path == 'string' || Array.isArray(path)) { // Called as (path, value, options?) Set sub-object
@@ -1049,12 +1065,11 @@ export default class TeraFyServer {
 	* @param {Boolean} [options.allowCancel=true] Allow cancelling the operation. Will throw `'CANCEL'` as the promise rejection if acationed
 	* @param {Boolean} [options.autoRequire=true] Run `requireProject()` automatically before continuing
 	* @param {Boolean} [options.showHiddenFiles=false] Whether hidden data.json files should be shown
-	* @param {FileFilters} [options.filter] Optional file filters
 	*
 	* @returns {Promise<ProjectFile>} The eventually selected file, if in save mode new files are created as stubs
 	*/
 	selectProjectFile(options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			title: 'Select a file',
 			hint: null,
 			save: false,
@@ -1114,7 +1129,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Array<ProjectFile>>} A collection of project files for the given project
 	*/
 	getProjectFiles(options?: any): Promise<any[]> {
-		let settings = {
+		const settings = {
 			autoRequire: true,
 			lazy: true,
 			meta: true,
@@ -1147,7 +1162,7 @@ export default class TeraFyServer {
 	* @returns {Promise<ProjectFile>} The eventual fetched ProjectFile (or requested subkey)
 	*/
 	getProjectFile(name: string, options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			subkey: null,
 			cache: true,
 			...(typeof options == 'string' ? {subkey: options} : options),
@@ -1190,7 +1205,7 @@ export default class TeraFyServer {
 				// Recursively search for the file to handle searching folders
 				return findFileRecursively(files, name);
 			})
-			.then((file: any) => file && settings.subkey ? (file as any)[settings.subkey] : file)
+			.then((file: any) => file && settings.subkey ? (file)[settings.subkey] : file)
 	}
 
 	/**
@@ -1204,7 +1219,7 @@ export default class TeraFyServer {
 	* @returns {*} The file contents in the requested format
 	*/
 	getProjectFileContents(id: string, options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			format: 'blob',
 			...options,
 		};
@@ -1397,7 +1412,7 @@ export default class TeraFyServer {
 
 		if (fileContents === undefined) throw new Error('setProjectFileContents requires contents to save.');
 
-		let settings = {
+		const settings = {
 			id: fileId,
 			autoRequire: true,
 			hint: null,
@@ -1474,7 +1489,7 @@ export default class TeraFyServer {
 
 		let cleanFolderPath = folderPath.trim();
 		if (cleanFolderPath.startsWith('/')) {
-			cleanFolderPath = cleanFolderPath.substring(1);
+			cleanFolderPath = cleanFolderPath.slice(1);
 		}
 		if (cleanFolderPath.endsWith('/')) {
 			cleanFolderPath = cleanFolderPath.slice(0, -1);
@@ -1536,7 +1551,7 @@ export default class TeraFyServer {
 
 		let cleanFolderPath = folderPath.trim();
 		if (cleanFolderPath.startsWith('/')) {
-			cleanFolderPath = cleanFolderPath.substring(1);
+			cleanFolderPath = cleanFolderPath.slice(1);
 		}
 		if (cleanFolderPath.endsWith('/')) {
 			cleanFolderPath = cleanFolderPath.slice(0, -1);
@@ -1603,7 +1618,7 @@ export default class TeraFyServer {
 	* @returns {Promise<Array<Ref>>} A collection of references from the selected file
 	*/
 	selectProjectLibrary(options?: any): Promise<any[]> {
-		let settings = {
+		const settings = {
 			title: 'Select a citation library',
 			hint: null,
 			allowUpload: true,
@@ -1613,7 +1628,7 @@ export default class TeraFyServer {
 			autoRequire: true,
 			filters: {
 				library: true,
-				...(options?.filters ?? {}), // Use filters from options if provided
+				...options?.filters, // Use filters from options if provided
 			},
 			...options,
 		};
@@ -1643,16 +1658,16 @@ export default class TeraFyServer {
 	* @returns {Promise<Array<Ref>>|Promise<*>} A collection of references (default bevahiour) or a whatever format was requested
 	*/
 	getProjectLibrary(id: string, options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			format: 'pojo',
 			autoRequire: true,
+			// eslint-disable-next-line no-unused-vars, @typescript-eslint/no-unused-vars
 			filter: (file: any) => true, // Default filter
-			// @ts-ignore
-			find: (files: any[]) => files.at(0), // Default find
+			find: (files: any[]) => files[0], // Default find
 			...options,
 		};
 
-		let filePath: string = app.service('$projects').decodeFilePath(id);
+		const filePath: string = app.service('$projects').decodeFilePath(id);
 
 		return Promise.resolve()
 			.then(()=> settings.autoRequire && this.requireProject())
@@ -1727,7 +1742,7 @@ export default class TeraFyServer {
 
 		if (libraryRefs === undefined) throw new Error('setProjectLibrary requires refs to save.');
 
-		let settings = {
+		const settings = {
 			id: fileId,
 			refs: libraryRefs,
 			format: 'auto',
@@ -1877,7 +1892,7 @@ export default class TeraFyServer {
 	* @returns {Promise} A promise which resolves when the alert has been dismissed
 	*/
 	uiAlert(text: string | any, options?: any): Promise<void> {
-		let settings = {
+		const settings = {
 			body: 'Alert!',
 			isHtml: false,
 			title: 'TERA',
@@ -1917,7 +1932,7 @@ export default class TeraFyServer {
 	* @returns {Promise} A promise which resolves with `Promise.resolve('OK')` or rejects with `Promise.reject('CANCEL')`
 	*/
 	uiConfirm(text: string | any, options?: any): Promise<'OK'> {
-		let settings = {
+		const settings = {
 			body: 'Confirm?',
 			isHtml: false,
 			title: 'TERA',
@@ -1946,7 +1961,7 @@ export default class TeraFyServer {
 					},
 				],
 			})
-				.then(()=> 'OK' as 'OK') // Resolve with 'OK' if OK button clicked
+				.then(()=> 'OK' as const) // Resolve with 'OK' if OK button clicked
 				.catch(()=> Promise.reject('CANCEL')) // Reject with 'CANCEL' if Cancel button clicked or closed
 		);
 	}
@@ -1956,7 +1971,7 @@ export default class TeraFyServer {
 	* Present some JSON to the user
 	*
 	* @function uiJson
-	* @param {String|Object} [data] Data to display, if (and this doesn't contain 'body' or 'title') this populates `options.body`
+	* @param {String|Object} data Data to display
 	*
 	* @param {Object} [options] Additional options to mutate behaviour
 	* @param {String} [options.body=""] The body text to display above the JSON
@@ -1967,15 +1982,11 @@ export default class TeraFyServer {
 	* @returns {Promise} A promise which resolves with `Promise.resolve('OK')`
 	*/
 	uiJson(data: any, options?: any): Promise<'OK'> {
-		let settings = {
+		const settings = {
 			body: '',
 			isHtml: false,
 			title: 'TERA',
-			...(
-				typeof data == 'string' ? {data: JSON.parse(data), ...options}
-				: typeof data == 'object' && !data.body && !data.title ? {data, ...options}
-				: options
-			),
+			...options,
 		};
 
 		return this.requestFocus(()=>
@@ -1986,10 +1997,10 @@ export default class TeraFyServer {
                 componentProps: {
                     body: settings.body,
                     isHtml: settings.isHtml,
-                    data: settings.data,
+                    data: data,
                 },
 			})
-				.then(()=> 'OK' as 'OK') // Resolve with 'OK' if OK button clicked
+				.then(()=> 'OK' as const) // Resolve with 'OK' if OK button clicked
 				.catch(()=> Promise.reject('CANCEL')) // Reject with 'CANCEL' if Cancel button clicked or closed
 		);
 	}
@@ -2028,7 +2039,7 @@ export default class TeraFyServer {
 	* @returns {Promise} A promise which resolves when the dialog has been updated
 	*/
 	uiProgress(options?: any): Promise<void> {
-		let currentOptions = options === false ? {close: true} : options || {};
+		const currentOptions = options === false ? {close: true} : options || {};
 
 		if (currentOptions.close) { // Asked to close the dialog
 			const closePromise = this._uiProgress.promise
@@ -2093,7 +2104,7 @@ export default class TeraFyServer {
 	* @returns {Promise<*>} Either the eventual user value or a throw with `Promise.reject('CANCEL')`
 	*/
 	uiPrompt(text: string | any, options?: any): Promise<any> {
-		let settings = {
+		const settings = {
 			body: '',
 			isHtml: false,
 			title: 'Input required',
@@ -2174,11 +2185,11 @@ export default class TeraFyServer {
 	*
 	* @returns {WindowProxy} The opened window object (if `noopener` is not set in permissions)
 	*/
-	uiWindow(url: string | URL, options?: any): WindowProxy | null {
+	uiWindow(url: string | URL, options?: UiWindowOptions): WindowProxy | null {
 		// Ensure this runs only in browser context
 		if (typeof window === 'undefined' || typeof screen === 'undefined') return null;
 
-		let settings = {
+		const settings = {
 			width: 500,
 			height: 600,
 			center: true,
@@ -2223,7 +2234,7 @@ export default class TeraFyServer {
 		// Ensure this runs only in browser context
 		if (typeof window === 'undefined' || typeof document === 'undefined') return;
 
-		let settings = {
+		const settings = {
 			logo: false,
 			...options,
 		};
@@ -2240,7 +2251,7 @@ export default class TeraFyServer {
 
 		let compiledContent: Element;
 		if (typeof content == 'string') {
-			let el = document.createElement('div')
+			const el = document.createElement('div')
 			el.innerHTML = content;
 			// If the string contained multiple top-level elements, wrap them
 			compiledContent = el.children.length === 1 ? el.firstElementChild! : el;
@@ -2252,7 +2263,7 @@ export default class TeraFyServer {
 		compiledContent.classList.add('tera-fy-uiSplat');
 
 		if (settings.logo) {
-			let logoEl = document.createElement('div');
+			const logoEl = document.createElement('div');
 			logoEl.innerHTML = `<img src="${typeof settings.logo == 'string' ? settings.logo : '/assets/logo/logo.svg'}" class="img-logo"/>`;
 			// Prepend logo within the content element
 			compiledContent.prepend(logoEl);
@@ -2264,14 +2275,14 @@ export default class TeraFyServer {
 
 	// Utility - debug() {{{
 
-	/* eslint-disable jsdoc/check-param-names */
 	/**
-	* Debugging output function
-	* This function will only act if `settings.devMode` is truthy
+	* Debugging output function.
+	* This function will only act if `settings.devMode` is truthy.
 	*
-	* @param {'INFO'|'LOG'|'WARN'|'ERROR'} [method='LOG'] Logging method to use
-	* @param {Number} [verboseLevel=1] The verbosity level to trigger at. If `settings.verbosity` is lower than this, the message is ignored
-	* @param {...*} [msg] Output to show
+	* @param {...any} inputArgs The arguments to process for debugging. The function signature is flexible: `debug([method], [verboseLevel], ...msg)`.
+	* - The first argument can optionally be a logging method string: 'INFO', 'LOG', 'WARN', or 'ERROR'. Defaults to 'LOG'.
+	* - The next argument can optionally be a numeric verbosity level. The message is ignored if `settings.verbosity` is lower than this level. Defaults to 1.
+	* - All subsequent arguments are passed to the console as the log message.
 	*/
 	debug(...inputArgs: any[]): void {
 		// Ensure console exists
@@ -2280,7 +2291,7 @@ export default class TeraFyServer {
 
 		let method: keyof Console = 'log'; // Default method
 		let verboseLevel = 1;
-		let msgArgs = [...inputArgs]; // Copy args to modify
+		const msgArgs = [...inputArgs]; // Copy args to modify
 
 		// Argument mangling for prefix method + verbosity level {{{
 		if (typeof msgArgs[0] == 'string' && ['INFO', 'LOG', 'WARN', 'ERROR'].includes(msgArgs[0].toUpperCase())) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iebh/tera-fy",
-  "version": "2.3.0",
+  "version": "2.3.2",
   "description": "TERA website worker",
   "scripts": {
     "dev": "esbuild --platform=browser --format=esm --bundle lib/terafy.client.ts --outfile=dist/terafy.js --minify --serve --servedir=.",
@@ -113,12 +113,12 @@
     "mitt": "^3.0.1",
     "nanoid": "^5.1.2",
     "p-retry": "^6.2.1",
-    "release-it": "^18.1.2",
+    "release-it": "^19.2.4",
     "uuid": "^11.1.0"
   },
   "devDependencies": {
     "@momsfriendlydevco/eslint-config": "^2.3.1",
-    "@release-it/conventional-changelog": "^10.0.0",
+    "@release-it/conventional-changelog": "^10.0.5",
     "@types/detect-port": "^2.0.0",
     "@types/http-proxy": "^1.17.16",
     "@types/lodash-es": "^4.17.12",
@@ -127,10 +127,12 @@
     "@typescript-eslint/eslint-plugin": "^8.41.0",
     "@typescript-eslint/parser": "^8.41.0",
     "concurrently": "^9.1.2",
+    "conventional-changelog-eslint": "^6.0.0",
     "documentation": "^14.0.3",
     "esbuild": "^0.25.0",
     "eslint": "^9.34.0",
     "eslint-plugin": "^1.0.1",
+    "globals": "^16.5.0",
     "nodemon": "^3.1.9",
     "typescript": "^5.9.2",
     "typescript-eslint": "^8.41.0"

package/plugins/base.ts

@@ -11,7 +11,7 @@ export default class TeraFyPlugin {
 	/**
 	* Optional function to be included when the main TeraFyClient is initalized
 	*/
-	init() {}
+	init(): any {}
 
 
 	/**
@@ -20,6 +20,7 @@ export default class TeraFyPlugin {
 	* @param {TeraFy} terafy The TeraFy client this plugin is being initalized against
 	* @param {Object} [options] Additional options to mutate behaviour
 	*/
-	constructor(terafy: TeraFy, options: Object) { // eslint-disable-line no-unused-vars
+	// eslint-disable-next-line @typescript-eslint/no-unused-vars
+	constructor(terafy: TeraFy, options: object) { // eslint-disable-line no-unused-vars
 	}
 }

package/plugins/firebase.ts

@@ -1,5 +1,5 @@
-import {initializeApp as Firebase, FirebaseApp} from 'firebase/app';
-import {getFirestore as Firestore, Firestore as FirestoreInstance} from 'firebase/firestore';
+import {initializeApp as Firebase} from 'firebase/app';
+import {getFirestore as Firestore} from 'firebase/firestore';
 import Supabasey from '@iebh/supabasey';
 import Syncro from '../lib/syncro/syncro.js';
 import TeraFyPluginBase from './base.js';
@@ -23,24 +23,25 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 	// Declare properties expected by the class methods or potentially inherited
 	getCredentials!: () => Promise<Record<string, any>>;
 	requireProject!: () => Promise<{ id: string }>;
+	// eslint-disable-next-line no-unused-vars
 	debug!: (...args: any[]) => void;
 
 
 	/**
 	* @interface
 	* The Syncro#reactive option to use when creating new Syncro instances
-	* This is expected to be overriden by other plugins
+	* This is expected to be overridden by other plugins
 	* If falsy the Syncro module will fall back to its internal (POJO only) getReactive() function
 	*
 	* @name getReactive
 	* @type {Function} A reactive function as defined in Syncro
 	*/
-	getReactive?: Function;
+	getReactive?: () => any;
 
 
 	/**
 	* Setup Firebase + Firestore + Supabase
-	* Default credentials (Firebase + Supabase) will be retrieved from `getCredentials()` unless overriden here
+	* Default credentials (Firebase + Supabase) will be retrieved from `getCredentials()` unless overridden here
 	*
 	* @param {Object} options Additional options to mutate behaviour (defaults to the main teraFy settings)
 	* @param {String} [options.firebaseApiKey] Firebase API key
@@ -53,7 +54,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 	* @returns {Promise} A Promise which will resolve when the init process has completed
 	*/
 	async init(options?: any): Promise<void> { // Add optional '?' and type 'any', keep async Promise<void>
-		let settings = {
+		const settings = {
 			firebaseApiKey: null,
 			firebaseAuthDomain: null,
 			firebaseProjectId: null,
@@ -64,7 +65,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 			...options,
 		};
 
-		let emptyValues = Object.keys(settings).filter(k => k === null);
+		const emptyValues = Object.keys(settings).filter(k => k === null);
 		if (emptyValues.length > 0)
 			throw new Error('Firebase plugin requires mandatory options: ' + emptyValues.join(', '));
 
@@ -88,7 +89,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 	/**
 	* Mount the given namespace against `namespaces[name]`
 	*
-	* @param {'_PROJECT'|String} name The name/Syncro path of the namespace to mount (or '_PROJECT' for the project mountpoint)
+	* @param {'_PROJECT'|String} name The name/Syncro path of the namespace to mount (or '_PROJECT' for the project mount-point)
 	*
 	* @returns {Promise} A promise which resolves when the operation has completed
 	*/
@@ -98,7 +99,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 		return Promise.resolve()
 			.then(()=> this.requireProject())
 			.then(project => {
-				let path = name == '_PROJECT'
+				const path = name == '_PROJECT'
 					? `projects::${project.id}`
 					: `project_namespaces::${project.id}::${name}`;
 
@@ -124,7 +125,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 	* @returns {Promise} A promise which resolves when the operation has completed
 	*/
 	_unmountNamespace(name: string): Promise<void | any[]> { // Add type 'string'
-		let syncro = this.syncros[name]; // Create local alias for Syncro before we detach it
+		const syncro = this.syncros[name]; // Create local alias for Syncro before we detach it
 
 		// Detach local state
 		delete this.namespaces[name];
@@ -132,7 +133,7 @@ export default class TeraFyPluginFirebase extends TeraFyPluginBase {
 
 		// Check if syncro exists before calling destroy
 		if (syncro) {
-			return syncro.destroy(); // Trigger Syncro distruction
+			return syncro.destroy(); // Trigger Syncro destruction
 		} else {
 			return Promise.resolve(); // Or handle the case where syncro doesn't exist
 		}

package/plugins/vue2.ts

@@ -27,7 +27,7 @@ import TeraFyPluginFirebase from './firebase.js';
 export default class TeraFyPluginVue2 extends TeraFyPluginFirebase {
 
 	/**
-	* Local Vue@2 library to use, set during constuctor
+	* Local Vue@2 library to use, set during constructor
 	*
 	* @type {Vue}
 	*/
@@ -68,7 +68,7 @@ export default class TeraFyPluginVue2 extends TeraFyPluginFirebase {
 	* @returns {Promise} A Promise which will resolve when the init process has completed
 	*/
 	async init(options: any) {
-		let settings = {
+		const settings = {
 			app: null,
 			Vue: null,
 			globalName: '$tera',
@@ -86,17 +86,17 @@ export default class TeraFyPluginVue2 extends TeraFyPluginFirebase {
 			this.Vue.prototype[settings.globalName] = this;
 
 		await super.init(settings); // Initalize parent class Firebase functionality
-		// @ts-ignore
+		// @ts-expect-error TODO: Track down why eslint throws error
 		this.project = await this.mountNamespace('_PROJECT');
 	}
 
 
 	/** @override */
-	// @ts-ignore
+	// @ts-expect-error TODO: Work out why TS doesn't like overrides
 	getReactive(value: any) {
-		let doc = this.Vue.observable(value);
+		const doc = this.Vue.observable(value);
 
-		let watcherPath = `_teraFy_${this.reactiveId++}`;
+		const watcherPath = `_teraFy_${this.reactiveId++}`;
 		this.app[watcherPath] = doc; // Attach onto app so we can use $watch later on
 
 		return {
@@ -110,6 +110,7 @@ export default class TeraFyPluginVue2 extends TeraFyPluginFirebase {
 			getState: () => {
 				return cloneDeep(doc);
 			},
+			// eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
 			watch: (cb: Function) => {
 				this.app.$watch(watcherPath, cb, {deep: true});
 			},