IndexDB fixes

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ztimson/utils",
-	"version": "0.26.11",
+	"version": "0.26.22",
 	"description": "Utility library",
 	"author": "Zak Timson",
 	"license": "MIT",

src/cache.ts

@@ -22,8 +22,10 @@ export class Cache<K extends string | number | symbol, T> {
 	[key: string | number | symbol]: CachedValue<T> | any;
 	/** Whether cache is complete */
 	complete = false;
+
+	private _loading!: Function;
 	/** Await initial loading */
-	loading!: Promise<void>;
+	loading = new Promise<void>(r => this._loading = r);
 
 	/**
 	 * Create new cache
@@ -31,9 +33,6 @@ export class Cache<K extends string | number | symbol, T> {
 	 * @param options
 	 */
 	constructor(public readonly key?: keyof T, public readonly options: CacheOptions = {}) {
-		let done!: Function;
-		this.loading = new Promise(r => done = r);
-
 		// Persistent storage
 		if(this.options.persistentStorage != null) {
 			if(typeof this.options.persistentStorage == 'string')
@@ -45,13 +44,15 @@ export class Cache<K extends string | number | symbol, T> {
 					const table: Table<any, any> = await persists.storage.createTable({name: persists.key, key: this.key});
 					const rows = await table.getAll();
 					Object.assign(this.store, rows.reduce((acc, row) => ({...acc, [this.getKey(row)]: row}), {}));
-					done();
+					this._loading();
 				})();
 			} else if((<any>this.options.persistentStorage?.storage)?.getItem != undefined) {
 				const stored = (<Storage>this.options.persistentStorage.storage).getItem(this.options.persistentStorage.key);
 				if(stored != null) try { Object.assign(this.store, JSON.parse(stored)); } catch { }
-				done();
+				this._loading();
 			}
+		} else {
+			this._loading();
 		}
 
 		// Handle index lookups

src/database.ts

@@ -14,8 +14,8 @@ class AsyncLock {
 		const res = this.p.then(fn, fn);
 		this.p = res.then(() => {}, () => {});
 		return res;
-}
 	}
