Enable ESLint prefer-const rule

This commit is contained in:
James Brunton 2025-09-02 17:52:11 +01:00
parent 65e331ae26
commit 76c07a171b
3 changed files with 7 additions and 13 deletions

View File

@ -20,7 +20,6 @@ export default defineConfig(
"no-undef": "off", // Temporarily disabled until codebase conformant "no-undef": "off", // Temporarily disabled until codebase conformant
"no-useless-escape": "off", // Temporarily disabled until codebase conformant "no-useless-escape": "off", // Temporarily disabled until codebase conformant
"no-case-declarations": "off", // Temporarily disabled until codebase conformant "no-case-declarations": "off", // Temporarily disabled until codebase conformant
"prefer-const": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/ban-ts-comment": "off", // Temporarily disabled until codebase conformant "@typescript-eslint/ban-ts-comment": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-empty-object-type": "off", // Temporarily disabled until codebase conformant "@typescript-eslint/no-empty-object-type": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-explicit-any": "off", // Temporarily disabled until codebase conformant "@typescript-eslint/no-explicit-any": "off", // Temporarily disabled until codebase conformant

View File

@ -182,7 +182,7 @@ export class EnhancedPDFProcessingService {
): Promise<ProcessedFile> { ): Promise<ProcessedFile> {
const arrayBuffer = await file.arrayBuffer(); const arrayBuffer = await file.arrayBuffer();
const pdf = await pdfWorkerManager.createDocument(arrayBuffer); const pdf = await pdfWorkerManager.createDocument(arrayBuffer);
try { try {
const totalPages = pdf.numPages; const totalPages = pdf.numPages;
@ -519,10 +519,7 @@ export class EnhancedPDFProcessingService {
this.notifyListeners(); this.notifyListeners();
// Force memory cleanup hint // Force memory cleanup hint
if (typeof window !== 'undefined' && window.gc) { setTimeout(() => window.gc?.(), 100);
let gc = window.gc;
setTimeout(() => gc(), 100);
}
} }
/** /**

View File

@ -73,7 +73,7 @@ class IndexedDBManager {
request.onsuccess = () => { request.onsuccess = () => {
const db = request.result; const db = request.result;
console.log(`Successfully opened ${config.name}`); console.log(`Successfully opened ${config.name}`);
// Set up close handler to clean up our references // Set up close handler to clean up our references
db.onclose = () => { db.onclose = () => {
console.log(`Database ${config.name} closed`); console.log(`Database ${config.name} closed`);
@ -87,13 +87,11 @@ class IndexedDBManager {
request.onupgradeneeded = (event) => { request.onupgradeneeded = (event) => {
const db = request.result; const db = request.result;
const oldVersion = event.oldVersion; const oldVersion = event.oldVersion;
console.log(`Upgrading ${config.name} from v${oldVersion} to v${config.version}`); console.log(`Upgrading ${config.name} from v${oldVersion} to v${config.version}`);
// Create or update object stores // Create or update object stores
config.stores.forEach(storeConfig => { config.stores.forEach(storeConfig => {
let store: IDBObjectStore;
if (db.objectStoreNames.contains(storeConfig.name)) { if (db.objectStoreNames.contains(storeConfig.name)) {
// Store exists - for now, just continue (could add migration logic here) // Store exists - for now, just continue (could add migration logic here)
console.log(`Object store '${storeConfig.name}' already exists`); console.log(`Object store '${storeConfig.name}' already exists`);
@ -109,7 +107,7 @@ class IndexedDBManager {
options.autoIncrement = storeConfig.autoIncrement; options.autoIncrement = storeConfig.autoIncrement;
} }
store = db.createObjectStore(storeConfig.name, options); const store = db.createObjectStore(storeConfig.name, options);
console.log(`Created object store '${storeConfig.name}'`); console.log(`Created object store '${storeConfig.name}'`);
// Create indexes // Create indexes
@ -168,7 +166,7 @@ class IndexedDBManager {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
const deleteRequest = indexedDB.deleteDatabase(name); const deleteRequest = indexedDB.deleteDatabase(name);
deleteRequest.onerror = () => reject(deleteRequest.error); deleteRequest.onerror = () => reject(deleteRequest.error);
deleteRequest.onsuccess = () => { deleteRequest.onsuccess = () => {
console.log(`Deleted database: ${name}`); console.log(`Deleted database: ${name}`);
@ -224,4 +222,4 @@ export const DATABASE_CONFIGS = {
} as DatabaseConfig } as DatabaseConfig
} as const; } as const;
export const indexedDBManager = IndexedDBManager.getInstance(); export const indexedDBManager = IndexedDBManager.getInstance();