Enable ESLint no-case-declarations rule (#4348)

# Description of Changes
Enable ESLint [no-case-declarations
rule](https://eslint.org/docs/latest/rules/no-case-declarations)
This commit is contained in:
James Brunton 2025-09-05 10:58:14 +01:00 committed by GitHub
parent b9cf7e7820
commit cd1fc682ab
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 20 additions and 16 deletions

View File

@ -16,7 +16,6 @@ export default defineConfig(
{
rules: {
"no-undef": "off", // Temporarily disabled until codebase conformant
"no-case-declarations": "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-explicit-any": "off", // Temporarily disabled until codebase conformant

View File

@ -174,7 +174,7 @@ export const useToolOperation = <TParams>(
let processedFiles: File[];
switch (config.toolType) {
case ToolType.singleFile:
case ToolType.singleFile: {
// Individual file processing - separate API call per file
const apiCallsConfig: ApiCallsConfig<TParams> = {
endpoint: config.endpoint,
@ -190,8 +190,9 @@ export const useToolOperation = <TParams>(
actions.setStatus
);
break;
}
case ToolType.multiFile:
case ToolType.multiFile: {
// Multi-file processing - single API call with all files
actions.setStatus('Processing files...');
const formData = config.buildFormData(params, validFiles);
@ -204,7 +205,7 @@ export const useToolOperation = <TParams>(
// Use custom responseHandler for multi-file (handles ZIP extraction)
processedFiles = await config.responseHandler(response.data, validFiles);
} else if (response.data.type === 'application/pdf' ||
(response.headers && response.headers['content-type'] === 'application/pdf')) {
(response.headers && response.headers['content-type'] === 'application/pdf')) {
// Single PDF response (e.g. split with merge option) - use original filename
const originalFileName = validFiles[0]?.name || 'document.pdf';
const singleFile = new File([response.data], originalFileName, { type: 'application/pdf' });
@ -219,6 +220,7 @@ export const useToolOperation = <TParams>(
}
}
break;
}
case ToolType.custom:
actions.setStatus('Processing files...');

View File

@ -459,11 +459,12 @@ export class EnhancedPDFProcessingService {
case 'failed':
this.metrics.failedFiles++;
break;
case 'cacheHit':
case 'cacheHit': {
// Update cache hit rate
const totalAttempts = this.metrics.totalFiles + 1;
this.metrics.cacheHitRate = (this.metrics.cacheHitRate * this.metrics.totalFiles + 1) / totalAttempts;
break;
}
}
}

View File

@ -148,15 +148,17 @@ export class FileAnalyzer {
case 'immediate_full':
return pageCount * baseTime;
case 'priority_pages':
case 'priority_pages': {
// Estimate time for priority pages (first 10)
const priorityPages = Math.min(pageCount, 10);
return priorityPages * baseTime;
}
case 'progressive_chunked':
case 'progressive_chunked': {
// Estimate time for first chunk (20 pages)
const firstChunk = Math.min(pageCount, 20);
return firstChunk * baseTime;
}
default:
return pageCount * baseTime;