+}
 
 export class Database {
 	private schemaLock = new AsyncLock();
@@ -28,7 +28,10 @@ export class Database {
 
 	constructor(public readonly database: string, tables?: (string | TableOptions)[], public version?: number) {
 		this.connection = new Promise((resolve, reject) => {
-			const req = indexedDB.open(this.database, this.version);
+			let req: IDBOpenDBRequest;
+			try { req = indexedDB.open(this.database, this.version); }
+			catch (err) { return reject(err); }
+
 			this.tables = !tables ? [] : tables.map(t => {
 				t = typeof t == 'object' ? t : {name: t};
 				return {...t, name: t.name.toString()};
@@ -37,13 +40,22 @@ export class Database {
 			req.onerror = () => reject(req.error);
 
 			req.onsuccess = () => {
-				const db = req.result;
+				let db: IDBDatabase;
+				try { db = req.result; }
+				catch (err) { return reject(err); }
+
 				const existing = Array.from(db.objectStoreNames);
-				if(!tables) this.tables = existing.map(t => {
-					const tx = db.transaction(t, 'readonly');
-					const store = tx.objectStore(t);
-					return {name: t, key: <string>store.keyPath};
-				});
+				if(!tables) {
+					this.tables = existing.map(t => {
+						try {
+							const tx = db.transaction(t, 'readonly');
+							const store = tx.objectStore(t);
+							return {name: t, key: <string>store.keyPath};
+						} catch {
+							return {name: t}; // 🛡️ fallback
+						}
+					});
+				}
 				const desired = new ASet((tables || []).map(t => typeof t == 'string' ? t : t.name));
 				if(tables && desired.symmetricDifference(new ASet(existing)).length) {
 					db.close();
@@ -58,19 +70,23 @@ export class Database {
 
 			req.onupgradeneeded = () => {
 				this.upgrading = true;
-				const db = req.result;
-				const existingTables = new ASet(Array.from(db.objectStoreNames));
-				if(tables) {
-					const desired = new ASet((tables || []).map(t => typeof t == 'string' ? t : t.name));
-					existingTables.difference(desired).forEach(name => db.deleteObjectStore(name));
-					desired.difference(existingTables).forEach(name => {
-						const t = this.tables.find(findByProp('name', name));
-						db.createObjectStore(name, {
-							keyPath: t?.key,
-							autoIncrement: t?.autoIncrement || !t?.key,
+				let db: IDBDatabase;
+				try { db = req.result; }
+				catch { return; }
+				try {
+					const existingTables = new ASet(Array.from(db.objectStoreNames));
+					if(tables) {
+						const desired = new ASet((tables || []).map(t => typeof t == 'string' ? t : t.name));
+						existingTables.difference(desired).forEach(name => db.deleteObjectStore(name));
+						desired.difference(existingTables).forEach(name => {
+							const t = this.tables.find(findByProp('name', name));
+							db.createObjectStore(name, {
+								keyPath: t?.key,
+								autoIncrement: t?.autoIncrement || !t?.key,
+							});
 						});
-					});
-				}
+					}
+				} catch { }
 			};
 		});
 	}
@@ -79,30 +95,30 @@ export class Database {
 
 	async createTable<K extends IDBValidKey = any, T = any>(table: string | TableOptions): Promise<Table<K, T>> {
 		return this.schemaLock.run(async () => {
-			if (typeof table == 'string') table = { name: table };
+			if(typeof table == 'string') table = { name: table };
 			const conn = await this.connection;
-			if (!this.includes(table.name)) {
+			if(!this.includes(table.name)) {
 				const newDb = new Database(this.database, [...this.tables, table], (this.version ?? 0) + 1);
 				conn.close();
-				Object.assign(this, newDb);
+				this.connection = newDb.connection;
 				await this.connection;
+				Object.assign(this, newDb);
 			}
 			return this.table<K, T>(table.name);
 		});
 	}
 
-
 	async deleteTable(table: string | TableOptions): Promise<void> {
 		return this.schemaLock.run(async () => {
-			if (typeof table == 'string') table = { name: table };
-			if (!this.includes(table.name)) return;
+			if(typeof table == 'string') table = { name: table };
+			if(!this.includes(table.name)) return;
 			const conn = await this.connection;
 			const newDb = new Database(this.database, this.tables.filter(t => t.name != (<TableOptions>table).name), (this.version ?? 0) + 1);
 			conn.close();
-			Object.assign(this, newDb);
+			this.connection = newDb.connection;
 			await this.connection;
+			Object.assign(this, newDb);
 		});
-
 	}
 
 	includes(name: any): boolean {
@@ -126,9 +142,8 @@ export class Table<K extends IDBValidKey = any, T = any> {
 		await this.database.waitForUpgrade();
 		const db = await this.database.connection;
 		const tx = db.transaction(table, readonly ? 'readonly' : 'readwrite');
-		const store = tx.objectStore(table);
 		return new Promise<R>((resolve, reject) => {
-			const request = fn(store);
+			const request = fn(tx.objectStore(table));
 			request.onsuccess = () => resolve(request.result as R);
 			request.onerror = () => reject(request.error);
 		});
@@ -168,7 +183,7 @@ export class Table<K extends IDBValidKey = any, T = any> {
 
 	put(value: T, key?: string): Promise<void> {
 		return this.tx(this.name, store => {
-			if (store.keyPath) return store.put(value);
+			if(store.keyPath) return store.put(value);
 			return store.put(value, key);
 		});
 	}

src/http.ts

@@ -78,7 +78,7 @@ export class Http {
 		if(!this.url && !opts.url) throw new Error('URL needs to be set');
 		let url = opts.url?.startsWith('http') ? opts.url : (this.url || '') + (opts.url || '');
 		url = url.replaceAll(/([^:]\/)\/+/g, '$1');
-		if(opts.fragment) url.includes('#') ? url.replace(/#.*(\?|\n)/g, (match, arg1) => `#${opts.fragment}${arg1}`) : `${url}#${opts.fragment}`;
+		if(opts.fragment) url.includes('#') ? url.replace(/#.*([?\n])/g, (match, arg1) => `#${opts.fragment}${arg1}`) : `${url}#${opts.fragment}`;
 		if(opts.query) {
 			const q = Array.isArray(opts.query) ? opts.query :
 				Object.keys(opts.query).map(k => ({key: k, value: (<any>opts.query)[k]}))

src/search.ts

@@ -13,8 +13,8 @@ export function search(rows: any[], search: string, regex?: boolean, transform:
 	if(!rows) return [];
 	return rows.filter(r => {
 		// Empty search
-		const value = transform(r);
 		if(!search) return true;
+		const value = transform(r);
 		// Regex search
 		if(regex) {
 			return !!Object.values(value).filter((v: any) => {
@@ -22,7 +22,7 @@ export function search(rows: any[], search: string, regex?: boolean, transform:
 				catch { return false; }
 			}).length
 		} else {
-			return logicTest(r, search);
+			return logicTest(value, search);
 		}
 	});
 }
@@ -39,8 +39,8 @@ export function logicTest(target: object, condition: string): boolean {
 			case '=':
 			case '==': return a == b;
 			case '!=': return a != b;
-			case '+': return typeof a?.includes != 'undefined' ? a.includes(b) : a == b;
-			case '-': return typeof a?.includes != 'undefined' ? !a.includes(b) : a != b;
+			case '+=': return a?.toString().includes(b);
+			case '-=': return !a?.toString().includes(b);
 			case '>': return a > b;
 			case '>=': return a >= b;
 			case '<': return a < b;
@@ -55,7 +55,7 @@ export function logicTest(target: object, condition: string): boolean {
 		const and = p.split('&&').map(p => p.trim()).filter(p => !!p);
 		return and.filter(p => {
 			// Boolean operator
-			const prop = /(\S+)\s*(==?|!=|\+|-|>=|>|<=|<)\s*(\S+)/g.exec(p);
+			const prop = /(\S+)\s*(==?|!=|\+=|-=|>=|>|<=|<)\s*(\S+)/g.exec(p);
 			if(prop) {
 				const key = Object.keys(target).find(k => k.toLowerCase() == prop[1].toLowerCase());
 				return evalBoolean(dotNotation<any>(target, key || prop[1]),  prop[2], JSONAttemptParse(prop[3]));

src/string.ts

@@ -43,6 +43,24 @@ export function formatBytes(bytes: number, decimals = 2) {
 	return parseFloat((bytes / Math.pow(k, i)).toFixed(decimals)) + ' ' + sizes[i];
 }
 
+/**
+ * Convert milliseconds to human-readable duration
+ * @param {string} ms milliseconds
+ * @param {boolean} short Use unit initial instead of word
+ * @return {string} formated duration
+ */
+export function formatMs(ms: number, short = false): string {
+	if (isNaN(ms) || ms < 0) return "Invalid input";
+	const seconds = ms / 1000;
+	const minutes = seconds / 60;
+	const hours = minutes / 60;
+	const days = hours / 24;
+	if (days >= 1) return `${days.toFixed(1)} ${short ? 'd' : 'days'}`;
+	else if (hours >= 1) return `${hours.toFixed(1)} ${short ? 'h' : 'hours'}`;
+	else if (minutes >= 1) return `${minutes.toFixed(1)} ${short ? 'm' : 'minutes'}`;
+	else return `${seconds.toFixed(1)} ${short ? 's' : 'seconds'}`;
+}
+
 /**
  * Extract numbers from a string & create a formated phone number: +1 (123) 456-7890
  *

tests/search.spec.ts

@@ -4,6 +4,7 @@ const rows = [
 	{id: 1, name: 'Alice', age: 30},
 	{id: 2, name: 'Bob', age: 24},
 	{id: 3, name: 'Carol', age: 30},
+	{id: 4, name: 'David', age: 35},
 ];
 
 describe('Search Utilities', () => {
@@ -37,6 +38,35 @@ describe('Search Utilities', () => {
 		it('returns all if search is falsy and regex enabled', () => {
 			expect(search(rows, '', true)).toEqual(rows);
 		});
+
+		it('handles regex search with special characters', () => {
+			expect(search(rows, '^[AC]', true)).toEqual([rows[0], rows[2]]);
+		});
+
+		it('ignores case when regex is applied', () => {
+			expect(search(rows, 'ALICE', true)).toEqual([]);
+		});
+
+		it('performs partial matches for properties when regex=false', () => {
+			expect(search(rows, 'Da')).toEqual([rows[3]]);
+		});
+
+		it('handles empty array input gracefully', () => {
+			expect(search([], 'test')).toEqual([]);
+		});
+
+		it('handles numeric values with comparison logic in strings', () => {
+			expect(search(rows, 'age < 31')).toEqual([rows[0], rows[1], rows[2]]);
+		});
+
+		// New test cases for `+` and `-` operators
+		it('filters rows using the + operator', () => {
+			expect(search(rows, 'name += Al')).toEqual([rows[0]]);
+		});
+
+		it('filters rows using the - operator', () => {
+			expect(search(rows, 'name -= Al')).toEqual([rows[1], rows[2], rows[3]]);
+		});
 	});
 
 	describe('logicTest', () => {
@@ -67,5 +97,46 @@ describe('Search Utilities', () => {
 		it('returns false for unsupported operators', () => {
 			expect(logicTest(obj, 'x === 10')).toBe(false);
 		});
+
+		it('handles invalid condition strings gracefully', () => {
+			expect(logicTest(obj, 'invalid condition')).toBe(false);
+		});
+
+		it('supports numeric operations with ranges', () => {
+			expect(logicTest(obj, 'x > 5 && x < 15')).toBe(true);
+			expect(logicTest(obj, 'x > 15')).toBe(false);
+		});
+
+		it('handles mixed case keys gracefully', () => {
+			const mixedCaseObj = {TestKey: 123};
+			expect(logicTest(mixedCaseObj, 'TestKey == 123')).toBe(true);
+			expect(logicTest(mixedCaseObj, 'testkey == 123')).toBe(true);
+		});
+
+		it('returns false if condition operators are missing', () => {
+			expect(logicTest(obj, 'x 10')).toBe(false);
+		});
+
+		// New test cases for `+` and `-` operators
+		it('handles the + operator for inclusion', () => {
+			expect(logicTest(obj, 'name += Alpha')).toBe(true);
+			expect(logicTest(obj, 'name += Alp')).toBe(true);
+			expect(logicTest(obj, 'name += Bet')).toBe(false);
+		});
+
+		it('handles the - operator for exclusion', () => {
+			expect(logicTest(obj, 'name -= Alpha')).toBe(false);
+			expect(logicTest(obj, 'name -= Alp')).toBe(false);
+			expect(logicTest(obj, 'name -= Bet')).toBe(true);
+		});
+
+		it('includes partial matches correctly with +', () => {
+			expect(logicTest(obj, 'name += lph')).toBe(true);
+		});
+
+		it('excludes partial matches correctly with -', () => {
+			expect(logicTest(obj, 'name -= lph')).toBe(false);
+			expect(logicTest(obj, 'name -= xyz')).toBe(true);
+		});
 	});
 });